Skip to content

Conversation

@pablo-cort-s73
Copy link
Contributor

@pablo-cort-s73 pablo-cort-s73 commented Oct 2, 2025

  • Migración del módulo l10n_es_partner a Odoo 19.0:

    • manifest.py: versión actualizada a "19.0.1.0.0" y añadido "oca_data_manual": ["wizard/data_banks.xml"].
  • Cambio en la generación de datos de bancos:

    • gen_src/README.rst: instrucciones modificadas para usar un CSV en lugar de XLS/xlrd (indica uso de la librería csv y conversión a .csv).
    • Añadido gen_src/REGBANESP_CONESTAB_A.csv con el listado completo de bancos españoles (archivo de datos grande).
  • Proceso de importación:

    • Se mantiene el asistente (wizard) que descarga el fichero del Banco de España, convierte de XLS a CSV (usa xlrd si está disponible) y llama a gen_bank_data_xml para generar el XML que luego importa con tools.convert_xml_import.
    • Si la descarga falla, el wizard usa la copia local CSV incluida en gen_src.

Depends on:

Borja L.S and others added 30 commits October 2, 2025 12:04
- Se han renombrado los módulos para usar la nomenclatura propuesta
  por OpenERP: l10n_es para el módulo base de localización (plan de
  cuentas), l10n_es_* para el resto de módulos.

