From f5867d9489132a26a347c017d667fb2c8ea9d311 Mon Sep 17 00:00:00 2001
From: "HDA (OpenERP)" <hda@tinyerp.com>
Date: Wed, 5 May 2010 19:43:30 +0530
Subject: [PATCH] usability improvement for sale

bzr revid: hda@tinyerp.com-20100505141330-zfbcuifv7nluauad
---
 bin/addons/base/res/partner/partner.py        |  6 ++--
 .../base/res/partner/partner_report.xml       |  4 +--
 bin/addons/base/res/partner/partner_view.xml  | 30 ++++++++++++++++---
 3 files changed, 32 insertions(+), 8 deletions(-)

diff --git a/bin/addons/base/res/partner/partner.py b/bin/addons/base/res/partner/partner.py
index 0cdf3d927d4..3e890e6efed 100644
--- a/bin/addons/base/res/partner/partner.py
+++ b/bin/addons/base/res/partner/partner.py
@@ -134,7 +134,7 @@ class res_partner(osv.osv):
         'title': fields.selection(_partner_title_get, 'Title', size=32),
         'parent_id': fields.many2one('res.partner','Parent Partner', select=2),
         'child_ids': fields.one2many('res.partner', 'parent_id', 'Partner Ref.'),
-        'ref': fields.char('Code', size=64),
+        'ref': fields.char('Reference', size=64),
         'lang': fields.selection(_lang_get, 'Language', size=5, help="If the selected language is loaded in the system, all documents related to this partner will be printed in this language. If not, it will be english."),
         'user_id': fields.many2one('res.users', 'Salesman', help='The internal user that is in charge of communicating with this partner if any.'),
         'vat': fields.char('VAT',size=32 ,help="Value Added Tax number. Check the box if the partner is subjected to the VAT. Used by the VAT legal statement."),
@@ -151,7 +151,9 @@ class res_partner(osv.osv):
         'supplier': fields.boolean('Supplier', help="Check this box if the partner is a supplier. If it's not checked, purchase people will not see it when encoding a purchase order."),
         'city': fields.related('address', 'city', type='char', string='City'), 
         'phone': fields.related('address', 'phone', type='char', string='Phone'), 
-        'country': fields.related('address', 'country_id', type='many2one', relation='res.country', string='Country'), 
+        'country': fields.related('address', 'country_id', type='many2one', relation='res.country', string='Country'),
+        'employee': fields.boolean('Employee', help="Check this box if the partner is an Employee."),
+        'email': fields.related('address', 'email', type='char', size=240, string='E-mail'),
         'company_id': fields.many2one('res.company', 'Company', select=1), 
     }
 
diff --git a/bin/addons/base/res/partner/partner_report.xml b/bin/addons/base/res/partner/partner_report.xml
index b4057414f56..61e3876527f 100644
--- a/bin/addons/base/res/partner/partner_report.xml
+++ b/bin/addons/base/res/partner/partner_report.xml
@@ -1,8 +1,8 @@
 <?xml version="1.0" encoding="utf-8"?>
 <openerp>
     <data>
-        <report id="res_partner_address_report" model="res.partner" name="res.partner.address" string="Labels" xml="base/res/partner/report/partner_address.xml" xsl="base/res/partner/report/partner_address.xsl"/>
-        <!-- 
+        <report id="res_partner_address_report" model="res.partner" name="res.partner.address" string="Labels" xml="base/res/partner/report/partner_address.xml" xsl="base/res/partner/report/partner_address.xsl" groups="base.group_extended"/>
+        <!--
 		<report string="Business Cards" model="res.partner" name="res.partner.businesscard" xml="base/res/partner/report/business_card.xml" xsl="base/res/partner/report/business_card.xsl"/>
 		-->
     </data>
diff --git a/bin/addons/base/res/partner/partner_view.xml b/bin/addons/base/res/partner/partner_view.xml
index 38dbeb34bc8..840ee132481 100644
--- a/bin/addons/base/res/partner/partner_view.xml
+++ b/bin/addons/base/res/partner/partner_view.xml
@@ -6,7 +6,7 @@
         <menuitem id="menu_base_config" name="Configuration" parent="menu_base_partner" sequence="9"
             groups="group_system"/>
 
-        <menuitem id="menu_base_config_partner" name="Partners" parent="menu_base_config" sequence="10" />
+        <menuitem id="menu_base_config_partner" name="Partners" parent="menu_base_config" sequence="10" groups="base.group_extended"/>
         <menuitem id="menu_base_config_contact" name="Contacts" parent="menu_base_config" sequence="20" />
 
         <!--
@@ -249,10 +249,11 @@
             <field name="arch" type="xml">
                 <tree string="Partners">
                     <field name="name"/>
-                    <field name="ref"/>
+                    <field name="ref" groups="base.group_extended"/>
                     <field name="phone"/>
                     <field name="city"/>
                     <field name="country"/>
+                    <field name="email"/>
                     <field name="lang"/>
                     <field name="user_id"/>
                 </tree>
@@ -275,6 +276,7 @@
                         <group colspan="1" col="2">
                             <field name="customer" select="1"/>
                             <field name="supplier"/>
+                            <field name="employee"/>
                         </group>
                     </group>
                     <notebook colspan="4">
@@ -315,7 +317,7 @@
                                 </tree>
                             </field>
                             <separator colspan="4" string="Categories"/>
-                            <field colspan="4" name="category_id" nolabel="1" height="90"/>
+                            <field colspan="4" name="category_id" nolabel="1" height="90" groups="base.group_extended"/>
                         </page>
                         <page string="Sales &amp; Purchases">
                             <separator string="General Information" colspan="4"/>
@@ -349,6 +351,7 @@
                         <separator orientation="vertical"/>
                         <filter string="Customers" icon="terp-partner" domain="[('customer','=',1)]" help="Customer Partners"/>
                         <filter string="Suppliers" icon="terp-partner" domain="[('supplier','=',1)]" help="Supplier Partners"/>
+                         <filter string="Employees" icon="terp-partner" domain="[('employee','=',1)]" help="Employee Partners"/>
                         <separator orientation="vertical"/>
                         <field name="name" select="1"/>
                         <field name="address" select="1"/>
@@ -406,6 +409,16 @@
             <field name="filter" eval="True"/>
         </record>
 
+        <record id="action_partner_employee_form" model="ir.actions.act_window">
+            <field name="name">Employees</field>
+            <field name="type">ir.actions.act_window</field>
+            <field name="res_model">res.partner</field>
+            <field name="view_type">form</field>
+            <field name="domain">[('employee','=',1)]</field>
+            <field name="context">{'default_customer':0, 'default_supplier':0 , 'default_employee':1}</field>
+            <field name="filter" eval="True"/>
+        </record>
+
         <record id="action_partner_other_form" model="ir.actions.act_window">
             <field name="name">Other Partners</field>
             <field name="type">ir.actions.act_window</field>
@@ -621,7 +634,16 @@
         <act_window domain="[('partner_id', '=', active_id)]"
             id="act_res_partner_event" name="Events"
             res_model="res.partner.event"
-            src_model="res.partner"/>
+            src_model="res.partner"
+            groups="base.group_extended"
+            />
+
+        <record id="ir_ui_view_sc_partner0" model="ir.ui.view_sc">
+            <field name="name">Partner</field>
+            <field name="resource">ir.ui.menu</field>
+            <field name="user_id" ref="base.user_root"/>
+             <field name="res_id" ref="menu_partner_form"/>
+        </record>
 
     </data>
 </openerp>