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] 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();