Cavemanon/piratepoet
7
Fork 0
Code Pull Requests Actions Releases 16 Activity

Set new hash when duplicating newsletter

Browse Source 
[MAILPOET-3275]
This commit is contained in:
Rostislav Wolny
2020-11-19 09:24:27 +01:00
committed by Veljko V
parent dfc4c286df
commit e22c192078
2 changed files with 12 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
lib/Newsletter/NewsletterSaveController.php
View File

@ -157,7 +157,7 @@ class NewsletterSaveController {
// reset status
$duplicate->setStatus(NewsletterEntity::STATUS_DRAFT);
// reset hash
$duplicate->setHash(null);
$duplicate->setHash(Security::generateHash());
// reset sent at date
$duplicate->setSentAt(null);

11
tests/integration/Newsletter/NewsletterSaveControllerTest.php
View File

@ -273,6 +273,17 @@ class NewsletterSaveControllerTest extends \MailPoetTest {
expect($settings->get('sender.address'))->same('test@example.com');
}
public function testItDuplicatesNewsletter() {
$newsletter = $this->createNewsletter(NewsletterEntity::TYPE_STANDARD, NewsletterEntity::STATUS_SENT);
$duplicate = $this->saveController->duplicate($newsletter);
expect($duplicate->getSubject())->equals('Copy of ' . $newsletter->getSubject());
expect($duplicate->getHash())->string();
expect($duplicate->getHash())->notEmpty();
expect($duplicate->getHash())->notEquals($newsletter->getHash());
expect($duplicate->getBody())->equals($newsletter->getBody());
expect($duplicate->getStatus())->equals(NewsletterEntity::STATUS_DRAFT);
}
public function testItCreatesNewNewsletter() {
$data = [
'subject' => 'My First Newsletter',