Fixed World.scale and Group.scale.

Author: photonstorm

README.md

@@ -54,7 +54,9 @@ Updates:
 
 Bug Fixes:
 
-* TBC
+* Cache.getImageKeys returned __missing in the array, now excluded.
+* Fixed Group.scale so you can now scale a Group directly.
+* Removed World.scale as it was preventing Group.scale from working - you can still scale the world, but you'll need to factor in Input changes yourself.
 
 
 You can view the Change Log for all previous versions at https://github.com/photonstorm/phaser/changelog.md

examples/groups/bring a child to top.js

@@ -2,12 +2,6 @@ var game = new Phaser.Game(800, 600, Phaser.CANVAS, 'phaser-example', { preload:
 
 function preload() {
 
-    // Enable scaling 
-    game.stage.scaleMode = Phaser.StageScaleMode.SHOW_ALL;
-    game.stage.scale.maxWidth = 1024;
-    game.stage.scale.maxHeight = 672;
-    game.stage.scale.refresh();
-
     game.load.image('atari1', 'assets/sprites/atari130xe.png');
     game.load.image('atari2', 'assets/sprites/atari800xl.png');
     game.load.image('atari4', 'assets/sprites/atari800.png');

examples/groups/bring a group to top.js

@@ -31,7 +31,6 @@ function create() {
     //  Now let's create some random sprites and enable them all for drag and 'bring to top'
     for (var i = 0; i < 10; i++)
     {
-
         var tempSprite = game.add.sprite(game.stage.randomX, game.stage.randomY, 'atari1');
 
         tempSprite.name = 'atari' + i;

src/PixiPatch.js

@@ -79,8 +79,8 @@ PIXI.CanvasRenderer.prototype.renderDisplayObject = function(displayObject)
                     displayObject.texture.frame.y,
                     displayObject.texture.frame.width,
                     displayObject.texture.frame.height,
-                    (displayObject.anchor.x) * -displayObject.texture.frame.width,
-                    (displayObject.anchor.y) * -displayObject.texture.frame.height,
+                    Math.floor((displayObject.anchor.x) * -displayObject.texture.frame.width),
+                    Math.floor((displayObject.anchor.y) * -displayObject.texture.frame.height),
                     displayObject.texture.frame.width,
                     displayObject.texture.frame.height);
             }

src/core/Group.js

@@ -77,10 +77,13 @@ Phaser.Group = function (game, parent, name, useStage) {
     */
     this.exists = true;
 
+    //  Replaces the PIXI.Point with a slightly more flexible one.
+    this._container.scale = new Phaser.Point(1, 1);
+
     /**
-    * @property {Phaser.Point} scale - Replaces the PIXI.Point with a slightly more flexible one.
+    * @property {Phaser.Point} scale - The scane of the Group container.
     */
-    this.scale = new Phaser.Point(1, 1);
+    this.scale = this._container.scale;
 
     /**
     * The cursor is a simple way to iterate through the objects in a Group using the Group.next and Group.previous functions.

src/core/World.js

@@ -12,18 +12,14 @@
 * the world at world-based coordinates. By default a world is created the same size as your Stage.
 *
 * @class Phaser.World
+* @extends Phaser.Group
 * @constructor
 * @param {Phaser.Game} game - Reference to the current game instance.
 */
 Phaser.World = function (game) {
 
     Phaser.Group.call(this, game, null, '__world', false);
 
-    /**
-    * @property {Phaser.Point} scale - Replaces the PIXI.Point with a slightly more flexible one.
-    */
-    this.scale = new Phaser.Point(1, 1);
-
     /**
     * The World has no fixed size, but it does have a bounds outside of which objects are no longer considered as being "in world" and you should use this to clean-up the display list and purge dead objects.
     * By default we set the Bounds to be from 0,0 to Game.width,Game.height. I.e. it will match the size given to the game constructor with 0,0 representing the top-left of the display.

src/gameobjects/Sprite.js

@@ -1101,8 +1101,6 @@ Object.defineProperty(Phaser.Sprite.prototype, "inputEnabled", {
 
     set: function (value) {
 
-        console.log('inputEnabled', value, this.input);
-
         if (value)
         {
             if (this.input.enabled === false)

src/input/InputHandler.js

@@ -28,36 +28,6 @@ Phaser.InputHandler = function (sprite) {
     */
     this.enabled = false;
 
-    /**
-    * @property {Description} parent - Description. 
-    * @default
-    */
-    // this.parent = null;
-    
-    /**
-    * @property {Description} next - Linked List
-    * @default
-    */
-    // this.next = null;
-    
-    /**
-    * @property {Description} prev - Description. 
-    * @default
-    */
-    // this.prev = null;
-    
-    /**
-    * @property {Description} last - Description. 
-    * @default
-    */
-    // this.last = this;
-    
-    /**
-    * @property {Description} first - Description. 
-    * @default
-    */
-    // this.first = this;
-
     /**
     * @property {number} priorityID - The PriorityID controls which Sprite receives an Input event first if they should overlap.
     * @default
@@ -199,8 +169,6 @@ Phaser.InputHandler.prototype = {
     */
     start: function (priority, useHandCursor) {
 
-        console.log('InputHandler start');
-
         priority = priority || 0;
         if (typeof useHandCursor == 'undefined') { useHandCursor = false; }
 

src/loader/Cache.js

@@ -745,7 +745,7 @@ Phaser.Cache.prototype = {
 
         for (var item in array)
         {
-            if (item !== '__default')
+            if (item !== '__default' && item !== '__missing')
             {
                 output.push(item);
             }

src/net/Net.js

@@ -128,8 +128,8 @@ Phaser.Net.prototype = {
         var output = {};
         var keyValues = location.search.substring(1).split('&');
 
-        for (var i in keyValues) {
-
+        for (var i in keyValues)
+        {
             var key = keyValues[i].split('=');
 
             if (key.length > 1)