Hooks alter/break array-formatted callbacks

[Veraxus]

When hooking to static methods of a class (plugin, theme, etc), there are two possible formats:

array('class','method') or 'class::method'

The prior works on PHP 4+, where the latter requires PHP 5.2.3+

The problem is that any time I try to use an array-formatted callback with a WordPress hook (for maximum compatibility), PHP throws an invalid callback error for wp-includes/plugin.php.

It turns out that the array ultimately being passed to call_user_func_array() always has an integer 1 appended to the array... thus making the callback invalid as call_user_func_array() is now being sent this:

array('class','function',1)

String-formatted callbacks work perfectly, but array-formatted callbacks are altered and therefore broken. This can be fixed by adding the following check to four different functions in plugin.php (apply_filters(), apply_filters_ref_array(), do_action(), and do_action_ref_array()), but I'm not sure this is the "best" solution.

Here's the "corrective" check:

if(is_array($the_['function']) && isset($the_['function'][2])){
   unset($the_['function'][2]);
}

[scribu]

I use array callbacks all the time without problems.

Are you sure there's not something else going on?

[scribu]

Maybe you're adding the priority in the array by mistake:

add_action( 'foo', array( 'class', 'method, 1 ) );

[westi]

Replying to scribu:

Maybe you're adding the priority in the array by mistake:

add_action( 'foo', array( 'class', 'method, 1 ) );

That exactly how he shows it in the ticket description which is incorrect.

[Veraxus]

No, I've confirmed the callback is being entered correctly.

For instance, here's one of the problem hooks:

add_action('admin_init', array('CTXPS_App','admin_init'));

Now if I write it like this, it works with no issues, Warnings, or Fatal Errors (but requires PHP 5.2.3)...

add_action('admin_init', 'CTXPS_App::admin_init');

Written as an array, I get this...

Warning: call_user_func_array() [function.call-user-func-array]: First argument is expected to be a valid callback, was given in /var/www/vhosts/contextureintl.com/subdomains/plugintest/httpdocs/wp-includes/plugin.php on line 488

If I print_r($the_['function']) everywhere it appears in plugin.php, I see that the array now looks like this (which is not how it's written in the hook)...

array('CTXPS_App','admin_init',1)

So far I just haven't been able to figure out where that 1 is coming from.

[Veraxus]

I loaded a fresh install of WordPress on a different server and there's no issues. There's definitely something wrong with the server, not WordPress.

I'll close this out unless I come up with something concrete to workaround.