Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0013958Openbravo ERP09. Financial managementpublic2010-07-14 17:022010-09-29 11:57
Reporternetworkb 
Assigned Tosathiyan 
PriorityhighSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS20OS VersionCommunity Appliance
Product Version 
Target Version2.50MP21Fixed in Version2.50MP21 
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
Summary0013958: cursor incorrect in process c_bankstatement_post
DescriptionIn c_bankstatement_post, the cursor "Cur_Debts" is wrong
This cursor returns all partners that they have got a debt_payment does not cancelled, and it runs process "C_BP_SOCREDITUSED_REFRESH" to each one.

For this reason, this process is very slow
Steps To Reproduce. create a bankstatement with a debt payment.
. process bankstatement
. execute the query:
    SELECT DISTINCT C_BPartner_ID
    FROM C_DEBT_PAYMENT
    WHERE C_Settlement_Cancel_ID=Cur_AutomaticSettlementCancel.C_SETTLEMENT_ID
    OR C_Settlement_Cancel_ID IS NULL
    OR C_Settlement_Generate_ID=Cur_AutomaticSettlementCancel.C_SETTLEMENT_ID AND ISRECEIPT = 'Y'

. a lot of partners are returned, and the process "C_BP_SOCREDITUSED_REFRESH" only should be run to partner of debt payment
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to defect 00146342.50MP22 closed adrianromero Cannot unprocess a bank statement with more than one business partner 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2010-07-14 17:02networkbNew Issue
2010-07-14 17:02networkbAssigned To => dalsasua
2010-07-14 17:02networkbOBNetwork customer => Yes
2010-07-14 17:10networkbTarget Version => 2.50MP21
2010-07-28 07:38sathiyanAssigned Todalsasua => sathiyan
2010-08-03 13:40hgbotCheckin
2010-08-03 13:40hgbotNote Added: 0029767
2010-08-03 13:40hgbotStatusnew => resolved
2010-08-03 13:40hgbotResolutionopen => fixed
2010-08-03 13:40hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/d42db16889e8833a9764bf007e1f0215b785e5a8 [^]
2010-08-05 09:16hudsonbotCheckin
2010-08-05 09:16hudsonbotNote Added: 0029798
2010-08-19 13:18psarobeNote Added: 0030150
2010-08-19 13:18psarobeStatusresolved => new
2010-08-19 13:18psarobeResolutionfixed => open
2010-08-19 13:21psarobeNote Edited: 0030150bug_revision_view_page.php?bugnote_id=0030150#r697
2010-08-19 13:48psarobeNote Deleted: 0030150
2010-08-19 13:48psarobeStatusnew => scheduled
2010-08-19 13:48psarobefix_in_branch => pi
2010-08-19 13:48psarobeStatusscheduled => resolved
2010-08-19 13:48psarobeResolutionopen => fixed
2010-08-25 09:50psarobeNote Added: 0030296
2010-08-25 09:50psarobeStatusresolved => closed
2010-08-25 09:50psarobeFixed in Version => 2.50MP21
2010-09-13 12:38anonymoussf_bug_id0 => 3065157
2010-09-21 13:07psarobeRelationship addedrelated to 0014634
2010-09-21 13:24hgbotCheckin
2010-09-21 13:24hgbotNote Added: 0031274
2010-09-29 11:57hudsonbotCheckin
2010-09-29 11:57hudsonbotNote Added: 0031452

Notes
(0029767)
hgbot   
2010-08-03 13:40   
Repository: erp/devel/pi
Changeset: d42db16889e8833a9764bf007e1f0215b785e5a8
Author: Sathiyan Sivaprakasam <sathiyan.sivaprakasam <at> openbravo.com>
Date: Tue Aug 03 17:08:58 2010 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/d42db16889e8833a9764bf007e1f0215b785e5a8 [^]

Fixes Issue 13958: cursor incorrect in process c_bankstatement_post

---
M src-db/database/model/functions/C_BANKSTATEMENT_POST.xml
---
(0029798)
hudsonbot   
2010-08-05 09:16   
A changeset related to this issue has been promoted to main after passing a series of tests and an OBX has been generated:

Changeset: http://code.openbravo.com/erp/devel/main/rev/d42db16889e8 [^]
Merge Changeset: http://code.openbravo.com/erp/devel/main/rev/02ad8e3730d2 [^]
Tests: http://builds.openbravo.com/view/int/ [^]
OBX: http://builds.openbravo.com/erp/core/obx/OpenbravoERP-2.50CI.18019.obx [^]
(0030296)
psarobe   
2010-08-25 09:50   
Tested working fine
(0031274)
hgbot   
2010-09-21 13:24   
Repository: erp/devel/pi
Changeset: 3249119dbc098164e2b74901ed9b0de384c24cab
Author: Adrián Romero <adrianromero <at> openbravo.com>
Date: Tue Sep 21 13:22:49 2010 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/3249119dbc098164e2b74901ed9b0de384c24cab [^]

Fixes issue 0014634: Cannot unprocess a bank statement with more than one business partner
It has been reverted the fix created in issue 13958 and applied a proper fix that consist in only removing the condition C_Settlement_Cancel_ID IS NULL when selecting business partners to update

---
M src-db/database/model/functions/C_BANKSTATEMENT_POST.xml
---
(0031452)
hudsonbot   
2010-09-29 11:57   
A changeset related to this issue has been promoted to main after passing a series of tests and an OBX has been generated:

Changeset: http://code.openbravo.com/erp/devel/main/rev/3249119dbc09 [^]
Merge Changeset: http://code.openbravo.com/erp/devel/main/rev/51c2c2967221 [^]
Tests: http://builds.openbravo.com/view/int/ [^]
OBX: http://builds.openbravo.com/erp/core/obx/OpenbravoERP-2.50CI.18452.obx [^]