sortable: only remove placeholder if necessary

opsb · opsb · commit cd5a4faed24a · 2014-03-21T12:59:09.000-06:00
diff --git a/ui/sortable.js b/ui/sortable.js
@@ -1267,7 +1267,8 @@ return $.widget("ui.sortable", $.ui.mouse, {
 		}
 
 		//$(this.placeholder[0]).remove(); would have been the jQuery way - unfortunately, it unbinds ALL events from the original node!
-		this.placeholder[0].parentNode.removeChild(this.placeholder[0]);
+		var parentNode = this.placeholder[0].parentNode;
+		if(parentNode) parentNode.removeChild(this.placeholder[0]);
 
 		if(this.helper[0] !== this.currentItem[0]) {
 			this.helper.remove();

Original file line number	Diff line number	Diff line change
`@@ -1267,7 +1267,8 @@ return $.widget("ui.sortable", $.ui.mouse, {`
`1267`	`1267`	`}`
`1268`	`1268`
`1269`	`1269`	`//$(this.placeholder[0]).remove(); would have been the jQuery way - unfortunately, it unbinds ALL events from the original node!`
`1270`		`- this.placeholder[0].parentNode.removeChild(this.placeholder[0]);`
	`1270`	`+ var parentNode = this.placeholder[0].parentNode;`
	`1271`	`+ if(parentNode) parentNode.removeChild(this.placeholder[0]);`
`1271`	`1272`
`1272`	`1273`	`if(this.helper[0] !== this.currentItem[0]) {`
`1273`	`1274`	`this.helper.remove();`