Differential D12507

Conpherence - sort participant list in a case insensitive way
ClosedPublic
Actions

Authored by btrahan on Apr 22 2015, 7:02 PM.

Tags

None

Referenced Files

	F15532981: D12507.id30028.diff
	Wed, Apr 23, 5:49 PM

	F15436453: D12507.id30028.diff
	Mar 25 2025, 2:16 PM

	F15428505: D12507.id30028.diff
	Mar 23 2025, 8:52 PM

	F15403903: D12507.diff
	Mar 18 2025, 5:24 AM

	F15373490: D12507.id30028.diff
	Mar 12 2025, 4:26 PM

	Unknown Object (File)
	Feb 8 2025, 3:52 AM

	Unknown Object (File)
	Feb 8 2025, 3:52 AM

	Unknown Object (File)
	Feb 8 2025, 3:52 AM

Subscribers

Details

Reviewers

Maniphest Tasks

T7857: Conpherence participants are sorted by alpha, but with capital letters first

Commits

Restricted Diffusion Commit
rP2e8649c29231: Conpherence - sort participant list in a case insensitive way

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

Branch

T7857

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 5441
Build 5459: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

btrahan updated this revision to Diff 30028.Apr 22 2015, 7:02 PM

btrahan added a task: T7857: Conpherence participants are sorted by alpha, but with capital letters first.

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.

Herald added subscribers: epriestley, Korvin. · View Herald TranscriptApr 22 2015, 7:02 PM

epriestley accepted this revision.Apr 22 2015, 7:12 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Apr 22 2015, 7:12 PM

Closed by commit rP2e8649c29231: Conpherence - sort participant list in a case insensitive way (authored by btrahan, committed by btrahan). · Explain WhyApr 22 2015, 7:19 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

conpherence/

view/

ConpherencePeopleWidgetView.php

11 lines

Diff 30028

src/applications/conpherence/view/ConpherencePeopleWidgetView.php

Loading...