fix toObject for extended attrs. close #86

CHANGELOG.md

@@ -9,6 +9,7 @@ This project adheres to [Semantic Versioning](http://semver.org/).
 
 ### Fixed
 - Correct calculation in `getClientRect` method of `Konva.Line` and `Konva.Container`.
+- Correct `toObject()` behaviour for node with attrs with extended native prototypes
 
 ### Changed
 - Dragging now works much better. If your pointer is out of stage content dragging will still continue.

src/Util.js

@@ -530,6 +530,9 @@
                 key;
 
             for(key in obj) {
+                if (!obj.hasOwnProperty(key)) {
+                    continue;
+                }
                 if(this._isFunction(obj[key])) {
                     names.push(key);
                 }

test/unit/Node-test.js

@@ -3015,4 +3015,15 @@ suite('Node', function() {
       };
       imageObj.src = 'assets/darth-vader.jpg';
   });
+
+  test('toObject with extended prototypes', function() {
+      var node = new Konva.Circle({
+          id: 'foo',
+          radius: 10
+      });
+      Number.prototype.customFunc = function() {};
+      console.dir(node.toObject());
+      assert.equal(node.toObject().attrs.radius, 10);
+      delete Number.prototype.customFunc;
+  });
 });