El 29/07/2014 18:47, "Raimon Esteve" <[email protected]> va escriure:
>
> > 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%),
>
> A - Instala el modulo antes de crear un pedido de venta.
> B- Haz un Pull request con la funcionalidad al instal·lar este modulo,
> aparezca un asistente que te pida datos para crear una nueva tienda y
> añade todos los pedidos con esta tienda (aqui tienes curro)
No se si cambiando el domain por éste se solucionaria el error:
If(Bool(Eval('shop')),
Eval('context', {})... *,
()),
*: aquí el domain actual.
Aunque no me cuadra que el domain lleve a la query del UPDATE que ponias.
La solución definitiva es la que dice Raimon de añadir la creación de la
tienda en la postinstalación del módulo... A ver si alguien se anima.
> > 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.
>
> Un usuario puede estar en varias tiendas y hacer ventas en una tienda
> en concreto según sus preferencias. Este sistema que propones a
> sale.configuration no vale, porque sin el dominio, podrias seleccionar
> tiendas que no tengas en tu perfil de usuario.
Exacto, tu propuesta es válida para para entornos de una sola tienda y el
módulo ya está hecho para multitienda pudiendo limitar las tiendas por
usuario (igual q las compañías).
Si se hace lo de crear la tienda en la postinstalación, debería asignarla
al usuario actual para poder empezar a trabajar desde el primer momento.
