Skip to content

Commit 632926d

Browse files
committed
MAGETWO-90739: Out of stock options for configurable products still show up in search and layered navigation if Elasticsearch is enabled
1 parent ffda0cc commit 632926d

File tree

2 files changed

+10
-3
lines changed

2 files changed

+10
-3
lines changed

app/code/Magento/Elasticsearch/Model/Config.php

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
namespace Magento\Elasticsearch\Model;
77

88
use Magento\Framework\App\Config\ScopeConfigInterface;
9+
use Magento\Framework\Search\EngineResolverInterface;
910
use Magento\Search\Model\EngineResolver;
1011
use Magento\Store\Model\ScopeInterface;
1112
use Magento\AdvancedSearch\Model\Client\ClientOptionsInterface;
@@ -55,23 +56,28 @@ class Config implements ClientOptionsInterface
5556
*/
5657
private $clientResolver;
5758

59+
/**
60+
* @var EngineResolverInterface
61+
*/
62+
private $engineResolver;
63+
5864
/**
5965
* Constructor
6066
*
6167
* @param ScopeConfigInterface $scopeConfig
6268
* @param ClientResolver|null $clientResolver
63-
* @param EngineResolver|null $engineResolver
69+
* @param EngineResolverInterface|null $engineResolver
6470
* @param string $prefix
6571
*/
6672
public function __construct(
6773
ScopeConfigInterface $scopeConfig,
6874
ClientResolver $clientResolver = null,
69-
EngineResolver $engineResolver = null,
75+
EngineResolverInterface $engineResolver = null,
7076
$prefix = null
7177
) {
7278
$this->scopeConfig = $scopeConfig;
7379
$this->clientResolver = $clientResolver ?: ObjectManager::getInstance()->get(ClientResolver::class);
74-
$this->engineResolver = $engineResolver ?: ObjectManager::getInstance()->get(EngineResolver::class);
80+
$this->engineResolver = $engineResolver ?: ObjectManager::getInstance()->get(EngineResolverInterface::class);
7581
$this->prefix = $prefix ?: $this->clientResolver->getCurrentEngine();
7682
}
7783

dev/tests/integration/testsuite/Magento/Elasticsearch/Model/Indexer/IndexHandlerTest.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -190,6 +190,7 @@ public function testReindexRowAfterDelete(): void
190190
}
191191

192192
/**
193+
* @magentoDbIsolation enabled
193194
* @magentoAppArea adminhtml
194195
* @magentoConfigFixture default/catalog/search/engine elasticsearch
195196
* @magentoConfigFixture current_store catalog/search/elasticsearch_index_prefix indexerhandlertest

0 commit comments

Comments
 (0)