Skip to content

Fix for missing argument notice #328

Merged
merged 11 commits into from Oct 26, 2013

2 participants

@jrfnl
jrfnl commented Oct 25, 2013

Fixed issue #327 Missing argument 3 for wpseo_upgrader_process_complete notice for WP 3.7+

Originally also contained unscheduling of yoast tracking cronjob, but I have moved that to a separate branch & pull request (#344) to make sure this fix can be committed now.

@coachmaria

Is the answer here somewhere for my problem?

After upgrading from WordPress 3.6.1 to 3.7, the following message appears each time a plug-in is updated:

Warning: Missing argument 3 for wpseo_upgrader_process_complete() in /.../wp-content/plugins/wordpress-seo/inc/wpseo-non-ajax-functions.php on line 259

This is being discussed by dozens of others experiencing the same problem on the WordPress.org repository at http://wordpress.org/support/topic/does-it-work-correctly-with-wordpress-37

@coachmaria

What is a human with no experience in php code supposed to do to fix the problem?

After upgrading from WordPress 3.6.1 to 3.7, the following message appears each time a plug-in is updated:

Warning: Missing argument 3 for wpseo_upgrader_process_complete() in /.../wp-content/plugins/wordpress-seo/inc/wpseo-non-ajax-functions.php on line 259

This is being discussed by dozens of others experiencing the same problem on the WordPress.org repository at http://wordpress.org/support/topic/does-it-work-correctly-with-wordpress-37

@jrfnl
jrfnl commented Oct 25, 2013

@coachmaria, posting twice will not get your message across more clearly.... sigh
Anyway, it is not in any way a serious problem. All it means is that the check for meta description tags is not being done on a theme upgrade.

I suggest you just wait for the next version of the plugin.

More detailed info is also available in the WP Forum thread you mention and in the original issue #327.

@jrfnl jrfnl referenced this pull request Oct 26, 2013
Closed

Missing argument #327

@jrfnl jrfnl merged commit cb3ed42 into Yoast:master Oct 26, 2013

1 check passed

Details default The Travis CI build passed
@jrfnl
jrfnl commented Oct 26, 2013

FYI: I've just committed the fix - if anyone prefers not to wait for the next official version, but wants the fix *now, you can download the current development version of the plugin and upload the unzipped files to your site via FTP.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.