Skip to content

Conversation

@OmarYepez29
Copy link
Contributor

No description provided.

Ivan-Binaural and others added 30 commits October 23, 2025 11:49
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, 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 []
@OmarYepez29 OmarYepez29 self-assigned this Nov 7, 2025
Copilot AI review requested due to automatic review settings November 7, 2025 14:24
@mergify
Copy link

mergify bot commented Nov 7, 2025

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🔴 Require a review from approvers team

This rule is failing.

This rule requires someone from the approver's team to approve the PR

  • approved-reviews-by = @probadores

Copy link
Contributor

Copilot AI left a 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_idreference_ids, move_line_ids_without_packagemove_line_ids)
  • Migrated XML view syntax (e.g., <div class="oe_chatter"><chatter/>, treelist)
  • 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_residual is captured before the context change but self is reassigned with a new context. The subsequent call to recalculate_bi_igtf should use the context-modified self. Consider capturing the method result in a new variable instead of reassigning self.
        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.

Comment on lines +556 to +558
invoice.amount_residual,
2,
f"Tras la desconciliación, el residual debe ser ${invoice.amount_residual} pero es ${invoice.amount_residual}"
Copy link

Copilot AI Nov 7, 2025

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.

Suggested change
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}"

Copilot uses AI. Check for mistakes.
Comment on lines +785 to +787
invoice.amount_residual,
2,
f"Tras la desconciliación, el residual debe ser ${invoice.amount_residual} pero es ${invoice.amount_residual}"
Copy link

Copilot AI Nov 7, 2025

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.

Suggested change
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}"

Copilot uses AI. Check for mistakes.

@api.depends("is_igtf")
def _compute_igtf_percentage(self):
""" Compute the igtf percetage defined in the company"""
Copy link

Copilot AI Nov 7, 2025

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.

Suggested change
""" Compute the igtf percetage defined in the company"""
""" Compute the igtf percentage defined in the company"""

Copilot uses AI. Check for mistakes.
Ivan-Binaural and others added 22 commits November 7, 2025 10:38
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
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
@OmarYepez29 OmarYepez29 requested a review from a team November 11, 2025 13:56
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants