From ed01fb3a68eeb0020eb629d156ca83b3f9349d3e Mon Sep 17 00:00:00 2001
From: Christian Lentfort <1284808+clentfort@users.noreply.github.com>
Date: Mon, 29 Jan 2024 11:39:30 +0100
Subject: [PATCH] fix: correctly handle type on plan items

---
 src/api/types.ts                             |   3 +-
 src/app/(app)/current-plan/[id].tsx          | 105 +++----------------
 src/app/(app)/current-plan/exercise/[id].tsx |   6 +-
 src/app/(app)/plan/[id].tsx                  |  52 +--------
 src/components/plan-items.tsx                | 101 ++++++++++++++++++
 5 files changed, 120 insertions(+), 147 deletions(-)
 create mode 100644 src/components/plan-items.tsx

diff --git a/src/api/types.ts b/src/api/types.ts
index a2a51bc..bbe9ecd 100644
--- a/src/api/types.ts
+++ b/src/api/types.ts
@@ -74,12 +74,13 @@ export interface PlanItem extends ParseObject {
   exercise: ParsePointer<"Exercise", Exercise>;
   finishedSets: PlanItemSet[];
   history: PlanItemSetHistory[];
+  name?: string;
   note: string;
   openSets: PlanItemSet[];
   plan: ParsePointer<"Plan", Plan>;
   position: number;
   sets: PlanItemSet[];
-  type: string;
+  type: "TEXT" | "EXERCISE";
 }
 
 interface UserExtended extends ParseObject {
diff --git a/src/app/(app)/current-plan/[id].tsx b/src/app/(app)/current-plan/[id].tsx
index f95e913..121903f 100644
--- a/src/app/(app)/current-plan/[id].tsx
+++ b/src/app/(app)/current-plan/[id].tsx
@@ -1,24 +1,15 @@
-import {
-  Button,
-  Card,
-  List,
-  ListItem,
-  ProgressBar,
-  Text,
-  useTheme,
-} from "@ui-kitten/components";
+import { Button, ProgressBar, Text } from "@ui-kitten/components";
 import { router, useLocalSearchParams } from "expo-router";
 import React from "react";
-import { Alert, Image, View } from "react-native";
+import { Alert } from "react-native";
 
-import { PlanItem as PlanItemType } from "@/api/types";
+import { PlanItem } from "@/api/types";
 import useObject from "@/api/use-object";
 import useObjects from "@/api/use-objects";
 import useUpdateObject from "@/api/use-update-object";
 import PageLayout from "@/components/page-layout";
+import PlanItems from "@/components/plan-items";
 import useAsyncStorage from "@/hooks/use-async-storage";
-import { PartiallyResolvedPointer } from "@/parse-client/pointer";
-import { space } from "@/styles";
 
 export default function PlanScreen() {
   const planId = useLocalSearchParams<{ id: string }>().id!;
@@ -43,8 +34,9 @@ export default function PlanScreen() {
     return <Text>Loading</Text>;
   }
 
+  const exercises = planItems.results.filter(({ type }) => type === "EXERCISE");
   const done = new Set(
-    planItems.results
+    exercises
       .filter((item) => item.openSets.length === 0)
       .map((item) => item.objectId),
   );
@@ -63,7 +55,7 @@ export default function PlanScreen() {
   };
 
   const handleEndSession = () => {
-    if (done.size < planItems.results.length) {
+    if (done.size < exercises.length) {
       Alert.alert(
         "Training beenden?",
         `Du hast erst ${done.size} von ${planItems.results.length} Übungen absolviert. Training wirklich beenden?`,
@@ -74,6 +66,10 @@ export default function PlanScreen() {
     }
   };
 
+  const handleItemPress = (item: PlanItem) => {
+    router.push(`/current-plan/exercise/${item.objectId}`);
+  };
+
   return (
     <PageLayout
       style={{
@@ -85,18 +81,7 @@ export default function PlanScreen() {
         progress={done.size / planItems.results.length}
         status="success"
       />
-      <List
-        data={planItems.results}
-        style={{
-          backgroundColor: "transparent",
-          gap: 0,
-          margin: 0,
-          padding: 0,
-        }}
-        renderItem={({ item }) => {
-          return <PlanItem item={item} />;
-        }}
-      />
+      <PlanItems items={planItems.results} onItemPress={handleItemPress} />
       <Button
         status={done.size < planItems.results.length ? "danger" : "primary"}
         onPress={handleEndSession}
@@ -106,69 +91,3 @@ export default function PlanScreen() {
     </PageLayout>
   );
 }
-
-interface PlanItemProps {
-  item: PartiallyResolvedPointer<PlanItemType, "exercise">;
-}
-
-function PlanItem({ item }: PlanItemProps) {
-  const theme = useTheme();
-  const isDone = item.openSets.length === 0;
-  return (
-    <ListItem
-      key={item.objectId}
-      style={{
-        paddingHorizontal: space[0],
-        paddingVertical: space[1],
-      }}
-    >
-      <Card
-        style={{
-          width: "100%",
-          backgroundColor: isDone ? theme["color-success-500"] : undefined,
-        }}
-        onPress={() => {
-          router.push(`/current-plan/exercise/${item.objectId}`);
-        }}
-      >
-        <View style={{ flexDirection: "row", gap: space[4] }}>
-          <View
-            style={{
-              backgroundColor: "white",
-              padding: space[1],
-              borderRadius: space[2],
-            }}
-          >
-            <Image
-              style={{ width: 90, height: 60 }}
-              source={{ uri: item.exercise.defaultImageUrl }}
-            />
-          </View>
-          <View
-            style={{
-              flexGrow: 1,
-              gap: space[2],
-            }}
-          >
-            <View style={{ flex: 1, flexDirection: "row" }}>
-              <Text
-                style={{
-                  fontWeight: "bold",
-                  flex: 1,
-                  fontSize: 16,
-                  lineHeight: 18,
-                }}
-              >
-                {item.exercise.name}
-              </Text>
-            </View>
-            <Text style={{ flex: 1 }}>
-              {item.sets.length - item.openSets.length} / {item.sets.length}{" "}
-              Sets
-            </Text>
-          </View>
-        </View>
-      </Card>
-    </ListItem>
-  );
-}
diff --git a/src/app/(app)/current-plan/exercise/[id].tsx b/src/app/(app)/current-plan/exercise/[id].tsx
index 8ecd216..a94baa3 100644
--- a/src/app/(app)/current-plan/exercise/[id].tsx
+++ b/src/app/(app)/current-plan/exercise/[id].tsx
@@ -72,10 +72,8 @@ const DEFAULT_PAUSE_DURATION = 60 + 40;
 
 export default function ExercisePlanScreen() {
   const planItemId = useLocalSearchParams<{ id: string }>().id!;
-  const [pauseDuration, setPauseDuration] = useState(20);
-  const [pauseStartedAt, setPauseStartedAt] = useState<null | number>(
-    Date.now(),
-  );
+  const [pauseDuration, setPauseDuration] = useState(0);
+  const [pauseStartedAt, setPauseStartedAt] = useState<null | number>(null);
   const [nextPauseDuration, setNextPauseDuration] = useAsyncStorage(
     "pause-duration",
     DEFAULT_PAUSE_DURATION,
diff --git a/src/app/(app)/plan/[id].tsx b/src/app/(app)/plan/[id].tsx
index 9871d8c..d497be1 100644
--- a/src/app/(app)/plan/[id].tsx
+++ b/src/app/(app)/plan/[id].tsx
@@ -1,14 +1,13 @@
-import { Button, Card, List, ListItem, Text } from "@ui-kitten/components";
+import { Button, Text } from "@ui-kitten/components";
 import { router, useLocalSearchParams } from "expo-router";
 import React from "react";
-import { Image, View } from "react-native";
 
 import useObject from "@/api/use-object";
 import useObjects from "@/api/use-objects";
 import useUpdateObject from "@/api/use-update-object";
 import PageLayout from "@/components/page-layout";
+import PlanItems from "@/components/plan-items";
 import useAsyncStorage from "@/hooks/use-async-storage";
-import { space } from "@/styles";
 
 export default function PlanScreen() {
   const planId = useLocalSearchParams<{ id: string }>().id!;
@@ -60,52 +59,7 @@ export default function PlanScreen() {
   return (
     <PageLayout>
       <Text category="h3">{plan.name}</Text>
-      <List
-        data={planItems.results}
-        style={{
-          backgroundColor: "transparent",
-        }}
-        renderItem={({ item }) => (
-          <ListItem key={item.objectId}>
-            <Card style={{ width: "100%" }}>
-              <View style={{ flexDirection: "row", gap: space[4] }}>
-                <View
-                  style={{
-                    backgroundColor: "white",
-                    padding: space[1],
-                    borderRadius: space[2],
-                  }}
-                >
-                  <Image
-                    style={{ width: 90, height: 60 }}
-                    source={{ uri: item.exercise.defaultImageUrl }}
-                  />
-                </View>
-                <View
-                  style={{
-                    flexGrow: 1,
-                    gap: space[2],
-                  }}
-                >
-                  <View style={{ flex: 1, flexDirection: "row" }}>
-                    <Text
-                      style={{
-                        fontWeight: "bold",
-                        flex: 1,
-                        fontSize: 16,
-                        lineHeight: 18,
-                      }}
-                    >
-                      {item.exercise.name}
-                    </Text>
-                  </View>
-                  <Text style={{ flex: 1 }}>0 / {item.sets.length} Sets</Text>
-                </View>
-              </View>
-            </Card>
-          </ListItem>
-        )}
-      />
+      <PlanItems items={planItems.results} />
       <Button onPress={handlePlanStart}>Training starten</Button>
     </PageLayout>
   );
diff --git a/src/components/plan-items.tsx b/src/components/plan-items.tsx
new file mode 100644
index 0000000..57975fa
--- /dev/null
+++ b/src/components/plan-items.tsx
@@ -0,0 +1,101 @@
+import { Card, List, ListItem, Text, useTheme } from "@ui-kitten/components";
+import React from "react";
+import { Image, View } from "react-native";
+
+import { PlanItem as PlanItemType } from "@/api/types";
+import { PartiallyResolvedPointer } from "@/parse-client/pointer";
+import { space } from "@/styles";
+
+interface PlanItemsProps {
+  items: PartiallyResolvedPointer<PlanItemType, "exercise">[];
+  onItemPress?: (item: PlanItemType) => void;
+}
+
+export default function PlanItems({ items, onItemPress }: PlanItemsProps) {
+  return (
+    <List
+      data={items}
+      style={{
+        backgroundColor: "transparent",
+        gap: 0,
+        margin: 0,
+        padding: 0,
+      }}
+      renderItem={({ item }) => {
+        return (
+          <ListItem
+            key={item.objectId}
+            style={{
+              paddingHorizontal: space[0],
+              paddingVertical: space[1],
+            }}
+          >
+            {item.type === "EXERCISE" ? (
+              <PlanItem item={item} onPress={onItemPress} />
+            ) : (
+              <Text category="label">{item.name}</Text>
+            )}
+          </ListItem>
+        );
+      }}
+    />
+  );
+}
+
+interface PlanItemProps {
+  item: PartiallyResolvedPointer<PlanItemType, "exercise">;
+  onPress?: (item: PlanItemType) => void;
+}
+
+function PlanItem({ item, onPress }: PlanItemProps) {
+  const theme = useTheme();
+  const isDone = item.openSets.length === 0;
+  return (
+    <Card
+      style={{
+        width: "100%",
+        backgroundColor: isDone ? theme["color-success-500"] : undefined,
+      }}
+      onPress={() => {
+        onPress?.(item);
+      }}
+    >
+      <View style={{ flexDirection: "row", gap: space[4] }}>
+        <View
+          style={{
+            backgroundColor: "white",
+            padding: space[1],
+            borderRadius: space[2],
+          }}
+        >
+          <Image
+            style={{ width: 90, height: 60 }}
+            source={{ uri: item.exercise?.defaultImageUrl }}
+          />
+        </View>
+        <View
+          style={{
+            flexGrow: 1,
+            gap: space[2],
+          }}
+        >
+          <View style={{ flex: 1, flexDirection: "row" }}>
+            <Text
+              style={{
+                fontWeight: "bold",
+                flex: 1,
+                fontSize: 16,
+                lineHeight: 18,
+              }}
+            >
+              {item.exercise?.name}
+            </Text>
+          </View>
+          <Text style={{ flex: 1 }}>
+            {item.sets.length - item.openSets.length} / {item.sets.length} Sets
+          </Text>
+        </View>
+      </View>
+    </Card>
+  );
+}