On Tuesday, July 29, 2014 10:59:28 AM UTC-5, oscar_alvarez wrote: > > > On 29/07/14 02:01, Raimon Esteve wrote: > > 2014-07-28 20:06 GMT+02:00 Oscar Alvarez <[email protected] > <javascript:>>: > >> 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. >
+1
