Opened 4 years ago
Closed 4 years ago
#12521 closed bug (fixed)
Deferred.promise( target) only works fine when typeof( target)=='object'
| Reported by: | lars.gersmann@… | Owned by: | jaubourg |
|---|---|---|---|
| Priority: | blocker | Milestone: | 1.8.2 |
| Component: | deferred | Version: | 1.8.1 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description
Deferred.promise( target) should work fine for any type of target except null and undefined.
Before version 1.8.0 (i used version 1.7.2) you could do
var tracker = $.Deferred(); var foo = function() { }; foo = tracker.promise( foo); foo();
Since version 1.8.0 the same code returns just the promise instead of the extend function.
Should be easy to fix.
Change History (2)
comment:1 Changed 4 years ago by jaubourg
- Component changed from unfiled to deferred
- Milestone changed from None to 1.8.2
- Owner set to jaubourg
- Priority changed from undecided to blocker
- Status changed from new to assigned
comment:2 Changed 4 years ago by jaubourg
- Resolution set to fixed
- Status changed from assigned to closed
deferred.promise(obj) should work with non-objects. Fixes #12521. Much needed unit tests added!
Changeset: 74cdd784975ba6e628d1934a859faeb017824a5f
Note: See
TracTickets for help on using
tickets.
Silly, silly regression. Thanks for catching this one.