[18.0][MIG] l10n_es_partner: Migration to 18.0 #3998
Conversation
- 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 openerp-spain@googlegroups.com
* [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
…ercial name field.
…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
…on los últimos datos disponibles del Banco de España: http://www.bde.es/webbde/es/secciones/servicio/regis/regent_excel.html
…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
…t de generacion de data_banks.xml
…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
[ADD] Mas caracteres a escapar
…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
mpascuall
force-pushed
the
18.0-mig-l10n_es_partner
branch
from
c33ed7d to
0f17c60
Compare
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
force-pushed
the
18.0-mig-l10n_es_partner
branch
2 times, most recently
from
816905d to
482a11c
Compare
Perfecto, muchas gracias! |
|
El problema que se está tratando en #4029 afecta aquí también /ocabot migration l10n_es_partner |
mpascuall
force-pushed
the
18.0-mig-l10n_es_partner
branch
from
482a11c to
ab949b6
Compare
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é. |
|
El tema es que ese parche no vale, por lo que hay que arreglarlo tanto en 17 como en 18. |
|
Haz por favor cherry-pick de #4058 antes de los commit de migración |
mpascuall
force-pushed
the
18.0-mig-l10n_es_partner
branch
from
ab949b6 to
190973b
Compare
Previous patch was overriding _auto_init method for injecting the comercial field in the list of fields to search for (_rec_names_search), but this method is only called on module installation/update, so if you restart the Odoo server after that, you lose the injection. Thus, let's inject it if not present when calling name_search, and this way we always have it available. TT55241
mpascuall
force-pushed
the
18.0-mig-l10n_es_partner
branch
2 times, most recently
from
455bcf3 to
49a30fc
Compare
Buenas @pedrobaeza, gracias por el comentario. He hecho el cherry-pick antes del commit de migración pero ahora el test da un ConnectionTimeour, a qué se puede deber? |
|
El problema es ajeno a vosotros, no te preocupes. Luego reviso. |
|
Hay que añadir #4131 , el ConnectTimeoutError sigue sucediendo? |
mpascuall
force-pushed
the
18.0-mig-l10n_es_partner
branch
from
49a30fc to
42216a0
Compare
|
This PR looks fantastic, let's merge it! |
|
Congratulations, your PR was merged at c833ddb. Thanks a lot for contributing to OCA. ❤️ |
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