Skip to content

Commit 98aebca

Browse files
author
Oleksii Korshenko
committed
MAGETWO-67680: Replace Zend_Mail (ZF1) with Zend\Mail (ZF2) #8608
- Merge Pull Request #8608 from schmengler/magento2:zend-mail-2 - Merged commits: 1. 3309db2 2. 584a17a 3. ca1a3b2 4. d1be154 5. 0d2f2a6 6. c1d8e0d 7. 0be511c 8. 00bc417 9. e02e28d 10. dd755bc 11. e7f62fe 12. b633cd3 13. b7186fc
2 parents 04c5b80 + b7186fc commit 98aebca

File tree

12 files changed

+291
-91
lines changed

12 files changed

+291
-91
lines changed

app/code/Magento/Newsletter/Model/Template.php

-6
Original file line numberDiff line numberDiff line change
@@ -38,12 +38,6 @@
3838
*/
3939
class Template extends \Magento\Email\Model\AbstractTemplate
4040
{
41-
/**
42-
* Mail object
43-
*
44-
* @var \Zend_Mail
45-
*/
46-
protected $_mail;
4741

4842
/**
4943
* Store manager to emulate design

composer.json

+1
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@
1717
"zendframework/zend-validator": "^2.6.0",
1818
"zendframework/zend-crypt": "^2.6.0",
1919
"zendframework/zend-console": "^2.6.0",
20+
"zendframework/zend-mail": "^2.8.0",
2021
"zendframework/zend-modulemanager": "^2.7",
2122
"zendframework/zend-mvc": "~2.6.3",
2223
"zendframework/zend-text": "^2.6.0",

composer.lock

+137-26
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dev/tests/integration/testsuite/Magento/Newsletter/Model/SubscriberTest.php

+4-1
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,8 @@
66

77
namespace Magento\Newsletter\Model;
88

9+
use Magento\TestFramework\Mail\Template\TransportBuilderMock;
10+
911
class SubscriberTest extends \PHPUnit_Framework_TestCase
1012
{
1113
/**
@@ -27,13 +29,14 @@ protected function setUp()
2729
public function testEmailConfirmation()
2830
{
2931
$this->_model->subscribe('[email protected]');
32+
/** @var TransportBuilderMock $transportBuilder */
3033
$transportBuilder = \Magento\TestFramework\Helper\Bootstrap::getObjectManager()
3134
->get(\Magento\TestFramework\Mail\Template\TransportBuilderMock::class);
3235
// confirmationCode 'ysayquyajua23iq29gxwu2eax2qb6gvy' is taken from fixture
3336
$this->assertContains(
3437
'/newsletter/subscriber/confirm/id/' . $this->_model->getSubscriberId()
3538
. '/code/ysayquyajua23iq29gxwu2eax2qb6gvy',
36-
$transportBuilder->getSentMessage()->getBodyHtml()->getRawContent()
39+
$transportBuilder->getSentMessage()->getRawMessage()
3740
);
3841
$this->assertEquals(Subscriber::STATUS_NOT_ACTIVE, $this->_model->getSubscriberStatus());
3942
}

dev/tests/integration/testsuite/Magento/ProductAlert/Model/EmailTest.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,7 @@ public function testSend($isCustomerIdUsed)
8282
);
8383
$this->assertContains(
8484
'John Smith,',
85-
$transportBuilder->getSentMessage()->getBodyHtml()->getRawContent()
85+
$transportBuilder->getSentMessage()->getRawMessage()
8686
);
8787
}
8888

dev/tests/integration/testsuite/Magento/ProductAlert/Model/ObserverTest.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ public function testProcess()
5656
);
5757
$this->assertContains(
5858
'John Smith,',
59-
$transportBuilder->getSentMessage()->getBodyHtml()->getRawContent()
59+
$transportBuilder->getSentMessage()->getRawMessage()
6060
);
6161
}
6262
}

dev/tests/integration/testsuite/Magento/Wishlist/Controller/IndexTest.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -158,7 +158,7 @@ public function testSendAction()
158158
);
159159

160160
$actualResult = \Zend_Mime_Decode::decodeQuotedPrintable(
161-
$transportBuilder->getSentMessage()->getBodyHtml()->getContent()
161+
$transportBuilder->getSentMessage()->getRawMessage()
162162
);
163163

164164
$this->assertStringMatchesFormat(

0 commit comments

Comments
 (0)