[FIX]_prepare_order_line_procurement should not return False (cause problem with super), instead add another function to know if the so can create a procurement or not

bzr revid: csn@openerp.com-20130911140034-5so2qbe4rd2m6131

1 files changed, 8 insertions, 4 deletions

diff --git a/addons/project_mrp/project_mrp.py b/addons/project_mrp/project_mrp.py
index 1d72375e03c..b4052509e38 100644
--- a/addons/project_mrp/project_mrp.py
+++ b/addons/project_mrp/project_mrp.py
@@ -153,14 +153,18 @@ class sale_order(osv.osv):
     _inherit = 'sale.order'
 
     def _prepare_order_line_procurement(self, cr, uid, order, line, group_id=False, context=None):
-        if not(line.product_id.type== "service" and not line.product_id.auto_create_task):
-            proc_data = super(sale_order, self)._prepare_order_line_procurement(cr,
+        proc_data = super(sale_order, self)._prepare_order_line_procurement(cr,
                 uid, order, line, group_id = group_id, context=context)
+        if not(line.product_id.type== "service" and not line.product_id.auto_create_task):
             proc_data['sale_line_id'] = line.id
-        else:
-            proc_data = False
         return proc_data
 
+    def _check_create_procurement(self, cr, uid, order, line, context=None):
+        create = super(sale_order, self)._check_create_procurement(cr, uid, order, line, context=context)
+        if (line.product_id.type== "service" and not line.product_id.auto_create_task):
+            create = True
+        return create
+
     def _picked_rate(self, cr, uid, ids, name, arg, context=None):
         if not ids:
             return {}