add validator #2168

MTES-MCT · Feb 27, 2024 · cc99774 · cc99774
1 parent 694fd45
commit cc99774
Show file tree

Hide file tree

Showing 17 changed files with 378 additions and 139 deletions.
diff --git a/src/Controller/Back/SignalementEditController.php b/src/Controller/Back/SignalementEditController.php
@@ -236,7 +236,10 @@ public function editInformationsLogement(
                 'json'
             );
 
-            $validationGroups = ['Default', 'EDIT'];
+            $validationGroups = ['Default'];
+            if ($signalement->getProfileDeclarant()) {
+                $validationGroups[] = $signalement->getProfileDeclarant()->value;
+            }
             $errorMessage = FormHelper::getErrorsFromRequest($validator, $informationsLogementRequest, $validationGroups);
 
             if (empty($errorMessage)) {
@@ -285,15 +288,11 @@ public function editCompositionLogement(
             );
 
             $validationGroups = ['Default'];
-            if ('autre' == $compositionLogementRequest->getType()) {
-                $validationGroups[] = 'TYPE_LOGEMENT_AUTRE';
-            }
             if ($signalement->getProfileDeclarant()) {
                 $validationGroups[] = $signalement->getProfileDeclarant()->value;
             }
 
             $errorMessage = FormHelper::getErrorsFromRequest($validator, $compositionLogementRequest, $validationGroups);
-
             if (empty($errorMessage)) {
                 $signalementManager->updateFromCompositionLogementRequest($signalement, $compositionLogementRequest);
                 $response = [
@@ -339,7 +338,10 @@ public function editSituationFoyer(
                 'json'
             );
 
-            $validationGroups = ['Default', 'EDIT'];
+            $validationGroups = ['Default'];
+            if ($signalement->getProfileDeclarant()) {
+                $validationGroups[] = $signalement->getProfileDeclarant()->value;
+            }
             $errorMessage = FormHelper::getErrorsFromRequest($validator, $situationFoyerRequest, $validationGroups);
 
             if (empty($errorMessage)) {
@@ -386,8 +388,10 @@ public function editProcedureDemarches(
                 ProcedureDemarchesRequest::class,
                 'json'
             );
-
-            $validationGroups = ['Default', 'EDIT'];
+            $validationGroups = ['Default'];
+            if ($signalement->getProfileDeclarant()) {
+                $validationGroups[] = $signalement->getProfileDeclarant()->value;
+            }
             $errorMessage = FormHelper::getErrorsFromRequest($validator, $procedureDemarchesRequest, $validationGroups);
 
             if (empty($errorMessage)) {

diff --git a/src/Dto/Request/Signalement/CompositionLogementRequest.php b/src/Dto/Request/Signalement/CompositionLogementRequest.php
@@ -4,14 +4,22 @@
 
 use App\Dto\Request\RequestInterface;
 use Symfony\Component\Validator\Constraints as Assert;
+use Symfony\Component\Validator\Context\ExecutionContextInterface;
 
 class CompositionLogementRequest implements RequestInterface
 {
     public function __construct(
         #[Assert\NotBlank(message: 'Merci de définir le type de logement.')]
         private readonly ?string $type = null,
-        #[Assert\NotBlank(message: 'Merci de préciser le type de logement autre.', groups: ['TYPE_LOGEMENT_AUTRE'])]
+        #[Assert\When(
+            expression: 'this.getType() == "autre"',
+            constraints: [
+                new Assert\NotBlank(message: 'Merci de préciser le type de logement autre.'),
+            ],
+        )]
         private readonly ?string $typeLogementNatureAutrePrecision = null,
+        #[Assert\NotBlank(message: 'Merci de selectioner pièce unique ou plusieurs pièces.')]
+        #[Assert\Choice(options: ['piece_unique', 'plusieurs_pieces'], message: 'Merci de selectioner pièce unique ou plusieurs pièces.')]
         private readonly ?string $typeCompositionLogement = null,
         #[Assert\NotBlank(message: 'Merci de saisir la superficie du logement.', groups: ['LOCATAIRE', 'BAILLEUR_OCCUPANT'])]
         #[Assert\Positive(message: 'Merci de saisir une information numérique dans le champs de superficie.')]
@@ -22,21 +30,70 @@ public function __construct(
         #[Assert\Positive(message: 'Merci de saisir une information numérique dans le champs nombre de pièces à vivre.')]
         private readonly ?string $compositionLogementNbPieces = null,
         private readonly ?string $nombreEtages = null,
+        #[Assert\When(
+            expression: 'this.getType() == "appartement"',
+            constraints: [
+                new Assert\NotBlank(message: 'Merci de préciser si le logement est au rez-de-chaussée.'),
+            ],
+        )]
         private readonly ?string $typeLogementRdc = null,
+        #[Assert\When(
+            expression: 'this.getType() == "appartement" && this.getTypeLogementRdc() == "non"',
+            constraints: [
+                new Assert\NotBlank(message: 'Merci de préciser si le logement est au dernier étage.'),
+            ],
+        )]
         private readonly ?string $typeLogementDernierEtage = null,
+        #[Assert\When(
+            expression: 'this.getType() == "appartement" && this.getTypeLogementDernierEtage() == "oui"',
+            constraints: [
+                new Assert\NotBlank(message: 'Merci de préciser si le logement est sous comble et sans fenêtre.'),
+            ],
+        )]
         private readonly ?string $typeLogementSousCombleSansFenetre = null,
