From bc17621cecb4a36d4d4e6c6272b2e1b57e51fa1f Mon Sep 17 00:00:00 2001
From: "sgo@tinyerp.com" <>
Date: Thu, 7 Feb 2013 12:21:26 +0530
Subject: [PATCH] [IMP]improve pos config form view and add group on warehouse

bzr revid: sgo@tinyerp.com-20130207065126-1liwmg1u3emnra6j
---
 addons/point_of_sale/point_of_sale.py       |  3 ++-
 addons/point_of_sale/point_of_sale_view.xml | 23 ++++++++++++++-------
 2 files changed, 17 insertions(+), 9 deletions(-)

diff --git a/addons/point_of_sale/point_of_sale.py b/addons/point_of_sale/point_of_sale.py
index e1fe577c922..ebb87d02f26 100644
--- a/addons/point_of_sale/point_of_sale.py
+++ b/addons/point_of_sale/point_of_sale.py
@@ -114,7 +114,8 @@ class pos_config(osv.osv):
         return res and res[0] or False
 
     def _default_warehouse(self, cr, uid, context=None):
-        res = self.pool.get('stock.warehouse').search(cr, uid, [])
+        user = self.pool.get('res.users').browse(cr, uid, uid, context)
+        res = self.pool.get('stock.warehouse').search(cr, uid, [('company_id','=',user.company_id.id)], context)
         return res and res[0] or False
     
     def _default_pricelist(self, cr, uid, context=None):
diff --git a/addons/point_of_sale/point_of_sale_view.xml b/addons/point_of_sale/point_of_sale_view.xml
index 6a8208bac0a..a3628761173 100644
--- a/addons/point_of_sale/point_of_sale_view.xml
+++ b/addons/point_of_sale/point_of_sale_view.xml
@@ -769,13 +769,20 @@
                     </header>
 
                     <sheet>
+                        <label for="name" class="oe_edit_only"/>
+                        <h1>
+                            <field name="name"/>
+                        </h1>
                         <group>
-                        <field name="name"/>
-                        <field name="warehouse_id" widget="selection" groups="stock.group_locations" />
-                        <field name="journal_id" widget="selection" />
-                        <field name="pricelist_id" groups="product.group_sale_pricelist"/>
-                        <field name="sequence_id" readonly="1" groups="base.group_no_one" />
-                        <field name="group_by" groups="account.group_account_user"  />
+                            <group>
+                                <field name="warehouse_id" widget="selection" groups="stock.group_locations" />
+                                <field name="pricelist_id" groups="product.group_sale_pricelist"/>
+                                <field name="sequence_id" readonly="1" groups="base.group_no_one" />
+                            </group>
+                            <group>
+                                 <field name="journal_id" widget="selection" />
+                                 <field name="group_by" groups="account.group_account_user"  />
+                            </group>
                         </group>
                         <separator string="Available Payment Methods" colspan="4"/>
                         <field name="journal_ids" colspan="4" nolabel="1">
@@ -810,7 +817,7 @@
             <field name="arch" type="xml">
                 <tree string="Point of Sale Configuration" colors="grey:state == 'inactive'">
                     <field name="name" />
-                    <field name="warehouse_id" />
+                    <field name="warehouse_id" groups="stock.group_locations"/>
                     <field name="state" />
                 </tree>
             </field>
@@ -824,7 +831,7 @@
                     <field name="name" />
                     <filter string="Active" domain="[('state', '=', 'active')]" />
                     <filter string="Inactive" domain="[('state', '=', 'inactive')]" />
-                    <field name="warehouse_id" />
+                    <field name="warehouse_id" groups="stock.group_locations" />
                 </search>
             </field>
         </record>