Skip to content
Permalink
Browse files

Draggable: Don't run stop methods for elements that have been removed…

…. Fixed #8269 - Removing draggable element on drop : a(this).data("draggable") is undefined.
  • Loading branch information...
tjvantoll authored and scottgonzalez committed Apr 30, 2012
1 parent 4ab7d53 commit 27d10235539e0f5baad6ee7e8d3d91cab65e2cfd
Showing with 8 additions and 2 deletions.
  1. +8 −2 ui/jquery.ui.draggable.js
@@ -207,8 +207,14 @@ $.widget("ui.draggable", $.ui.mouse, {
this.dropped = false;
}

//if the original element is removed, don't bother to continue
if((!this.element[0] || !this.element[0].parentNode) && this.options.helper === "original")
//if the original element is no longer in the DOM don't bother to continue (see #8269)
var element = this.element[0], elementInDom = false;
while ( element && (element = element.parentNode) ) {
if (element == document ) {
elementInDom = true;
}
}
if ( !elementInDom && this.options.helper === "original" )
return false;

if((this.options.revert == "invalid" && !dropped) || (this.options.revert == "valid" && dropped) || this.options.revert === true || ($.isFunction(this.options.revert) && this.options.revert.call(this.element, dropped))) {

0 comments on commit 27d1023

Please sign in to comment.
You can’t perform that action at this time.