From 5e74d37e723ab0521b222126ebf1c770f30636d4 Mon Sep 17 00:00:00 2001
From: Yogesh Sakhreliya <ysa@tinyerp.com>
Date: Tue, 28 Dec 2010 15:50:06 +0530
Subject: [PATCH 1/4] [IMP] add change user password wizard.

bzr revid: ysa@tinyerp.com-20101228102006-u2z5ve9d302xqe4v
---
 bin/addons/base/base_update.xml | 32 ++++++++++++++++++++++++++++++--
 bin/addons/base/res/res_user.py | 32 ++++++++++++++++++++++++++++++--
 2 files changed, 60 insertions(+), 4 deletions(-)

diff --git a/bin/addons/base/base_update.xml b/bin/addons/base/base_update.xml
index 2448306ad79..5a008bd7d7e 100644
--- a/bin/addons/base/base_update.xml
+++ b/bin/addons/base/base_update.xml
@@ -69,6 +69,33 @@
     Users
     ======================
     -->
+
+        <record id="view_change_user_password_form" model="ir.ui.view">
+            <field name="name">change.user.password</field>
+            <field name="model">change.user.password</field>
+            <field name="type">form</field>
+            <field name="arch" type="xml">
+                <form string="Change Password">
+	                <field name="current_password" password="True" readonly="0" colspan="4"/>
+	                <field name="new_password" password="True" readonly="0" colspan="4"/>
+	                <field name="confirm_password" password="True" readonly="0" colspan="4"/>
+                    <separator colspan="4" />
+                    <label align="0.0" colspan="2" string=""/>
+                    <button colspan="1" icon="gtk-cancel" special="cancel" string="Cancel"/>
+                    <button colspan="1" icon="gtk-execute" name="change_password" string="Change" type="object"/>
+                </form>
+            </field>
+        </record>
+
+        <record id="action_view_change_password_form" model="ir.actions.act_window">
+            <field name="name">Change Password</field>
+            <field name="type">ir.actions.act_window</field>
+            <field name="res_model">change.user.password</field>
+            <field name="view_type">form</field>
+            <field name="view_mode">form</field>
+            <field name="target">new</field>
+        </record>
+
         <record id="view_users_form_simple_modif" model="ir.ui.view">
             <field name="name">res.users.form.modif</field>
             <field name="model">res.users</field>
@@ -87,12 +114,13 @@
                             <newline/>
                         </page>
                          <page string="Preferences">
-                            <field name="password" password="True" readonly="0" />
                             <field name="context_lang" completion="1" readonly="0"/>
+                            <field name="context_tz" completion="1" readonly="0"/>
                             <label string="" colspan="1"/>
                             <label colspan="3" string="You must logout and login again after changing your password."/>
-                            <field name="context_tz" completion="1" readonly="0"/>
                             <field name="menu_tips" colspan="2" readonly="0"/>
+                            <label string="" colspan="1"/>
+                            <button name="%(action_view_change_password_form)d" string="Change Password" type="action" icon="gtk-execute"/>
                             <separator string="Email &amp; Signature" colspan="4"/>
                             <group colspan="4"><field name="user_email" widget="email" readonly="0"/></group>
                             <field colspan="4" name="signature" readonly="0" nolabel="1"/>
diff --git a/bin/addons/base/res/res_user.py b/bin/addons/base/res/res_user.py
index e7f881c1c9d..e7f6bfd7de0 100644
--- a/bin/addons/base/res/res_user.py
+++ b/bin/addons/base/res/res_user.py
@@ -448,7 +448,7 @@ class users(osv.osv):
             return True
         cr = pooler.get_db(db).cursor()
         try:
