Fixes #11264 or rather seriously limits the risk of global ajaxSettings screwing with script loading in domManip. Gotta love globals and sneaky dependencies. Unit test added.

3 files changed, 23 insertions, 0 deletions

diff --git a/src/manipulation.js b/src/manipulation.js
index 0b42810da..2a3757904 100644
--- a/src/manipulation.js
+++ b/src/manipulation.js
@@ -351,6 +351,8 @@ jQuery.fn.extend({
 				jQuery.each( scripts, function( i, elem ) {
 					if ( elem.src ) {
 						jQuery.ajax({
+							type: "GET",
+							global: false,
 							url: elem.src,
 							async: false,
 							dataType: "script"
diff --git a/test/data/evalScript.php b/test/data/evalScript.php
new file mode 100644
index 000000000..ea9b8c55f
--- /dev/null
+++ b/test/data/evalScript.php
@@ -0,0 +1 @@
+ok( "<?php echo $_SERVER['REQUEST_METHOD'] ?>" === "GET", "request method is <?php echo $_SERVER['REQUEST_METHOD'] ?>" );
\ No newline at end of file
diff --git a/test/unit/ajax.js b/test/unit/ajax.js
index cca457db1..163ba4f05 100644
--- a/test/unit/ajax.js
+++ b/test/unit/ajax.js
@@ -2336,6 +2336,26 @@ test( "jQuery.ajax - loading binary data shouldn't throw an exception in IE (#11
 	});
 });
 
+test( "jQuery.domManip - no side effect because of ajaxSetup or global events (#11264)", function() {
+	expect( 1 );
+
+	jQuery.ajaxSetup({
+		type: "POST"
+	});
+
+	jQuery( document ).bind( "ajaxStart ajaxStop", function() {
+		ok( false, "Global event triggered" );
+	});
+
+	jQuery( "#qunit-fixture" ).append( "<script src='data/evalScript.php'></script>" );
+
+	jQuery( document ).unbind( "ajaxStart ajaxStop" );
+
+	jQuery.ajaxSetup({
+		type: "GET"
+	});
+});
+
 test("jQuery.ajax - active counter", function() {
     ok( jQuery.active == 0, "ajax active counter should be zero: " + jQuery.active );
 });