Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0015566Openbravo ERP09. Financial managementpublic2011-01-03 12:562011-05-24 10:56
Reportermalsasua 
Assigned Todalsasua 
PriorityurgentSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS20OS VersionCommunity Appliance
Product Versionpi 
Target Version2.50MP26Fixed in Version 
Merge Request Status
Review Assigned To
OBNetwork customerOBPS
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0015566: 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
TagsclosingMay2011
Relationships
blocks backport 0015794 closed dalsasua bankstatement in a wrong status: it is not processed and debt payments are cancelled 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2011-01-03 12:56malsasuaNew Issue
2011-01-03 12:56malsasuaAssigned To => dalsasua
2011-01-03 12:56malsasuaModules => Core
2011-01-03 12:56malsasuaOBNetwork customer => Yes
2011-01-10 10:15psarobeNote Added: 0033556
2011-01-10 18:05malsasuaIssue Monitored: networkb
2011-01-27 13:19adrianromeroStatusnew => scheduled
2011-01-27 13:19adrianromerofix_in_branch => pi
2011-01-27 13:19adrianromeroIssue cloned0015794
2011-01-27 13:19adrianromeroRelationship addedblocks 0015794
2011-01-27 13:20adrianromeroversion2.50MP25 => pi
2011-01-27 13:20adrianromerofix_in_branchpi =>
2011-01-31 19:25shuehnerIssue Monitored: shuehner
2011-02-02 14:18hgbotCheckin
2011-02-02 14:18hgbotNote Added: 0034014
2011-02-02 14:18hgbotStatusscheduled => resolved
2011-02-02 14:18hgbotResolutionopen => fixed
2011-02-02 14:18hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/3eee3bd161cf70bdc9d639832d881571f05da9b8 [^]
2011-02-02 14:22adrianromeroNote Added: 0034016
2011-02-03 11:30hgbotCheckin
2011-02-03 11:30hgbotNote Added: 0034024
2011-02-03 11:30hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/devel/pi/rev/3eee3bd161cf70bdc9d639832d881571f05da9b8 [^] => http://code.openbravo.com/erp/devel/pi/rev/d57526d04e89c2b6fd312c97debfc3a392b625ba [^]
2011-05-24 10:08dalsasuaTag Attached: closingMay2011
2011-05-24 10:56dalsasuaStatusresolved => closed

Notes
(0033556)
psarobe   
2011-01-10 10:15   
This is very strange since the PL update the field processing='Y'. This is just a comment that might help when fixing the issue.
(0034014)
hgbot   
2011-02-02 14:18   
Repository: erp/devel/pi
Changeset: 3eee3bd161cf70bdc9d639832d881571f05da9b8
Author: Adrián Romero <adrianromero <at> openbravo.com>
Date: Wed Feb 02 14:17:42 2011 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/3eee3bd161cf70bdc9d639832d881571f05da9b8 [^]

Fixes issue 0015566: 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
---
(0034016)
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.
(0034024)
hgbot   
2011-02-03 11:30   
Repository: erp/devel/pi
Changeset: d57526d04e89c2b6fd312c97debfc3a392b625ba
Author: Adrián Romero <adrianromero <at> openbravo.com>
Date: Thu Feb 03 11:29:36 2011 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/d57526d04e89c2b6fd312c97debfc3a392b625ba [^]

Fixes issue 0015566: 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
---