From 5d43c2d42e6265c87556a03aaa4caa30d84df7a6 Mon Sep 17 00:00:00 2001
From: "noah.lancien" <noah.lancien@ldc.fr>
Date: Thu, 6 Apr 2023 09:55:47 +0000
Subject: [PATCH 1/3] Added more text in system messages

---
 app/Listeners/AddStatusMessages.php | 31 +++++++++++++++++++++++++++++
 1 file changed, 31 insertions(+)

diff --git a/app/Listeners/AddStatusMessages.php b/app/Listeners/AddStatusMessages.php
index 491fcf8..6d55011 100644
--- a/app/Listeners/AddStatusMessages.php
+++ b/app/Listeners/AddStatusMessages.php
@@ -2,6 +2,7 @@
 
 namespace App\Listeners;
 
+use App\Enums\TicketStatus;
 use App\Events\TicketStatusChanged;
 use App\Models\Message;
 
@@ -25,6 +26,36 @@ class AddStatusMessages
         } else {
             $message->message = "Le statut est passé à : <b>{$event->ticket->status->label}</b>";
         }
+        switch ($event->ticket->status) {
+            case TicketStatus::new():
+                $message->message .= "<br>Votre ticket a été créé, nous allons en prendre connaissance.<br>";
+                break;
+            case TicketStatus::estimateTodo():
+                // phpcs:ignore Generic.Files.LineLength.TooLong
+                $message->message .= "<br>Votre ticket nécessite une estimation, nous allons vous proposer un devis.<br>";
+                break;
+            case TicketStatus::takenInAccount():
+                $message->message .= "<br>Votre ticket est dans le processus de traitement.<br>";
+                break;
+            case TicketStatus::waiting():
+                // phpcs:ignore Generic.Files.LineLength.TooLong
+                $message->message .= "<br>Nous attendons une réponse de votre part pour procéder à la suite du traitement.<br>";
+                break;
+            case TicketStatus::waitingValidation():
+                $message->message .= "<br>Nous attendons une validation de votre part pour procéder au traitement.<br>";
+                break;
+            case TicketStatus::recette():
+                // phpcs:ignore Generic.Files.LineLength.TooLong
+                $message->message .= "<br>Le traitement à été réalisé et mis à disposition sur la plateforme de recette. Vous pouvez vérifier que la solution mise en œuvre correspond à la demande.<br>";
+                break;
+            case TicketStatus::production():
+                // phpcs:ignore Generic.Files.LineLength.TooLong
+                $message->message .= "<br>La solution à été mise en ligne. Vous pouvez vérifier sur le site de production que tout fonctionne comme attendu.<br>";
+                break;
+            case TicketStatus::archived():
+                $message->message .= "<br>Ce ticket est clos.<br>";
+                break;
+        }
         $message->save();
     }
 }

From a13124f2c6eea751456f9a3f8889d7fc5c9d0f96 Mon Sep 17 00:00:00 2001
From: "noah.lancien" <noah.lancien@ldc.fr>
Date: Tue, 11 Apr 2023 15:22:29 +0000
Subject: [PATCH 2/3] Used sprintf() and __() in texts

---
 app/Listeners/AddStatusMessages.php | 26 ++++++++++++++++++--------
 1 file changed, 18 insertions(+), 8 deletions(-)

