fix: clean up useEffect dependencies

dhis2 · Dec 20, 2024 · de622f2 · de622f2
1 parent 6b550ae
commit de622f2
Show file tree

Hide file tree

Showing 8 changed files with 14 additions and 11 deletions.
diff --git a/src/data-workspace/inputs/boolean-radios.js b/src/data-workspace/inputs/boolean-radios.js
@@ -27,7 +27,7 @@ export const BooleanRadios = ({
         if (syncTouched) {
             setValueSynced(value === lastSyncedValue)
         }
-    }, [value, lastSyncedValue, syncTouched])
+    }, [value, lastSyncedValue, syncTouched, setValueSynced])
 
     const { mutate } = useSetDataValueMutation({ deId, cocId })
 

diff --git a/src/data-workspace/inputs/date-input.js b/src/data-workspace/inputs/date-input.js
@@ -29,7 +29,7 @@ export const DateInput = ({
         if (syncTouched) {
             setValueSynced(value === lastSyncedValue)
         }
-    }, [value, lastSyncedValue, syncTouched])
+    }, [value, lastSyncedValue, syncTouched, setValueSynced])
 
     const { data: userInfo } = useUserInfo()
     const keyUiLocale = userInfo?.settings?.keyUiLocale

diff --git a/src/data-workspace/inputs/datetime-input.js b/src/data-workspace/inputs/datetime-input.js
@@ -44,7 +44,7 @@ export const DateTimeInput = ({
             const dateWithTime = convertToDateWithTime({ date, time })
             setValueSynced(dateWithTime === lastSyncedValue)
         }
-    }, [date, time, lastSyncedValue, syncTouched])
+    }, [date, time, lastSyncedValue, syncTouched, setValueSynced])
 
     const { mutate } = useSetDataValueMutation({ deId, cocId })
 

diff --git a/src/data-workspace/inputs/file-inputs.js b/src/data-workspace/inputs/file-inputs.js
@@ -45,7 +45,7 @@ export const FileResourceInput = ({
                 )
             }
         }
-    }, [value, lastSyncedValue, syncTouched])
+    }, [value, lastSyncedValue, syncTouched, setValueSynced])
 
     const dataValueParams = useDataValueParams({ deId, cocId })
     const fileLink = useFileInputUrl(dataValueParams)

diff --git a/src/data-workspace/inputs/generic-input.js b/src/data-workspace/inputs/generic-input.js
@@ -63,7 +63,7 @@ export const GenericInput = ({
         if (syncTouched) {
             setValueSynced(value === lastSyncedValue)
         }
-    }, [value, lastSyncedValue, syncTouched])
+    }, [value, lastSyncedValue, syncTouched, setValueSynced])
 
     const limits = useMinMaxLimits(deId, cocId)
 
@@ -75,9 +75,12 @@ export const GenericInput = ({
 
     // check if the initial value has any associated warnings
     useEffect(() => {
-        const warningValidationResult = warningValidator(value)
-        setWarning(fieldname, warningValidationResult)
-    }, [initialValue, warningValidator, setWarning, fieldname])
+        // only check if the value has not been updated (i.e. is initial value)
+        if (!syncTouched) {
+            const warningValidationResult = warningValidator(value)
+            setWarning(fieldname, warningValidationResult)
+        }
+    }, [value, syncTouched, warningValidator, setWarning, fieldname])
 
     const { mutate } = useSetDataValueMutation({ deId, cocId })
 

diff --git a/src/data-workspace/inputs/long-text.js b/src/data-workspace/inputs/long-text.js
@@ -22,7 +22,7 @@ export const LongText = ({
         if (syncTouched) {
             setValueSynced(value === lastSyncedValue)
         }
-    }, [value, lastSyncedValue, syncTouched])
+    }, [value, lastSyncedValue, syncTouched, setValueSynced])
 
     const { mutate } = useSetDataValueMutation({ deId, cocId })
     const syncData = (newValue) => {

diff --git a/src/data-workspace/inputs/option-set.js b/src/data-workspace/inputs/option-set.js
@@ -78,7 +78,7 @@ export const OptionSet = ({
         if (syncTouched) {
             setValueSynced(value === lastSyncedValue)
         }
-    }, [value, lastSyncedValue, syncTouched])
+    }, [value, lastSyncedValue, syncTouched, setValueSynced])
 
     const { mutate } = useSetDataValueMutation({ deId, cocId })
     const syncData = (newValue) => {

diff --git a/src/data-workspace/inputs/true-only-checkbox.js b/src/data-workspace/inputs/true-only-checkbox.js
@@ -25,7 +25,7 @@ export const TrueOnlyCheckbox = ({
         if (syncTouched) {
             setValueSynced(value === lastSyncedValue)
         }
-    }, [value, lastSyncedValue, syncTouched])
+    }, [value, lastSyncedValue, syncTouched, setValueSynced])
 
     const { mutate } = useSetDataValueMutation({ deId, cocId })
     const syncData = (newValue) => {