Merge branch 'better-error-on-demo-bug' into 'master'

[FIX][sdk] Better error on demo bug

See merge request codingame/game-engine!139

Author: Julien Poulton

engine/core/src/main/resources/view/core/Drawer.js

@@ -215,14 +215,19 @@ export class Drawer {
       }
 
       var demoContainer = new PIXI.Container()
-
-      this.initDefaultFrames(this.demo.playerCount, this.demo.frames, this.demo.agents)
-      /** **************************************************************************************************************************************** */
-      this.preconstructScene(this.scope, container, this.initWidth, this.initHeight)
-      this.initScene(this.scope, demoContainer, this.frames, true)
-      this.updateScene(this.scope, this.question, this.frames, this.currentFrame, this.progress, 1, this.reasons[this.currentFrame], true)
-      /** **************************************************************************************************************************************** */
-
+      try {
+        this.initDefaultFrames(this.demo.playerCount, this.demo.frames, this.demo.agents)
+        /** **************************************************************************************************************************************** */
+        this.preconstructScene(this.scope, container, this.initWidth, this.initHeight)
+        this.initScene(this.scope, demoContainer, this.frames, true)
+        this.updateScene(this.scope, this.question, this.frames, this.currentFrame, this.progress, 1, this.reasons[this.currentFrame], true)
+        /** **************************************************************************************************************************************** */
+      } catch (error) {
+        ErrorLog.push({
+          cause: error,
+          message: 'Cannot load demo, you might want to reset the demo'
+        })
+      }
       scope.demo = demoContainer
       scope.demotime = 0
 

runner/src/main/resources/view/test.css

@@ -129,7 +129,18 @@ button {
     font-weight: normal;
     word-break: break-word;
 }
-
+.outputs-demo {
+    height: calc(100% - 20px);
+    padding: 10px 15px;
+    background-color: #29323c;
+    border: solid 1px #363e48;
+    overflow: auto;
+    white-space: pre-wrap;
+    font-size: 13px;
+    font-weight: normal;
+    word-break: break-word;
+    color: white;
+}
 .players-outputs-player-console-title, .game-outputs-title, .game-outputs-down-referee-title, .players-outputs-player-header-title {
     font-size: 13px;
     font-weight: 600;

runner/src/main/resources/view/test.html

@@ -53,7 +53,8 @@
 					shareable="false"
 					show-replay-prompt="false"
 					game-params="gameParams">
-				</cg-player>			
+				</cg-player>
+				<div id="console" class="outputs-demo" ng-bind="errors" ng-if="errors"></div>
 				<div class="close-button-wrapper">
 					<button class="close-button" ng-click="closeViewReplayPopup()">Close</button>
 				</div>