refactor: better code to send mail

formulaire-adhesion/traitement-adhesion.php

@@ -6,6 +6,8 @@
  */
 //variables pour le mail
 define("SUBJECT_ADULLACT", "Nouvelle demande de renouvellement d'adhésion");
+define("SUBJECT_ERROR_MAIL", "Une erreur est survenue lors d'un envoi de mail récupéré sur le formulaire d'adhésion.");
+define("MESSAGE_ERROR_MAIL", "Lorsqu'un utilisateur a validé le formulaire d'adhésion, le mail de confirmation n'a pas été envoyé à cet utilisateur. Une erreur est survenue, veuillez consulter les logs pour en savoir plus.");
 define("SUBJECT_USER", "Confirmation de demande de renouvellement d'adhésion");
 
 $bonjourUser = "Bonjour,";
@@ -143,9 +145,8 @@ function sendAdhesionMail($tab, $trash, $replacements)
     }
     $emailUser = trim($tab["mailUser"]);
 
-    $header = "";
-    $header.="Content-type: text/html; charset=utf-8\r\n";
-    $header.="Content-Transfer-Encoding: 8bit\r\n";
+    $header = "Content-type: text/html; charset=utf-8\r\n";
+    $header .= "Content-Transfer-Encoding: 8bit\r\n";
 
     $logMessage = "\n";
     $messageAdullact = "Nouvelle requête pour le formulaire d'adhésion : <br><table>";
@@ -157,11 +158,15 @@ function sendAdhesionMail($tab, $trash, $replacements)
         $logMessage = $logMessage . $key . " : " . $value . "\n";
     }
 
-    if (wp_mail($emailUser, html_entity_decode(SUBJECT_USER,ENT_QUOTES, 'UTF-8'), TEXT_MAIL_USER, $header)) {
-        wp_mail(MAIL_CIBLE_ADULLACT, html_entity_decode(SUBJECT_ADULLACT,ENT_QUOTES,'UTF-8'), $messageAdullact, $header);
-        writeInLog("Nouvelle demande", $logMessage);
+    //on tente d'envoyer le mail à l'utilisateur
+    if (wp_mail($emailUser, html_entity_decode(SUBJECT_USER, ENT_QUOTES, 'UTF-8'), TEXT_MAIL_USER, $header)) {
+        //si ça fonctionne, on envoie un mail positif à l'Adullact.
+        wp_mail(MAIL_CIBLE_ADULLACT, html_entity_decode(SUBJECT_ADULLACT, ENT_QUOTES, 'UTF-8'), $messageAdullact, $header);
+        writeInLog("Nouvelle demande", $logMessage, MEMBERSHIP_LOGS_DIRECTORY);
     } else {
-        writeInLog("Erreur de l'envoi mail.", "Header : \n" . $header . "\nInfos :" . $logMessage);
+        //ça n'a visiblement pas fonctionné. On envoie donc un mail négatif à l'Adullact et on l'écrit dans les logs.
+        wp_mail(MAIL_CIBLE_ADULLACT, html_entity_decode(SUBJECT_ERROR_MAIL, ENT_QUOTES, 'UTF-8'), MESSAGE_ERROR_MAIL, $header);
+        writeInLog("Erreur de l'envoi mail.", "Header : \n" . $header . "\nInfos :" . $logMessage, MEMBERSHIP_LOGS_DIRECTORY);
     }
 }
 
@@ -169,15 +174,15 @@ function sendAdhesionMail($tab, $trash, $replacements)
  * Utile pour écrire un quelconque message dans les logs
  * @param string $intitule
  * @param string $text
+ * @param string $filePath : chemin d'accès au fichier de log
  */
-function writeInLog($intitule, $text)
+function writeInLog($intitule, $text, $filePath)
 {
     //fonction qui écrit dans les logs.
     date_default_timezone_set('Europe/Paris');
 
     $date = date("d-m-Y");
     $hour = date("H:i:s");
-    $filePath = MEMBERSHIP_LOGS_DIRECTORY;
     $beforeMessage = "\n--------------\nLe " . $date . ", à " . $hour . ", \n" . $intitule . " : \n";
     $finalMessage = $beforeMessage . $text;
     $fichier = fopen($filePath, "a");