marcel/familienarchiv
1
0
Fork 0
Code Issues 105 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

fix(aktivitaeten): narrow File cast and use null payload for missing notificationId

Browse Source 
Replace 'as string | null' cast (which silently accepts File values) with an explicit
typeof check. Use error: null instead of hardcoded German so the client falls through
to the generic i18n-keyed error banner.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Marcel
2026-05-20 07:06:15 +02:00
committed by marcel
parent 978a2b3cdb
commit 35fbaf8154
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
frontend/src/routes/aktivitaeten/+page.server.ts
View File

@@ -71,8 +71,9 @@ export async function load({ fetch, url }) {
export const actions = { export const actions = {
'dismiss-notification': async ({ request, fetch }) => { 'dismiss-notification': async ({ request, fetch }) => {
const data = await request.formData(); const data = await request.formData();
const notificationId = data.get('notificationId') as string | null; const raw = data.get('notificationId');
if (!notificationId) return fail(400, { error: 'Ungültige Benachrichtigungs-ID' }); const notificationId = typeof raw === 'string' ? raw : null;
if (!notificationId) return fail(400, { error: null });
const api = createApiClient(fetch); const api = createApiClient(fetch);
const result = await api.PATCH('/api/notifications/{id}/read', { const result = await api.PATCH('/api/notifications/{id}/read', {
params: { path: { id: notificationId } } params: { path: { id: notificationId } }