From 343731e7e894f25c317bdc7b7e1a51a2a4a61f4b Mon Sep 17 00:00:00 2001
From: ntr <ntr@smokestack.io>
Date: Sun, 27 Oct 2019 14:12:38 +1100
Subject: [PATCH] draw fixes and client side showing

---
 client/src/components/game.ctrl.btns.top.jsx | 14 +++++++++++---
 client/src/components/player.box.jsx         |  9 +++++++--
 server/src/instance.rs                       | 19 +------------------
 3 files changed, 19 insertions(+), 23 deletions(-)

diff --git a/client/src/components/game.ctrl.btns.top.jsx b/client/src/components/game.ctrl.btns.top.jsx
index 4b81670c..8e09718c 100644
--- a/client/src/components/game.ctrl.btns.top.jsx
+++ b/client/src/components/game.ctrl.btns.top.jsx
@@ -8,6 +8,7 @@ const addState = connect(
         const {
             ws,
             game,
+            account,
         } = state;
 
         function sendAbandon() {
@@ -20,6 +21,7 @@ const addState = connect(
 
         return {
             game,
+            account,
 
             sendAbandon,
             sendDraw,
@@ -39,6 +41,7 @@ const addState = connect(
 function GameCtrlTopBtns(args) {
     const {
         game,
+        account,
 
         leave,
         sendAbandon,
@@ -48,6 +51,9 @@ function GameCtrlTopBtns(args) {
     const finished = game && game.phase === 'Finished';
     const { abandonState, drawState } = this.state;
 
+    const player = game.players.find(p => p.id === account.id);
+    const drawOffered = player && player.draw_offered;
+
     const abandonStateTrue = e => {
         e.stopPropagation();
         this.setState({ abandonState: true });
@@ -66,10 +72,12 @@ function GameCtrlTopBtns(args) {
 
     const abandonBtn = <button class={abandonClasses} disabled={finished} onClick={abandonAction}>{abandonText}</button>;
 
-    const drawClasses = `draw ${drawState ? 'confirming' : ''}`;
-    const drawText = drawState ? 'Draw' : 'Offer';
+    const drawClasses = `draw ${drawState || drawOffered ? 'confirming' : ''}`;
+    const drawText = drawOffered
+        ? 'Offered'
+        : drawState ? 'Draw' : 'Offer';
     const drawAction = drawState ? sendDraw : drawStateTrue;
-    const drawBtn = <button class={drawClasses} disabled={finished} onClick={drawAction}>{drawText}</button>;
+    const drawBtn = <button class={drawClasses} disabled={finished || drawOffered} onClick={drawAction}>{drawText}</button>;
 
     return (
         <div class="instance-ctrl-btns">
diff --git a/client/src/components/player.box.jsx b/client/src/components/player.box.jsx
index 2790d69f..9d55606e 100644
--- a/client/src/components/player.box.jsx
+++ b/client/src/components/player.box.jsx
@@ -68,6 +68,11 @@ function Scoreboard(args) {
     };
 
     const winner = player.score === 'Win';
+    const chatText = player.draw_offered
+        ? 'draw?'
+        : chat || '\u00A0';
+
+    console.log(chatText);
 
     if (!isPlayer) {
         const nameClass = `name ${player.img ? 'subscriber' : ''}`;
@@ -77,7 +82,7 @@ function Scoreboard(args) {
                 <div class="score">{scoreText()}</div>
                 <div class={nameClass}>{player.name}</div>
                 <Img img={player.img} id={player.id} />
-                <div class="msg">{chat || '\u00A0'}</div>
+                <div class="msg">{chatText}</div>
             </div>
         );
     }
@@ -87,7 +92,7 @@ function Scoreboard(args) {
 
     return (
         <div class={boxClass}>
-            <div class="msg">{chat || '\u00A0'}</div>
+            <div class="msg">{chatText}</div>
             <div class="score">{scoreText()}</div>
             <div class={nameClass}>{player.name}</div>
             <Img img={player.img} id={player.id} />
diff --git a/server/src/instance.rs b/server/src/instance.rs
index eec93a0d..07498300 100644
--- a/server/src/instance.rs
+++ b/server/src/instance.rs
@@ -335,24 +335,7 @@ impl Instance {
     }
 
     fn finish_condition(&mut self) -> bool {
-        // tennis
-        for player in self.players.iter() {
-            if player.score == Score::Win {
-                self.winner = Some(player.id);
-                return true;
-            }
-        }
-        // Game defaults to lose otherwise
-        if self.rounds.len() < 4 {
-            return false;
-        }
-
-        // both players afk
-        if self.players.iter().all(|p| p.score == Score::Zero) {
-            return true;
-        }
-
-        return false;
+        self.players.iter().any(|p| p.score == Score::Win)
     }
 
     pub fn finish(&mut self) -> &mut Instance {