From 9f2720839f5123a43d694ce6db85eae79e355e08 Mon Sep 17 00:00:00 2001 From: Timo Tijhof Date: Wed, 30 Apr 2014 16:40:06 +0200 Subject: [PATCH] resourceloader: Use $.ajax instead of duplicating its "script" logic Also removed isFunction check. This is an internal method and the parameter is either a function or omitted/nulled. No need for aggressive validation. Change-Id: I6bbd82ca1b986b45a2f4653427396b9a6fceb6e8 --- resources/src/mediawiki/mediawiki.js | 83 +++++++--------------------- 1 file changed, 21 insertions(+), 62 deletions(-) diff --git a/resources/src/mediawiki/mediawiki.js b/resources/src/mediawiki/mediawiki.js index 47b00638c3..fc4635acbb 100644 --- a/resources/src/mediawiki/mediawiki.js +++ b/resources/src/mediawiki/mediawiki.js @@ -1043,71 +1043,30 @@ * Ignored (and defaulted to `true`) if the document-ready event has already occurred. */ function addScript( src, callback, async ) { - /*jshint evil:true */ - var script, head, done; - - // Using isReady directly instead of storing it locally from - // a $.fn.ready callback (bug 31895). + // Using isReady directly instead of storing it locally from a $().ready callback (bug 31895) if ( $.isReady || async ) { - // Can't use jQuery.getScript because that only uses