Account - Financial Tools

Merge lp:~yann-papouin/account-financial-tools/7.0-bug-1255543-chart-update-records-wrong-dict into lp:~account-core-editors/account-financial-tools/7.0

  1. 7.0-bug-1255543-chart-update-records-wrong-dict
  2. Merge into 7.0
Proposed by Yann Papouin
Status: Merged
Merged at revision: 128
Proposed branch: lp:~yann-papouin/account-financial-tools/7.0-bug-1255543-chart-update-records-wrong-dict
Merge into: lp:~account-core-editors/account-financial-tools/7.0
Diff against target: 11 lines (+1/-1)
1 file modified
account_chart_update/wizard/wizard_chart_update.py (+1/-1)
To merge this branch: bzr merge lp:~yann-papouin/account-financial-tools/7.0-bug-1255543-chart-update-records-wrong-dict
Reviewer Review Type Date Requested Status
Joaquin Gutierrez (community) code review and test Approve
Pedro Manuel Baeza code review, no test Approve
Review via email: mp+196912@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

Thanks for catching up!

Regards.

review: Approve (code review, no test)
Revision history for this message
Joaquin Gutierrez (joaquing-pedrosa) wrote :

Hello Yann:

Good job.

Thank

review: Approve (code review and test)
Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

I proceed with the merge, having two reviews and one functional, because I have readed that this fix is needed for some tasks now on l10n_fr.

Regards.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'account_chart_update/wizard/wizard_chart_update.py'
2--- account_chart_update/wizard/wizard_chart_update.py 2013-10-18 16:44:19 +0000
3+++ account_chart_update/wizard/wizard_chart_update.py 2013-11-27 14:38:17 +0000
4@@ -1095,7 +1095,7 @@
5 taxes_res = self._update_taxes(
6 cr, uid, wizard, log, tax_codes_res['mapping'], context=context)
7 accounts_res = self._update_accounts(
8- cr, uid, wizard, log, taxes_res['pending'], context=context)
9+ cr, uid, wizard, log, taxes_res['mapping'], context=context)
10 self._update_taxes_pending_for_accounts(cr, uid, wizard, log, taxes_res['pending'], accounts_res['mapping'], context=context)
11 fps_res = self._update_fiscal_positions(cr, uid, wizard, log, taxes_res['mapping'], accounts_res['mapping'], context=context)
12

Subscribers

People subscribed via source and target branches