[FIX]stock_ux: origin description + product name #856
Conversation
|
|
There was a problem hiding this comment.
Pull request overview
Este PR añade una nueva opción de configuración que permite incluir el nombre del producto junto con la descripción de origen en los comprobantes de transferencia, complementando la funcionalidad existente delivery_slip_use_origin.
Changes:
- Se añade un nuevo campo booleano de configuración
delivery_slip_add_product_nameenres.config.settings - Se modifica la lógica de
_get_aggregated_propertiespara combinar el nombre del producto con la descripción limpia cuando la nueva opción está activa - Se incrementa la versión del módulo de 19.0.1.1.0 a 19.0.1.2.0
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| stock_ux/manifest.py | Incrementa la versión del módulo de 19.0.1.1.0 a 19.0.1.2.0 para reflejar la nueva funcionalidad |
| stock_ux/models/res_config_settings.py | Agrega el campo de configuración delivery_slip_add_product_name con su parámetro correspondiente |
| stock_ux/views/res_config_settings_views.xml | Añade el campo de configuración en la vista, anidado condicionalmente bajo delivery_slip_use_origin |
| stock_ux/models/stock_move_line.py | Implementa la lógica para combinar nombre de producto y descripción limpia cuando ambas opciones están activas |
|
|
||
| name = clean_description if clean_description else origin_description | ||
| if add_product_name and clean_description and clean_description != origin_description: | ||
| name = f"{product.name} {'-'} {clean_description}" |
There was a problem hiding this comment.
El formato de la cadena incluye un guión entre espacios innecesarios. La expresión f"{product.name} {'-'} {clean_description}" genera algo como "Producto - Descripción", pero debería ser simplemente f"{product.name} - {clean_description}" para evitar la redundancia de espacios alrededor del guión.
| name = f"{product.name} {'-'} {clean_description}" | |
| name = f"{product.name} - {clean_description}" |
| add_product_name = ( | ||
| self.env["ir.config_parameter"].sudo().get_param("stock_ux.delivery_slip_add_product_name", "False") == "True" | ||
| ) |
There was a problem hiding this comment.
Se está llamando a get_param() con sudo() dentro de un método que puede ejecutarse múltiples veces durante la agregación de líneas. Esto puede impactar el rendimiento. Considera almacenar el resultado en una variable fuera del loop o en el contexto si este método se llama repetidamente para múltiples líneas de movimiento.
mav-adhoc
force-pushed
the
19.0-t-56560-mav-1
branch
from
ccd675f to
53f29b4
Compare
2 participants
No description provided.