Differential D20890

Allow username changes which modify letter case to go through as valid
ClosedPublic
Actions

Authored by epriestley on Nov 6 2019, 4:08 PM.

Tags

None

Referenced Files

	F14441942: D20890.diff
	Thu, Dec 26, 7:20 AM

	F14435922: D20890.id49796.diff
	Wed, Dec 25, 11:42 PM

	Unknown Object (File)
	Fri, Dec 20, 4:57 AM

	Unknown Object (File)
	Sat, Dec 14, 10:37 AM

	Unknown Object (File)
	Sun, Dec 8, 9:14 PM

	Unknown Object (File)
	Tue, Dec 3, 12:52 AM

	Unknown Object (File)
	Wed, Nov 27, 5:18 PM

	Unknown Object (File)
	Wed, Nov 27, 1:27 PM

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13446: Allow username changes to change the letter case of an existing username
Commits: rPf5f2a0bc5696: Allow username changes which modify letter case to go through as valid

Summary

Fixes T13446. Currently, the validation logic here rejects a rename like "alice" to "ALICE" (which changes only letter case) but this is a permissible rename.

Allow collisions that collide with the same user to permit this rename.

Also, fix an issue where an empty rename was treated improperly.

Test Plan

Renamed "alice" to "ALICE".
- Before: username collision error.
- After: clean rename.
Renamed "alice" to "orange" (an existing user). Got an error.
Renamed "alice" to "", "!@#$", etc (invalid usernames). Got sensible errors.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Nov 6 2019, 4:08 PM

Harbormaster completed remote builds in B23626: Diff 49796.Nov 6 2019, 4:10 PM

This revision was not accepted when it landed; it landed in state Needs Review.Nov 6 2019, 4:10 PM

epriestley requested review of this revision.

Closed by commit rPf5f2a0bc5696: Allow username changes which modify letter case to go through as valid (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rPf5f2a0bc5696: Allow username changes which modify letter case to go through as valid.

Revision Contents
Changeset List

Path

Size

src/

applications/

people/

xaction/

PhabricatorUserUsernameTransaction.php

21 lines

Diff 49797

src/applications/people/xaction/PhabricatorUserUsernameTransaction.php

Loading...