[
https://issues.apache.org/jira/browse/OFBIZ-6330?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15856988#comment-15856988
]
Paul Foxworthy commented on OFBIZ-6330:
---------------------------------------
Hi Nicolas,
The code here generates individual accounting entries for the tax authorities.
There is nothing adding the amounts up, so when the tax-inclusive price is
calculated, the variable invoiceTaxTotal has never been used and will have a
value of zero.
One reason I didn't take that approach (and I guess Kongrath thought the same)
is that createAcctgTransForSalesInvoice also just calls getInvoiceTaxTotal.
I understand that adding up the tax amounts would avoid a call to one more
method, at the expense of making the code a little more complicated. If you
still think that's a good idea, I'll change all three methods
createAcctgTransForCustomerReturnInvoice, createAcctgTransForSalesInvoice,
createAcctgTransForPurchaseInvoice to work that way.
What do you think?
Cheers
Paul
> The invoiceTaxTotal value is missing from createAcctgTransForPurchaseInvoice
> service
> ------------------------------------------------------------------------------------
>
> Key: OFBIZ-6330
> URL: https://issues.apache.org/jira/browse/OFBIZ-6330
> Project: OFBiz
> Issue Type: Bug
> Components: accounting
> Affects Versions: Trunk
> Reporter: Kongrath Suankaewmanee
> Assignee: Paul Foxworthy
> Attachments: GeneralLedgerServices.patch,
> OFBIZ-6330_TaxAccountingOnPurchasesAndReturns.patch
>
>
> Hi All,
> Scenario: The sum of debit and credit in InvoiceAcctgTransEntriesPdf of
> purchase invoice are not equal.
> Question: I'm not sure why the createAcctgTransForPurchaseInvoice service did
> not call the method to get invoiceTaxTotal.
> <call-class-method method-name="getInvoiceTaxTotal"
> class-name="org.ofbiz.accounting.invoice.InvoiceWorker"
> ret-field="invoiceTaxTotal">
> <field field="invoice" type="GenericValue"/>
> </call-class-method>
> And the invoiceTaxTotal value needs to add to totalAmountFromInvoice via code
> below:
> <calculate field="totalAmountFromInvoice" type="BigDecimal"
> decimal-scale="${ledgerDecimals}" rounding-mode="${roundingMode}">
> <calcop operator="add">
> <calcop operator="get" field="totalAmountFromInvoice"/>
> <calcop operator="get" field="invoiceTaxTotal"/>
> </calcop>
> </calculate>
> That it should work like the createAcctgTransForSalesInvoice service of the
> sales invoice.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
