Split create/update field input resolvers for relationship fields (#6317

)
keystonejs · Aug 16, 2021 · 1cbcf54 · 1cbcf54
1 parent 81b95c5
commit 1cbcf54
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 6 deletions.
diff --git a/.changeset/brown-plums-kneel.md b/.changeset/brown-plums-kneel.md
@@ -0,0 +1,5 @@
+---
+'@keystone-next/keystone': patch
+---
+
+Separated the resolving of non-relationship field from relationship fields in create/update inputs to allow for better error handling.
diff --git a/packages/keystone/src/lib/core/mutations/create-update.ts b/packages/keystone/src/lib/core/mutations/create-update.ts
@@ -180,21 +180,32 @@ async function getResolvedData(
     );
   }
 
-  // Apply field type input resolvers
+  // Apply non-relationship field type input resolvers
   resolvedData = Object.fromEntries(
     await promiseAllRejectWithAllErrors(
       Object.entries(list.fields).map(async ([fieldKey, field]) => {
         const inputResolver = field.input?.[operation]?.resolve;
         let input = resolvedData[fieldKey];
-        if (inputResolver) {
+        if (inputResolver && field.dbField.kind !== 'relation') {
+          input = await inputResolver(input, context, undefined);
+        }
+        return [fieldKey, input] as const;
+      })
+    )
+  );
+
+  // Apply relationship field type input resolvers
+  resolvedData = Object.fromEntries(
+    await promiseAllRejectWithAllErrors(
+      Object.entries(list.fields).map(async ([fieldKey, field]) => {
+        const inputResolver = field.input?.[operation]?.resolve;
+        let input = resolvedData[fieldKey];
+        if (inputResolver && field.dbField.kind === 'relation') {
           input = await inputResolver(
             input,
             context,
+            // This third argument only applies to relationship fields
             (() => {
-              // This third argument only applies to relationship fields
-              if (field.dbField.kind !== 'relation') {
-                return undefined;
-              }
               if (input === undefined) {
                 // No-op: This is what we want
                 return () => undefined;