Fixed internal callback handlers

Author: photonstorm

src/gameobjects/group/Group.js

@@ -223,7 +223,7 @@ var Group = new Class({
          * @private
          * @since 3.22.0
          */
-        this.internalCreateCallback = null;
+        this.internalCreateCallback = GetFastValue(config, 'internalCreateCallback', null);
 
         /**
          * A function to be called when removing group members.
@@ -234,7 +234,7 @@ var Group = new Class({
          * @private
          * @since 3.22.0
          */
-        this.internalRemoveCallback = null;
+        this.internalRemoveCallback = GetFastValue(config, 'internalRemoveCallback', null);
 
         if (config)
         {

src/physics/arcade/PhysicsGroup.js

@@ -38,20 +38,42 @@ var PhysicsGroup = new Class({
 
     function PhysicsGroup (world, scene, children, config)
     {
-        if (IsPlainObject(children))
+        if (!children && !config)
+        {
+            config = {
+                internalCreateCallback: this.createCallbackHandler,
+                internalRemoveCallback: this.removeCallbackHandler
+            };
+        }
+        else if (IsPlainObject(children))
         {
             //  children is a plain object, so swizzle them:
             config = children;
             children = null;
+
+            config.internalCreateCallback = this.createCallbackHandler;
+            config.internalRemoveCallback = this.removeCallbackHandler;
         }
         else if (Array.isArray(children) && IsPlainObject(children[0]))
         {
             //  children is an array of plain objects
             config = children[0];
+
+            var _this = this;
+
+            children.forEach(function (singleConfig)
+            {
+                singleConfig.internalCreateCallback = _this.createCallbackHandler;
+                singleConfig.internalRemoveCallback = _this.removeCallbackHandler;
+            });
         }
-        else if (config === undefined || children === undefined)
+        else
         {
-            config = {};
+            // config is not defined and children is not a plain object nor an array of plain objects
+            config = {
+                internalCreateCallback: this.createCallbackHandler,
+                internalRemoveCallback: this.removeCallbackHandler
+            };
         }
 
         /**
@@ -125,9 +147,6 @@ var PhysicsGroup = new Class({
 
         Group.call(this, scene, children, config);
 
-        this.internalCreateCallback = this.createCallbackHandler;
-        this.internalRemoveCallback = this.removeCallbackHandler;
-
         /**
          * A textual representation of this Game Object.
          * Used internally by Phaser but is available for your own custom classes to populate.

src/physics/arcade/StaticPhysicsGroup.js

@@ -41,8 +41,8 @@ var StaticPhysicsGroup = new Class({
         if (!children && !config)
         {
             config = {
-                createCallback: this.createCallbackHandler,
-                removeCallback: this.removeCallbackHandler,
+                internalCreateCallback: this.createCallbackHandler,
+                internalRemoveCallback: this.removeCallbackHandler,
                 createMultipleCallback: this.createMultipleCallbackHandler,
                 classType: ArcadeSprite
             };
@@ -53,8 +53,8 @@ var StaticPhysicsGroup = new Class({
             config = children;
             children = null;
 
-            config.createCallback = this.createCallbackHandler;
-            config.removeCallback = this.removeCallbackHandler;
+            config.internalCreateCallback = this.createCallbackHandler;
+            config.internalRemoveCallback = this.removeCallbackHandler;
             config.createMultipleCallback = this.createMultipleCallbackHandler;
             config.classType = GetFastValue(config, 'classType', ArcadeSprite);
         }
@@ -66,8 +66,8 @@ var StaticPhysicsGroup = new Class({
 
             config.forEach(function (singleConfig)
             {
-                singleConfig.createCallback = this.createCallbackHandler;
-                singleConfig.removeCallback = this.removeCallbackHandler;
+                singleConfig.internalCreateCallback = this.createCallbackHandler;
+                singleConfig.internalRemoveCallback = this.removeCallbackHandler;
                 singleConfig.createMultipleCallback = this.createMultipleCallbackHandler;
                 singleConfig.classType = GetFastValue(singleConfig, 'classType', ArcadeSprite);
             });