Merge lp:~vauxoo/addons-vauxoo/7.0-addons-vauxoo-her-cancel-dev-julio into lp:addons-vauxoo/7.0

Proposed by Julio Serna-http://www.vauxoo.com
Status: Merged
Merged at revision: 1016
Proposed branch: lp:~vauxoo/addons-vauxoo/7.0-addons-vauxoo-her-cancel-dev-julio
Merge into: lp:addons-vauxoo/7.0
Diff against target: 30 lines (+12/-4)
1 file modified
hr_expense_replenishment/model/hr_expense.py (+12/-4)
To merge this branch: bzr merge lp:~vauxoo/addons-vauxoo/7.0-addons-vauxoo-her-cancel-dev-julio
Reviewer Review Type Date Requested Status
Moisés López - http://www.vauxoo.com Pending
Julio Serna-http://www.vauxoo.com Pending
Vauxoo Pending
Review via email: mp+219604@code.launchpad.net

Description of the change

Modified function cancel of expense, changed _remove_move_reconcile by unlink of ids of concilation partial or full

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'hr_expense_replenishment/model/hr_expense.py'
2--- hr_expense_replenishment/model/hr_expense.py 2014-05-09 21:51:34 +0000
3+++ hr_expense_replenishment/model/hr_expense.py 2014-05-14 21:33:13 +0000
4@@ -34,14 +34,22 @@
5 def expense_canceled(self, cr, uid, ids, context=None):
6 obj_move_line = self.pool.get('account.move.line')
7 obj_move = self.pool.get('account.move')
8+ obj_move_rec = self.pool.get('account.move.reconcile')
9+
10 res = super(hr_expense_expense,
11 self).expense_canceled(cr, uid, ids, context=context)
12 for expense in self.browse(cr, uid, ids, context=context):
13 if expense.account_move_id:
14- obj_move_line._remove_move_reconcile(cr, uid,
15- [move_line.id
16- for move_line in expense.account_move_id.line_id],
17- context=context)
18+ reconcile_id = [move_line.reconcile_id.id
19+ for move_line in expense.account_move_id.line_id
20+ if move_line.reconcile_id]
21+ reconcile_partial_id = [move_line.reconcile_partial_id.id
22+ for move_line in expense.account_move_id.line_id
23+ if move_line.reconcile_partial_id]
24+ recs = reconcile_id + reconcile_partial_id
25+ all_moves_recs = list( set(recs) )
26+ if all_moves_recs:
27+ obj_move_rec.unlink(cr, uid, all_moves_recs)
28 obj_move.unlink(cr, uid, [expense.account_move_id.id],
29 context=context)
30 return res