Ever since using the nightly versions of WordPress, I got used to doing regular updates and not checking if all plugins work fine afterward.
Therefore only today I discovered that leaving comments on this blog has been impossible for a while already. Turns out that changes in the WordPress core have made the OpenID plugin unaware of the comment author’s name and e-mail.
Haven’t found a solution yet.
The Solution
Open comments.php within the OpenID plugin folder and move the name & email input verification from line 73:
// duplicate name and email check from wp-comments-post.php
openid_require_name_email();
right after line 68, like this:
// just ignore the error... it just means the user entered a normal URL.
if (array_key_exists('openid_identifier', $_POST)) {
openid_repost_comment_anonymously($_SESSION['openid_comment_post']);
// duplicate name and email check from wp-comments-post.php
openid_require_name_email();
}
Why this works
We need to check name and e-mail input only if the URL is not an OpenID.
heh… I have a bad habit of fixing things in trunk but forgetting to release them publicly. Looks like this was fixed several weeks ago, but never released. I’ll see about making a new release tonight if I can.
will, great that you had already solved this. I’ll update to trunk.
Hmm, will. Just tried the trunk version and it still has the
openid_require_name_email();at the wrong place.The Tabbed Widgets plugin was working great on my site prior to the new WordPress 2.8 upgrade. I noticed that others are have problems too. Any fix in the works?
Kaspars: the authoritative repository for the plugin is actually:
http://diso.googlecode.com/svn/wordpress/openid/trunk/
I typically sync changes over to the WordPress.org svn repository when I’m ready to cut a release. Really need to find a better workflow for this stuff.
just realized though, that the current change in trunk was to fix requiring of names in trackbacks and pingbacks. It may actually still be broken with regards to OpenID. oops.
Thanks, will. I have two plugins to update as well, so I understand :)