Merge lp:~openerp-dev/openobject-server/7.0-bug-1157839-dharmang into lp:openobject-server/7.0

Proposed by Dharmang Soni (OpenERP)
Status: Needs review
Proposed branch: lp:~openerp-dev/openobject-server/7.0-bug-1157839-dharmang
Merge into: lp:openobject-server/7.0
Diff against target: 13 lines (+3/-0)
1 file modified
openerp/addons/base/res/res_partner.py (+3/-0)
To merge this branch: bzr merge lp:~openerp-dev/openobject-server/7.0-bug-1157839-dharmang
Reviewer Review Type Date Requested Status
Ruchir Shukla(BizzAppDev) (community) Needs Fixing
OpenERP Core Team Pending
Review via email: mp+157843@code.launchpad.net

Description of the change

Hello,

I have fixed the issue of reset password when users with write access on Partners can change any user's password if "Enable password reset from Login page" is enabled.

Thank You,
Dharmang Soni

To post a comment you must log in.
Revision history for this message
Ruchir Shukla(BizzAppDev) (ruchir.shukla) wrote :

What If I am a second level Admin and not a Super user? I think it will not allow me to change any partner data for any user.

Is that mean To change partner data of user, you need to be a SUPER ADMIN ?

review: Needs Fixing

Unmerged revisions

4927. By Dharmang Soni (OpenERP)

[FIX] [7.0] users with write access on Partners can change any user's password if 'Enable password reset from Login page' is enabled

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'openerp/addons/base/res/res_partner.py'
2--- openerp/addons/base/res/res_partner.py 2013-03-29 11:23:17 +0000
3+++ openerp/addons/base/res/res_partner.py 2013-04-09 10:47:23 +0000
4@@ -365,6 +365,9 @@
5 if vals.get('is_company')==False:
6 vals.update({'child_ids' : [(5,)]})
7 for partner in self.browse(cr, uid, ids, context=context):
8+ user = self.pool.get("res.users").search(cr, uid, [("partner_id", "=", partner.id)], context=context);
9+ if user and user[0] != uid and uid != SUPERUSER_ID:
10+ raise osv.except_osv(_('Warning!'), _("You are not authorized to update user information."))
11 update_ids = []
12 if partner.is_company:
13 domain_children = [('parent_id', '=', partner.id), ('use_parent_address', '=', True)]