Login  Register

[jira] [Commented] (OFBIZ-4326) VAT Correction for ProductPrice that has priceWithTax =N is not correct (with patch)

Posted by Nicolas Malin (Jira) on Jun 28, 2011; 12:53am
URL: http://ofbiz.116.s1.nabble.com/jira-Created-OFBIZ-4326-VAT-Correction-for-ProductPrice-that-has-priceWithTax-N-is-not-correct-with--tp3622553p3629233.html


    [ https://issues.apache.org/jira/browse/OFBIZ-4326?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13056234#comment-13056234 ]

Paul Foxworthy commented on OFBIZ-4326:
---------------------------------------

Hi Stéphane,

Could you say a bit more about the problem you've found? I don't understand what the problem is with rounding.

I don't think your suggested fix will do the job. The itemPrice parameter will have the price the user entered, which might be different from the list price or any other standard price. If the list price includes tax, lines 434-537 are working out the difference between the tax including in the list price and the tax included in the actual price for this order, and putting in the difference as an order adjustment.

If the problem is with rounding, why not just round the itemPrice, or round enteredTotalPriceWithTax when it's calculated on line 515?

Thanks

Paul Foxworthy


> VAT Correction for ProductPrice that has priceWithTax =N is not correct (with patch)
> ------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-4326
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-4326
>             Project: OFBiz
>          Issue Type: Bug
>          Components: accounting
>    Affects Versions: SVN trunk
>            Reporter: Stéphane DUCAS
>            Priority: Minor
>         Attachments: vatCorrection.patch, vatCorrection_2.patch, vatCorrection_2.patch
>
>
> The VAT correction adjustment is not correct because it's based on the price not rounded (while tax adjustment is calculated from the rounded price).
> I provide the patch for that.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira