From: Benjamin Auder <benjamin.auder@somewhere>
Date: Sun, 18 Apr 2021 21:15:45 +0000 (+0200)
Subject: Small fixes
X-Git-Url: https://git.auder.net/DESCRIPTION?a=commitdiff_plain;h=2a2936023cda0888e215ff699d420090372899b0;p=vchess.git

Small fixes
---

diff --git a/client/src/utils/printDiagram.js b/client/src/utils/printDiagram.js
index ebc4d114..ef0d2b55 100644
--- a/client/src/utils/printDiagram.js
+++ b/client/src/utils/printDiagram.js
@@ -73,6 +73,7 @@ export function getDiagram(args) {
   // Obtain the array of pieces images names:
   const board = V.GetBoard(args.position);
   const orientation = args.orientation || "w";
+  const darkBottomRight = !!args.darkBottomRight;
   const markArray = getMarkArray(args.marks);
   const shadowArray = getShadowArray(args.shadow);
   const vr = new V(); //just for pieces images paths
diff --git a/client/src/views/Game.vue b/client/src/views/Game.vue
index 0e430fe2..2968b8f8 100644
--- a/client/src/views/Game.vue
+++ b/client/src/views/Game.vue
@@ -1618,7 +1618,8 @@ export default {
         if (["all","byrow"].includes(V.ShowMoves)) {
           this.curDiag = getDiagram({
             position: position,
-            orientation: V.CanFlip ? this.game.mycolor : "w"
+            orientation: V.CanFlip ? this.game.mycolor : "w",
+            darkBottomRight: V.DarkBottomRight
           });
           document.querySelector("#confirmDiv > .card").style.width =
             boardDiv.offsetWidth + "px";
diff --git a/client/src/views/Hall.vue b/client/src/views/Hall.vue
index bdb28050..c8ca3e6d 100644
--- a/client/src/views/Hall.vue
+++ b/client/src/views/Hall.vue
@@ -1097,16 +1097,18 @@ export default {
       }
       window.V = this.newchallenge.V;
       let chall = Object.assign({}, this.newchallenge, { options: {} });
-      // Get/set options variables (if any) / TODO: v-model?!
-      for (const check of this.newchallenge.V.Options.check || []) {
-        const elt = document.getElementById(check.variable + "_opt");
-        chall.options[check.variable] = elt.checked;
-      }
-      for (const select of this.newchallenge.V.Options.select || []) {
-        const elt = document.getElementById(select.variable + "_opt");
-        const tryIntVal = parseInt(elt.value, 10);
-        chall.options[select.variable] =
-          (isNaN(tryIntVal) ? elt.value : tryIntVal);
+      if (V.Options) {
+        // Get/set options variables (if any) / TODO: v-model?!
+        for (const check of this.newchallenge.V.Options.check || []) {
+          const elt = document.getElementById(check.variable + "_opt");
+          chall.options[check.variable] = elt.checked;
+        }
+        for (const select of this.newchallenge.V.Options.select || []) {
+          const elt = document.getElementById(select.variable + "_opt");
+          const tryIntVal = parseInt(elt.value, 10);
+          chall.options[select.variable] =
+            (isNaN(tryIntVal) ? elt.value : tryIntVal);
+        }
       }
       error = checkChallenge(chall);
       if (error) {