From df0ba6c7e8f17c24111b27769951d0d68e545312 Mon Sep 17 00:00:00 2001
From: Olivier Dony <odo@openerp.com>
Date: Tue, 17 Dec 2013 11:43:01 +0100
Subject: [PATCH] [FIX] project_mrp: local copy of the "Procurement" process
 node to avoid errors during updates when sale_stock is not installed

The various attempts to fix this error without duplicating the node
have all caused other errors. Having a duplicate process node has
no consequence on business operations, so it is an acceptable
workaround until the node can be moved to the sale module.

bzr revid: odo@openerp.com-20131217104301-t8fsf28jgjqq9q7k
---
 addons/project_mrp/process/project_mrp_process.xml | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/addons/project_mrp/process/project_mrp_process.xml b/addons/project_mrp/process/project_mrp_process.xml
index 59915715999..a6aaccedbbc 100644
--- a/addons/project_mrp/process/project_mrp_process.xml
+++ b/addons/project_mrp/process/project_mrp_process.xml
@@ -46,14 +46,15 @@
         Process Transition
     -->
 
-        <!-- This "foreign" process node is duplicated here from the corresponding sale_stock node because
+        <!-- This process node is duplicated here from the corresponding sale_stock node because
              `project_mrp` implements a basic procurement system for services without actually using the
-             full-fledged procurement process from sale_stock, and without the dependency. So it stil
-             represents a "procurement system".
-             TODO: To cleanup this invalid foreign external ID, the node should probably be moved to the
+             full-fledged procurement process from sale_stock, and without the dependency. So it still
+             represents a "procurement system". Sharing the external ID causes other problems, so
+             we duplicate it instead.
+             TODO: To cleanup this duplicate node, it should probably be moved to the
              `sale` module directly, and removed from both `sale_stock` and `project_mrp`.
         --> 
-        <record id="sale_stock.process_node_saleprocurement0" model="process.node">
+        <record id="process_node_saleprocurement0" model="process.node">
             <field name="menu_id" ref="procurement.menu_stock_procurement_action"/>
             <field name="model_id" ref="procurement.model_procurement_order"/>
             <field name="kind">subflow</field>
@@ -68,7 +69,7 @@
             <field eval="&quot;&quot;&quot;Procurement Task&quot;&quot;&quot;" name="name"/>
             <field eval="&quot;&quot;&quot;if product type is 'service' then it creates the task.&quot;&quot;&quot;" name="note"/>
             <field model="process.node" name="target_node_id" ref="process_node_procuretasktask0"/>
-            <field model="process.node" name="source_node_id" ref="sale_stock.process_node_saleprocurement0"/>
+            <field model="process.node" name="source_node_id" ref="process_node_saleprocurement0"/>
         </record>
 
         <record id="process_transition_createtask0" model="process.transition">