+        #[Assert\When(
+            expression: 'this.getType() == "appartement" && this.getTypeLogementRdc() == "non" && this.getTypeLogementDernierEtage() == "non"',
+            constraints: [
+                new Assert\NotBlank(message: 'Merci de préciser si le logement est au sous-sol et sans fenêtre.'),
+            ],
+        )]
         private readonly ?string $typeLogementSousSolSansFenetre = null,
         #[Assert\NotBlank(message: 'Merci de définir si une pièce fait plus de 9m².', groups: ['LOCATAIRE', 'BAILLEUR_OCCUPANT', 'BAILLEUR', 'TIERS_PARTICULIER'])]
         private readonly ?string $typeLogementCommoditesPieceAVivre9m = null,
         #[Assert\NotBlank(message: 'Merci de définir si il y a une cuisine.', groups: ['LOCATAIRE', 'BAILLEUR_OCCUPANT', 'BAILLEUR', 'TIERS_PARTICULIER', 'TIERS_PRO', 'SERVICE_SECOURS'])]
         private readonly ?string $typeLogementCommoditesCuisine = null,
+        #[Assert\When(
+            expression: 'this.getTypeLogementCommoditesCuisine() == "non"',
+            constraints: [
+                new Assert\NotBlank(message: 'Merci de préciser s\'il y a une cuisine collective.'),
+            ],
+        )]
         private readonly ?string $typeLogementCommoditesCuisineCollective = null,
         #[Assert\NotBlank(message: 'Merci de définir si il y a une salle de bain.', groups: ['LOCATAIRE', 'BAILLEUR_OCCUPANT', 'BAILLEUR', 'TIERS_PARTICULIER', 'TIERS_PRO', 'SERVICE_SECOURS'])]
         private readonly ?string $typeLogementCommoditesSalleDeBain = null,
+        #[Assert\When(
+            expression: 'this.getTypeLogementCommoditesSalleDeBain() == "non"',
+            constraints: [
+                new Assert\NotBlank(message: 'Merci de préciser s\'il y a une salle de bain collective.'),
+            ],
+        )]
         private readonly ?string $typeLogementCommoditesSalleDeBainCollective = null,
         #[Assert\NotBlank(message: 'Merci de définir si il y a des WC.', groups: ['LOCATAIRE', 'BAILLEUR_OCCUPANT', 'BAILLEUR', 'TIERS_PARTICULIER', 'TIERS_PRO', 'SERVICE_SECOURS'])]
         private readonly ?string $typeLogementCommoditesWc = null,
+        #[Assert\When(
+            expression: 'this.getTypeLogementCommoditesWc() == "non"',
+            constraints: [
+                new Assert\NotBlank(message: 'Merci de préciser s\'il y a des wc collectifs.'),
+            ],
+        )]
         private readonly ?string $typeLogementCommoditesWcCollective = null,
