From 10f77d5ee4f9aa12a82151c9ff2ec738339c1e5c Mon Sep 17 00:00:00 2001
From: malmen237 <linda.malm@eyevinn.se>
Date: Tue, 28 Jan 2025 16:21:19 +0100
Subject: [PATCH] fix: set the api-fetch into it's own hook

---
 .../create-production-page.tsx                | 42 +++----------
 .../use-create-production.tsx                 | 62 +++++++++++++++++++
 2 files changed, 70 insertions(+), 34 deletions(-)
 create mode 100644 src/components/create-production/use-create-production.tsx

diff --git a/src/components/create-production/create-production-page.tsx b/src/components/create-production/create-production-page.tsx
index c5b74396..8bd45e93 100644
--- a/src/components/create-production/create-production-page.tsx
+++ b/src/components/create-production/create-production-page.tsx
@@ -16,7 +16,6 @@ import {
   PrimaryButton,
   SecondaryButton,
 } from "../landing-page/form-elements.tsx";
-import { API } from "../../api/api.ts";
 import { useGlobalState } from "../../global-state/context-provider.tsx";
 import { Spinner } from "../loader/loader.tsx";
 import { FlexContainer } from "../generic-components.ts";
@@ -27,13 +26,7 @@ import { NavigateToRootButton } from "../navigate-to-root-button/navigate-to-roo
 import { ResponsiveFormContainer } from "../user-settings/user-settings.tsx";
 import { isMobile } from "../../bowser.ts";
 import { Checkbox } from "../checkbox/checkbox.tsx";
-
-type FormValues = {
-  productionName: string;
-  defaultLine: string;
-  defaultLineProgramOutput: boolean;
-  lines: { name: string; programOutputLine?: boolean }[];
-};
+import { FormValues, useCreateProduction } from "./use-create-production.tsx";
 
 const HeaderWrapper = styled.div`
   display: flex;
@@ -88,10 +81,8 @@ const CheckboxWrapper = styled.div`
 
 export const CreateProductionPage = () => {
   const [, dispatch] = useGlobalState();
-  const [createdProductionId, setCreatedProductionId] = useState<string | null>(
-    null
-  );
-  const [loading, setLoading] = useState<boolean>(false);
+  const [createNewProduction, setCreateNewProduction] =
+    useState<FormValues | null>(null);
   const [copiedUrl, setCopiedUrl] = useState<boolean>(false);
   const {
     formState: { errors },
@@ -116,33 +107,16 @@ export const CreateProductionPage = () => {
     },
   });
 
+  const { createdProductionId, loading } = useCreateProduction({
+    createNewProduction,
+  });
+
   const { error: productionFetchError, production } = useFetchProduction(
     createdProductionId ? parseInt(createdProductionId, 10) : null
   );
 
   const onSubmit: SubmitHandler<FormValues> = (value) => {
-    setLoading(true);
-    API.createProduction({
-      name: value.productionName,
-      lines: [
-        {
-          name: value.defaultLine,
-          programOutputLine: value.defaultLineProgramOutput,
-        },
-        ...value.lines,
-      ],
-    })
-      .then((v) => {
-        setCreatedProductionId(v.productionId);
-        setLoading(false);
-      })
-      .catch((error) => {
-        dispatch({
-          type: "ERROR",
-          payload: error,
-        });
-        setLoading(false);
-      });
+    setCreateNewProduction(value);
   };
 
   // Reset form values when created production id changes
diff --git a/src/components/create-production/use-create-production.tsx b/src/components/create-production/use-create-production.tsx
new file mode 100644
index 00000000..b0de035b
--- /dev/null
+++ b/src/components/create-production/use-create-production.tsx
@@ -0,0 +1,62 @@
+import { useEffect, useState } from "react";
+import { useGlobalState } from "../../global-state/context-provider";
+import { API } from "../../api/api";
+
+type TUseCreateProduction = {
+  createNewProduction: FormValues | null;
+};
+
+export type FormValues = {
+  productionName: string;
+  defaultLine: string;
+  defaultLineProgramOutput: boolean;
+  lines: { name: string; programOutputLine?: boolean }[];
+};
+
+export const useCreateProduction = ({
+  createNewProduction,
+}: TUseCreateProduction) => {
+  const [newProduction, setNewProduction] = useState<string | null>(null);
+  const [loading, setLoading] = useState<boolean>(false);
+  const [, dispatch] = useGlobalState();
+
+  useEffect(() => {
+    let aborted = false;
+
+    if (createNewProduction) {
+      setLoading(true);
+      API.createProduction({
+        name: createNewProduction.productionName,
+        lines: [
+          {
+            name: createNewProduction.defaultLine,
+            programOutputLine: createNewProduction.defaultLineProgramOutput,
+          },
+          ...createNewProduction.lines,
+        ],
+      })
+        .then((v) => {
+          if (aborted) return;
+
+          setNewProduction(v.productionId);
+          setLoading(false);
+        })
+        .catch((error) => {
+          dispatch({
+            type: "ERROR",
+            payload: error,
+          });
+          setLoading(false);
+        });
+    }
+
+    return () => {
+      aborted = true;
+    };
+  }, [createNewProduction, dispatch]);
+
+  return {
+    createdProductionId: newProduction,
+    loading,
+  };
+};