-            cr.execute('SELECT COUNT(1) FROM res_users WHERE id=%s AND password=%s AND active=%s', 
+            cr.execute('SELECT COUNT(1) FROM res_users WHERE id=%s AND password=%s AND active=%s',
                         (int(uid), passwd, True))
             res = cr.fetchone()[0]
             if not bool(res):
@@ -547,7 +547,7 @@ class groups2(osv.osv): ##FIXME: Is there a reason to inherit this object ?
             if len(user_names) >= 5:
                 user_names = user_names[:5]
                 user_names += '...'
-            raise osv.except_osv(_('Warning !'), 
+            raise osv.except_osv(_('Warning !'),
                         _('Group(s) cannot be deleted, because some user(s) still belong to them: %s !') % \
                             ', '.join(user_names))
         return super(groups2, self).unlink(cr, uid, ids, context=context)
@@ -574,5 +574,33 @@ class res_config_view(osv.osv_memory):
 
 res_config_view()
 
+class change_user_password(osv.osv_memory):
+    _name = 'change.user.password'
+    _columns = {
+        'current_password':fields.char('Current Password', size=64, required=True, help="Enter your current password."),
+        'new_password': fields.char('New Password', size=64, required=True, help="Enter new password."),
+        'confirm_password': fields.char('Confirm Password', size=64, required=True, help="Enter new password again for confirmation."),
+    }
+    _defaults={
+        'current_password' : '',
+        'new_password' : '',
+        'confirm_password' : '',
+    }
+
+    def change_password(self, cr, uid, ids, context=None):
+        user_obj = self.pool.get('res.users')
+        if ids:
+            current_password = user_obj.browse(cr, uid, uid, context).password
+            password_rec = self.browse(cr, uid, ids[0], context)
+            if password_rec.new_password != password_rec.confirm_password:
+                raise osv.except_osv(_('Warning !'), _('New password and confirm password does not match !'))
+            elif current_password != password_rec.current_password:
+                raise osv.except_osv(_('Warning !'), _('Current password does not match !'))
+            else:
+                user_obj.write(cr, uid, [uid], {'password': password_rec.new_password}, context)
+        return {}
+
+change_user_password()
+
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 

From cfad3fdbb9d13b9a605f09d2ab6dbf7be5c77130 Mon Sep 17 00:00:00 2001
From: Yogesh Sakhreliya <ysa@tinyerp.com>
Date: Tue, 28 Dec 2010 16:39:29 +0530
Subject: [PATCH 2/4] [FIX] chnage the create new address record and change
 address_id in demo user. and fix problem in report file.

bzr revid: ysa@tinyerp.com-20101228110929-tr369m30tccn78up
---
 bin/addons/base/base_demo.xml      | 17 +++++++++++++++--
 bin/report/render/rml2pdf/utils.py |  2 +-
 2 files changed, 16 insertions(+), 3 deletions(-)

diff --git a/bin/addons/base/base_demo.xml b/bin/addons/base/base_demo.xml
index 7779171fe08..73f6fa400a5 100644
--- a/bin/addons/base/base_demo.xml
+++ b/bin/addons/base/base_demo.xml
@@ -1,14 +1,27 @@
 <?xml version="1.0" encoding="utf-8"?>
 <openerp>
     <data noupdate="1">
+
+        <record id="demo_address" model="res.partner.address">
+            <field name="name">Fabien Dupont</field>
+            <field name="street">Chaussee de Namur</field>
+            <field name="zip">1367</field>
+            <field name="city">Gerompont</field>
+            <field name="phone">(+32).81.81.37.00</field>
+            <field name="type">default</field>
+            <field model="res.country" name="country_id" ref="be"/>
+            <!-- Company ID will be set later -->
+            <field name="company_id" eval="None"/>
+        </record>
+
         <record id="user_demo" model="res.users">
             <field name="login">demo</field>
             <field name="password">demo</field>
             <field name="name">Demo User</field>
             <field name="signature">Mr Demo</field>
-            <field name="address_id" ref="main_address"/>
+            <field name="address_id" ref="demo_address"/>
             <field name="company_id" ref="main_company"/>
-            <field name="groups_id" eval="[(6,0,[ref('base.group_user')])]"/> 
+            <field name="groups_id" eval="[(6,0,[ref('base.group_user')])]"/>
         </record>
     </data>
 </openerp>
diff --git a/bin/report/render/rml2pdf/utils.py b/bin/report/render/rml2pdf/utils.py
index 6f8616da96d..e631fcd9566 100644
--- a/bin/report/render/rml2pdf/utils.py
+++ b/bin/report/render/rml2pdf/utils.py
@@ -131,7 +131,7 @@ def _process_text(self, txt):
                     pass
                 if isinstance(txt, basestring):
                     result += str2xml(txt)
-                elif (txt is not None) and (txt is not False):
+                elif txt and (txt is not None) and (txt is not False):
                     result += ustr(txt)
         return result
 

From bc524902eb7c33ae3ec48801a7481236617e86eb Mon Sep 17 00:00:00 2001
From: Yogesh Sakhreliya <ysa@tinyerp.com>
Date: Wed, 29 Dec 2010 13:19:51 +0530
Subject: [PATCH 3/4] [FIX] fix height of configuration wizard.

bzr revid: ysa@tinyerp.com-20101229074951-s3yhn5regplzei5l
---
 bin/addons/base/res/res_config.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/bin/addons/base/res/res_config.xml b/bin/addons/base/res/res_config.xml
index 79e1e257139..4b0db4c64f5 100644
--- a/bin/addons/base/res/res_config.xml
+++ b/bin/addons/base/res/res_config.xml
@@ -7,7 +7,7 @@
       <field name="arch" type="xml">
        <form>
          <group colspan="4" col="8">
-           <group colspan="1" width="220" height="250">
+           <group colspan="1" width="220">
              <field name="config_logo" widget="image" width="220" height="130" nolabel="1"/>
              <newline/>
              <label align="0.0" string="description" width="200" colspan="2"/>

From 6112db95ddc52bc09b197c9df8b2a9181d57fc5f Mon Sep 17 00:00:00 2001
From: Yogesh Sakhreliya <ysa@tinyerp.com>
Date: Thu, 29 Dec 2011 18:04:13 +0530
Subject: [PATCH 4/4] [FIX-695338] fix assert tag in currency rate.

bzr revid: ysa@tinyerp.com-20111229123413-vba5v6etaf1vrh74
---
 bin/addons/base/test/test_context.xml | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/bin/addons/base/test/test_context.xml b/bin/addons/base/test/test_context.xml
index 44751644403..d308fc3dea8 100644
--- a/bin/addons/base/test/test_context.xml
+++ b/bin/addons/base/test/test_context.xml
@@ -6,15 +6,15 @@
 <!--            model="res.currency"-->
 <!--            id="INR"-->
 <!--            string="Indian rupee rate in 2009">-->
-            <!-- date specified in data element's context --> 
+            <!-- date specified in data element's context -->
 <!--            <test expr="str(rate)">65.8287</test>-->
 <!--        </assert> -->
-        <assert 
+        <assert
             model="res.currency"
             id="INR"
             string="Indian rupee rate in 2002"
             context="{'date': '2010-06-01'}">
-            <test expr="str(rate)">59.9739</test>
+            <test expr="rate_ids and str(rate_ids[0].rate)">59.9739</test>
         </assert>
     </data>
 </openerp>