[IMP] account_ux: Add branch_order to re order journal given more priority to branch journals if you are setting from a branch #853
Conversation
|
|
There was a problem hiding this comment.
Pull request overview
Este PR implementa una restricción para evitar que los diarios de compra que utilizan documentos de Latinoamérica (l10n_latam_use_documents) sean compartidos con sucursales (branches), manteniendo la política actual para diarios de tipo general y de venta.
Cambios principales:
- Se agrega una advertencia en el método
_onchange_shared_to_branchescuando se intenta compartir un diario de compra con documentos latinoamericanos - Se actualiza la lógica del método compute
_compute_shared_to_branchespara excluir automáticamente estos diarios de compra del compartido predeterminado
account_ux/models/account_journal.py
Outdated
| @api.depends("type", "l10n_latam_use_documents") | ||
| def _compute_shared_to_branches(self): | ||
| shared = self.filtered(lambda j: j.type in ["general", "purchase"]) | ||
| # Journals of type general are shared, and purchase journals that do not use l10n_latam documents | ||
| has_latam_documents = "l10n_latam_use_documents" in self._fields | ||
| shared = self.filtered( | ||
| lambda j: j.type == "general" | ||
| or (j.type == "purchase" and (not has_latam_documents or not j.l10n_latam_use_documents)) | ||
| ) | ||
| shared.shared_to_branches = True | ||
| (self - shared).shared_to_branches = False |
There was a problem hiding this comment.
Este cambio modifica el comportamiento de un campo compute almacenado (shared_to_branches tiene store=True), por lo que se debe incrementar la versión del módulo en el __manifest__.py (de 19.0.1.3.0 a 19.0.1.4.0).
account_ux/models/account_journal.py
Outdated
| @api.depends("type", "l10n_latam_use_documents") | ||
| def _compute_shared_to_branches(self): | ||
| shared = self.filtered(lambda j: j.type in ["general", "purchase"]) | ||
| # Journals of type general are shared, and purchase journals that do not use l10n_latam documents | ||
| has_latam_documents = "l10n_latam_use_documents" in self._fields | ||
| shared = self.filtered( | ||
| lambda j: j.type == "general" | ||
| or (j.type == "purchase" and (not has_latam_documents or not j.l10n_latam_use_documents)) | ||
| ) | ||
| shared.shared_to_branches = True | ||
| (self - shared).shared_to_branches = False |
There was a problem hiding this comment.
Al modificar la lógica del método _compute_shared_to_branches (campo compute almacenado), es necesario crear un script de migración post en migrations/19.0.1.4.0/post_recompute_shared_to_branches.py que recalcule el campo shared_to_branches para todos los diarios de tipo "purchase" existentes, especialmente aquellos que tienen l10n_latam_use_documents=True, ya que ahora deberían tener shared_to_branches=False. El script debe procesar los registros en lotes para evitar problemas de rendimiento.
account_ux/models/account_journal.py
Outdated
| } | ||
|
|
||
| @api.depends("type") | ||
| @api.depends("type", "l10n_latam_use_documents") |
There was a problem hiding this comment.
El decorador @api.depends incluye l10n_latam_use_documents, pero este campo puede no existir si el módulo de localización latinoamericana no está instalado. En Odoo, los decoradores @api.depends con campos inexistentes pueden causar warnings o errores. Considera usar una verificación condicional para agregar esta dependencia solo si el campo existe, o documentar que este módulo requiere que el campo esté siempre presente cuando se usa la funcionalidad de sucursales.
| @api.depends("type", "l10n_latam_use_documents") | |
| @api.depends("type") |
rov-adhoc
force-pushed
the
19.0-t-60604-rov
branch
5 times, most recently
from
8251691 to
074b718
Compare
rov-adhoc
changed the title
rov-adhoc
force-pushed
the
19.0-t-60604-rov
branch
from
074b718 to
709bc4c
Compare
…ority to branch journals if you are setting from a branch
rov-adhoc
force-pushed
the
19.0-t-60604-rov
branch
from
709bc4c to
312ab75
Compare
3 participants
No description provided.