Skip to main content

Search and Top Navigation

#5581 closed bug (fixed)

Opened May 05, 2010 02:55AM UTC

Closed May 05, 2010 01:05PM UTC

Last modified May 20, 2010 11:14AM UTC

_setOption being called with 'self' rather than 'this'

Reported by: Owned by:
Priority: blocker Milestone: 1.8.2
Component: ui.sortable Version: 1.8.1
Keywords: Cc:
Blocked by: Blocking:

In the _setOption method the following line appears:

$.Widget.prototype._setOption.apply(self, arguments);

(line 3049 in the current full source code library)

Under some complex conditions in our code, self points to the window, not the sortable object. Changing 'self' to 'this' makes everything good.

Note: I haven't spent enough time in this code to be that this change is valid. If it's really required, maybe I can produce a test case that shows the problem, but at this point, the code is intertwined with a jquery.linkselect

Attachments (0)
Change History (3)

Changed May 05, 2010 11:54AM UTC by rdworth comment:1

milestone: TBD1.9
priority: majorblocker

Changed May 05, 2010 01:05PM UTC by rdworth comment:2

resolution: → fixed
status: newclosed

Fixed in a4488af

Changed May 20, 2010 11:14AM UTC by rdworth comment:3