DDF-2687 Update vendored DrawHelper to not remove overlapping points

- See leforthomas/cesium-drawhelper#13 for details. - At lower zooms, points were being erroneously removed from drawings.
andrewkfiedler · Jan 6, 2017 · c73e009 · c73e009
1 parent b35cfa1
commit c73e009
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 12 deletions.
diff --git a/catalog/ui/catalog-ui-search/src/main/webapp/lib/cesium-drawhelper/DrawHelper.js b/catalog/ui/catalog-ui-search/src/main/webapp/lib/cesium-drawhelper/DrawHelper.js
@@ -933,12 +933,8 @@ var DrawHelper = (function() {
                     if (cartesian) {
                         _self.stopDrawing();
                         if(typeof options.callback == 'function') {
-                            // remove overlapping ones
-                            var index = positions.length - 1;
-                            // TODO - calculate some epsilon based on the zoom level
-                            var epsilon = Cesium.Math.EPSILON3;
-                            for(; index > 0 && positions[index].equalsEpsilon(positions[index - 1], epsilon); index--) {}
-                            options.callback(positions.splice(0, index + 1));
+                            //https://github.com/leforthomas/cesium-drawhelper/issues/13
+                            options.callback(positions);
                         }
                     }
                 }

diff --git a/catalog/ui/search-ui/standard/src/main/webapp/lib/cesium-drawhelper/DrawHelper.js b/catalog/ui/search-ui/standard/src/main/webapp/lib/cesium-drawhelper/DrawHelper.js
@@ -933,12 +933,8 @@ var DrawHelper = (function() {
                     if (cartesian) {
                         _self.stopDrawing();
                         if(typeof options.callback == 'function') {
-                            // remove overlapping ones
-                            var index = positions.length - 1;
-                            // TODO - calculate some epsilon based on the zoom level
-                            var epsilon = Cesium.Math.EPSILON3;
-                            for(; index > 0 && positions[index].equalsEpsilon(positions[index - 1], epsilon); index--) {}
-                            options.callback(positions.splice(0, index + 1));
+                            //https://github.com/leforthomas/cesium-drawhelper/issues/13
+                            options.callback(positions);
                         }
                     }
                 }