Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0035498 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Openbravo ERP] 09. Financial management | major | always | 2017-03-13 18:58 | 2017-03-22 18:41 | |||
Reporter | maite | View Status | public | |||||
Assigned To | mcobas | |||||||
Priority | urgent | Resolution | fixed | Fixed in Version | 3.0PR17Q2 | |||
Status | closed | Fix in branch | Fixed in SCM revision | 987adac146a2 | ||||
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 | 45868 | |||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0035498: Not possible to post Goods Receipt when using "Book Using Purchase Order Price" and order is Voided | |||||||
Description | When configuring Product as "Book Using Purchase Order Price", it is not possible to post goods receipt document (it is set as "Document disabled for accounting") in case Purchase Order is voided before receipt is posted. According with documentation (http://wiki.openbravo.com/wiki/Product [^]) "product price is required for posting", which is still set in Purchase Order Line | |||||||
Steps To Reproduce | 1. Configure Product as "Expense type" and "Book using purchase order price" 2. Register Purchase Order using this product and process 3. Register related Goods Receipt using "Create lines from" previous order and process. Not post the receipt 4. Access Purchase Order and run "Reactivate" process. After that run "Void" process. Realize that "Ordered qty" has been set to 0 5. Access Goods Receipt and try to post it. Error "Document disabled for accounting" will be raised | |||||||
Proposed Solution | Seems that getDocumentConfirmation method in DocInOut.java class is checking existence of line net amount instead of unit price | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
![]() |
|
![]() |
|
(0095052) mcobas (viewer) 2017-03-15 15:37 |
1. Go to Product screen and create one new. 1.1 Configure Product as "Expense type" and "Book using purchase order price". 1.2 Sale checkbox = unselected. 2. Register a Purchase Order using previous product and process. 3. Register related Goods Receipt using "Create lines from" previous order and process. 3.1 Not post the receipt. 4. Go to previous Purchase Order and run "Reactivate" process. 4.1 Run "Reactivate" process. 4.2 After that run "Void" process. 4.3 Notice that "Ordered qty" has been set to 0. 5. Go to previous Goods Receipt. 5.1 Run "Post" process. 6. Notice that open a new screen named "Journal Entries Report" which shows the ledger entries created for the document. |
(0095499) hgbot (developer) 2017-03-22 12:01 |
Repository: erp/devel/pi Changeset: 987adac146a2f02834f05ce1870256dc3b330c77 Author: Miguel <martencobasmiguel <at> gmail.com> Date: Wed Mar 15 10:45:37 2017 -0400 URL: http://code.openbravo.com/erp/devel/pi/rev/987adac146a2f02834f05ce1870256dc3b330c77 [^] Fixes issue 35498: Error while posting a Good Receipt related an Order is Void The error is caused because when post a Goods Receipt using "Book Using Purchase Order Price" and order is Voided, is being used the net amount of the line corresponding to the Order instead of the net price. To solve this problem, consider the proposed solution by modifying the getDocumentConfirmation method so that it takes into account the net price of the line instead of its net amount. --- M src/org/openbravo/erpCommon/ad_forms/DocInOut.java --- |
(0095500) hgbot (developer) 2017-03-22 12:02 |
Repository: erp/devel/pi Changeset: 1d1a20c06e8cdfee1ca23bf7ce320738b82850c6 Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com> Date: Wed Mar 22 11:55:56 2017 +0100 URL: http://code.openbravo.com/erp/devel/pi/rev/1d1a20c06e8cdfee1ca23bf7ce320738b82850c6 [^] Related to issue 35498: Code review improvements Transaction cost should be Goods Receipt Line movement quantity * Purchase Order Line net unit price. --- M src/org/openbravo/erpCommon/ad_forms/DocInOut.java --- |
(0095501) aferraz (viewer) 2017-03-22 12:05 |
Code review + Testing OK |
(0095515) hudsonbot (viewer) 2017-03-22 18:41 |
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/fdc639ebd8e0 [^] Maturity status: Test |
(0095516) hudsonbot (viewer) 2017-03-22 18:41 |
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/fdc639ebd8e0 [^] Maturity status: Test |
![]() |
|||
Date Modified | Username | Field | Change |
2017-03-13 18:58 | maite | New Issue | |
2017-03-13 18:58 | maite | Assigned To | => Triage Finance |
2017-03-13 18:58 | maite | OBNetwork customer | => Yes |
2017-03-13 18:58 | maite | Modules | => Core |
2017-03-13 18:58 | maite | Support ticket | => 45868 |
2017-03-13 18:58 | maite | Resolution time | => 1491170400 |
2017-03-13 18:58 | maite | Triggers an Emergency Pack | => No |
2017-03-13 18:58 | maite | Issue Monitored: networkb | |
2017-03-14 17:09 | markmm82 | Assigned To | Triage Finance => mcobas |
2017-03-15 15:31 | markmm82 | Status | new => scheduled |
2017-03-15 15:37 | mcobas | Note Added: 0095052 | |
2017-03-22 12:01 | hgbot | Checkin | |
2017-03-22 12:01 | hgbot | Note Added: 0095499 | |
2017-03-22 12:01 | hgbot | Status | scheduled => resolved |
2017-03-22 12:01 | hgbot | Resolution | open => fixed |
2017-03-22 12:01 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/987adac146a2f02834f05ce1870256dc3b330c77 [^] |
2017-03-22 12:02 | hgbot | Checkin | |
2017-03-22 12:02 | hgbot | Note Added: 0095500 | |
2017-03-22 12:05 | aferraz | Review Assigned To | => aferraz |
2017-03-22 12:05 | aferraz | Note Added: 0095501 | |
2017-03-22 12:05 | aferraz | Status | resolved => closed |
2017-03-22 12:05 | aferraz | Fixed in Version | => 3.0PR17Q2 |
2017-03-22 18:41 | hudsonbot | Checkin | |
2017-03-22 18:41 | hudsonbot | Note Added: 0095515 | |
2017-03-22 18:41 | hudsonbot | Checkin | |
2017-03-22 18:41 | hudsonbot | Note Added: 0095516 |
Copyright © 2000 - 2009 MantisBT Group |