Merge lp:~camptocamp/banking-addons/already_delete_reconcile_id_resubmit into lp:banking-addons/bank-statement-reconcile-70
Proposed by
Vincent Renaville@camptocamp
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Guewen Baconnier @ Camptocamp | ||||
Approved revision: | 103 | ||||
Merged at revision: | 108 | ||||
Proposed branch: | lp:~camptocamp/banking-addons/already_delete_reconcile_id_resubmit | ||||
Merge into: | lp:banking-addons/bank-statement-reconcile-70 | ||||
Diff against target: |
16 lines (+4/-1) 1 file modified
account_statement_ext/account.py (+4/-1) |
||||
To merge this branch: | bzr merge lp:~camptocamp/banking-addons/already_delete_reconcile_id_resubmit | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Guewen Baconnier @ Camptocamp | code review | Approve | |
Omar (Pexego) | Approve | ||
Leonardo Pistone | Approve | ||
Joël Grand-Guillaume @ camptocamp | code review, no tests | Needs Fixing | |
Review via email: mp+194679@code.launchpad.net |
This proposal supersedes a proposal from 2013-07-26.
Description of the change
[FIX] remove account.reconcile before deleting a account.move
To post a comment you must log in.
I think a more efficient and straightforward way would be something along those lines:
[...] get('account. move.reconcile' ) reconcile_ id:
reconcile_ ids.add( move_line. reconcile_ id.id) obj.unlink( cr, uid, list(reconcile_ ids), context=context)
reconcile_obj = self.pool.
reconcile_ids = set()
for move_line in move.line_id:
if move_line.
reconcile_
return [...]
So you'll do only 1 DELETE at the end of the loop.