From 77f16c7fd64439bfc468b825e10d6a94b8a17e88 Mon Sep 17 00:00:00 2001
From: tsauerwein <tobias.sauerwein@camptocamp.com>
Date: Fri, 24 Apr 2015 09:26:31 +0200
Subject: [PATCH] Fix XYZ coordinates in snap interaction

---
 src/ol/interaction/snapinteraction.js         |  2 +-
 .../ol/interaction/snapinteraction.test.js    | 60 +++++++++++++++++++
 2 files changed, 61 insertions(+), 1 deletion(-)

diff --git a/src/ol/interaction/snapinteraction.js b/src/ol/interaction/snapinteraction.js
index dc93a1b437..fceed0af19 100644
--- a/src/ol/interaction/snapinteraction.js
+++ b/src/ol/interaction/snapinteraction.js
@@ -583,7 +583,7 @@ ol.interaction.Snap.SegmentDataType;
 ol.interaction.Snap.handleEvent_ = function(evt) {
   var result = this.snapTo(evt.pixel, evt.coordinate, evt.map);
   if (result.snapped) {
-    evt.coordinate = result.vertex;
+    evt.coordinate = result.vertex.slice(0, 2);
     evt.pixel = result.vertexPixel;
   }
   return ol.interaction.Pointer.handleEvent.call(this, evt);
diff --git a/test/spec/ol/interaction/snapinteraction.test.js b/test/spec/ol/interaction/snapinteraction.test.js
index 5420431046..f595fa459b 100644
--- a/test/spec/ol/interaction/snapinteraction.test.js
+++ b/test/spec/ol/interaction/snapinteraction.test.js
@@ -11,6 +11,66 @@ describe('ol.interaction.Snap', function() {
 
   });
 
+  describe('handleEvent_', function() {
+    var target, map;
+
+    var width = 360;
+    var height = 180;
+
+    beforeEach(function(done) {
+      target = document.createElement('div');
+
+      var style = target.style;
+      style.position = 'absolute';
+      style.left = '-1000px';
+      style.top = '-1000px';
+      style.width = width + 'px';
+      style.height = height + 'px';
+      document.body.appendChild(target);
+
+      map = new ol.Map({
+        target: target,
+        view: new ol.View({
+          projection: 'EPSG:4326',
+          center: [0, 0],
+          resolution: 1
+        })
+      });
+
+      map.on('postrender', function() {
+        done();
+      });
+    });
+
+    afterEach(function() {
+      goog.dispose(map);
+      document.body.removeChild(target);
+    });
+
+    it('can handle XYZ coordinates', function() {
+      var point = new ol.Feature(new ol.geom.Point([0, 0, 123]));
+      var snapInteraction = new ol.interaction.Snap({
+        features: new ol.Collection([point])
+      });
+      snapInteraction.setMap(map);
+
+      var event = {
+        pixel: [width / 2, height / 2],
+        coordinate: [0, 0],
+        map: map
+      };
+      ol.interaction.Snap.handleEvent_.call(snapInteraction, event);
+      // check that the coordinate is in XY and not XYZ
+      expect(event.coordinate).to.eql([0, 0]);
+    });
+
+  });
+
 });
 
+goog.require('ol.Collection');
+goog.require('ol.Feature');
+goog.require('ol.Map');
+goog.require('ol.View');
+goog.require('ol.geom.Point');
 goog.require('ol.interaction.Snap');