Fix getLegendGraphicUrl to not append multiple questionmarks

terrestris · Sep 5, 2018 · d1888ee · d1888ee
1 parent 37d59f0
commit d1888ee
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/src/MapUtil/MapUtil.js b/src/MapUtil/MapUtil.js
@@ -295,6 +295,7 @@ export class MapUtil {
    * Get the getlegendGraphic url of a layer. Designed for geoserver.
    * Currently supported Sources:
    *  - ol.source.TileWms (with url configured)
+   *  - ol.source.ImageWms (with url configured)
    *
    * @param {ol.layer.Layer} layer The layer that you want to have a legendUrlfor.
    * @return {String|undefined} The getLegendGraphicUrl.
@@ -330,7 +331,7 @@ export class MapUtil {
       const queryString = UrlUtil.objectToRequestString(
         Object.assign(params, extraParams));
 
-      return `${url}?${queryString}`;
+      return url.endsWith('?') ? `${url}${queryString}` : `${url}?${queryString}`;
     } else {
       Logger.warn(`Source of "${layer.get('name')}" is currently not supported `
         + `by MapUtil.getLegendGraphicUrl.`);

diff --git a/src/MapUtil/MapUtil.spec.js b/src/MapUtil/MapUtil.spec.js
@@ -474,7 +474,7 @@ describe('MapUtil', () => {
       layer1 = new OlLayerTile({
         name: 'OSM-WMS',
         source: new OlSourceTileWMS({
-          url: 'https://ows.terrestris.de/osm-gray/service',
+          url: 'https://ows.terrestris.de/osm-gray/service?',
           params: {'LAYERS': 'OSM-WMS', 'TILED': true},
           serverType: 'geoserver'
         })
@@ -551,6 +551,12 @@ describe('MapUtil', () => {
       });
     });
 
+    it('Does not append multiple questionmarks in URL', () => {
+      const legendUrl = MapUtil.getLegendGraphicUrl(layer1);
+      expect(legendUrl).toEqual(expect.stringContaining('?'));
+      expect(legendUrl).toEqual(expect.not.stringContaining('??'));
+    });
+
     it('accepts extraParams for the request', () => {
       const extraParams = {
         HEIGHT: 10,