Skip to content

Commit d009296

Browse files
committed
magento#16069: Configurable product price is not displayed if all children are out of stock and even if Display Out of Stock Products is set to "yes"
- refactor variable names.
1 parent 3351aa9 commit d009296

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

app/code/Magento/ConfigurableProduct/Model/ResourceModel/Product/StockStatusBaseSelectProcessor.php

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -35,21 +35,21 @@ class StockStatusBaseSelectProcessor implements BaseSelectProcessorInterface
3535
/**
3636
* @var StockStatusConfigurableInterface
3737
*/
38-
private $stockStatusConfigurableResource;
38+
private $stockConfigurable;
3939

4040
/**
4141
* @param StockConfigurationInterface $stockConfig
4242
* @param StockStatusResource $stockStatusResource
43-
* @param StockStatusConfigurableInterface $stockStatusConfigurableResource
43+
* @param StockStatusConfigurableInterface $stockConfigurable
4444
*/
4545
public function __construct(
4646
StockConfigurationInterface $stockConfig,
4747
StockStatusResource $stockStatusResource,
48-
StockStatusConfigurableInterface $stockStatusConfigurableResource = null
48+
StockStatusConfigurableInterface $stockConfigurable = null
4949
) {
5050
$this->stockConfig = $stockConfig;
5151
$this->stockStatusResource = $stockStatusResource;
52-
$this->stockStatusConfigurableResource = $stockStatusConfigurableResource ?:
52+
$this->stockConfigurable = $stockConfigurable ?:
5353
ObjectManager::getInstance()->get(StockStatusConfigurableInterface::class);
5454
}
5555

@@ -84,6 +84,6 @@ public function process(Select $select, int $productId): Select
8484
*/
8585
private function isAllChildOutOfStock(int $productId): bool
8686
{
87-
return $this->stockStatusConfigurableResource->isAllChildOutOfStock($productId);
87+
return $this->stockConfigurable->isAllChildOutOfStock($productId);
8888
}
8989
}

app/code/Magento/ConfigurableProduct/Pricing/Render/FinalPriceBox.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ class FinalPriceBox extends \Magento\Catalog\Pricing\Render\FinalPriceBox
4141
* @param array $data
4242
* @param LowestPriceOptionsProviderInterface $lowestPriceOptionsProvider
4343
* @param SalableResolverInterface|null $salableResolver
44-
* @param MinimalPriceCalculatorInterface|null $minimalPriceCalculator
44+
* @param MinimalPriceCalculatorInterface|null $minPriceCalculator
4545
* @param StockStatus $stockStatus
4646
* @SuppressWarnings(PHPMD.UnusedFormalParameter)
4747
*/
@@ -54,7 +54,7 @@ public function __construct(
5454
array $data = [],
5555
LowestPriceOptionsProviderInterface $lowestPriceOptionsProvider = null,
5656
SalableResolverInterface $salableResolver = null,
57-
MinimalPriceCalculatorInterface $minimalPriceCalculator = null,
57+
MinimalPriceCalculatorInterface $minPriceCalculator = null,
5858
StockStatus $stockStatus = null
5959
) {
6060
parent::__construct(
@@ -64,7 +64,7 @@ public function __construct(
6464
$rendererPool,
6565
$data,
6666
$salableResolver,
67-
$minimalPriceCalculator
67+
$minPriceCalculator
6868
);
6969
$this->lowestPriceOptionsProvider = $lowestPriceOptionsProvider ?:
7070
ObjectManager::getInstance()->get(LowestPriceOptionsProviderInterface::class);

0 commit comments

Comments
 (0)