+
+        #[Assert\When(
+            expression: 'this.getTypeLogementCommoditesWc() == "oui" && this.getTypeLogementCommoditesCuisine() == "oui"',
+            constraints: [
+                new Assert\NotBlank(message: 'Merci de préciser s\'il y a des wc dans la cuisine.'),
+            ],
+        )]
         private readonly ?string $typeLogementCommoditesWcCuisine = null,
     ) {
     }
@@ -135,4 +192,18 @@ public function getTypeLogementCommoditesWcCuisine(): ?string
     {
         return $this->typeLogementCommoditesWcCuisine;
     }
+
+    #[Assert\Callback]
+    public function validate(ExecutionContextInterface $context, mixed $payload): void
+    {
+        if ($this->getTypeLogementDernierEtage() === $this->getTypeLogementRdc()) {
+            $context->buildViolation('Merci de bien préciser si le logement est au rez-de-chaussée ou dernier étage.')
+                ->atPath('typeLogementRdc')
+                ->addViolation();
+
+            $context->buildViolation('Merci de bien préciser si le logement est au rez-de-chaussée ou dernier étage.')
+                ->atPath('typeLogementDernierEtage')
+                ->addViolation();
+        }
+    }
 }
diff --git a/src/Dto/Request/Signalement/CoordonneesFoyerRequest.php b/src/Dto/Request/Signalement/CoordonneesFoyerRequest.php
@@ -11,12 +11,19 @@ class CoordonneesFoyerRequest implements RequestInterface
 {
     public function __construct(
         private readonly ?string $typeProprio = null,
+        #[Assert\When(
+            expression: 'this.getTypeProprio() == "ORGANISME_SOCIETE"',
+            constraints: [
+                new Assert\NotBlank(message: 'Merci de saisir un nom de structure.'),
+            ],
+        )]
         private readonly ?string $nomStructure = null,
+        #[Assert\NotBlank(message: 'Merci de selectionner une civilité.', groups: ['LOCATAIRE'])]
         private readonly ?string $civilite = null,
-        #[Assert\NotBlank(message: 'Merci de saisir un nom.', groups: ['LOCATAIRE', 'BAILLEUR_OCCUPANT'])]
+        #[Assert\NotBlank(message: 'Merci de saisir un nom.', groups: ['LOCATAIRE', 'BAILLEUR_OCCUPANT', 'TIERS_PARTICULIER', 'TIERS_PRO', 'BAILLEUR'])]
         #[Assert\Length(max: 50, maxMessage: 'Le nom ne doit pas dépasser {{ limit }} caractères.')]
         private readonly ?string $nom = null,
-        #[Assert\NotBlank(message: 'Merci de saisir un prénom.', groups: ['LOCATAIRE', 'BAILLEUR_OCCUPANT'])]
+        #[Assert\NotBlank(message: 'Merci de saisir un prénom.', groups: ['LOCATAIRE', 'BAILLEUR_OCCUPANT', 'TIERS_PARTICULIER', 'TIERS_PRO', 'BAILLEUR'])]
         #[Assert\Length(max: 50, maxMessage: 'Le prénom ne doit pas dépasser {{ limit }} caractères.')]
         private readonly ?string $prenom = null,
         #[Assert\NotBlank(message: 'Merci de saisir un email', groups: ['LOCATAIRE', 'BAILLEUR_OCCUPANT'])]

diff --git a/src/Dto/Request/Signalement/CoordonneesTiersRequest.php b/src/Dto/Request/Signalement/CoordonneesTiersRequest.php
@@ -24,6 +24,12 @@ public function __construct(
         #[AppAssert\TelephoneFormat]
         private readonly ?string $telephone = null,
         private readonly ?string $lien = null,
+        #[Assert\When(
+            expression: 'this.getLien() == "PRO" || this.getLien() == "SECOURS"',
+            constraints: [
+                new Assert\NotBlank(message: 'Merci de saisir un nom de structure.'),
+            ],
+        )]
         private readonly ?string $structure = null,
     ) {
     }

