Last modified: 2010-01-02 22:04:49 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T23857, the corresponding Phabricator task for complete and up-to-date bug report information.
First Last Prev Next    This bug is not in your last search results.
Bug 21857 - User attribute initialization should be forced when user is created
User attribute initialization should be forced when user is created
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
OpenID (Other open bugs)
unspecified
All All
: Normal enhancement (vote)
: ---
Assigned To: Sergey Chernyshev
: patch
Depends on:
Blocks:
  Show dependency tree / graph
 
Reported: 2009-12-15 18:08 UTC by Sergey Chernyshev
Modified: 2010-01-02 22:04 UTC (History)
0 users

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---

Attachments
Forces user attribute initialization when user is created (1.02 KB, patch)
2009-12-15 18:18 UTC, Sergey Chernyshev 		Details
Add an attachment (proposed patch, testcase, etc.)

Description Sergey Chernyshev 2009-12-15 18:08:46 UTC 
Code was refactored to use SpecialOpenIDLogin::updateUser within SpecialOpenIDLogin::createUser which is affected by user OpenID preferences which are not necessarily set up properly for new user (see also bug 19347).

It's better to add "force" parameter to SpecialOpenIDLogin::updateUser to use when called from SpecialOpenIDLogin::createUser that will ignore user's preferences.

Discussion on the mailing list:
http://groups.google.com/group/mediawiki-openid/browse_thread/thread/8aef4327cf268090
Comment 1 Sergey Chernyshev 2009-12-15 18:18:48 UTC 
Created attachment 6884 [details]
Forces user attribute initialization when user is created

Didn't have time to test it, but the change is pretty simple so hoping to add this to the code soon.
Comment 2 Sergey Chernyshev 2010-01-02 22:04:49 UTC 
Tested and it works. Checked in as r60565.
Note You need to log in before you can comment on or make changes to this bug.
First Last Prev Next    This bug is not in your last search results.

Navigation
Links