Commit e7fdda98 authored by nanto_vi's avatar nanto_vi

Propagete context of returned deferred object in Deferred.then(). Fixes #13160

parent c6115042
...@@ -35,7 +35,9 @@ jQuery.extend({ ...@@ -35,7 +35,9 @@ jQuery.extend({
newDefer[ action + "With" ]( this === promise ? newDefer.promise() : this, [ returned ] ); newDefer[ action + "With" ]( this === promise ? newDefer.promise() : this, [ returned ] );
} }
} : } :
newDefer[ action ] function() {
newDefer[ action + "With" ]( this === promise ? newDefer.promise() : this, arguments );
}
); );
}); });
fns = null; fns = null;
...@@ -72,7 +74,7 @@ jQuery.extend({ ...@@ -72,7 +74,7 @@ jQuery.extend({
// deferred[ resolve | reject | notify ] // deferred[ resolve | reject | notify ]
deferred[ tuple[0] ] = function() { deferred[ tuple[0] ] = function() {
deferred[ tuple[0] + "With" ]( promise, arguments ); deferred[ tuple[0] + "With" ]( this === deferred ? promise : this, arguments );
return this; return this;
}; };
deferred[ tuple[0] + "With" ] = list.fireWith; deferred[ tuple[0] + "With" ] = list.fireWith;
......
...@@ -273,7 +273,7 @@ test( "jQuery.Deferred.then - deferred (progress)", function() { ...@@ -273,7 +273,7 @@ test( "jQuery.Deferred.then - deferred (progress)", function() {
test( "jQuery.Deferred.then - context", function() { test( "jQuery.Deferred.then - context", function() {
expect( 4 ); expect( 7 );
var context = {}; var context = {};
...@@ -284,6 +284,12 @@ test( "jQuery.Deferred.then - context", function() { ...@@ -284,6 +284,12 @@ test( "jQuery.Deferred.then - context", function() {
strictEqual( value, 6, "proper value received" ); strictEqual( value, 6, "proper value received" );
}); });
jQuery.Deferred().resolve().then(function() {
return jQuery.Deferred().resolveWith(context);
}).done(function() {
strictEqual( this, context, "custom context of returned deferred correctly propagated" );
});
var defer = jQuery.Deferred(), var defer = jQuery.Deferred(),
piped = defer.then(function( value ) { piped = defer.then(function( value ) {
return value * 3; return value * 3;
...@@ -295,6 +301,16 @@ test( "jQuery.Deferred.then - context", function() { ...@@ -295,6 +301,16 @@ test( "jQuery.Deferred.then - context", function() {
strictEqual( this, piped, "default context gets updated to latest promise in the chain" ); strictEqual( this, piped, "default context gets updated to latest promise in the chain" );
strictEqual( value, 6, "proper value received" ); strictEqual( value, 6, "proper value received" );
}); });
var defer2 = jQuery.Deferred(),
piped2 = defer2.then();
defer2.resolve( 2 );
piped2.done(function( value ) {
strictEqual( this, piped2, "default context gets updated to latest promise in the chain (without passing function)" );
strictEqual( value, 2, "proper value received (without passing function)" );
});
}); });
test( "jQuery.when", function() { test( "jQuery.when", function() {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment