fix(de form async validation): prevent other field validation

dhis2 · Oct 30, 2023 · bb839b2 · bb839b2
1 parent a3a4135
commit bb839b2
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/src/pages/dataElements/form/fields.tsx b/src/pages/dataElements/form/fields.tsx
@@ -46,7 +46,10 @@ export function NameField() {
             helpText={i18n.t(
                 'A data element name should be concise and easy to recognize.'
             )}
-            validate={checkIsValueTaken}
+            // Do not pass checkIsValueTaken directly, otherwise memoization
+            // won't work due to additional arguments being passed
+            validate={(name: string) => checkIsValueTaken(name)}
+            validateFields={[]}
         />
     )
 }
@@ -70,7 +73,10 @@ export function ShortNameField() {
             })}
             name="shortName"
             helpText={i18n.t('Often used in reports where space is limited')}
-            validate={checkIsValueTaken}
+            // Do not pass checkIsValueTaken directly, otherwise memoization
+            // won't work due to additional arguments being passed
+            validate={(shortName: string) => checkIsValueTaken(shortName)}
+            validateFields={[]}
         />
     )
 }