Closed (fixed)
Project:
OpenID
Version:
5.x-1.x-dev
Component:
OpenID Client
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
9 Apr 2007 at 20:54 UTC
Updated:
11 Aug 2007 at 03:49 UTC
A user logs in with a MyOpenID.com URL, or any openID url i suppose.
If you then go to edit the account, Drupal grumbles at the username containing illegal characters, and the required email address is missing.
The email address can be forgiven maybe, but the illegal characters issue is a pain, and probably a bit of a problem if the user changes it?
Comments
Comment #1
boris mann commentedBoth of these should be solved by implementing OpenID SREG -- aka simple registration extension.
Note: all Drupal distributed auth systems that don't use email as a key have this problem -- including the Drupal module currently in core.
The best use case would be, on login, to immediately go to user/edit (or some clone of it with just the required fields).
Comment #2
buddaIf the new user is pushed on to the user edit page, would it not mess things up if they edited their username? Next time they log in to the site I assume it would create a new user account again with the original openID url - giving the user two accounts, one of which they can no longer access because they changed the username.
of course this all depends if the user even has the access control to change his own username. If he doesn't then the form validation would never let him make any changes.
Maybe the OpenID module could replace the standard user edit form validation through hook_form_alter() ?
Comment #3
buddaI found this only occurs if the user has not set up their openid with a default persona containing a username.
Another OpenID module I used checked for this, and asked for a username - or to at least confirm the username they wanted to use whilst on the Drupal site. That was a nice touch. The module can be found at http://www.jirwin.net/?q=node/5
Comment #4
walkah commentedfixed. all accounts created now must be valid.
Comment #5
(not verified) commented