Fixed the null issue while parsing the ARM deployment output (#14334) (#14363)

ddivanshu · pradudev · Pradeep Chouhan · web-flow · commit 1d46bc5f7118 · 2021-02-09T14:59:40.000+05:30
* Fixed the null issue while parsing the deployment output

* updating version

Co-authored-by: Pradeep Chouhan &lt;pradeep.chouhan@bupa.com.au&gt;
Co-authored-by: ddivanshu &lt;36132271+ddivanshu@users.noreply.github.com&gt;
Co-authored-by: Divanshu Aggarwal &lt;divanshuaggarwal@Divanshus-MacBook-Pro.local&gt;

Co-authored-by: Pradeep Chouhan &lt;pradeepchouhan1986@gmail.com&gt;
Co-authored-by: Pradeep Chouhan &lt;pradeep.chouhan@bupa.com.au&gt;
Co-authored-by: Divanshu Aggarwal &lt;divanshuaggarwal@Divanshus-MacBook-Pro.local&gt;
diff --git a/Tasks/AzureResourceGroupDeploymentV2/operations/ResourceGroup.ts b/Tasks/AzureResourceGroupDeploymentV2/operations/ResourceGroup.ts
@@ -559,7 +559,7 @@ export class ResourceGroup {
                     if (result && result["properties"] && result["properties"]["outputs"] && utils.isNonEmpty(this.taskParameters.deploymentOutputs)) {
                         const setVariablesInObject = (path: string, obj: any) => {
                             for (var key of Object.keys(obj)) {
-                                if (typeof(obj[key]) === "object") {
+                                if (obj[key] && typeof(obj[key]) === "object") {
                                     setVariablesInObject(`${path}.${key}`, obj[key]);
                                 }
                                 else {
diff --git a/Tasks/AzureResourceGroupDeploymentV2/task.json b/Tasks/AzureResourceGroupDeploymentV2/task.json
@@ -14,7 +14,7 @@
     "author": "Microsoft Corporation",
     "version": {
         "Major": 2,
-        "Minor": 181,
+        "Minor": 182,
         "Patch": 0
     },
     "demands": [],
diff --git a/Tasks/AzureResourceGroupDeploymentV2/task.loc.json b/Tasks/AzureResourceGroupDeploymentV2/task.loc.json
@@ -14,7 +14,7 @@
   "author": "Microsoft Corporation",
   "version": {
     "Major": 2,
-    "Minor": 181,
+    "Minor": 182,
     "Patch": 0
   },
   "demands": [],
diff --git a/Tasks/AzureResourceManagerTemplateDeploymentV3/operations/DeploymentScopeBase.ts b/Tasks/AzureResourceManagerTemplateDeploymentV3/operations/DeploymentScopeBase.ts
@@ -73,7 +73,7 @@ export class DeploymentScopeBase {
                     if (result && result["properties"] && result["properties"]["outputs"] && utils.isNonEmpty(this.taskParameters.deploymentOutputs)) {
                         const setVariablesInObject = (path: string, obj: any) => {
                             for (var key of Object.keys(obj)) {
-                                if (typeof(obj[key]) === "object") {
+                                if (obj[key] && typeof(obj[key]) === "object") {
                                     setVariablesInObject(`${path}.${key}`, obj[key]);
                                 }
                                 else {
diff --git a/Tasks/AzureResourceManagerTemplateDeploymentV3/task.json b/Tasks/AzureResourceManagerTemplateDeploymentV3/task.json
@@ -14,7 +14,7 @@
     "author": "Microsoft Corporation",
     "version": {
         "Major": 3,
-        "Minor": 181,
+        "Minor": 182,
         "Patch": 0
     },
     "demands": [],
diff --git a/Tasks/AzureResourceManagerTemplateDeploymentV3/task.loc.json b/Tasks/AzureResourceManagerTemplateDeploymentV3/task.loc.json
@@ -14,7 +14,7 @@
   "author": "Microsoft Corporation",
   "version": {
     "Major": 3,
-    "Minor": 181,
+    "Minor": 182,
     "Patch": 0
   },
   "demands": [],

Original file line number	Diff line number	Diff line change
`@@ -559,7 +559,7 @@ export class ResourceGroup {`
`559`	`559`	`if (result && result["properties"] && result["properties"]["outputs"] && utils.isNonEmpty(this.taskParameters.deploymentOutputs)) {`
`560`	`560`	`const setVariablesInObject = (path: string, obj: any) => {`
`561`	`561`	`for (var key of Object.keys(obj)) {`
`562`		`- if (typeof(obj[key]) === "object") {`
	`562`	`+ if (obj[key] && typeof(obj[key]) === "object") {`
`563`	`563`	setVariablesInObject(`${path}.${key}`, obj[key]);
`564`	`564`	`}`
`565`	`565`	`else {`
Original file line number	Diff line number	Diff line change
`@@ -73,7 +73,7 @@ export class DeploymentScopeBase {`
`73`	`73`	`if (result && result["properties"] && result["properties"]["outputs"] && utils.isNonEmpty(this.taskParameters.deploymentOutputs)) {`
`74`	`74`	`const setVariablesInObject = (path: string, obj: any) => {`
`75`	`75`	`for (var key of Object.keys(obj)) {`
`76`		`- if (typeof(obj[key]) === "object") {`
	`76`	`+ if (obj[key] && typeof(obj[key]) === "object") {`
`77`	`77`	setVariablesInObject(`${path}.${key}`, obj[key]);
`78`	`78`	`}`
`79`	`79`	`else {`