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

	Unknown Object (File)
	Wed, Apr 24, 11:54 PM

	Unknown Object (File)
	Wed, Apr 24, 10:47 AM

	Unknown Object (File)
	Thu, Apr 11, 8:10 AM

	Unknown Object (File)
	Mon, Apr 1, 2:55 PM

	Unknown Object (File)
	Feb 16 2024, 5:29 AM

	Unknown Object (File)
	Jan 31 2024, 1:36 PM

	Unknown Object (File)
	Dec 23 2023, 4:11 AM

	Unknown Object (File)
	Dec 19 2023, 10:42 PM

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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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 30033

src/applications/conpherence/view/ConpherencePeopleWidgetView.php

Loading...