From 53e478e4603ff3d076b73c8fbe5551198d019c13 Mon Sep 17 00:00:00 2001
From: Mark Ross <krazkidd@gmail.com>
Date: Mon, 16 Oct 2023 08:32:18 -0700
Subject: [PATCH] Don't allow negative scores. (#25)

---
 components/scoreboard/ScoreboardTeamTile.vue | 13 +++++++++----
 firestore.rules                              |  4 ++--
 pages/scoreboard/[[id]].vue                  |  6 ++----
 3 files changed, 13 insertions(+), 10 deletions(-)

diff --git a/components/scoreboard/ScoreboardTeamTile.vue b/components/scoreboard/ScoreboardTeamTile.vue
index fa94b22..ddd49b0 100644
--- a/components/scoreboard/ScoreboardTeamTile.vue
+++ b/components/scoreboard/ScoreboardTeamTile.vue
@@ -13,10 +13,15 @@ const props = withDefaults(defineProps<{
 });
 
 const emit = defineEmits<{
-    (e: "addRun"): void;
-    (e: "removeRun"): void;
+    (e: "incrementScore", payload: number): void;
 }>();
 
+function onClick(increment: number) {
+  if (props.score + increment >= 0) {
+    emit('incrementScore', increment);
+  }
+}
+
 const cardPassThroughOptions = computed(() => ({
   root: {
     class: 'accent m-1 w-64',
@@ -51,7 +56,7 @@ const buttonPassThroughOptions = {
         <template #content>
             <div class="flex justify-around">
                 <Button
-                    @click="emit('addRun')"
+                    @click="onClick(1)"
                     icon="pi pi-caret-up"
                     severity="success"
                     rounded
@@ -62,7 +67,7 @@ const buttonPassThroughOptions = {
                 />
 
                 <Button
-                    @click="emit('removeRun')"
+                    @click="onClick(-1)"
                     icon="pi pi-caret-down"
                     severity="danger"
                     rounded
diff --git a/firestore.rules b/firestore.rules
index 10d7255..bc62683 100644
--- a/firestore.rules
+++ b/firestore.rules
@@ -27,8 +27,8 @@ service cloud.firestore {
       // only allow writing Scoreboard type
       allow write:
         if request.resource.data.size() == 2
-        && request.resource.data.teamScore is int
-        && request.resource.data.otherTeamScore is int;
+        && request.resource.data.teamScore is int && request.resource.data.teamScore >= 0
+        && request.resource.data.otherTeamScore is int && request.resource.data.otherTeamScore >= 0;
     }
 
     match /emotes/{id} {
diff --git a/pages/scoreboard/[[id]].vue b/pages/scoreboard/[[id]].vue
index 0c0f3a8..7cd113c 100644
--- a/pages/scoreboard/[[id]].vue
+++ b/pages/scoreboard/[[id]].vue
@@ -45,8 +45,7 @@ const fakeTeam = {
                 :team="team"
                 :score="scoreboard?.teamScore ?? 0"
                 :show-buttons="!route.params.id"
-                @add-run="incrementTeamScore(1)"
-                @remove-run="incrementTeamScore(-1)"
+                @increment-score="incrementTeamScore($event)"
                 class="w-64"
             />
 
@@ -56,8 +55,7 @@ const fakeTeam = {
                 :team="fakeTeam"
                 :score="scoreboard?.otherTeamScore ?? 0"
                 :show-buttons="!route.params.id"
-                @add-run="incrementOtherTeamScore(1)"
-                @remove-run="incrementOtherTeamScore(-1)"
+                @increment-score="incrementOtherTeamScore($event)"
                 class="w-64"
             />
         </div>