Skip to content

Commit b635415

Browse files
committed
MC-33026: Upgrade magento with MySQL search engine removed
- fix search engine blacklist
1 parent 20da669 commit b635415

File tree

3 files changed

+7
-6
lines changed

3 files changed

+7
-6
lines changed

app/code/Magento/Elasticsearch/etc/di.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -536,7 +536,7 @@
536536
<type name="Magento\Search\Model\SearchEngine\Validator">
537537
<arguments>
538538
<argument name="engineBlacklist" xsi:type="array">
539-
<item name="elasticsearch2" xsi:type="string">elasticsearch2</item>
539+
<item name="elasticsearch" xsi:type="string">Elasticsearch 2</item>
540540
</argument>
541541
<argument name="engineValidators" xsi:type="array">
542542
<item name="elasticsearch5" xsi:type="object">Magento\Elasticsearch\Setup\Validator</item>

app/code/Magento/Search/Model/SearchEngine/Validator.php

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ class Validator implements ValidatorInterface
2222
/**
2323
* @var array
2424
*/
25-
private $engineBlacklist = ['mysql'];
25+
private $engineBlacklist = ['mysql' => 'MySQL'];
2626

2727
/**
2828
* @var ValidatorInterface[]
@@ -55,8 +55,9 @@ public function validate(array $searchConfig = []): array
5555
? $searchConfig['search-engine']
5656
: $this->scopeConfig->getValue('catalog/search/engine');
5757

58-
if (in_array($currentEngine, $this->engineBlacklist)) {
59-
$errors[] = "Search engine '{$currentEngine}' is not supported. Fix search configuration and try again.";
58+
if (isset($this->engineBlacklist[$currentEngine])) {
59+
$blacklistedEngine = $this->engineBlacklist[$currentEngine];
60+
$errors[] = "Search engine '{$blacklistedEngine}' is not supported. Fix search configuration and try again.";
6061
}
6162

6263
if (isset($this->engineValidators[$currentEngine])) {

app/code/Magento/Search/Test/Unit/Model/SearchEngine/ValidatorTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ protected function setUp()
3434
[
3535
'scopeConfig' => $this->scopeConfigMock,
3636
'engineValidators' => ['otherEngine' => $this->otherEngineValidatorMock],
37-
'engineBlacklist' => ['badEngine']
37+
'engineBlacklist' => ['badEngine' => 'Bad Engine']
3838
]
3939
);
4040
}
@@ -56,7 +56,7 @@ public function testValidateValid()
5656

5757
public function testValidateBlacklist()
5858
{
59-
$expectedErrors = ["Search engine 'badEngine' is not supported. Fix search configuration and try again."];
59+
$expectedErrors = ["Search engine 'Bad Engine' is not supported. Fix search configuration and try again."];
6060

6161
$this->assertEquals($expectedErrors, $this->validator->validate(['search-engine' => 'badEngine']));
6262
}

0 commit comments

Comments
 (0)