Will Norris

try rebuilding the tables from the bottom of the wp-openid config screen.

that’s pretty odd. I’m not at my normal workstation right now, but will take a look at it tomorrow. If you haven’t already, please add a bug report to the DiSo issue tracker so I can follow it up there.

admittedly, we haven’t done a real good job of testing new versions of the plugin against wpmu. That “files” rewriting thing sounds pretty crappy… I’ll definitely have to take a look at that. Sounds like a bug in wpmu, as I can’t understand why they would have a rewrite rules that grabs *all* folders with that name. Please add a bug to the DiSo issue tracker if there isn’t already one there regarding wpmu compatibility… we definitely don’t want to break that.

if you are comfortable with using subversion, I’d recommend taking a look at my latest blog post about wp-openid. I believe a lot of the changes I made cleaned up the loading code. If it looks like it is still loading unnecessarily, please file a bug in the DiSo issue tracker if it’s not already there.

you’re absolutely right… there is still a bug. I’m not at my normal workstation right now, so I’ll have to fix it tomorrow when I’m back in the office.

We are certainly aware of the ID Selector, and are looking at adding the option of using it with wp-openid. Because you have to sign-up with idselector.com, it provides a few minor challenges, but we are looking into it.

yes, a lot of the DB code has been needing work for a while… I’ll see if I can clean that stuff up a bit. I’ll follow up in this thread with what I come up with.

ahh… didn’t realize that was only in php5. I’ve applied your change and tagged v1.2. Thanks.

first attempt at localization has been checked into trunk (in DiSo repository). There’s still a ways to go, but it should give translators enough to get started.

https://www.ads-software.com/extend/plugins/openid/other_notes/

If you want to view the actual code changes:
https://dev.wp-plugins.org/changeset?new=openid%2Ftrunk%4047054&old=openid%2Ftrunk%4037878

yes, we’ll probably look into integrating it, though not sure when that will happen. There are several higher priority items, plus we’ll have to figure out the best user experience for it, since the javascript for the ID selector is site-specific at the moment. They are supposed to be working on a non-hosted version, which would likely be the preferred route for including it into wp-openid.

fixed in DiSo repository trunk… will be included in the next release.

@link2caro, I’ve committed versions of a couple of your patches here:

Your patch for the create_new_user() would have broken backward compatibility, but I’ve fixed that… thanks.

I don’t really like the idea of prepending “oid-” to user logins… the idea is for a more seamless experience for all users, regardless of whether they use OpenID. I also don’t want to hide a user’s Identity URL from them or completely prevent them from being able to remove it. We as programmers don’t necessarily know what is better for the user… only they do. What I did do however is to add a warning prompt if a user is about to remove the last OpenID for their account. It alerts them that it may interfere with their ability to login, but it still allows them to go ahead with it if they so choose. I think that should be a happy medium.

These changes, and a few other minor ones can currently be found in the DiSo repository. I’ll cut a new release sometime this week after I get a few more bugs patched.

Good point. would you mind filing an enhancement request at:
https://code.google.com/p/diso/issues/list

I can’t promise when I’ll get to it, but that’s the best place to record it.

duplicate of https://www.ads-software.com/support/topic/165141