refactor(use has field value): add "id" field to query to suppress co…

…nsole.warn
dhis2 · Oct 30, 2023 · 2a908ba · 2a908ba
1 parent 526fc8b
commit 2a908ba
Showing 1 changed file with 9 additions and 8 deletions.
diff --git a/src/pages/dataElements/form/useHasFieldValue.ts b/src/pages/dataElements/form/useHasFieldValue.ts
@@ -8,7 +8,7 @@ const HAS_FIELD_VALUE_QUERY = {
         resource: 'dataElements',
         params: (variables: Record<string, string>) => ({
             pageSize: 1,
-            fields: '',
+            fields: 'id',
             filter: [`${variables.field}:eq:${variables.value}`],
         }),
     },
@@ -38,13 +38,14 @@ export function useHasFieldValue(field: string) {
                     // so we have to ignore the type error
                     // eslint-disable-next-line @typescript-eslint/ban-ts-comment
                     // @ts-ignore
-                    (data: QueryResponse) =>
-                        data.dataElements.pager.total
-                            ? i18n.t(
-                                  'This {{field}} already exists, please choose antoher one',
-                                  { field }
-                              )
-                            : undefined
+                    (data: QueryResponse) => {
+                        if (data.dataElements.pager.total) {
+                            return i18n.t(
+                                'This {{field}} already exists, please choose antoher one',
+                                { field }
+                            )
+                        }
+                    }
                 )
             },
         }),