Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0035514 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Openbravo ERP] 09. Financial management | major | always | 2017-03-14 17:15 | 2017-03-29 15:06 | |||
| Reporter | maite | View Status | public | |||||
| Assigned To | collazoandy4 | |||||||
| Priority | urgent | Resolution | fixed | Fixed in Version | 3.0PR17Q2 | |||
| Status | closed | Fix in branch | Fixed in SCM revision | c6ab374457d8 | ||||
| Projection | none | ETA | none | Target Version | ||||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | SCM revision | |||||||
| Review Assigned To | aferraz | |||||||
| Web browser | ||||||||
| Modules | Core | |||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0035514: Wrong "Product Expense" account used when posting purchase invoice referring "tax not deductible" | |||||||
| Description | In case invoice has different product with different "Product Expense" account configured, accounting entry is not posting "tax not deductible" amount properly to each account | |||||||
| Steps To Reproduce | 1. Create new Tax Rate for "F&B España, S.A" org, "IVA+Deductible" name, valid from today and summary level=yes 2. Create new Tax Rate for "F&B España, S.A" org, "IVA" name, valid from today, rate=20 and parent=IVA+Deductible 3. Create new Tax Rate for "F&B España, S.A" org, "Deductible" name, valid from today, rate=1, "Tax not deductible=YES" and parent=IVA+Deductible 4. Register Purchase Invoice for "España" org, "Bebidas alegres" BP. Add line for product "Cerveza Ale 0,5L" set price=100 and tax=IVA+Deductible Add line for product "Cerveza Lager 0,5L" set price=100 and tax=IVA+Deductible Process invoice 5. Navigate to "Cerveza Ale 0,5L" and modify Accounting tab of "F&B España, S.A. US/A/Euro" to set "Product Expense" account to "56000 - Fianzas recibidas a corto plazo" (or any different from 60000 - Compras de mercaderías) 6. Post Purchase Invoice. Access Accounting tab, filter by "F&B España, S.A. US/A/Euro" general ledger and realize that deductible amount of 2eur has been posted to one of the product's "Product Expense" account (when expected behaviour is to have both Product Expense accounts affected, each one with 1eur amount) | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
|
 Relationships |