diff --git a/src/Dto/Request/Signalement/ProcedureDemarchesRequest.php b/src/Dto/Request/Signalement/ProcedureDemarchesRequest.php
@@ -8,10 +8,16 @@
 class ProcedureDemarchesRequest implements RequestInterface
 {
     public function __construct(
-        #[Assert\NotBlank(['message' => 'Merci d\'indiquer si le bailleur a été averti', 'groups' => ['LOCATAIRE']])]
+        #[Assert\NotBlank(['message' => 'Merci d\'indiquer si le bailleur a été averti.', 'groups' => ['LOCATAIRE']])]
         private readonly ?string $isProprioAverti = null,
-        #[Assert\NotBlank(['message' => 'Merci d\'indiquer si l\'assurance a été contactée', 'groups' => ['LOCATAIRE', 'BAILLEUR_OCCUPANT', 'BAILLEUR']])]
+        #[Assert\NotBlank(['message' => 'Merci d\'indiquer si l\'assurance a été contactée.', 'groups' => ['LOCATAIRE', 'BAILLEUR_OCCUPANT', 'BAILLEUR']])]
         private readonly ?string $infoProcedureAssuranceContactee = null,
+        #[Assert\When(
+            expression: 'this.getInfoProcedureAssuranceContactee() == "oui"',
+            constraints: [
+                new Assert\NotBlank(message: 'Merci de noter la réponde de l\'assurance.'),
+            ],
+        )]
         private readonly ?string $infoProcedureReponseAssurance = null,
         #[Assert\NotBlank(['message' => 'Merci d\'indiquer si l\'occupant souhaite garder son logement après travaux ', 'groups' => ['LOCATAIRE', 'BAILLEUR_OCCUPANT']])]
         private readonly ?string $infoProcedureDepartApresTravaux = null,

diff --git a/src/Dto/Request/Signalement/SituationFoyerRequest.php b/src/Dto/Request/Signalement/SituationFoyerRequest.php
@@ -14,11 +14,23 @@ public function __construct(
         #[Assert\NotBlank(['message' => 'Veuillez définir le champ allocataire', 'groups' => ['LOCATAIRE', 'BAILLEUR_OCCUPANT', 'BAILLEUR', 'TIERS_PARTICULIER', 'TIERS_PRO']])]
         private readonly ?string $isAllocataire = null,
         #[Assert\DateTime('Y-m-d')]
+        #[Assert\When(
+            expression: 'this.getIsAllocataire() == "oui" || this.getIsAllocataire() == "CAF" || this.getIsAllocataire() == "MSA"',
+            constraints: [
+                new Assert\NotBlank(message: 'Merci de préciser la date de naissance.'),
+            ],
+        )]
         private readonly ?string $dateNaissanceOccupant = null,
         private readonly ?string $numAllocataire = null,
         private readonly ?string $logementSocialMontantAllocation = null,
         #[Assert\NotBlank(['message' => 'Veuillez définir le champ souhaite quitter le logement', 'groups' => ['LOCATAIRE', 'BAILLEUR_OCCUPANT', 'TIERS_PARTICULIER', 'TIERS_PRO']])]
         private readonly ?string $travailleurSocialQuitteLogement = null,
+        #[Assert\When(
+            expression: 'this.getTravailleurSocialQuitteLogement() == "oui"',
+            constraints: [
+                new Assert\NotBlank(message: 'Merci de préciser s\'il y a un préavis de départ.'),
+            ],
+        )]
         private readonly ?string $travailleurSocialPreavisDepart = null,
         #[Assert\NotBlank(['message' => 'Veuillez définir le champ accompagnement par un travailleur social', 'groups' => ['LOCATAIRE', 'BAILLEUR_OCCUPANT', 'TIERS_PARTICULIER', 'TIERS_PRO']])]
         private readonly ?string $travailleurSocialAccompagnementDeclarant = null,

diff --git a/src/Serializer/SignalementDraftRequestNormalizer.php b/src/Serializer/SignalementDraftRequestNormalizer.php
@@ -2,8 +2,7 @@
 
 namespace App\Serializer;
 
-use App\Dto\Request\Signalement\AdresseOccupantRequest;
-use App\Dto\Request\Signalement\CompositionLogementRequest;
+use App\Dto\Request\RequestInterface;
 use App\Dto\Request\Signalement\SignalementDraftRequest;
 use App\Entity\Model\TypeCompositionLogement;
 use App\Entity\SignalementDraft;
