1.5.0-beta2

Fix glyph access key for openmaptiles

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "maputnik",
-  "version": "1.5.0-beta",
+  "version": "1.5.0-beta2",
   "lockfileVersion": 1,
   "requires": true,
   "dependencies": {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "maputnik",
-  "version": "1.5.0-beta",
+  "version": "1.5.0-beta2",
   "description": "A MapboxGL visual style editor",
   "main": "''",
   "scripts": {

src/libs/style.js

@@ -54,27 +54,27 @@ function indexOfLayer(layers, layerId) {
   return null
 }
 
-function getAccessToken(key, mapStyle, opts) {
+function getAccessToken(sourceName, mapStyle, opts) {
-  if(key === "thunderforest_transport" || key === "thunderforest_outdoors") {
+  if(sourceName === "thunderforest_transport" || sourceName === "thunderforest_outdoors") {
-    key = "thunderforest";
+    sourceName = "thunderforest"
   }
 
   const metadata = mapStyle.metadata || {}
-  let accessToken = metadata['maputnik:'+key+'_access_token'];
+  let accessToken = metadata[`maputnik:${sourceName}_access_token`]
 
   if(opts.allowFallback && !accessToken) {
-    accessToken = tokens[key];
+    accessToken = tokens[sourceName]
   }
 
   return accessToken;
 }
 
-function replaceSourceAccessToken(mapStyle, key, opts={}) {
+function replaceSourceAccessToken(mapStyle, sourceName, opts={}) {
-  const source = mapStyle.sources[key]
+  const source = mapStyle.sources[sourceName]
   if(!source) return mapStyle
   if(!source.hasOwnProperty("url")) return mapStyle
 
-  const accessToken = getAccessToken(key, mapStyle, opts)
+  const accessToken = getAccessToken(sourceName, mapStyle, opts)
 
   if(!accessToken) {
     // Early exit.
@@ -83,7 +83,7 @@ function replaceSourceAccessToken(mapStyle, key, opts={}) {
 
   const changedSources = {
     ...mapStyle.sources,
-    [key]: {
+    [sourceName]: {
       ...source,
       url: source.url.replace('{key}', accessToken)
     }
@@ -92,24 +92,19 @@ function replaceSourceAccessToken(mapStyle, key, opts={}) {
     ...mapStyle,
     sources: changedSources
   }
-
+  if (mapStyle.glyphs) {
+    changedStyle.glyphs = changedStyle.glyphs.replace('{key}', accessToken)
+  }
   return changedStyle
 }
 
 function replaceAccessTokens(mapStyle, opts={}) {
-  let changedStyle = mapStyle;
+  let changedStyle = mapStyle
 
-  Object.keys(mapStyle.sources).forEach((tokenKey) => {
+  Object.keys(mapStyle.sources).forEach((sourceName) => {
-    changedStyle = replaceSourceAccessToken(changedStyle, tokenKey, opts);
+    changedStyle = replaceSourceAccessToken(changedStyle, sourceName, opts);
   })
 
-  if(mapStyle.glyphs && mapStyle.glyphs.match(/\.tileserver\.org/)) {
-    changedStyle = {
-      ...changedStyle,
-      glyphs: mapStyle.glyphs ? mapStyle.glyphs.replace('{key}', getAccessToken("openmaptiles", mapStyle, opts)) : mapStyle.glyphs
-    }
-  }
-
   return changedStyle
 }