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

Garrett Albright’s picture

Garrett Albright commented

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.

Garrett Albright’s picture

Garrett Albright commented

Okay, I just created a Beta 2 release. Please give it a try.

mikesir87’s picture

mikesir87 commented

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! :-)

Garrett Albright’s picture

Garrett Albright commented

Oh, God, do I hate CVS.

Okay, trying again with a Beta 3.

mikesir87’s picture

mikesir87 commented
Status: Active » Fixed

Haha...I know the feeling! I have to say, I loved the comment in the release notes! :-)

And fyi... this release works perfectly! Thanks!

Garrett Albright’s picture

Garrett Albright commented

Wonderful. Thanks again for reporting the glitch, and for trying the module.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.