Change use of maybe-defined "mkws_jQuery" to avoid assigning to an undefined variable.
authorMike Taylor <mike@indexdata.com>
Fri, 27 Jun 2014 09:05:30 +0000 (10:05 +0100)
committerMike Taylor <mike@indexdata.com>
Fri, 27 Jun 2014 09:05:30 +0000 (10:05 +0100)
Fixes the "make phantomjs" Jenkins error:
Strict mode forbids implicit creation of global property 'mkws_jQuery'

Note: not yet tested as part of installation (which we need so we can
check that Seb's widgets embedded in the blog still work). I am
pushing so that I can onto the live site and verify that. If there's a
problem there, I'll revert and try again.

src/mkws-core.js

index 7c74664..dd106f7 100644 (file)
@@ -96,8 +96,11 @@ var mkws = {
 };
 
 // We may be using a separate copy
-if (typeof(mkws_jQuery) === "undefined") mkws_jQuery = jQuery;
-mkws.$ = mkws_jQuery;
+if (typeof(mkws_jQuery) !== "undefined") {
+  mkws.$ = mkws_jQuery;
+} else {
+  mkws.$ = jQuery;
+}
 
 mkws.log = function(string) {
   if (!mkws.log_level)
@@ -609,4 +612,4 @@ mkws.pagerNext = function(tname) {
     var widgetSelector = selectorForAllWidgets();
     if (widgetSelector && $(widgetSelector).length !== 0) init();
   });
-})(mkws_jQuery);
+})(mkws.$);