openerp-community-reviewer team mailing list archive

[Stéphane Bidoul (Acsone) <stephane.bidoul@xxxxxxxxx>]

Stéphane Bidoul (Acsone) has proposed merging lp:~sbi/ocb-addons/7.0-opw-595596-han into lp:ocb-addons.

Requested reviews:
  Stéphane Bidoul (Acsone) (sbi): code review and test
  OpenERP Community Backports Team (ocb)
Related bugs:
  Bug #1203108 in OpenERP Community Backports (Addons): "[7.0][stock] Position of the flag active in Product form view"
  https://bugs.launchpad.net/ocb-addons/+bug/1203108

For more details, see:
https://code.launchpad.net/~sbi/ocb-addons/7.0-opw-595596-han/+merge/189049

Automatically derived from https://code.launchpad.net/~openerp-dev/openobject-addons/7.0-opw-595596-han for https://code.launchpad.net/~openerp/openobject-addons/7.0. Below is a copy of the original description.

Hello,

Relocated the active field into Inventory/ Status from Procurement/ Delay in product form.
It's current position is confusing and let the user believe the active flag is related to the delay instead of the product itself.

With reference of maintenance case: 595596.

Thanks
-- 
https://code.launchpad.net/~sbi/ocb-addons/7.0-opw-595596-han/+merge/189049
Your team OpenERP Community Backports Team is requested to review the proposed merge of lp:~sbi/ocb-addons/7.0-opw-595596-han into lp:ocb-addons.

=== modified file 'stock/product_view.xml'
--- stock/product_view.xml	2012-10-23 16:05:04 +0000
+++ stock/product_view.xml	2013-10-03 11:26:40 +0000
@@ -74,7 +74,6 @@
                         <div attrs="{'invisible':[('type','=','service')]}">
                             <field name="produce_delay" class="oe_inline"/> days
                         </div>
-                        <field name="active"/>
                     </group>
                 </group>
                 <xpath expr="//group[@string='Sale Conditions']" position="inside">
@@ -83,6 +82,9 @@
                             <field name="sale_delay" class="oe_inline"/> days
                         </div>
                 </xpath>
+                <group name="status" position="inside" version="7.0">
+                    <field name="active"/>
+                </group>
                 <group name="status" position="before" version="7.0">
                     <group string="Stock and Expected Variations" attrs="{'invisible': [('type', '=', 'service')]}" groups="base.group_user">
                         <label for="qty_available"/>