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
_getItemsAsjQueryand
_refreshItems
and now it works!
Greetings Thomas
Attachments (0)
Change History (3)
Changed September 27, 2011 11:57AM UTC by comment:1
| resolution: | → fixed |
|---|---|
| status: | new → closed |
Changed September 27, 2011 12:00PM UTC by 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 comment:3
| milestone: | 1.9 → 1.8.17 |
|---|
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