Page MenuHomePhabricator

Correct "msort()" vs "msortv()" to more fully stabilize transaction sorts after recent changes
ClosedPublic

Authored by epriestley on Jun 17 2019, 5:40 PM.
Tags
None
Referenced Files
F15418970: D20583.id49092.diff
Fri, Mar 21, 2:45 AM
F15416803: D20583.id49092.diff
Thu, Mar 20, 1:37 PM
F15414292: D20583.id49088.diff
Thu, Mar 20, 12:01 AM
F15392554: D20583.diff
Sat, Mar 15, 3:55 PM
F15390327: D20583.id49092.diff
Sat, Mar 15, 6:07 AM
F15377255: D20583.id49088.diff
Thu, Mar 13, 8:28 AM
F15359082: D20583.diff
Tue, Mar 11, 7:40 AM
F15337585: D20583.id49092.diff
Sun, Mar 9, 6:38 AM
Subscribers
None

Details

Summary

Ref T13303. I upgraded this to a vector-based sort but forgot to type a "v", which means the sort has different stability under PHP 5.5. See D20582 for a root cause fix.

Test Plan

Locally, on PHP7, not much changes. I expect this to fix the odd selection of title stories in mail and notification stories on secure, which is running PHP 5.5.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable