#4087 closed bug (fixed)

Opened February 07, 2009 03:36PM UTC

Closed February 10, 2009 09:15AM UTC

Removing ui.draggable immediately after the drop callback raises an error.

Reported by: bohdan.ganicky Owned by:
Priority: critical Milestone: 1.7
Component: ui.draggable Version: 1.6rc6
Keywords: Cc:
Blocked by: Blocking:
Description

The code is simple:

$("#draggable").draggable();
$("#droppable").droppable({
	drop: function(event, ui) {
		ui.draggable.remove();
	}
});

Firebug output:

$(this).data("draggable") is undefined
http://localhost/Dev/jquery-ui/ui/ui.draggable.js
Line 570

testcase here:

http://jquery-ui.googlecode.com/svn/trunk/tests/visual/droppable/droppable_ticket_4087.html

Attachments (0)
Change History (4)

Changed February 07, 2009 04:19PM UTC by bohdan.ganicky comment:1

component: ui.droppable → ui.draggable

Changed February 07, 2009 05:48PM UTC by rdworth comment:2

description: The code is simple: \ {{{ \ $("#draggable").draggable(); \ $("#droppable").droppable({ \ drop: function(event, ui) { \ ui.draggable.remove(); \ } \ }); \ }}} \ \ Firebug output: \ {{{ \ $(this).data("draggable") is undefined \ http://localhost/Dev/jquery-ui/ui/ui.draggable.js \ Line 570 \ }}} \ \ testcase here: \ http://jquery-ui.googlecode.com/svn/trunk/demos/droppable/remove-draggable-after-drop.html → The code is simple: \ {{{ \ $("#draggable").draggable(); \ $("#droppable").droppable({ \ drop: function(event, ui) { \ ui.draggable.remove(); \ } \ }); \ }}} \ \ Firebug output: \ {{{ \ $(this).data("draggable") is undefined \ http://localhost/Dev/jquery-ui/ui/ui.draggable.js \ Line 570 \ }}} \ \ testcase here: \ http://jquery-ui.googlecode.com/svn/trunk/tests/visual/droppable/droppable_ticket_4087.html

Changed February 09, 2009 09:32AM UTC by sukerman comment:3

I'm seeing the same thing, but also seems jammed in a loop my firefox error console instantly fills to capacity with this error repeating and CPU usage goes to 50% hanging firefox. I can provide test case but my case is quite complex to setup and I guess its the same problem.

Changed February 10, 2009 09:15AM UTC by paul comment:4

resolution: → fixed
status: new → closed

Fixed in r2061.