Ajax: simplify one ajax call and add explanatory comment

markelog · markelog · commit 0ac28ed29368 · 2015-02-15T02:00:38.000+03:00
* Remove "async = true" from script transport since it was needed
  for FF &lt; 4 and old Opera which we do not support anymore

* Add comment to "evalUrl" method on why "type" field should be explicit
diff --git a/src/ajax/script.js b/src/ajax/script.js
@@ -39,7 +39,6 @@ jQuery.ajaxTransport( "script", function( s ) {
 		return {
 			send: function( _, complete ) {
 				script = jQuery("<script>").prop({
-					async: true,
 					charset: s.scriptCharset,
 					src: s.url
 				}).on(
diff --git a/src/manipulation/_evalUrl.js b/src/manipulation/_evalUrl.js
@@ -5,6 +5,8 @@ define([
 jQuery._evalUrl = function( url ) {
 	return jQuery.ajax({
 		url: url,
+
+		// Make this explicit, since user can override this through ajaxSetup (#11264)
 		type: "GET",
 		dataType: "script",
 		cache: true,
