Módulos no funcionan con versión trunk (8.0 dev)

Bug #1284140 reported by Javier Matos Odut
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenERP Spanish Localization Project
Fix Released
Undecided
Unassigned

Bug Description

Hola,

estoy probando la nueva versión trunk (8.0 dev) a fecha 24 de Febrero de 2014 y por desgracia no funcionan los módulos asociados al proyecto de OpenERP-Spain. He visto que el error está en que cambian las rutas de algunos archivos en OpenERP, de manera que los import fallan.

Nos hace falta usar justo esta versión así que como parecía no ser muy difícil cambié los import y conseguí que funcionara (al menos inicia el servidor, si bien no he hecho pruebas o ejecutado tests).

Todo cuanto hube de hacer lo he simplificado en un script, y es que en mi caso que hago despliegues automáticos de OpenERP me pareció la forma más fácil de modificar el código del repositorio una vez se descargara.

Aquí comparto con ustedes los cambios que hice, por si pudieran incorporarse a la versión 7.0. Esto lo digo porque no sé si tal vez la versión 7.0 permite los imports de ambas rutas (la actual y la que usarán en la versión 8.0 que es la nueva), de manera que hagamos que el código ya sea portable sin más a la nueva versión. En este sentido, he visto que el módulo "partner-contact-management" funciona en la versión 7.0 como en trunk, así que sería buena cosa hacer lo mismo con vuestro desarrollo.

--> Script aplicado:
#!/bin/bash

REPO=$1

# Fix netsvc
grep -rl "^import netsvc" $REPO | xargs sed -i 's/^import netsvc/from openerp import netsvc/g'

# Fix pooler
grep -rl "^import pooler" $REPO | xargs sed -i 's/^import pooler/from openerp import pooler/g'

# Fix report
grep -rl "^import report" $REPO | xargs sed -i 's/^import report/from openerp import report/g'

# Fix tools
grep -rl "^import tools" $REPO | xargs sed -i 's/^import tools/from openerp import tools/g'
grep -rl "^from tools" $REPO | xargs sed -i 's/^from tools/from openerp.tools/g'

Revision history for this message
Javier Matos Odut (niseteocurra) wrote :
Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

Buenas, Javier,

La v8 aún no es una versión soportada por la localización española, sobre todo porque es una versión aún no finalizada que puede cambiar, y resultaría contraproducente realizar cambios que luego pudieran no servir. Aún así, en la v7 se ha hecho mucho más hincapié en una refactorización que minimice el número de cambios entre una versión y otra.

El cambio que comentas precisamente es por la forma en la que se carga el servidor (añadiendo el espacio de nombres openerp), que en la v7 ya estaba presente, junto con la otra forma compatible, y cuya compatibilidad se ha eliminado en la v8. Puesto que ese cambio también sirve para la v7, lo he aplicado sin más en la revisión 440. Gracias por el script que lo hace automáticamente.

Seguramente te encontrarás con más problemas trabajando con los módulos de la localización en la v8, ya que también se eliminan los alias osv.osv y osv.osv_memory en favor de orm.Model y orm.TransientModel, teniendo que cambiar también el 'from osv import osv, fields' a 'from openerp.osv import orm, fields'. De todas formas, en todos los módulos en los que yo he hecho la migración o he colaborado (salvo el primero si no recuerdo mal), ese cambio ya está hecho.

Gracias de nuevo por colaborar, y si encuentras algún otro problema, háznoslo saber.

Un saludo.

Changed in openerp-spain:
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.