Refactored into new Base Game Object class, that Image and Sprite (and everything else) will now extend. Much more versatile, and a lot less code duplication.

Author: photonstorm

build/config.php

@@ -178,6 +178,7 @@
     <script src="$path/src/components/Transform.js"></script>
     <script src="$path/src/components/UpdateManager.js"></script>
 
+    <script src="$path/src/gameobjects/GameObject.js"></script>
     <script src="$path/src/gameobjects/Factory.js"></script>
     <script src="$path/src/gameobjects/GameObjectCreator.js"></script>
 

src/components/Color.js

@@ -64,7 +64,7 @@ Phaser.Component.Color.prototype = {
             this._a = (alpha) ? alpha : 1;
         }
 
-        this.setDirty();
+        this.dirty = true;
     },
 
     clearTint: function ()
@@ -90,21 +90,7 @@ Phaser.Component.Color.prototype = {
 
         this._hasTint = true;
 
-        this.setDirty();
-    },
-
-    getBlendMode: function ()
-    {
-        return this._blendMode;
-    },
-
-    setBlendMode: function (blendMode)
-    {
-        if (blendMode >= 0 && blendMode <= 16)
-        {
-            this._blendMode = blendMode;
-            this.setDirty();
-        }
+        this.dirty = true;
     },
 
     //  Called by the Dirty Manager
@@ -122,16 +108,6 @@ Phaser.Component.Color.prototype = {
 
     },
 
-    setDirty: function ()
-    {
-        if (!this._dirty)
-        {
-            this.game.updates.add(this);
-        }
-
-        this._dirty = true;
-    },
-
     getColor: function (value)
     {
         return (value >> 16) + (value & 0xff00) + ((value & 0xff) << 16);
@@ -155,6 +131,34 @@ Phaser.Component.Color.prototype = {
 
 Object.defineProperties(Phaser.Component.Color.prototype, {
 
+    dirty: {
+
+        enumerable: true,
+
+        get: function ()
+        {
+            return this._dirty;
+        },
+
+        set: function (value)
+        {
+            if (value)
+            {
+                if (!this._dirty)
+                {
+                    this.game.updates.add(this);
+                }
+
+                this._dirty = true;
+            }
+            else
+            {
+                this._dirty = false;
+            }
+        }
+
+    },
+
     tintTopLeft: {
 
         enumerable: true,
@@ -168,7 +172,7 @@ Object.defineProperties(Phaser.Component.Color.prototype, {
         {
             this._tint.topLeft = value;
             this._glTint.topLeft = this.getColor(value);
-            this.setDirty();
+            this.dirty = true;
         }
 
     },
@@ -186,7 +190,7 @@ Object.defineProperties(Phaser.Component.Color.prototype, {
         {
             this._tint.topRight = value;
             this._glTint.topRight = this.getColor(value);
-            this.setDirty();
+            this.dirty = true;
         }
 
     },
@@ -204,7 +208,7 @@ Object.defineProperties(Phaser.Component.Color.prototype, {
         {
             this._tint.bottomLeft = value;
             this._glTint.bottomLeft = this.getColor(value);
-            this.setDirty();
+            this.dirty = true;
         }
 
     },
@@ -222,7 +226,7 @@ Object.defineProperties(Phaser.Component.Color.prototype, {
         {
             this._tint.bottomRight = value;
             this._glTint.bottomRight = this.getColor(value);
-            this.setDirty();
+            this.dirty = true;
         }
 
     },
@@ -257,7 +261,27 @@ Object.defineProperties(Phaser.Component.Color.prototype, {
             if (value !== this._alpha)
             {
                 this._alpha = value;
-                this.setDirty();
+                this.dirty = true;
+            }
+        }
+
+    },
+
+    blendMode: {
+
+        enumerable: true,
+
+        get: function ()
+        {
+            return this._blendMode;
+        },
+
+        set: function (value)
+        {
+            if (value !== this._blendMode && value >= 0 && value <= 16)
+            {
+                this._blendMode = value;
+                this.dirty = true;
             }
         }
 
@@ -294,7 +318,7 @@ Object.defineProperties(Phaser.Component.Color.prototype, {
             {
                 this._a = value;
                 this._hasBackground = true;
-                this.setDirty();
+                this.dirty = true;
             }
         }
 
@@ -315,7 +339,7 @@ Object.defineProperties(Phaser.Component.Color.prototype, {
             {
                 this._r = value | 0;
                 this._hasBackground = true;
-                this.setDirty();
+                this.dirty = true;
             }
         }
 
@@ -336,7 +360,7 @@ Object.defineProperties(Phaser.Component.Color.prototype, {
             {
                 this._g = value | 0;
                 this._hasBackground = true;
-                this.setDirty();
+                this.dirty = true;
             }
         }
 
@@ -357,7 +381,7 @@ Object.defineProperties(Phaser.Component.Color.prototype, {
             {
                 this._b = value | 0;
                 this._hasBackground = true;
-                this.setDirty();
+                this.dirty = true;
             }
         }
 

src/components/Transform.js

@@ -51,8 +51,10 @@ Phaser.Component.Transform = function (gameObject, x, y, scaleX, scaleY)
     this._worldScaleX = scaleX;
     this._worldScaleY = scaleY;
 
+    // this._glVertexDataX1 = 
+
     this._dirty = true;
-    
+   
     this.game.updates.add(this);
 
     //  The parent Transform (NOT the parent GameObject, although very often they are related)