Skip to content

Commit c7beb3c

Browse files
committed
MC-33297: ES validation fails during install on Cloud
1 parent 2805b8e commit c7beb3c

File tree

3 files changed

+22
-26
lines changed

3 files changed

+22
-26
lines changed

setup/src/Magento/Setup/Controller/ConfigureCatalogSearch.php

Lines changed: 17 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,22 @@ class ConfigureCatalogSearch extends AbstractActionController
2222
*/
2323
private $searchConfigOptionsList;
2424

25+
/**
26+
* Default values to prefill form
27+
*
28+
* @var array
29+
*/
30+
private $prefillConfigValues = [
31+
'engine' => 'elasticsearch7',
32+
'elasticsearch' => [
33+
'hostname' => 'localhost',
34+
'port' => '9200',
35+
'timeout' => '15',
36+
'indexPrefix' => 'magento2',
37+
'enableAuth' => false
38+
]
39+
];
40+
2541
/**
2642
* @param SearchConfigOptionsList $searchConfigOptionsList
2743
*/
@@ -51,17 +67,6 @@ public function indexAction(): ViewModel
5167
*/
5268
public function defaultParametersAction(): JsonModel
5369
{
54-
$defaults = [
55-
'engine' => SearchConfigOptionsList::DEFAULT_SEARCH_ENGINE,
56-
'elasticsearch' => [
57-
'hostname' => SearchConfigOptionsList::DEFAULT_ELASTICSEARCH_HOST,
58-
'port' => SearchConfigOptionsList::DEFAULT_ELASTICSEARCH_PORT,
59-
'timeout' => SearchConfigOptionsList::DEFAULT_ELASTICSEARCH_TIMEOUT,
60-
'indexPrefix' => SearchConfigOptionsList::DEFAULT_ELASTICSEARCH_INDEX_PREFIX,
61-
'enableAuth' => false
62-
]
63-
];
64-
65-
return new JsonModel($defaults);
70+
return new JsonModel($this->prefillConfigValues);
6671
}
6772
}

setup/src/Magento/Setup/Model/SearchConfigOptionsList.php

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -28,15 +28,6 @@ class SearchConfigOptionsList
2828
const INPUT_KEY_ELASTICSEARCH_INDEX_PREFIX = 'elasticsearch-index-prefix';
2929
const INPUT_KEY_ELASTICSEARCH_TIMEOUT = 'elasticsearch-timeout';
3030

31-
/**
32-
* Default values
33-
*/
34-
const DEFAULT_SEARCH_ENGINE = 'elasticsearch7';
35-
const DEFAULT_ELASTICSEARCH_HOST = 'localhost';
36-
const DEFAULT_ELASTICSEARCH_PORT = '9200';
37-
const DEFAULT_ELASTICSEARCH_INDEX_PREFIX = 'magento2';
38-
const DEFAULT_ELASTICSEARCH_TIMEOUT = 15;
39-
4031
/**
4132
* Get options list for search engine configuration
4233
*

setup/src/Magento/Setup/Test/Unit/Controller/ConfigureCatalogSearchTest.php

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -50,12 +50,12 @@ public function testDefaultParametersAction()
5050
$this->assertInstanceOf(JsonModel::class, $jsonModel);
5151

5252
$expectedDefaultParameters = [
53-
'engine' => SearchConfigOptionsList::DEFAULT_SEARCH_ENGINE,
53+
'engine' => 'elasticsearch7',
5454
'elasticsearch' => [
55-
'hostname' => SearchConfigOptionsList::DEFAULT_ELASTICSEARCH_HOST,
56-
'port' => SearchConfigOptionsList::DEFAULT_ELASTICSEARCH_PORT,
57-
'timeout' => SearchConfigOptionsList::DEFAULT_ELASTICSEARCH_TIMEOUT,
58-
'indexPrefix' => SearchConfigOptionsList::DEFAULT_ELASTICSEARCH_INDEX_PREFIX,
55+
'hostname' => 'localhost',
56+
'port' => '9200',
57+
'timeout' => '15',
58+
'indexPrefix' => 'magento2',
5959
'enableAuth' => false
6060
]
6161
];

0 commit comments

Comments
 (0)