From 99019ddeef45c2c61e792172c5e6c0f0a6e9b30d Mon Sep 17 00:00:00 2001
From: Noah Spannbauer <noahspannbauer@icloud.com>
Date: Tue, 25 Feb 2025 11:33:16 -0600
Subject: [PATCH] fixing logbook entry form pilot name

---
 app/src/components/logForm/LogForm.tsx | 70 +++++++++++---------------
 1 file changed, 29 insertions(+), 41 deletions(-)

diff --git a/app/src/components/logForm/LogForm.tsx b/app/src/components/logForm/LogForm.tsx
index 711d234..13c0aad 100644
--- a/app/src/components/logForm/LogForm.tsx
+++ b/app/src/components/logForm/LogForm.tsx
@@ -123,6 +123,15 @@ const LogForm: React.FC<ILogFormProps> = ({
         );
         const entry = response.data;
 
+        // if (mode !== FormMode.ADD) {
+        //   const pilot = pilots?.find((pilot) => pilot.id === entry.pilotId);
+        //   console.log(pilot.name)
+        //   dispatch({
+        //     type: 'SET_SELECTED_ENTRY_PILOT_NAME',
+        //     payload: pilot.name
+        //   });
+        // }
+
         methods.reset(entry);
       } catch (error) {
         const axiosError = error as AxiosError;
@@ -194,50 +203,29 @@ const LogForm: React.FC<ILogFormProps> = ({
                 name="pilotId"
                 control={methods.control}
                 render={({ field: { onChange, value } }) => {
-                  useEffect(() => {
-                    if (value && mode !== FormMode.ADD) {
-                      const pilot = pilots?.find((pilot) => pilot.id === value);
-
-                      dispatch({
-                        type: 'SET_SELECTED_ENTRY_PILOT_NAME',
-                        payload: pilot.name
-                      });
-                    }
-                  }, [value]);
-
+                  console.log(value)
                   return (
-                    <>
-                      {mode === FormMode.ADD && (
-                        <Select
-                          disabled={state.isDisabled}
-                          fullWidth
-                          onChange={(event) => {
-                            const pilot = pilots?.find(
-                              (pilot) => (pilot.id = event.target.value)
-                            );
+                    <Select
+                      disabled={state.isDisabled}
+                      fullWidth
+                      onChange={(event) => {
+                        const pilot = pilots?.find(
+                          (pilot) => (pilot.id = event.target.value)
+                        );
 
-                            if (pilot) {
-                              methods.setValue('pilotName', pilot.name);
-                            }
+                        if (pilot) {
+                          methods.setValue('pilotName', pilot.name);
+                        }
 
-                            methods.setValue('pilotId', event.target.value);
-                          }}
-                          options={
-                            state.pilotOptions && state.pilotOptions.length > 0
-                              ? state.pilotOptions
-                              : []
-                          }
-                          value={value}
-                        />
-                      )}
-                      {mode !== FormMode.ADD && (
-                        <TextField
-                          disabled={true}
-                          fullWidth
-                          value={state.selectedEntryPilotName}
-                        />
-                      )}
-                    </>
+                        methods.setValue('pilotId', event.target.value);
+                      }}
+                      options={
+                        state.pilotOptions && state.pilotOptions.length > 0
+                          ? state.pilotOptions
+                          : []
+                      }
+                      value={value ? value : ''}
+                    />
                   );
                 }}
               />