From 086f180c8098bc8d4387172d0b6e8e46adcf7fa8 Mon Sep 17 00:00:00 2001
From: "AMP (OpenERP)" <amp@tinyerp.com>
Date: Mon, 26 Jul 2010 10:45:51 +0530
Subject: [PATCH] [MOD] usability improvement in access rights

bzr revid: amp@tinyerp.com-20100726051551-i1udiuboxtn4vfve
---
 addons/account/security/ir.model.access.csv        | 1 +
 addons/account_report/account_view.xml             | 2 +-
 addons/account_report/security/ir.model.access.csv | 1 +
 addons/delivery/delivery_view.xml                  | 2 +-
 addons/delivery/security/ir.model.access.csv       | 2 +-
 addons/point_of_sale/pos_view.xml                  | 2 +-
 addons/point_of_sale/security/ir.model.access.csv  | 2 +-
 7 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/addons/account/security/ir.model.access.csv b/addons/account/security/ir.model.access.csv
index 0203f1b57a4..63c096d0215 100644
--- a/addons/account/security/ir.model.access.csv
+++ b/addons/account/security/ir.model.access.csv
@@ -91,3 +91,4 @@
 "access_account_account_report_manager","account.account.report","model_account_account_report","account.group_account_manager",1,0,0,0
 "access_account_entries_report_manager","account.entries.report","model_account_entries_report","account.group_account_manager",1,1,1,1
 "access_analytic_entries_report_manager","analytic.entries.report","model_analytic_entries_report","account.group_account_manager",1,1,1,1
+"access_res_partner_group_account_manager","res_partner group_account_manager","model_res_partner","account.group_account_manager",1,1,1,1
diff --git a/addons/account_report/account_view.xml b/addons/account_report/account_view.xml
index 82fb48e5e8b..a935afc3617 100644
--- a/addons/account_report/account_view.xml
+++ b/addons/account_report/account_view.xml
@@ -91,7 +91,7 @@
             <field name="view_id" ref="view_account_report_tree_simple"/>
         </record>
 
-        <menuitem action="action_account_report_tree" id="menu_action_account_report_tree_define" parent="account.menu_finance_configuration" sequence="20"/>
+        <menuitem action="action_account_report_tree" id="menu_action_account_report_tree_define" parent="account.menu_finance_configuration" sequence="20" groups="base.group_system"/>
 
         <record id="view_account_report_tree" model="ir.ui.view">
             <field name="name">account.report.report.tree</field>
diff --git a/addons/account_report/security/ir.model.access.csv b/addons/account_report/security/ir.model.access.csv
index 1abb810f073..d404998221e 100644
--- a/addons/account_report/security/ir.model.access.csv
+++ b/addons/account_report/security/ir.model.access.csv
@@ -1,3 +1,4 @@
 "id","name","model_id:id","group_id:id","perm_read","perm_write","perm_create","perm_unlink"
 "access_account_report_report","account.report.report","model_account_report_report","base.group_system",1,1,1,1
 "access_account_report_history","account.report.history","model_account_report_history","account.group_account_manager",1,0,0,0
+"access_account_report_report","account.report.report","model_account_report_report","account.group_account_user",1,1,0,0
diff --git a/addons/delivery/delivery_view.xml b/addons/delivery/delivery_view.xml
index 704442c778e..fdef01e2ef2 100644
--- a/addons/delivery/delivery_view.xml
+++ b/addons/delivery/delivery_view.xml
@@ -36,7 +36,7 @@
             <field name="view_type">form</field>
             <field name="view_mode">tree,form</field>
         </record>
-        <menuitem action="action_delivery_carrier_form" id="menu_action_delivery_carrier_form" parent="menu_delivery"/>
+        <menuitem action="action_delivery_carrier_form" id="menu_action_delivery_carrier_form" parent="menu_delivery" groups="base.group_system"/>
 
         <!-- Delivery Grids -->
         <record id="view_delivery_grid_tree" model="ir.ui.view">
diff --git a/addons/delivery/security/ir.model.access.csv b/addons/delivery/security/ir.model.access.csv
index 59558170047..948a85c35c7 100644
--- a/addons/delivery/security/ir.model.access.csv
+++ b/addons/delivery/security/ir.model.access.csv
@@ -6,4 +6,4 @@
 "access_delivery_grid_manager","delivery.grid","model_delivery_grid","sale.group_sale_manager",1,1,1,1
 "access_delivery_grid_line_manager","delivery.grid.line","model_delivery_grid_line","sale.group_sale_manager",1,1,1,1
 "access_delivery_carrier_partner_manager","delivery.carrier partner_manager","model_delivery_carrier","base.group_partner_manager",1,0,0,0
-"access_delivery_carrier_stock_worker","delivery.carrier stock_worker","model_delivery_carrier","base.group_system",1,1,1,1
+"access_delivery_carrier_stock_worker","delivery.carrier stock_worker","model_delivery_carrier","stock.group_stock_user",1,0,0,0
diff --git a/addons/point_of_sale/pos_view.xml b/addons/point_of_sale/pos_view.xml
index ea8be78274b..150febfcfa6 100644
--- a/addons/point_of_sale/pos_view.xml
+++ b/addons/point_of_sale/pos_view.xml
@@ -746,7 +746,7 @@
         </record>
 
         <menuitem name="Configuration" parent="menu_point_root"
-            id="menu_point_config_product" sequence="25"/>
+            id="menu_point_config_product" sequence="25" groups="base.group_system"/>
 
         <record model="ir.actions.act_window" id="action_product_input">
             <field name="name">Products</field>
diff --git a/addons/point_of_sale/security/ir.model.access.csv b/addons/point_of_sale/security/ir.model.access.csv
index 1be98fd8686..15353f01d7c 100644
--- a/addons/point_of_sale/security/ir.model.access.csv
+++ b/addons/point_of_sale/security/ir.model.access.csv
@@ -44,7 +44,7 @@
 "access_account_period","account.period","account.model_account_period","point_of_sale.group_pos_user",1,1,1,0
 "access_account_fiscalyear","account.fiscalyear","account.model_account_fiscalyear","point_of_sale.group_pos_user",1,1,1,0
 "access_account_cashbox_line","account.cashbox.line","account.model_account_cashbox_line","point_of_sale.group_pos_user",1,1,1,0
-"access_product_product","product.product","product.model_product_product","base.group_system",1,1,1,1
+"access_product_product","product.product","product.model_product_product","point_of_sale.group_pos_user",1,1,1,0
 "access_account_move_line","account.move.line","account.model_account_move_line","point_of_sale.group_pos_user",1,1,1,0
 "access_account_move","account.move","account.model_account_move","point_of_sale.group_pos_user",1,1,1,0
 "access_account_journal","account.journal","account.model_account_journal","point_of_sale.group_pos_user",1,1,1,0