Fixed coercion of boolean false.

Author: ciscoheat

CHANGELOG.md

@@ -5,6 +5,12 @@ 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).
 
+## [Unreleased]
+
+### Fixed
+
+- A boolean schema field didn't accept a boolean `false` value when posted, it was coerced as `true`.
+
 ## [1.4.0] - 2023-07-20
 
 ### Fixed

src/lib/superValidate.ts

@@ -211,7 +211,7 @@ function formDataToValidation<T extends AnyZodObject>(
         ? parseInt(value ?? '', 10)
         : parseFloat(value ?? '');
     } else if (zodType._def.typeName == 'ZodBoolean') {
-      return Boolean(value).valueOf();
+      return Boolean(value == 'false' ? '' : value).valueOf();
     } else if (zodType._def.typeName == 'ZodDate') {
       return new Date(value ?? '');
     } else if (zodType._def.typeName == 'ZodArray') {

src/routes/tests/issue-235/+page.server.ts

@@ -0,0 +1,26 @@
+import { superValidate, message } from '$lib/server';
+import { fail } from '@sveltejs/kit';
+
+import type { Actions, PageServerLoad } from './$types';
+import { exampleSchema } from './schema.js';
+
+///// Load function /////
+
+export const load: PageServerLoad = async () => {
+  const form = await superValidate(exampleSchema);
+  return { form };
+};
+
+///// Form actions /////
+
+export const actions: Actions = {
+  default: async ({ request }) => {
+    const form = await superValidate(request, exampleSchema);
+
+    console.log('POST', form);
+
+    if (!form.valid) return fail(400, { form });
+
+    return message(form, 'Form posted successfully!');
+  }
+};

src/routes/tests/issue-235/+page.svelte

@@ -0,0 +1,97 @@
+<script lang="ts">
+  import { superForm } from '$lib/client';
+  import type { PageData } from './$types';
+  import SuperDebug from '$lib/client/SuperDebug.svelte';
+  import { page } from '$app/stores';
+
+  export let data: PageData;
+
+  const { form, errors, message, constraints, enhance, validate } =
+    superForm(data.form);
+</script>
+
+<SuperDebug
+  data={{
+    $form: $form,
+    $errors: $errors,
+    validate: validate
+  }}
+/>
+
+{#if $message}
+  <div
+    class="status"
+    class:error={$page.status >= 400}
+    class:success={$page.status == 200}
+  >
+    {$message}
+  </div>
+{/if}
+
+<form method="POST" use:enhance>
+  <fieldset role="radiogroup">
+    <legend>Radio Group</legend>
+    <label>
+      <input
+        type="radio"
+        name="radioGroup"
+        value={true}
+        bind:group={$form.radioGroup}
+      />
+      Yes
+    </label>
+    <label>
+      <input
+        type="radio"
+        name="radioGroup"
+        value={false}
+        bind:group={$form.radioGroup}
+      />
+      No
+    </label>
+  </fieldset>
+  <hr />
+  <label>
+    <input type="checkbox" name="checkbox" bind:checked={$form.checkbox} /> Checkbox
+  </label>
+  <button>Submit</button>
+</form>
+
+<style>
+  .invalid {
+    color: red;
+  }
+
+  .status {
+    color: white;
+    padding: 4px;
+    padding-left: 8px;
+    border-radius: 2px;
+    font-weight: 500;
+  }
+
+  .status.success {
+    background-color: seagreen;
+  }
+
+  .status.error {
+    background-color: #ff2a02;
+  }
+
+  input {
+    background-color: #ddd;
+  }
+
+  a {
+    text-decoration: underline;
+  }
+
+  hr {
+    margin-top: 4rem;
+  }
+
+  form {
+    padding-top: 1rem;
+    padding-bottom: 1rem;
+  }
+</style>

src/routes/tests/issue-235/schema.ts

@@ -0,0 +1,6 @@
+import { z } from 'zod';
+
+export const exampleSchema = z.object({
+  radioGroup: z.boolean().default(false),
+  checkbox: z.boolean()
+});