6.0 reconciliation doesn't work
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Banking Addons |
Fix Released
|
Critical
|
Stefan Rijnhart (Opener) |
Bug Description
I am not sure if you are interested to fix version 6.0. (In 6.1 I see it is different concept)
I get with my own developed parser that import bank statement wizard recognises remote_account and partner and even says that matched statement line. I see that invoice account move is marked with created reconciliation ID. But this reconciliation ID doesn't appear on bank statement line and doesn't create full reconciliation after confirming the bank statement.
I have changed some lines in bank_import.py wizard and it started to work.
Generally I added reconciliation ID directly from create method because it cannot be taken later until whole wizard finishes.
Exact line numbers can differ as I added some log entries to analyse the code. I have changed lines with comment # GG. Another change is at the end.
around line 79:
if partial:
)
else:
if move_line.
else:
}
)
return retval
Another change around line 834 (again one change marked with # GG)
if move_info:
Related branches
- Grzegorz Grzelak (OpenGLOBE.pl): Pending (test) requested
- Dimitri John Ledkov (ex-credativ): Pending requested
-
Diff: 48 lines (+22/-0)3 files modifiedaccount_banking/__init__.py (+1/-0)
account_banking/browse_record.py (+20/-0)
account_banking/wizard/bank_import.py (+1/-0)
Changed in banking-addons: | |
status: | New → Confirmed |
importance: | Undecided → Critical |
assignee: | nobody → Stefan Rijnhart (Therp) (stefan-therp) |
Changed in banking-addons: | |
status: | Fix Committed → Fix Released |
Hi Gregorz,
thank you for reporting this problem and suggesting a fix! Does this problem occur with every matched transaction? In general, reconciliation of matched transaction is known to work well, except that your modification is in a recently modified part of the code. It may be this recent modification that is at fault (http:// bazaar. launchpad. net/~banking- addons- team/banking- addons/ 6.0/revision/ 71).
Looking at your code, it registers the id of the generated reconciliation on the move_info struct, separately from the move line. The OpenERP reconciliation logic dictates that the id of the reconciliation should also be available from the move_line itself, like the original code retrieves it. However, we are aware of caching issues with browse objects such as the move_line in this piece of the code. We will need to refresh the browse record after having effectively written its reconcile_id by creating the reconciliation object.
Thanks again,
Stefan.