Merge lp:~camptocamp/openobject-addons/7.0-fix-form_ref_context_lp1214801-afe into lp:openobject-addons/7.0

[Alexandre Fayolle - camptocamp]

Use the qualified name when specifying sub views

[Olivier Dony (Odoo) (odo-openerp)]

Looks fine, but since the list view "stock.view_move_picking_tree" is configured to have the highest priority, it's in fact redundant to force it every time, and the `tree_view_ref` elements could be dropped altogether.

The `form_view_ref` element will on the other hand cause the stock.move view to switch to the simplified version meant for use inside pickings, which was the intended effect.

Thanks!

review: Approve

Unmerged revisions

9372. By Alexandre Fayolle - camptocamp on 2013-08-21

[FIX] stock: use qualified xml id in form_view_ref and tree_view_ref context values

=== modified file 'stock/stock_view.xml'
--- stock/stock_view.xml	2013-08-21 08:53:04 +0000
+++ stock/stock_view.xml	2013-08-21 10:03:27 +0000
@@ -776,7 +776,7 @@
                     </group>
                     <notebook>
                         <page string="Products">
-                            <field name="move_lines" context="{'address_in_id': partner_id, 'form_view_ref':'view_move_picking_form', 'tree_view_ref':'view_move_picking_tree', 'picking_type': 'internal'}" options='{"reload_on_button": true}'/>
+                            <field name="move_lines" context="{'address_in_id': partner_id, 'form_view_ref':'stock.view_move_picking_form', 'tree_view_ref':'stock.view_move_picking_tree', 'picking_type': 'internal'}" options='{"reload_on_button": true}'/>
                             <field name="note" placeholder="Add an internal note..." class="oe_inline"/>
                         </page>
                         <page string="Additional Info">
@@ -911,7 +911,7 @@
                     <field name="partner_id" on_change="onchange_partner_in(partner_id)" string="Customer" domain="[('customer','=',True)]" />
                 </xpath>
                 <xpath expr="//field[@name='move_lines']" position="replace">
-                    <field name="move_lines" context="{'address_out_id': partner_id, 'picking_type': 'out', 'form_view_ref':'view_move_picking_form', 'tree_view_ref':'view_move_picking_tree'}" options='{"reload_on_button": true}'/>
+                    <field name="move_lines" context="{'address_out_id': partner_id, 'picking_type': 'out', 'form_view_ref':'stock.view_move_picking_form', 'tree_view_ref':'stock.view_move_picking_tree'}" options='{"reload_on_button": true}'/>
                 </xpath>
                 <xpath expr="/form/sheet" position="after">
                     <div class="oe_chatter">
@@ -1038,7 +1038,7 @@
                     <field name="partner_id" on_change="onchange_partner_in(partner_id)" string="Supplier" domain="[('supplier','=',True)]" />
                 </xpath>
                 <xpath expr="//field[@name='move_lines']" position="replace">
-                    <field name="move_lines" context="{'address_in_id': partner_id, 'picking_type': 'in', 'form_view_ref':'view_move_picking_form', 'tree_view_ref':'view_move_picking_tree'}" options='{"reload_on_button": true}'/>
+                    <field name="move_lines" context="{'address_in_id': partner_id, 'picking_type': 'in', 'form_view_ref':'stock.view_move_picking_form', 'tree_view_ref':'stock.view_move_picking_tree'}" options='{"reload_on_button": true}'/>
                 </xpath>
                 <xpath expr="/form/sheet" position="after">
                     <div class="oe_chatter">