odoomrp / odoomrp-wip

WIP of OdooMRP
http://www.odoomrp.com
GNU Affero General Public License v3.0
117 stars 248 forks source link

[FIX][8.0] Error when new product without line attributes from template #1385

Closed igallart closed 6 years ago

igallart commented 7 years ago

If list variant products from templates and your create new product with a template without line attributes, it crash. ValueError: Wrong value for product.product.product_id: product.product(<openerp.models.NewId object at 0x746cb0918b50>, 31)

pedrobaeza commented 7 years ago

Product without attributes should be created normally, not this way...

igallart commented 7 years ago

@pedrobaeza, aunque sea como tú dices, ¿no debería estar contemplado ese caso y como mínimo, si no se debe/puede hacerlo así, dar un mensaje de error más "elegante"? Salta el típico pantallazo de error de python.

pedrobaeza commented 7 years ago

Es que tampoco creo que salte ese error, porque la variante ya debería estar creada. Por eso digo que no se debe utilizar en el caso que estés creando un producto para una plantilla que no tenga atributos. Con esto que haces, estás estropeando la funcionalidad de seleccionar automáticamente la variante en los pedidos de venta y de compra.

igallart commented 7 years ago

El caso es el siguiente, tienes la plantilla con no crear las variantes automáticamente, o si instalas una demo con datos cargados, lo dejas como está. Vas a una plantilla, si es la de demo, la primera, Advance. Le das a lista de variantes, y a continuación crear, y te salta el error.

Desconozco si es inapropiado hacerlo así, pero independientemente de eso, el error existe haciendo eso.

pedrobaeza commented 7 years ago

@oihane a ti se te ha presentado ese caso? Yo ya no trabajo con la versión 8 desde luego, pero lo que veo es que aunque sea ése el problema, la solución no es lo que hace este PR, que elimina una funcionalidad.