From 97ef1f2612a9c5bd453d34146fdfd552cf9cee67 Mon Sep 17 00:00:00 2001 From: Oleg Gaidarenko Date: Wed, 11 Feb 2015 16:40:40 +0300 Subject: [PATCH] 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 --- src/ajax/load.js | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/ajax/load.js b/src/ajax/load.js index 595061443..f8f0c396a 100644 --- 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 ] ); }); } -- 2.39.5