Add JSHint configuration, tidy up any warnings.

3 files changed, 24 insertions, 19 deletions

diff --git a/web/js/map-OpenLayers.js b/web/js/map-OpenLayers.js
index 31f5f49d8..00f3e8e70 100644
--- a/web/js/map-OpenLayers.js
+++ b/web/js/map-OpenLayers.js
@@ -22,6 +22,23 @@ $.extend(fixmystreet.utils, {
 
     fixmystreet.maps = fixmystreet.maps || {};
 
+    var drag = {
+        activate: function() {
+            this._drag = new OpenLayers.Control.DragFeatureFMS( fixmystreet.markers, {
+                onComplete: function(feature, e) {
+                    fixmystreet.update_pin( feature.geometry );
+                }
+            } );
+            fixmystreet.map.addControl( this._drag );
+            this._drag.activate();
+        },
+        deactivate: function() {
+            if (this._drag) {
+              this._drag.deactivate();
+            }
+        }
+    };
+
     $.extend(fixmystreet.maps, {
       // This function might be passed either an OpenLayers.LonLat (so has
       // lon and lat), or an OpenLayers.Geometry.Point (so has x and y).
@@ -215,23 +232,6 @@ $.extend(fixmystreet.utils, {
       }
     });
 
-    var drag = {
-        activate: function() {
-            this._drag = new OpenLayers.Control.DragFeatureFMS( fixmystreet.markers, {
-                onComplete: function(feature, e) {
-                    fixmystreet.update_pin( feature.geometry );
-                }
-            } );
-            fixmystreet.map.addControl( this._drag );
-            this._drag.activate();
-        },
-        deactivate: function() {
-            if (this._drag) {
-              this._drag.deactivate();
-            }
-        }
-    };
-
     /* Make sure pins aren't going to reload just because we're zooming out,
      * we already have the pins when the page loaded */
     function zoomToBounds(bounds) {
@@ -862,8 +862,9 @@ OpenLayers.Protocol.FixMyStreet = OpenLayers.Class(OpenLayers.Protocol.HTTP, {
                 options.params[key] = val;
             }
         });
+        var page;
         if (this.use_page) {
-            var page = $('.pagination').data('page');
+            page = $('.pagination').data('page');
             this.use_page = false;
         } else if (this.initial_page) {
             page = 1;
@@ -883,6 +884,7 @@ OpenLayers.Format.FixMyStreet = OpenLayers.Class(OpenLayers.Format.JSON, {
         // Remove loading indicator
         $('#loading-indicator').addClass('hidden');
         $('#loading-indicator').attr('aria-hidden', true);
+        var obj;
         if (typeof json == 'string') {
             obj = OpenLayers.Format.JSON.prototype.read.apply(this, [json, filter]);
         } else {
diff --git a/web/js/map-google-ol.js b/web/js/map-google-ol.js
index 7369a8e9f..99670d4f2 100644
--- a/web/js/map-google-ol.js
+++ b/web/js/map-google-ol.js
@@ -10,6 +10,7 @@ $(function(){
             fixmystreet.map.setBaseLayer(fixmystreet.map.layers[1]);
         }
     });
+    // jshint undef:false
     if (typeof fixmystreet_google_default !== 'undefined' && fixmystreet_google_default == 'satellite') {
         $('#map_layer_toggle').click();
     }
@@ -36,7 +37,9 @@ fixmystreet.maps.config = function() {
     var road_layer = {}; // Empty object defaults to standard road layer
 
     function apply_map_styles() {
+        // jshint undef:false
         var styledMapType = new google.maps.StyledMapType(fixmystreet_google_maps_custom_style);
+        // jshint undef:true
         this.mapObject.mapTypes.set('styled', styledMapType);
         this.mapObject.setMapTypeId('styled');
     }
diff --git a/web/js/map-google.js b/web/js/map-google.js
index 75a1b25a1..bf9909f02 100644
--- a/web/js/map-google.js
+++ b/web/js/map-google.js
@@ -41,7 +41,7 @@ fixmystreet.maps = {};
             });
             fixmystreet.report_marker = marker;
             google.maps.event.removeListener(fixmystreet.event_update_map);
-            for (m=0; m<fixmystreet.markers.length; m++) {
+            for (var m=0; m<fixmystreet.markers.length; m++) {
                 fixmystreet.markers[m].setMap(null);
             }
         }