View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0006761 | OXID eShop (all versions) | 1.02. Price calculations (discounts, coupons, additional costs etc.) | public | 2017-12-27 00:41 | 2018-01-05 11:56 |
| Reporter | vanilla thunder | Assigned To | |||
| Priority | normal | Severity | minor | Reproducibility | always |
| Status | closed | Resolution | fixed | ||
| Product Version | 6.0.0 | ||||
| Fixed in Version | 6.0.1 | ||||
| Summary | 0006761: oxVariantHandler always uses brutto price for MD Variants | ||||
| Description | regardless configured netto / brutto price mode, oxvarianthandler always uses brutto price for building md variants https://github.com/OXID-eSales/oxideshop_ce/blob/master/source/Application/Model/VariantHandler.php#L70 found originally in 4.10 but also appears in 6.0 | ||||
| Steps To Reproduce | 1) enable at least "Show net prices in frontend (B2B)" ( master settings -> core settings -> settings -> VAT ) 2) insert my code from "additional information" into productmain.tpl line ~ 151 3) open Kuyichi Jeans ANNA product price says "from 78,07 €" and you see "plus tax" in footer but my table with md variants array shows brutto prices | ||||
| Additional Information | <table class="table table-striped table-bordered"> <tr> [{foreach from=$aVariantSelections.selections item=oList key=iKey}] <td>[{$oList->getLabel()}]</td> [{/foreach}] <td>[{oxmultilang ident="PRICE"}]</td> </tr> [{foreach from=$oDetailsProduct->getMdSubvariants() item=_mdVar key=iKey}] [{foreach from=$_mdVar->getMdSubvariants() item=_mdSubVar key=iSubKey name="mdsubvariants"}] <tr> [{if $smarty.foreach.mdsubvariants.first}] <td valign="middle" style="vertical-align:middle;" rowspan="[{$_mdVar->getMdSubvariants()|@count}]">[{$_mdVar->getName()}]</td> [{/if}] <td>[{$_mdSubVar->getName()}]</td> <td>[{oxprice price=$_mdSubVar->getDPrice()}]</td> </tr> [{/foreach}] [{/foreach}] </table> | ||||
| Tags | Solution Provided, Variants, VAT | ||||
| Theme | Not defined | ||||
| Browser | Not defined | ||||
| PHP Version | Not defined | ||||
| Database Version | Not defined | ||||
|
|
PR with suggested fix: https://github.com/OXID-eSales/oxideshop_ce/pull/618 |
|
|
Thanks, pull request had been merged recently to b-6.x |
