Merge pull request #1553 from stweil/lgtm

Add some missing variable declarations

js/lib/isfahan.js

@@ -45,6 +45,7 @@ var Isfahan = function(configObject) {
     n = group.length,
     x = rect.x,
     y = rect.y,
+    d,
     o;
     var offset = 0;
     while (++i < n) {

js/src/annotations/osd-region-draw-tool.js

@@ -137,7 +137,7 @@
       if (typeof annotation === 'object' && annotation.on) {
         for (var i = 0; i < strategies.length; i++) {
           if (strategies[i].isThisType(annotation)) {
-            shapeArray = strategies[i].parseRegion(annotation, this);
+            var shapeArray = strategies[i].parseRegion(annotation, this);
             return shapeArray;
           }
         }

js/src/utils/saveController.js

@@ -87,8 +87,8 @@
         return false;
       }
 
-      saveModule = config.jsonStorageEndpoint.module,
-      saveOptions = config.jsonStorageEndpoint.options;
+      var saveModule = config.jsonStorageEndpoint.module;
+      var saveOptions = config.jsonStorageEndpoint.options;
       _this.storageModule = new $[saveModule](saveOptions);
 
       var sessionID = window.location.hash.substring(1); // will return empty string if none exists, causing the or statement below to evaluate to false, generating a new sesssionID.

js/src/viewer/manifestListItem.js

@@ -108,7 +108,7 @@
 
         var aspectRatio = canvas.height/canvas.width,
         width = (_this.thumbHeight/aspectRatio);
-        url = _this.manifest.getThumbnailForCanvas(canvas, width);
+        var url = _this.manifest.getThumbnailForCanvas(canvas, width);
 
         _this.allImages.push({
           url: url,

js/src/widgets/searchTab.js

@@ -137,7 +137,7 @@
         };
       });
 
-      templateData = {
+      var templateData = {
         searchService: searchServiceIdArray
       };