From c60f60d426e1c0adb83d9948d1011a59ec66fef8 Mon Sep 17 00:00:00 2001
From: efalcy <efalcy@afi-sa.fr>
Date: Thu, 12 Nov 2015 11:25:07 +0100
Subject: [PATCH] dev #32614 : duplicate newsletter: set duplicate to draft
---
library/Class/Newsletter.php | 2 +-
.../modules/admin/controllers/NewsletterControllerTest.php | 6 ++++++
2 files changed, 7 insertions(+), 1 deletion(-)
diff --git a/library/Class/Newsletter.php b/library/Class/Newsletter.php
index 79b9303291b..08b8015ed36 100644
--- a/library/Class/Newsletter.php
+++ b/library/Class/Newsletter.php
@@ -310,7 +310,7 @@ class Class_Newsletter extends Storm_Model_Abstract {
$duplicate = new Class_Newsletter();
$duplicate->updateAttributes($attributes);
-
+ $duplicate->setDraft(true);
if (!$duplicate->save())
return false;
diff --git a/tests/application/modules/admin/controllers/NewsletterControllerTest.php b/tests/application/modules/admin/controllers/NewsletterControllerTest.php
index add99fd2c4a..c4309b1a2dd 100644
--- a/tests/application/modules/admin/controllers/NewsletterControllerTest.php
+++ b/tests/application/modules/admin/controllers/NewsletterControllerTest.php
@@ -1115,6 +1115,12 @@ class Admin_NewsletterControllerDuplicateActionTest extends Admin_AbstractContro
}
+ /** @test */
+ public function duplicatedNewsletterShouldBeDraft() {
+ $this->assertEquals(1,Class_Newsletter::find($this->_new_id)->getDraft());
+ }
+
+
/** @test */
public function duplicatedNewsletterShouldHaveSameContentThanSource() {
$this->assertEquals(Class_Newsletter::find(1)->getContenu(), Class_Newsletter::find($this->_new_id)->getContenu());
--
GitLab