From: Paul Bakaus <paul.bakaus@googlemail.com>
Date: Tue, 10 Feb 2009 10:07:25 +0000 (+0000)
Subject: sortable: if current item was removed by user, it was reappended by the clearing... 
X-Git-Tag: 1.6~10
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=2e0f967680447e93e04eea2764f53ea3977b4f5f;p=jquery-ui.git

sortable: if current item was removed by user, it was reappended by the clearing logic (fixes #4088)
---

diff --git a/ui/ui.sortable.js b/ui/ui.sortable.js
index 56c613d04..e3a3f3d3d 100644
--- a/ui/ui.sortable.js
+++ b/ui/ui.sortable.js
@@ -901,8 +901,9 @@ $.widget("ui.sortable", $.extend({}, $.ui.mouse, {
 		// everything else normalized again
 		var delayedTriggers = [], self = this;
 
-		//We first have to update the dom position of the actual currentItem
-		if(!this._noFinalSort) this.placeholder.before(this.currentItem);
+		// We first have to update the dom position of the actual currentItem
+		// Note: don't do it if the current item is already removed (by a user), or it gets reappended (see #4088)
+		if(!this._noFinalSort && this.currentItem[0].parentNode) this.placeholder.before(this.currentItem);
 		this._noFinalSort = null;
 
 		if(this.helper[0] == this.currentItem[0]) {