Skip to content

Commit 2583036

Browse files
committed
Revert AC-1883
1 parent 926b334 commit 2583036

File tree

3 files changed

+20
-22
lines changed

3 files changed

+20
-22
lines changed

app/code/Magento/Elasticsearch/Model/DataProvider/Base/Suggestions.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -226,7 +226,6 @@ private function addSuggestFields($searchQuery, $searchSuggestionsCount)
226226
[
227227
'field' => $field,
228228
'min_word_length' => 3,
229-
'min_doc_freq' => 1,
230229
]
231230
],
232231
],

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

Lines changed: 15 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -53,19 +53,19 @@
5353
*/
5454
class Subscriber extends AbstractModel
5555
{
56-
const STATUS_SUBSCRIBED = 1;
57-
const STATUS_NOT_ACTIVE = 2;
58-
const STATUS_UNSUBSCRIBED = 3;
59-
const STATUS_UNCONFIRMED = 4;
56+
public const STATUS_SUBSCRIBED = 1;
57+
public const STATUS_NOT_ACTIVE = 2;
58+
public const STATUS_UNSUBSCRIBED = 3;
59+
public const STATUS_UNCONFIRMED = 4;
6060

61-
const XML_PATH_CONFIRM_EMAIL_TEMPLATE = 'newsletter/subscription/confirm_email_template';
62-
const XML_PATH_CONFIRM_EMAIL_IDENTITY = 'newsletter/subscription/confirm_email_identity';
63-
const XML_PATH_SUCCESS_EMAIL_TEMPLATE = 'newsletter/subscription/success_email_template';
64-
const XML_PATH_SUCCESS_EMAIL_IDENTITY = 'newsletter/subscription/success_email_identity';
65-
const XML_PATH_UNSUBSCRIBE_EMAIL_TEMPLATE = 'newsletter/subscription/un_email_template';
66-
const XML_PATH_UNSUBSCRIBE_EMAIL_IDENTITY = 'newsletter/subscription/un_email_identity';
67-
const XML_PATH_CONFIRMATION_FLAG = 'newsletter/subscription/confirm';
68-
const XML_PATH_ALLOW_GUEST_SUBSCRIBE_FLAG = 'newsletter/subscription/allow_guest_subscribe';
61+
public const XML_PATH_CONFIRM_EMAIL_TEMPLATE = 'newsletter/subscription/confirm_email_template';
62+
public const XML_PATH_CONFIRM_EMAIL_IDENTITY = 'newsletter/subscription/confirm_email_identity';
63+
public const XML_PATH_SUCCESS_EMAIL_TEMPLATE = 'newsletter/subscription/success_email_template';
64+
public const XML_PATH_SUCCESS_EMAIL_IDENTITY = 'newsletter/subscription/success_email_identity';
65+
public const XML_PATH_UNSUBSCRIBE_EMAIL_TEMPLATE = 'newsletter/subscription/un_email_template';
66+
public const XML_PATH_UNSUBSCRIBE_EMAIL_IDENTITY = 'newsletter/subscription/un_email_identity';
67+
public const XML_PATH_CONFIRMATION_FLAG = 'newsletter/subscription/confirm';
68+
public const XML_PATH_ALLOW_GUEST_SUBSCRIBE_FLAG = 'newsletter/subscription/allow_guest_subscribe';
6969

7070
/**
7171
* Prefix of model events names
@@ -91,7 +91,6 @@ class Subscriber extends AbstractModel
9191
protected $_isStatusChanged = false;
9292

9393
/**
94-
* Newsletter data
9594
*
9695
* @var Data
9796
*/
@@ -105,7 +104,6 @@ class Subscriber extends AbstractModel
105104
protected $_scopeConfig;
106105

107106
/**
108-
* Customer session
109107
*
110108
* @var \Magento\Customer\Model\Session
111109
*/
@@ -118,7 +116,6 @@ class Subscriber extends AbstractModel
118116
private $dateTime;
119117

120118
/**
121-
* Store manager
122119
*
123120
* @var StoreManagerInterface
124121
*/
@@ -565,8 +562,9 @@ private function sendEmail(string $emailTemplatePath, string $emailIdentityPath,
565562
]
566563
)->setTemplateVars(
567564
$templateVars
568-
)->setFrom(
569-
$identity
565+
)->setFromByScope(
566+
$identity,
567+
$this->getStoreId()
570568
)->addTo(
571569
$this->getEmail(),
572570
$this->getName()

app/code/Magento/Newsletter/Test/Unit/Model/SubscriberTest.php

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -115,6 +115,7 @@ protected function setUp(): void
115115
'setTemplateOptions',
116116
'setTemplateVars',
117117
'setFrom',
118+
'setFromByScope',
118119
'addTo',
119120
'getTransport'
120121
]
@@ -133,8 +134,8 @@ protected function setUp(): void
133134
$this->inlineTranslation = $this->getMockForAbstractClass(StateInterface::class);
134135
$this->resource = $this->getMockBuilder(SubscriberResourceModel::class)
135136
->addMethods(
136-
['loadByCustomer']
137-
)
137+
['loadByCustomer']
138+
)
138139
->onlyMethods(
139140
['loadByEmail', 'getIdFieldName', 'save', 'received', 'loadBySubscriberEmail', 'loadByCustomerId']
140141
)
@@ -379,8 +380,8 @@ private function sendEmailCheck(string $templateConfigPath, string $identityTemp
379380
->with($vars)
380381
->willReturnSelf();
381382
$this->transportBuilder->expects($this->once())
382-
->method('setFrom')
383-
->with($identity)
383+
->method('setFromByScope')
384+
->with($identity, $this->subscriber->getStoreId())
384385
->willReturnSelf();
385386
$this->transportBuilder->expects($this->once())
386387
->method('addTo')

0 commit comments

Comments
 (0)