From 386a9a7bb200609dc3795c494830f393781b2781 Mon Sep 17 00:00:00 2001
From: Brandon Presley <brandon@presley.nz>
Date: Tue, 29 Aug 2023 11:25:05 +1200
Subject: [PATCH] Pass whole updated set instead of just its id

Since we already have the whole updated set,
might as well pass it around instead of
re-fetching it.
---
 EditSet.tsx         | 2 +-
 SetList.tsx         | 7 ++++---
 home-page-params.ts | 4 ++--
 3 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/EditSet.tsx b/EditSet.tsx
index b32d49a..a4d4296 100644
--- a/EditSet.tsx
+++ b/EditSet.tsx
@@ -105,7 +105,7 @@ export default function EditSet() {
     const saved = await setRepo.save(newSet);
     if (typeof set.id !== "number") return added(saved);
     if (createdDirty) navigate("Sets", { reset: saved.id });
-    else navigate("Sets", { refresh: saved.id });
+    else navigate("Sets", { refresh: saved });
   };
 
   const changeImage = useCallback(async () => {
diff --git a/SetList.tsx b/SetList.tsx
index c1d24fd..2aa68a0 100644
--- a/SetList.tsx
+++ b/SetList.tsx
@@ -29,10 +29,11 @@ export default function SetList() {
   const { params } = useRoute<RouteProp<HomePageParams, "Sets">>();
   const [term, setTerm] = useState(params?.search || "");
 
-  const refresh = async (id: number) => {
+  const refresh = async (gymSet: GymSet) => {
     if (!sets) return;
-    const set = await setRepo.findOne({ where: { id } });
-    let newSets = sets.map((oldSet) => (oldSet.id === id ? set : oldSet));
+    const newSets = sets.map((oldSet) =>
+      oldSet.id === gymSet.id ? gymSet : oldSet
+    );
     setSets(newSets);
   };
 
diff --git a/home-page-params.ts b/home-page-params.ts
index e016ab5..3f972c8 100644
--- a/home-page-params.ts
+++ b/home-page-params.ts
@@ -5,9 +5,9 @@ export type HomePageParams = {
     search?: string;
 
     /**
-     * Reload the specified set by ID.
+     * Update the specified set
      */
-    refresh?: number;
+    refresh?: GymSet;
 
     /**
      * Reload the list with limit = 0