|View Issue Details|
|Type||Category||Severity||Reproducibility||Date Submitted||Last Update|
|defect||[Openbravo ERP] 09. Financial management||minor||always||2016-02-17 19:51||2016-04-08 11:00|
|Priority||immediate||Resolution||fixed||Fixed in Version|
|Status||closed||Fix in branch||Fixed in SCM revision||143d5c808926|
|OS Version||Database version||Ant version|
|Product Version||SCM revision|
|Review Assigned To||vmromanos|
|Regression level||Production - Confirmed Stable|
|Regression introduced in release||3.0PR14Q4|
|Regression introduced by commit||https://code.openbravo.com/erp/devel/pi/rev/179d97ee602d [^]|
|Triggers an Emergency Pack||No|
0032268: Wrong payment status when unmatching bankstatement line manually matched against an invoice
|Description||Wrong payment status when unmatching bankstatement line manually matched against an invoice|
|Steps To Reproduce||0. Configure "Bank" financial account to use "Advanced Matching algorithm"|
1. Access Financial Account, create Bank Statement record and add line with amount in =100. Process bank statement
2. Register Sales invoice with any line and total invoice amount = 100
3. Come back to Financial Account and run "Match Statement" process
4. Select "Cancel" option to not execute automatic matching
5. Run "add transaction" process by pressing + button
6. Create payment by pressing "+" button in payment's selector and select your invoice. Process Done and Done to finish with matching process
7. Filter in the grid by "matched=yes" records and unmatch record. Press OK to save reconciliation
8. Realize that transaction has correct status "deposited not cleared"
9. Navigate to related payment in and realize that status is "payment cleared" instead of "deposited not cleared"
NOTE: I followed same steps in an standard MP30.3 version and both payment and transaction has status "deposited not cleared"
|Tags||No tags attached.|
edited on: 2016-02-22 14:40
- Configure "Bank" financial account to use "Advanced Matching algorithm"
- Access Financial Account, create Bank Statement record and add line with amount in =100. Process bank statement
- Register Sales invoice with any line and total invoice amount = 100
- Come back to Financial Account and run "Match Statement" process
- Select "Cancel" option to not execute automatic matching
- Remove the filter.
- Run "add transaction" process by pressing + button
- Create payment by pressing "+" button in payment's selector and select your invoice. Process Done and Done to finish with matching process
- Filter in the grid by "matched=yes" records and unmatch record. Press OK to save reconciliation
- Check that financial account transaction has correct status "deposited not cleared".
- Go to related payment in and check that status is "deposited not cleared" which is correct.
- Match the payment and the statement line and check that payment status changes to "Payment Cleared" same as Financial Account Transaction.
Repeat same flow for purchase out, therefore Status is "Withdrawn not Cleared" for both payment out created and financial account transaction.
Test plan above verified.
Issue can be code reviewed and closed.
|Verified also with the new "Unmatch Selected" process|
Author: Atul Gaware <atul.gaware <at> openbravo.com>
Date: Mon Feb 22 15:33:27 2016 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/143d5c8089260d63d8f645b73c5cf43bf1216828 [^]
Fixes Issue 32268: Wrong payment status when unmatching bankstatement line
manually matched against an invoice
Incase a finTransaction is unmatched which has finPayment,status of finPayment
is also updated accordingly as per isReceipt flag
|Code review + testing OK|
Tests in TestLink:
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/b22fb0500156 [^]
Maturity status: Test
Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com>
Date: Wed Mar 09 19:05:46 2016 +0100
URL: http://code.openbravo.com/tools/automation/int/rev/a0e4097727b38183d7209a473e0e709f9b5bfea5 [^]
Related to issue 32268: Added some automated tests
|2016-02-17 19:51||maite||New Issue|
|2016-02-17 19:51||maite||Assigned To||=> Triage Finance|
|2016-02-17 19:51||maite||Resolution time||=> 1456268400|
|2016-02-17 19:51||maite||Issue Monitored: networkb|
|2016-02-17 19:52||maite||Steps to Reproduce Updated||View Revisions|
|2016-02-22 10:26||AtulOpenbravo||Assigned To||Triage Finance => AtulOpenbravo|
|2016-02-22 10:26||AtulOpenbravo||Status||new => scheduled|
|2016-02-22 10:39||AtulOpenbravo||Regression introduced by commit||=> https://code.openbravo.com/erp/devel/pi/rev/179d97ee602d [^]|
|2016-02-22 10:39||AtulOpenbravo||Regression level||=> Production - Confirmed Stable|
|2016-02-22 10:39||AtulOpenbravo||Resolution time||1456268400 => 1411423200|
|2016-02-22 10:41||AtulOpenbravo||Relationship added||caused by 0029390|
|2016-02-22 10:41||AtulOpenbravo||Status||scheduled => acknowledged|
|2016-02-22 10:45||vmromanos||Project||Modules => Openbravo ERP|
|2016-02-22 10:46||vmromanos||Modules||=> Core|
|2016-02-22 10:46||vmromanos||Triggers an Emergency Pack||=> No|
|2016-02-22 10:46||vmromanos||Category||Advanced Matching Algorithm => 09. Financial management|
|2016-02-22 10:46||AtulOpenbravo||Status||acknowledged => scheduled|
|2016-02-22 10:47||AtulOpenbravo||Resolution time||1411423200 =>|
|2016-02-22 10:47||AtulOpenbravo||Regression date||=> 2014-09-23|
|2016-02-22 10:52||AtulOpenbravo||Regression introduced in release||=> 3.0PR14Q4|
|2016-02-22 11:13||AtulOpenbravo||Note Added: 0084399|
|2016-02-22 14:08||psanjuan||Note Added: 0084405|
|2016-02-22 14:27||psanjuan||Note Edited: 0084399||View Revisions|
|2016-02-22 14:33||psanjuan||Note Edited: 0084399||View Revisions|
|2016-02-22 14:40||psanjuan||Note Edited: 0084399||View Revisions|
|2016-02-22 14:42||psanjuan||Note Added: 0084407|
|2016-02-22 16:45||vmromanos||Resolution time||=> 1456268400|
|2016-02-22 18:00||vmromanos||Note Added: 0084410|
|2016-02-23 14:00||hgbot||Note Added: 0084422|
|2016-02-23 14:00||hgbot||Status||scheduled => resolved|
|2016-02-23 14:00||hgbot||Resolution||open => fixed|
|2016-02-23 14:00||hgbot||Fixed in SCM revision||=> http://code.openbravo.com/erp/devel/pi/rev/143d5c8089260d63d8f645b73c5cf43bf1216828 [^]|
|2016-02-23 14:02||vmromanos||Review Assigned To||=> vmromanos|
|2016-02-23 14:02||vmromanos||Note Added: 0084429|
|2016-02-23 14:02||vmromanos||Status||resolved => closed|
|2016-03-03 18:53||aferraz||Note Added: 0084697|
|2016-03-17 10:56||hudsonbot||Note Added: 0085162|
|2016-04-08 11:00||hgbot||Note Added: 0085527|
|Copyright © 2000 - 2009 MantisBT Group|