OpenERPv6 - l10n_es_partner_seq no encuentra las cuentas tipo 'terceros - rec' y 'terceros - pay'

[Juanjo A]

Al crear las cuentas de un cliente mediante el asistente sale el error de '...cuentas del tipo 'terceros - rec' y 'terceros - pay' no encontradas, no creando las cuentas para el cliente.

Sol:
Mirando el wizard (create_accounts.py) del módulo tenemos las dos líneas que buscan cuentas del tipo mencionado:
res_ids = account_type_obj.search(cr, uid, [('code', '=', 'terceros_- _rec')])
...
res_ids = account_type_obj.search(cr, uid, [('code', '=', 'terceros_- _pay')])

Pero el 'code' de esos tipos de cuenta es realmente 'terceros - rec' y 'terceros - pay' (espacios y no guión bajo alrededor del guión normal) con lo que no los encuentra.

Si cambio las líneas por estas otras funciona, crea las cuentas, aunque con el cliente web, no cierra la
ventana y refresca la ficha del cliente.
Se queda en la misma ventana de configuración de las nuevas cuentas:

res_ids = account_type_obj.search(cr, uid, [('code', '=', 'terceros - rec')])
res_ids = account_type_obj.search(cr, uid, [('code', '=', 'terceros - pay')])

Un saludo
Juanjo A

[Juanjo A (juanjoa)]

La versión de la localización es la 217

addons60: 4495
extra-addons60: 5355
openerp-server:3367
cliente-web: 4535
Ubuntu 10.04 / Postgresql 8.4.6

Gracias
Juanjo A

[Borja López Soilán (NeoPolus) (borjals)]

Hola Juanjo.

El problema se debe a esta modificación que hice esta semana: http://bazaar.launchpad.net/~openerp-spain-team/openerp-spain/6.0/revision/212

    [FIX] l10n_es: Cambiados ids con espacios en ellos (a imagen de addons).

    Algunos identificadores de registro de cuentas, en los archivos XML,
    usaban espacios y eso (aparentemente) puede provocar problemas en
    algunos casos.
    El 2010-12-30 niv-openerp <email address hidden>
    corrigió esto para la rama addons (lp:openobject-addons/6.0),
    pero la corrección no se había aplicado en esta rama hasta ahora.

Como ves, parece que en la versión 6 los códigos con espacios ('terceros - rec') dan algún tipo de problema, así que OpenERP cambió los códigos en el módulo l10n_es que está en los addons oficiales para quitar los espacios ('terceros - rec'), por desgracia se olvidaron de avisarnos ;)
En cualquier caso, para evitar incompatibilidades futuras entre las dos ramas, opté por aplicar el mismo cambio sobre nuestra rama.

Por supuesto esto implica que, si creaste una base de datos usando el módulo de openerp-spain entre el 2010-12-30 y el 2011-03-25, los tipos de cuenta ya están creados 'erroneamente' con códigos con espacios, y al actualizar el módulo (por desgracia) no te va a sobreescribir los tipos de cuenta automáticamente: si la localización española de la versión 6 ya fuese oficialmente estable me habría preocupado de escribir un script de migración, en este caso he dado por hecho que los 'alumnos aventajados' serán capaces de realizar la modificación manualmente ;)

Es decir, lo correcto es el código que busca cuentas sin espacios, si creas una nueva base de datos te funcionará, si usas una base de datos antigua tendrás que editar manualmente los dos tipos de cuenta para reemplazar el código.

Un saludo y gracias por reportarlo.

[Juanjo A (juanjoa)]

Ok, podría haber cambiado los ids en la base de datos, no me imaginaba que era una actualización.

Y gracias a ti, realizas una gran labor en la lista.

Un saludo
Juanjo A.

[Eduardo Ruiz (readylan)]

Hola,

a día de hoy compruebo que el wizard no crea las cuentas tal y como reportaba Juanjo.

Borja,
según puedo ver en la revisión http://bazaar.launchpad.net/~openerp-spain-team/openerp-spain/6.0/revision/212
corriges el problema de los ids en los xml.

<record id="terceros_-_pay" model="account.account.type">
            <field name="name">Terceros - A Pagar</field>
            <field name="code">terceros - pay</field>

Sin embargo, la modificación del fichero "l10n_es_partner_seq/wizard/create_accounts.py" creo que sobra ya que el campo 'code' va con los espacios.

res_ids = account_type_obj.search(cr, uid, [('code', '=', 'terceros_-_rec')]) # NO encuentra el registro

Podrías confirmármelo?

Saludos,

Eduardo

[Borja López Soilán (NeoPolus) (borjals)]

Ups, tienes razón Eduardo, metedura de pata 100% mía :)
Ahora lo corrijo...

[Borja López Soilán (NeoPolus) (borjals)]

Vale, he deshecho el cambio al asistente, efectivamente no debía haber sido modificado, ¡perdón! y gracias por insistir :)

[Eduardo Ruiz (readylan)]

Muchas gracias a ti, Borja.