change way to unserialize form request

MTES-MCT · Oct 13, 2023 · 39ecb98 · 39ecb98
1 parent 7080979
commit 39ecb98
Show file tree

Hide file tree

Showing 5 changed files with 39 additions and 9 deletions.
diff --git a/src/Controller/Back/SignalementActionController.php b/src/Controller/Back/SignalementActionController.php
@@ -82,7 +82,7 @@ public function validationResponseSignalement(
             $doctrine->getManager()->persist($suivi);
             $doctrine->getManager()->flush();
 
-            $this->addFlash('success', 'Statut du signalement mis à jour avec succés !');
+            $this->addFlash('success', 'Statut du signalement mis à jour avec succès !');
         } else {
             $this->addFlash('error', 'Une erreur est survenue...');
         }
@@ -152,7 +152,7 @@ public function reopenSignalement(Signalement $signalement, Request $request, Ma
             $suivi->setType(SUIVI::TYPE_AUTO);
             $doctrine->getManager()->persist($suivi);
             $doctrine->getManager()->flush();
-            $this->addFlash('success', 'Signalement rouvert avec succés !');
+            $this->addFlash('success', 'Signalement rouvert avec succès !');
         } else {
             $this->addFlash('error', 'Erreur lors de la réouverture du signalement! ');
         }

diff --git a/src/Controller/Back/SignalementController.php b/src/Controller/Back/SignalementController.php
@@ -262,7 +262,7 @@ public function editSignalement(
                 $signalement->setInseeOccupant($form->getExtraData()['inseeOccupant']);
                 $doctrine->getManager()->persist($signalement);
                 $doctrine->getManager()->flush();
-                $this->addFlash('success', 'Signalement modifié avec succés !');
+                $this->addFlash('success', 'Signalement modifié avec succès !');
 
                 return $this->json(['response' => 'success_edited']);
             }

diff --git a/src/Controller/Back/SignalementEditController.php b/src/Controller/Back/SignalementEditController.php
@@ -8,24 +8,52 @@
 use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;
-use Symfony\Component\HttpKernel\Attribute\MapRequestPayload;
 use Symfony\Component\Routing\Annotation\Route;
+use Symfony\Component\Serializer\SerializerInterface;
+use Symfony\Component\Validator\Validator\ValidatorInterface;
 
 #[Route('/bo/signalements')]
 class SignalementEditController extends AbstractController
 {
     #[Route('/{uuid}/edit-address', name: 'back_signalement_edit_address', methods: 'POST')]
     public function validationResponseSignalement(
         Signalement $signalement,
-        #[MapRequestPayload] AdresseOccupantRequest $adresseOccupantRequest,
         Request $request,
         SignalementManager $signalementManager,
+        SerializerInterface $serializer,
+        ValidatorInterface $validator,
     ): Response {
         $this->denyAccessUnlessGranted('SIGN_VALIDATE', $signalement);
         if ($this->isCsrfTokenValid('signalement_edit_address_'.$signalement->getId(), $request->get('_token'))) {
-            $signalementManager->updateFromAdresseOccupantRequest($signalement, $adresseOccupantRequest);
+            /** @var AdresseOccupantRequest $adresseOccupantRequest */
+            $adresseOccupantRequest = new AdresseOccupantRequest(
+                adresse: $request->get('adresse'),
+                codePostal: $request->get('codePostal'),
+                ville: $request->get('ville'),
+                etage: $request->get('etage'),
+                escalier: $request->get('escalier'),
+                numAppart: $request->get('numAppart'),
+                autre: $request->get('autre'),
+                geolocLng: $request->get('geolocLng'),
+                geolocLat: $request->get('geolocLat'),
+                insee: $request->get('insee'),
+            );
 
-            $this->addFlash('success', 'Adresse du logement mise à jour avec succés !');
+            $errorMessage = '';
+            $errors = $validator->validate($adresseOccupantRequest);
+            if (\count($errors) > 0) {
+                $errorMessage = '';
+                foreach ($errors as $error) {
+                    $errorMessage .= $error->getPropertyPath().' : '.$error->getMessage().' ';
+                }
+            }
+
+            if (empty($errorMessage)) {
+                $signalementManager->updateFromAdresseOccupantRequest($signalement, $adresseOccupantRequest);
+                $this->addFlash('success', 'Adresse du logement mise à jour avec succès !');
+            } else {
+                $this->addFlash('error', 'Erreur de saisie : '.$errorMessage);
+            }
         } else {
             $this->addFlash('error', 'Une erreur est survenue...');
         }

diff --git a/src/Dto/Request/Signalement/AdresseOccupantRequest.php b/src/Dto/Request/Signalement/AdresseOccupantRequest.php
@@ -7,10 +7,12 @@
 class AdresseOccupantRequest
 {
     public function __construct(
+        #[Assert\NotBlank()]
         private readonly ?string $adresse = null,
+        #[Assert\NotBlank()]
         private readonly ?string $codePostal = null,
-        private readonly ?string $ville = null,
         #[Assert\NotBlank()]
+        private readonly ?string $ville = null,
         private readonly ?string $etage = null,
         private readonly ?string $escalier = null,
         private readonly ?string $numAppart = null,

diff --git a/templates/back/signalement/view/edit-modals/edit-address.html.twig b/templates/back/signalement/view/edit-modals/edit-address.html.twig
@@ -2,7 +2,7 @@
     <div class="fr-container fr-container--fluid fr-container-md">
         <div class="fr-grid-row fr-grid-row--center">
             <div class="fr-col-12 fr-col-md-8">
-                <form method="POST" id="form-edit-address" enctype="multipart/form-data" action="{{ path('back_signalement_edit_address',{uuid:signalement.uuid}) }}">
+                <form method="POST" id="form-edit-address" enctype="application/json" action="{{ path('back_signalement_edit_address',{uuid:signalement.uuid}) }}">
                     <div class="fr-modal__body">
                         <div class="fr-modal__header">
                             <button class="fr-btn--close fr-btn" title="Fermer la fenêtre modale" aria-controls="fr-modal-edit-address">Fermer</button>