Page MenuHomePhabricator

Conpherence - sort participant list in a case insensitive way
ClosedPublic

Authored by btrahan on Apr 22 2015, 7:02 PM.
Tags
None
Referenced Files
F18944845: D12507.id30033.diff
Tue, Nov 11, 5:04 PM
F18864521: D12507.diff
Mon, Nov 3, 5:54 AM
F18842658: D12507.id.diff
Oct 28 2025, 5:25 PM
F18838951: D12507.diff
Oct 27 2025, 1:38 PM
F18682150: D12507.id30033.diff
Sep 26 2025, 5:15 AM
F18680907: D12507.id30033.diff
Sep 26 2025, 3:26 AM
F18647173: D12507.diff
Sep 19 2025, 12:29 PM
F18639431: D12507.id30028.diff
Sep 17 2025, 1:56 PM
Subscribers

Details

Summary

Fixes T7857. Some sorting gymnastics - make a nice list for natcasesort and then array_select_key to sort - but nothing too crazy.

Test Plan

made a room with user "btrahan" and other users "BTRAHA", "BTRAHANCAPS", and "xerxes". chat room correctly ordered "btrahan" (since logged in user always at top), "BTRAHA", "BTRAHANCAPS", "xerxes". Viewed room as "xerxes" and saw ordering "xerxes", "BTRAHA", "btrahan", "BTRAHANCAPS".

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

btrahan retitled this revision from to Conpherence - sort participant list in a case insensitive way.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Apr 22 2015, 7:12 PM
This revision was automatically updated to reflect the committed changes.