@@ -48,7 +47,11 @@ public function denormalize($data, $type, $format = null, array $context = []):
 
                 $transformedData[SignalementDraftRequest::FILE_UPLOAD_KEY][$keyUpdated] = $data[$key];
             } else {
-                $transformedData[$key] = !empty($value) ? $value : null;
+                if ('isProprioAverti' === $key) {
+                    $transformedData[$key] = $value;
+                } else {
+                    $transformedData[$key] = !empty($value) ? $value : null;
+                }
             }
         }
 
@@ -89,8 +92,7 @@ public function getSupportedTypes(?string $format): array
         return [
             SignalementDraftRequest::class => true,
             TypeCompositionLogement::class => true,
-            CompositionLogementRequest::class => true,
-            AdresseOccupantRequest::class => true,
+            RequestInterface::class => true,
         ];
     }
 }
diff --git a/src/Twig/AppExtension.php b/src/Twig/AppExtension.php
@@ -3,11 +3,14 @@
 namespace App\Twig;
 
 use App\Entity\Enum\OccupantLink;
+use App\Entity\Enum\ProfileDeclarant;
 use App\Entity\Enum\QualificationStatus;
 use App\Service\Esabora\EsaboraPartnerTypeSubscription;
 use App\Service\Files\ImageBase64Encoder;
 use App\Service\Notification\NotificationCounter;
 use App\Service\Signalement\Qualification\QualificationStatusService;
+use Symfony\Component\Validator\Constraints\NotBlank;
+use Symfony\Component\Validator\Constraints\When;
 use Twig\Extension\AbstractExtension;
 use Twig\TwigFilter;
 use Twig\TwigFunction;
@@ -56,6 +59,28 @@ public function getFunctions(): array
             new TwigFunction('can_see_nde_qualification', [QualificationStatusService::class, 'canSeenNDEQualification']),
             new TwigFunction('can_see_nde_edit_zone', [QualificationStatusService::class, 'canSeenNDEEditZone']),
             new TwigFunction('can_edit_esabora_credentials', [EsaboraPartnerTypeSubscription::class, 'isSubscribed']),
+            new TwigFunction('show_label_facultatif', [$this, 'showLabelAsFacultatif']),
         ];
     }
+
+    public function showLabelAsFacultatif(string $class, string $field, ?ProfileDeclarant $profileDeclarant): string
+    {
+        if (null === $profileDeclarant) {
+            return '';
+        }
+
+        if ('nomStructure' === $field) {
+            $reflector = new \ReflectionClass($class);
+            $attributes = $reflector->getProperty($field)->getAttributes(When::class);
+            $constraints = $attributes[0]->getArguments()['constraints'] ?? [];
+
+            return \count($constraints) > 0 ? '(facultatif)' : '';
+        }
+
+        $reflector = new \ReflectionClass($class);
+        $attributes = $reflector->getProperty($field)->getAttributes(NotBlank::class);
+        $groups = $attributes[0]->getArguments()['groups'] ?? [];
+
+        return !empty($groups) && !\in_array($profileDeclarant->value, $groups) ? '(facultatif)' : '';
+    }
 }
diff --git a/src/Validator/TelephoneFormatValidator.php b/src/Validator/TelephoneFormatValidator.php
@@ -33,13 +33,11 @@ public function validate(mixed $value, Constraint $constraint): void
             if (!$isPossible) {
                 $this->context->buildViolation($constraint->message)
                     ->setParameter('{{ value }}', $value)
-                    ->atPath('telephone')
                     ->addViolation();
             }
         } catch (NumberParseException $e) {
             $this->context->buildViolation($constraint->message)
                 ->setParameter('{{ value }}', $value)
-                ->atPath('telephone')
                 ->addViolation();
         }
     }

diff --git a/templates/back/signalement/view/edit-modals/edit-address.html.twig b/templates/back/signalement/view/edit-modals/edit-address.html.twig
@@ -53,7 +53,7 @@
                                     <input type="hidden" name="geolocLng" value="{% if signalement.geoloc.lng is defined %}{{ signalement.geoloc.lng }}{% endif %}">
                                 </div>
                                 <div class="fr-col-12 fr-col-md-6">
-                                    <h2 class="fr-h5">Complément d'adresse</h2>
+                                    <h2 class="fr-h5">Complément d'adresse (facultatif)</h2>
 
                                     <div class="fr-input-group">
                                         <label class="fr-label" for="etage">Etage