From 1eec6338e1146439b1dbc6207843fbc44dd13088 Mon Sep 17 00:00:00 2001 From: Owen Jacobson Date: Wed, 30 Oct 2024 18:01:48 -0400 Subject: Don't leave field binding vars uninitialized. This was causing problems for changing passwords: if the user didn't type anything in the "original password" field, the code path to sending that field to the server was just straight-up omitting the field from the message, rather than setting it to empty string, causing a 422 Unprocessable Entity. On investigation we had latent bugs related to this in a bunch of spots. --- ui/routes/(app)/me/+page.svelte | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'ui/routes/(app)') diff --git a/ui/routes/(app)/me/+page.svelte b/ui/routes/(app)/me/+page.svelte index 7559dbe..82af3c7 100644 --- a/ui/routes/(app)/me/+page.svelte +++ b/ui/routes/(app)/me/+page.svelte @@ -3,7 +3,7 @@ import Invites from '$lib/components/Invites.svelte'; - let currentPassword, newPassword, confirmPassword, passwordForm; + let currentPassword = "", newPassword = "", confirmPassword = "", passwordForm; let pending = false; $: valid = (newPassword === confirmPassword) && (newPassword !== currentPassword); $: disabled = pending || !valid; -- cgit v1.2.3