Fix some more JSHint issues

app/assets/javascripts/index/directions/mapquest.js

@@ -48,9 +48,10 @@ function MapQuestEngine(id, vehicleParam) {
           if (data.info.statuscode !== 0)
             return callback(true);
 
+          var i;
           var line = [];
           var shape = data.route.shape.shapePoints;
-          for (var i = 0; i < shape.length; i += 2) {
+          for (i = 0; i < shape.length; i += 2) {
             line.push(L.latLng(shape[i], shape[i + 1]));
           }
 
@@ -58,7 +59,7 @@ function MapQuestEngine(id, vehicleParam) {
           // data.route.legs[0].maneuvers is list of turns
           var steps = [];
           var mq = data.route.legs[0].maneuvers;
-          for (var i = 0; i < mq.length; i++) {
+          for (i = 0; i < mq.length; i++) {
             var s = mq[i];
             var d;
             var linesegstart, linesegend, lineseg;

app/assets/javascripts/index/note.js

@@ -1,6 +1,5 @@
 OSM.Note = function (map) {
-  var noteLayer = map.noteLayer,
+  var content = $('#sidebar_content'),
-    content = $('#sidebar_content'),
     page = {},
     halo, currentNote;
 

app/assets/javascripts/index/query.js

@@ -87,9 +87,10 @@ OSM.Query = function(map) {
       });
     } else {
       var prefixes = I18n.t("geocoder.search_osm_nominatim.prefix");
+      var key, value;
 
-      for (var key in tags) {
+      for (key in tags) {
-        var value = tags[key];
+        value = tags[key];
 
         if (prefixes[key]) {
           if (prefixes[key][value]) {
@@ -98,8 +99,8 @@ OSM.Query = function(map) {
         }
       }
 
-      for (var key in tags) {
+      for (key in tags) {
-        var value = tags[key];
+        value = tags[key];
 
         if (prefixes[key]) {
           var first = value.substr(0, 1).toUpperCase(),

app/assets/javascripts/leaflet.layers.js

@@ -34,12 +34,12 @@ L.OSM.layers = function(options) {
       .attr('class', 'section base-layers')
       .appendTo($ui);
 
-    list = $('<ul>')
+    var baseLayers = $('<ul>')
       .appendTo(baseSection);
 
     layers.forEach(function(layer) {
       var item = $('<li>')
-        .appendTo(list);
+        .appendTo(baseLayers);
 
       if (map.hasLayer(layer)) {
         item.addClass('active');
@@ -117,7 +117,7 @@ L.OSM.layers = function(options) {
         .attr("class", "deemphasize")
         .appendTo(overlaySection);
 
-      var list = $('<ul>')
+      var overlays = $('<ul>')
         .appendTo(overlaySection);
 
       var addOverlay = function (layer, name, maxArea) {
@@ -125,7 +125,7 @@ L.OSM.layers = function(options) {
           .tooltip({
             placement: 'top'
           })
-          .appendTo(list);
+          .appendTo(overlays);
 
         var label = $('<label>')
           .appendTo(item);

app/assets/javascripts/leaflet.zoom.js

@@ -37,7 +37,7 @@ L.OSM.Zoom = L.Control.extend({
 		link.href = '#';
 		link.title = title;
 
-		var sprite = L.DomUtil.create('span', 'icon ' + className, link);
+		L.DomUtil.create('span', 'icon ' + className, link);
 
 		var stop = L.DomEvent.stopPropagation;
 

config/jshint.yml

@@ -15,11 +15,14 @@ options:
   unused: true
   trailing: true
   globals:
-    I18n: true
+    I18n: false
     L: true
     OSM: true
-    Piwik: true
+    Piwik: false
+    alert: false
+    escape: false
     maximiseMap: false
     minimiseMap: false
-    querystring: false
+    querystring: true
+    require: false
     updateLinks: false