-
Notifications
You must be signed in to change notification settings - Fork 19
Dev 19 mig igtf 19 #365
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: dev-19
Are you sure you want to change the base?
Dev 19 mig igtf 19 #365
Conversation
…on #58513 Corrijiendo error al intentar instalar debido a las vistas. Link:https://binaural.odoo.com/web/#id=58513&cids=2&menu_id=975&action=341&model=project.task&view_type=form
Se suben ajustes correspondientes para la migración de binaural_mobile a la versión 19.0, se indican las siguientes observaciones: .- Los módulos modificados para poder ser instalados fueron los siguientes: *l10n_ve_location *l10n_ve_payment_extension *l10n_ve_stock Tarea (Link): https://binaural.odoo.com/web\#id\=58139\&cids\=2\&menu_id\=975\&action\=341\&model\=project.task\&view_type\=form Tarea de proyecto [x] Ticket de soporte []
Corrijiendo vistas para que se muestren los calculos foraneos Link:https://binaural.odoo.com/web/#id=58983&cids=2&menu_id=975&action=341&model=project.task&view_type=form
Corrijiendo vistas para que se muestren los calculos foraneos Link:https://binaural.odoo.com/web/#id=58983&cids=2&menu_id=975&action=341&model=project.task&view_type=form
Corrijiendo vistas para que se muestren los calculos foraneos, tanto en moneda alterna como en totales impuestos Link:https://binaural.odoo.com/web/#id=58983&cids=2&menu_id=975&action=341&model=project.task&view_type=form
Corrijiendo vistas para que se muestren los calculos foraneos, tanto en moneda alterna como en totales impuestos Link:https://binaural.odoo.com/web/#id=58983&cids=2&menu_id=975&action=341&model=project.task&view_type=form
Problema: -Se requiere que la lista de precios en las cotizaciones, pueda ser seleccionada sin importar la moneda base, es decir cualquier lista de precios Solución: -se quita el domain que solo limitaba la lista de precios a las de la moneda base Tarea (Link): https://binaural.odoo.com/web#id=57482&cids=2&menu_id=255&action=327&active_id=614&model=project.task&view_type=form Tarea de proyecto [x] Ticket de soporte []
Corrijiendo vistas para que se muestren los calculos foraneos, tanto en moneda alterna como en totales impuestos Link:https://binaural.odoo.com/web/#id=58983&cids=2&menu_id=975&action=341&model=project.task&view_type=form
Corrijiendo vistas para que se muestren los calculos foraneos, tanto en moneda alterna como en totales impuestos Link:https://binaural.odoo.com/web/#id=58983&cids=2&menu_id=975&action=341&model=project.task&view_type=form
Problema: -Se requiere que la lista de precios en las cotizaciones, pueda ser seleccionada sin importar la moneda base, es decir cualquier lista de precios -Se remueve de la factura ahora tambien Solución: -se quita el domain que solo limitaba la lista de precios a las de la moneda base Tarea (Link): https://binaural.odoo.com/web#id=57482&cids=2&menu_id=255&action=327&active_id=614&model=project.task&view_type=form Tarea de proyecto [x] Ticket de soporte []
Se corrige código duplicado en manifest, se ajusta codigo comentando en stock_picking, y se corrige grupo creado. Tarea (Link): https://binaural.odoo.com/web\#id\=58139\&cids\=2\&menu_id\=975\&action\=341\&model\=project.task\&view_type\=form Tarea de proyecto [x] Ticket de soporte []
Corrijiendo error al intentar instalar y al pasar facturas de proveedores Link:https://binaural.odoo.com/web#id=58454&cids=2&model=project.task&view_type=form
Problema: -Al intentar registrar un pago de cliente, muestra un error. Solución: -Se refactoriza la función action_confirm para que pueda acceder correctamente a los valores de order, y recorra la respectiva lógica. Tarea (Link): https://binaural.odoo.com/web#id=59281&cids=2&menu_id=975&action=341&model=project.task&view_type=form Tarea de proyecto [x] Ticket de soporte []
Corrijiendo vistas para que se muestren solo los calculos en bolivares a peticion del cliente en un componente personalizado Link:https://binaural.odoo.com/web/#id=58983&cids=2&menu_id=975&action=341&model=project.task&view_type=form
Corrijiendo vistas para que se muestren solo los calculos en bolivares a peticion del cliente en un componente personalizado agregando dichos campos en el componente de javascript tax_totals para que se reflejen en las vistas correspondientes Link:https://binaural.odoo.com/web/#id=58983&cids=2&menu_id=975&action=341&model=project.task&view_type=form
Corrijiendo vistas para que se muestren solo los calculos en bolivares a peticion del cliente en un componente personalizado agregando dichos campos en el componente de javascript tax_totals para que se reflejen en las vistas correspondientes y se toma en cuenta como estan organizados tanto en backend como en front-end Link:https://binaural.odoo.com/web/#id=58983&cids=2&menu_id=975&action=341&model=project.task&view_type=form
Problema: -Se muestra un boton de agregar nota de debito adicional Solución: Se remueve el boton de ve_invoice, al ya estar definido de forma nativa Tarea (Link): https://binaural.odoo.com/web/#id=59177&cids=2&menu_id=975&action=341&model=project.task&view_type=form Tarea de proyecto [x] Ticket de soporte []
Problema: -Se requiere corregir algunos campos que ya no existen en la 19 y generan problemas y errores al actualizar Solución: - se remueven los campos para evitar problemas y en algunos casos se actualizan por unos ya existentes Tarea (Link): https://binaural.odoo.com/web/#id=59362&cids=2&menu_id=975&action=341&model=project.task&view_type=form Tarea de proyecto [x] Ticket de soporte []
Problema: -Al intentar crear una orden de compra, muestra error ya que intenta acceder a un objeto de tax_totals que no existe. Solución: -Se cambia el acceso de _prepare_foreign_base_line_for_taxes_computation al campo taxes_id por tax_ids Tarea (Link): https://binaural.odoo.com/web#id=59487&ci%20reds=2&menu_id=255&action=327&active_id=614&model=project.task&view_type=form Tarea de proyecto [x] Ticket de soporte []
Se realiza ajustes para poder crear almacenes de consignacion y almacenes en general Link:https://binaural.odoo.com/web#id=58454&cids=2&model=project.task&view_type=form
Problema: -Se mostraba un error al no conseguir una en el diccionario de baseline. Solución: -Se cre una tupla con los valores que normalmente se evalúa en baseline si traen valor o son none, se utiliza setdefault para setear none si no existe la llave o no tiene valor. Tarea (Link): https://binaural.odoo.com/web#id=59487&ci%20reds=2&menu_id=255&action=327&active_id=614&model=project.task&view_type=form Tarea de proyecto [x] Ticket de soporte []
Merge ProtectionsYour pull request matches the following merge protections and will not be merged until they are valid. 🔴 Require a review from approvers teamThis rule is failing.This rule requires someone from the approver's team to approve the PR
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR migrates the codebase to Odoo 19, updating deprecated fields, methods, and XML syntax across multiple modules including stock, accounting, POS, and IGTF functionality.
Key Changes:
- Updated deprecated Odoo fields and methods (e.g.,
group_id→reference_ids,move_line_ids_without_package→move_line_ids) - Migrated XML view syntax (e.g.,
<div class="oe_chatter">→<chatter/>,tree→list) - Updated tax computation methods and field names for Odoo 19 compatibility
- Added comprehensive IGTF test suites for both providers and partners
Reviewed Changes
Copilot reviewed 45 out of 45 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| l10n_ve_stock/models/stock_picking.py | Replaced deprecated group_id with reference_ids field |
| l10n_ve_stock/views/stock_location_views.xml | Simplified chatter implementation using new <chatter/> tag |
| l10n_ve_stock/models/stock_warehouse.py | Updated deprecated _get_picking_type_create_values method |
| l10n_ve_payment_extension/views/*.xml | Updated search view syntax removing deprecated attributes |
| l10n_ve_igtf/models/account_tax.py | Migrated _prepare_tax_totals to _get_tax_totals_summary |
| l10n_ve_igtf/tests/* | Added new comprehensive test files for IGTF functionality |
| l10n_ve_accountant/static/src/components/tax_totals/* | Updated JavaScript and XML templates for Odoo 19 |
Comments suppressed due to low confidence (1)
l10n_ve_igtf/models/account_move.py:299
- The variable
amount_residualis captured before the context change butselfis reassigned with a new context. The subsequent call torecalculate_bi_igtfshould use the context-modifiedself. Consider capturing the method result in a new variable instead of reassigningself.
self = self.with_context(from_widget=True)
res = super().js_assign_outstanding_line(line_id)
self.recalculate_bi_igtf(
line_id,
initial_residual=amount_residual
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| invoice.amount_residual, | ||
| 2, | ||
| f"Tras la desconciliación, el residual debe ser ${invoice.amount_residual} pero es ${invoice.amount_residual}" |
Copilot
AI
Nov 7, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The assertion compares invoice.amount_residual with itself, which will always pass. Based on the pattern in test_igtf_providers.py line 556-559, this should compare against invoice_amount instead.
| invoice.amount_residual, | |
| 2, | |
| f"Tras la desconciliación, el residual debe ser ${invoice.amount_residual} pero es ${invoice.amount_residual}" | |
| invoice_amount, | |
| 2, | |
| f"Tras la desconciliación, el residual debe ser ${invoice_amount} pero es ${invoice.amount_residual}" |
| invoice.amount_residual, | ||
| 2, | ||
| f"Tras la desconciliación, el residual debe ser ${invoice.amount_residual} pero es ${invoice.amount_residual}" |
Copilot
AI
Nov 7, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The assertion compares invoice.amount_residual with itself, which will always pass. This should compare against the expected value invoice_amount as seen in the providers test.
| invoice.amount_residual, | |
| 2, | |
| f"Tras la desconciliación, el residual debe ser ${invoice.amount_residual} pero es ${invoice.amount_residual}" | |
| invoice_amount, | |
| 2, | |
| f"Tras la desconciliación, el residual debe ser ${invoice_amount} pero es ${invoice.amount_residual}" |
|
|
||
| @api.depends("is_igtf") | ||
| def _compute_igtf_percentage(self): | ||
| """ Compute the igtf percetage defined in the company""" |
Copilot
AI
Nov 7, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Corrected spelling of 'percentage' in docstring.
| """ Compute the igtf percetage defined in the company""" | |
| """ Compute the igtf percentage defined in the company""" |
Corrijiendo error en ventas debido a nuevas caracteristicas de odoo 19 en las variantes de producto y los template Link:https://binaural.odoo.com/web#id=58454&cids=2&model=project.task&view_type=form
Problema: -Se requiere migrar configuraciones de stock de Integra a Homologado Solución: -Se migra la configuración not_allow_negative_inventory_adjustments -Se migra la configuración allow_scrap_more_than_available -Se migra not_allow_scrap_more_than_what_was_manufactured -Se migra not_allow_negative_stock_movement Tarea (Link): Tarea de proyecto [] Ticket de soporte [x]
Problema: -Se requiere incluir los movimientos de inventario de tipo entrega en la validación para ver si hay suficiente stock disponible para el movimiento a realizar. Solución: -Se agrega el tipo outgoing en la validación para que entre la configuración en función. Tarea (Link): https://binaural.odoo.com/web#id=9422&cids=2&menu_id=293&action=389&model=helpdesk.ticket&view_type=form Tarea de proyecto [] Ticket de soporte [x]
… github.com:binaural-dev/odoo-venezuela into HEAD
Problema: -Al cambiar la moneda de la factura, los asientos se muestran desbalanceados en las líneas de cŕedito y débito alterno. Solución: -Se añaden validaciones para ajustar los casos según la moneda de la factura, si la moneda de la factura es la principal, sigue el flujo que estaba anteriormente. Si la moneda de la factura es la misma que la foranea, el precio foraneo es igual al precio standar. En el caso de tener una moneda diferente a la principal y secundaria, entonces se realiza una conversión a la moneda principal y a partir de esta, se convierte a la foranea. Tarea (Link): https://binaural.odoo.com/web#id=59280&cids=2&menu_id=309&model=project.task&view_type=form Tarea de proyecto [x] Ticket de soporte []
Problema: -Al intentar validar un stock picking, el botón no realiza ninguna acción. Solución: -Se agregan return para los casos necesarios. Tarea (Link): Tarea de proyecto [x] Ticket de soporte []
Problema: -Al intentar actualizar cantidades de un producto en inventario a través de la interfaz de actualizar cantidades, muestra error ya que _apply_inventory está esperando un parámetro. Solución: -Se agrega date=None en los argumentos del método _apply_inventory, este en V19 es requerido para la ejecución de la función correctamente. Tarea (Link): https://binaural.odoo.com/web#id=59671&cids=2&model=project.task&view_type=form Tarea de proyecto [x] Ticket de soporte []
Corrijiendo vistas para que se muestren solo los calculos en bolivares a peticion del cliente en un componente personalizado agregando dichos campos en el componente de javascript tax_totals para que se reflejen en las vistas correspondientes y se toma en cuenta como estan organizados tanto en backend como en front-end. Ajuste sugerido por omar, en donde se tiene en cuenta los montos de la moneda local, para la conversion, queda probar con diferentes monedas, debido a que mi ambiente esta algo raro al no mostrar el campo de moneda correspondiente Link:https://binaural.odoo.com/web/#id=58983&cids=2&menu_id=975&action=341&model=project.task&view_type=form
Corrijiendo vistas para que se muestren solo los calculos en bolivares a peticion del cliente en un componente personalizado agregando dichos campos en el componente de javascript tax_totals para que se reflejen en las vistas correspondientes y se toma en cuenta como estan organizados tanto en backend como en front-end. Ajuste sugerido por omar, en donde se tiene en cuenta los montos de la moneda local, para la conversion, queda probar con diferentes monedas, debido a que mi ambiente esta algo raro al no mostrar el campo de moneda correspondiente Link:https://binaural.odoo.com/web/#id=58983&cids=2&menu_id=975&action=341&model=project.task&view_type=form
Corrijiendo vistas para que se muestren solo los calculos en bolivares a peticion del cliente en un componente personalizado agregando dichos campos en el componente de javascript tax_totals para que se reflejen en las vistas correspondientes Link:https://binaural.odoo.com/web/#id=58983&cids=2&menu_id=975&action=341&model=project.task&view_type=form
Corrijiendo vistas para que se muestren solo los calculos en bolivares a peticion del cliente en un componente personalizado agregando dichos campos en el componente de javascript tax_totals para que se reflejen en las vistas correspondientes y se toma en cuenta como estan organizados tanto en backend como en front-end Link:https://binaural.odoo.com/web/#id=58983&cids=2&menu_id=975&action=341&model=project.task&view_type=form
Corrijiendo vistas para que se muestren solo los calculos en bolivares a peticion del cliente en un componente personalizado agregando dichos campos en el componente de javascript tax_totals para que se reflejen en las vistas correspondientes y se toma en cuenta como estan organizados tanto en backend como en front-end. Ajuste sugerido por omar, en donde se tiene en cuenta los montos de la moneda local, para la conversion, queda probar con diferentes monedas, debido a que mi ambiente esta algo raro al no mostrar el campo de moneda correspondiente
Corrijiendo vistas para que se muestren solo los calculos en bolivares a peticion del cliente en un componente personalizado agregando dichos campos en el componente de javascript tax_totals para que se reflejen en las vistas correspondientes y se toma en cuenta como estan organizados tanto en backend como en front-end. Ajuste sugerido por omar, en donde se tiene en cuenta los montos de la moneda local, para la conversion, queda probar con diferentes monedas, debido a que mi ambiente esta algo raro al no mostrar el campo de moneda correspondiente
Se coloca el sql constrains que esta en la V17 de Odoo-venezuela
Corrijiendo vistas para que se muestren solo los calculos en bolivares a peticion del cliente en un componente personalizado agregando dichos campos en el componente de javascript tax_totals para que se reflejen en las vistas correspondientes y se toma en cuenta como estan organizados tanto en backend como en front-end. Ajuste sugerido por omar, en donde se tiene en cuenta los montos de la moneda local, para la conversion, queda probar con diferentes monedas, debido a que mi ambiente esta algo raro al no mostrar el campo de moneda correspondiente
…unts' into dev-19-MIG-IGTF-19
Corrijiendo vistas para que se muestren solo los calculos en bolivares a peticion del cliente en un componente personalizado agregando dichos campos en el componente de javascript tax_totals para que se reflejen en las vistas correspondientes y se toma en cuenta como estan organizados tanto en backend como en front-end. Ajuste sugerido por omar, en donde se tiene en cuenta los montos de la moneda local, para la conversion, queda probar con diferentes monedas, debido a que mi ambiente esta algo raro al no mostrar el campo de moneda correspondiente 1 file changed, 3 insertions(+), 3 deletions(-)
Al intentar hacer una retencion ocurria el error de intento de sobreescribir el context. Adicionalmente se quita el onchange_foreign_retention_amount. Esto porque en la version 19 solo se manejaran escenarios con la moneda base del sistema en Bs. Link:https://binaural.odoo.com/web#id=59695&cids=2&menu_id=257&action=341&model=project.task&view_type=form
Se agrega nuevamente los campos de igtf al tax_totals considerando los cambios fueron realizados por nosotros en l10n_ve_accountant. Link:https://binaural.odoo.com/web#id=59648&cids=2&menu_id=257&action=341&model=project.task&view_type=form
No description provided.