diff --git a/app/Listeners/AddStatusMessages.php b/app/Listeners/AddStatusMessages.php
index 6d55011..2e00837 100644
--- a/app/Listeners/AddStatusMessages.php
+++ b/app/Listeners/AddStatusMessages.php
@@ -28,32 +28,42 @@ class AddStatusMessages
         }
         switch ($event->ticket->status) {
             case TicketStatus::new():
-                $message->message .= "<br>Votre ticket a été créé, nous allons en prendre connaissance.<br>";
+                $message->message .= sprintf("<br>" .
+                __("Votre ticket a été créé, nous allons en prendre connaissance.") . "<br>");
                 break;
             case TicketStatus::estimateTodo():
                 // phpcs:ignore Generic.Files.LineLength.TooLong
-                $message->message .= "<br>Votre ticket nécessite une estimation, nous allons vous proposer un devis.<br>";
+                $message->message .= sprintf("<br>" .
+                __("Votre ticket nécessite une estimation, nous allons vous proposer un devis.") . "<br>");
                 break;
             case TicketStatus::takenInAccount():
-                $message->message .= "<br>Votre ticket est dans le processus de traitement.<br>";
+                $message->message .= sprintf("<br>" .
+                __("Votre ticket est dans le processus de traitement.") . "<br>");
                 break;
             case TicketStatus::waiting():
                 // phpcs:ignore Generic.Files.LineLength.TooLong
-                $message->message .= "<br>Nous attendons une réponse de votre part pour procéder à la suite du traitement.<br>";
+                $message->message .= sprintf("<br>" .
+                __("Nous attendons une réponse de votre part pour procéder à la suite du traitement.") . "<br>");
                 break;
             case TicketStatus::waitingValidation():
-                $message->message .= "<br>Nous attendons une validation de votre part pour procéder au traitement.<br>";
+                $message->message .= sprintf("<br>" .
+                __("Nous attendons une validation de votre part pour procéder au traitement.") . "<br>");
                 break;
             case TicketStatus::recette():
                 // phpcs:ignore Generic.Files.LineLength.TooLong
-                $message->message .= "<br>Le traitement à été réalisé et mis à disposition sur la plateforme de recette. Vous pouvez vérifier que la solution mise en œuvre correspond à la demande.<br>";
+                $message->message .= sprintf("<br>" .
+                __("Le traitement à été réalisé et mis à disposition sur la plateforme de recette. 
+                Vous pouvez vérifier que la solution mise en œuvre correspond à la demande.") . "<br>");
                 break;
             case TicketStatus::production():
                 // phpcs:ignore Generic.Files.LineLength.TooLong
-                $message->message .= "<br>La solution à été mise en ligne. Vous pouvez vérifier sur le site de production que tout fonctionne comme attendu.<br>";
+                $message->message .= sprintf("<br>" .
+                __("La solution à été mise en ligne. 
+                Vous pouvez vérifier sur le site de production que tout fonctionne comme attendu.") . "<br>");
                 break;
             case TicketStatus::archived():
-                $message->message .= "<br>Ce ticket est clos.<br>";
+                $message->message .= sprintf("<br>" .
+                __("Ce ticket est clos.") . "<br>");
                 break;
         }
         $message->save();

From 8e0a51e3e6b7e340c796e09449a1e833652117bb Mon Sep 17 00:00:00 2001
From: "noah.lancien" <noah.lancien@ldc.fr>
Date: Tue, 11 Apr 2023 15:33:58 +0000
Subject: [PATCH 3/3] Cleaned sprintf

---
 app/Listeners/AddStatusMessages.php | 70 ++++++++++++++++++++---------
 1 file changed, 48 insertions(+), 22 deletions(-)

diff --git a/app/Listeners/AddStatusMessages.php b/app/Listeners/AddStatusMessages.php
index 2e00837..cb6546a 100644
--- a/app/Listeners/AddStatusMessages.php
+++ b/app/Listeners/AddStatusMessages.php
@@ -22,48 +22,74 @@ class AddStatusMessages
             $message->message = $event->message;
         } elseif ($event->user) {
             // phpcs:ignore Generic.Files.LineLength.TooLong
-            $message->message = "<b>{$event->user->name}</b> a modifié le statut en <b>{$event->ticket->status->label}</b>";
+            $message->message = sprintf(
+                "<b>%s</b> %s <b>%s</b>",
+                $event->user->name,
+                __("a modifié le statut en"),
+                $event->ticket->status->label
+            );
         } else {
-            $message->message = "Le statut est passé à : <b>{$event->ticket->status->label}</b>";
+            $message->message = sprintf(
+                "%s : <b>%s</b>",
+                __("Le statut est passé à"),
+                $event->ticket->status->label
+            );
         }
         switch ($event->ticket->status) {
             case TicketStatus::new():
-                $message->message .= sprintf("<br>" .
-                __("Votre ticket a été créé, nous allons en prendre connaissance.") . "<br>");
+                $message->message = sprintf(
+                    "%s<br>%s<br>",
+                    $message->message,
+                    __("Votre ticket a été créé, nous allons en prendre connaissance.")
+                );
                 break;
             case TicketStatus::estimateTodo():
-                // phpcs:ignore Generic.Files.LineLength.TooLong
-                $message->message .= sprintf("<br>" .
-                __("Votre ticket nécessite une estimation, nous allons vous proposer un devis.") . "<br>");
+                $message->message = sprintf(
+                    "%s<br>%s<br>",
+                    $message->message,
+                    __("Votre ticket nécessite une estimation, nous allons vous proposer un devis.")
+                );
                 break;
             case TicketStatus::takenInAccount():
                 $message->message .= sprintf("<br>" .
                 __("Votre ticket est dans le processus de traitement.") . "<br>");
                 break;
             case TicketStatus::waiting():
-                // phpcs:ignore Generic.Files.LineLength.TooLong
-                $message->message .= sprintf("<br>" .
-                __("Nous attendons une réponse de votre part pour procéder à la suite du traitement.") . "<br>");
+                $message->message = sprintf(
+                    "%s<br>%s<br>",
+                    $message->message,
+                    __("Nous attendons une réponse de votre part pour procéder à la suite du traitement.")
+                );
                 break;
             case TicketStatus::waitingValidation():
-                $message->message .= sprintf("<br>" .
-                __("Nous attendons une validation de votre part pour procéder au traitement.") . "<br>");
+                $message->message = sprintf(
+                    "%s<br>%s<br>",
+                    $message->message,
+                    __("Nous attendons une validation de votre part pour procéder au traitement.")
+                );
                 break;
             case TicketStatus::recette():
-                // phpcs:ignore Generic.Files.LineLength.TooLong
-                $message->message .= sprintf("<br>" .
-                __("Le traitement à été réalisé et mis à disposition sur la plateforme de recette. 
-                Vous pouvez vérifier que la solution mise en œuvre correspond à la demande.") . "<br>");
+                $message->message = sprintf(
+                    "%s<br>%s<br>",
+                    $message->message,
+                    __("Le traitement à été réalisé et mis à disposition sur la plateforme de recette. 
+                    Vous pouvez vérifier que la solution mise en œuvre correspond à la demande.")
+                );
                 break;
             case TicketStatus::production():
-                // phpcs:ignore Generic.Files.LineLength.TooLong
-                $message->message .= sprintf("<br>" .
-                __("La solution à été mise en ligne. 
-                Vous pouvez vérifier sur le site de production que tout fonctionne comme attendu.") . "<br>");
+                $message->message = sprintf(
+                    "%s<br>%s<br>",
+                    $message->message,
+                    __("La solution à été mise en ligne. 
+                    Vous pouvez vérifier sur le site de production que tout fonctionne comme attendu.")
+                );
                 break;
             case TicketStatus::archived():
-                $message->message .= sprintf("<br>" .
-                __("Ce ticket est clos.") . "<br>");
+                $message->message = sprintf(
+                    "%s<br>%s<br>",
+                    $message->message,
+                    __("Ce ticket est clos.")
+                );
                 break;
         }
         $message->save();