Merge branch 'eulerscheZahl-TinyToggleModule' into 'master'

[FIX][SDK] ToggleModule serialised data is now smaller

See merge request codingame/game-engine!284

Author: CodinGameCommunity

engine/modules/toggle/src/main/java/com/codingame/gameengine/module/toggle/ToggleModule.java

@@ -2,6 +2,7 @@
 
 import java.util.HashMap;
 import java.util.Map;
+import java.util.Map.Entry;
 
 import com.codingame.gameengine.core.AbstractPlayer;
 import com.codingame.gameengine.core.GameManager;
@@ -13,9 +14,9 @@
 
 /**
  * @author Jean Porée
- * 
+ *
  *         This module allows you to display or hide elements of the GraphicEntityModule using the viewer's options menu.
- * 
+ *
  */
 @Singleton
 public class ToggleModule implements Module {
@@ -67,20 +68,29 @@ public void onAfterGameTurn() {
     }
 
     @Override
-    public void onAfterOnEnd() {}
+    public void onAfterOnEnd() {
+    }
 
     private void sendFrameData() {
-        Object[] data = { newRegistration };
-        gameManager.setViewData("toggles", data);
-
-        newRegistration.clear();
+        if (newRegistration.size() > 0) {
+            Map<String, String> data = new HashMap<>();
+            newRegistration.forEach((entityId, toggle) -> {
+                data.put(toggle.name, data.getOrDefault(toggle.name, "") + entityId + (toggle.state ? "+" : "-"));
+            });
+            gameManager.setViewData("toggles", data);
+            newRegistration.clear();
+        }
     }
+
     /**
      * Will display the entity only when the toggle state matches the state you set
-     * 
-     * @param entity which will be displayed
-     * @param toggle the name of the toggle you want to use
-     * @param state the state of the toggle where the entity will be displayed at
+     *
+     * @param entity
+     *            which will be displayed
+     * @param toggle
+     *            the name of the toggle you want to use
+     * @param state
+     *            the state of the toggle where the entity will be displayed at
      */
     public void displayOnToggleState(Entity<?> entity, String toggle, boolean state) {
         int id = entity.getId();
@@ -90,4 +100,4 @@ public void displayOnToggleState(Entity<?> entity, String toggle, boolean state)
             registered.put(id, associatedToggle);
         }
     }
-}
+}

engine/modules/toggle/src/main/resources/view/toggle-module/ToggleModule.js

@@ -55,10 +55,19 @@ export class ToggleModule {
   }
 
   handleFrameData (frameInfo, data) {
-    if (!data) {
-      return
+    const newRegistration = {}
+    if (data) {
+      Object.entries(data).forEach(([key, value]) => {
+        value.match(/\d+./g).forEach(m => {
+          const entityId = m.slice(0, -1)
+          const state = m.slice(-1) === '+'
+          newRegistration[entityId] = {
+            name: key,
+            state: state
+          }
+        })
+      })
     }
-    const newRegistration = data[0]
     const registered = { ...this.previousFrame.registered, ...newRegistration }
     const frame = { registered, number: frameInfo.number }
     this.previousFrame = frame
@@ -81,7 +90,7 @@ function checkDuplicates (option) {
     let matchedPair = ToggleModule.optionValues[option.toggle].find(elem => elem.value === value)
 
     if (!matchedPair) {
-      matchedPair = { keys: [ ], value }
+      matchedPair = { keys: [], value }
       ToggleModule.optionValues[option.toggle].push(matchedPair)
     }
 

playground/misc/misc-3-release-notes.md

@@ -7,6 +7,7 @@ The CodinGame SDK is regularly updated and improved. This document lets you know
 ### 🐞 Bug fix
 
 - The statement preview page now presents HTML as it would show in the CodinGame IDE
+- Fixed `ToggleModule` taking up a lot of characters in the game result from the allowed quota.
 
 ## 3.15.2