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

Closes gh-2033

diff --git a/src/ajax/load.js b/src/ajax/load.js
index 5950614437f24b856987dd9a995781d25d41a85a..f8f0c396a8c4c9436c0eb82741c0d7cac9986704 100644 (file)
--- 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 ] );
                });
        }