Merge lp:~rr.clearcorp/openerp-ccorp-addons/6.1-account_move_reverse into lp:openerp-ccorp-addons/6.1

Proposed by Ronald Rubi
Status: Merged
Merged at revision: 751
Proposed branch: lp:~rr.clearcorp/openerp-ccorp-addons/6.1-account_move_reverse
Merge into: lp:openerp-ccorp-addons/6.1
Diff against target: 42 lines (+13/-4)
1 file modified
account_move_reverse/account_move_reverse.py (+13/-4)
To merge this branch: bzr merge lp:~rr.clearcorp/openerp-ccorp-addons/6.1-account_move_reverse
Reviewer Review Type Date Requested Status
ClearCorp drivers Pending
Review via email: mp+193500@code.launchpad.net

Description of the change

[FIX] Fix account_move_reverse, bad matching lines

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
=== modified file 'account_move_reverse/account_move_reverse.py'
--- account_move_reverse/account_move_reverse.py 2013-10-21 16:54:53 +0000
+++ account_move_reverse/account_move_reverse.py 2013-10-31 21:17:26 +0000
@@ -61,6 +61,7 @@
61 def reverse(self, cr, uid, ids, context=None):61 def reverse(self, cr, uid, ids, context=None):
6262
63 for move_original_id in ids:63 for move_original_id in ids:
64 new_reconcile_line_ids = []
64 move_original = self.pool.get('account.move').browse(cr, 1, move_original_id, context=context)65 move_original = self.pool.get('account.move').browse(cr, 1, move_original_id, context=context)
65 66
66 if move_original.move_reverse_id:67 if move_original.move_reverse_id:
@@ -110,9 +111,15 @@
110 if len(reconcile.line_id) > 2:111 if len(reconcile.line_id) > 2:
111 reconcile_line_ids = []112 reconcile_line_ids = []
112 for line_id in reconcile.line_id:113 for line_id in reconcile.line_id:
113 reconcile_line_ids.append(line_id.id)114 if line_id.id not in new_reconcile_line_ids:
114 self.pool.get('account.move.line').write(cr,uid,reconcile_line_ids,{'reconcile_id': False, 'reconcile_partial_id':reconcile.id})115 reconcile_line_ids.append(line_id.id)
115 self.pool.get('account.move.line').write(cr,uid,line.id,{'reconcile_partial_id': False})116 if len(reconcile_line_ids) > 2:
117 self.pool.get('account.move.line').write(cr,uid,reconcile_line_ids,{'reconcile_id': False, 'reconcile_partial_id':reconcile.id})
118 self.pool.get('account.move.line').write(cr,uid,line.id,{'reconcile_partial_id': False})
119 else:
120 move_reconcile_obj.unlink(cr,uid,[reconcile.id],context=context)
121 new_reconcile_line_ids.append(line.id) #Workaround, commit database
122
116 else:123 else:
117 move_reconcile_obj.unlink(cr,uid,[reconcile.id],context=context)124 move_reconcile_obj.unlink(cr,uid,[reconcile.id],context=context)
118125
@@ -125,9 +132,11 @@
125132
126 if line.account_id.reconcile:133 if line.account_id.reconcile:
127 reconcile_id = self.pool.get('account.move.reconcile').create(cr, uid, {'type': 'Account Reverse'})134 reconcile_id = self.pool.get('account.move.reconcile').create(cr, uid, {'type': 'Account Reverse'})
135
136 #The move don't support write this line
128 cr.execute('UPDATE account_move_line '\137 cr.execute('UPDATE account_move_line '\
129 'SET reconcile_id=%s '\138 'SET reconcile_id=%s '\
130 'WHERE id IN %s', (reconcile_id, tuple([line.id]),))139 'WHERE id IN %s', (reconcile_id, tuple([line.id]),))
131 #self.pool.get('account.move.line').write(cr,uid,[line.id],{'reconcile_id': reconcile_id})140 #self.pool.get('account.move.line').write(cr,uid,[line.id],{'reconcile_id': reconcile_id})
132 self.pool.get('account.move.line').write(cr,uid,[line_created_id],{'reconcile_id': reconcile_id})141 self.pool.get('account.move.line').write(cr,uid,[line_created_id],{'reconcile_id': reconcile_id})
133142

Subscribers

People subscribed via source and target branches