From 6b8c334049f255ef9f82f729e6b99ad4ef9abe36 Mon Sep 17 00:00:00 2001
From: Frederic Junod <frederic.junod@camptocamp.com>
Date: Tue, 27 Nov 2018 13:36:23 +0100
Subject: [PATCH] Fix layer with no data

---
 contribs/gmf/apps/desktop/Controller.js  |  5 ++--
 contribs/gmf/apps/desktop/index.html.ejs |  2 +-
 src/profile/d3Elevation.js               | 33 ++++++++++++++----------
 3 files changed, 23 insertions(+), 17 deletions(-)

diff --git a/contribs/gmf/apps/desktop/Controller.js b/contribs/gmf/apps/desktop/Controller.js
index a89878a84a99..70e31cc0f0c2 100644
--- a/contribs/gmf/apps/desktop/Controller.js
+++ b/contribs/gmf/apps/desktop/Controller.js
@@ -58,7 +58,7 @@ const exports = function($scope, $injector) {
    * @type {Array.<string>}
    * @export
    */
-  this.elevationLayers = ['aster', 'srtm'];
+  this.elevationLayers = ['aster', 'srtm', 'srtm-partial'];
 
   /**
    * @type {Object.<string, gmf.raster.component.LayerConfig>}
@@ -78,7 +78,8 @@ const exports = function($scope, $injector) {
    */
   this.profileLinesconfiguration = {
     'aster': {color: '#0000A0'},
-    'srtm': {color: '#00A000'}
+    'srtm': {color: '#00A000'},
+    'srtm-partial': {color: '#FF66FF'},
   };
 
   /**
diff --git a/contribs/gmf/apps/desktop/index.html.ejs b/contribs/gmf/apps/desktop/index.html.ejs
index 73e0ad9e1fc9..bd4b9ed9992c 100644
--- a/contribs/gmf/apps/desktop/index.html.ejs
+++ b/contribs/gmf/apps/desktop/index.html.ejs
@@ -310,7 +310,7 @@
     <% } %>
     <script>
     document.write(
-      '<script src="https://geomapfish-demo.camptocamp.com/2.3/wsgi/dynamic.js?' +
+      '<script src="https://geomapfish-demo.camptocamp.com/sbrunner23/wsgi/dynamic.js?' +
       'interface=desktop&' +
       'query=' + encodeURIComponent(document.location.search) +
       '"><' + '/script>');
diff --git a/src/profile/d3Elevation.js b/src/profile/d3Elevation.js
index 365c26861c7f..7796010aa695 100644
--- a/src/profile/d3Elevation.js
+++ b/src/profile/d3Elevation.js
@@ -361,11 +361,13 @@ const exports = function(options) {
       // each lines.
       const yDomain = function() {
         let elevationsValues = [];
-        let extent, name;
         // Get min/max values (extent) of each lines.
-        for (name in linesConfiguration) {
-          extent = d3.extent(data, d => linesConfiguration[name].zExtractor(d));
-          elevationsValues = elevationsValues.concat(extent);
+        for (const name in linesConfiguration) {
+          const extent = d3.extent(data, d => linesConfiguration[name].zExtractor(d));
+          // only include defined extent
+          if (extent.every(Number.isFinite)) {
+            elevationsValues = elevationsValues.concat(extent);
+          }
         }
         return [
           Math.min.apply(null, elevationsValues),
@@ -519,24 +521,27 @@ const exports = function(options) {
 
     for (lineName in linesConfiguration) {
       elevation = linesConfiguration[lineName].zExtractor(point);
-      elevations.push(elevation);
-      elevationsRef[lineName] = elevation;
-      g.select(`.y.grid-hover.${lineName}`)
-        .style('display', 'inline')
-        .select('line')
-        .attr('x1', x(0))
-        .attr('y1', y(elevation))
-        .attr('x2', width)
-        .attr('y2', y(elevation));
+      if (Number.isFinite(elevation)) {
+        elevations.push(elevation);
+        elevationsRef[lineName] = elevation;
+        g.select(`.y.grid-hover.${lineName}`)
+          .style('display', 'inline')
+          .select('line')
+          .attr('x1', x(0))
+          .attr('y1', y(elevation))
+          .attr('x2', width)
+          .attr('y2', y(elevation));
+      }
     }
 
+    const y2 = y(Math.max.apply(null, elevations));
     g.select('.x.grid-hover')
       .style('display', 'inline')
       .select('line')
       .attr('x1', x(dist))
       .attr('y1', height)
       .attr('x2', x(dist))
-      .attr('y2', y(Math.max.apply(null, elevations)));
+      .attr('y2', Number.isFinite(y2) ? y2 : 0);
 
     const right = dist > xDomain[1] / 2;
     let xtranslate = x(dist);