Add multiple forms test

Author: unlocomqx

src/lib/client/superForm.ts

@@ -1,6 +1,6 @@
 /* eslint-disable dci-lint/atomic-role-binding */
 import type { TaintedFields, SuperFormValidated, SuperValidated } from '$lib/superValidate.js';
-import type { ActionResult, Page, SubmitFunction } from '@sveltejs/kit';
+import type { ActionResult, BeforeNavigate, Page, SubmitFunction } from '@sveltejs/kit';
 import {
 	derived,
 	get,
@@ -383,6 +383,28 @@ try {
 	// No Storybook
 }
 
+const onDestroyCallbacks = new Set<() => void>();
+const beforeNavigateCallbacks = new Set<(nav: BeforeNavigate) => Promise<void>>();
+let lifeCycleHandlersInited = false;
+function initLifeCycleCallbacks() {
+	if (lifeCycleHandlersInited) return;
+	lifeCycleHandlersInited = true;
+
+	onDestroy(() => {
+		for (const callback of onDestroyCallbacks) {
+			callback();
+		}
+		onDestroyCallbacks.clear();
+	});
+
+	beforeNavigate((nav: BeforeNavigate) => {
+		for (const callback of beforeNavigateCallbacks) {
+			callback(nav);
+		}
+		beforeNavigateCallbacks.clear();
+	});
+}
+
 /////////////////////////////////////////////////////////////////////
 
 /**
@@ -404,6 +426,8 @@ export function superForm<
 	// To check if a full validator is used when switching options.validators dynamically
 	let initialValidator: FormOptions<T, M, In>['validators'] | undefined = undefined;
 
+	initLifeCycleCallbacks();
+
 	{
 		if (options.legacy ?? LEGACY_MODE) {
 			if (options.resetForm === undefined) options.resetForm = false;
@@ -518,7 +542,7 @@ export function superForm<
 
 		///// From here, form is properly initialized /////
 
-		onDestroy(() => {
+		onDestroyCallbacks.add(() => {
 			Unsubscriptions_unsubscribe();
 			NextChange_clear();
 			EnhancedForm_destroy();
@@ -1350,7 +1374,7 @@ export function superForm<
 		// Tainted check
 		const defaultMessage = 'Leave page? Changes that you made may not be saved.';
 		let forceRedirection = false;
-		beforeNavigate(async (nav) => {
+		beforeNavigateCallbacks.add(async (nav: BeforeNavigate) => {
 			if (options.taintedMessage && !Data.submitting && !forceRedirection) {
 				if (Tainted_isTainted()) {
 					const { taintedMessage } = options;

src/routes/(v2)/v2/Navigation.svelte

@@ -14,6 +14,7 @@
 		'issue-337-checkboxes',
 		'issue-345',
 		'letters',
+		'multiple-forms',
 		'multiple-files',
 		'multistep-client',
 		'multistep-server',

src/routes/(v2)/v2/multiple-forms/+page.server.ts

@@ -0,0 +1,58 @@
+import { zod } from '$lib/adapters/zod.js';
+import { message, superValidate } from '$lib/server/index.js';
+import { type Actions, fail } from '@sveltejs/kit';
+import { schema } from './schema.js';
+
+const items = [
+	{
+		id: 1,
+		label: 'One'
+	},
+	{
+		id: 2,
+		label: 'Two'
+	},
+	{
+		id: 3,
+		label: 'Three'
+	}
+];
+
+export const load = async () => {
+	const item_forms = await Promise.all(
+		items.map((item) =>
+			superValidate(item, zod(schema), {
+				id: item.id.toString()
+			})
+		)
+	);
+
+	return { item_forms };
+};
+
+export const actions: Actions = {
+	async create() {
+		items.push({
+			id: items.length + 1,
+			label: (items.length + 1).toString()
+		});
+
+		return { success: true };
+	},
+
+	async save({ request }) {
+		const form = await superValidate(request, zod(schema));
+
+		if (!form.valid) {
+			// Again, return { form } and things will just work.
+			return fail(400, { form });
+		}
+
+		const index = items.findIndex((item) => item.id === form.data.id);
+		if(index !== -1) {
+			items[index].label = form.data.label;
+		}
+
+		return message(form, `Item ${form.data.id} updated`);
+	}
+};

src/routes/(v2)/v2/multiple-forms/+page.svelte

@@ -0,0 +1,26 @@
+<script lang="ts">
+	import type { PageData } from './$types.js';
+	import { superForm } from '$lib/client/index.js';
+	import Form from './Form.svelte';
+	import { enhance } from '$app/forms';
+
+	let {
+		data
+	}: {
+		data: PageData;
+	} = $props();
+
+	const superforms = $derived(data.item_forms.map(item_form => superForm(item_form, {
+		id: item_form.id,
+		dataType: 'json'
+	})));
+</script>
+
+{#each superforms as superform (superform.formId)}
+	<Form {superform} />
+	<hr>
+{/each}
+
+<form action="?/create" method=post use:enhance>
+	<button type=submit>Add</button>
+</form>

src/routes/(v2)/v2/multiple-forms/Form.svelte

@@ -0,0 +1,28 @@
+<script lang="ts">
+	import type { SuperForm } from '$lib/index.js';
+	import type { schema } from './schema.js';
+	import { z } from 'zod';
+	import SuperDebug from '$lib/client/SuperDebug.svelte';
+
+	let { superform }: {
+		superform: SuperForm<z.infer<typeof schema>>;
+	} = $props();
+
+	let { form, message, enhance } = superform;
+</script>
+
+<form action="?/save" method="post" use:enhance>
+	<input type="hidden" name="id" value={$form.id} />
+
+	<SuperDebug data={$form} />
+
+	<label>
+		<input type="text" name="label" bind:value={$form.label} />
+	</label>
+
+	<button type="submit">Save</button>
+
+	{#if $message}
+		<p>{$message}</p>
+	{/if}
+</form>

src/routes/(v2)/v2/multiple-forms/schema.ts

@@ -0,0 +1,6 @@
+import { z } from 'zod';
+
+export const schema = z.object({
+	id: z.number(),
+	label: z.string()
+});