Closed (fixed)
Project:
Permissions API
Version:
6.x-2.5
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Reporter:
Created:
27 Oct 2009 at 13:27 UTC
Updated:
23 Mar 2010 at 14:40 UTC
In the permissions_revoke_permissions function there is the following error:
foreach($new_permissions as $permission){
$index = array_search($permission, $permissions);
if($index >= FALSE){
/* The above question is a mistake because if the searched permission is not in the array you get FALSE that is converted to 0 so you revoke the
first permission on the array.
the code should be changed to if($index !== FALSE)
*/
unset($permissions[$index]);
}
}
I don't fix it my self because i dont know how to commit the change
Comments
Comment #1
ebeyrent commentedI'll take a look at this right away; thanks for the bug report!
Comment #2
ebeyrent commentedThis has been committed; I'll update the release soon.
Comment #3
ebeyrent commentedFixed in the 2.6 release