Skip to content
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

[18.0][MIG] l10n_es_partner: Migration to 18.0 #3998

Open
wants to merge 67 commits into
base: 18.0
Choose a base branch
from

Conversation

mpascuall
Copy link
Contributor

@mpascuall mpascuall commented Feb 10, 2025

Module migrated to version 18.0

He tenido que modificar la vista del wizard ya que en Odoo 18 no existe el modelo 'res.config.installer' y lo he añadido al menú Bancos de Facturación.

cc https://github.com/APSL 166331

@miquelalzanillas @lbarry-apsl @javierobcn @peluko00 @ppyczko @BernatObrador please review

Borja L.S and others added 30 commits January 29, 2025 10:11
- 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-git-bot and others added 14 commits January 29, 2025 10:11
It seems that BDE is mangling somehow the automated download of the
file detecting some patterns. We change the URL for trying to maximize
the possibilities of being downloaded, but also provide a local file
as a fallback.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-spain-16.0/l10n-spain-16.0-l10n_es_partner
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-16-0/l10n-spain-16-0-l10n_es_partner/
The elements are not shown correctly due to the styling changes in v16.
@mpascuall mpascuall force-pushed the 18.0-mig-l10n_es_partner branch from d964bcf to c33ed7d Compare February 10, 2025 10:02
@mpascuall
Copy link
Contributor Author

@pedrobaeza Buenas! En esta migración fallan los tests debido a que da un error descargando la información desde internet, pero en local me pasan correctamente. Debo hacer algún cambio?

@juancarlosonate-tecnativa

Justo acabo de realizar una migración con un problema similar relacionado con res_config_installer, revisa #4027, es la alternativa más óptima.

@mpascuall mpascuall force-pushed the 18.0-mig-l10n_es_partner branch from c33ed7d to 0f17c60 Compare February 18, 2025 11:28
@mpascuall
Copy link
Contributor Author

Justo acabo de realizar una migración con un problema similar relacionado con res_config_installer, revisa #4027, es la alternativa más óptima.

Buenas @juancarlosonate-tecnativa! Muchas gracias por tu comentario, básicamente he hecho lo mismo, crear una nueva vista con el wizard manteniendo la misma funcionalidad. Lo único es que hay un problema con los tests que fallan por el warning de la conexión a internet.

@EmilioPascual
Copy link
Contributor

Justo acabo de realizar una migración con un problema similar relacionado con res_config_installer, revisa #4027, es la alternativa más óptima.

Buenas @juancarlosonate-tecnativa! Muchas gracias por tu comentario, básicamente he hecho lo mismo, crear una nueva vista con el wizard manteniendo la misma funcionalidad. Lo único es que hay un problema con los tests que fallan por el warning de la conexión a internet.

No te fijes en el wizard, fijate en el test y como resuelve el problema https://github.com/OCA/l10n-spain/pull/4027/files#diff-995bf288bfd4b78d663bac240b335fd72353933ba075274853f3e608bade9f80R17

@mpascuall mpascuall force-pushed the 18.0-mig-l10n_es_partner branch 2 times, most recently from 816905d to 482a11c Compare February 20, 2025 12:24
@mpascuall
Copy link
Contributor Author

Justo acabo de realizar una migración con un problema similar relacionado con res_config_installer, revisa #4027, es la alternativa más óptima.

Buenas @juancarlosonate-tecnativa! Muchas gracias por tu comentario, básicamente he hecho lo mismo, crear una nueva vista con el wizard manteniendo la misma funcionalidad. Lo único es que hay un problema con los tests que fallan por el warning de la conexión a internet.

No te fijes en el wizard, fijate en el test y como resuelve el problema https://github.com/OCA/l10n-spain/pull/4027/files#diff-995bf288bfd4b78d663bac240b335fd72353933ba075274853f3e608bade9f80R17

Perfecto, muchas gracias!

@pedrobaeza
Copy link
Member

El problema que se está tratando en #4029 afecta aquí también

/ocabot migration l10n_es_partner

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Feb 20, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request Feb 20, 2025
31 tasks
@mpascuall mpascuall force-pushed the 18.0-mig-l10n_es_partner branch from 482a11c to ab949b6 Compare February 21, 2025 07:12
@mpascuall
Copy link
Contributor Author

El problema que se está tratando en #4029 afecta aquí también

/ocabot migration l10n_es_partner

Gracias por tu comentario @pedrobaeza! ¿Debería añadir un commit específico para este FIX? Ahora mismo he agregado los cambios en el commit de migración, pero si ves que es mejor en un commit específico, lo cambiaré.

@pedrobaeza
Copy link
Member

El tema es que ese parche no vale, por lo que hay que arreglarlo tanto en 17 como en 18.

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.