From 74140f41dbfbd27ad9289aa27339f65d2b44f042 Mon Sep 17 00:00:00 2001
From: Scott Idem <stidem@gmail.com>
Date: Tue, 18 Nov 2025 19:23:33 -0500
Subject: [PATCH] fix: Update event settings forms for Svelte 5 reactivity

Corrected event settings components to use Svelte 5 bindable props for two-way data binding. This ensures that changes in child form components (ae_comp__event_settings_form.svelte, ae_comp__event_settings_pres_mgmt_form.svelte) are reactively reflected in the parent page (settings/+page.svelte) and properly handled during save operations.
---
 src/routes/events/[event_id]/settings/+page.svelte     |  4 ++--
 .../settings/ae_comp__event_settings_form.svelte       | 10 ++++------
 .../ae_comp__event_settings_pres_mgmt_form.svelte      | 10 ++++------
 3 files changed, 10 insertions(+), 14 deletions(-)

diff --git a/src/routes/events/[event_id]/settings/+page.svelte b/src/routes/events/[event_id]/settings/+page.svelte
index a617dd7a..ac7e1fb8 100644
--- a/src/routes/events/[event_id]/settings/+page.svelte
+++ b/src/routes/events/[event_id]/settings/+page.svelte
@@ -50,7 +50,7 @@
 		<div>
 			<h2 class="h2">General Config (cfg_json)</h2>
 			<Ae_comp_event_settings_form
-				cfg_json={event_obj.cfg_json}
+				bind:cfg_json={event_obj.cfg_json}
 				on:save={(e) => handle_save('cfg_json', e.detail)}
 			/>
 		</div>
@@ -58,7 +58,7 @@
 		<div>
 			<h2 class="h2">Presentation Management (mod_pres_mgmt_json)</h2>
 			<Ae_comp_event_settings_pres_mgmt_form
-				mod_pres_mgmt_json={event_obj.mod_pres_mgmt_json}
+				bind:mod_pres_mgmt_json={event_obj.mod_pres_mgmt_json}
 				on:save={(e) => handle_save('mod_pres_mgmt_json', e.detail)}
 			/>
 		</div>
diff --git a/src/routes/events/[event_id]/settings/ae_comp__event_settings_form.svelte b/src/routes/events/[event_id]/settings/ae_comp__event_settings_form.svelte
index 6beaf28d..bf5248aa 100644
--- a/src/routes/events/[event_id]/settings/ae_comp__event_settings_form.svelte
+++ b/src/routes/events/[event_id]/settings/ae_comp__event_settings_form.svelte
@@ -6,14 +6,12 @@
 		cfg_json: key_val;
 	}
 
-	let { cfg_json }: Props = $props();
-
-	let local_cfg_json = $state(cfg_json);
+	let { cfg_json = $bindable() }: Props = $props();
 
 	const dispatch = createEventDispatcher();
 
 	function save() {
-		dispatch('save', local_cfg_json);
+		dispatch('save', cfg_json);
 	}
 </script>
 
@@ -21,13 +19,13 @@
 	<div>
 		<label class="label">
 			<span>Short Name</span>
-			<input type="text" class="input" bind:value={local_cfg_json.short_name} />
+			<input type="text" class="input" bind:value={cfg_json.short_name} />
 		</label>
 	</div>
 	<div>
 		<label class="label">
 			<span>Medium Name</span>
-			<input type="text" class="input" bind:value={local_cfg_json.med_name} />
+			<input type="text" class="input" bind:value={cfg_json.med_name} />
 		</label>
 	</div>
 	<button class="btn preset-tonal-primary" on:click={save}>Save</button>
diff --git a/src/routes/events/[event_id]/settings/ae_comp__event_settings_pres_mgmt_form.svelte b/src/routes/events/[event_id]/settings/ae_comp__event_settings_pres_mgmt_form.svelte
index 56753e9c..89032a47 100644
--- a/src/routes/events/[event_id]/settings/ae_comp__event_settings_pres_mgmt_form.svelte
+++ b/src/routes/events/[event_id]/settings/ae_comp__event_settings_pres_mgmt_form.svelte
@@ -6,21 +6,19 @@
 		mod_pres_mgmt_json: key_val;
 	}
 
-	let { mod_pres_mgmt_json }: Props = $props();
-
-	let local_mod_pres_mgmt_json = $state(mod_pres_mgmt_json);
+	let { mod_pres_mgmt_json = $bindable() }: Props = $props();
 
 	const dispatch = createEventDispatcher();
 
 	function save() {
-		dispatch('save', local_mod_pres_mgmt_json);
+		dispatch('save', mod_pres_mgmt_json);
 	}
 </script>
 
 <div class="space-y-4">
 	<div>
 		<label class="label">
-			<input type="checkbox" class="checkbox" bind:checked={local_mod_pres_mgmt_json.lock_config} />
+			<input type="checkbox" class="checkbox" bind:checked={mod_pres_mgmt_json.lock_config} />
 			<span>Lock Config</span>
 		</label>
 	</div>
@@ -29,7 +27,7 @@
 			<input
 				type="checkbox"
 				class="checkbox"
-				bind:checked={local_mod_pres_mgmt_json.hide__location_code}
+				bind:checked={mod_pres_mgmt_json.hide__location_code}
 			/>
 			<span>Hide Location Code</span>
 		</label>