Yaltik/account-financial-tools
2
0
Fork 0
Code Pull Requests Activity

[FIX] account_chart_update: Don't write code on account update

Browse Source 
As there's a constraint that forbids to write the code of the account
if it has moves, we delete the corresponding entry on update only,
because the method that prepares the values is the same for new and
existing accounts.
This commit is contained in:
Pedro M. Baeza 2016-01-27 20:38:30 +01:00 committed by Luis J. Salvatierra
parent 7d35cd2705
commit a560d62acd
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
account_chart_update/wizard/wizard_chart_update.py
View File

@ -1026,12 +1026,14 @@ class WizardUpdateChartsAccounts(models.TransientModel):
else: else:
# Update the account # Update the account
account = wiz_account.update_account_id account = wiz_account.update_account_id
# Don't write again the same code - it may give an error
code = vals.pop('code')
try: try:
account.write(vals) account.write(vals)
log.add(_("Updated account %s.\n") % vals['code']) log.add(_("Updated account %s.\n") % code)
except (exceptions.Warning, except_orm, except_osv) as ex: except (exceptions.Warning, except_orm, except_osv) as ex:
log.add(_("Exception writing account %s: %s - %s.\n") % log.add(_("Exception writing account %s: %s - %s.\n") %
(vals['code'], ex.name, ex.value), True) (code, ex.name, ex.value), True)
# Set this account as the parent of the accounts that seem to # Set this account as the parent of the accounts that seem to
# be its children (brothers starting with the same code). # be its children (brothers starting with the same code).
if self.update_children_accounts_parent: if self.update_children_accounts_parent: