From 7692f9d503a1f54627110fc8b670dfe881275cc6 Mon Sep 17 00:00:00 2001
From: Levente Orban <leventeorb@gmail.com>
Date: Tue, 28 Oct 2025 19:16:36 +0100
Subject: [PATCH] fix: error when editing the events, location_type

---
 src/routes/event/[id]/edit/+page.server.ts |  7 +++----
 src/routes/event/[id]/edit/+page.svelte    | 19 ++++++++++++++-----
 2 files changed, 17 insertions(+), 9 deletions(-)

diff --git a/src/routes/event/[id]/edit/+page.server.ts b/src/routes/event/[id]/edit/+page.server.ts
index fd61dd0..c866f6b 100644
--- a/src/routes/event/[id]/edit/+page.server.ts
+++ b/src/routes/event/[id]/edit/+page.server.ts
@@ -76,7 +76,7 @@ export const actions: Actions = {
 		const date = formData.get('date') as string;
 		const time = formData.get('time') as string;
 		const location = formData.get('location') as string;
-		const locationType = formData.get('location_type') as 'none' | 'text' | 'maps';
+		const locationType = formData.get('locationType') as string;
 		const locationUrl = formData.get('location_url') as string;
 		const type = formData.get('type') as 'limited' | 'unlimited';
 		const attendeeLimit = formData.get('attendee_limit') as string;
@@ -109,7 +109,7 @@ export const actions: Actions = {
 			});
 		}
 
-		// Check if date is in the past using local timezone (but allow editing past events for corrections)
+		// Check if date is in the past using local timezone
 		const [year, month, day] = date.split('-').map(Number);
 		const eventDate = new Date(year, month - 1, day);
 		const today = new Date();
@@ -148,7 +148,6 @@ export const actions: Actions = {
 				}
 			});
 		}
-
 		// Update the event
 		await database
 			.update(events)
@@ -157,7 +156,7 @@ export const actions: Actions = {
 				date: date,
 				time: time,
 				location: location?.trim() || '',
-				locationType: locationType,
+				locationType: locationType as 'none' | 'text' | 'maps',
 				locationUrl: locationType === 'maps' ? locationUrl?.trim() : null,
 				type: type,
 				attendeeLimit: type === 'limited' ? parseInt(attendeeLimit) : null,
diff --git a/src/routes/event/[id]/edit/+page.svelte b/src/routes/event/[id]/edit/+page.svelte
index 676f3be..bbf0ebd 100644
--- a/src/routes/event/[id]/edit/+page.svelte
+++ b/src/routes/event/[id]/edit/+page.svelte
@@ -1,5 +1,5 @@
 <script lang="ts">
-	import type { EventType, LocationType } from '$lib/types';
+	import type { CreateEventData, EventType, LocationType } from '$lib/types';
 	import { enhance } from '$app/forms';
 	import { goto } from '$app/navigation';
 	import { t } from '$lib/i18n/i18n.js';
@@ -7,7 +7,7 @@
 	export let data;
 	export let form;
 
-	let eventData = {
+	let eventData: CreateEventData = {
 		name: data.event.name,
 		date: data.event.date,
 		time: data.event.time,
@@ -15,7 +15,7 @@
 		location_type: data.event.locationType || 'none',
 		location_url: data.event.locationUrl || '',
 		type: data.event.type,
-		attendee_limit: data.event.attendeeLimit,
+		attendee_limit: data.event.attendeeLimit || undefined,
 		visibility: data.event.visibility
 	};
 
@@ -44,14 +44,14 @@
 			attendee_limit: (values as any).attendee_limit
 				? // eslint-disable-next-line @typescript-eslint/no-explicit-any
 					parseInt(String((values as any).attendee_limit))
-				: null
+				: undefined
 		};
 	}
 
 	const handleTypeChange = (type: EventType) => {
 		eventData.type = type;
 		if (type === 'unlimited') {
-			eventData.attendee_limit = null;
+			eventData.attendee_limit = undefined;
 		}
 	};
 
@@ -189,6 +189,9 @@
 
 					<!-- Location Type -->
 					<div>
+						<!-- Hidden input to submit locationType value -->
+						<input type="hidden" name="locationType" bind:value={eventData.location_type} />
+
 						<fieldset>
 							<legend class="text-dark-800 mb-3 block text-sm font-semibold">
 								{t('create.locationTypeLabel')}
@@ -279,6 +282,9 @@
 
 					<!-- Event Type -->
 					<div>
+						<!-- Hidden input to submit type value -->
+						<input type="hidden" name="type" bind:value={eventData.type} />
+
 						<fieldset>
 							<legend class="text-dark-800 mb-3 block text-sm font-semibold">
 								{t('common.type')} <span class="text-red-400">{t('common.required')}</span>
@@ -333,6 +339,9 @@
 
 					<!-- Event Visibility -->
 					<div>
+						<!-- Hidden input to submit visibility value -->
+						<input type="hidden" name="visibility" bind:value={eventData.visibility} />
+
 						<fieldset>
 							<legend class="text-dark-800 mb-3 block text-sm font-semibold">
 								{t('common.visibility')} <span class="text-red-400">{t('common.required')}</span>