Persist email in lower case (#10228)

parti-renaissance · May 16, 2024 · e510580 · e510580
1 parent 1f938eb
commit e510580
Show file tree

Hide file tree

Showing 6 changed files with 45 additions and 24 deletions.
diff --git a/src/Adhesion/Command/PersistAdhesionEmailCommand.php b/src/Adhesion/Command/PersistAdhesionEmailCommand.php
@@ -4,10 +4,17 @@
 
 class PersistAdhesionEmailCommand
 {
-    public function __construct(
-        public readonly string $email,
-        public readonly ?string $utmSource = null,
-        public readonly ?string $utmCampaign = null,
-    ) {
+    private string $email;
+    public ?string $utmSource = null;
+    public ?string $utmCampaign = null;
+
+    public function getEmail(): string
+    {
+        return $this->email;
+    }
+
+    public function setEmail(string $email): void
+    {
+        $this->email = mb_strtolower($email);
     }
 }
diff --git a/src/Adhesion/Handler/PersistAdhesionEmailCommandHandler.php b/src/Adhesion/Handler/PersistAdhesionEmailCommandHandler.php
@@ -21,13 +21,13 @@ public function __construct(
 
     public function __invoke(PersistAdhesionEmailCommand $command): ?string
     {
-        if ($adherent = $this->adherentRepository->findOneByEmail($command->email)) {
+        if ($adherent = $this->adherentRepository->findOneByEmail($command->getEmail())) {
             $this->membershipNotifier->sendConnexionDetailsMessage($adherent);
 
             return null;
         }
 
-        $this->entityManager->persist($object = AdherentRequest::createForEmail($command->email));
+        $this->entityManager->persist($object = AdherentRequest::createForEmail($command->getEmail()));
 
         $object->utmCampaign = $command->utmCampaign;
         $object->utmSource = $command->utmSource;

diff --git a/src/BesoinDEurope/Inscription/Command/PersistInscriptionEmailCommand.php b/src/BesoinDEurope/Inscription/Command/PersistInscriptionEmailCommand.php
@@ -4,11 +4,18 @@
 
 class PersistInscriptionEmailCommand
 {
-    public function __construct(
-        public readonly string $email,
-        public readonly ?string $utmSource = null,
-        public readonly ?string $utmCampaign = null,
-        public ?string $clientIp = null
-    ) {
+    private string $email;
+    public ?string $utmSource = null;
+    public ?string $utmCampaign = null;
+    public ?string $clientIp = null;
+
+    public function getEmail(): string
+    {
+        return $this->email;
+    }
+
+    public function setEmail(string $email): void
+    {
+        $this->email = mb_strtolower($email);
     }
 }
diff --git a/src/BesoinDEurope/Inscription/Handler/PersistInscriptionEmailCommandHandler.php b/src/BesoinDEurope/Inscription/Handler/PersistInscriptionEmailCommandHandler.php
@@ -22,13 +22,13 @@ public function __construct(
 
     public function __invoke(PersistInscriptionEmailCommand $command): ?string
     {
-        if ($adherent = $this->adherentRepository->findOneByEmail($command->email)) {
+        if ($adherent = $this->adherentRepository->findOneByEmail($command->getEmail())) {
             $this->membershipNotifier->sendConnexionDetailsMessage($adherent, AppCodeEnum::BESOIN_D_EUROPE);
 
             return null;
         }
 
-        $this->entityManager->persist($object = InscriptionRequest::createForEmail($command->email));
+        $this->entityManager->persist($object = InscriptionRequest::createForEmail($command->getEmail()));
 
         $object->utmCampaign = $command->utmCampaign;
         $object->utmSource = $command->utmSource;

diff --git a/src/Procuration/Command/PersistProcurationEmailCommand.php b/src/Procuration/Command/PersistProcurationEmailCommand.php
@@ -6,12 +6,19 @@
 
 class PersistProcurationEmailCommand
 {
-    public function __construct(
-        public readonly string $email,
-        public readonly InitialRequestTypeEnum $type,
-        public readonly ?string $utmSource = null,
-        public readonly ?string $utmCampaign = null,
-        public ?string $clientIp = null
-    ) {
+    private string $email;
+    public InitialRequestTypeEnum $type;
+    public ?string $utmSource = null;
+    public ?string $utmCampaign = null;
+    public ?string $clientIp = null;
+
+    public function getEmail(): string
+    {
+        return $this->email;
+    }
+
+    public function setEmail(string $email): void
+    {
+        $this->email = mb_strtolower($email);
     }
 }
diff --git a/src/Procuration/Handler/PersistProcurationEmailCommandHandler.php b/src/Procuration/Handler/PersistProcurationEmailCommandHandler.php
@@ -19,10 +19,10 @@ public function __construct(
 
     public function __invoke(PersistProcurationEmailCommand $command): ?string
     {
-        $initialRequest = $this->findInitialRequest($command->email);
+        $initialRequest = $this->findInitialRequest($command->getEmail());
 
         if (!$initialRequest) {
-            $initialRequest = ProcurationRequest::createForEmail($command->email);
+            $initialRequest = ProcurationRequest::createForEmail($command->getEmail());
 
             $this->entityManager->persist($initialRequest);
         }