diff --git a/CLAUDE.md b/CLAUDE.md index 51df179..8807557 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -177,7 +177,11 @@ Funktion, sie ist die einzige Stelle, an der aus JSON HTML entsteht. - **Varianten statt Zweitformular:** das Kontaktformular hat zwei Ausprägungen, gesteuert vom Hidden-Feld `department` (`''` = allgemein, `'turnen'` = Turnabteilung auf `/turnen`). `contact_variant()` (`helpers.php`) ist die **einzige** Quelle für Auswahlliste, Feldregeln - (Betreff-Feld, Pflichtfelder) und Empfänger; Komponente und Action lesen beide dort. Ein + (Betreff-Feld, Geburtsdatum-Feld, Pflichtfelder) und Empfänger; Komponente und Action lesen beide + dort. Ein variantenspezifisches Feld braucht **drei** Stellen: den Schalter in + `contact_variant()`, die bedingte Ausgabe in `contact-form.php` und in der Action beides — + Validierung **und** ein Zurücksetzen auf leer, wenn die Variante das Feld nicht anbietet (sonst + könnte ein manipulierter POST es an einer Variante vorbei einschmuggeln). Ein unbekannter Wert fällt auf die allgemeine Variante zurück, ein manipuliertes Feld erreicht also keinen fremden Empfänger. Die Turn-Variante geht an `club.email_turnen`, hat die Disziplinen aus `data/turnen.json` als Auswahl (Pflicht, ersetzt den Betreff) und eine freiwillige Nachricht. diff --git a/app/actions/contact-submit.php b/app/actions/contact-submit.php index 022f912..8d537d3 100644 --- a/app/actions/contact-submit.php +++ b/app/actions/contact-submit.php @@ -60,6 +60,7 @@ $name = $field('name'); $email = $field('email'); $phone = $field('phone'); $interest = $field('interest'); +$birthdate = $field('birthdate'); $subject = $field('subject'); $message = $field('message'); $privacy = ($_POST['privacy'] ?? '') === '1'; @@ -74,6 +75,9 @@ if (!$variant['interest_required']) { if (!$variant['ask_subject']) { $subject = ''; // Feld gibt es dort nicht, ein mitgeschicktes ignorieren } +if (!$variant['ask_birthdate']) { + $birthdate = ''; // dito — nur die Turn-Variante fragt danach +} // --- Validierung (Whitelist-Prinzip, Fehler pro Feld) --- $errors = []; @@ -86,6 +90,21 @@ if (filter_var($email, FILTER_VALIDATE_EMAIL) === false) { if (mb_strlen($phone) > 50) { $errors['contact-phone'] = 'Die Telefonnummer ist zu lang (max. 50 Zeichen).'; } +// Geburtsdatum: optional, aber wenn angegeben, dann ein echtes Datum in einem +// plausiblen Bereich. type=date liefert YYYY-MM-DD; createFromFormat mit +// anschließendem Vergleich weist auch Scheindaten wie 2019-02-31 ab (PHP würde +// die sonst stillschweigend auf den 3. März verschieben). +if ($birthdate !== '') { + $d = DateTimeImmutable::createFromFormat('!Y-m-d', $birthdate); + $heute = new DateTimeImmutable('today'); + if ($d === false || $d->format('Y-m-d') !== $birthdate) { + $errors['contact-birthdate'] = 'Bitte gib das Geburtsdatum im Format TT.MM.JJJJ an.'; + } elseif ($d > $heute) { + $errors['contact-birthdate'] = 'Das Geburtsdatum kann nicht in der Zukunft liegen.'; + } elseif ($d < new DateTimeImmutable('1900-01-01')) { + $errors['contact-birthdate'] = 'Bitte prüfe das Geburtsdatum.'; + } +} if (!in_array($interest, $interests, true)) { $errors['contact-interest'] = $variant['interest_required'] ? 'Bitte wähle aus, für welche Gruppe du dich interessierst.' @@ -110,7 +129,8 @@ if ($errors !== []) { // Ohne JS: Seite mit Eingaben und Feldfehlern neu rendern statt PRG (Werte bleiben erhalten). render_form_invalid($return, [ 'name' => $name, 'email' => $email, 'phone' => $phone, 'interest' => $interest, - 'subject' => $subject, 'message' => $message, 'privacy' => $privacy ? '1' : '', + 'birthdate' => $birthdate, 'subject' => $subject, 'message' => $message, + 'privacy' => $privacy ? '1' : '', ], $errors); } @@ -148,6 +168,7 @@ $body = $variant['context'] . " über tsv08kulmbach.de\n" . "E-Mail: {$email}\n" . ($phone !== '' ? "Telefon: {$phone}\n" : '') . ($interest !== '' ? "Interesse: {$interest}\n" : '') + . ($birthdate !== '' ? 'Geburtstag des Kindes: ' . german_date($birthdate) . "\n" : '') . ($subject !== '' ? "Betreff: {$subject}\n" : '') . str_repeat('-', 40) . "\n\n" . ($message !== '' ? $message . "\n\n" : "(keine Nachricht angegeben)\n\n") diff --git a/app/components/contact-form.php b/app/components/contact-form.php index f1e1350..0cf65d9 100644 --- a/app/components/contact-form.php +++ b/app/components/contact-form.php @@ -43,6 +43,14 @@ $f = [ 'email' => ['type' => 'email', 'name' => 'email', 'id' => 'contact-email', 'label' => 'E-Mail', 'required' => true, 'autocomplete' => 'email', 'maxlength' => 200], 'phone' => ['type' => 'tel', 'name' => 'phone', 'id' => 'contact-phone', 'label' => 'Telefonnummer', 'autocomplete' => 'tel', 'maxlength' => 50], 'interest' => ['type' => 'select', 'name' => 'interest', 'id' => 'contact-interest', 'label' => 'Ich interessiere mich für', 'options' => $interests, 'required' => $variant['interest_required']], + // Optional und nur in der Turn-Variante (siehe contact_variant). max/min + // begrenzen die Auswahl im Datepicker schon im Browser; die Action prüft es + // serverseitig noch einmal. + 'birthdate' => [ + 'type' => 'date', 'name' => 'birthdate', 'id' => 'contact-birthdate', + 'label' => 'Geburtsdatum des Kindes', 'hint' => 'Optional. Hilft uns, die passende Gruppe zu finden.', + 'min' => '1900-01-01', 'max' => date('Y-m-d'), + ], 'subject' => ['type' => 'text', 'name' => 'subject', 'id' => 'contact-subject', 'label' => 'Betreff', 'required' => true, 'maxlength' => 200], 'message' => ['type' => 'textarea', 'name' => 'message', 'id' => 'contact-message', 'label' => 'Nachricht', 'required' => $variant['message_required'], 'maxlength' => 5000, 'rows' => 6], 'privacy' => [ @@ -79,6 +87,9 @@ $f = [