Last modified: 2013-06-18 15:54:03 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 T18075, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 16075 - Special:RenameUser should ignore the case of the first letter of the current username
Special:RenameUser should ignore the case of the first letter of the current ...
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
Renameuser (Other open bugs)
unspecified
All All
: Low normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-10-23 15:19 UTC by Nicholas
Modified: 2013-06-18 15:54 UTC (History)
5 users (show)

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


Attachments

Description Nicholas 2008-10-23 15:19:55 UTC
http://en.wikipedia.org/w/index.php?title=Special:RenameUser

Mediawiki should ignore the case of the first letter of the current username. Too many false positives reported when renaming.
Comment 1 X! 2008-10-23 16:54:55 UTC
I just added a fix to the CHU template that automatically uppercases the names onwiki.
Comment 2 Brion Vibber 2008-10-23 17:10:23 UTC
False positive whats?

Note that Special:RenameUser needs to be able to take lowercase usernames as input on wikis supporting lowercase page titles, since old user accounts may have to be fixed.
Comment 3 Chad H. 2009-01-02 18:31:42 UTC
Fixed in r45331. RenameUser was force-capitalizing all target usernames, even if the wiki allows lowercase names. Allowing Title to handle the normalization for us fixes this.
Comment 4 Chad H. 2009-01-02 19:10:38 UTC
Self-reverted in r16075.
Comment 5 Aryeh Gregor (not reading bugmail, please e-mail directly) 2009-01-02 19:12:02 UTC
That should be r45332.  16075 is this bug's number.  :)

We should check for the lowercase username first, and if that doesn't exist try the uppercase one.
Comment 6 Chad H. 2009-02-04 23:09:06 UTC
(In reply to comment #2)
> False positive whats?
> 
> Note that Special:RenameUser needs to be able to take lowercase usernames as
> input on wikis supporting lowercase page titles, since old user accounts may
> have to be fixed.
> 

Agreed. There shouldn't be anything wrong with how RenameUser does things at the moment.

We accept upper or lower-case user names (as some lower-cased ones could still exist and need renaming). Output should always be first-letter upper-cased, as User is hardcoded to only allow first-letter upper-cased names. The current behavior is exactly this, and I don't see any reason it should change. Pending more clarification as to what the actual issue is, I suggest WORKSFORME, as this is all designed as intended.
Comment 7 Andrew Garrett 2009-02-04 23:13:01 UTC
Changing bug summary to remove weird jargon.
Comment 8 Brion Vibber 2009-02-04 23:13:48 UTC
Clarifying subject in summary.
Comment 9 Siebrand Mazeland 2012-05-22 11:15:07 UTC
The comment message of r45332 mentioned in comment 4 states "Because User is hardcoded to only allow upper-cased-first usernames, this breaks login and pretty much everything in the User class. The (coming?) changes to $wgCapitalLinks should fix this."

Has that ever happened, and if so, is this still an issue?
Comment 10 Chad H. 2012-05-22 12:43:41 UTC
(In reply to comment #9)
> The comment message of r45332 mentioned in comment 4 states "Because User is
> hardcoded to only allow upper-cased-first usernames, this breaks login and
> pretty much everything in the User class. The (coming?) changes to
> $wgCapitalLinks should fix this."
> 
> Has that ever happened, and if so, is this still an issue?

It shouldn't be an issue anymore--unless wikis with $wgCapitalLinks = false are still wanting lowercased usernames.

When we did the $wgCapitalLink(Override)s work, we forced NS_USER to always be ucfirst and the User class stayed the same.

There might be some issues with dangling lowercased pages in NS_USER(_TALK) from wikis with $wgCapitalLinks = false that existed before the fixes--but that wouldn't be part of this bug.
Comment 11 Chad H. 2012-05-22 12:44:21 UTC
(Pressed enter too soon)

So yeah: I'd be inclined to mark this FIXED, but someone should definitely test it out because it's been *ages* since I worked on this.
Comment 12 Siebrand Mazeland 2012-05-22 12:53:11 UTC
Marking resolved based on comment 11.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links