Posted by CoPut on August 6, 2009 at 1:34pm
| Project: | ipAuthenticator |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | critical |
| Assigned: | Unassigned |
| Status: | needs review |
| Issue tags: | anonymous, authentication, ip, register, registration |
Issue Summary
In my situation email_registration module needs to override user registration form but ipAuthenticator call user_register() on form construction directly, so there is no changes for email_registration.
Comments
#1
That's my solution.
#2
That looks good--thank you! I will see how it functions and then make an update.
#3
The patch does not work for me. After patching, when I access /user/register I get the following error:
PHP Fatal error: Cannot unset string offsets in /MYPATH/includes/form.inc on line 496Please help. This is a critical issue.
#4
I was forced to stop using this module more than a month ago because of this problem. Is there any solution, please?
#5
Any further thoughts on this, anyone? I'd like to use the module but this bug prevents it.
#6
"Any further thoughts on this, anyone? I'd like to use the module but this bug prevents it."
It might be a module weighting issues. I manually changed the weighting of the IPAuth module to 10 via this SQL:
UPDATE `myDB`.`system` SET `weight` = '10' WHERE CONVERT( `system`.`filename` USING utf8 ) = 'sites/all/modules/ipAuthenticator/ipauth.module' LIMIT 1 ;What was also helpful was an alteration to the user calls to add an extra 'true' from this thread.
http://drupal.org/node/555096
Basically it changed all instances of
'access arguments' => array(FALSE, TRUE, TRUEE),to
'access arguments' => array(FALSE, TRUE, TRUE, TRUE),The following line is also changed to add an extra TRUE at the end:
'access arguments' => array(FALSE, variable_get('user_register', 1), TRUE),to
'access arguments' => array(FALSE, variable_get('user_register', 1), TRUE, TRUE),This allows for the login window to appear if people are outside the IP ranges we specify.
PS. both these changes should be incorporated into the next version. I do beleive they are bugs.