Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0022590 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Openbravo ERP] 09. Financial management | major | always | 2012-12-17 05:18 | 2013-01-03 13:12 | |||
| Reporter | pjuvara | View Status | public | |||||
| Assigned To | Sandrahuguet | |||||||
| Priority | normal | Resolution | fixed | Fixed in Version | ||||
| Status | closed | Fix in branch | pi | Fixed in SCM revision | 34510269985f | |||
| Projection | none | ETA | none | Target Version | ||||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | 3.0MP17.1 | SCM revision | ||||||
| Review Assigned To | dmiguelez | |||||||
| Web browser | ||||||||
| Modules | Core | |||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0022590: Data corruption when creating a financial account transaction in a closed period | |||||||
| Description | When you create a financial account transaction in a closed period, an unmanaged exception is raised (org.openbravo.base.exception.OBException - see attachment1.png). After the exception, the transaction is committed but the integrity of the transaction is broken and the financial account balance is not updated. Compare attachment1.png (when the exception takes place) and attachment2.png (after the exception). You can see that the current balance for Bank Account 2 has not changed but in attachment2.png there is a new transaction, so the current balance in that picture should be 149,990.00. | |||||||
| Steps To Reproduce | In the demo environment: 1) Login with role F&B US Inc. Admin 2) Navigate to Open/Close Period and close all periods in 2011 for F&B US 3) Navigate to Financial Account and press Add Transactions 4) Create a G/L Item transaction during 2011. Press OK. The exception is raised. 5) Acknowledge the exception and refresh the Transaction tab; the transaction is retrieved but the account balance has not been updated. | |||||||
| Proposed Solution | You should not be able to save a transaction in a closed period. Rather than a technical exception, a user friendly error should be displayed. The Add Transaction window should not close and the user should stay there. | |||||||
| Tags | No tags attached. | |||||||
| Attached Files |  attachment1.png [^] (141,126 bytes) 2012-12-17 05:18
attachment2.png [^] (145,100 bytes) 2012-12-17 05:18
| |||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
|
 Relationships |
|
Notes |
|
|
(0054953) pjuvara (reporter) 2012-12-17 05:20 |
Note: this is a very serious issue as an innocent end user mistake (mistype the year on a date field) can result in data corruption where the financial account balance does not match the transactions and the value in the balance sheet. |
|
(0055104) Sandrahuguet (developer) 2012-12-20 10:52 |
Test Plan: 1) Login with role F&B US Inc. Admin 2) Navigate to Open/Close Period and close all periods in 2011 for F&B US 3) Navigate to Financial Account and press Add Transactions 4) Create a G/L Item transaction during 2011. Press OK 5) Notice that you get an error message and the transacction is not added |
|
(0055235) hgbot (developer) 2012-12-27 10:15 |
Repository: erp/devel/pi Changeset: 34510269985f2c3407b04a7886f22a770077a360 Author: Sandra Huguet <sandra.huguet <at> openbravo.com> Date: Thu Dec 27 10:06:22 2012 +0100 URL: http://code.openbravo.com/erp/devel/pi/rev/34510269985f2c3407b04a7886f22a770077a360 [^] Fixed bug 22590 Display the error message properly --- M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/ad_actionbutton/AddTransaction.html M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/ad_actionbutton/AddTransaction.java --- |
|
(0055236) dmiguelez (developer) 2012-12-27 11:34 |
Code review + testing ok |
|
(0055327) hudsonbot (developer) 2013-01-03 13:12 |
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/792728467ed0 [^] Maturity status: Test |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2012-12-17 05:18 | pjuvara | New Issue | |
| 2012-12-17 05:18 | pjuvara | Assigned To | => dmiguelez |
| 2012-12-17 05:18 | pjuvara | File Added: attachment1.png | |
| 2012-12-17 05:18 | pjuvara | Modules | => Core |
| 2012-12-17 05:18 | pjuvara | Triggers an Emergency Pack | => No |
| 2012-12-17 05:18 | pjuvara | File Added: attachment2.png | |
| 2012-12-17 05:20 | pjuvara | Note Added: 0054953 | |
| 2012-12-19 17:40 | Sandrahuguet | Assigned To | dmiguelez => Sandrahuguet |
| 2012-12-19 17:41 | Sandrahuguet | Status | new => scheduled |
| 2012-12-19 17:41 | Sandrahuguet | fix_in_branch | => pi |
| 2012-12-20 10:52 | Sandrahuguet | Note Added: 0055104 | |
| 2012-12-27 10:15 | hgbot | Checkin | |
| 2012-12-27 10:15 | hgbot | Note Added: 0055235 | |
| 2012-12-27 10:15 | hgbot | Status | scheduled => resolved |
| 2012-12-27 10:15 | hgbot | Resolution | open => fixed |
| 2012-12-27 10:15 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/34510269985f2c3407b04a7886f22a770077a360 [^] |
| 2012-12-27 11:34 | dmiguelez | Review Assigned To | => dmiguelez |
| 2012-12-27 11:34 | dmiguelez | Note Added: 0055236 | |
| 2012-12-27 11:34 | dmiguelez | Status | resolved => closed |
| 2013-01-03 13:12 | hudsonbot | Checkin | |
| 2013-01-03 13:12 | hudsonbot | Note Added: 0055327 | |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |