@pnp/cli-microsoft365 5.5.0-beta.4e973d9 → 5.5.0-beta.8bd301f

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

@@ -21,12 +21,17 @@ class PlannerBucketRemoveCommand extends GraphCommand_1.default {
         telemetryProps.name = typeof args.options.name !== 'undefined';
         telemetryProps.planId = typeof args.options.planId !== 'undefined';
         telemetryProps.planName = typeof args.options.planName !== 'undefined';
+        telemetryProps.planTitle = typeof args.options.planTitle !== 'undefined';
         telemetryProps.ownerGroupId = typeof args.options.ownerGroupId !== 'undefined';
         telemetryProps.ownerGroupName = typeof args.options.ownerGroupName !== 'undefined';
         telemetryProps.confirm = args.options.confirm || false;
         return telemetryProps;
     }
     commandAction(logger, args, cb) {
+        if (args.options.planName) {
+            args.options.planTitle = args.options.planName;
+            this.warn(logger, `Option 'planName' is deprecated. Please use 'planTitle' instead`);
+        }
         if (utils_1.accessToken.isAppOnlyAccessToken(Auth_1.default.service.accessTokens[this.resource].accessToken)) {
             this.handleError('This command does not support application permissions.', logger, cb);
             return;
@@ -101,13 +106,13 @@ class PlannerBucketRemoveCommand extends GraphCommand_1.default {
         });
     }
     getPlanId(args) {
-        const { planId, planName } = args.options;
+        const { planId, planTitle } = args.options;
         if (planId) {
             return Promise.resolve(planId);
         }
         return this
             .getGroupId(args)
-            .then(groupId => planner_1.planner.getPlanByName(planName, groupId))
+            .then(groupId => planner_1.planner.getPlanByTitle(planTitle, groupId))
             .then(plan => plan.id);
     }
     getGroupId(args) {
@@ -133,6 +138,9 @@ class PlannerBucketRemoveCommand extends GraphCommand_1.default {
             {
                 option: '--planName [planName]'
             },
+            {
+                option: "--planTitle [planTitle]"
+            },
             {
                 option: '--ownerGroupId [ownerGroupId]'
             },
@@ -153,23 +161,23 @@ class PlannerBucketRemoveCommand extends GraphCommand_1.default {
     }
     validate(args) {
         if (args.options.id) {
-            if (args.options.planId || args.options.planName || args.options.ownerGroupId || args.options.ownerGroupName) {
-                return 'Don\'t specify planId, planName, ownerGroupId or ownerGroupName when using id';
+            if (args.options.planId || args.options.planName || args.options.planTitle || args.options.ownerGroupId || args.options.ownerGroupName) {
+                return 'Don\'t specify planId, planTitle, ownerGroupId or ownerGroupName when using id';
             }
         }
         else {
-            if (!args.options.planId && !args.options.planName) {
-                return 'Specify either planId or planName when using name';
+            if (!args.options.planId && !args.options.planName && !args.options.planTitle) {
+                return 'Specify either planId or planTitle when using name';
             }
-            if (args.options.planId && args.options.planName) {
-                return 'Specify either planId or planName when using name but not both';
+            if (args.options.planId && (args.options.planName || args.options.planTitle)) {
+                return 'Specify either planId or planTitle when using name but not both';
             }
-            if (args.options.planName) {
+            if (args.options.planName || args.options.planTitle) {
                 if (!args.options.ownerGroupId && !args.options.ownerGroupName) {
-                    return 'Specify either ownerGroupId or ownerGroupName when using planName';
+                    return 'Specify either ownerGroupId or ownerGroupName when using planTitle';
                 }
                 if (args.options.ownerGroupId && args.options.ownerGroupName) {
-                    return 'Specify either ownerGroupId or ownerGroupName when using planName but not both';
+                    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`;

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

@@ -20,6 +20,7 @@ class PlannerBucketSetCommand extends GraphCommand_1.default {
         telemetryProps.name = typeof args.options.name !== 'undefined';
         telemetryProps.planId = typeof args.options.planId !== 'undefined';
         telemetryProps.planName = typeof args.options.planName !== 'undefined';
+        telemetryProps.planTitle = typeof args.options.planTitle !== 'undefined';
         telemetryProps.ownerGroupId = typeof args.options.ownerGroupId !== 'undefined';
         telemetryProps.ownerGroupName = typeof args.options.ownerGroupName !== 'undefined';
         telemetryProps.newName = typeof args.options.newName !== 'undefined';
@@ -27,6 +28,10 @@ class PlannerBucketSetCommand extends GraphCommand_1.default {
         return telemetryProps;
     }
     commandAction(logger, args, cb) {
+        if (args.options.planName) {
+            args.options.planTitle = args.options.planName;
+            this.warn(logger, `Option 'planName' is deprecated. Please use 'planTitle' instead`);
+        }
         if (utils_1.accessToken.isAppOnlyAccessToken(Auth_1.default.service.accessTokens[this.resource].accessToken)) {
             this.handleError('This command does not support application permissions.', logger, cb);
             return;
@@ -89,13 +94,13 @@ class PlannerBucketSetCommand extends GraphCommand_1.default {
         });
     }
     getPlanId(args) {
-        const { planId, planName } = args.options;
+        const { planId, planTitle } = args.options;
         if (planId) {
             return Promise.resolve(planId);
         }
         return this
             .getGroupId(args)
-            .then(groupId => planner_1.planner.getPlanByName(planName, groupId))
+            .then(groupId => planner_1.planner.getPlanByTitle(planTitle, groupId))
             .then(plan => plan.id);
     }
     getGroupId(args) {
@@ -121,6 +126,9 @@ class PlannerBucketSetCommand extends GraphCommand_1.default {
             {
                 option: '--planName [planName]'
             },
+            {
+                option: "--planTitle [planTitle]"
+            },
             {
                 option: '--ownerGroupId [ownerGroupId]'
             },
@@ -144,23 +152,23 @@ class PlannerBucketSetCommand extends GraphCommand_1.default {
     }
     validate(args) {
         if (args.options.id) {
-            if (args.options.planId || args.options.planName || args.options.ownerGroupId || args.options.ownerGroupName) {
-                return 'Don\'t specify planId, planName, ownerGroupId or ownerGroupName when using id';
+            if (args.options.planId || args.options.planName || args.options.planTitle || args.options.ownerGroupId || args.options.ownerGroupName) {
+                return 'Don\'t specify planId, planTitle, ownerGroupId or ownerGroupName when using id';
             }
         }
         if (args.options.name) {
-            if (!args.options.planId && !args.options.planName) {
-                return 'Specify either planId or planName when using name';
+            if (!args.options.planId && !args.options.planName && !args.options.planTitle) {
+                return 'Specify either planId or planTitle when using name';
             }
-            if (args.options.planId && args.options.planName) {
-                return 'Specify either planId or planName when using name but not both';
+            if (args.options.planId && (args.options.planName || args.options.planTitle)) {
+                return 'Specify either planId or planTitle when using name but not both';
             }
-            if (args.options.planName) {
+            if (args.options.planName || args.options.planTitle) {
                 if (!args.options.ownerGroupId && !args.options.ownerGroupName) {
-                    return 'Specify either ownerGroupId or ownerGroupName when using planName';
+                    return 'Specify either ownerGroupId or ownerGroupName when using planTitle';
                 }
                 if (args.options.ownerGroupId && args.options.ownerGroupName) {
-                    return 'Specify either ownerGroupId or ownerGroupName when using planName but not both';
+                    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`;

package/dist/m365/planner/commands/plan/plan-add.js

@@ -1,11 +1,10 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-const utils_1 = require("../../../../utils");
 const Auth_1 = require("../../../../Auth");
 const request_1 = require("../../../../request");
+const utils_1 = require("../../../../utils");
 const GraphCommand_1 = require("../../../base/GraphCommand");
 const commands_1 = require("../../commands");
-const aadGroup_1 = require("../../../../utils/aadGroup");
 class PlannerPlanAddCommand extends GraphCommand_1.default {
     get name() {
         return commands_1.default.PLAN_ADD;
@@ -17,6 +16,8 @@ class PlannerPlanAddCommand extends GraphCommand_1.default {
         const telemetryProps = super.getTelemetryProperties(args);
         telemetryProps.ownerGroupId = typeof args.options.ownerGroupId !== 'undefined';
         telemetryProps.ownerGroupName = typeof args.options.ownerGroupName !== 'undefined';
+        telemetryProps.shareWithUserIds = typeof args.options.shareWithUserIds !== 'undefined';
+        telemetryProps.shareWithUserNames = typeof args.options.shareWithUserNames !== 'undefined';
         return telemetryProps;
     }
     defaultProperties() {
@@ -43,16 +44,96 @@ class PlannerPlanAddCommand extends GraphCommand_1.default {
             };
             return request_1.default.post(requestOptions);
         })
+            .then(newPlan => this.updatePlanDetails(args.options, newPlan))
             .then((res) => {
             logger.log(res);
             cb();
         }, (err) => this.handleRejectedODataJsonPromise(err, logger, cb));
     }
+    updatePlanDetails(options, newPlan) {
+        const planId = newPlan.id;
+        if (!options.shareWithUserIds && !options.shareWithUserNames) {
+            return Promise.resolve(newPlan);
+        }
+        return Promise
+            .all([this.generateSharedWith(options), this.getPlanDetailsEtag(planId)])
+            .then(resArray => {
+            const sharedWith = resArray[0];
+            const etag = resArray[1];
+            const requestOptionsPlanDetails = {
+                url: `${this.resource}/v1.0/planner/plans/${planId}/details`,
+                headers: {
+                    'accept': 'application/json;odata.metadata=none',
+                    'If-Match': etag,
+                    'Prefer': 'return=representation'
+                },
+                responseType: 'json',
+                data: {
+                    sharedWith: sharedWith
+                }
+            };
+            return request_1.default.patch(requestOptionsPlanDetails);
+        })
+            .then(planDetails => {
+            return Object.assign(Object.assign({}, newPlan), planDetails);
+        });
+    }
+    getPlanDetailsEtag(planId) {
+        const requestOptions = {
+            url: `${this.resource}/v1.0/planner/plans/${planId}/details`,
+            headers: {
+                accept: 'application/json'
+            },
+            responseType: 'json'
+        };
+        return request_1.default
+            .get(requestOptions)
+            .then((response) => response['@odata.etag']);
+    }
+    generateSharedWith(options) {
+        const sharedWith = {};
+        return this
+            .getUserIds(options)
+            .then((userIds) => {
+            userIds.map(x => sharedWith[x] = true);
+            return Promise.resolve(sharedWith);
+        });
+    }
+    getUserIds(options) {
+        if (options.shareWithUserIds) {
+            return Promise.resolve(options.shareWithUserIds.split(','));
+        }
+        // Hitting this section means assignedToUserNames won't be undefined
+        const userNames = options.shareWithUserNames;
+        const userArr = userNames.split(',').map(o => o.trim());
+        const promises = userArr.map(user => {
+            const requestOptions = {
+                url: `${this.resource}/v1.0/users?$filter=userPrincipalName eq '${utils_1.formatting.encodeQueryParameter(user)}'&$select=id,userPrincipalName`,
+                headers: {
+                    'content-type': 'application/json'
+                },
+                responseType: 'json'
+            };
+            return request_1.default.get(requestOptions);
+        });
+        return Promise
+            .all(promises)
+            .then((usersRes) => {
+            const userUpns = usersRes.map(res => { var _a; return (_a = res.value[0]) === null || _a === void 0 ? void 0 : _a.userPrincipalName; });
+            const userIds = usersRes.map(res => { var _a; return (_a = res.value[0]) === null || _a === void 0 ? void 0 : _a.id; });
+            // Find the members where no graph response was found
+            const invalidUsers = userArr.filter(user => !userUpns.some((upn) => (upn === null || upn === void 0 ? void 0 : upn.toLowerCase()) === user.toLowerCase()));
+            if (invalidUsers && invalidUsers.length > 0) {
+                return Promise.reject(`Cannot proceed with planner plan creation. The following users provided are invalid : ${invalidUsers.join(',')}`);
+            }
+            return Promise.resolve(userIds);
+        });
+    }
     getGroupId(args) {
         if (args.options.ownerGroupId) {
             return Promise.resolve(args.options.ownerGroupId);
         }
-        return aadGroup_1.aadGroup
+        return utils_1.aadGroup
             .getGroupByDisplayName(args.options.ownerGroupName)
             .then(group => group.id);
     }
@@ -62,10 +143,16 @@ class PlannerPlanAddCommand extends GraphCommand_1.default {
                 option: '-t, --title <title>'
             },
             {
-                option: "--ownerGroupId [ownerGroupId]"
+                option: '--ownerGroupId [ownerGroupId]'
             },
             {
-                option: "--ownerGroupName [ownerGroupName]"
+                option: '--ownerGroupName [ownerGroupName]'
+            },
+            {
+                option: '--shareWithUserIds [shareWithUserIds]'
+            },
+            {
+                option: '--shareWithUserNames [shareWithUserNames]'
             }
         ];
         const parentOptions = super.options();
@@ -81,6 +168,12 @@ class PlannerPlanAddCommand extends GraphCommand_1.default {
         if (args.options.ownerGroupId && !utils_1.validation.isValidGuid(args.options.ownerGroupId)) {
             return `${args.options.ownerGroupId} is not a valid GUID`;
         }
+        if (args.options.shareWithUserIds && args.options.shareWithUserNames) {
+            return 'Specify either shareWithUserIds or shareWithUserNames but not both';
+        }
+        if (args.options.shareWithUserIds && !utils_1.validation.isValidGuidArray(args.options.shareWithUserIds.split(','))) {
+            return 'shareWithUserIds contains invalid GUID';
+        }
         return true;
     }
 }

package/dist/m365/planner/commands/plan/plan-get.js

@@ -6,15 +6,21 @@ const GraphCommand_1 = require("../../../base/GraphCommand");
 const commands_1 = require("../../commands");
 const Auth_1 = require("../../../../Auth");
 const aadGroup_1 = require("../../../../utils/aadGroup");
+const request_1 = require("../../../../request");
 class PlannerPlanGetCommand extends GraphCommand_1.default {
     get name() {
         return commands_1.default.PLAN_GET;
     }
+    alias() {
+        return [commands_1.default.PLAN_DETAILS_GET];
+    }
     get description() {
         return 'Get a Microsoft Planner plan';
     }
     getTelemetryProperties(args) {
         const telemetryProps = super.getTelemetryProperties(args);
+        telemetryProps.planId = typeof args.options.planId !== 'undefined';
+        telemetryProps.planTitle = typeof args.options.planTitle !== 'undefined';
         telemetryProps.id = typeof args.options.id !== 'undefined';
         telemetryProps.title = typeof args.options.title !== 'undefined';
         telemetryProps.ownerGroupId = typeof args.options.ownerGroupId !== 'undefined';
@@ -25,13 +31,21 @@ class PlannerPlanGetCommand extends GraphCommand_1.default {
         return ['id', 'title', 'createdDateTime', 'owner', '@odata.etag'];
     }
     commandAction(logger, args, cb) {
+        this.showDeprecationWarning(logger, commands_1.default.PLAN_DETAILS_GET, commands_1.default.PLAN_GET);
         if (utils_1.accessToken.isAppOnlyAccessToken(Auth_1.default.service.accessTokens[this.resource].accessToken)) {
             this.handleError('This command does not support application permissions.', logger, cb);
             return;
         }
+        if (args.options.planId) {
+            args.options.id = args.options.planId;
+        }
+        if (args.options.planTitle) {
+            args.options.title = args.options.planTitle;
+        }
         if (args.options.id) {
             planner_1.planner
                 .getPlanById(args.options.id)
+                .then(plan => this.getPlanDetails(plan))
                 .then((res) => {
                 logger.log(res);
                 cb();
@@ -40,15 +54,31 @@ class PlannerPlanGetCommand extends GraphCommand_1.default {
         else {
             this
                 .getGroupId(args)
-                .then(groupId => planner_1.planner.getPlanByName(args.options.title, groupId))
-                .then((plan) => {
-                if (plan) {
-                    logger.log(plan);
+                .then(groupId => planner_1.planner.getPlanByTitle(args.options.title, groupId))
+                .then(plan => this.getPlanDetails(plan))
+                .then((res) => {
+                if (res) {
+                    logger.log(res);
                 }
                 cb();
             }, (err) => this.handleRejectedODataJsonPromise(err, logger, cb));
         }
     }
+    getPlanDetails(plan) {
+        const requestOptionsTaskDetails = {
+            url: `${this.resource}/v1.0/planner/plans/${plan.id}/details`,
+            headers: {
+                'accept': 'application/json;odata.metadata=none',
+                'Prefer': 'return=representation'
+            },
+            responseType: 'json'
+        };
+        return request_1.default
+            .get(requestOptionsTaskDetails)
+            .then(planDetails => {
+            return Object.assign(Object.assign({}, plan), planDetails);
+        });
+    }
     getGroupId(args) {
         if (args.options.ownerGroupId) {
             return Promise.resolve(args.options.ownerGroupId);
@@ -59,6 +89,12 @@ class PlannerPlanGetCommand extends GraphCommand_1.default {
     }
     options() {
         const options = [
+            {
+                option: '--planId [planId]'
+            },
+            {
+                option: '--planTitle [planTitle]'
+            },
             {
                 option: '-i, --id [id]'
             },
@@ -76,20 +112,25 @@ class PlannerPlanGetCommand extends GraphCommand_1.default {
         return options.concat(parentOptions);
     }
     validate(args) {
-        if (!args.options.id && !args.options.title) {
-            return 'Specify either id or title';
-        }
-        if (args.options.id && args.options.title) {
-            return 'Specify either id or title';
-        }
-        if (args.options.title && !args.options.ownerGroupId && !args.options.ownerGroupName) {
-            return 'Specify either ownerGroupId or ownerGroupName';
-        }
-        if (args.options.title && args.options.ownerGroupId && args.options.ownerGroupName) {
-            return 'Specify either ownerGroupId or ownerGroupName but not both';
+        if (args.options.planId && args.options.planTitle ||
+            args.options.id && args.options.title ||
+            args.options.planId && args.options.title ||
+            args.options.id && args.options.planTitle) {
+            return 'Specify either id or title but not both';
         }
-        if (args.options.ownerGroupId && !utils_1.validation.isValidGuid(args.options.ownerGroupId)) {
-            return `${args.options.ownerGroupId} is not a valid GUID`;
+        if (!args.options.planId && !args.options.id) {
+            if (!args.options.planTitle && !args.options.title) {
+                return 'Specify either id or title';
+            }
+            if ((args.options.title || args.options.planTitle) && !args.options.ownerGroupId && !args.options.ownerGroupName) {
+                return 'Specify either ownerGroupId or ownerGroupName';
+            }
+            if ((args.options.title || args.options.planTitle) && args.options.ownerGroupId && args.options.ownerGroupName) {
+                return 'Specify either ownerGroupId or ownerGroupName 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;
     }

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

@@ -0,0 +1,141 @@
+"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());
+    });
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const Auth_1 = require("../../../../Auth");
+const cli_1 = require("../../../../cli");
+const request_1 = require("../../../../request");
+const utils_1 = require("../../../../utils");
+const aadGroup_1 = require("../../../../utils/aadGroup");
+const planner_1 = require("../../../../utils/planner");
+const GraphCommand_1 = require("../../../base/GraphCommand");
+const commands_1 = require("../../commands");
+class PlannerPlanRemoveCommand extends GraphCommand_1.default {
+    get name() {
+        return commands_1.default.PLAN_REMOVE;
+    }
+    get description() {
+        return 'Removes the Microsoft Planner plan';
+    }
+    getTelemetryProperties(args) {
+        const telemetryProps = super.getTelemetryProperties(args);
+        telemetryProps.id = typeof args.options.id !== 'undefined';
+        telemetryProps.title = typeof args.options.title !== 'undefined';
+        telemetryProps.ownerGroupId = typeof args.options.ownerGroupId !== 'undefined';
+        telemetryProps.ownerGroupName = typeof args.options.ownerGroupName !== 'undefined';
+        telemetryProps.confirm = !!args.options.confirm;
+        return telemetryProps;
+    }
+    commandAction(logger, args, cb) {
+        if (utils_1.accessToken.isAppOnlyAccessToken(Auth_1.default.service.accessTokens[this.resource].accessToken)) {
+            this.handleError('This command does not support application permissions.', logger, cb);
+            return;
+        }
+        const removePlan = () => __awaiter(this, void 0, void 0, function* () {
+            try {
+                const plan = yield this.getPlan(args);
+                const requestOptions = {
+                    url: `${this.resource}/v1.0/planner/plans/${plan.id}`,
+                    headers: {
+                        accept: 'application/json;odata.metadata=none',
+                        'if-match': plan['@odata.etag']
+                    },
+                    responseType: 'json'
+                };
+                yield request_1.default.delete(requestOptions);
+                cb();
+            }
+            catch (err) {
+                this.handleRejectedODataJsonPromise(err, logger, cb);
+            }
+        });
+        if (args.options.confirm) {
+            removePlan();
+        }
+        else {
+            cli_1.Cli.prompt({
+                type: 'confirm',
+                name: 'continue',
+                default: false,
+                message: `Are you sure you want to remove the plan ${args.options.id || args.options.title}?`
+            }, (result) => {
+                if (!result.continue) {
+                    cb();
+                }
+                else {
+                    removePlan();
+                }
+            });
+        }
+    }
+    getPlan(args) {
+        return __awaiter(this, void 0, void 0, function* () {
+            const { id, title } = args.options;
+            if (id) {
+                return planner_1.planner.getPlanById(id, 'minimal');
+            }
+            const groupId = yield this.getGroupId(args);
+            return yield planner_1.planner.getPlanByTitle(title, groupId);
+        });
+    }
+    getGroupId(args) {
+        return __awaiter(this, void 0, void 0, function* () {
+            const { ownerGroupId, ownerGroupName } = args.options;
+            if (ownerGroupId) {
+                return ownerGroupId;
+            }
+            const group = yield aadGroup_1.aadGroup.getGroupByDisplayName(ownerGroupName);
+            return group.id;
+        });
+    }
+    optionSets() {
+        return [['id', 'title']];
+    }
+    options() {
+        const options = [
+            {
+                option: '-i, --id [id]'
+            },
+            {
+                option: '-t, --title [title]'
+            },
+            {
+                option: '--ownerGroupId [ownerGroupId]'
+            },
+            {
+                option: '--ownerGroupName [ownerGroupName]'
+            },
+            {
+                option: '--confirm'
+            }
+        ];
+        const parentOptions = super.options();
+        return options.concat(parentOptions);
+    }
+    validate(args) {
+        if (args.options.title) {
+            if (!args.options.ownerGroupId && !args.options.ownerGroupName) {
+                return 'Specify either ownerGroupId or ownerGroupName';
+            }
+            if (args.options.ownerGroupId && args.options.ownerGroupName) {
+                return 'Specify either ownerGroupId or ownerGroupName but not both';
+            }
+            if (args.options.ownerGroupId && !utils_1.validation.isValidGuid(args.options.ownerGroupId)) {
+                return `${args.options.ownerGroupId} is not a valid GUID`;
+            }
+        }
+        else if (args.options.ownerGroupId || args.options.ownerGroupName) {
+            return 'Don\'t specify ownerGroupId or ownerGroupName when using id';
+        }
+        return true;
+    }
+}
+module.exports = new PlannerPlanRemoveCommand();
+//# sourceMappingURL=plan-remove.js.map

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

@@ -24,6 +24,7 @@ class PlannerTaskAddCommand extends GraphCommand_1.default {
         const telemetryProps = super.getTelemetryProperties(args);
         telemetryProps.planId = typeof args.options.planId !== 'undefined';
         telemetryProps.planName = typeof args.options.planName !== 'undefined';
+        telemetryProps.planTitle = typeof args.options.planTitle !== 'undefined';
         telemetryProps.ownerGroupId = typeof args.options.ownerGroupId !== 'undefined';
         telemetryProps.ownerGroupName = typeof args.options.ownerGroupName !== 'undefined';
         telemetryProps.bucketId = typeof args.options.bucketId !== 'undefined';
@@ -42,6 +43,10 @@ class PlannerTaskAddCommand extends GraphCommand_1.default {
         return telemetryProps;
     }
     commandAction(logger, args, cb) {
+        if (args.options.planName) {
+            args.options.planTitle = args.options.planName;
+            this.warn(logger, `Option 'planName' is deprecated. Please use 'planTitle' instead`);
+        }
         if (utils_1.accessToken.isAppOnlyAccessToken(Auth_1.default.service.accessTokens[this.resource].accessToken)) {
             this.handleError('This command does not support application permissions.', logger, cb);
             return;
@@ -96,13 +101,7 @@ class PlannerTaskAddCommand extends GraphCommand_1.default {
         };
         return request_1.default
             .get(requestOptions)
-            .then((response) => {
-            const etag = response ? response['@odata.etag'] : undefined;
-            if (!etag) {
-                return Promise.reject(`Error fetching task details`);
-            }
-            return Promise.resolve(etag);
-        });
+            .then((response) => response['@odata.etag']);
     }
     generateAppliedCategories(options) {
         if (!options.appliedCategories) {
@@ -181,7 +180,7 @@ class PlannerTaskAddCommand extends GraphCommand_1.default {
         }
         return this
             .getGroupId(args)
-            .then((groupId) => planner_1.planner.getPlanByName(args.options.planName, groupId))
+            .then((groupId) => planner_1.planner.getPlanByTitle(args.options.planTitle, groupId))
             .then(plan => plan.id);
     }
     getGroupId(args) {
@@ -229,6 +228,7 @@ class PlannerTaskAddCommand extends GraphCommand_1.default {
             { option: '-t, --title <title>' },
             { option: '--planId [planId]' },
             { option: '--planName [planName]' },
+            { option: '--planTitle [planTitle]' },
             { option: '--ownerGroupId [ownerGroupId]' },
             { option: '--ownerGroupName [ownerGroupName]' },
             { option: '--bucketId [bucketId]' },
@@ -255,17 +255,17 @@ class PlannerTaskAddCommand extends GraphCommand_1.default {
         return options.concat(parentOptions);
     }
     validate(args) {
-        if (!args.options.planId && !args.options.planName) {
-            return 'Specify either planId or planName';
+        if (!args.options.planId && !args.options.planName && !args.options.planTitle) {
+            return 'Specify either planId or planTitle';
         }
-        if (args.options.planId && args.options.planName) {
-            return 'Specify either planId or planName but not both';
+        if (args.options.planId && (args.options.planName || args.options.planTitle)) {
+            return 'Specify either planId or planTitle but not both';
         }
-        if (args.options.planName && !args.options.ownerGroupId && !args.options.ownerGroupName) {
-            return 'Specify either ownerGroupId or ownerGroupName when using planName';
+        if ((args.options.planName || args.options.planTitle) && !args.options.ownerGroupId && !args.options.ownerGroupName) {
+            return 'Specify either ownerGroupId or ownerGroupName when using planTitle';
         }
-        if (args.options.planName && args.options.ownerGroupId && args.options.ownerGroupName) {
-            return 'Specify either ownerGroupId or ownerGroupName when using planName but not both';
+        if ((args.options.planName || args.options.planTitle) && 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`;