Event: Don't warn on ajax .load(), only event .load()

dmethvin · dmethvin · commit f0cbdf16c3fc · 2016-02-25T15:05:51.000-05:00
Fixes #138 Closes #144 (cherry picked from commit ca716e9)
diff --git a/src/event.js b/src/event.js
@@ -43,16 +43,17 @@ jQuery.each( [ "load", "unload", "error" ], function( _, name ) {
 
 	jQuery.fn[ name ] = function() {
 		var args = Array.prototype.slice.call( arguments, 0 );
-		migrateWarn( "jQuery.fn." + name + "() is deprecated" );
 
 		// If this is an ajax load() the first arg should be the string URL;
 		// technically this could also be the "Anything" arg of the event .load()
 		// which just goes to show why this dumb signature has been deprecated!
 		// jQuery custom builds that exclude the Ajax module justifiably die here.
-		if ( name === "load" && typeof arguments[ 0 ] === "string" ) {
-			return oldLoad.apply( this, arguments );
+		if ( name === "load" && typeof args[ 0 ] === "string" ) {
+			return oldLoad.apply( this, args );
 		}
 
+		migrateWarn( "jQuery.fn." + name + "() is deprecated" );
+
 		args.splice( 0, 0, name );
 		if ( arguments.length ) {
 			return this.bind.apply( this, args );
diff --git a/test/event.js b/test/event.js
@@ -734,7 +734,7 @@ test( "error() event method", function() {
 });
 
 test( "load() and unload() event methods", function() {
-	expect( 4 );
+	expect( 5 );
 
 	expectWarning( "jQuery.fn.load()", function() {
 		jQuery( "<img />" )
@@ -757,6 +757,16 @@ test( "load() and unload() event methods", function() {
 			.unload()
 			.remove();
 	});
+
+	expectNoWarning( "ajax load", function() {
+		stop();
+		jQuery( "<div id=load138></div>" )
+			.appendTo( "#qunit-fixture" )
+			.load( "not-found.file", function() {
+				jQuery( "#load138" ).remove();
+				start();
+			});
+	});
 });
 
 test( "hover pseudo-event", function() {
