#10723 closed bug (fixed)
jqXHR.always() returns a Promise instead of a jqXHR object
Reported by: | madblueimp | Owned by: | jaubourg |
---|---|---|---|
Priority: | blocker | Milestone: | 1.7.1 |
Component: | ajax | Version: | 1.7 |
Keywords: | Cc: | jaubourg | |
Blocked by: | Blocking: |
Description
Test case:
$.get().always($.noop).abort();
Expected behavior: Aborts the GET request.
Observed behavior:
TypeError: Object #<Object> has no method 'abort'
Change History (6)
comment:1 Changed 12 years ago by
Cc: | jaubourg added |
---|---|
Component: | unfiled → ajax |
Keywords: | needsreview added |
Priority: | undecided → low |
Status: | new → open |
comment:3 Changed 12 years ago by
Keywords: | needsreview removed |
---|---|
Milestone: | None → 1.7.1 |
Owner: | set to jaubourg |
Priority: | low → blocker |
Status: | open → assigned |
comment:4 Changed 12 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Have Deferred.always return the object onto which it is currently attached to enable true chainability. Fixes #10723. Unit tests added.
Changeset: 2a9f0681dee3a6421bafd2fbc4451c9fe364b6ca
comment:6 Changed 11 years ago by
- Changed from Review to Test Case rio orange portabilite du numero imc
- Status changed from reopened to closed comparateur forfait forfait sans engagement forfait illimite sms illimite forfait internet mobile forfait bloque forfait blackberry
Note: See
TracTickets for help on using
tickets.
.always() returns the deferred without the methods added using deferred.promise( jqXHR ) internally. I 'defer' to jaubourg here. It may be worth defer.promise extending the actual deferred rather than creating a new object so that the jqXHR can be chained.