From d41f07017281460974504a3b687e4082bf6aa96f Mon Sep 17 00:00:00 2001
From: Julian L <julian.lawrance@gmail.com>
Date: Sat, 6 Jul 2024 22:17:24 +0100
Subject: [PATCH] Fix bug where assigning a table and first breaker to the
 first fixture in a round robin  resulted in that fixture being de-prioritised
 when the second fixture was assigned to a table

---
 src/components/play/FixtureCard.vue | 4 ++++
 src/composables/usePhase.ts         | 4 +++-
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/src/components/play/FixtureCard.vue b/src/components/play/FixtureCard.vue
index 30ad921..72bbea5 100644
--- a/src/components/play/FixtureCard.vue
+++ b/src/components/play/FixtureCard.vue
@@ -79,6 +79,10 @@ const fixtureCardClass = (fixture: Fixture) => [
         v-if="fixture"
         class="border-round-md border-1"
         :class="fixtureCardClass(fixture)">
+        <div v-if="uiStore.isDebugMode" class="text-center">
+            {{ fixture.id }}
+        </div>
+
         <div v-if="table && !fixture.finishTime" class="text-center">
             <TableBadge :table="table" />
         </div>
diff --git a/src/composables/usePhase.ts b/src/composables/usePhase.ts
index aa78ea1..7b2dafa 100644
--- a/src/composables/usePhase.ts
+++ b/src/composables/usePhase.ts
@@ -73,7 +73,9 @@ export const usePhase = (p: Phase | null) => {
         const status = getFixtureStatus(fixture)
 
         const validStatuses = [
-            FixtureStatus.ReadyToStart,
+            // TODO: FixtureStatus.ReadyToStart, but only if this fixture would
+            // be brought forwards in time. Probably needs a check in the place
+            // that's calling this function...
             FixtureStatus.WaitingForAssignment,
         ]