|
Notes |
|
|
(0095519) collazoandy4 (reporter) 2017-03-22 22:09 |
As F&B International Group Create new Tax Rate: Organization: F&B España, S.A Name: IVA+Deductible Valid from: Today Summary level: yes Create new Tax Rate: Organization: F&B España, S.A Name: IVA Valid from: Today Rate: 20 Parent Tax Rate: IVA+Deductible Create new Tax Rate: Organization: F&B España, S.A Name: Deductible Valid from: Today Rate: 1 Parent Tax Rate: IVA+Deductible Tax Not Deductible: Yes Create a Purchase Invoice: Organization: F&B España, S.A Add a line: Product: Cerveza Ale 0,5L Price: 100 Tax: IVA+Deductible Add a line: Product: Cerveza Lager 0,5L Price: 100 Tax: IVA+Deductible Process invoice Go to Product window and edit Cerveza Ale 0,5L record In the Accounting tab of F&B España, S.A. US/A/Euro set Product Expense account to 56000 - Fianzas recibidas a corto plazo Post the created invoice Check that the accounts 56000 - Fianzas recibidas a corto plazo and 60000 - Compras de mercaderías is used with an amount of 1 |
|
(0095600) hgbot (developer) 2017-03-28 11:31 |
Repository: erp/devel/pi Changeset: c6ab374457d88933377393d920cad2d8485dd808 Author: Armaignac <collazoandy4 <at> gmail.com> Date: Wed Mar 22 17:30:36 2017 -0400 URL: http://code.openbravo.com/erp/devel/pi/rev/c6ab374457d88933377393d920cad2d8485dd808 [^] Fixes issue 35514: Wrong Product Expense account used with tax not deductible When posting a Purchase Invoice with lines that contains a not deductible tax the query to get the tax amount by lines was failing, also in the facacct create lines process always was get the first one ignoring the product account Now the amount its get from the invoicelinetax and a new method was created to get the product accounts in the purchase invoice filtered by the product_id --- M src/org/openbravo/erpCommon/ad_forms/DocInvoice.java M src/org/openbravo/erpCommon/ad_forms/DocInvoice_data.xsql M src/org/openbravo/erpCommon/ad_forms/DocLineInvoice_data.xsql --- |
|
(0095601) hgbot (developer) 2017-03-28 11:31 |
Repository: erp/devel/pi Changeset: aceca41e08a189758d8eba6244b3c88f3b3607af Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com> Date: Tue Mar 28 11:21:34 2017 +0200 URL: http://code.openbravo.com/erp/devel/pi/rev/aceca41e08a189758d8eba6244b3c88f3b3607af [^] Related to issue 35514: Code review improvements Remove duplicated DocInvoiceData.selectProductAcct method making productId parameter optional. Update copyrights. --- M src/org/openbravo/erpCommon/ad_forms/DocInvoice.java M src/org/openbravo/erpCommon/ad_forms/DocInvoice_data.xsql M src/org/openbravo/erpCommon/ad_forms/DocLineInvoice_data.xsql --- |
|
(0095602) aferraz (manager) 2017-03-28 11:34 |
Code review + Testing OK |
|
(0095650) hudsonbot (developer) 2017-03-29 15:06 |
A changeset related to this issue has been promoted main and to the Central Repository, after passing a series of tests. Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/f901bb68d428 [^] Maturity status: Test |
|
(0095651) hudsonbot (developer) 2017-03-29 15:06 |
A changeset related to this issue has been promoted main and to the Central Repository, after passing a series of tests. Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/f901bb68d428 [^] Maturity status: Test |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2017-03-14 17:15 | maite | New Issue | |
| 2017-03-14 17:15 | maite | Assigned To | => Triage Finance |
| 2017-03-14 17:15 | maite | Modules | => Core |
| 2017-03-14 17:15 | maite | Resolution time | => 1491256800 |
| 2017-03-14 17:15 | maite | Triggers an Emergency Pack | => No |
| 2017-03-14 17:16 | maite | Issue Monitored: networkb | |
| 2017-03-14 18:26 | markmm82 | Assigned To | Triage Finance => luisrs8608 |
| 2017-03-20 17:28 | collazoandy4 | Assigned To | luisrs8608 => collazoandy4 |
| 2017-03-20 17:29 | collazoandy4 | Status | new => scheduled |
| 2017-03-22 22:09 | collazoandy4 | Note Added: 0095519 | |
| 2017-03-28 11:31 | hgbot | Checkin | |
| 2017-03-28 11:31 | hgbot | Note Added: 0095600 | |
| 2017-03-28 11:31 | hgbot | Status | scheduled => resolved |
| 2017-03-28 11:31 | hgbot | Resolution | open => fixed |
| 2017-03-28 11:31 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/c6ab374457d88933377393d920cad2d8485dd808 [^] |
| 2017-03-28 11:31 | hgbot | Checkin | |
| 2017-03-28 11:31 | hgbot | Note Added: 0095601 | |
| 2017-03-28 11:34 | aferraz | Review Assigned To | => aferraz |
| 2017-03-28 11:34 | aferraz | Note Added: 0095602 | |
| 2017-03-28 11:34 | aferraz | Status | resolved => closed |
| 2017-03-28 11:34 | aferraz | Fixed in Version | => 3.0PR17Q2 |
| 2017-03-29 15:06 | hudsonbot | Checkin | |
| 2017-03-29 15:06 | hudsonbot | Note Added: 0095650 | |
| 2017-03-29 15:06 | hudsonbot | Checkin | |
| 2017-03-29 15:06 | hudsonbot | Note Added: 0095651 | |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |