From c5a6c50c1d48e3eb4b5a40a0cdf2cfe798a2b487 Mon Sep 17 00:00:00 2001
From: mtr <mtr@mtr>
Date: Tue, 5 Oct 2010 15:30:54 +0530
Subject: [PATCH] [IMP] sale,sale_journal:-improvements in the security access
 rights

bzr revid: mtr@mtr-20101005100054-m64tzbfrjlz3i2d1
---
 addons/sale/sale_view.xml                 | 7 ++++---
 addons/sale/security/sale_security.xml    | 4 ++++
 addons/sale_journal/sale_journal_view.xml | 4 ++--
 3 files changed, 10 insertions(+), 5 deletions(-)

diff --git a/addons/sale/sale_view.xml b/addons/sale/sale_view.xml
index db1818602ce..b30aacaa1fa 100644
--- a/addons/sale/sale_view.xml
+++ b/addons/sale/sale_view.xml
@@ -1,6 +1,7 @@
 <?xml version="1.0" encoding="utf-8"?>
 <openerp>
     <data>
+            
         <menuitem icon="terp-partner" id="base.menu_base_partner" name="Sales" sequence="0" groups="base.group_sale_salesman,base.group_sale_manager,base.group_system,base.group_partner_manager"/>
         <menuitem id="base.menu_sales" name="Sales" parent="base.menu_base_partner" sequence="1"/>
 
@@ -48,7 +49,7 @@
 
         <menuitem action="action_shop_form" id="menu_action_shop_form"
             parent="product.prod_config_main" sequence="0"
-            groups="base.group_extended"/>
+            groups="base.group_extended,base.group_sale_manager"/>
 
         <record id="view_sale_order_calendar" model="ir.ui.view">
             <field name="name">sale.order.calendar</field>
@@ -293,7 +294,7 @@ orders, purchases, etc. It support several invoicing methods according
 to your configuration: from the sales order, from the pickings, etc.
             </field>
         </record>
-        <menuitem action="action_order_form" id="menu_sale_order" parent="base.menu_sales" sequence="3"/>
+        <menuitem action="action_order_form" id="menu_sale_order" parent="base.menu_sales" sequence="3" groups="base.group_sale_salesman,base.group_sale_manager"/>
 
         <record id="action_order_tree2" model="ir.actions.act_window">
             <field name="name">Sales in Exception</field>
@@ -517,7 +518,7 @@ to your configuration: from the sales order, from the pickings, etc.
         groups="base.group_extended"/>
 
         <menuitem id="menu_invoiced" name="Billing" parent="base.menu_base_partner" sequence="5"/>
-        <menuitem action="action_order_line_tree2" id="menu_invoicing_sales_order_lines" parent="menu_invoiced" sequence="2"/>
+        <menuitem action="action_order_line_tree2" id="menu_invoicing_sales_order_lines" parent="menu_invoiced" sequence="2" groups="base.group_sale_salesman,base.group_sale_manager"/>
         <!-- configartion view -->
 
         <record id="view_config_picking_policy" model="ir.ui.view">
diff --git a/addons/sale/security/sale_security.xml b/addons/sale/security/sale_security.xml
index 19da6a6f81f..36a8fc15118 100644
--- a/addons/sale/security/sale_security.xml
+++ b/addons/sale/security/sale_security.xml
@@ -12,6 +12,10 @@
     <record model="ir.ui.menu" id="base.menu_base_partner">
         <field eval="[(4, ref('base.group_sale_salesman'))]" name="groups_id"/>
     </record>
+    
+    <record model="ir.ui.menu" id="base.menu_base_config">
+        <field eval="[(4, ref('base.group_sale_manager'))]" name="groups_id"/>
+    </record>
 
     <record model="ir.rule" id="sale_order_comp_rule">
         <field name="name">Sale Order multi-company</field>
diff --git a/addons/sale_journal/sale_journal_view.xml b/addons/sale_journal/sale_journal_view.xml
index 8a8eb0d2c1e..8c9103d25f0 100644
--- a/addons/sale_journal/sale_journal_view.xml
+++ b/addons/sale_journal/sale_journal_view.xml
@@ -1,9 +1,9 @@
 <openerp>
    <data>
         <menuitem id="base.menu_base_config" name="Configuration" parent="base.menu_base_partner" sequence="30"
-            groups="base.group_system,base.group_sale_salesman"/>
+            groups="base.group_sale_salesman,base.group_system,base.group_sale_manager"/>
         <menuitem id="menu_sale_journal_config" name="Sale Journals"
-            parent="base.menu_base_config" sequence="5" groups="base.group_system,base.group_sale_salesman"/>
+            parent="base.menu_base_config" sequence="5" groups="base.group_sale_salesman"/>
 
    <!--  sale journal invoice Type's Form  view -->