Skip to content

Commit f952bd9

Browse files
committed
MC-20710: Add Elasticsearch configuration parameters to console
1 parent 9264df8 commit f952bd9

File tree

2 files changed

+18
-5
lines changed

2 files changed

+18
-5
lines changed

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

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,8 @@ private function validateSearchEngineSelection(array $searchOptions)
6969
{
7070
if (isset($searchOptions[SearchConfigOptionsList::INPUT_KEY_SEARCH_ENGINE])) {
7171
$selectedEngine = $searchOptions[SearchConfigOptionsList::INPUT_KEY_SEARCH_ENGINE];
72-
if (!in_array($selectedEngine, SearchConfigOptionsList::AVAILABLE_SEARCH_ENGINES)) {
72+
$availableEngines = $this->searchConfigOptionsList->getAvailableSearchEngineList();
73+
if (!isset($availableEngines[$selectedEngine])) {
7374
throw new SetupException("Search engine '{$selectedEngine}' is not an available search engine.");
7475
}
7576
}

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

Lines changed: 16 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,6 @@
1717
*/
1818
class SearchConfigOptionsList
1919
{
20-
const AVAILABLE_SEARCH_ENGINES = ['elasticsearch5', 'elasticsearch6', 'elasticsearch7'];
21-
2220
/**
2321
* Input key for the options
2422
*/
@@ -51,9 +49,9 @@ public function getOptionsList(): array
5149
new SelectConfigOption(
5250
self::INPUT_KEY_SEARCH_ENGINE,
5351
SelectConfigOption::FRONTEND_WIZARD_SELECT,
54-
self::AVAILABLE_SEARCH_ENGINES,
52+
array_keys($this->getAvailableSearchEngineList()),
5553
'',
56-
'Search engine.',
54+
'Search engine. Values: ' . implode(', ', array_keys($this->getAvailableSearchEngineList())),
5755
self::DEFAULT_SEARCH_ENGINE
5856
),
5957
new TextConfigOption(
@@ -104,4 +102,18 @@ public function getOptionsList(): array
104102
)
105103
];
106104
}
105+
106+
/**
107+
* Get UI friendly list of available search engines
108+
*
109+
* @return array
110+
*/
111+
public function getAvailableSearchEngineList(): array
112+
{
113+
return [
114+
'elasticsearch5' => 'Elasticsearch 5.x (deprecated)',
115+
'elasticsearch6' => 'Elasticsearch 6.x',
116+
'elasticsearch7' => 'Elasticsearch 7.x'
117+
];
118+
}
107119
}

0 commit comments

Comments
 (0)