Merge lp:~openerp-dev/openobject-addons/trunk-bug-1297736-hsa into lp:openobject-addons

[Hardik Sanchawat (OpenERP)]

Hello,

I fixed this issue in trunk.

Issue Scenario :
When I try to delete the Bill of meterial I can successfully delete it even I have the manufacturing order for that same product with any state. It should not be allow to delete it .
After that If I have any manufacturing order in "ready to produce" state and I deleted the BOM then after If I try to produce the product it doesn't allow me beccuase bom_id is required field on mrp.production.

I fixed this bug lp:1297736

Thanks,
-HSA

[lp:~openerp-dev/openobject-addons/trunk-bug-1297736-hsa]

9363. By Hardik Sanchawat (OpenERP) on 2014-04-18

[IMP]update the code

Unmerged revisions

9363. By Hardik Sanchawat (OpenERP) on 2014-04-18

[IMP]update the code

9362. By Hardik Sanchawat (OpenERP) on 2014-04-18

[FIX] MRP: Deleting Bill of Material when there are Manufacturing orders

=== modified file 'mrp/mrp.py'
--- mrp/mrp.py	2014-03-11 14:52:05 +0000
+++ mrp/mrp.py	2014-04-18 09:54:27 +0000
@@ -376,7 +376,12 @@
         bom_data = self.read(cr, uid, id, [], context=context)
         default.update(name=_("%s (copy)") % (bom_data['name']), bom_id=False)
         return super(mrp_bom, self).copy_data(cr, uid, id, default, context=context)
-
+    
+    def unlink(self, cr, uid, ids, context=None):        
+        product_ids = self.pool.get('mrp.production').search(cr, uid, [('bom_id','in',ids)])
+        if product_ids:
+            raise osv.except_osv(_('Warning!'),_("You can not delete BOM because already assigned in Manufacturing Order."))
+        return super(mrp_bom,self).unlink(cr, uid, ids, context)
 
 def rounding(f, r):
     # TODO for trunk: log deprecation warning