Merge pull request #818 from Kienz/patch-1

Fix `fabric.Circle.set` for radius

src/shapes/circle.class.js

@@ -35,9 +35,22 @@
 
       this.set('radius', options.radius || 0);
       this.callSuper('initialize', options);
+    },
 
-      var diameter = this.get('radius') * 2;
-      this.set('width', diameter).set('height', diameter);
+    /**
+     * @private
+     * @param {String} key
+     * @param {Any} value
+     * @return {fabric.Circle} thisArg
+     */
+    _set: function(key, value) {
+      this.callSuper('_set', key, value);
+
+      if (key === 'radius') {
+        this.setRadius(value);
+      }
+
+      return this;
     },
 
     /**

test/unit/circle.js

@@ -42,6 +42,13 @@
     var circle = new fabric.Circle({ radius: 10 });
 
     ok(typeof circle.setRadius == 'function');
+
+    equal(circle.getRadiusX(), 10);
+    equal(circle.getRadiusY(), 10);
+
+    equal(circle.getWidth(), 20);
+    equal(circle.getHeight(), 20);
+
     circle.setRadius(20);
 
     equal(circle.getRadiusX(), 20);