Skip to main content

Search and Top Navigation

#7741 closed bug (fixed)

Opened September 27, 2011 07:04AM UTC

Closed September 27, 2011 11:57AM UTC

Last modified September 27, 2011 12:01PM UTC

ui.sortable is not working when it is extended

Reported by: TomWolk Owned by:
Priority: minor Milestone: 1.8.17
Component: ui.sortable Version: 1.8.15
Keywords: Cc:
Blocked by: Blocking:
Description

I extended the ui.sortable widget to add some additional functionality. I gave the new widget a new name so I can still use the original sortable.

But with a new name 2 methods stopped working because the name of the widget was hard coded in those methods.

I changed the line

var inst = $.data(cur[j], 'sortable');

to

var inst = $.data(cur[j], this.widgetName);

in the methods

_getItemsAsjQuery
and
_refreshItems

and now it works!

Greetings Thomas

Attachments (0)
Change History (3)

Changed September 27, 2011 11:57AM UTC by Richard D. Worth comment:1

resolution: → fixed
status: newclosed

Sortable: replaced hard-coded sortable with this.widgetName, and removed lines from destroy that are handled by super. Fixes #7741 - ui.sortable is not working when it is extended

Changeset: 228b1b191b4f668a74e7005fc9c8c9250c090aa9

Changed September 27, 2011 12:00PM UTC by Richard D. Worth comment:2

Sortable: replaced hard-coded sortable with this.widgetName, and removed lines from destroy that are handled by super. Fixes #7741 - ui.sortable is not working when it is extended

(cherry picked from commit 228b1b191b4f668a74e7005fc9c8c9250c090aa9)

Changeset: 5c6cfe2645588e6a3d3f1ed002534d454209e33f

Changed September 27, 2011 12:01PM UTC by rdworth comment:3

milestone: 1.91.8.17