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
F19312966: D19642.id46936.diff
Thu, Dec 25, 12:57 AM
F19018922: D19642.id46936.diff
Nov 23 2025, 12:45 PM
F18866374: D19642.id.diff
Nov 3 2025, 4:48 PM
F18860737: D19642.diff
Nov 2 2025, 12:32 PM
F18832437: D19642.id46949.diff
Oct 25 2025, 9:54 PM
F18832436: D19642.id46936.diff
Oct 25 2025, 9:54 PM
F18832435: D19642.id.diff
Oct 25 2025, 9:54 PM
F18831612: D19642.diff
Oct 25 2025, 1:48 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