From f5fc4532f2f3c6193913b5845fe2760cececa207 Mon Sep 17 00:00:00 2001
From: "Quentin (OpenERP)" <qdp-launchpad@openerp.com>
Date: Tue, 6 Nov 2012 10:03:00 +0100
Subject: [PATCH] [IMP] lunch: code review. usability / fixes

bzr revid: qdp-launchpad@openerp.com-20121106090300-bk9up9ugprfcyqn4
---
 addons/lunch/lunch.py       |  2 +-
 addons/lunch/lunch_view.xml | 13 ++++++++-----
 2 files changed, 9 insertions(+), 6 deletions(-)

diff --git a/addons/lunch/lunch.py b/addons/lunch/lunch.py
index 6aef154ad0a..0d224ef3a8e 100644
--- a/addons/lunch/lunch.py
+++ b/addons/lunch/lunch.py
@@ -79,7 +79,7 @@ class lunch_order(osv.Model):
         get the alerts to display on the order form 
         """
         result = {}
-        alert_msg = self._default_alerts_get(cr, uid, arg, context=context)
+        alert_msg = self._default_alerts_get(cr, uid, context=context)
         for order in self.browse(cr, uid, ids, context=context):
             if order.state == 'new':
                 result[order.id] = alert_msg
diff --git a/addons/lunch/lunch_view.xml b/addons/lunch/lunch_view.xml
index 31540c63e14..32b978dfab0 100644
--- a/addons/lunch/lunch_view.xml
+++ b/addons/lunch/lunch_view.xml
@@ -37,9 +37,10 @@
             <field name='type'>search</field>
             <field name='arch' type='xml'>
                 <search string="lunch employee payment">
-                    <field name="user_id"/>
                     <field name="description"/>
+                    <field name="user_id"/>
                     <filter name='is_payment' string="Payment" domain="[('state','=','payment')]"/>
+                    <separator/>
                     <filter name='is_mine' string="My Account" domain="[('user_id','=',uid)]"/>
                 </search>
             </field>
@@ -51,9 +52,11 @@
             <field name='type'>search</field>
             <field name='arch' type='xml'>
                 <search string="lunch cashmove">
-                    <field name="user_id"/>
                     <field name="description"/>
-                    <filter name='group_by_user' string="By Employee" context="{'group_by':'user_id'}"/>
+                    <field name="user_id"/>
+                    <group expand="0" string="Group By...">
+                        <filter name='group_by_user' string="By Employee" context="{'group_by':'user_id'}"/>
+                    </group>
                 </search>
             </field>
         </record>
@@ -86,11 +89,11 @@
 
         <!--Action for Your Orders-->
         <record model="ir.actions.act_window" id="action_lunch_order_form">
-            <field name="name">Your Order</field>
+            <field name="name">New Order</field>
             <field name="res_model">lunch.order</field>
             <field name="view_mode">form</field>
         </record>
-        <menuitem name="Your Order" parent="menu_lunch_title" id="menu_lunch_order_form" action="action_lunch_order_form" sequence="1"/>
+        <menuitem name="New Order" parent="menu_lunch_title" id="menu_lunch_order_form" action="action_lunch_order_form" sequence="1"/>
 
         <record model="ir.actions.act_window" id="action_lunch_order_tree">
             <field name="name">Your Orders</field>