#6 Path list

v1.5
open
Common (4)
5
2011-10-28
2011-10-28
Jacq
No

With an administrator user account I cannot see the permissions for all the paths. I don't know if this issue is related to other changes I made to the source or if this is a desired behaviour. I think this issue is present in the original 1.5.

Anyway, to "fix" this I edit accespathslist.php around line 38:
Before:
if ($appEngine->getAclManager()->isUserAccessPathManager($currentUsername))
$list = $appEngine->getAclManager()->filterAccessPathsList($currentUsername, $list);
After:
$isAdmin = $appEngine->getAclManager()->userHasRole($currentUsername,$appEngine->getAclManager()->administrator_role_name);
if(!$isAdmin)
{
if ($appEngine->getAclManager()->isUserAccessPathManager($currentUsername))
$list = $appEngine->getAclManager()->filterAccessPathsList($currentUsername, $list);
}

If there is another way more "elegante" to fix this just let me know.
I couldn't find any use of the function userHasRole of the AclManager anywhere in the current code, is there a reason not to use it?
Jacq

Discussion


Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks