diff --git a/addons/l10n_ar/models/account_journal.py b/addons/l10n_ar/models/account_journal.py index d8f501e5db6..25ffcae5c45 100644 --- a/addons/l10n_ar/models/account_journal.py +++ b/addons/l10n_ar/models/account_journal.py @@ -167,11 +167,15 @@ class AccountJournal(models.Model): @api.constrains('l10n_ar_afip_pos_number') def _check_afip_pos_number(self): - missing_pos_number = self.filtered( - lambda x: x.type == 'sale' and x.l10n_latam_use_documents and x.company_id.country_id == self.env.ref('base.ar') - and x.l10n_ar_afip_pos_number == 0) - if missing_pos_number: - raise ValidationError(_('Please define a valid AFIP POS number')) + to_review = self.filtered( + lambda x: x.type == 'sale' and x.l10n_latam_use_documents and + x.company_id.country_id == self.env.ref('base.ar')) + + if to_review.filtered(lambda x: x.l10n_ar_afip_pos_number == 0): + raise ValidationError(_('Please define an AFIP POS number')) + + if to_review.filtered(lambda x: x.l10n_ar_afip_pos_number > 99999): + raise ValidationError(_('Please define a valid AFIP POS number (5 digits max)')) @api.onchange('l10n_ar_afip_pos_system') def _onchange_l10n_ar_afip_pos_system(self):