@pnp/cli-microsoft365 5.4.0-beta.d01ee78 → 5.4.0-beta.fb9344d

package/dist/m365/planner/commands/task/task-remove.js

@@ -0,0 +1,201 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const os = require("os");
+const cli_1 = require("../../../../cli");
+const request_1 = require("../../../../request");
+const utils_1 = require("../../../../utils");
+const planner_1 = require("../../../../utils/planner");
+const GraphCommand_1 = require("../../../base/GraphCommand");
+const commands_1 = require("../../commands");
+class PlannerTaskRemoveCommand extends GraphCommand_1.default {
+    get name() {
+        return commands_1.default.TASK_REMOVE;
+    }
+    get description() {
+        return 'Removes the Microsoft Planner task from a plan';
+    }
+    commandAction(logger, args, cb) {
+        const removeTask = () => {
+            this
+                .getTask(args.options)
+                .then(task => {
+                const requestOptions = {
+                    url: `${this.resource}/v1.0/planner/tasks/${task.id}`,
+                    headers: {
+                        accept: 'application/json;odata.metadata=none',
+                        'if-match': task['@odata.etag']
+                    },
+                    responseType: 'json'
+                };
+                return request_1.default.delete(requestOptions);
+            })
+                .then(_ => cb(), (err) => this.handleRejectedODataJsonPromise(err, logger, cb));
+        };
+        if (args.options.confirm) {
+            removeTask();
+        }
+        else {
+            cli_1.Cli.prompt({
+                type: 'confirm',
+                name: 'continue',
+                default: false,
+                message: `Are you sure you want to remove the task ${args.options.id || args.options.title}?`
+            }, (result) => {
+                if (!result.continue) {
+                    cb();
+                }
+                else {
+                    removeTask();
+                }
+            });
+        }
+    }
+    getTask(options) {
+        const { id, title } = options;
+        if (id) {
+            const requestOptions = {
+                url: `${this.resource}/v1.0/planner/tasks/${id}`,
+                headers: {
+                    accept: 'application/json'
+                },
+                responseType: 'json'
+            };
+            return request_1.default.get(requestOptions);
+        }
+        return this
+            .getBucketId(options)
+            .then(bucketId => {
+            // $filter is not working on the buckets/{bucketId}/tasks endpoint, hence it is not being used.
+            return utils_1.odata.getAllItems(`${this.resource}/v1.0/planner/buckets/${bucketId}/tasks?$select=title,id`, 'minimal');
+        })
+            .then(tasks => {
+            const filteredtasks = tasks.filter(b => title.toLocaleLowerCase() === b.title.toLocaleLowerCase());
+            if (filteredtasks.length === 0) {
+                return Promise.reject(`The specified task ${title} does not exist`);
+            }
+            if (filteredtasks.length > 1) {
+                return Promise.reject(`Multiple tasks with title ${title} found: Please disambiguate: ${os.EOL}${filteredtasks.map(f => `- ${f.id}`).join(os.EOL)}`);
+            }
+            return Promise.resolve(filteredtasks[0]);
+        });
+    }
+    getBucketId(options) {
+        const { bucketId, bucketName } = options;
+        if (bucketId) {
+            return Promise.resolve(bucketId);
+        }
+        return this
+            .getPlanId(options)
+            .then(planId => {
+            const requestOptions = {
+                url: `${this.resource}/v1.0/planner/plans/${planId}/buckets?$select=id,name`,
+                headers: {
+                    accept: 'application/json;odata.metadata=none'
+                },
+                responseType: 'json'
+            };
+            return request_1.default.get(requestOptions);
+        })
+            .then((buckets) => {
+            const filteredBuckets = buckets.value.filter(b => bucketName.toLocaleLowerCase() === b.name.toLocaleLowerCase());
+            if (filteredBuckets.length === 0) {
+                return Promise.reject(`The specified bucket ${bucketName} does not exist`);
+            }
+            if (filteredBuckets.length > 1) {
+                return Promise.reject(`Multiple buckets with name ${bucketName} found: Please disambiguate:${os.EOL}${filteredBuckets.map(f => `- ${f.id}`).join(os.EOL)}`);
+            }
+            return Promise.resolve(filteredBuckets[0].id);
+        });
+    }
+    getPlanId(options) {
+        const { planId, planTitle } = options;
+        if (planId) {
+            return Promise.resolve(planId);
+        }
+        return this
+            .getGroupId(options)
+            .then(groupId => planner_1.planner.getPlanByName(planTitle, groupId))
+            .then(plan => plan.id);
+    }
+    getGroupId(options) {
+        const { ownerGroupId, ownerGroupName } = options;
+        if (ownerGroupId) {
+            return Promise.resolve(ownerGroupId);
+        }
+        const requestOptions = {
+            url: `${this.resource}/v1.0/groups?$filter=displayName eq '${encodeURIComponent(ownerGroupName)}'`,
+            headers: {
+                accept: 'application/json;odata.metadata=none'
+            },
+            responseType: 'json'
+        };
+        return request_1.default
+            .get(requestOptions)
+            .then(response => {
+            if (response.value.length === 0) {
+                return Promise.reject(`The specified owner group ${ownerGroupName} does not exist`);
+            }
+            if (response.value.length > 1) {
+                return Promise.reject(`Multiple owner groups with name ${ownerGroupName} found: ${response.value.map(x => x.id)}`);
+            }
+            return Promise.resolve(response.value[0].id);
+        });
+    }
+    options() {
+        const options = [
+            { option: '-i, --id [id]' },
+            { option: '-t, --title [title]' },
+            { option: '--bucketId [bucketId]' },
+            { option: '--bucketName [bucketName]' },
+            { option: '--planId [planId]' },
+            { option: '--planTitle [planTitle]' },
+            { option: '--ownerGroupId [ownerGroupId]' },
+            { option: '--ownerGroupName [ownerGroupName]' },
+            { option: '--confirm' }
+        ];
+        const parentOptions = super.options();
+        return options.concat(parentOptions);
+    }
+    optionSets() {
+        return [
+            ['id', 'title']
+        ];
+    }
+    validate(args) {
+        if (args.options.id) {
+            if (args.options.bucketId || args.options.bucketName || args.options.planId || args.options.planTitle || args.options.ownerGroupId || args.options.ownerGroupName) {
+                return 'Don\'t specify bucketId,bucketName, planId, planTitle, ownerGroupId or ownerGroupName when using id';
+            }
+        }
+        if (args.options.title) {
+            if (!args.options.bucketId && !args.options.bucketName) {
+                return 'Specify either bucketId or bucketName when using title';
+            }
+            if (args.options.bucketId && args.options.bucketName) {
+                return 'Specify either bucketId or bucketName when using title but not both';
+            }
+            if (args.options.bucketName) {
+                if (!args.options.planId && !args.options.planTitle) {
+                    return 'Specify either planId or planTitle when using bucketName';
+                }
+                if (args.options.planId && args.options.planTitle) {
+                    return 'Specify either planId or planTitle when using bucketName but not both';
+                }
+            }
+            if (args.options.planTitle) {
+                if (!args.options.ownerGroupId && !args.options.ownerGroupName) {
+                    return 'Specify either ownerGroupId or ownerGroupName when using planTitle';
+                }
+                if (args.options.ownerGroupId && args.options.ownerGroupName) {
+                    return 'Specify either ownerGroupId or ownerGroupName when using planTitle but not both';
+                }
+            }
+            if (args.options.ownerGroupId && !utils_1.validation.isValidGuid(args.options.ownerGroupId)) {
+                return `${args.options.ownerGroupId} is not a valid GUID`;
+            }
+        }
+        return true;
+    }
+}
+module.exports = new PlannerTaskRemoveCommand();
+//# sourceMappingURL=task-remove.js.map

