From de4ab6739057916ea0770082f1cd3b4bf876a9ff Mon Sep 17 00:00:00 2001
From: "Jagdish Panchal (Open ERP)" <jap@tinyerp.com>
Date: Wed, 25 Apr 2012 18:16:26 +0530
Subject: [PATCH] [IMP] sale,stock:apply group_locations on UoM Categories and
 Unit of Measure Categories and also improve the code in sale/res_config.py

bzr revid: jap@tinyerp.com-20120425124626-xys9jf1o002bl3km
---
 addons/product/product_view.xml | 6 +++---
 addons/sale/res_config.py       | 2 +-
 addons/sale/res_config_view.xml | 2 +-
 addons/stock/stock_view.xml     | 4 ++--
 4 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/addons/product/product_view.xml b/addons/product/product_view.xml
index 82dfcb46ec7..6ee597dd537 100644
--- a/addons/product/product_view.xml
+++ b/addons/product/product_view.xml
@@ -440,8 +440,8 @@
             <field name="view_id" ref="product_uom_tree_view"/>
             <field name="help">Create and manage the units of measure you want to be used in your system. You can define a conversion rate between several Units of Measure within the same category.</field>
         </record>
-        <menuitem id="next_id_16" name="Units of Measure" parent="prod_config_main" sequence="30" groups="product.group_uom"/>
-        <menuitem action="product_uom_form_action" id="menu_product_uom_form_action" parent="base.menu_base_config" sequence="30"/>
+        <menuitem id="next_id_16" name="Units of Measure" parent="prod_config_main" sequence="30" groups="product.group_uom,stock.group_locations"/>
+        <menuitem action="product_uom_form_action" id="menu_product_uom_form_action" parent="base.menu_base_config" sequence="30" groups="stock.group_locations"/>
 
         <record id="product_uom_categ_form_view" model="ir.ui.view">
             <field name="name">product.uom.categ.form</field>
@@ -461,7 +461,7 @@
             <field name="view_mode">tree,form</field>
             <field name="help">Create and manage the units of measure categories you want to be used in your system. If several units of measure are in the same category, they can be converted to each other. For example, in the unit of measure category "Time", you will have the following UoM: Hours, Days.</field>
         </record>
-        <menuitem action="product_uom_categ_form_action" id="menu_product_uom_categ_form_action" parent="base.menu_base_config" sequence="25" groups="product.group_uom"/>
+        <menuitem action="product_uom_categ_form_action" id="menu_product_uom_categ_form_action" parent="base.menu_base_config" sequence="25" groups="product.group_uom,stock.group_locations"/>
 
         <record id="product_ul_form_view" model="ir.ui.view">
             <field name="name">product.ul.form.view</field>
diff --git a/addons/sale/res_config.py b/addons/sale/res_config.py
index cb08443d41e..3d2f63790c5 100644
--- a/addons/sale/res_config.py
+++ b/addons/sale/res_config.py
@@ -50,7 +50,7 @@ class sale_configuration(osv.osv_memory):
             It installs the account_analytic_analysis module."""),
         'default_order_policy': fields.selection(
             [('manual', 'Invoice Based on Sales Orders'), ('picking', 'Invoice Based on Deliveries')],
-            'Default Method', required=True, default_model='sale.order',
+            'Default Method', default_model='sale.order',
             help="You can generate invoices based on sales orders or based on shippings."),
         'module_delivery': fields.boolean('Charge Shipping Cost',
             help ="""Allows you to add delivery methods in sale orders and delivery orders.
diff --git a/addons/sale/res_config_view.xml b/addons/sale/res_config_view.xml
index ac943d9a18c..1341a446162 100644
--- a/addons/sale/res_config_view.xml
+++ b/addons/sale/res_config_view.xml
@@ -11,7 +11,7 @@
                 <group name="config_sale" position="after">
                     <separator string="Invoicing Policy" colspan="4"/>
                     <field name="group_invoice_so_lines" on_change="onchange_invoice_methods(group_invoice_so_lines, group_invoice_deli_orders)"/>
-                    <field name="default_order_policy" attrs="{'invisible':['|',('group_invoice_so_lines','=',False),('group_invoice_deli_orders','=',False)]}"/>
+                    <field name="default_order_policy" attrs="{'invisible':['|',('group_invoice_so_lines','=',False),('group_invoice_deli_orders','=',False)],'required': ['|',('group_invoice_so_lines','=',True),('group_invoice_deli_orders','=',True)]}"/>
                     <newline/>
                     <field name="group_invoice_deli_orders" on_change="onchange_invoice_methods(group_invoice_so_lines, group_invoice_deli_orders)"/>
                     <field name="module_delivery" attrs="{'invisible':[('group_invoice_so_lines','=',False), ('group_invoice_deli_orders','=',False)]}"/>
diff --git a/addons/stock/stock_view.xml b/addons/stock/stock_view.xml
index b37bbd063e6..52ca0299b2a 100644
--- a/addons/stock/stock_view.xml
+++ b/addons/stock/stock_view.xml
@@ -29,10 +29,10 @@
             parent="stock.menu_product_in_config_stock"  sequence="35" groups="product.group_uom"/>
          <menuitem
              action="product.product_uom_categ_form_action" id="menu_stock_uom_categ_form_action"
-             parent="menu_stock_configuration" sequence="30"/>
+             parent="menu_stock_configuration" sequence="30" groups="stock.group_locations"/>
          <menuitem
               action="product.product_uom_form_action" id="menu_stock_uom_form_action"
-              parent="menu_stock_configuration" sequence="35"/>
+              parent="menu_stock_configuration" sequence="35" groups="stock.group_locations"/>
 
         <record id="stock_inventory_line_tree" model="ir.ui.view">
             <field name="name">stock.inventory.line.tree</field>