From b90d2f53b6206f0ea82338273279bf6657d0fb95 Mon Sep 17 00:00:00 2001
From: llaffont <llaffont@afi-sa.fr>
Date: Tue, 10 Nov 2015 17:04:40 +0100
Subject: [PATCH] rel #31761 notify article validators: mail deduplication
---
application/modules/admin/controllers/CmsController.php | 2 +-
.../modules/admin/controllers/CmsControllerTest.php | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/application/modules/admin/controllers/CmsController.php b/application/modules/admin/controllers/CmsController.php
index ddc825c9a3f..b69685ae34e 100644
--- a/application/modules/admin/controllers/CmsController.php
+++ b/application/modules/admin/controllers/CmsController.php
@@ -378,7 +378,7 @@ class Admin_CmsController extends ZendAfi_Controller_Action {
$mail = new ZendAfi_Mail('utf8');
$mail
->setFrom('no-reply@afi-sa.fr')
- ->addTo(implode(',', $mails->getArrayCopy()))
+ ->addTo(implode(',', array_unique($mails->getArrayCopy())))
->setSubject($this->_('[Bokeh] Validation d\'article en attente: ') . $article->getTitre())
->setBodyText($body);
diff --git a/tests/application/modules/admin/controllers/CmsControllerTest.php b/tests/application/modules/admin/controllers/CmsControllerTest.php
index 9e8a9c25b4e..4ad82618f13 100644
--- a/tests/application/modules/admin/controllers/CmsControllerTest.php
+++ b/tests/application/modules/admin/controllers/CmsControllerTest.php
@@ -1713,8 +1713,8 @@ class CmsControllerNewsAddActionPostWithWorkflowTest
$this->_cat_a_la_une);
$this->postDispatch('/admin/cms/add/id_cat/23',
$data);
- $this->assertEquals(['laurent@afi-sa.fr', 'bernard@afi-sa.fr', 'admin@afi-sa.fr'],
- $this->mock_transport->getSentMails()[0]->getRecipients());
+ $this->assertEquals(['<laurent@afi-sa.fr>', '<bernard@afi-sa.fr>', '<admin@afi-sa.fr>', 'append' => true],
+ $this->mock_transport->getSentMails()[0]->getHeaders()['To']);
}
--
GitLab