[IMP] _credit_limit: compatibility with account_multicompany_ux #1524
Conversation
No es hermoso depender de ese módulo pero a "account_multicompany_ux" lo tenemos en todos donde tenemos este otro
|
|
There was a problem hiding this comment.
Pull request overview
Este PR mejora la compatibilidad del módulo sale_exception_credit_limit con el módulo account_multicompany_ux, el cual aparentemente redefine la jerarquía de vistas de propiedades del partner. Los cambios adaptan la herencia de vistas y controlan la visibilidad de elementos relacionados con la configuración multicompañía del límite de crédito.
- Cambio de la vista padre de
account.view_partner_property_formaproduct.view_partner_property_formpara alinearse con la estructura deaccount_multicompany_ux - Adición de control de visibilidad al botón de propiedades multicompañía basado en permisos de usuario
- Bump de versión a 18.0.1.2.0 y declaración explícita de dependencia con
account_multicompany_ux
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| sale_exception_credit_limit/manifest.py | Bump de versión a 18.0.1.2.0 y adición de dependencia account_multicompany_ux |
| sale_exception_credit_limit/views/res_partner_views.xml | Cambio de vista heredada de account a product y adición de xpath para controlar visibilidad del botón de propiedades multicompañía |
| <attribute name="invisible">not use_partner_credit_limit</attribute> | ||
| <attribute name="readonly">not user_credit_config</attribute> | ||
| </field> | ||
|
|
There was a problem hiding this comment.
El selector xpath //div[@name='property_credit_limit_ids']/button[@name='action_company_properties'] es muy específico y depende de la estructura interna del módulo account_multicompany_ux. Si ese módulo cambia la estructura de su vista, este xpath podría fallar silenciosamente.
Considera agregar un comentario explicando qué elemento se está modificando y de qué módulo proviene, para facilitar el mantenimiento futuro. Por ejemplo: <!-- Botón de propiedades multicompañía proporcionado por account_multicompany_ux -->
| <!-- Botón de propiedades multicompañía proporcionado por account_multicompany_ux --> |
|
@roboadhoc r+ |
|
@roboadhoc r- |
2 participants
No es hermoso depender de ese módulo pero a "account_multicompany_ux" lo tenemos en todos donde tenemos este otro