Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0020262Openbravo ERP09. Financial managementpublic2012-04-13 09:502012-05-29 03:27
psarobe 
Sandrahuguet 
normalminoralways
closedfixed 
5
pi 
3.0MP12 
Core
No
0020262: Post a reconciliation fails when exists a G/L Item+Invoice and posting is disabled for reconciliation
Configuration to reproduce the issue:
1. Reconciliation must be disabled for accounting for the payment method of the invoice
2. But in the Accounting configuration of the financial account must be "enabled"

See attach configuration_bank1

With this configuration if you reconcile a purchase invoice and a G/L Item then the system does strange things.

See second attach
1. Go to financial account
2. Create a transactions that contains a purchase invoice
3. Create a transaction that contains a G/L Item
4. Create a bankstatement (manually) that contains two lines similar to the above transactions
5. Matching both transactions
6. Reconcile and post. See attach reconciliation error

VMA-Reviewed
jpg Configuration_bank1.JPG (116,067) 2012-05-17 18:02
https://issues.openbravo.com/file_download.php?file_id=5239&type=bug
jpg

jpg Reconciliaton_error.JPG (141,837) 2012-05-17 18:02
https://issues.openbravo.com/file_download.php?file_id=5240&type=bug
jpg
Issue History
2012-04-13 09:50psarobeNew Issue
2012-04-13 09:50psarobeAssigned To => vmromanos
2012-04-13 09:50psarobeModules => Core
2012-04-13 10:01vmromanosTag Attached: VMA-Reviewed
2012-05-10 12:12gorka_gilAssigned Tovmromanos => mirurita
2012-05-17 16:05SandrahuguetStatusnew => scheduled
2012-05-17 16:05SandrahuguetAssigned Tomirurita => Sandrahuguet
2012-05-17 16:05Sandrahuguetfix_in_branch => pi
2012-05-17 18:02psarobeFile Added: Configuration_bank1.JPG
2012-05-17 18:02psarobeFile Added: Reconciliaton_error.JPG
2012-05-17 18:09psarobefix_in_branchpi =>
2012-05-17 18:09psarobeDescription Updatedbug_revision_view_page.php?rev_id=3479#r3479
2012-05-17 18:09psarobeSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=3481#r3481
2012-05-17 18:09psarobeSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=3482#r3482
2012-05-17 18:10psarobeDescription Updatedbug_revision_view_page.php?rev_id=3483#r3483
2012-05-21 09:12SandrahuguetNote Added: 0048791
2012-05-28 10:37hgbotCheckin
2012-05-28 10:37hgbotNote Added: 0049342
2012-05-28 10:37hgbotStatusscheduled => resolved
2012-05-28 10:37hgbotResolutionopen => fixed
2012-05-28 10:37hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/d7397900046854bcaa432ded37119a022fc87311 [^]
2012-05-28 10:37hgbotCheckin
2012-05-28 10:37hgbotNote Added: 0049343
2012-05-28 10:56miruritaNote Added: 0049344
2012-05-28 10:56miruritaStatusresolved => closed
2012-05-28 10:56miruritaFixed in Version => 3.0MP12
2012-05-29 03:27hudsonbotCheckin
2012-05-29 03:27hudsonbotNote Added: 0049386
2012-05-29 03:27hudsonbotCheckin
2012-05-29 03:27hudsonbotNote Added: 0049387

Notes
(0048791)
Sandrahuguet   
2012-05-21 09:12   
Test Plan:
1. Go to financial account and select bank-account 1
   -Reconciliation must be disabled for accounting for the payment method of the invoice
   -But in the Accounting configuration of the financial account must be "enabled" (In Payment Out, Cleared Payment Account: F&B International Group-1110)
   -Matching Algorithm: Standard
2. Click Add Transacction button and create a transactions that contains a purchase invoice
3. Click Add Transacction button and create a transaction that contains a G/L Item
4. Create a bankstatement (manually) that contains two lines similar to the above transactions
5. Matching both transactions
6. Reconcile and post. Notice that you can reconcile both transactions
(0049342)
hgbot   
2012-05-28 10:37   
Repository: erp/devel/pi
Changeset: d7397900046854bcaa432ded37119a022fc87311
Author: Sandra Huguet <sandra.huguet <at> openbravo.com>
Date: Mon May 21 09:43:34 2012 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/d7397900046854bcaa432ded37119a022fc87311 [^]

Fixed bug 20262 Reconcile transactions disabled and enabled for accounting
Add logic in DocFINReconciliation.java to reconcile transactions
disabled and enabled for accounting

---
M src/org/openbravo/erpCommon/ad_forms/DocFINReconciliation.java
---
(0049343)
hgbot   
2012-05-28 10:37   
Repository: erp/devel/pi
Changeset: c49d26d67c207dbcf43f2095f8fb555f3e5433d3
Author: Mikel Irurita <mikel.irurita <at> openbravo.com>
Date: Fri May 25 14:47:07 2012 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/c49d26d67c207dbcf43f2095f8fb555f3e5433d3 [^]

Related to issue 20262: remove unused code

---
M src/org/openbravo/erpCommon/ad_forms/DocFINReconciliation.java
---
(0049344)
mirurita   
2012-05-28 10:56   
code review + testing OK

Testing Plan
1) Create a new payment method for "Bank - Account 2"
  - Name: Paypal
  - Upon Reconciliation Use Out: empty
2) Create a new payment method for "Bank - Account 2"
  - Name: Wire Transfer2
  - Upon Reconciliation Use Out: Cleared Payment Account
3) In Accounting Configuration tab, set a value for Cleared Payment Account (OUT)
4) Create a Purchase Invoice using Paypal
5) Create other Purchase Invoice using Wire Transfer 2
6) Create manually a new bank statement with two lines (process it)
  - Line 1 Amount Out: 10
  - Line 2 Amount Out: 20
7) Click on Match Statement button
8) Match line 1 with purchase invoice of step 4). Use greed plus icon.
9) Match line 2 with purchase invoice of step 5). Use greed plus icon.
10) Click on Reconcile button
11) Go to the reconciliation and post:
Only the purchase invoice of Wire Transfer 2 (step 5) should be posted.
12) Unpost the reconciliation
13) For Wire Transfer 2, remove the value on "upon Reconciliation Use Out"
14) Post the reconciliation and you will get: account disables for accounting
(0049386)
hudsonbot   
2012-05-29 03:27   
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/ced2fdfb9aee [^]

Maturity status: Test
(0049387)
hudsonbot   
2012-05-29 03:27   
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/ced2fdfb9aee [^]

Maturity status: Test