[FIX] stock_dropshipping: do not raise a warning for dropshipping product
When selling a product with a dropshipping route, the available stock was checked and a warning got raise if it was the case. The _check_routing was not working properly as is called in a context of an onchange and 'self' was the record in the database, not in the browser. Just use check there is a dropshipping route on the product instead of checking it on the routes of the sale order line (not reliable info anyway). Add a todo for master to improve it to change the signature of the method. opw 657760
This commit is contained in:
@@ -7,6 +7,7 @@ from openerp.exceptions import UserError
|
|||||||
class sale_order_line(models.Model):
|
class sale_order_line(models.Model):
|
||||||
_inherit = 'sale.order.line'
|
_inherit = 'sale.order.line'
|
||||||
|
|
||||||
|
# TODO @api.model as called from onchange
|
||||||
@api.multi
|
@api.multi
|
||||||
def _check_routing(self, product, warehouse):
|
def _check_routing(self, product, warehouse):
|
||||||
""" skip stock verification if the route goes from supplier to customer
|
""" skip stock verification if the route goes from supplier to customer
|
||||||
@@ -14,12 +15,11 @@ class sale_order_line(models.Model):
|
|||||||
"""
|
"""
|
||||||
res = super(sale_order_line, self)._check_routing(product, warehouse)
|
res = super(sale_order_line, self)._check_routing(product, warehouse)
|
||||||
if not res:
|
if not res:
|
||||||
for line in self:
|
for route in product.route_ids:
|
||||||
for pull_rule in line.route_id.pull_ids:
|
for pull_rule in route.pull_ids:
|
||||||
if (pull_rule.picking_type_id.default_location_src_id.usage == 'supplier' and
|
if (pull_rule.picking_type_id.default_location_src_id.usage == 'supplier' and
|
||||||
pull_rule.picking_type_id.default_location_dest_id.usage == 'customer'):
|
pull_rule.picking_type_id.default_location_dest_id.usage == 'customer'):
|
||||||
res = True
|
return True
|
||||||
break
|
|
||||||
return res
|
return res
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user