Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0036028 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Openbravo ERP] 04. Warehouse management | major | always | 2017-05-17 18:52 | 2017-06-09 02:49 | |||
Reporter | ngarcia | View Status | public | |||||
Assigned To | collazoandy4 | |||||||
Priority | urgent | Resolution | fixed | Fixed in Version | 3.0PR17Q3 | |||
Status | closed | Fix in branch | Fixed in SCM revision | 40bb5c5e4947 | ||||
Projection | none | ETA | none | Target Version | ||||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Merge Request Status | ||||||||
Review Assigned To | aferraz | |||||||
OBNetwork customer | OBPS | |||||||
Web browser | ||||||||
Modules | Core | |||||||
Support ticket | 46806 | |||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0036028: Wrong stock in Valued Stock Report if there exists more than one record in Transaction Costs tab with different Accounting Date | |||||||
Description | Wrong stock in Valued Stock Report if there exists more than one record in Transaction Costs tab with different Accounting Date The problem is that some transactions are taken into account more than once as they are joint with more than one record of the following subquery: LEFT JOIN (SELECT Sum(cost) AS trxcost, m_transaction_id, c_currency_id, COALESCE(dateacct, costdate) AS movementdate FROM m_transaction_cost WHERE dateacct < To_date('18-05-2017') AND dateacct > To_date('01-01-0001') GROUP BY m_transaction_id, c_currency_id, COALESCE(dateacct, costdate)) tc | |||||||
Steps To Reproduce | As group admin role: Create a product and add a purchase price list to it Create a goods receipt and receive 100 units of it Launch the Costing Background Process and check it has finished successfully Add a manual cost adjustment to it with an accounting date one day after the goods receipt movement date Launch the Valued Stock Report for this accounting date and check the stock is shown duplicated | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
![]() |
|
![]() |
|
(0097173) collazoandy4 (viewer) 2017-06-06 21:17 |
As group admin role: Go to Product window and create a new record: Search Key/Name: IssueProduct Product Category: Fruit juice Create a new record in Price tab: Price List Version: Tarifa Bebidas Alegres Unit Price/List Price: 10 Go to goods receipt window and create a new record: Business Partner: Bebidas Alegres, S.L. Create a new record in Lines tab: Product: IssueProduct Movement Quantity: 100 Complete the receipt Launch the Costing Background Process and check it has finished successfully Go to Product window and select the previous one created: Check a new entry in Costing tab with total movement quantity = 100 In Transaction tab select the record and clic on Manual Cost Adjustment Total Cost amount: 500 Accounting date: Tomorrow Launch the Valued Stock Report for F&B EspaƱa S.A organization Fruit juice Product Category Tomorrow Date and check the stock doesn't shown duplicated |
(0097222) hgbot (developer) 2017-06-08 17:10 |
Repository: erp/devel/pi Changeset: 40bb5c5e4947774629da8c8cfa2c63bc0eb7a6ba Author: Armaignac <collazoandy4 <at> gmail.com> Date: Tue Jun 06 18:36:25 2017 -0400 URL: http://code.openbravo.com/erp/devel/pi/rev/40bb5c5e4947774629da8c8cfa2c63bc0eb7a6ba [^] Fixes issue 36028: Wrong stock in Valued Stock Report Some transactions are taken into account more than once as they are joint with more than one record of the m_transaction_cost with different Accounting date making a duplicate product stock. The accounting date was removed from the joint query for a correct group of records with m_transaction_id and c_currency_id. The transaction movement date is now used instance of accounting date for group the stock records --- M src/org/openbravo/erpCommon/ad_reports/ReportValuationStock_data.xsql --- |
(0097223) aferraz (viewer) 2017-06-08 17:11 |
Code review + Testing OK |
(0097264) hudsonbot (viewer) 2017-06-09 02:49 |
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/682983c2eead [^] Maturity status: Test |
![]() |
|||
Date Modified | Username | Field | Change |
2017-05-17 18:52 | ngarcia | New Issue | |
2017-05-17 18:52 | ngarcia | Assigned To | => Triage Finance |
2017-05-17 18:52 | ngarcia | OBNetwork customer | => No |
2017-05-17 18:52 | ngarcia | Modules | => Core |
2017-05-17 18:52 | ngarcia | Support ticket | => 46806 |
2017-05-17 18:52 | ngarcia | Resolution time | => 1496786400 |
2017-05-17 18:52 | ngarcia | Triggers an Emergency Pack | => No |
2017-05-17 18:52 | ngarcia | Issue Monitored: networkb | |
2017-05-17 18:57 | heccam | Issue Monitored: heccam | |
2017-05-18 12:54 | ngarcia | OBNetwork customer | No => Yes |
2017-05-19 13:30 | aferraz | Assigned To | Triage Finance => collazoandy4 |
2017-05-22 15:33 | rbianchini | Issue Monitored: rbianchini | |
2017-06-03 02:18 | collazoandy4 | Status | new => scheduled |
2017-06-06 21:17 | collazoandy4 | Note Added: 0097173 | |
2017-06-08 17:10 | hgbot | Checkin | |
2017-06-08 17:10 | hgbot | Note Added: 0097222 | |
2017-06-08 17:10 | hgbot | Status | scheduled => resolved |
2017-06-08 17:10 | hgbot | Resolution | open => fixed |
2017-06-08 17:10 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/40bb5c5e4947774629da8c8cfa2c63bc0eb7a6ba [^] |
2017-06-08 17:11 | aferraz | Review Assigned To | => aferraz |
2017-06-08 17:11 | aferraz | Note Added: 0097223 | |
2017-06-08 17:11 | aferraz | Status | resolved => closed |
2017-06-08 17:11 | aferraz | Fixed in Version | => 3.0PR17Q3 |
2017-06-09 02:49 | hudsonbot | Checkin | |
2017-06-09 02:49 | hudsonbot | Note Added: 0097264 |
Copyright © 2000 - 2009 MantisBT Group |