Page MenuHomePhabricator

Begin transitioning audits to modern (string) status constants, from legacy (integer) status constants
ClosedPublic

Authored by epriestley on Sep 5 2018, 5:32 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mar 5 2024, 1:43 AM
Unknown Object (File)
Dec 14 2023, 11:02 AM
Unknown Object (File)
Nov 30 2023, 4:15 AM
Unknown Object (File)
Nov 17 2023, 2:14 AM
Unknown Object (File)
Oct 15 2023, 2:13 AM
Unknown Object (File)
Sep 8 2023, 2:10 PM
Unknown Object (File)
Aug 31 2023, 1:38 PM
Unknown Object (File)
Jul 23 2023, 2:30 PM
Subscribers
None

Details

Summary

Ref T13195. See PHI851. Audits currently have older integer status constants. We've moved almost all object types away from this to string constants (which are better in basically every way, and particularly way better for exposing over the API).

Commits/audits are currently accessible over the API and expose these constants via a "statuses" constraint.

Prepare to move toward modern string constants by defining a new, more modern map of status details and defining the existing methods in terms of it.

Test Plan

Browsed audits checking for icons/names/open-ness, saw no changes. This change should have no user-visible effects, as it just reorganizes code.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable