Opened 14 years ago
Closed 14 years ago
#4282 closed bug (fixed)
Droppable changes options object
Reported by: | d.wachss | Owned by: | |
---|---|---|---|
Priority: | critical | Milestone: | 1.8 |
Component: | ui.droppable | Version: | 1.6rc6 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
based on this discussion http://groups.google.com/group/jquery-ui-dev/browse_thread/thread/abf6a10a8af2cac7
Droppable shouldn't modify the accept option, but should add a key to the widget object itself. Change line 23 from
this.options.accept = accept && $.isFunction(accept) ? accept : function(d) {
to
this.accept = $.isFunction(accept) ? accept : function(d) {
and line 53 from
this.options.accept = value && $.isFunction(value) ? value : function(d) {
to
this.accept = $.isFunction(value) ? value : function(d) {
(I also removed the unnecessary guard expressions) and change all the this.options.accept.call expressions to this.accept.call
Attachments (1)
Change History (3)
comment:1 Changed 14 years ago by
Milestone: | TBD → 1.8 |
---|---|
Priority: | minor → critical |
Changed 14 years ago by
Attachment: | ui.droppable.js.patch added |
---|
comment:2 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Applied in r2517, fixes 14 tests. Thanks for the contribution!
Note: See
TracTickets for help on using
tickets.
complete tested fix starting from the ticket description