Closed (fixed)
Project:
Easy Login
Version:
6.x-1.2
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
8 Apr 2009 at 08:37 UTC
Updated:
23 Apr 2009 at 05:00 UTC
I have a demo site set up on which I am using this module (nice little module by the way). One of the users I have set up an easy login for is a site administrator - I am doing this so our clients can see what they can do as administrators. However, I don't want them to be able to reset their URL. The problem is in this code:
function _easylogin_access_profile($uid){
global $user;
if ($uid == 1 || user_is_anonymous() || !user_access('login from url') || ($user->uid != $uid && !user_access('administer users'))){
return FALSE;
}
return TRUE;
}It appears that anyone with 'administer users' permissions can reset their URL, even if they don't have 'reset own url' permissions. I think these should be separated - as there will be situations, like mine, where we need these permissions to be separated.
Thank you.
Comments
Comment #1
George2 commentedinteresting use case.
anyway, i've jiggled permissions about to allow for this. and will be in dev at utc midnight
Comment #2
George2 commentedComment #3
jaypanThank you sir! I appreciate the quick response and action.