OpenID Comments for MT 1.6

My OpenID Comments plugin for Movable Type is now available in version 1.6. New:

  • Finds userpic/avatar images in commenters' FOAF profiles, and provides them with the MTCommentAuthorPictureURL tag.
  • Properly handles cancelled signins.
  • Signing out of the blog no longer signs commenters out of TypeKey.
  • Per-blog option to disable inline styles in the SignOnThunk tags.

You might see PictureURL in action on this very blog, which is sporting a slight variation on the new style I made for my LiveJournal.

I tried to figure out how to use a fake email address to get around the "Require email addresses" setting when OpenID commenters are commenting, but faking and unfaking it in all the right places seems too ugly to bother with. As you can reasonably argue that's fine, and it warns commenters when they first try to sign in that the setting is wrong, I probably won't fix it in the future either.

6 comments

Comments

[LiveJournal user info] assert(♥)

comment

Here’s an OpenID comment with a userpic!

This blog style makes it tiny, but you can mouse over it to show full size.

Neil T.

comment

Lots of bugs with this one, I’m afraid :( .

On my site, when rebuilding, I get this error after having added the tag: Can't call method "pic_url" on an undefined value at /home/neiltur/public_html/scgi-bin/blog/plugins/openid-comment/openid-comment.pl line 83.

And on your site I get ‘Insertion test failed on SQL error Duplicate entry ‘0’ for key 1’ when trying to sign-in using OpenID for both my LiveJournal and TypeKey accounts.

And on here you’ve put ‘profiles.typekey.com/name’ - it’s ‘profile.typekey.com/name)’. Otherwise you get a nasty ‘could not connect’ error :) .

Neil T.

comment

Oh, in the hope that this narrows things down - the SQL Insertion bug does not seem to appear on my blog.

[LiveJournal user info] assert(♥)

comment

The Can't call method pic_url error should go away once someone signs in and the schema is upgraded, I think?

Yeah, someone else got that error, I had forgot about it. Good to know you aren’t getting it (I did a pretty manual sidegrade to MySQL recently so it could be entirely that).

And yeah, it’s profile, totally my bad as well. Thanks!

[LiveJournal user info] assert(♥)

comment

I was misreading the Can't call method error. Apparently that means your database is inconsistent: you have authors with names starting with openid\n that don’t have a corresponding MT::Plugin::OpenIDIdentity object… Not sure what to recommend, but I made it not error there at least.

The insertion test error was a migration issue, yeah: my openididentity_id field wasn’t set auto_increment. Oops.

Here’s a 1.6.1.

[OpenID Commenter Profile] totalxsive [typekey.com]

comment

Excellent. It’s working fine now, thanks :) .