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

Proposed by Alexandre Fayolle - camptocamp
Status: Needs review
Proposed branch: lp:~camptocamp/openobject-addons/7.0-fix-form_ref_context_lp1214801-afe
Merge into: lp:openobject-addons/7.0
Diff against target: 30 lines (+3/-3)
1 file modified
stock/stock_view.xml (+3/-3)
To merge this branch: bzr merge lp:~camptocamp/openobject-addons/7.0-fix-form_ref_context_lp1214801-afe
Reviewer Review Type Date Requested Status
Olivier Dony (Odoo) Approve
Review via email: mp+181237@code.launchpad.net

Description of the change

Use the qualified name when specifying sub views

To post a comment you must log in.
Revision history for this message
Olivier Dony (Odoo) (odo-openerp) wrote :

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

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

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== 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 @@
776 </group>776 </group>
777 <notebook>777 <notebook>
778 <page string="Products">778 <page string="Products">
779 <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}'/>779 <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}'/>
780 <field name="note" placeholder="Add an internal note..." class="oe_inline"/>780 <field name="note" placeholder="Add an internal note..." class="oe_inline"/>
781 </page>781 </page>
782 <page string="Additional Info">782 <page string="Additional Info">
@@ -911,7 +911,7 @@
911 <field name="partner_id" on_change="onchange_partner_in(partner_id)" string="Customer" domain="[('customer','=',True)]" />911 <field name="partner_id" on_change="onchange_partner_in(partner_id)" string="Customer" domain="[('customer','=',True)]" />
912 </xpath>912 </xpath>
913 <xpath expr="//field[@name='move_lines']" position="replace">913 <xpath expr="//field[@name='move_lines']" position="replace">
914 <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}'/>914 <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}'/>
915 </xpath>915 </xpath>
916 <xpath expr="/form/sheet" position="after">916 <xpath expr="/form/sheet" position="after">
917 <div class="oe_chatter">917 <div class="oe_chatter">
@@ -1038,7 +1038,7 @@
1038 <field name="partner_id" on_change="onchange_partner_in(partner_id)" string="Supplier" domain="[('supplier','=',True)]" />1038 <field name="partner_id" on_change="onchange_partner_in(partner_id)" string="Supplier" domain="[('supplier','=',True)]" />
1039 </xpath>1039 </xpath>
1040 <xpath expr="//field[@name='move_lines']" position="replace">1040 <xpath expr="//field[@name='move_lines']" position="replace">
1041 <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}'/>1041 <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}'/>
1042 </xpath>1042 </xpath>
1043 <xpath expr="/form/sheet" position="after">1043 <xpath expr="/form/sheet" position="after">
1044 <div class="oe_chatter">1044 <div class="oe_chatter">