From 653a6cc82d802e82beabc90b968d12ba385543a5 Mon Sep 17 00:00:00 2001
From: Jon Atkins <github@jonatkins.com>
Date: Sun, 19 May 2013 15:29:36 +0100
Subject: [PATCH] rewrite default base map layer handling. it remembers by
 name, rather than id (so will work if order changes), and also works if
 plugins add additional base layers

---
 code/boot.js | 44 ++++++++++++++++++++++++++++++++------------
 1 file changed, 32 insertions(+), 12 deletions(-)

diff --git a/code/boot.js b/code/boot.js
index 7d5a45ec..1f4d9633 100644
--- a/code/boot.js
+++ b/code/boot.js
@@ -200,14 +200,6 @@ window.setupMap = function() {
 
   map.addControl(window.layerChooser);
 
-  // set the map AFTER adding the layer chooser, or Chrome reorders the
-  // layers. This likely leads to broken layer selection because the
-  // views/cookie order does not match the layer chooser order.
-  try {
-    convertCookieToLocalStorage('ingress.intelmap.type');
-    map.addLayer(views[localStorage['ingress.intelmap.type']]);
-  } catch(e) { map.addLayer(views[0]); }
-
   map.attributionControl.setPrefix('');
   // listen for changes and store them in cookies
   map.on('moveend', window.storeMapPosition);
@@ -228,10 +220,6 @@ window.setupMap = function() {
     console.log('Remove all resonators');
   });
 
-  map.on('baselayerchange', function () {
-    var selInd = $('[name=leaflet-base-layers]:checked').parent().index();
-    localStorage['ingress.intelmap.type']=selInd;
-  });
 
   // map update status handling & update map hooks
   // ensures order of calls
@@ -248,6 +236,36 @@ window.setupMap = function() {
 
 };
 
+//adds a base layer to the map. done separately from the above, so that plugins that add base layers can be the default
+window.setMapBaseLayer = function() {
+  //create a map name -> layer mapping - depends on internals of L.Control.Layers
+  var nameToLayer = {};
+  var firstLayer = null;
+
+  for (i in window.layerChooser._layers) {
+    var obj = window.layerChooser._layers[i];
+    if (!obj.overlay) {
+      nameToLayer[obj.name] = obj.layer;
+      if (!firstLayer) firstLayer = obj.layer;
+    }
+  }
+
+  var baseLayer = nameToLayer[localStorage['iitc-base-map']] || firstLayer;
+  map.addLayer(baseLayer);
+
+  map.on('baselayerchange', function() {
+    for(i in window.layerChooser._layers) {
+      var obj = window.layerChooser._layers[i];
+      if (!obj.overlay && map.hasLayer(obj.layer)) {
+        localStorage['iitc-base-map'] = obj.name;
+        break;
+      }
+    }
+
+  });
+
+}
+
 // renders player details into the website. Since the player info is
 // included as inline script in the original site, the data is static
 // and cannot be updated.
@@ -412,6 +430,8 @@ function boot() {
       }
     });
 
+  window.setMapBaseLayer();
+
   window.runOnSmartphonesAfterBoot();
 
   // workaround for #129. Not sure why this is required.