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.

CommentFileSizeAuthor
#1 ipauth-541770-1.patch915 bytesCoPut

Comments

CoPut’s picture

StatusFileSize
new915 bytes

That's my solution.

jonfrancisskydiver’s picture

That looks good--thank you! I will see how it functions and then make an update.

vacilando’s picture

Version: 6.x-1.5 » 6.x-1.x-dev
Priority: Normal » Critical
Issue tags: +authentication, +register, +anonymous, +ip, +registration

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 496

Please help. This is a critical issue.

vacilando’s picture

I was forced to stop using this module more than a month ago because of this problem. Is there any solution, please?

vacilando’s picture

Any further thoughts on this, anyone? I'd like to use the module but this bug prevents it.

kingmanic’s picture

"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.