added new after func for factor setters. The filters can now hook into after, rather than a validator

src/Factory.js

@@ -42,9 +42,9 @@
         Y = 'y';
 
     Kinetic.Factory = {
-        addGetterSetter: function(constructor, attr, def, validator) {
+        addGetterSetter: function(constructor, attr, def, validator, after) {
             this.addGetter(constructor, attr, def);
-            this.addSetter(constructor, attr, validator);
+            this.addSetter(constructor, attr, validator, after);
             this.addOverloadedGetterSetter(constructor, attr);
         },
         addGetter: function(constructor, attr, def) {
@@ -56,7 +56,7 @@
                 return val === undefined ? def : val;
             };
         },
-        addSetter: function(constructor, attr, validator) {
+        addSetter: function(constructor, attr, validator, after) {
             var method = SET + Kinetic.Util._capitalize(attr);
 
             constructor.prototype[method] = function(val) {
@@ -65,10 +65,15 @@
                 }
 
                 this._setAttr(attr, val);
+
+                if (after) {
+                    after.call(this);
+                }
+
                 return this;  
             };
         },
-        addComponentsGetterSetter: function(constructor, attr, components, validator) {
+        addComponentsGetterSetter: function(constructor, attr, components, validator, after) {
             var len = components.length,
                 capitalize = Kinetic.Util._capitalize,
                 getter = GET + capitalize(attr), 
@@ -102,6 +107,10 @@
 
                 this._fireChangeEvent(attr, oldVal, val);
                 
+                if (after) {
+                    after.call(this);
+                }
+
                 return this;  
             };
 
@@ -131,6 +140,9 @@
             for (key in methods) {
                 constructor.prototype[key] = constructor.prototype[methods[key]];
             }
+        },
+        afterSetFilter: function() {
+            this._filterUpToDate = false;
         }
     };
 

src/filters/Blur.js

@@ -337,7 +337,7 @@
         }
     };
 
-    Kinetic.Factory.addGetterSetter(Kinetic.Node, 'blurRadius', 0, function(val) {this._filterUpToDate = false;return val;});
+    Kinetic.Factory.addGetterSetter(Kinetic.Node, 'blurRadius', 0, null, Kinetic.Factory.afterSetFilter);
 
     /**
     * get/set blur radius

src/filters/Brighten.js

@@ -21,7 +21,7 @@
         }
     };
 
-    Kinetic.Factory.addGetterSetter(Kinetic.Node, 'brightness', 0, function(val) {this._filterUpToDate = false;return val;});
+    Kinetic.Factory.addGetterSetter(Kinetic.Node, 'brightness', 0, null, Kinetic.Factory.afterSetFilter);
     /**
     * get/set filter brightness.  The brightness is a number between -1 and 1.  Positive values 
     *  brighten the pixels and negative values darken them.

src/filters/Emboss.js

@@ -123,7 +123,7 @@
       } while (--y);
   };
 
-  Kinetic.Factory.addGetterSetter(Kinetic.Node, 'embossStrength', 0.5, function(val) {this._filterUpToDate = false;return val;});
+  Kinetic.Factory.addGetterSetter(Kinetic.Node, 'embossStrength', 0.5, null, Kinetic.Factory.afterSetFilter);
   /**
   * get/set emboss strength
   * @name embossStrength
@@ -133,7 +133,7 @@
   * @returns {Number}
   */
 
-  Kinetic.Factory.addGetterSetter(Kinetic.Node, 'embossWhiteLevel', 0.5, function(val) {this._filterUpToDate = false;return val;});
+  Kinetic.Factory.addGetterSetter(Kinetic.Node, 'embossWhiteLevel', 0.5, null, Kinetic.Factory.afterSetFilter);
   /**
   * get/set emboss white level
   * @name embossWhiteLevel
@@ -143,7 +143,7 @@
   * @returns {Number}
   */
 
-  Kinetic.Factory.addGetterSetter(Kinetic.Node, 'embossDirection', 'top-left', function(val) {this._filterUpToDate = false;return val;});
+  Kinetic.Factory.addGetterSetter(Kinetic.Node, 'embossDirection', 'top-left', null, Kinetic.Factory.afterSetFilter);
   /**
   * get/set emboss direction
   * @name embossDirection
@@ -154,7 +154,7 @@
   * @returns {String}
   */
 
-  Kinetic.Factory.addGetterSetter(Kinetic.Node, 'embossBlend', false, function(val) {this._filterUpToDate = false;return val;});
+  Kinetic.Factory.addGetterSetter(Kinetic.Node, 'embossBlend', false, null, Kinetic.Factory.afterSetFilter);
   /**
   * get/set emboss blend
   * @name embossBlend

src/filters/Enhance.js

@@ -105,7 +105,7 @@
     }
   };
 
-  Kinetic.Factory.addGetterSetter(Kinetic.Node, 'enhance', 0, function(val) {this._filterUpToDate = false;return val;});
+  Kinetic.Factory.addGetterSetter(Kinetic.Node, 'enhance', 0, null, Kinetic.Factory.afterSetFilter);
 
     /**
     * get/set enhance

src/filters/HSV.js

@@ -57,7 +57,7 @@
 
   };
 
-  Kinetic.Factory.addGetterSetter(Kinetic.Node, 'hue', 0, function(val) {this._filterUpToDate = false;return val;});
+  Kinetic.Factory.addGetterSetter(Kinetic.Node, 'hue', 0, null, Kinetic.Factory.afterSetFilter);
   /**
   * get/set hsv hue in degrees
   * @name hue
@@ -67,7 +67,7 @@
   * @returns {Number}
   */
 