package/dist/m365/planner/commands/task/task-set.js

@@ -6,6 +6,8 @@ const request_1 = require("../../../../request");
 const planner_1 = require("../../../../utils/planner");
 const GraphCommand_1 = require("../../../base/GraphCommand");
 const commands_1 = require("../../commands");
+const aadGroup_1 = require("../../../../utils/aadGroup");
+const taskPriority_1 = require("../../taskPriority");
 class PlannerTaskSetCommand extends GraphCommand_1.default {
     constructor() {
         super(...arguments);
@@ -35,6 +37,7 @@ class PlannerTaskSetCommand extends GraphCommand_1.default {
         telemetryProps.description = typeof args.options.description !== 'undefined';
         telemetryProps.appliedCategories = typeof args.options.appliedCategories !== 'undefined';
         telemetryProps.orderHint = typeof args.options.orderHint !== 'undefined';
+        telemetryProps.priority = typeof args.options.priority !== 'undefined';
         return telemetryProps;
     }
     commandAction(logger, args, cb) {
@@ -230,22 +233,9 @@ class PlannerTaskSetCommand extends GraphCommand_1.default {
         if (options.ownerGroupId) {
             return Promise.resolve(options.ownerGroupId);
         }
-        const requestOptions = {
-            url: `${this.resource}/v1.0/groups?$filter=displayName eq '${encodeURIComponent(options.ownerGroupName)}'&$select=id`,
-            headers: {
-                accept: 'application/json;odata.metadata=none'
-            },
-            responseType: 'json'
-        };
-        return request_1.default
-            .get(requestOptions)
-            .then(response => {
-            const group = response.value[0];
-            if (!group) {
-                return Promise.reject(`The specified owner group does not exist`);
-            }
-            return Promise.resolve(group.id);
-        });
+        return aadGroup_1.aadGroup
+            .getGroupByDisplayName(options.ownerGroupName)
+            .then(group => group.id);
     }
     mapRequestBody(options, appliedCategories) {
         const requestBody = {};
@@ -276,6 +266,9 @@ class PlannerTaskSetCommand extends GraphCommand_1.default {
         if (options.orderHint) {
             requestBody.orderHint = options.orderHint;
         }
+        if (options.priority !== undefined) {
+            requestBody.priority = taskPriority_1.taskPriority.getPriorityValue(options.priority);
+        }
         return requestBody;
     }
     options() {
@@ -296,7 +289,8 @@ class PlannerTaskSetCommand extends GraphCommand_1.default {
             { option: '--assigneePriority [assigneePriority]' },
             { option: '--description [description]' },
             { option: '--appliedCategories [appliedCategories]' },
-            { option: '--orderHint [orderHint]' }
+            { option: '--orderHint [orderHint]' },
+            { option: '--priority [priority]', autocomplete: taskPriority_1.taskPriority.priorityValues }
         ];
         const parentOptions = super.options();
         return options.concat(parentOptions);
@@ -341,6 +335,16 @@ class PlannerTaskSetCommand extends GraphCommand_1.default {
         if (args.options.appliedCategories && args.options.appliedCategories.split(',').filter(category => this.allowedAppliedCategories.indexOf(category.toLocaleLowerCase()) < 0).length !== 0) {
             return 'The appliedCategories contains invalid value. Specify either category1, category2, category3, category4, category5 and/or category6 as properties';
         }
+        if (args.options.priority !== undefined) {
+            if (typeof args.options.priority === "number") {
+                if (isNaN(args.options.priority) || args.options.priority < 0 || args.options.priority > 10 || !Number.isInteger(args.options.priority)) {
+                    return 'priority should be an integer between 0 and 10.';
+                }
+            }
+            else if (taskPriority_1.taskPriority.priorityValues.map(l => l.toLowerCase()).indexOf(args.options.priority.toString().toLowerCase()) === -1) {
+                return `${args.options.priority} is not a valid priority value. Allowed values are ${taskPriority_1.taskPriority.priorityValues.join('|')}.`;
+            }
+        }
         return true;
     }
 }

package/dist/m365/planner/commands/tenant/tenant-settings-list.js

@@ -0,0 +1,33 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const request_1 = require("../../../../request");
+const PlannerCommand_1 = require("../../../base/PlannerCommand");
+const commands_1 = require("../../commands");
+class PlannerTenantSettingsListCommand extends PlannerCommand_1.default {
+    get name() {
+        return commands_1.default.TENANT_SETTINGS_LIST;
+    }
+    get description() {
+        return 'Lists the Microsoft Planner configuration of the tenant';
+    }
+    defaultProperties() {
+        return ['isPlannerAllowed', 'allowCalendarSharing', 'allowTenantMoveWithDataLoss', 'allowTenantMoveWithDataMigration', 'allowRosterCreation', 'allowPlannerMobilePushNotifications'];
+    }
+    commandAction(logger, args, cb) {
+        const requestOptions = {
+            url: `${this.resource}/taskAPI/tenantAdminSettings/Settings`,
+            headers: {
+                accept: 'application/json;odata.metadata=none'
+            },
+            responseType: 'json'
+        };
+        request_1.default
+            .get(requestOptions)
+            .then((result) => {
+            logger.log(result);
+            cb();
+        }, (err) => this.handleRejectedODataJsonPromise(err, logger, cb));
+    }
+}
+module.exports = new PlannerTenantSettingsListCommand();
+//# sourceMappingURL=tenant-settings-list.js.map

package/dist/m365/planner/commands.js

@@ -3,20 +3,26 @@ Object.defineProperty(exports, "__esModule", { value: true });
 const prefix = 'planner';
 exports.default = {
     BUCKET_ADD: `${prefix} bucket add`,
+    BUCKET_GET: `${prefix} bucket get`,
     BUCKET_LIST: `${prefix} bucket list`,
     BUCKET_SET: `${prefix} bucket set`,
     BUCKET_REMOVE: `${prefix} bucket remove`,
-    BUCKET_GET: `${prefix} bucket get`,
     PLAN_ADD: `${prefix} plan add`,
     PLAN_DETAILS_GET: `${prefix} plan details get`,
     PLAN_GET: `${prefix} plan get`,
     PLAN_LIST: `${prefix} plan list`,
     TASK_ADD: `${prefix} task add`,
+    TASK_CHECKLISTITEM_ADD: `${prefix} task checklistitem add`,
+    TASK_CHECKLISTITEM_LIST: `${prefix} task checklistitem list`,
+    TASK_CHECKLISTITEM_REMOVE: `${prefix} task checklistitem remove`,
     TASK_DETAILS_GET: `${prefix} task details get`,
     TASK_GET: `${prefix} task get`,
     TASK_LIST: `${prefix} task list`,
     TASK_REFERENCE_ADD: `${prefix} task reference add`,
     TASK_REFERENCE_LIST: `${prefix} task reference list`,
-    TASK_SET: `${prefix} task set`
+    TASK_REFERENCE_REMOVE: `${prefix} task reference remove`,
+    TASK_REMOVE: `${prefix} task remove`,
+    TASK_SET: `${prefix} task set`,
+    TENANT_SETTINGS_LIST: `${prefix} tenant settings list`
 };
 //# sourceMappingURL=commands.js.map

package/dist/m365/planner/taskPriority.js

@@ -0,0 +1,27 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.taskPriority = void 0;
+exports.taskPriority = {
+    /** Priority labels used in Planner. */
+    priorityValues: ["Urgent", "Important", "Medium", "Low"],
+    /**
+     * Transform priority label to the corresponding number value.
+     * @param priority Priority label or number.
+     */
+    getPriorityValue(priority) {
+        if (typeof priority === "string") {
+            switch (priority.toLowerCase()) {
+                case "urgent":
+                    return 1;
+                case "important":
+                    return 3;
+                case "medium":
+                    return 5;
+                case "low":
+                    return 9;
+            }
+        }
+        return priority;
+    }
+};
+//# sourceMappingURL=taskPriority.js.map

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

@@ -116,7 +116,7 @@ class SpoContentTypeAddCommand extends SpoCommand_1.default {
                     logger.logToStderr(`Retrieving list id...`);
                 }
                 const requestOptions = {
-                    url: `${webUrl}/_api/web/lists/getByTitle('${encodeURIComponent(listTitle)}')?$select=Id`,
+                    url: `${webUrl}/_api/web/lists/getByTitle('${utils_1.formatting.encodeQueryParameter(listTitle)}')?$select=Id`,
                     headers: {
                         accept: 'application/json;odata=nometadata'
                     },

package/dist/m365/spo/commands/contenttype/contenttype-field-remove.js

@@ -70,7 +70,7 @@ class SpoContentTypeFieldRemoveCommand extends SpoCommand_1.default {
                 }
                 // Request for the ListId
                 const requestOptions = {
-                    url: `${args.options.webUrl}/_api/lists/GetByTitle('${encodeURIComponent(args.options.listTitle)}')?$select=Id`,
+                    url: `${args.options.webUrl}/_api/lists/GetByTitle('${utils_1.formatting.encodeQueryParameter(args.options.listTitle)}')?$select=Id`,
                     headers: {
                         accept: 'application/json;odata=nometadata'
                     },

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

@@ -18,7 +18,7 @@ class SpoContentTypeGetCommand extends SpoCommand_1.default {
         };
     }
     commandAction(logger, args, cb) {
-        let requestUrl = `${args.options.webUrl}/_api/web/${(args.options.listTitle ? `lists/getByTitle('${encodeURIComponent(args.options.listTitle)}')/` : '')}contenttypes`;
+        let requestUrl = `${args.options.webUrl}/_api/web/${(args.options.listTitle ? `lists/getByTitle('${utils_1.formatting.encodeQueryParameter(args.options.listTitle)}')/` : '')}contenttypes`;
         if (args.options.id) {
             requestUrl += `('${encodeURIComponent(args.options.id)}')`;
         }

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

@@ -26,14 +26,14 @@ class SpoEventreceiverGetCommand extends SpoCommand_1.default {
         let listUrl = '';
         let filter = '?$filter=';
         if (args.options.listId) {
-            listUrl = `lists(guid'${encodeURIComponent(args.options.listId)}')/`;
+            listUrl = `lists(guid'${utils_1.formatting.encodeQueryParameter(args.options.listId)}')/`;
         }
         else if (args.options.listTitle) {
-            listUrl = `lists/getByTitle('${encodeURIComponent(args.options.listTitle)}')/`;
+            listUrl = `lists/getByTitle('${utils_1.formatting.encodeQueryParameter(args.options.listTitle)}')/`;
         }
         else if (args.options.listUrl) {
             const listServerRelativeUrl = utils_1.urlUtil.getServerRelativePath(args.options.webUrl, args.options.listUrl);
-            listUrl = `GetList('${encodeURIComponent(listServerRelativeUrl)}')/`;
+            listUrl = `GetList('${utils_1.formatting.encodeQueryParameter(listServerRelativeUrl)}')/`;
         }
         if (!args.options.scope || args.options.scope === 'web') {
             requestUrl += `web/${listUrl}eventreceivers`;

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

@@ -0,0 +1,102 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const request_1 = require("../../../../request");
+const utils_1 = require("../../../../utils");
+const SpoCommand_1 = require("../../../base/SpoCommand");
+const commands_1 = require("../../commands");
+class SpoEventreceiverListCommand extends SpoCommand_1.default {
+    get name() {
+        return commands_1.default.EVENTRECEIVER_LIST;
+    }
+    get description() {
+        return 'Retrieves event receivers for the specified web, site or list.';
+    }
+    defaultProperties() {
+        return ['ReceiverId', 'ReceiverName'];
+    }
+    getTelemetryProperties(args) {
+        const telemetryProps = super.getTelemetryProperties(args);
+        telemetryProps.listId = typeof args.options.listId !== 'undefined';
+        telemetryProps.listTitle = typeof args.options.listTitle !== 'undefined';
+        telemetryProps.listUrl = typeof args.options.listUrl !== 'undefined';
+        telemetryProps.scope = typeof args.options.scope !== 'undefined';
+        return telemetryProps;
+    }
+    commandAction(logger, args, cb) {
+        let requestUrl = `${args.options.webUrl}/_api/`;
+        let listUrl = '';
+        if (args.options.listId) {
+            listUrl = `lists(guid'${encodeURIComponent(args.options.listId)}')/`;
+        }
+        else if (args.options.listTitle) {
+            listUrl = `lists/getByTitle('${encodeURIComponent(args.options.listTitle)}')/`;
+        }
+        else if (args.options.listUrl) {
+            const listServerRelativeUrl = utils_1.urlUtil.getServerRelativePath(args.options.webUrl, args.options.listUrl);
+            listUrl = `GetList('${encodeURIComponent(listServerRelativeUrl)}')/`;
+        }
+        if (!args.options.scope || args.options.scope === 'web') {
+            requestUrl += `web/${listUrl}eventreceivers`;
+        }
+        else {
+            requestUrl += 'site/eventreceivers';
+        }
+        const requestOptions = {
+            url: requestUrl,
+            headers: {
+                'accept': 'application/json;odata=nometadata'
+            },
+            responseType: 'json'
+        };
+        request_1.default
+            .get(requestOptions)
+            .then((res) => {
+            logger.log(res.value);
+            cb();
+        }, (err) => this.handleRejectedODataJsonPromise(err, logger, cb));
+    }
+    options() {
+        const options = [
+            {
+                option: '-u, --webUrl <webUrl>'
+            },
+            {
+                option: '--listTitle [listTitle]'
+            },
+            {
+                option: '--listId  [listId]'
+            },
+            {
+                option: '--listUrl [listUrl]'
+            },
+            {
+                option: '-s, --scope [scope]',
+                autocomplete: ['web', 'site']
+            }
+        ];
+        const parentOptions = super.options();
+        return options.concat(parentOptions);
+    }
+    validate(args) {
+        const isValidSharePointUrl = utils_1.validation.isValidSharePointUrl(args.options.webUrl);
+        if (isValidSharePointUrl !== true) {
+            return isValidSharePointUrl;
+        }
+        const listOptions = [args.options.listId, args.options.listTitle, args.options.listUrl];
+        if (listOptions.some(item => item !== undefined) && listOptions.filter(item => item !== undefined).length > 1) {
+            return `Specify either list id or title or list url`;
+        }
+        if (args.options.listId && !utils_1.validation.isValidGuid(args.options.listId)) {
+            return `${args.options.listId} is not a valid GUID`;
+        }
+        if (args.options.scope && ['web', 'site'].indexOf(args.options.scope) === -1) {
+            return `${args.options.scope} is not a valid type value. Allowed values web|site.`;
+        }
+        if (args.options.scope && args.options.scope === 'site' && (args.options.listId || args.options.listUrl || args.options.listTitle)) {
+            return 'Scope cannot be set to site when retrieving list event receivers.';
+        }
+        return true;
+    }
+}
+module.exports = new SpoEventreceiverListCommand();
+//# sourceMappingURL=eventreceiver-list.js.map

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

@@ -16,7 +16,7 @@ class SpoFieldAddCommand extends SpoCommand_1.default {
             .getRequestDigest(args.options.webUrl)
             .then((res) => {
             const requestOptions = {
-                url: `${args.options.webUrl}/_api/web/${(args.options.listTitle ? `lists/getByTitle('${encodeURIComponent(args.options.listTitle)}')/` : '')}fields/CreateFieldAsXml`,
+                url: `${args.options.webUrl}/_api/web/${(args.options.listTitle ? `lists/getByTitle('${utils_1.formatting.encodeQueryParameter(args.options.listTitle)}')/` : '')}fields/CreateFieldAsXml`,
                 headers: {
                     'X-RequestDigest': res.FormDigestValue,
                     accept: 'application/json;odata=nometadata'

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

@@ -23,21 +23,21 @@ class SpoFieldGetCommand extends SpoCommand_1.default {
     commandAction(logger, args, cb) {
         let listRestUrl = '';
         if (args.options.listId) {
-            listRestUrl = `lists(guid'${encodeURIComponent(args.options.listId)}')/`;
+            listRestUrl = `lists(guid'${utils_1.formatting.encodeQueryParameter(args.options.listId)}')/`;
         }
         else if (args.options.listTitle) {
-            listRestUrl = `lists/getByTitle('${encodeURIComponent(args.options.listTitle)}')/`;
+            listRestUrl = `lists/getByTitle('${utils_1.formatting.encodeQueryParameter(args.options.listTitle)}')/`;
         }
         else if (args.options.listUrl) {
             const listServerRelativeUrl = utils_1.urlUtil.getServerRelativePath(args.options.webUrl, args.options.listUrl);
-            listRestUrl = `GetList('${encodeURIComponent(listServerRelativeUrl)}')/`;
+            listRestUrl = `GetList('${utils_1.formatting.encodeQueryParameter(listServerRelativeUrl)}')/`;
         }
         let fieldRestUrl = '';
         if (args.options.id) {
-            fieldRestUrl = `/getbyid('${encodeURIComponent(args.options.id)}')`;
+            fieldRestUrl = `/getbyid('${utils_1.formatting.encodeQueryParameter(args.options.id)}')`;
         }
         else {
-            fieldRestUrl = `/getbyinternalnameortitle('${encodeURIComponent(args.options.fieldTitle)}')`;
+            fieldRestUrl = `/getbyinternalnameortitle('${utils_1.formatting.encodeQueryParameter(args.options.fieldTitle)}')`;
         }
         const requestOptions = {
             url: `${args.options.webUrl}/_api/web/${listRestUrl}fields${fieldRestUrl}`,

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

@@ -24,14 +24,14 @@ class SpoFieldListCommand extends SpoCommand_1.default {
     commandAction(logger, args, cb) {
         let listUrl = '';
         if (args.options.listId) {
-            listUrl = `lists(guid'${encodeURIComponent(args.options.listId)}')/`;
+            listUrl = `lists(guid'${utils_1.formatting.encodeQueryParameter(args.options.listId)}')/`;
         }
         else if (args.options.listTitle) {
-            listUrl = `lists/getByTitle('${encodeURIComponent(args.options.listTitle)}')/`;
+            listUrl = `lists/getByTitle('${utils_1.formatting.encodeQueryParameter(args.options.listTitle)}')/`;
         }
         else if (args.options.listUrl) {
             const listServerRelativeUrl = utils_1.urlUtil.getServerRelativePath(args.options.webUrl, args.options.listUrl);
-            listUrl = `GetList('${encodeURIComponent(listServerRelativeUrl)}')/`;
+            listUrl = `GetList('${utils_1.formatting.encodeQueryParameter(listServerRelativeUrl)}')/`;
         }
         const requestOptions = {
             url: `${args.options.webUrl}/_api/web/${listUrl}fields`,

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

@@ -37,10 +37,10 @@ class SpoFieldRemoveCommand extends SpoCommand_1.default {
             }
             let fieldRestUrl = '';
             if (fieldId) {
-                fieldRestUrl = `/getbyid('${encodeURIComponent(fieldId)}')`;
+                fieldRestUrl = `/getbyid('${utils_1.formatting.encodeQueryParameter(fieldId)}')`;
             }
             else {
-                fieldRestUrl = `/getbyinternalnameortitle('${encodeURIComponent(fieldTitle)}')`;
+                fieldRestUrl = `/getbyinternalnameortitle('${utils_1.formatting.encodeQueryParameter(fieldTitle)}')`;
             }
             const requestOptions = {
                 url: `${args.options.webUrl}/_api/web/${listRestUrl}fields${fieldRestUrl}`,
@@ -57,14 +57,14 @@ class SpoFieldRemoveCommand extends SpoCommand_1.default {
         const prepareRemoval = () => {
             let listRestUrl = '';
             if (args.options.listId) {
-                listRestUrl = `lists(guid'${encodeURIComponent(args.options.listId)}')/`;
+                listRestUrl = `lists(guid'${utils_1.formatting.encodeQueryParameter(args.options.listId)}')/`;
             }
             else if (args.options.listTitle) {
-                listRestUrl = `lists/getByTitle('${encodeURIComponent(args.options.listTitle)}')/`;
+                listRestUrl = `lists/getByTitle('${utils_1.formatting.encodeQueryParameter(args.options.listTitle)}')/`;
             }
             else if (args.options.listUrl) {
                 const listServerRelativeUrl = utils_1.urlUtil.getServerRelativePath(args.options.webUrl, args.options.listUrl);
-                listRestUrl = `GetList('${encodeURIComponent(listServerRelativeUrl)}')/`;
+                listRestUrl = `GetList('${utils_1.formatting.encodeQueryParameter(listServerRelativeUrl)}')/`;
             }
             if (args.options.group) {
                 if (this.verbose) {

package/dist/m365/spo/commands/file/file-sharinginfo-get.js

@@ -32,7 +32,7 @@ class SpoFileSharinginfoGetCommand extends SpoCommand_1.default {
                 logger.logToStderr(`Retrieving sharing information report for the file with item Id  ${fileInformation.fileItemId}`);
             }
             const requestOptions = {
-                url: `${args.options.webUrl}/_api/web/lists/getbytitle('${fileInformation.libraryName}')/items(${fileInformation.fileItemId})/GetSharingInformation?$select=permissionsInformation&$Expand=permissionsInformation`,
+                url: `${args.options.webUrl}/_api/web/lists/getbytitle('${utils_1.formatting.encodeQueryParameter(fileInformation.libraryName)}')/items(${fileInformation.fileItemId})/GetSharingInformation?$select=permissionsInformation&$Expand=permissionsInformation`,
                 headers: {
                     'accept': 'application/json;odata=nometadata'
                 },

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

@@ -42,7 +42,7 @@ class SpoHubSiteListCommand extends SpoCommand_1.default {
         })
             .then((res) => {
             hubSites = res.value;
-            if (args.options.includeAssociatedSites !== true || args.options.output !== 'json') {
+            if (args.options.includeAssociatedSites !== true || args.options.output && args.options.output !== 'json') {
                 return Promise.resolve();
             }
             else {

package/dist/m365/spo/commands/list/ListPrincipalType.js

@@ -0,0 +1,13 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.ListPrincipalType = void 0;
+var ListPrincipalType;
+(function (ListPrincipalType) {
+    ListPrincipalType[ListPrincipalType["None"] = 0] = "None";
+    ListPrincipalType[ListPrincipalType["User"] = 1] = "User";
+    ListPrincipalType[ListPrincipalType["DistributionList"] = 2] = "DistributionList";
+    ListPrincipalType[ListPrincipalType["SecurityGroup"] = 4] = "SecurityGroup";
+    ListPrincipalType[ListPrincipalType["SharePointGroup"] = 8] = "SharePointGroup";
+    ListPrincipalType[ListPrincipalType["All"] = 15] = "All";
+})(ListPrincipalType = exports.ListPrincipalType || (exports.ListPrincipalType = {}));
+//# sourceMappingURL=ListPrincipalType.js.map