Page MenuHomePhabricator
Differential D10786

Change usernames to `sort64` to make them case-insensitive
ClosedPublic
Actions

Authored by epriestley on Nov 5 2014, 12:38 PM.
Tags
None
Referenced Files
F13180275: D10786.id25884.diff
Wed, May 8, 10:53 PM
F13179941: D10786.diff
Wed, May 8, 9:33 PM
F13176646: D10786.id25893.diff
Wed, May 8, 3:55 PM
Unknown Object (File)
Tue, May 7, 10:08 AM
Unknown Object (File)
Fri, May 3, 8:52 PM
Unknown Object (File)
Fri, May 3, 1:01 PM
Unknown Object (File)
Fri, May 3, 10:15 AM
Unknown Object (File)
Sat, Apr 27, 12:14 PM
View All Files
Subscribers
epriestley

Details

Reviewers
btrahan
Maniphest Tasks
T1191: Comments are truncated at first non-base-plane character
Commits
Restricted Diffusion Commit
rPf7fbf9ccc174: Change usernames to `sort64` to make them case-insensitive
Summary

Ref T1191. After adjustment, usernames currently end up case-sensitive, which means alincoln and Alincoln are different users. Make them case-sensitive so these names collie.

Test Plan

Ran bin/storage adjust.

Diff Detail

Repository
rP Phabricator
Branch
usercase
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 2949
Build 2953: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 25884.Nov 5 2014, 12:38 PM
epriestley retitled this revision from to Change usernames to `sort64` to make them case-insensitive.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added a task: T1191: Comments are truncated at first non-base-plane character.
Herald added a subscriber: epriestley. · View Herald TranscriptNov 5 2014, 12:38 PM
Harbormaster completed remote builds in B2949: Diff 25884.Nov 5 2014, 12:39 PM
btrahan accepted this revision.Nov 5 2014, 6:10 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Nov 5 2014, 6:10 PM
Closed by commit rPf7fbf9ccc174: Change usernames to `sort64` to make them case-insensitive (authored by epriestley, committed by epriestley). · Explain WhyNov 5 2014, 8:32 PM
This revision was automatically updated to reflect the committed changes.
bluehawk added a subscriber: locutus.Nov 6 2014, 5:25 PM
locutus removed a subscriber: locutus.Nov 6 2014, 5:56 PM
epriestley mentioned this in D10806: Make user emails case-insensitive.Nov 7 2014, 1:17 PM
epriestley mentioned this in rPaa0ca6fe4c22: Make user emails case-insensitive.Nov 7 2014, 5:48 PM

Revision Contents
Changeset List

PathSize
src/
applications/
people/
storage/
2 lines

Diff 25884

View Options

src/applications/people/storage/PhabricatorUser.php

Loading...
Phabricator · Built by Phacility