[FIX] l10n_it_riba: remove account and journal domain #4455
Conversation
SirAionTech
added
the
is porting
There was a problem hiding this comment.
Review tecnica e funzionale OK per il porting.
PS. Cambierei solo il titolo della PR perchè ero partito con l'idea di togliere le restrizioni del dominio anche per il registro di banca, ma poi non l'ho fatto e non ricordo di preciso il motivo.
Allora non l'ho cambiato, ma visto che questa è ancora aperta si può fare.
Funzionalmente invece lasciando la forzatura di un conto di tipo banca, la PR va bene se l’insoluto viene accreditato e poi stornato dalla banca altrimenti anche il registro dovrebbe essere libero.
| @@ -65,7 +65,7 @@ class RibaConfiguration(models.Model): | |||
| "account.account", | |||
| "A/C Bank Account", | |||
| check_company=True, | |||
| domain="[('company_id', '=', company_id), ('internal_type', '=', 'liquidity')]", | |||
| domain="[('company_id', '=', company_id)]", | |||
There was a problem hiding this comment.
Questo si può togliere: quando il dominio è solo per multicompany, l'attributo check_company inietta già il dominio corretto (vedi https://github.com/odoo/odoo/blob/d7f13b50148d0a3ec764dc37b0ca3631c9a35483/odoo/fields.py#L2541).
| @@ -79,7 +79,7 @@ class RibaConfiguration(models.Model): | |||
| "account.account", | |||
| "Past Due Bills Account", | |||
| check_company=True, | |||
| domain="[('company_id', '=', company_id), ('internal_type', '=', 'receivable')]", | |||
| domain="[('company_id', '=', company_id)]", | |||
There was a problem hiding this comment.
Anche questo si può togliere, vedi l'altro commento sul codice simile.
|
Dato che si tratta della v.14.0 sarebbe meglio modificare la descrizione del commit e della PR come già indicato qui. |
No description provided.