Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FO - Formulaire] Empêcher la date de naissance dans le futur #3756

Merged
merged 2 commits into from
Feb 27, 2025

Conversation

hmeneuvrier
Copy link
Collaborator

@hmeneuvrier hmeneuvrier commented Feb 25, 2025

Ticket

#3723

Description

Je suis usager
Dans la situation du foyer, sur la question de la date de naissance
Je peux indiquer une date de naissance dans le futur sans être bloqué
-> On devrait m'empêcher de sélectionner une date dans le futur

Changements apportés

  • Ajout d'un paramètre dans les json du formulaire et d'une condition dans le component Validator
  • Ajout d'une assertion en callback dans les différents Dto
  • Mise à jour des tests

Pré-requis

make composer
npm run watch-form

Tests

  • déposer un signalement en allocataire et vérifier qu'on ne peut pas mettre une date de naissance dans le futur (tout profil sauf service secours, si bailleur vérifier pour la date de naissance de l'allocataire et celle du bailleur)
  • vérifier qu'on peut soumettre le signalement
  • dans le BO vérifier l'édition de la date de naissance de l'occupant et du bailleur


class CoordonneesBailleurRequest implements RequestInterface
{
use DateNaissanceValidatorTrait;
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'ai choisi de faire un Trait, car sinon on aurait pu utiliser
#[Assert\LessThan(value: 'today')]
Mais il aurait fallu transformer tous les champs de date de naissance qui sont des string pour en faire des DateTimeInterface. J'avais peur des effets de bord.

Copy link
Collaborator

@emilschn emilschn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK lecture et test

@emilschn emilschn merged commit 720dcd3 into develop Feb 27, 2025
4 checks passed
@numew numew modified the milestone: 2.10.7 Feb 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants