diff options
| author | Victor Feyens <vfe@odoo.com> | 2020-09-15 13:55:09 +0000 |
|---|---|---|
| committer | Victor Feyens <vfe@odoo.com> | 2020-10-22 13:10:54 +0000 |
| commit | 037012bc4e2935eb00a7f353529bb7ec55066733 (patch) | |
| tree | 3aa2efc3cfb905e9c15d382e9751cd15bdacc2db /addons/l10n_ar/models/account_chart_template.py | |
| parent | 4ff9ff09193390d9b39b15046855f94a98ad8409 (diff) | |
[FIX] *: avoid direct references to countries
Ensure the code works fine even if countries are deleted.
Also consider "re"-created countries after deletion, by only considering
the country code, not the data reference.
This should reduce support requests related to deleted countries, and
ease the resolution of such problems by the users themselves.
TASK ID - 2368842
closes odoo/odoo#57837
Related: odoo/enterprise#13275
Signed-off-by: Victor Feyens (vfe) <vfe@odoo.com>
Diffstat (limited to 'addons/l10n_ar/models/account_chart_template.py')
| -rw-r--r-- | addons/l10n_ar/models/account_chart_template.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/addons/l10n_ar/models/account_chart_template.py b/addons/l10n_ar/models/account_chart_template.py index 8ce14066993..a53709251a6 100644 --- a/addons/l10n_ar/models/account_chart_template.py +++ b/addons/l10n_ar/models/account_chart_template.py @@ -11,7 +11,7 @@ class AccountChartTemplate(models.Model): def _get_fp_vals(self, company, position): res = super()._get_fp_vals(company, position) - if company.country_id == self.env.ref('base.ar'): + if company.country_id.code == "AR": res['l10n_ar_afip_responsibility_type_ids'] = [ (6, False, position.l10n_ar_afip_responsibility_type_ids.ids)] return res @@ -20,7 +20,7 @@ class AccountChartTemplate(models.Model): """ If Argentinian chart, we modify the defaults values of sales journal to be a preprinted journal """ res = super()._prepare_all_journals(acc_template_ref, company, journals_dict=journals_dict) - if company.country_id == self.env.ref('base.ar'): + if company.country_id.code == "AR": for vals in res: if vals['type'] == 'sale': vals.update({ @@ -58,7 +58,7 @@ class AccountChartTemplate(models.Model): company_responsibility = company.l10n_ar_afip_responsibility_type_id company.write({ 'l10n_ar_afip_responsibility_type_id': coa_responsibility.id, - 'country_id': self.env.ref('base.ar').id, + 'country_id': self.env['res.country'].search([('code', '=', 'AR')]).id, 'tax_calculation_rounding_method': 'round_globally', }) # set CUIT identification type (which is the argentinian vat) in the created company partner instead of |