Merge lp:~bruno-bottacini/openobject-italia/6.1-fix-bug-1167773 into lp:~openobject-italia-core-devs/openobject-italia/italian-addons-6.1

Proposed by bruno bottacini
Status: Needs review
Proposed branch: lp:~bruno-bottacini/openobject-italia/6.1-fix-bug-1167773
Merge into: lp:~openobject-italia-core-devs/openobject-italia/italian-addons-6.1
Diff against target: 25 lines (+8/-1)
1 file modified
l10n_it_ricevute_bancarie/wizard/wizard_unsolved.py (+8/-1)
To merge this branch: bzr merge lp:~bruno-bottacini/openobject-italia/6.1-fix-bug-1167773
Reviewer Review Type Date Requested Status
Lorenzo Battistini Needs Fixing
Review via email: mp+158863@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Lorenzo Battistini (elbati) wrote :

Nella 'acceptance_move_id' cercherei anche lì per conto 'effects_account_id'.
Cercando per debit > 0, se per qualunque motivo l'utente avesse usato un conto diverso o avesse aggiunto delle move.line, dentro 'to_be_reconciled' andrebbero a finire cose sbagliate

review: Needs Fixing
Revision history for this message
Lorenzo Battistini (elbati) wrote :

This project is now hosted on https://github.com/OCA/l10n-italy. Please move your proposal there. This guide may help you https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub

Revision history for this message
Lorenzo Battistini (elbati) wrote :

Unmerged revisions

231. By bruno bottacini

Bug #1167773

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'l10n_it_ricevute_bancarie/wizard/wizard_unsolved.py'
2--- l10n_it_ricevute_bancarie/wizard/wizard_unsolved.py 2013-01-04 14:02:10 +0000
3+++ l10n_it_ricevute_bancarie/wizard/wizard_unsolved.py 2013-04-15 08:47:31 +0000
4@@ -151,13 +151,20 @@
5 ]
6 }
7 move_id = move_pool.create(cr, uid, move_vals, context=context)
8-
9+
10+ to_be_reconciled = []
11 for move_line in move_pool.browse(cr, uid, move_id, context=context).line_id:
12 if move_line.account_id.id == wizard.overdue_effects_account_id.id:
13 for riba_move_line in distinta_line.move_line_ids:
14 invoice_pool.write(cr, uid, riba_move_line.move_line_id.invoice.id, {
15 'unsolved_move_line_ids': [(4, move_line.id)],
16 }, context=context)
17+ if move_line.account_id.id == wizard.effects_account_id.id:
18+ to_be_reconciled.append(move_line.id)
19+ for acceptance_move_line in distinta_line.acceptance_move_id.line_id:
20+ if acceptance_move_line.debit > 0.0:
21+ to_be_reconciled.append(acceptance_move_line.id)
22+ move_line_pool.reconcile_partial(cr, uid, to_be_reconciled, context=context)
23
24 distinta_line.write({
25 'unsolved_move_id': move_id,

Subscribers

People subscribed via source and target branches