register_hook and register_new_user are broken

[whooami]

In the SVN, hooking into register_post with a plugin, doesnt work:

function email_check() {
		global $errors;
		if ($user_email != "secret@secret.org")
		{
		$errors->add('icky_email', __('<strong>ERROR</strong>: your email blows.'));
		

		}

add_action('register_post', 'email_check');

This was fixable this by adding the the global $errors to the function, register_new_user:

function register_new_user($user_login, $user_email) {
	global $errors; 
...
...

[ryan]

(In [6821]) Pass login, email, and errors to register_post action. fixes #5763

[dlo]

Hooking to register_post in a plugin still doesn't work, even after change 6821.

$errors variable remain unchanged when returning from hooked function in the register_new_user function (in login.php) as $errors is certainly passed by value and not by reference.

[dlo]

To add some precision:
Running PHP 4.4.3, I had to revert to the solution given by whooami at the beginning of this ticket to be able to get back the errors triggered by my custom function hooked to 'register_form' action in the $error variable of the register_new_user function.

Is this due to the fact that objects are passed by value in PHP 4 when they are passed by reference in PHP 5, thus making the [6821] change ineffective with some configuration? Nevertheless, it's a bit annoying to have to modify the wp-login.php file to get a plugin working. Ehh?

Is it possible to consider the implementation of the whooami solution as a quick workaround for WordPress 2.5 ?

[ryan]

Why not use the registration_errors filter for modifying $errors?

 $errors = apply_filters( 'registration_errors', $errors );

[whooami]

Ryan,

$errors = apply_filters( 'registration_errors', $errors );

I'm dense, can you explain to me where that would be used inside a plugin that is calling $errors? I cant seem to figure it out.

[dlo]

Ryan,

Using the filter 'registration_errors' is just fine.
You were right in pointing that at me. I was focused on the action 'register_form' and didn't pay attention to the filter just one line below !
Nethertheless, is it a feature or a bug the fact that you can't modify the $errors variable during the 'register_form' action execution ?

[Denis-de-Bernardy]

Nethertheless, is it a feature or a bug the fact that you can't modify the $errors variable during the 'register_form' action execution ?

It's php related. in php4, you'd need a reference to modify the $error. but call_user_func() et al won't let you pass one.