Merge lp:~akretion-team/openobject-addons/mrp-demo-fix-template-ref into lp:openobject-addons

Proposed by Raphaël Valyi - http://www.akretion.com
Status: Rejected
Rejected by: qdp (OpenERP)
Proposed branch: lp:~akretion-team/openobject-addons/mrp-demo-fix-template-ref
Merge into: lp:openobject-addons
Diff against target: 48 lines (+5/-5)
1 file modified
mrp/mrp_demo.xml (+5/-5)
To merge this branch: bzr merge lp:~akretion-team/openobject-addons/mrp-demo-fix-template-ref
Reviewer Review Type Date Requested Status
OpenERP Core Team Pending
Review via email: mp+93231@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Ronald Portier (Therp) (rportier1962) wrote :

I also had to comment out the last record added through this xml file:

    <!-- Run Scheduler -->
    <!-- BUG <function model="procurement.order" name="run_scheduler"/> -->

Otherwise got the error "Couldn't find a bill of material for this product." when trying to install mrp.

Revision history for this message
qdp (OpenERP) (qdp) wrote :

as explained on tyhe bug: this will not occure anymore with the trunk. Let me reject this MP to clear our list of TODO merge props.

Thanks
Quentin

Unmerged revisions

6594. By Raphaël Valyi - http://www.akretion.com

[FIX] mrp: fixes MRP demo data to link supplierinfo data to proper product.template id and stops assuming it's the same as the product.product id as it won't work if some variant product exist.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'mrp/mrp_demo.xml'
2--- mrp/mrp_demo.xml 2011-12-21 15:39:24 +0000
3+++ mrp/mrp_demo.xml 2012-02-15 16:16:19 +0000
4@@ -930,7 +930,7 @@
5 <field name="uom_po_id" ref="product.product_uom_kgm"/>
6 </record>
7 <record id="orange_supplier_asus" model="product.supplierinfo">
8- <field name="product_id" ref="product_orange"/>
9+ <field name="product_id" ref="product_orange_product_template"/>
10 <field name="name" ref="base.res_partner_asus"/>
11 <field name="min_qty">1.0</field>
12 <field name="product_uom" ref="product.product_uom_kgm"/>
13@@ -945,7 +945,7 @@
14 <field name="uom_po_id" ref="product.product_uom_kgm"/>
15 </record>
16 <record id="sugar_supplier_desertic" model="product.supplierinfo">
17- <field name="product_id" ref="product_sugar"/>
18+ <field name="product_id" ref="product_sugar_product_template"/>
19 <field name="name" ref="base.res_partner_desertic_hispafuentes"/>
20 <field name="product_uom" ref="product.product_uom_kgm"/>
21 <field name="min_qty">2.0</field>
22@@ -960,7 +960,7 @@
23 <field name="uom_po_id" ref="product_uom_litre"/>
24 </record>
25 <record id="water_supplier_agrolait" model="product.supplierinfo">
26- <field name="product_id" ref="product_water"/>
27+ <field name="product_id" ref="product_water_product_template"/>
28 <field name="name" ref="base.res_partner_agrolait"/>
29 <field name="product_uom" ref="product_uom_litre"/>
30 <field name="min_qty">2.0</field>
31@@ -1050,7 +1050,7 @@
32 <field name="uom_po_id" ref="product.product_uom_unit"/>
33 </record>
34 <record id="cloth_supplier_maxtor" model="product.supplierinfo">
35- <field name="product_id" ref="product_cloth"/>
36+ <field name="product_id" ref="product_cloth_product_template"/>
37 <field name="name" ref="base.res_partner_maxtor"/>
38 <field name="product_uom" ref="product.product_uom_unit"/>
39 <field name="min_qty">300.0</field>
40@@ -1065,7 +1065,7 @@
41 <field name="uom_po_id" ref="product.product_uom_unit"/>
42 </record>
43 <record id="buttons_supplier_asus" model="product.supplierinfo">
44- <field name="product_id" ref="product_buttons"/>
45+ <field name="product_id" ref="product_buttons_product_template"/>
46 <field name="name" ref="base.res_partner_asus"/>
47 <field name="product_uom" ref="product.product_uom_unit"/>
48 <field name="min_qty">100.0</field>

Subscribers

People subscribed via source and target branches

to all changes: