Permissions not being assigned
designerbrent - September 16, 2008 - 21:47
| Project: | Admin Role |
| Version: | 6.x-1.0-beta2 |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed |
Jump to:
Description
adminrole was not assigning permissions when I would add new modules as it should. I had the role setup and everything. The attached patch will fix it. Basically, it appears that the module_list() function was not refreshing. The attached patch will fix this.
| Attachment | Size |
|---|---|
| adminrole_checkall.diff | 364 bytes |
#1
can you make a .patch file from this
i can not patch with this file on my vps root
#2
Babalu: This is a patch file. If you so desire, you can rename it to .patch. Or you can open it and make the small change manually.
http://drupal.org/patch/apply
#3
rename and works. thx :)
#4
This patch works great... I applied a few weeks ago and have been checking periodically to make sure all permissions are applied - have tested many modules and no issues yet. Is it too bold to say it's 'reviewed & tested by the community'? ;)
#5
I'm experiencing the same problem, and the warnings on the drupal.org instructions for patching make me very hesitant to apply the patch (hesitant to apply any patch, actually). Any idea when this patch will be added either to .dev or beta for 6.x?
#6
@femrich, I understand your concerns, but this is a really trivial patch, just look at it and make the change manually. Basically, the first argument to module_list needs to be changed from FALSE to TRUE. It is much safer to apply the patch than to use the dev version, since the dev version most likely has other changes as well.
#7
Would be great to get this in and roll a new release. The current release does not have the fix for #297920: Wrong role gets admin access. thanks.
#8
Sorry, but I am a coding newbie. I'll look into this, but it will take me a while just to figure out what the instructions in your reply mean, mariuss. My bad, of course. But thanks for the feedback and I will try to figure out how to apply this patch...
#9
Subscribing.
#10
It'd be really good to get this committed and released. I think JacobSingh (maintainer) is swamped with larger things with his new job at Acquia right now. I have pinged him.
#11
Committed, will be released as RC2
#12
#13
Automatically closed -- issue fixed for two weeks with no activity.