Posted by mikesir87 on September 22, 2009 at 7:51pm
3 followers
Jump to:
| Project: | Secure by Role |
| Version: | 6.x-1.0-beta1 |
| Component: | User interface |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
In the config page for the module, the current server setting isn't showing. By replacing line 74 to show
'#default_value' => variable_get('secrole_server', 0),I was able to get it to work. Was there a reason to hide that? Might be something to fix in the next release.
Comments
#1
Ha ha ha, wow, what an oversight. Thanks. Clearly, I have only really used the "make a guess" option with regards to this feature.
The second parameter for that variable_get() call should be a blank string, though.
I'll patch this up and get a release made quickly. Thanks.
#2
Okay, I just created a Beta 2 release. Please give it a try.
#3
Thanks for the quick response! It seems like the code didn't make it in for some strange reason. It still has the default_vale as ''.
I must also say that this module has really saved us on one of our projects! Thanks for developing it! :-)
#4
Oh, God, do I hate CVS.
Okay, trying again with a Beta 3.
#5
Haha...I know the feeling! I have to say, I loved the comment in the release notes! :-)
And fyi... this release works perfectly! Thanks!
#6
Wonderful. Thanks again for reporting the glitch, and for trying the module.
#7
Automatically closed -- issue fixed for 2 weeks with no activity.