Skip to content

Commit 0c01a36

Browse files
author
Cristian Partica
committed
MAGETWO-31324: Tax labels aren't displayed for Bundle options of "multiple select" & "drop-down" types
- QA plus format fix plus test
1 parent 2e33874 commit 0c01a36

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

app/code/Magento/Tax/Model/Observer.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -320,7 +320,7 @@ public function updateProductOptions(\Magento\Framework\Event\Observer $observer
320320
$options['optionTemplate'] = sprintf(
321321
'<%%= data.label %%>'
322322
. '<%% if (data.basePrice.value) { %%>'
323-
. ' <%%= data.basePrice.formatted %%>'
323+
. ' +<%%= data.basePrice.formatted %%>'
324324
. '<%% } %%>'
325325
);
326326
}

app/code/Magento/Tax/Test/Unit/Model/ObserverTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ public function dataProviderUpdateProductOptions()
9191
'expected' => [
9292
'calculationAlgorithm' => 'TOTAL_BASE_CALCULATION',
9393
'optionTemplate' => '<%= data.label %><% if (data.finalPrice.value) '.
94-
'{ %> <%= data.finalPrice.formatted %> (Excl. tax: <%= data.basePrice.formatted %>)<% } %>',
94+
'{ %> +<%= data.finalPrice.formatted %> (Excl. tax: <%= data.basePrice.formatted %>)<% } %>',
9595
],
9696
'displayBothPrices' => true,
9797
'priceIncludesTax' => false,
@@ -101,7 +101,7 @@ public function dataProviderUpdateProductOptions()
101101
'expected' => [
102102
'calculationAlgorithm' => 'TOTAL_BASE_CALCULATION',
103103
'optionTemplate' => '<%= data.label %><% if (data.basePrice.value) '.
104-
'{ %> <%= data.basePrice.formatted %><% } %>',
104+
'{ %> +<%= data.basePrice.formatted %><% } %>',
105105
],
106106
'displayBothPrices' => false,
107107
'priceIncludesTax' => true,

0 commit comments

Comments
 (0)