[IMP] mrp_product_variants: delete default domain

oihane commented 8 years ago

Se elimino está línea en https://github.com/odoomrp/odoomrp-wip/commit/7ccdf654d79a8f14df2763a00f268834377f5cf4#diff-0ac97c08c3bad0031f23566211c42f7aL22

segun @Daniel-CA:

El tema está en la función que define si el producto definido en el BoM es apto para la variante, tal y como esta enfocada en este momento se deben cumplir todas las variantes en la linea para que el producto se añada. Por lo que si un componente en el BoM tiene las variantes de Grande y Pequeño definidas en la misma linea, a la hora de fabricar un producto con cualquiera de esas dos variantes no se añadirá ya que no cumple las dos (cosa que no ocurrirá nunca ya que se excluyen mutuamente).

Otro enfoque sería que se añada en caso de que cumpla cualquiera de las dos variantes, pero de hacerse así si queremos añadir variantes de otro tipo habría que hacerlo de forma diferente ya que puede que el componente solo se tenga que añadir si se cumplen que sea Grande y Negro por ejemplo.

codecov-io commented 8 years ago

Current coverage is 57.56% (diff: 100%)

Merging #1335 into 8.0 will not change coverage

@@                8.0      #1335   diff @@
==========================================
  Files           268        268          
  Lines          8027       8027          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits           4621       4621          
  Misses         3406       3406          
  Partials          0          0

Powered by Codecov. Last update 4bbfe76...f573543

Daniel-CA commented 8 years ago

:+1:

anajuaristi commented 8 years ago

Functionally tested in runbot (light test) It seems to be OK :+1:

odoomrp / odoomrp-wip

[IMP] mrp_product_variants: delete default domain #1335

Current coverage is 57.56% (diff: 100%)