Allow PhutilSortVector to self-sort
ClosedPublic
Actions

Authored by epriestley on Mar 13 2018, 2:00 PM.

Tags

None

Referenced Files

	F19097684: D19212.id.diff
	Thu, Dec 4, 2:35 PM

	F19088680: D19212.diff
	Wed, Dec 3, 4:48 AM

	F18877587: D19212.id.diff
	Nov 6 2025, 10:36 AM

	F18859708: D19212.diff
	Nov 2 2025, 4:56 AM

	F18816330: D19212.diff
	Oct 21 2025, 6:05 AM

	F18672955: D19212.id46008.diff
	Sep 25 2025, 8:46 AM

	F18638779: D19212.diff
	Sep 17 2025, 11:28 AM

	F18509431: D19212.id.diff
	Sep 5 2025, 3:24 AM

View All Files

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13103: Document favicon customization and validate the "ui.favicons" configuration option
Commits: rPHUa3e02f01c34f: Allow PhutilSortVector to self-sort

Summary

Ref T13103 . msortv() requires a list which can be mpull()'d, so it's hard to use in an ad-hoc way. Allow a list of Vectors to sort itself.

Test Plan

See D19209.

Diff Detail

Repository

rPHU libphutil

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Mar 13 2018, 2:00 PM

Harbormaster completed remote builds in B19825: Diff 46007.Mar 13 2018, 2:01 PM

This revision was not accepted when it landed; it landed in state Needs Review.Mar 13 2018, 2:01 PM

epriestley requested review of this revision.

Closed by commit rPHUa3e02f01c34f: Allow PhutilSortVector to self-sort (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley edited the summary of this revision. (Show Details)Mar 13 2018, 4:30 PM

epriestley added a task: T13103: Document favicon customization and validate the "ui.favicons" configuration option.

Revision Contents
Changeset List

Path

Size

src/

utils/

PhutilSortVector.php

7 lines

Diff 46008

View Options

src/utils/PhutilSortVector.php

Allow PhutilSortVector to self-sortClosedPublicActions