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.

1 files changed, 5 insertions, 5 deletions

diff --git a/ui/routes/(login)/setup/+page.svelte b/ui/routes/(login)/setup/+page.svelte
index 2503502..a1974b8 100644
--- a/ui/routes/(login)/setup/+page.svelte
+++ b/ui/routes/(login)/setup/+page.svelte
@@ -4,19 +4,19 @@
 
     import LogIn from '$lib/components/LogIn.svelte';
 
-    let disabled;
-    let username;
-    let password;
+    let username = "", password = "";
+    let pending = false;
+    $: disabled = pending;
 
     async function onSubmit() {
-        disabled = true;
+        pending = true;
         const response = await setup(username, password);
         if (200 <= response.status && response.status < 300) {
             username = '';
             password = '';
             goto('/');
         }
-        disabled = false;
+        pending = false;
     }
 </script>