OpenID Plugin and WordPress 2.8

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.

You may also like

7 Comments

willnorris Jun 12, 6:00

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.

Kaspars Jun 12, 11:19

will, great that you had already solved this. I’ll update to trunk.

Kaspars Jun 12, 11:34

Hmm, will. Just tried the trunk version and it still has the openid_require_name_email(); at the wrong place.

Ian McAbee Jun 12, 15:36

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?

willnorris Jun 12, 16:06

Kaspars: the authoritative repository for the plugin is actually:
http://diso.googlecode.com/svn.....nid/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.

willnorris Jun 12, 16:07

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.

Kaspars Jun 12, 17:04

Thanks, will. I have two plugins to update as well, so I understand :)

Leave a Comment