Page MenuHomePhabricator

Make all revision status readers explicitly read modern or legacy status
ClosedPublic

Authored by epriestley on Aug 11 2017, 11:23 PM.
Tags
None
Referenced Files
F15573925: D18417.id44261.diff
Mon, May 5, 10:36 PM
F15571165: D18417.diff
Mon, May 5, 4:35 AM
F15557215: D18417.id.diff
Mon, Apr 28, 8:30 PM
F15554466: D18417.id44252.diff
Mon, Apr 28, 7:13 AM
F15548887: D18417.diff
Sun, Apr 27, 2:43 AM
F15513532: D18417.id44261.diff
Thu, Apr 17, 7:36 PM
F15485596: D18417.diff
Thu, Apr 10, 12:56 AM
F15461258: D18417.id44261.diff
Apr 1 2025, 6:46 AM
Subscribers
None

Details

Summary

Ref T2543. All writers now write modern statuses. Make all readers explicit about whether they are reading modern or legacy statuses, so I can swap the storage format.

Test Plan
  • Grepped for getStatus(), scanned the list. Other applications have methods with this name so it's possible I missed something.
  • Browed around, changed revision statuses.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable