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

markelog · markelog · commit fd80f5970f38 · 2015-02-15T02:06:34.000+03:00
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
diff --git a/src/ajax/load.js b/src/ajax/load.js
@@ -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 ] );
 		});
 	}