-  Kinetic.Factory.addGetterSetter(Kinetic.Node, 'saturation', 1, function(val) {this._filterUpToDate = false;return val;});
+  Kinetic.Factory.addGetterSetter(Kinetic.Node, 'saturation', 1, null, Kinetic.Factory.afterSetFilter);
   /**
   * get/set hsv saturation
   * @name saturation
@@ -77,7 +77,7 @@
   * @returns {Number}
   */
 
-  Kinetic.Factory.addGetterSetter(Kinetic.Node, 'value', 1, function(val) {this._filterUpToDate = false;return val;});
+  Kinetic.Factory.addGetterSetter(Kinetic.Node, 'value', 1, null, Kinetic.Factory.afterSetFilter);
   /**
   * get/set hsv value
   * @name value

src/filters/Kaleidoscope.js

@@ -310,9 +310,6 @@
     FromPolar(scratchData,imageData,{polarRotation:0});
   };
 
-    Kinetic.Factory.addGetterSetter(Kinetic.Node, 'kaleidoscopePower', 2, function(val) {this._filterUpToDate = false;return val;});
-    Kinetic.Factory.addGetterSetter(Kinetic.Node, 'kaleidoscopeAngle', 0, function(val) {this._filterUpToDate = false;return val;});
-
   /**
   * get/set kaleidoscope power
   * @name kaleidoscopePower
@@ -321,6 +318,7 @@
   * @param {Integer} power of kaleidoscope
   * @returns {Integer}
   */
+  Kinetic.Factory.addGetterSetter(Kinetic.Node, 'kaleidoscopePower', 2, null, Kinetic.Factory.afterSetFilter);
 
   /**
   * get/set kaleidoscope angle
@@ -330,4 +328,6 @@
   * @param {Integer} degrees
   * @returns {Integer}
   */
+  Kinetic.Factory.addGetterSetter(Kinetic.Node, 'kaleidoscopeAngle', 0, null, Kinetic.Factory.afterSetFilter);
+
 })();

src/filters/Mask.js

@@ -188,5 +188,5 @@
 		return imageData;
 	};
 
-	Kinetic.Factory.addGetterSetter(Kinetic.Node, 'threshold', 0, function(val) {this._filterUpToDate = false;return val;});
+	Kinetic.Factory.addGetterSetter(Kinetic.Node, 'threshold', 0, null, Kinetic.Factory.afterSetFilter);
 })();

src/filters/Noise.js

@@ -21,7 +21,7 @@
       }
   };
 
-  Kinetic.Factory.addGetterSetter(Kinetic.Node, 'noise', 0.2, function(val) {this._filterUpToDate = false;return val;});
+  Kinetic.Factory.addGetterSetter(Kinetic.Node, 'noise', 0.2, null, Kinetic.Factory.afterSetFilter);
 
   /**
   * get/set noise amount.  Must be a value between 0 and 1

src/filters/Pixelate.js

@@ -76,7 +76,7 @@
     
   };
 
-  Kinetic.Factory.addGetterSetter(Kinetic.Node, 'pixelSize', 8, function(val) {this._filterUpToDate = false;return val;});
+  Kinetic.Factory.addGetterSetter(Kinetic.Node, 'pixelSize', 8, null, Kinetic.Factory.afterSetFilter);
 
   /**
   * get/set pixel size

src/filters/Posterize.js

@@ -23,7 +23,7 @@
     }
   };
 
-  Kinetic.Factory.addGetterSetter(Kinetic.Node, 'levels', 0.5, function(val) {this._filterUpToDate = false;return val;});
+  Kinetic.Factory.addGetterSetter(Kinetic.Node, 'levels', 0.5, null, Kinetic.Factory.afterSetFilter);
 
   /**
   * get/set levels.  Must be a number between 0 and 1

src/filters/RGB.js

@@ -65,18 +65,7 @@
   * @returns {Integer}
   */
 
-  Kinetic.Factory.addGetterSetter(Kinetic.Node, 'blue', 0, function(val) {
+  Kinetic.Factory.addGetterSetter(Kinetic.Node, 'blue', 0, Kinetic.Validators.RGBComponent, Kinetic.Factory.afterSetFilter);
-      this._filterUpToDate = false;
-      if (val > 255) {
-          return 255;
-      }
-      else if (val < 0) {
-          return 0;
-      }
-      else {
-          return Math.round(val);
-      }
-  });
   /**
   * get/set filter blue value
   * @name blue

src/filters/Threshold.js

@@ -21,7 +21,7 @@
     }
   };
 
-  Kinetic.Factory.addGetterSetter(Kinetic.Node, 'threshold', 0.5, function(val) {this._filterUpToDate = false;return val;});
+  Kinetic.Factory.addGetterSetter(Kinetic.Node, 'threshold', 0.5, null, Kinetic.Factory.afterSetFilter);
 
   /**
   * get/set threshold.  Must be a value between 0 and 1