On 29/07/14 02:01, Raimon Esteve wrote:
2014-07-28 20:06 GMT+02:00 Oscar Alvarez <[email protected]>:
Hola devs
Al instalar sale_shop en una base de datos con ventas previas genera el
siguiente error
Simple. Los pedidos van relacionados con tienda. Primero debes crear
una tienda, tanto si tienes ya o no pedidos de venta.
Haicendo una depuracion de la query que genera el error obtengo esto:
('UPDATE "sale_sale" SET "shop" = %s', (None,))
En el directorio doc, tienes el INSTALL q tienes el comando SQL aplicar:
https://bitbucket.org/zikzakmedia/trytond-sale_shop/src/266d39ac406885ad851844f3b238fa585fb4802e/doc/INSTALL?at=default
Seria bueno activar el seguimiento a fallos bitbucket, para no hacerlo a
traves de mail. Por otro lado pienso que el campo shop en sale no deberia
ser obligatorio, quizas esto elimine el problema, ahora la instalacion solo
funciona en base de datos nueva.
Un pedido va relacionado con una tienda. Es la gracia del mòdulo. Si
no usas tiendas físicas o virtuales, no lo instales.
Estoy de acuerdo que si el modulo se instala es para utilizarlo, y que
el campo sale_shop incluso reconozco que puede ser obligatorio sin que
genere bug, el problema es el atributo domain de este campo, que busca
en el contexto el sale_shop cuando este no existe genera el error en la
instalacion, pienso que es un poco extraño (tener que ejecutar un
comando sql manualmente, para que funcione 100%), teniendo en cuenta que
esto se podria solventar simplemente quitando el atributo domain, para
que no genere el error, y fijando una tienda por defecto desde
configuracion de sale_shop en un modelo singleton, luego se toma este
atributo en la creación de una nueva venta, con esto seria innecesario
el sql.
Solo es mi sugerencia de mejora del modulo.
