r2062 - trunk/ui

r2062 - trunk/ui


Author: paul.bakaus
Date: Tue Feb 10 02:07:25 2009
New Revision: 2062
Modified:
trunk/ui/ui.sortable.js
Log:
sortable: if current item was removed by user, it was reappended by the
clearing logic (fixes #4088)
Modified: trunk/ui/ui.sortable.js
==============================================================================
--- trunk/ui/ui.sortable.js    (original)
+++ trunk/ui/ui.sortable.js    Tue Feb 10 02:07:25 2009
@@ -901,8 +901,9 @@
        // 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]) {