Skip to content

Commit 44c5fb0

Browse files
committed
ACP2E-1388:Bundle Products Not Combining Simple Product Prices When Set to Dynamic Pricing
fixed CR feedback
1 parent e53c375 commit 44c5fb0

File tree

2 files changed

+1
-3
lines changed

2 files changed

+1
-3
lines changed

app/code/Magento/Bundle/Model/Plugin/ProductPriceIndexModifier.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -130,7 +130,6 @@ private function filterProductsFromDynamicPriceBundle(string $priceTableName, ar
130130
);
131131
$select->where('selection.product_id IN (?)', $productIds);
132132
$select->where('product.type_id = ?', Type::TYPE_BUNDLE);
133-
$select->group('selection.product_id');
134133
$bundleProducts = $connection->fetchAll($select);
135134

136135
if (empty($bundleProducts)) {
@@ -139,7 +138,7 @@ private function filterProductsFromDynamicPriceBundle(string $priceTableName, ar
139138

140139
$filteredProducts = [];
141140
foreach($bundleProducts as $bundle) {
142-
if ($bundle['bundle_price_type'] !== Price::PRICE_TYPE_DYNAMIC) {
141+
if ($bundle['bundle_price_type'] != Price::PRICE_TYPE_DYNAMIC) {
143142
$filteredProducts[] = $bundle['child_product_id'];
144143
}
145144
}

app/code/Magento/Bundle/Test/Unit/Model/Plugin/ProductPriceIndexModifierTest.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,6 @@ public function testAroundModifyPriceFilteredEntities()
9898
->method('joinInner');
9999
$select->expects($this->exactly(2))
100100
->method('where');
101-
$select->expects($this->once())->method('group')->with('selection.product_id');
102101
$connection = $this->createMock(AdapterInterface::class);
103102
$connection->expects($this->once())
104103
->method('select')

0 commit comments

Comments
 (0)