From 40ddf9586768c15f3d6cb7c8ce16b4e21fd6aaea Mon Sep 17 00:00:00 2001
From: Josse Colpaert <jco@openerp.com>
Date: Wed, 23 Apr 2014 11:40:26 +0200
Subject: [PATCH 1/8] [IMP] Remove procurement creation when confirming
 production without produce line

bzr revid: jco@openerp.com-20140423094026-gfmghsgpxvk5vng9
---
 addons/mrp/mrp.py | 48 -----------------------------------------------
 1 file changed, 48 deletions(-)

diff --git a/addons/mrp/mrp.py b/addons/mrp/mrp.py
index d2ff8553a35..61b6b598d7e 100644
--- a/addons/mrp/mrp.py
+++ b/addons/mrp/mrp.py
@@ -697,8 +697,6 @@ class mrp_production(osv.osv):
         for production in self.browse(cr, uid, ids, context=context):
             if not production.move_created_ids:
                 self._make_production_produce_line(cr, uid, production, context=context)
-                for scheduled in production.product_lines:
-                    self._make_production_line_procurement(cr, uid, scheduled, False, context=context)
 
             if production.move_prod_id and production.move_prod_id.location_id.id != production.location_dest_id.id:
                 move_obj.write(cr, uid, [production.move_prod_id.id],
@@ -960,52 +958,6 @@ class mrp_production(osv.osv):
                 res = True
         return res
 
-    def _make_production_line_procurement(self, cr, uid, production_line, shipment_move_id, context=None):
-        procurement_order = self.pool.get('procurement.order')
-        production = production_line.production_id
-        location_id = production.location_src_id.id
-        date_planned = production.date_planned
-        procurement_name = (production.origin or '').split(':')[0] + ':' + production.name
-        procurement_id = procurement_order.create(cr, uid, {
-                    'name': procurement_name,
-                    'origin': procurement_name,
-                    'date_planned': date_planned,
-                    'product_id': production_line.product_id.id,
-                    'product_qty': production_line.product_qty,
-                    'product_uom': production_line.product_uom.id,
-                    'product_uos_qty': production_line.product_uos and production_line.product_qty or False,
-                    'product_uos': production_line.product_uos and production_line.product_uos.id or False,
-                    'location_id': location_id,
-                    'move_id': shipment_move_id,
-                    'company_id': production.company_id.id,
-        })
-        procurement_order.signal_button_confirm(cr, uid, [procurement_id])
-        return procurement_id
-
-    def _make_production_produce_line(self, cr, uid, production, context=None):
-        stock_move = self.pool.get('stock.move')
-        source_location_id = production.product_id.property_stock_production.id
-        destination_location_id = production.location_dest_id.id
-        data = {
-            'name': production.name,
-            'date': production.date_planned,
-            'product_id': production.product_id.id,
-            'product_qty': production.product_qty,
-            'product_uom': production.product_uom.id,
-            'product_uom_qty': production.product_qty,
-            'product_uos_qty': production.product_uos and production.product_uos_qty or False,
-            'product_uos': production.product_uos and production.product_uos.id or False,
-            'location_id': source_location_id,
-            'location_dest_id': destination_location_id,
-            'move_dest_id': production.move_prod_id.id,
-            'company_id': production.company_id.id,
-            'production_id': production.id,
-            'origin': production.name,
-        }
-        move_id = stock_move.create(cr, uid, data, context=context)
-        #a phantom bom cannot be used in mrp order so it's ok to assume the list returned by action_confirm
-        #is 1 element long, so we can take the first.
-        return stock_move.action_confirm(cr, uid, [move_id], context=context)[0]
 
     def _get_raw_material_procure_method(self, cr, uid, product, context=None):
         '''This method returns the procure_method to use when creating the stock move for the production raw materials'''

From af0468c418194d8139e4a8736260bc46a72e17e4 Mon Sep 17 00:00:00 2001
From: Josse Colpaert <jco@openerp.com>
Date: Wed, 23 Apr 2014 11:48:07 +0200
Subject: [PATCH 2/8] [WIP] Remove picking except state on mrp order

bzr revid: jco@openerp.com-20140423094807-za7855wkx2vlr0di
---
 addons/mrp/mrp.py                       |  8 +-------
 addons/mrp/mrp_view.xml                 | 12 +++++-------
 addons/mrp_operations/mrp_operations.py |  2 +-
 3 files changed, 7 insertions(+), 15 deletions(-)

diff --git a/addons/mrp/mrp.py b/addons/mrp/mrp.py
index 61b6b598d7e..19b1625b105 100644
--- a/addons/mrp/mrp.py
+++ b/addons/mrp/mrp.py
@@ -493,7 +493,7 @@ class mrp_production(osv.osv):
         'workcenter_lines': fields.one2many('mrp.production.workcenter.line', 'production_id', 'Work Centers Utilisation',
             readonly=True, states={'draft': [('readonly', False)]}),
         'state': fields.selection(
-            [('draft', 'New'), ('cancel', 'Cancelled'), ('picking_except', 'Picking Exception'), ('confirmed', 'Awaiting Raw Materials'),
+            [('draft', 'New'), ('cancel', 'Cancelled'), ('confirmed', 'Awaiting Raw Materials'),
                 ('ready', 'Ready to Produce'), ('in_production', 'Production Started'), ('done', 'Done')],
             string='Status', readonly=True,
             track_visibility='onchange',
@@ -614,12 +614,6 @@ class mrp_production(osv.osv):
         }
         return {'value': result}
 
-    def action_picking_except(self, cr, uid, ids):
-        """ Changes the state to Exception.
-        @return: True
-        """
-        self.write(cr, uid, ids, {'state': 'picking_except'})
-        return True
 
     def _action_compute_lines(self, cr, uid, ids, properties=None, context=None):
         """ Compute product_lines and workcenter_lines from BoM structure
diff --git a/addons/mrp/mrp_view.xml b/addons/mrp/mrp_view.xml
index 417d4236b21..84ad35382ec 100644
--- a/addons/mrp/mrp_view.xml
+++ b/addons/mrp/mrp_view.xml
@@ -674,12 +674,10 @@
                     <button name="button_confirm" states="draft" string="Confirm Production" class="oe_highlight"/>
                     <button name="%(act_mrp_product_produce)d" states="ready,in_production" string="Produce" type="action" class="oe_highlight"/>
                     <button name="force_production" states="confirmed" string="Force Reservation" type="object"/>
-                    <button name="force_production" states="picking_except" string="Force Reservation" type="object"/>
                     <button name="button_produce" states="ready" string="Mark as Started"/>
-                    <button name="button_recreate" states="picking_except" string="Recreate Picking"/>
-                    <button name="button_cancel" states="draft,ready,in_production,picking_except" string="Cancel Production"/>
+                    <button name="button_cancel" states="draft,ready,in_production" string="Cancel Production"/>
                     <button name="action_cancel" type="object" states="confirmed" string="Cancel Production"/>
-                    <field name="state" widget="statusbar" statusbar_visible="draft,ready,in_production,done" statusbar_colors='{"picking_except":"red","confirmed":"blue"}'/>
+                    <field name="state" widget="statusbar" statusbar_visible="draft,ready,in_production,done" statusbar_colors='{"confirmed":"blue"}'/>
                 </header>
                 <sheet>
                     <div class="oe_title">
@@ -720,7 +718,7 @@
                             <group>
                                 <group string="Products to Consume">
                                     <field name="move_lines" nolabel="1" options="{'reload_on_button': true}">
-                                        <tree colors="blue:state == 'draft';black:state in ('ready','assigned','in_production');gray:state in ('cancel','done');red:state in ('confirmed','picking_except','waiting')" string="Products to Consume">
+                                        <tree colors="blue:state == 'draft';black:state in ('ready','assigned','in_production');gray:state in ('cancel','done');red:state in ('confirmed','waiting')" string="Products to Consume">
                                             <field name="product_id"/>
                                             <field name="product_qty"  string="Quantity"/>
                                             <field name="product_uom" string="Unit of Measure" groups="product.group_uom"/>
@@ -739,7 +737,7 @@
                                 </group>
                                 <group string="Consumed Products">
                                     <field name="move_lines2" nolabel="1" options="{'reload_on_button': true}">
-                                       <tree colors="red:scrapped==True;blue:state == 'draft';black:state in('picking_except','confirmed','ready','in_production');gray:state == 'cancel' " string="Consumed Products" editable="bottom">
+                                       <tree colors="red:scrapped==True;blue:state == 'draft';black:state in ('confirmed','ready','in_production');gray:state == 'cancel' " string="Consumed Products" editable="bottom">
                                            <field name="product_id" readonly="1"/>
                                            <field name="restrict_lot_id" context="{'product_id': product_id}" groups="stock.group_tracking_lot"/>
                                            <field name="product_qty" readonly="1"/>
@@ -766,7 +764,7 @@
                                 </group>
                                 <group string="Produced Products">
                                     <field name="move_created_ids2" nolabel="1" options="{'reload_on_button': true}">
-                                        <tree colors="red:scrapped==True;blue:state == 'draft';black:state in('picking_except','confirmed','ready','in_production');gray:state in('cancel','done') " string="Finished Products">
+                                        <tree colors="red:scrapped==True;blue:state == 'draft';black:state in('confirmed','ready','in_production');gray:state in('cancel','done') " string="Finished Products">
                                             <field name="product_id" readonly="1"/>
                                             <field name="product_qty" readonly="1"/>
                                             <field name="restrict_lot_id" groups="stock.group_tracking_lot"/>
diff --git a/addons/mrp_operations/mrp_operations.py b/addons/mrp_operations/mrp_operations.py
index b464f7045e6..14acd667866 100644
--- a/addons/mrp_operations/mrp_operations.py
+++ b/addons/mrp_operations/mrp_operations.py
@@ -97,7 +97,7 @@ class mrp_production_workcenter_line(osv.osv):
        'delay': fields.float('Working Hours',help="The elapsed time between operation start and stop in this Work Center",readonly=True),
        'production_state':fields.related('production_id','state',
             type='selection',
-            selection=[('draft','Draft'),('picking_except', 'Picking Exception'),('confirmed','Waiting Goods'),('ready','Ready to Produce'),('in_production','In Production'),('cancel','Canceled'),('done','Done')],
+            selection=[('draft','Draft'),('confirmed','Waiting Goods'),('ready','Ready to Produce'),('in_production','In Production'),('cancel','Canceled'),('done','Done')],
             string='Production Status', readonly=True),
        'product':fields.related('production_id','product_id',type='many2one',relation='product.product',string='Product',
             readonly=True),

From 8015980e33af8ab8f3da58e55663cf23797369f3 Mon Sep 17 00:00:00 2001
From: Josse Colpaert <jco@openerp.com>
Date: Wed, 23 Apr 2014 11:52:58 +0200
Subject: [PATCH 3/8] [FIX] Add makeproductionproduceline

bzr revid: jco@openerp.com-20140423095258-vpy4dz57sus1l8vp
---
 addons/mrp/mrp.py | 27 ++++++++++++++++++++++++++-
 1 file changed, 26 insertions(+), 1 deletion(-)

diff --git a/addons/mrp/mrp.py b/addons/mrp/mrp.py
index 19b1625b105..26409161904 100644
--- a/addons/mrp/mrp.py
+++ b/addons/mrp/mrp.py
@@ -951,7 +951,32 @@ class mrp_production(osv.osv):
             if production.ready_production:
                 res = True
         return res
-
+    
+    
+    def _make_production_produce_line(self, cr, uid, production, context=None):
+        stock_move = self.pool.get('stock.move')
+        source_location_id = production.product_id.property_stock_production.id
+        destination_location_id = production.location_dest_id.id
+        data = {
+            'name': production.name,
+            'date': production.date_planned,
+            'product_id': production.product_id.id,
+            'product_qty': production.product_qty,
+            'product_uom': production.product_uom.id,
+            'product_uom_qty': production.product_qty,
+            'product_uos_qty': production.product_uos and production.product_uos_qty or False,
+            'product_uos': production.product_uos and production.product_uos.id or False,
+            'location_id': source_location_id,
+            'location_dest_id': destination_location_id,
+            'move_dest_id': production.move_prod_id.id,
+            'company_id': production.company_id.id,
+            'production_id': production.id,
+            'origin': production.name,
+        }
+        move_id = stock_move.create(cr, uid, data, context=context)
+        #a phantom bom cannot be used in mrp order so it's ok to assume the list returned by action_confirm
+        #is 1 element long, so we can take the first.
+        return stock_move.action_confirm(cr, uid, [move_id], context=context)[0]
 
     def _get_raw_material_procure_method(self, cr, uid, product, context=None):
         '''This method returns the procure_method to use when creating the stock move for the production raw materials'''

From 7a8222631e62424b09d72f5107f26176aaa26bd8 Mon Sep 17 00:00:00 2001
From: Josse Colpaert <jco@openerp.com>
Date: Wed, 23 Apr 2014 16:17:00 +0200
Subject: [PATCH 4/8] [IMP] Make sure the propagation of moves cancels
 procurements where necessary + manufacture and purchase cancel is not
 propagated by default

bzr revid: jco@openerp.com-20140423141700-60gbuxb4atp1z6ek
---
 addons/mrp/mrp.py        | 9 +++++++++
 addons/mrp/stock.py      | 2 +-
 addons/purchase/stock.py | 2 +-
 addons/stock/stock.py    | 9 +++++++++
 4 files changed, 20 insertions(+), 2 deletions(-)

diff --git a/addons/mrp/mrp.py b/addons/mrp/mrp.py
index 26409161904..03263195f30 100644
--- a/addons/mrp/mrp.py
+++ b/addons/mrp/mrp.py
@@ -679,6 +679,11 @@ class mrp_production(osv.osv):
                 move_obj.action_cancel(cr, uid, [x.id for x in production.move_created_ids])
             move_obj.action_cancel(cr, uid, [x.id for x in production.move_lines])
         self.write(cr, uid, ids, {'state': 'cancel'})
+        # Put related procurements in exception
+        proc_obj = self.pool.get("procurement.order")
+        procs = proc_obj.search(cr, uid, [('production_id', 'in', ids)], context=context)
+        if procs:
+            proc_obj.write(cr, uid, procs, {'state': 'exception'}, context=context)
         return True
 
     def action_ready(self, cr, uid, ids, context=None):
@@ -704,6 +709,10 @@ class mrp_production(osv.osv):
         for production in self.browse(cr, uid, ids):
             self._costs_generate(cr, uid, production)
         write_res = self.write(cr, uid, ids, {'state': 'done', 'date_finished': time.strftime('%Y-%m-%d %H:%M:%S')})
+        # Check related procurements
+        proc_obj = self.pool.get("procurement.order")
+        procs = proc_obj.search(cr, uid, [('production_id', 'in', ids)], context=context)
+        proc_obj.check(cr, uid, procs, context=context)
         return write_res
 
     def test_production_done(self, cr, uid, ids):
diff --git a/addons/mrp/stock.py b/addons/mrp/stock.py
index 11cdffbbb64..1fb28712fad 100644
--- a/addons/mrp/stock.py
+++ b/addons/mrp/stock.py
@@ -227,7 +227,7 @@ class stock_warehouse(osv.osv):
             'route_id': manufacture_route_id,
             'action': 'manufacture',
             'picking_type_id': warehouse.int_type_id.id,
-            'procure_method': 'make_to_order',
+            'propagate': False, 
             'warehouse_id': warehouse.id,
         }
 
diff --git a/addons/purchase/stock.py b/addons/purchase/stock.py
index f14087723f5..4715acdd6b0 100644
--- a/addons/purchase/stock.py
+++ b/addons/purchase/stock.py
@@ -116,7 +116,7 @@ class stock_warehouse(osv.osv):
             'route_id': buy_route_id,
             'action': 'buy',
             'picking_type_id': warehouse.in_type_id.id,
-            'procure_method': 'make_to_order',
+            'propagate': False, 
             'warehouse_id': warehouse.id,
         }
 
diff --git a/addons/stock/stock.py b/addons/stock/stock.py
index 3cfb10ac05d..9050a4bd173 100644
--- a/addons/stock/stock.py
+++ b/addons/stock/stock.py
@@ -2144,6 +2144,14 @@ class stock_move(osv.osv):
                 #cancel chained moves
                 if move.propagate:
                     self.action_cancel(cr, uid, [move.move_dest_id.id], context=context)
+                    # If we have a long chain of moves to be cancelled, it is easier for the user to handle 
+                    # only the last procurement which will go into exception, instead of all procurements
+                    # along the chain going into exception
+                    if move.procurement_id:
+                        proc = move.procurement_id
+                        if all([x.state == 'cancel' for x in proc.move_orig_ids]):
+                            procurement_obj.write(cr, uid, [proc.id], {'state': 'cancel'})
+                        
                 elif move.move_dest_id.state == 'waiting':
                     self.write(cr, uid, [move.move_dest_id.id], {'state': 'confirmed'}, context=context)
         return self.write(cr, uid, ids, {'state': 'cancel', 'move_dest_id': False}, context=context)
@@ -2360,6 +2368,7 @@ class stock_move(osv.osv):
             'restrict_lot_id': restrict_lot_id,
             'restrict_partner_id': restrict_partner_id,
             'split_from': move.id,
+            'move_dest_id': move.move_dest_id.id,
         }
         if context.get('source_location_id'):
             defaults['location_id'] = context['source_location_id']

From 2b3ec22a262808ec604775db26981f9b8fbb61d5 Mon Sep 17 00:00:00 2001
From: Josse Colpaert <jco@openerp.com>
Date: Wed, 23 Apr 2014 17:16:00 +0200
Subject: [PATCH 5/8] [FIX] Correct the fact that moves have not been changed
 to cancel yet

bzr revid: jco@openerp.com-20140423151600-j9i3jhiyy3nzshvl
---
 addons/stock/stock.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/addons/stock/stock.py b/addons/stock/stock.py
index 9050a4bd173..17a25eb879c 100644
--- a/addons/stock/stock.py
+++ b/addons/stock/stock.py
@@ -2146,10 +2146,10 @@ class stock_move(osv.osv):
                     self.action_cancel(cr, uid, [move.move_dest_id.id], context=context)
                     # If we have a long chain of moves to be cancelled, it is easier for the user to handle 
                     # only the last procurement which will go into exception, instead of all procurements
-                    # along the chain going into exception
+                    # along the chain going into exception.  We need to check if there are no split moves not cancelled however
                     if move.procurement_id:
                         proc = move.procurement_id
-                        if all([x.state == 'cancel' for x in proc.move_orig_ids]):
+                        if all([x.state == 'cancel' for x in proc.move_ids if x.id != move.id]):
                             procurement_obj.write(cr, uid, [proc.id], {'state': 'cancel'})
                         
                 elif move.move_dest_id.state == 'waiting':

From 7e733fcfaae559290ebd7b4371d67e38da0f5ee8 Mon Sep 17 00:00:00 2001
From: Josse Colpaert <jco@openerp.com>
Date: Thu, 24 Apr 2014 11:17:31 +0200
Subject: [PATCH 6/8] [IMP] Do not check rule when cancelling procurement in
 propagation, works through stock move

bzr revid: jco@openerp.com-20140424091731-3ptkt7qr26i9ofnf
---
 addons/stock/procurement.py | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/addons/stock/procurement.py b/addons/stock/procurement.py
index 0214eef6da1..88a02bc8efc 100644
--- a/addons/stock/procurement.py
+++ b/addons/stock/procurement.py
@@ -99,8 +99,7 @@ class procurement_order(osv.osv):
         #set the context for the propagation of the procurement cancelation
         ctx['cancel_procurement'] = True
         for procurement in self.browse(cr, uid, to_cancel_ids, context=ctx):
-            if procurement.rule_id and procurement.rule_id.propagate:
-                self.propagate_cancel(cr, uid, procurement, context=ctx)
+            self.propagate_cancel(cr, uid, procurement, context=ctx)
         return super(procurement_order, self).cancel(cr, uid, to_cancel_ids, context=ctx)
 
     def _find_parent_locations(self, cr, uid, procurement, context=None):

From bf5585851a1ba47ad35fb9c7d0ca79c8afaad9fe Mon Sep 17 00:00:00 2001
From: Josse Colpaert <jco@openerp.com>
Date: Thu, 24 Apr 2014 11:36:54 +0200
Subject: [PATCH 7/8] [IMP] Make sure tests pass without procurement_jit
 installation

bzr revid: jco@openerp.com-20140424093654-r04xvcp544du1lkj
---
 addons/sale_mrp/test/cancellation_propagated.yml | 9 +++++++--
 addons/sale_mrp/test/sale_mrp.yml                | 6 +++---
 2 files changed, 10 insertions(+), 5 deletions(-)

diff --git a/addons/sale_mrp/test/cancellation_propagated.yml b/addons/sale_mrp/test/cancellation_propagated.yml
index 770601bac6d..0005e15db53 100644
--- a/addons/sale_mrp/test/cancellation_propagated.yml
+++ b/addons/sale_mrp/test/cancellation_propagated.yml
@@ -51,12 +51,17 @@
   Confirm sales order
 - 
   !workflow {model: sale.order, action: order_confirm, ref: sale_order_product_manu}
+-
+  I run scheduler.
+-
+  !python {model: procurement.order}: |
+    self.run_scheduler(cr, uid)
 - 
   Check the propagation when we cancel the main procurement 
     * Retrieve related procurements and check that there are all running
     * Check that the purchase order has been well created
     * Cancel the main procurement
-    * Check that all procurements related and the purchase order has been well cancelled   
+    * Check that all procurements related and the purchase order has been well cancelled
 -
   !python {model: procurement.order}: |
     # Retrieve related procu
@@ -65,7 +70,7 @@
     assert len(procu_ids)>0, 'No procurements are found for sale order "%s" (with id : %d)' %(so.name, so.id)
 
     # Check that all procurements are running 
-    for procu in self.browse(cr, uid, procu_ids, context=context):        
+    for procu in self.browse(cr, uid, procu_ids, context=context):
         assert procu.state == u'running', 'Procurement with id %d should be "running" but is with a state : %s!' %(procu.id, procu.state)
 
     # Check that one production order exist
diff --git a/addons/sale_mrp/test/sale_mrp.yml b/addons/sale_mrp/test/sale_mrp.yml
index 82c8b6e8b12..85e25b502f3 100644
--- a/addons/sale_mrp/test/sale_mrp.yml
+++ b/addons/sale_mrp/test/sale_mrp.yml
@@ -100,7 +100,7 @@
   !python {model: procurement.order}: |
     sale_order_obj = self.pool.get('sale.order')
     so = sale_order_obj.browse(cr, uid, ref("sale_order_so0"))
-    proc_ids = self.search(cr, uid, [('origin','=',so.name)])
+    proc_ids = self.search(cr, uid, [('origin','like',so.name)])
     self.run(cr, uid, proc_ids)
 -
   I verify that a procurement state is "running"
@@ -108,8 +108,8 @@
   !python {model: procurement.order}: |
     sale_order_obj = self.pool.get('sale.order')
     so = sale_order_obj.browse(cr, uid, ref("sale_order_so0"))
-    proc_ids = self.search(cr, uid, [('origin','like',so.name), ('state','=','running')])
-    # Check that all procurement are running 
+    proc_ids = self.search(cr, uid, [('origin','like',so.name)])
+    # Check that all procurement are running
     for procu in self.browse(cr,uid,proc_ids,context=context):        
         assert procu.state == u'running', 'Procurement with id %d should be with a state "running" but is with a state : %s!' %(procu.id,procu.state)
 -

From d958ce968ee0bb4126e4a31d3e605d5a84d0e7b3 Mon Sep 17 00:00:00 2001
From: Josse Colpaert <jco@openerp.com>
Date: Thu, 24 Apr 2014 11:53:47 +0200
Subject: [PATCH 8/8] [IMP] Make sure tests of stock_dropshipping also pass
 without procurement_jit

bzr revid: jco@openerp.com-20140424095347-xcqj0m4oxilroxhk
---
 addons/stock_dropshipping/test/cancellation_propagated.yml | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/addons/stock_dropshipping/test/cancellation_propagated.yml b/addons/stock_dropshipping/test/cancellation_propagated.yml
index cb68b582a6a..71f5e75385d 100644
--- a/addons/stock_dropshipping/test/cancellation_propagated.yml
+++ b/addons/stock_dropshipping/test/cancellation_propagated.yml
@@ -40,6 +40,11 @@
   Confirm the sale order
 - 
   !workflow {model: sale.order, action: order_confirm, ref: sale_order_product_mto}
+-
+  I run scheduler.
+-
+  !python {model: procurement.order}: |
+    self.run_scheduler(cr, uid)
 - 
   Check the propagation when we cancel the main procurement 
     * Retrieve related procurements and check that there are all running
@@ -64,7 +69,7 @@
     # Cancel the main procurement
     main_procu_id = self.search(cr, uid, [('origin', '=', so.name)])
     assert len(main_procu_id) == 1, 'Main procurement not identified !'
-    self.cancel(cr, uid, main_procu_id, context=context)    
+    self.cancel(cr, uid, main_procu_id, context=context)
     assert self.browse(cr, uid, main_procu_id[0]).state == u'cancel', 'Main procurement should be cancelled !'
     
     # Check that all procurements related are cancelled