Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0015794Openbravo ERP09. Financial managementpublic2011-01-27 13:192011-03-01 13:56
Reporteradrianromero 
Assigned Todalsasua 
PriorityurgentSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS20OS VersionCommunity Appliance
Product Version2.50MP25 
Target VersionFixed in Version 
Merge Request Status
Review Assigned To
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0015794: bankstatement in a wrong status: it is not processed and debt payments are cancelled
Descriptionif a bankstatement has a lot of payments (we have reproduced the problem with 1200 lines, but the end client has got 368 lines), the execution of process of this bankstatement could be along to several seconds.
If during this execution, the header is saved by other user, this request is paused until the process is finished.
When the request is executed, the field "processed" is saved with value "N", but the bank statement has been processed, because the debt payments are cancelled in a setlement generated by bankstatement_post process
Steps To Reproduce1. create a bankstatement with a lot of lines (close to 1000)
2. process bankstatement.
3. before that bankstatement_post finish, other user edits the bankstatement header, changes the name (for example), and saves the bankstatement
4. the execution of step 2 finish, and the bankstatement is saved like "processed" = N
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
depends on defect 00155662.50MP26 closed dalsasua bankstatement in a wrong status: it is not processed and debt payments are cancelled 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2011-01-27 13:19adrianromeroNew Issue
2011-01-27 13:19adrianromeroAssigned To => dalsasua
2011-01-27 13:19adrianromeroModules => Core
2011-01-27 13:19adrianromeroIssue generated from0015566
2011-01-27 13:19adrianromeroRelationship addeddepends on 0015566
2011-01-27 13:20adrianromeroStatusnew => scheduled
2011-01-27 13:20adrianromerofix_in_branch => pi
2011-01-28 16:51dalsasuafix_in_branchpi => 2.50
2011-02-02 14:14hgbotCheckin
2011-02-02 14:14hgbotNote Added: 0034013
2011-02-02 14:14hgbotStatusscheduled => resolved
2011-02-02 14:14hgbotResolutionopen => fixed
2011-02-02 14:14hgbotFixed in SCM revision => http://code.openbravo.com/erp/stable/2.50/rev/820242d519ab4a5e0e4972290f750341e63fa980 [^]
2011-02-02 14:22adrianromeroNote Added: 0034015
2011-02-03 11:27hgbotCheckin
2011-02-03 11:27hgbotNote Added: 0034023
2011-02-03 11:27hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/stable/2.50/rev/820242d519ab4a5e0e4972290f750341e63fa980 [^] => http://code.openbravo.com/erp/stable/2.50/rev/63c194f03f6cf44c2ff04098a92c5dbed3824939 [^]
2011-03-01 13:56dalsasuaStatusresolved => closed

Notes
(0034013)
hgbot   
2011-02-02 14:14   
Repository: erp/stable/2.50
Changeset: 820242d519ab4a5e0e4972290f750341e63fa980
Author: Adrián Romero <adrianromero <at> openbravo.com>
Date: Wed Feb 02 14:13:34 2011 +0100
URL: http://code.openbravo.com/erp/stable/2.50/rev/820242d519ab4a5e0e4972290f750341e63fa980 [^]

Fixes issue 0015794: bankstatement in a wrong status: it is not processed and debt payments are cancelled
It has been added a condition in the C_BANKSTATEMENT trigger that verifies when saving that there has not been running process concurrently

---
M src-db/database/model/triggers/C_BSTMT_CHK_RESTRICTIONS_TRG.xml
---
(0034015)
adrianromero   
2011-02-02 14:22   
* Testing the issue

Kindly follow the steps to test and verify that now in one window the process finish successfully and in the window where the user tried to save a record it appears a message saying that there is other process active.

Verify also that the message appears when unprocessing bankstatements.

Verify also that records in bankstatemens can be created/deleted/updated without any impediment.

And finally check the correct behavior in both database engines PostgreSQL and Oracle.

* Other areas affected

It has been modified the trigger of table C_BANKSTATEMENS. All actions related to this table can be affected.
(0034023)
hgbot   
2011-02-03 11:27   
Repository: erp/stable/2.50
Changeset: 63c194f03f6cf44c2ff04098a92c5dbed3824939
Author: Adrián Romero <adrianromero <at> openbravo.com>
Date: Thu Feb 03 11:25:55 2011 +0100
URL: http://code.openbravo.com/erp/stable/2.50/rev/63c194f03f6cf44c2ff04098a92c5dbed3824939 [^]

Fixes issue 0015794: bankstatement in a wrong status: it is not processed and debt payments are cancelled
It has been added a condition in the C_BANKSTATEMENT trigger that verifies when saving that there has not been running process concurrently

---
M src-db/database/model/triggers/C_BSTMT_CHK_RESTRICTIONS_TRG.xml
---