Caught exception so grouped SVG elements without a pin ID would not t… (#96)

* Caught exception so grouped SVG elements without a pin ID would not terminate the processing of a skin file.

* Removed unnecessary semicolon for nop statement.

* Indented variable declarations.

Co-authored-by: xesscorp <[email protected]>

Author: xesscorp

built/Skin.js

@@ -8,8 +8,13 @@ var Skin;
     Skin.skin = null;
     function getPortsWithPrefix(template, prefix) {
         var ports = _.filter(template, function (e) {
-            if (e instanceof Array && e[0] === 'g') {
-                return e[1]['s:pid'].startsWith(prefix);
+            try {
+                if (e instanceof Array && e[0] === 'g') {
+                    return e[1]['s:pid'].startsWith(prefix);
+                }
+            }
+            catch (exception) {
+                // Do nothing if the SVG group doesn't have a pin id.
             }
         });
         return ports;

lib/Skin.ts

@@ -1,3 +1,4 @@
+
 import onml = require('onml');
 import _ = require('lodash');
 import { ElkModel } from './elkGraph';
@@ -8,8 +9,12 @@ export namespace Skin {
 
     export function getPortsWithPrefix(template: any[], prefix: string) {
         const ports = _.filter(template, (e) => {
-            if (e instanceof Array && e[0] === 'g') {
-                return e[1]['s:pid'].startsWith(prefix);
+            try {
+                if (e instanceof Array && e[0] === 'g') {
+                    return e[1]['s:pid'].startsWith(prefix);
+                }
+            } catch (exception) {
+                // Do nothing if the SVG group doesn't have a pin id.
             }
         });
         return ports;

tsconfig.json

@@ -2,7 +2,7 @@
     "compilerOptions": {
         "outDir": "./built",
         "allowJs": true,
-        "lib": [ "es6" ],
+        "lib": [ "es6", "dom" ],
         "target": "es5"
     },
     "include": [