fix: null check

ljacobsson · ljacobsson · commit a2d8372b420e · 2020-12-23T15:09:08.000+01:00
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@mhlabs/cfn-diagram",
-  "version": "1.1.11",
+  "version": "1.1.12",
   "description": "CLI tool to visualise CloudFormation templates as draw.io or HTML diagrams",
   "main": "index.js",
   "scripts": {
diff --git a/resources/JsonUtil.js b/resources/JsonUtil.js
@@ -1,7 +1,7 @@
 function createPseudoResources(template, current) {
   current = current || template.Resources;
   for (var k in current) {
-    if (current[k]["Fn::Join"]) {
+    if (current[k] && current[k]["Fn::Join"]) {
       const joinList = current[k]["Fn::Join"][1]    
       current[k] = (Array.isArray(joinList) ? joinList : [joinList]).join(current[k]["Fn::Join"][0]);
     }

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "@mhlabs/cfn-diagram",`
`3`		`- "version": "1.1.11",`
	`3`	`+ "version": "1.1.12",`
`4`	`4`	`"description": "CLI tool to visualise CloudFormation templates as draw.io or HTML diagrams",`
`5`	`5`	`"main": "index.js",`
`6`	`6`	`"scripts": {`
Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,7 @@`
`1`	`1`	`function createPseudoResources(template, current) {`
`2`	`2`	`current = current \|\| template.Resources;`
`3`	`3`	`for (var k in current) {`
`4`		`- if (current[k]["Fn::Join"]) {`
	`4`	`+ if (current[k] && current[k]["Fn::Join"]) {`
`5`	`5`	`const joinList = current[k]["Fn::Join"][1]`
`6`	`6`	`current[k] = (Array.isArray(joinList) ? joinList : [joinList]).join(current[k]["Fn::Join"][0]);`
`7`	`7`	`}`