Merge pull request #495 from paulkaplan/fix-target-id

paulkaplan · web-flow · commit 2ebb112d3048 · 2017-03-08T12:13:10.000-05:00
Add check to make sure `target` exists
diff --git a/src/virtual-machine.js b/src/virtual-machine.js
@@ -336,7 +336,7 @@ VirtualMachine.prototype.emitWorkspaceUpdate = function () {
  */
 VirtualMachine.prototype.getTargetIdForDrawableId = function (drawableId) {
     var target = this.runtime.getTargetByDrawableId(drawableId);
-    if (target.hasOwnProperty('id') && target.hasOwnProperty('isStage') && !target.isStage) {
+    if (target && target.hasOwnProperty('id') && target.hasOwnProperty('isStage') && !target.isStage) {
         return target.id;
     }
     return null;

Original file line number	Diff line number	Diff line change
`@@ -336,7 +336,7 @@ VirtualMachine.prototype.emitWorkspaceUpdate = function () {`
`336`	`336`	`*/`
`337`	`337`	`VirtualMachine.prototype.getTargetIdForDrawableId = function (drawableId) {`
`338`	`338`	`var target = this.runtime.getTargetByDrawableId(drawableId);`
`339`		`- if (target.hasOwnProperty('id') && target.hasOwnProperty('isStage') && !target.isStage) {`
	`339`	`+ if (target && target.hasOwnProperty('id') && target.hasOwnProperty('isStage') && !target.isStage) {`
`340`	`340`	`return target.id;`
`341`	`341`	`}`
`342`	`342`	`return null;`