Group.remove now checks the child to see if it's a member of the root Group before removing it, otherwise Pixi throws an Error.

Author: photonstorm

README.md

@@ -68,7 +68,7 @@ Version 2.0.4 - "Mos Shirare" - in development
 * Group.setProperty will now check if the property exists before setting it, this applies to Group.setAll and anything else using setProperty internally.
 * QuadTree.retrieve now checks to see if the given Sprite has a body before carrying on.
 * ArcadePhysics.collideSpriteVsGroup checks if Sprite has a body before carrying on, now safely skips sub-groups or other non-Sprite group children. 
-
+* Group.remove now checks the child to see if it's a member of the root Group before removing it, otherwise Pixi throws an Error.
 
 
 ### New Features

src/core/Group.js

@@ -1413,7 +1413,8 @@ Phaser.Group.prototype.getRandom = function (startIndex, length) {
 };
 
 /**
-* Removes the given child from this Group and sets its group property to null.
+* Removes the given child from this Group. This will dispatch an onRemovedFromGroup event from the child (if it has one),
+* reset the Group cursor and optionally destroy the child.
 *
 * @method Phaser.Group#remove
 * @param {Any} child - The child to remove.
@@ -1424,7 +1425,7 @@ Phaser.Group.prototype.remove = function (child, destroy) {
 
     if (typeof destroy === 'undefined') { destroy = false; }
 
-    if (this.children.length === 0)
+    if (this.children.length === 0 || this.children.indexOf(child) === -1)
     {
         return false;
     }