[FIX] l10n_ar_stock_ux: fix + minor improvements#277
Merged
jjscarafia merged 3 commits intoingadhoc:19.0from Dec 16, 2025
Merged
[FIX] l10n_ar_stock_ux: fix + minor improvements#277jjscarafia merged 3 commits intoingadhoc:19.0from
jjscarafia merged 3 commits intoingadhoc:19.0from
Conversation
1. arreglamos el if de calculo de barcode 2. pasamos a depender de declared value. Lo definimos así porque: a) si bien no es obligatorio el dato, por uso y costumbre se usa un montón b) además que si no para mandar un valor a COT deberíamos hacer un chino c) eventualmente el declared value podría estar como campo en stock_ux y todo el cálculo automático en módulo separado (y extra?)
Contributor
|
|
Contributor
There was a problem hiding this comment.
Pull request overview
Este PR corrige errores y realiza mejoras menores en el módulo l10n_ar_stock_ux relacionado con la generación de códigos COT (Código de Operaciones de Traslado) de ARBA/Santa Fe para remitos argentinos.
- Corrección del cálculo del código de barras AFIP eliminando una línea que sobrescribía incorrectamente el valor de
pos_number - Integración con el campo
declared_valuedel módulostock_declared_valuepara prellenar el importe en el wizard de COT - Simplificación de condiciones en templates de reportes eliminando verificaciones redundantes de existencia de campos
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| l10n_ar_stock_ux/manifest.py | Agrega dependencias a stock_ux y stock_declared_value para soportar el campo declared_value |
| l10n_ar_stock_ux/models/stock_picking.py | Corrige bug en cálculo de barcode eliminando línea que sobrescribía pos_number y simplifica condiciones del if |
| l10n_ar_stock_ux/wizards/arba_cot_wizard.py | Elimina comentario TODO obsoleto sobre implementar asistente de importe |
| l10n_ar_stock_ux/views/stock_picking_views.xml | Actualiza botón COT removiendo "(BETA)" y agrega context para prellenar importe con declared_value |
| l10n_ar_stock_ux/views/report_deliveryslip.xml | Simplifica condiciones en template eliminando verificaciones redundantes con _fields.get() |
| l10n_ar_stock_delivery/views/report_deliveryslip.xml | Simplifica condiciones en template eliminando verificaciones redundantes con _fields.get() |
| # esta dependencia es solo por el campo declared value para: | ||
| # a) reporte de remito | ||
| # b) mandar valor declarado a wizard de COT | ||
| # eventualmente se podría moer dicho campo a stock_ux y evitar esta dependencia |
There was a problem hiding this comment.
Error de ortografía en el comentario: "moer" debería ser "mover".
Suggested change
| # eventualmente se podría moer dicho campo a stock_ux y evitar esta dependencia | |
| # eventualmente se podría mover dicho campo a stock_ux y evitar esta dependencia |
jjscarafia
force-pushed
the
19.0-t-56799-jjs
branch
from
cac18fb to
226609d
Compare
Member
Author
|
@roboadhoc r+ |
Contributor
|
@jjscarafia because this PR has multiple commits, I need to know how to merge it:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
arreglamos el if de calculo de barcode
pasamos a depender de declared value. Lo definimos así porque: a) si bien no es obligatorio el dato, por uso y costumbre se usa un montón b) además que si no para mandar un valor a COT deberíamos hacer un chino c) eventualmente el declared value podría estar como campo en stock_ux y todo el cálculo automático en módulo separado (y extra?)
en el commit 3 se ve mucho diff pero es solo arreglar indentación