From 22404b8e7cc81189ddd37cbbb418070ee304e899 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ma=C3=AFeul=20Rouquette?= <maieul@maieul.net> Date: Thu, 27 Feb 2020 12:15:56 +0100 Subject: [PATCH] =?UTF-8?q?Ne=20filtrer=20qu'une=20seule=20fois=20les=20de?= =?UTF-8?q?stinataires=20des=20emails=20pour=20s'assurer=20des=20doublons?= =?UTF-8?q?=20+=20appliquer=20trim=20par=20securit=C3=A9?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- traiter/email.php | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/traiter/email.php b/traiter/email.php index 05544152..09a2d857 100644 --- a/traiter/email.php +++ b/traiter/email.php @@ -68,7 +68,6 @@ function traiter_email_dist($args, $retours) { $destinataires_plus = explode(',', $options['destinataires_plus']); $destinataires_plus = array_map('trim', $destinataires_plus); $destinataires = array_merge($destinataires, $destinataires_plus); - $destinataires = array_unique($destinataires); } // On ajoute les destinataires en fonction des choix de saisie dans le formulaire @@ -77,10 +76,15 @@ function traiter_email_dist($args, $retours) { if (!empty($options['destinataires_selon_champ'])) { if ($destinataires_selon_champ = formidable_traiter_email_destinataire_selon_champ($options['destinataires_selon_champ'])) { $destinataires = array_merge($destinataires, $destinataires_selon_champ); - $destinataires = array_unique($destinataires); } } + + + // S'assurer que les destinataires ne soient pas en doublons + $destinataires = array_filter($destinataires, 'trim'); + $destinataires = array_unique($destinataires); + // On récupère le courriel de l'envoyeur s'il existe if ($options['champ_courriel']) { $courriel_envoyeur = _request($options['champ_courriel']); @@ -410,7 +414,6 @@ function formidable_traiter_email_destinataire_selon_champ($description) { if ($ok) { $destinataires = array_merge($destinataires, $mails); - $destinataires = array_unique($destinataires); } } } -- GitLab