@pnp/cli-microsoft365 5.8.0-beta.f956c5d → 5.9.0-beta.21a8d94

package/dist/m365/spo/commands/page/page-text-add.js

@@ -17,7 +17,9 @@ var _SpoPageTextAddCommand_instances, _SpoPageTextAddCommand_initTelemetry, _Spo
 Object.defineProperty(exports, "__esModule", { value: true });
 const util_1 = require("util");
 const request_1 = require("../../../../request");
-const utils_1 = require("../../../../utils");
+const spo_1 = require("../../../../utils/spo");
+const urlUtil_1 = require("../../../../utils/urlUtil");
+const validation_1 = require("../../../../utils/validation");
 const SpoCommand_1 = require("../../../base/SpoCommand");
 const commands_1 = require("../../commands");
 const clientsidepages_1 = require("./clientsidepages");
@@ -36,50 +38,50 @@ class SpoPageTextAddCommand extends SpoCommand_1.default {
     get description() {
         return 'Adds text to a modern page';
     }
-    commandAction(logger, args, cb) {
-        let requestDigest = '';
-        let pageName = args.options.pageName;
-        if (args.options.pageName.indexOf('.aspx') < 0) {
-            pageName += '.aspx';
-        }
-        if (this.verbose) {
-            logger.logToStderr(`Retrieving request digest...`);
-        }
-        utils_1.spo
-            .getRequestDigest(args.options.webUrl)
-            .then((res) => {
-            // Keep the reference of request digest for subsequent requests
-            requestDigest = res.FormDigestValue;
-            if (this.verbose) {
-                logger.logToStderr(`Retrieving modern page ${pageName}...`);
+    commandAction(logger, args) {
+        return __awaiter(this, void 0, void 0, function* () {
+            let requestDigest = '';
+            let pageName = args.options.pageName;
+            if (args.options.pageName.indexOf('.aspx') < 0) {
+                pageName += '.aspx';
             }
-            // Get Client Side Page
-            return Page_1.Page.getPage(pageName, args.options.webUrl, logger, this.debug, this.verbose);
-        })
-            .then((page) => {
-            const section = (args.options.section || 1) - 1;
-            const column = (args.options.column || 1) - 1;
-            // Make sure the section is in range
-            if (section >= page.sections.length) {
-                throw new Error(`Invalid section '${section + 1}'`);
-            }
-            // Make sure the column is in range
-            if (column >= page.sections[section].columns.length) {
-                throw new Error(`Invalid column '${column + 1}'`);
+            if (this.verbose) {
+                logger.logToStderr(`Retrieving request digest...`);
             }
-            const text = new clientsidepages_1.ClientSideText(args.options.text);
-            if (typeof args.options.order === 'undefined') {
-                page.sections[section].columns[column].addControl(text);
+            try {
+                const reqDigest = yield spo_1.spo.getRequestDigest(args.options.webUrl);
+                // Keep the reference of request digest for subsequent requests
+                requestDigest = reqDigest.FormDigestValue;
+                if (this.verbose) {
+                    logger.logToStderr(`Retrieving modern page ${pageName}...`);
+                }
+                // Get Client Side Page
+                const page = yield Page_1.Page.getPage(pageName, args.options.webUrl, logger, this.debug, this.verbose);
+                const section = (args.options.section || 1) - 1;
+                const column = (args.options.column || 1) - 1;
+                // Make sure the section is in range
+                if (section >= page.sections.length) {
+                    throw new Error(`Invalid section '${section + 1}'`);
+                }
+                // Make sure the column is in range
+                if (column >= page.sections[section].columns.length) {
+                    throw new Error(`Invalid column '${column + 1}'`);
+                }
+                const text = new clientsidepages_1.ClientSideText(args.options.text);
+                if (typeof args.options.order === 'undefined') {
+                    page.sections[section].columns[column].addControl(text);
+                }
+                else {
+                    const order = args.options.order - 1;
+                    page.sections[section].columns[column].insertControl(text, order);
+                }
+                // Save the Client Side Page with updated information
+                yield this.saveClientSidePage(page, logger, args, pageName, requestDigest);
             }
-            else {
-                const order = args.options.order - 1;
-                page.sections[section].columns[column].insertControl(text, order);
+            catch (err) {
+                this.handleRejectedODataJsonPromise(err);
             }
-            // Save the Client Side Page with updated information
-            return this.saveClientSidePage(page, logger, args, pageName, requestDigest);
-        })
-            .then(_ => cb())
-            .catch((err) => this.handleRejectedODataJsonPromise(err, logger, cb));
+        });
     }
     saveClientSidePage(clientSidePage, logger, args, pageName, requestDigest) {
         const updatedContent = clientSidePage.toHtml();
@@ -90,7 +92,7 @@ class SpoPageTextAddCommand extends SpoCommand_1.default {
         }
         const requestOptions = {
             url: `${args.options
-                .webUrl}/_api/web/getfilebyserverrelativeurl('${utils_1.urlUtil.getServerRelativeSiteUrl(args.options.webUrl)}/sitepages/${pageName}')/ListItemAllFields`,
+                .webUrl}/_api/web/getfilebyserverrelativeurl('${urlUtil_1.urlUtil.getServerRelativeSiteUrl(args.options.webUrl)}/sitepages/${pageName}')/ListItemAllFields`,
             headers: {
                 'X-RequestDigest': requestDigest,
                 'content-type': 'application/json;odata=nometadata',
@@ -136,7 +138,7 @@ _SpoPageTextAddCommand_instances = new WeakSet(), _SpoPageTextAddCommand_initTel
         if (args.options.column && (!(0, util_1.isNumber)(args.options.column) || args.options.column < 1)) {
             return 'The value of parameter column must be 1 or higher';
         }
-        return utils_1.validation.isValidSharePointUrl(args.options.webUrl);
+        return validation_1.validation.isValidSharePointUrl(args.options.webUrl);
     }));
 };
 module.exports = new SpoPageTextAddCommand();

package/dist/m365/spo/commands/propertybag/propertybag-base.js

@@ -3,7 +3,8 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.SpoPropertyBagBaseCommand = void 0;
 const config_1 = require("../../../../config");
 const request_1 = require("../../../../request");
-const utils_1 = require("../../../../utils");
+const formatting_1 = require("../../../../utils/formatting");
+const spo_1 = require("../../../../utils/spo");
 const SpoCommand_1 = require("../../../base/SpoCommand");
 const base_permissions_1 = require("../../base-permissions");
 class SpoPropertyBagBaseCommand extends SpoCommand_1.default {
@@ -120,7 +121,7 @@ class SpoPropertyBagBaseCommand extends SpoCommand_1.default {
             headers: {
                 'X-RequestDigest': formDigest
             },
-            data: `<Request AddExpandoFieldTypeSuffix="true" SchemaVersion="15.0.0.0" LibraryVersion="16.0.0.0" ApplicationName="${config_1.default.applicationName}" xmlns="http://schemas.microsoft.com/sharepoint/clientquery/2009"><Actions><Method Name="SetFieldValue" Id="206" ObjectPathId="205"><Parameters><Parameter Type="String">${utils_1.formatting.escapeXml(name)}</Parameter><Parameter Type="String">${utils_1.formatting.escapeXml(value)}</Parameter></Parameters></Method><Method Name="Update" Id="207" ObjectPathId="198" /></Actions><ObjectPaths><Property Id="205" ParentId="198" Name="${objectType}" /><Identity Id="198" Name="${identityResp.objectIdentity}" /></ObjectPaths></Request>`
+            data: `<Request AddExpandoFieldTypeSuffix="true" SchemaVersion="15.0.0.0" LibraryVersion="16.0.0.0" ApplicationName="${config_1.default.applicationName}" xmlns="http://schemas.microsoft.com/sharepoint/clientquery/2009"><Actions><Method Name="SetFieldValue" Id="206" ObjectPathId="205"><Parameters><Parameter Type="String">${formatting_1.formatting.escapeXml(name)}</Parameter><Parameter Type="String">${formatting_1.formatting.escapeXml(value)}</Parameter></Parameters></Method><Method Name="Update" Id="207" ObjectPathId="198" /></Actions><ObjectPaths><Property Id="205" ParentId="198" Name="${objectType}" /><Identity Id="198" Name="${identityResp.objectIdentity}" /></ObjectPaths></Request>`
         };
         return new Promise((resolve, reject) => {
             request_1.default.post(requestOptions).then((res) => {
@@ -145,7 +146,7 @@ class SpoPropertyBagBaseCommand extends SpoCommand_1.default {
      */
     static isNoScriptSite(webUrl, formDigest, webIdentityResp, logger, debug) {
         return new Promise((resolve, reject) => {
-            utils_1.spo.getEffectiveBasePermissions(webIdentityResp.objectIdentity, webUrl, formDigest, logger, debug)
+            spo_1.spo.getEffectiveBasePermissions(webIdentityResp.objectIdentity, webUrl, formDigest, logger, debug)
                 .then((basePermissionsResp) => {
                 resolve(basePermissionsResp.has(base_permissions_1.PermissionKind.AddAndCustomizePages) === false);
             })

package/dist/m365/spo/commands/propertybag/propertybag-get.js

@@ -15,7 +15,8 @@ var __classPrivateFieldGet = (this && this.__classPrivateFieldGet) || function (
 };
 var _SpoPropertyBagGetCommand_instances, _SpoPropertyBagGetCommand_initTelemetry, _SpoPropertyBagGetCommand_initOptions, _SpoPropertyBagGetCommand_initValidators;
 Object.defineProperty(exports, "__esModule", { value: true });
-const utils_1 = require("../../../../utils");
+const spo_1 = require("../../../../utils/spo");
+const validation_1 = require("../../../../utils/validation");
 const commands_1 = require("../../commands");
 const propertybag_base_1 = require("./propertybag-base");
 class SpoPropertyBagGetCommand extends propertybag_base_1.SpoPropertyBagBaseCommand {
@@ -32,30 +33,32 @@ class SpoPropertyBagGetCommand extends propertybag_base_1.SpoPropertyBagBaseComm
     get description() {
         return 'Gets the value of the specified property from the property bag';
     }
-    commandAction(logger, args, cb) {
-        utils_1.spo
-            .getRequestDigest(args.options.webUrl)
-            .then((contextResponse) => {
-            this.formDigestValue = contextResponse.FormDigestValue;
-            return utils_1.spo.getCurrentWebIdentity(args.options.webUrl, this.formDigestValue);
-        })
-            .then((identityResp) => {
-            const opts = args.options;
-            if (opts.folder) {
-                return this.getFolderPropertyBag(identityResp, opts.webUrl, opts.folder, logger);
+    commandAction(logger, args) {
+        return __awaiter(this, void 0, void 0, function* () {
+            try {
+                const contextResponse = yield spo_1.spo.getRequestDigest(args.options.webUrl);
+                this.formDigestValue = contextResponse.FormDigestValue;
+                const identityResp = yield spo_1.spo.getCurrentWebIdentity(args.options.webUrl, this.formDigestValue);
+                let propertyBagData;
+                const opts = args.options;
+                if (opts.folder) {
+                    propertyBagData = yield this.getFolderPropertyBag(identityResp, opts.webUrl, opts.folder, logger);
+                }
+                else {
+                    propertyBagData = yield this.getWebPropertyBag(identityResp, opts.webUrl, logger);
+                }
+                const property = this.filterByKey(propertyBagData, args.options.key);
+                if (property) {
+                    logger.log(property.value);
+                }
+                else if (this.verbose) {
+                    logger.logToStderr('Property not found.');
+                }
             }
-            return this.getWebPropertyBag(identityResp, opts.webUrl, logger);
-        })
-            .then((propertyBagData) => {
-            const property = this.filterByKey(propertyBagData, args.options.key);
-            if (property) {
-                logger.log(property.value);
+            catch (err) {
+                this.handleRejectedPromise(err);
             }
-            else if (this.verbose) {
-                logger.logToStderr('Property not found.');
-            }
-            cb();
-        }, (err) => this.handleRejectedPromise(err, logger, cb));
+        });
     }
     filterByKey(propertyBag, key) {
         const keys = Object.keys(propertyBag);
@@ -86,7 +89,7 @@ _SpoPropertyBagGetCommand_instances = new WeakSet(), _SpoPropertyBagGetCommand_i
         option: '-f, --folder [folder]'
     });
 }, _SpoPropertyBagGetCommand_initValidators = function _SpoPropertyBagGetCommand_initValidators() {
-    this.validators.push((args) => __awaiter(this, void 0, void 0, function* () { return utils_1.validation.isValidSharePointUrl(args.options.webUrl); }));
+    this.validators.push((args) => __awaiter(this, void 0, void 0, function* () { return validation_1.validation.isValidSharePointUrl(args.options.webUrl); }));
 };
 module.exports = new SpoPropertyBagGetCommand();
 //# sourceMappingURL=propertybag-get.js.map

package/dist/m365/spo/commands/propertybag/propertybag-list.js

@@ -15,7 +15,8 @@ var __classPrivateFieldGet = (this && this.__classPrivateFieldGet) || function (
 };
 var _SpoPropertyBagListCommand_instances, _SpoPropertyBagListCommand_initTelemetry, _SpoPropertyBagListCommand_initOptions, _SpoPropertyBagListCommand_initValidators;
 Object.defineProperty(exports, "__esModule", { value: true });
-const utils_1 = require("../../../../utils");
+const spo_1 = require("../../../../utils/spo");
+const validation_1 = require("../../../../utils/validation");
 const commands_1 = require("../../commands");
 const propertybag_base_1 = require("./propertybag-base");
 class SpoPropertyBagListCommand extends propertybag_base_1.SpoPropertyBagBaseCommand {
@@ -32,24 +33,26 @@ class SpoPropertyBagListCommand extends propertybag_base_1.SpoPropertyBagBaseCom
     get description() {
         return 'Gets property bag values';
     }
-    commandAction(logger, args, cb) {
-        utils_1.spo
-            .getRequestDigest(args.options.webUrl)
-            .then((contextResponse) => {
-            this.formDigestValue = contextResponse.FormDigestValue;
-            return utils_1.spo.getCurrentWebIdentity(args.options.webUrl, this.formDigestValue);
-        })
-            .then((identityResp) => {
-            const opts = args.options;
-            if (opts.folder) {
-                return this.getFolderPropertyBag(identityResp, opts.webUrl, opts.folder, logger);
+    commandAction(logger, args) {
+        return __awaiter(this, void 0, void 0, function* () {
+            try {
+                const contextResponse = yield spo_1.spo.getRequestDigest(args.options.webUrl);
+                this.formDigestValue = contextResponse.FormDigestValue;
+                const identityResp = yield spo_1.spo.getCurrentWebIdentity(args.options.webUrl, this.formDigestValue);
+                let propertyBagData;
+                const opts = args.options;
+                if (opts.folder) {
+                    propertyBagData = yield this.getFolderPropertyBag(identityResp, opts.webUrl, opts.folder, logger);
+                }
+                else {
+                    propertyBagData = yield this.getWebPropertyBag(identityResp, opts.webUrl, logger);
+                }
+                logger.log(this.formatOutput(propertyBagData));
             }
-            return this.getWebPropertyBag(identityResp, opts.webUrl, logger);
-        })
-            .then((propertyBagData) => {
-            logger.log(this.formatOutput(propertyBagData));
-            cb();
-        }, (err) => this.handleRejectedPromise(err, logger, cb));
+            catch (err) {
+                this.handleRejectedPromise(err);
+            }
+        });
     }
     /**
      * The property bag data returned from the client.svc/ProcessQuery response
@@ -84,7 +87,7 @@ _SpoPropertyBagListCommand_instances = new WeakSet(), _SpoPropertyBagListCommand
         option: '-f, --folder [folder]'
     });
 }, _SpoPropertyBagListCommand_initValidators = function _SpoPropertyBagListCommand_initValidators() {
-    this.validators.push((args) => __awaiter(this, void 0, void 0, function* () { return utils_1.validation.isValidSharePointUrl(args.options.webUrl); }));
+    this.validators.push((args) => __awaiter(this, void 0, void 0, function* () { return validation_1.validation.isValidSharePointUrl(args.options.webUrl); }));
 };
 module.exports = new SpoPropertyBagListCommand();
 //# sourceMappingURL=propertybag-list.js.map

package/dist/m365/spo/commands/propertybag/propertybag-remove.js

@@ -15,10 +15,12 @@ var __classPrivateFieldGet = (this && this.__classPrivateFieldGet) || function (
 };
 var _SpoPropertyBagRemoveCommand_instances, _SpoPropertyBagRemoveCommand_initTelemetry, _SpoPropertyBagRemoveCommand_initOptions, _SpoPropertyBagRemoveCommand_initValidators;
 Object.defineProperty(exports, "__esModule", { value: true });
-const cli_1 = require("../../../../cli");
+const Cli_1 = require("../../../../cli/Cli");
 const config_1 = require("../../../../config");
 const request_1 = require("../../../../request");
-const utils_1 = require("../../../../utils");
+const formatting_1 = require("../../../../utils/formatting");
+const spo_1 = require("../../../../utils/spo");
+const validation_1 = require("../../../../utils/validation");
 const commands_1 = require("../../commands");
 const propertybag_base_1 = require("./propertybag-base");
 class SpoPropertyBagRemoveCommand extends propertybag_base_1.SpoPropertyBagBaseCommand {
@@ -35,45 +37,39 @@ class SpoPropertyBagRemoveCommand extends propertybag_base_1.SpoPropertyBagBaseC
     get description() {
         return 'Removes specified property from the property bag';
     }
-    commandAction(logger, args, cb) {
-        const removeProperty = () => {
-            utils_1.spo
-                .getRequestDigest(args.options.webUrl)
-                .then((contextResponse) => {
-                this.formDigestValue = contextResponse.FormDigestValue;
-                return utils_1.spo.getCurrentWebIdentity(args.options.webUrl, this.formDigestValue);
-            })
-                .then((identityResp) => {
-                const opts = args.options;
-                if (opts.folder) {
-                    // get the folder guid instead of the web guid
-                    return utils_1.spo.getFolderIdentity(identityResp.objectIdentity, opts.webUrl, opts.folder, this.formDigestValue);
+    commandAction(logger, args) {
+        return __awaiter(this, void 0, void 0, function* () {
+            const removeProperty = () => __awaiter(this, void 0, void 0, function* () {
+                try {
+                    const contextResponse = yield spo_1.spo.getRequestDigest(args.options.webUrl);
+                    this.formDigestValue = contextResponse.FormDigestValue;
+                    let identityResp = yield spo_1.spo.getCurrentWebIdentity(args.options.webUrl, this.formDigestValue);
+                    const opts = args.options;
+                    if (opts.folder) {
+                        // get the folder guid instead of the web guid
+                        identityResp = yield spo_1.spo.getFolderIdentity(identityResp.objectIdentity, opts.webUrl, opts.folder, this.formDigestValue);
+                    }
+                    yield this.removeProperty(identityResp, args.options);
                 }
-                return new Promise(resolve => { return resolve(identityResp); });
-            })
-                .then((identityResp) => {
-                return this.removeProperty(identityResp, args.options);
-            })
-                .then(_ => cb(), (err) => this.handleRejectedPromise(err, logger, cb));
-        };
-        if (args.options.confirm) {
-            removeProperty();
-        }
-        else {
-            cli_1.Cli.prompt({
-                type: 'confirm',
-                name: 'continue',
-                default: false,
-                message: `Are you sure you want to remove the ${args.options.key} property?`
-            }, (result) => {
-                if (!result.continue) {
-                    cb();
-                }
-                else {
-                    removeProperty();
+                catch (err) {
+                    this.handleRejectedPromise(err);
                 }
             });
-        }
+            if (args.options.confirm) {
+                yield removeProperty();
+            }
+            else {
+                const result = yield Cli_1.Cli.prompt({
+                    type: 'confirm',
+                    name: 'continue',
+                    default: false,
+                    message: `Are you sure you want to remove the ${args.options.key} property?`
+                });
+                if (result.continue) {
+                    yield removeProperty();
+                }
+            }
+        });
     }
     removeProperty(identityResp, options) {
         let objectType = 'AllProperties';
@@ -85,7 +81,7 @@ class SpoPropertyBagRemoveCommand extends propertybag_base_1.SpoPropertyBagBaseC
             headers: {
                 'X-RequestDigest': this.formDigestValue
             },
-            data: `<Request AddExpandoFieldTypeSuffix="true" SchemaVersion="15.0.0.0" LibraryVersion="16.0.0.0" ApplicationName="${config_1.default.applicationName}" xmlns="http://schemas.microsoft.com/sharepoint/clientquery/2009"><Actions><Method Name="SetFieldValue" Id="206" ObjectPathId="205"><Parameters><Parameter Type="String">${utils_1.formatting.escapeXml(options.key)}</Parameter><Parameter Type="Null" /></Parameters></Method><Method Name="Update" Id="207" ObjectPathId="198" /></Actions><ObjectPaths><Property Id="205" ParentId="198" Name="${objectType}" /><Identity Id="198" Name="${identityResp.objectIdentity}" /></ObjectPaths></Request>`
+            data: `<Request AddExpandoFieldTypeSuffix="true" SchemaVersion="15.0.0.0" LibraryVersion="16.0.0.0" ApplicationName="${config_1.default.applicationName}" xmlns="http://schemas.microsoft.com/sharepoint/clientquery/2009"><Actions><Method Name="SetFieldValue" Id="206" ObjectPathId="205"><Parameters><Parameter Type="String">${formatting_1.formatting.escapeXml(options.key)}</Parameter><Parameter Type="Null" /></Parameters></Method><Method Name="Update" Id="207" ObjectPathId="198" /></Actions><ObjectPaths><Property Id="205" ParentId="198" Name="${objectType}" /><Identity Id="198" Name="${identityResp.objectIdentity}" /></ObjectPaths></Request>`
         };
         return new Promise((resolve, reject) => {
             request_1.default.post(requestOptions).then((res) => {
@@ -119,7 +115,7 @@ _SpoPropertyBagRemoveCommand_instances = new WeakSet(), _SpoPropertyBagRemoveCom
         option: '--confirm'
     });
 }, _SpoPropertyBagRemoveCommand_initValidators = function _SpoPropertyBagRemoveCommand_initValidators() {
-    this.validators.push((args) => __awaiter(this, void 0, void 0, function* () { return utils_1.validation.isValidSharePointUrl(args.options.webUrl); }));
+    this.validators.push((args) => __awaiter(this, void 0, void 0, function* () { return validation_1.validation.isValidSharePointUrl(args.options.webUrl); }));
 };
 module.exports = new SpoPropertyBagRemoveCommand();
 //# sourceMappingURL=propertybag-remove.js.map

package/dist/m365/spo/commands/propertybag/propertybag-set.js

@@ -15,7 +15,8 @@ var __classPrivateFieldGet = (this && this.__classPrivateFieldGet) || function (
 };
 var _SpoPropertyBagSetCommand_instances, _SpoPropertyBagSetCommand_initTelemetry, _SpoPropertyBagSetCommand_initOptions, _SpoPropertyBagSetCommand_initValidators;
 Object.defineProperty(exports, "__esModule", { value: true });
-const utils_1 = require("../../../../utils");
+const spo_1 = require("../../../../utils/spo");
+const validation_1 = require("../../../../utils/validation");
 const commands_1 = require("../../commands");
 const propertybag_base_1 = require("./propertybag-base");
 class SpoPropertyBagSetCommand extends propertybag_base_1.SpoPropertyBagBaseCommand {
@@ -32,35 +33,30 @@ class SpoPropertyBagSetCommand extends propertybag_base_1.SpoPropertyBagBaseComm
     get description() {
         return 'Sets the value of the specified property in the property bag. Adds the property if it does not exist';
     }
-    commandAction(logger, args, cb) {
-        let webIdentityResp;
-        utils_1.spo
-            .getRequestDigest(args.options.webUrl)
-            .then((contextResponse) => {
-            this.formDigestValue = contextResponse.FormDigestValue;
-            return utils_1.spo.getCurrentWebIdentity(args.options.webUrl, this.formDigestValue);
-        })
-            .then((identityResp) => {
-            webIdentityResp = identityResp;
-            // Check if web no script enabled or not
-            // Cannot set property bag value if no script is enabled
-            return this.isNoScriptSite(identityResp, args.options, logger);
-        })
-            .then((isNoScriptSite) => {
-            if (isNoScriptSite) {
-                return Promise.reject('Site has NoScript enabled, and setting property bag values is not supported');
+    commandAction(logger, args) {
+        return __awaiter(this, void 0, void 0, function* () {
+            try {
+                const contextResponse = yield spo_1.spo.getRequestDigest(args.options.webUrl);
+                this.formDigestValue = contextResponse.FormDigestValue;
+                let identityResp = yield spo_1.spo.getCurrentWebIdentity(args.options.webUrl, this.formDigestValue);
+                const webIdentityResp = identityResp;
+                // Check if web no script enabled or not
+                // Cannot set property bag value if no script is enabled
+                const isNoScriptSite = yield this.isNoScriptSite(identityResp, args.options, logger);
+                if (isNoScriptSite) {
+                    throw 'Site has NoScript enabled, and setting property bag values is not supported';
+                }
+                const opts = args.options;
+                if (opts.folder) {
+                    // get the folder guid instead of the web guid
+                    identityResp = yield spo_1.spo.getFolderIdentity(webIdentityResp.objectIdentity, opts.webUrl, opts.folder, this.formDigestValue);
+                }
+                yield this.setProperty(identityResp, args.options, logger);
             }
-            const opts = args.options;
-            if (opts.folder) {
-                // get the folder guid instead of the web guid
-                return utils_1.spo.getFolderIdentity(webIdentityResp.objectIdentity, opts.webUrl, opts.folder, this.formDigestValue);
+            catch (err) {
+                this.handleRejectedPromise(err);
             }
-            return new Promise(resolve => { return resolve(webIdentityResp); });
-        })
-            .then((identityResp) => {
-            return this.setProperty(identityResp, args.options, logger);
-        })
-            .then(_ => cb(), (err) => this.handleRejectedPromise(err, logger, cb));
+        });
     }
     setProperty(identityResp, options, logger) {
         return propertybag_base_1.SpoPropertyBagBaseCommand.setProperty(options.key, options.value, options.webUrl, this.formDigestValue, identityResp, logger, this.debug, options.folder);
@@ -86,7 +82,7 @@ _SpoPropertyBagSetCommand_instances = new WeakSet(), _SpoPropertyBagSetCommand_i
         option: '-f, --folder [folder]'
     });
 }, _SpoPropertyBagSetCommand_initValidators = function _SpoPropertyBagSetCommand_initValidators() {
-    this.validators.push((args) => __awaiter(this, void 0, void 0, function* () { return utils_1.validation.isValidSharePointUrl(args.options.webUrl); }));
+    this.validators.push((args) => __awaiter(this, void 0, void 0, function* () { return validation_1.validation.isValidSharePointUrl(args.options.webUrl); }));
 };
 module.exports = new SpoPropertyBagSetCommand();
 //# sourceMappingURL=propertybag-set.js.map

package/dist/m365/spo/commands/roledefinition/roledefinition-add.js

@@ -0,0 +1,117 @@
+"use strict";
+var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
+    function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
+    return new (P || (P = Promise))(function (resolve, reject) {
+        function fulfilled(value) { try { step(generator.next(value)); } catch (e) { reject(e); } }
+        function rejected(value) { try { step(generator["throw"](value)); } catch (e) { reject(e); } }
+        function step(result) { result.done ? resolve(result.value) : adopt(result.value).then(fulfilled, rejected); }
+        step((generator = generator.apply(thisArg, _arguments || [])).next());
+    });
+};
+var __classPrivateFieldGet = (this && this.__classPrivateFieldGet) || function (receiver, state, kind, f) {
+    if (kind === "a" && !f) throw new TypeError("Private accessor was defined without a getter");
+    if (typeof state === "function" ? receiver !== state || !f : !state.has(receiver)) throw new TypeError("Cannot read private member from an object whose class did not declare it");
+    return kind === "m" ? f : kind === "a" ? f.call(receiver) : f ? f.value : state.get(receiver);
+};
+var _SpoRoleDefinitionAddCommand_instances, _SpoRoleDefinitionAddCommand_initTelemetry, _SpoRoleDefinitionAddCommand_initOptions, _SpoRoleDefinitionAddCommand_initValidators;
+Object.defineProperty(exports, "__esModule", { value: true });
+const request_1 = require("../../../../request");
+const validation_1 = require("../../../../utils/validation");
+const SpoCommand_1 = require("../../../base/SpoCommand");
+const base_permissions_1 = require("../../base-permissions");
+const commands_1 = require("../../commands");
+class SpoRoleDefinitionAddCommand extends SpoCommand_1.default {
+    constructor() {
+        super();
+        _SpoRoleDefinitionAddCommand_instances.add(this);
+        __classPrivateFieldGet(this, _SpoRoleDefinitionAddCommand_instances, "m", _SpoRoleDefinitionAddCommand_initTelemetry).call(this);
+        __classPrivateFieldGet(this, _SpoRoleDefinitionAddCommand_instances, "m", _SpoRoleDefinitionAddCommand_initOptions).call(this);
+        __classPrivateFieldGet(this, _SpoRoleDefinitionAddCommand_instances, "m", _SpoRoleDefinitionAddCommand_initValidators).call(this);
+    }
+    get name() {
+        return commands_1.default.ROLEDEFINITION_ADD;
+    }
+    get description() {
+        return 'Adds a new roledefinition to web';
+    }
+    get permissionsKindMap() {
+        const result = [];
+        for (const kind in base_permissions_1.PermissionKind) {
+            if (typeof base_permissions_1.PermissionKind[kind] === 'number') {
+                result.push(kind);
+            }
+        }
+        return result;
+    }
+    commandAction(logger, args) {
+        return __awaiter(this, void 0, void 0, function* () {
+            if (this.verbose) {
+                logger.logToStderr(`Adding role definition to ${args.options.webUrl}...`);
+            }
+            const description = args.options.description || '';
+            const permissions = new base_permissions_1.BasePermissions();
+            if (args.options.rights) {
+                const rights = args.options.rights.split(',');
+                for (const item of rights) {
+                    const kind = base_permissions_1.PermissionKind[item.trim()];
+                    permissions.set(kind);
+                }
+            }
+            const requestOptions = {
+                url: `${args.options.webUrl}/_api/web/roledefinitions`,
+                headers: {
+                    'accept': 'application/json;odata=nometadata'
+                },
+                responseType: 'json',
+                data: {
+                    'BasePermissions': {
+                        'High': permissions.high.toString(),
+                        'Low': permissions.low.toString()
+                    },
+                    'Description': `${description}`,
+                    'Name': `${args.options.name}`
+                }
+            };
+            try {
+                yield request_1.default.post(requestOptions);
+            }
+            catch (err) {
+                this.handleRejectedODataJsonPromise(err);
+            }
+        });
+    }
+}
+_SpoRoleDefinitionAddCommand_instances = new WeakSet(), _SpoRoleDefinitionAddCommand_initTelemetry = function _SpoRoleDefinitionAddCommand_initTelemetry() {
+    this.telemetry.push((args) => {
+        Object.assign(this.telemetryProperties, {
+            rights: args.options.rights,
+            description: (!(!args.options.description)).toString()
+        });
+    });
+}, _SpoRoleDefinitionAddCommand_initOptions = function _SpoRoleDefinitionAddCommand_initOptions() {
+    this.options.unshift({
+        option: '-u, --webUrl <webUrl>'
+    }, {
+        option: '-n, --name <name>'
+    }, {
+        option: '-d, --description [description]'
+    }, {
+        option: '--rights [rights]',
+        autocomplete: this.permissionsKindMap
+    });
+}, _SpoRoleDefinitionAddCommand_initValidators = function _SpoRoleDefinitionAddCommand_initValidators() {
+    this.validators.push((args) => __awaiter(this, void 0, void 0, function* () {
+        if (args.options.rights) {
+            const rights = args.options.rights.split(',');
+            for (const item of rights) {
+                const kind = base_permissions_1.PermissionKind[item.trim()];
+                if (!kind) {
+                    return `Rights option '${item}' is not recognized as valid PermissionKind choice. Please note it is case-sensitive. Allowed values are ${this.permissionsKindMap.join('|')}.`;
+                }
+            }
+        }
+        return validation_1.validation.isValidSharePointUrl(args.options.webUrl);
+    }));
+};
+module.exports = new SpoRoleDefinitionAddCommand();
+//# sourceMappingURL=roledefinition-add.js.map