Skip to content

Commit eca73d0

Browse files
authoredFeb 5, 2018
Merge forwardport of #13033 to 2.3-develop branch
Applied pull request patch https://github.com/magento/magento2/pull/13033.patch (created by @devamitbera) based on commit(s): 1. 7559783 2. b853497 3. be8b258 Fixed GitHub Issues in 2.3-develop branch: - #12787: Newsletter\Model\Subscriber::loadByEmail() does not use MySQL index (reported by @schmengler)
2 parents 917a03f + 54bc3a1 commit eca73d0

File tree

2 files changed

+37
-1
lines changed

2 files changed

+37
-1
lines changed
 
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,36 @@
1+
<?php
2+
/**
3+
* Copyright © Magento, Inc. All rights reserved.
4+
* See COPYING.txt for license details.
5+
*/
6+
namespace Magento\Newsletter\Setup;
7+
8+
use Magento\Framework\Setup\ModuleContextInterface;
9+
use Magento\Framework\Setup\SchemaSetupInterface;
10+
use Magento\Framework\Setup\UpgradeSchemaInterface;
11+
12+
/**
13+
* Upgrade the Newsletter module DB scheme
14+
*/
15+
class UpgradeSchema implements UpgradeSchemaInterface
16+
{
17+
/**
18+
* {@inheritdoc}
19+
*/
20+
public function upgrade(SchemaSetupInterface $setup, ModuleContextInterface $context)
21+
{
22+
$setup->startSetup();
23+
24+
if (version_compare($context->getVersion(), '2.0.1', '<')) {
25+
$connection = $setup->getConnection();
26+
27+
$connection->addIndex(
28+
$setup->getTable('newsletter_subscriber'),
29+
$setup->getIdxName('newsletter_subscriber', ['subscriber_email']),
30+
['subscriber_email']
31+
);
32+
}
33+
34+
$setup->endSetup();
35+
}
36+
}

‎app/code/Magento/Newsletter/etc/module.xml

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
*/
77
-->
88
<config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="urn:magento:framework:Module/etc/module.xsd">
9-
<module name="Magento_Newsletter" setup_version="2.0.0">
9+
<module name="Magento_Newsletter" setup_version="2.0.1">
1010
<sequence>
1111
<module name="Magento_Store"/>
1212
<module name="Magento_Customer"/>

0 commit comments

Comments
 (0)