From ec6a0548d1527572739ac174e946107daa428774 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=89ric=20Lemoine?= <eric.lemoine@gmail.com>
Date: Tue, 24 Jun 2014 18:12:19 +0200
Subject: [PATCH] Add ol.Feature#clone

---
 src/ol/feature.js            | 21 ++++++++++++++++++++
 test/spec/ol/feature.test.js | 37 ++++++++++++++++++++++++++++++++++++
 2 files changed, 58 insertions(+)

diff --git a/src/ol/feature.js b/src/ol/feature.js
index aedcd12995..d7e510f873 100644
--- a/src/ol/feature.js
+++ b/src/ol/feature.js
@@ -84,6 +84,27 @@ ol.Feature = function(opt_geometryOrValues) {
 goog.inherits(ol.Feature, ol.Object);
 
 
+/**
+ * Clone this feature. If the original feature has a geometry it
+ * is also cloned. The feature id is not set in the clone.
+ * @return {ol.Feature} The clone.
+ * @todo api
+ */
+ol.Feature.prototype.clone = function() {
+  var clone = new ol.Feature(this.getProperties());
+  clone.setGeometryName(this.getGeometryName());
+  var geometry = this.getGeometry();
+  if (goog.isDefAndNotNull(geometry)) {
+    clone.setGeometry(geometry.clone());
+  }
+  var style = this.getStyle();
+  if (!goog.isNull(style)) {
+    clone.setStyle(style);
+  }
+  return clone;
+};
+
+
 /**
  * @return {ol.geom.Geometry|undefined} Geometry.
  * @todo api
diff --git a/test/spec/ol/feature.test.js b/test/spec/ol/feature.test.js
index 76f5ea4d69..b9760ec2ea 100644
--- a/test/spec/ol/feature.test.js
+++ b/test/spec/ol/feature.test.js
@@ -371,6 +371,43 @@ describe('ol.Feature', function() {
 
   });
 
+  describe('#clone', function() {
+
+    it('correctly clones features', function() {
+      var feature = new ol.Feature();
+      feature.setValues({'fookey': 'fooval'});
+      feature.setId(1);
+      feature.setGeometryName('geom');
+      var geometry = new ol.geom.Point([1, 2]);
+      feature.setGeometry(geometry);
+      var style = new ol.style.Style({});
+      feature.setStyle(style);
+      feature.set('barkey', 'barval');
+
+      var clone = feature.clone();
+      expect(clone.get('fookey')).to.be('fooval');
+      expect(clone.getId()).to.be(undefined);
+      expect(clone.getGeometryName()).to.be('geom');
+      var geometryClone = clone.getGeometry();
+      expect(geometryClone).not.to.be(geometry);
+      var coordinates = geometryClone.getFlatCoordinates();
+      expect(coordinates[0]).to.be(1);
+      expect(coordinates[1]).to.be(2);
+      expect(clone.getStyle()).to.be(style);
+      expect(clone.get('barkey')).to.be('barval');
+    });
+
+    it('correctly clones features with no geometry and no style', function() {
+      var feature = new ol.Feature();
+      feature.set('fookey', 'fooval');
+
+      var clone = feature.clone();
+      expect(clone.get('fookey')).to.be('fooval');
+      expect(clone.getGeometry()).to.be(undefined);
+      expect(clone.getStyle()).to.be(null);
+    });
+  });
+
 
 });