Permalink
Browse files

Ajax: replace "jqXHR.complete" callback with "always"

Since it was deprecated since 1.8.

Also add additional comments which explains tricky
behaviour of "always" callback

(cherry-picked from 97ef1f2)
Closes gh-2033
  • Loading branch information...
markelog committed Feb 11, 2015
1 parent 1d3d2b1 commit fd80f5970f38ff12c7fa2caa8c1de955301a6979
Showing with 4 additions and 1 deletion.
  1. +4 −1 src/ajax/load.js
View
@@ -66,7 +66,10 @@ jQuery.fn.load = function( url, params, callback ) {
// Otherwise use the full result
responseText );
- }).complete( callback && function( jqXHR, status ) {
+ // If the request succeeds, this function gets "data", "status", "jqXHR"
+ // but they are ignored because response was set above.
+ // If it fails, this function gets "jqXHR", "status", "error"
+ }).always( callback && function( jqXHR, status ) {
self.each( callback, response || [ jqXHR.responseText, status, jqXHR ] );
});
}

0 comments on commit fd80f59

Please sign in to comment.