Merge branch '4001-BufferedGroups-optimisation' into 'master'

[FIX][SDK] BufferedGroup optimisation

See merge request codingame/game-engine!122

Author: Julien Poulton

engine/modules/entities/src/main/resources/view/entity-module/BufferedGroup.js

@@ -11,17 +11,28 @@ export class BufferedGroup extends Group {
     flagForDestructionOnReinit(this.gameTexture)
     this.graphics = new PIXI.Sprite(this.gameTexture)
     this.buffer = new PIXI.Container()
+    this.needsRender = true
   }
 
   updateDisplay (state, changed, globalData) {
     super.updateDisplay(state, changed, globalData)
   }
 
   postUpdate () {
-    getRenderer().render(this.buffer, this.gameTexture)
+    if (this.needsRender) {
+      getRenderer().render(this.buffer, this.gameTexture)
+      this.needsRender = false
+    }
   }
 
   get childrenContainer () {
     return this.buffer
   }
+
+  notifyChange () {
+    this.needsRender = true
+    if (this.parent) {
+      this.parent.notifyChange()
+    }
+  }
 }

engine/modules/entities/src/main/resources/view/entity-module/Entity.js

@@ -110,12 +110,21 @@ export class Entity {
         if (changed.mask) {
           globalData.maskUpdates[this.id] = state.mask
         }
+        if (Object.keys(changed).length !== 0 && this.parent) {
+          this.parent.notifyChange()
+        }
       }
     } else {
       Object.assign(this.currentState, this.defaultState)
     }
   }
 
+  notifyChange () {
+    if (this.parent) {
+      this.parent.notifyChange()
+    }
+  }
+
   initDisplay () {
     this.container = new PIXI.Container()
     this.container.zIndex = this.defaultState.zIndex

engine/modules/entities/src/main/resources/view/entity-module/GraphicEntityModule.js

@@ -173,7 +173,9 @@ export class GraphicEntityModule {
       if (e instanceof Group) {
         e.childrenContainer.removeChildren()
         e.currentState.children.forEach(id => {
-          e.childrenContainer.addChild(this.entities.get(id).container)
+          const child = this.entities.get(id)
+          e.childrenContainer.addChild(child.container)
+          child.parent = e
         })
       }
     })