[ADD] l10n_ar_edi_ux: required fields #875
Conversation
only require document type and number when validating the invoice
|
|
There was a problem hiding this comment.
Pull request overview
Este PR modifica la vista de facturas argentinas para remover el atributo required de los campos l10n_latam_document_number y l10n_latam_document_type_id, permitiendo que estos campos no sean obligatorios hasta el momento de validar la factura.
Changes:
- Se modifica la vista heredada
view_move_formpara eliminar la obligatoriedad de los campos de tipo y número de documento - Se incrementa la versión del módulo de 19.0.1.1.0 a 19.0.1.3.0
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| l10n_ar_edi_ux/views/account_move_view.xml | Agrega modificaciones de atributos para remover el required de campos de documento |
| l10n_ar_edi_ux/manifest.py | Incrementa la versión del módulo |
| <button name="check_valid_boarding_permission" string="Validar permisos de embarque" type="object" invisible="l10n_ar_afip_ws != 'wsfex'" icon="fa-arrow-right" class="oe_link" colspan="2"/> | ||
| </field> | ||
|
|
||
| <!-- Solo hacemos requeridos estos campo cuando validamos la factura, no antes--> |
There was a problem hiding this comment.
Error de tipeo: 'estos campo' debería ser 'estos campos' (plural).
| <!-- Solo hacemos requeridos estos campo cuando validamos la factura, no antes--> | |
| <!-- Solo hacemos requeridos estos campos cuando validamos la factura, no antes--> |
| <attribute name="required"/> | ||
| </field> | ||
| <field name="l10n_latam_document_type_id" position="attributes"> | ||
| <attribute name="required"/> |
There was a problem hiding this comment.
El uso de <attribute name='required'/> sin valor elimina el atributo required de la vista padre, haciendo que los campos NO sean obligatorios. Sin embargo, el comentario en la línea 18 dice 'Solo hacemos requeridos estos campo cuando validamos la factura', lo cual es contradictorio. Si la intención es remover la obligatoriedad inicial (como sugiere la descripción del PR), el comentario debería aclarar que se están haciendo NO obligatorios en la vista pero se validarán programáticamente al validar la factura. Si la intención es hacerlos obligatorios, debe usarse una sintaxis como <attribute name='required'>1</attribute> o agregar un atributo states para condicionar la obligatoriedad.
| { | ||
| "name": "Argentinian Electronic Invoicing UX", | ||
| "version": "19.0.1.1.0", | ||
| "version": "19.0.1.3.0", |
There was a problem hiding this comment.
El bump de versión de 19.0.1.1.0 a 19.0.1.3.0 salta la versión 19.0.1.2.0. Si bien técnicamente válido, se recomienda incrementar de forma secuencial (debería ser 19.0.1.2.0) para mantener un versionado coherente, a menos que 19.0.1.2.0 ya exista en otra rama.
| <field name="l10n_latam_document_number" position="attributes"> | ||
| <attribute name="required"/> | ||
| </field> | ||
| <field name="l10n_latam_document_type_id" position="attributes"> | ||
| <attribute name="required"/> | ||
| </field> |
There was a problem hiding this comment.
Si la intención es remover la obligatoriedad visual pero mantener la validación al momento de confirmar la factura, falta la lógica en el modelo (constraint o validación en método _post o similar) que garantice que estos campos estén completados al validar. Revisa si existe esta validación en el modelo account.move heredado; si no existe, este cambio podría permitir validar facturas sin tipo o número de documento.
2 participants
only require document type and number when validating the invoice