From 4b3e63066dd480d07b9ce8057cb0c02b8ad7e990 Mon Sep 17 00:00:00 2001 From: Oleg Gaidarenko Date: Sun, 15 Feb 2015 01:21:05 +0300 Subject: [PATCH] Ajax: simplify one ajax call and add explanatory comment * Remove "async = true" from script transport since it was needed for FF < 4 and old Opera which we do not support anymore * Add comment to "evalUrl" method on why "type" field should be explicit Ref 0ac28ed293681cb8f2e9fdd11efa0021da039c84 --- src/ajax/script.js | 2 -- src/manipulation/_evalUrl.js | 2 ++ 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/ajax/script.js b/src/ajax/script.js index be422adf3..2e937fe65 100644 --- a/src/ajax/script.js +++ b/src/ajax/script.js @@ -46,8 +46,6 @@ jQuery.ajaxTransport( "script", function(s) { script = document.createElement("script"); - script.async = true; - if ( s.scriptCharset ) { script.charset = s.scriptCharset; } diff --git a/src/manipulation/_evalUrl.js b/src/manipulation/_evalUrl.js index 0fdacbcee..007e7293c 100644 --- a/src/manipulation/_evalUrl.js +++ b/src/manipulation/_evalUrl.js @@ -8,6 +8,8 @@ jQuery._evalUrl = function( url ) { type: "GET", dataType: "script", cache: true, + + // Make this explicit, since user can override this through ajaxSetup (#11264) async: false, global: false, "throws": true -- 2.39.5