- Se eliminan los módulos extra_addons/* que deberían moverse a
  los extra-addons genéricos (no son específicos de España).

- Se renombran los __terp__.py por __openerp__.py
…ciertos módulos, por los correspondientes que se modificaron para esta versión 6.0 y en ciertos __init__ adaptamos los imports a los nuevos nombres de los ficheros. Renombrado de los archivos de traducción españoles de es_ES.po a es.po y pequeñas refactorizaciones
[FIX] Renombrado de los archivos de traducción catalanas de ca_ES.po a ca.po
[IMP] Eliminadas reglas de seguridad en modelos osv.osv_memory
[IMP] Añadido website en la definición del módulo
[IMP] Actualizado listado bancos y cajas españoles con los últimos datos disponibles del Banco de España: http://www.bde.es/webbde/es/secciones/servicio/regis/regent_excel.html
[IMP] Improved the introduction of the VAT (uppercase conversion).
[RFR] Cambio de licencia de GPL3 a AGPL3 en el resto de módulos de localización española, con el consentimiento de todos los autores expresado vía lista de correo del equipo de localización [email protected]
* [IMP] Cambio de la denomición del banco 0216. Actualizado por BDE el 22/11/2011
* [IMP] Añadido ING DIRECT a las lista de bancos
* [FIX] Añadido campo code, eliminado en res.bank de la versión 6.1.
* [FIX] Bug #950256
…emoved from res.bank so I have added it in the localization as it is required to lookup banks when entering the bank account number. It isn't a normal merge cause of different bzr version in the branches
…ron marcados como no instalables mientras no se compruebe que funcionan o migren, para poder usarlos o probarlos se debería poner el atributo installable de los ficheros __openerp__.py de cada módulo a True
[IMP] Refactorización de código
[IMP] Asistente de importación puesto como res.config.installer que se ejecuta al instalar el módulo.
[IMP] Limpieza del archivo XML de bancos.
[FIX] Adaptación de las vistas a la versión 7.0.
[IMP] PEP8 + mejoras en la herencia de vistas

Launchpad automatic translations update.
…ltima versión

Launchpad automatic translations update.
[IMP] l10n_es_partner: Mejorada descripción del módulo.
[IMP] l10n_es_partner: Incluido icono.
[IMP] l10n_es_partner: Añadido BIC de Pastor Serfin.
[FIX] l10n_es_partner: Eliminados espacios antes de comprobar CCC para que dé correcto.
[IMP] l10n_es_partner: Eliminación de la mayoría de warnings que da runbot.

Launchpad automatic translations update.
* [FIX] Cerrar manejador de fichero y clase en PEP8
* [FIX] Campo code duplicado
* [IMP] Test
* [FIX] Añadida dependecia a base_vat
* [FIX] Aplicar API 8.0 y mas test
* Importar los toponimos
…y + Eliminar apóstrofes de los identificadores externos + Forzar cambio de id externo para el banco res_bank_bnpparibasse
Se ha corregido además el script de importación para que ya no cause más problemas

OCA Transbot updated translations from Transifex
…ved on update

As the data generated by `l10n.es.partner.import.wizard` is created using
the flag `noupdate=False`, the subsequent update of the module is removing
all the created data. This causes the Bank Accounts (`res.partner.bank`)
to be silently losing their assigned Bank (res.bank).

OCA Transbot updated translations from Transifex
* Better search on partner
* Review fixes
* More tests
* Migration script
* Improve search over other terms:

  When searching on the omnibar, you can include other filters and
  the default search includes other different arguments, so the
  condition for adding the commercial name should be expanded.
* Make name_search work

  As Odoo overwrites name_search on res.partner in a non inheritable way,
  we have to include explicitly name_search method and search first for
  the partners with that commercial, and then the rest.

OCA Transbot updated translations from Transifex
OCA Transbot updated translations from Transifex
* Updated banks local file.
* Updated import script
* Adapted res.partner.bank logic and views
* Adapted account.journal logic
OCA Transbot updated translations from Transifex
oca-ci and others added 7 commits October 2, 2025 12:05
As the CI fails with warnings, let's put info level if performing tests.
[FIX] l10n_es_partner: Fix vista formulario banco recuperando el ancho

[FIX] l10n_es_partner: Fix vista formulario banco, respetando ancho y eliminando select attribute.
@pablo-cort-s73 pablo-cort-s73 force-pushed the 19.0-mig-l10n_es_partner branch 6 times, most recently from 921e3d8 to 30f6371 Compare October 7, 2025 13:10
@pablo-cort-s73 pablo-cort-s73 force-pushed the 19.0-mig-l10n_es_partner branch from 30f6371 to d7165a0 Compare October 16, 2025 06:21
@pablo-cort-s73 pablo-cort-s73 mentioned this pull request Oct 16, 2025
33 tasks
@pedrobaeza
Copy link
Member

/ocabot migration l10n_es_partner

@OCA-git-bot OCA-git-bot added this to the 19.0 milestone Oct 16, 2025
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

En una prueba funcional, todo funciona bien, excepto el patrón para mostrar, que estando establecido, en el propio registro el display name sale cambiado:

Image

pero luego en los many2one, no está utilizando ese display name:

Image

¿Puedes mirar a ver por qué?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Podrías aprovechar para volver a descargar este archivo local, regenerar el XML, y actualizar las referencias de fechas. También incluso se podría cambiar para que fuera un CSV y ocupara menos y cargara más rápido.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok @pedrobaeza, ahora lo vuelvo a regenerar y lo paso a csv.

@pablo-cort-s73 pablo-cort-s73 force-pushed the 19.0-mig-l10n_es_partner branch 4 times, most recently from 1826198 to 3950cad Compare October 16, 2025 11:44
@pablo-cort-s73
Copy link
Contributor Author

pablo-cort-s73 commented Oct 16, 2025

¿Puedes mirar a ver por qué?

@pedrobaeza
Hay un nuevo context en el _compute_display_name que se llama desde web_name_search que lo modifica

https://github.com/odoo/odoo/blob/9900375bc0bac2754150fd8cd6a1e45ecad8da83/odoo/addons/base/models/res_partner.py#L1015

def web_name_search(self, name, specification, domain=None, operator='ilike', limit=100):

Hacemos el super y comprobamos que está ese context para volver a poner el patrón.

@api.depends("comercial")
@api.depends_context("no_display_commercial")
@api.depends(
"complete_name",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No hace falta declarar todos los depends, solo los nuevos.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perdona si no fui claro. Lo que me refería de convertir a CSV es este archivo, no el otro (aunque el otro tampoco venga mal).

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Que va... igual debería haber preguntado yo mejor si te referias al xml, había entendido de pasar el xls a csv en todos los pasos.
Entonces quito lo modificado para utilizar csv?
Aparte de eso, modifico para que sea un csv el archivo data_banks.xml

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

En teoría ese Excel lo conseguí del Banco de España, y la idea es que en un momento dado, se actualice de nuevo para tener los últimos BICs. En una búsqueda rápida, he encontrado este PDF: https://www.bde.es/f/webbde/SPA/sispago/t2/TARGET2_BE_BIC.pdf, aunque el listado parece algo corto. Entonces la idea es poder actualizarlo de forma sencilla.

@pablo-cort-s73 pablo-cort-s73 force-pushed the 19.0-mig-l10n_es_partner branch 3 times, most recently from ae84967 to f570156 Compare October 20, 2025 08:30
@pablo-cort-s73 pablo-cort-s73 force-pushed the 19.0-mig-l10n_es_partner branch from f570156 to 4a30f7c Compare October 21, 2025 06:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.