Fixed exception message when using nested data in normal posting.

Author: ciscoheat

CHANGELOG.md

@@ -5,12 +5,18 @@ Headlines: Added, Changed, Deprecated, Removed, Fixed, Security
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## [1.2.0]
+## [Unreleased]
+
+### Fixed
+
+- Fixed exception message when the `dataType` option isn't set to `'json'` and the schema contains a nested object.
+
+## [1.2.0] - 2023-07-06
 
 ### Added
 
 - The `scrollToError` option can now take the same parameters as [scrollIntoView](https://developer.mozilla.org/en-US/docs/Web/API/Element/scrollIntoView), which makes it work for nested scrollbars.
-- Added signature to more conveniently set a form-level error with `setError`.
+- Added `setError` signature, to more conveniently set a form-level error: `setError(form, 'Form-level error message')`
 
 ### Fixed
 

src/errors.test.ts

@@ -277,3 +277,32 @@ test('Refined errors on leaf node', async () => {
     flavours: { _errors: ["Can't order more flavours than scoops!"] }
   });
 });
+
+test('Error on posting nested data in dataType form mode', async () => {
+  const registrationSchema = z.object({
+    username: z.string(),
+    credential: z.object({
+      id: z.string(),
+      publicKey: z.string(),
+      algorithm: z.string()
+    }),
+    authenticatorData: z.string(),
+    clientData: z.string()
+  });
+
+  const schema = z.object({
+    email: z.string().email(),
+    registration: registrationSchema.optional()
+  });
+
+  const formData = new FormData();
+  formData.set('email', '[email protected]');
+  formData.set(
+    'registration',
+    '{"username":"[email protected]","credential":{"id":"6igsmeLIsd2jTAraOL1QX4qUWjdtvBX3gMeEHOR-QcU","publicKey":"MFkwEwYHKoZIzj0CAQYIKoZIzj0DAQcDQgAEOdtZo4FdtR2AAU6pR0u9d4qJcLsfnCM18No1lwTjx-7sJds6sr4SI721yzwDMYIB1L8frZkuUs1JK4Rq5C4fYg==","algorithm":"ES256"},"authenticatorData":"SZYN5YgOjGh0NBcPZHZgW4_krrmihjLHmVzzuoMdl2NFAAAAAQECAwQFBgcIAQIDBAUGBwgAIOooLJniyLHdo0wK2ji9UF-KlFo3bbwV94DHhBzkfkHFpQECAyYgASFYIDnbWaOBXbUdgAFOqUdLvXeKiXC7H5wjNfDaNZcE48fuIlgg7CXbOrK-EiO9tcs8AzGCAdS_H62ZLlLNSSuEauQuH2I=","clientData":"eyJ0eXBlIjoid2ViYXV0aG4uY3JlYXRlIiwiY2hhbGxlbmdlIjoiU1llTVR5aHpmaHFXaUx4ZyIsIm9yaWdpbiI6Imh0dHA6Ly9sb2NhbGhvc3Q6NTE3MyIsImNyb3NzT3JpZ2luIjpmYWxzZX0="}'
+  );
+
+  await expect(superValidate(formData, schema)).rejects.toThrow(
+    /Object found in form field "registration"/
+  );
+});

src/lib/client/index.ts

@@ -374,8 +374,8 @@ export function superForm<
       for (const form of forms) {
         if (duplicateId.has(form.id)) {
           console.warn(
-            `Duplicate form id found: "${form.id}"` +
-              '. Multiple forms will receive the same data. Use the id option to differentiate between them, ' +
+            `Duplicate form id found: "${form.id}". ` +
+              'Multiple forms will receive the same data. Use the id option to differentiate between them, ' +
               'or if this is intended, set warnings.duplicateId option to false to disable this message.'
           );
         } else {
@@ -472,7 +472,9 @@ export function superForm<
       if (value.length > 0) Form_checkForNestedData(key, value[0]);
     } else if (!(value instanceof Date)) {
       throw new SuperFormError(
-        `Object found in form field "${key}". Set options.dataType = 'json' and use:enhance to use nested data structures.`
+        `Object found in form field "${key}". ` +
+          `Set the dataType option to "json" and add use:enhance to use nested data structures. ` +
+          `More information: https://superforms.rocks/concepts/nested-data`
       );
     }
   }

src/lib/superValidate.ts

@@ -258,6 +258,14 @@ function formDataToValidation<T extends AnyZodObject>(
       return Symbol(String(value));
     }
 
+    if (zodType._def.typeName == 'ZodObject') {
+      throw new SuperFormError(
+        `Object found in form field "${field}". ` +
+          `Set the dataType option to "json" and add use:enhance on the client to use nested data structures. ` +
+          `More information: https://superforms.rocks/concepts/nested-data`
+      );
+    }
+
     throw new SuperFormError(
       'Unsupported Zod default type: ' + zodType.constructor.name
     );