Page MenuHomePhabricator

Make legacy revision statuses from "differential.query" have type "string" again
ClosedPublic

Authored by epriestley on Aug 29 2017, 7:44 PM.
Tags
None
Referenced Files
F19946888: D18493.diff
Fri, Apr 17, 3:21 AM
F19910858: D18493.id44421.diff
Thu, Apr 2, 9:57 AM
F19903603: D18493.id.diff
Tue, Mar 31, 2:39 AM
F19870908: D18493.diff
Mar 15 2026, 8:51 AM
F19813078: D18493.diff
Mar 4 2026, 7:35 AM
F19509760: D18493.diff
Jan 11 2026, 12:46 AM
F19500623: D18493.diff
Jan 8 2026, 2:32 PM
F19500613: D18493.diff
Jan 8 2026, 2:31 PM
Subscribers
None

Details

Summary

Ref T2543. The type on these got changed by accident, it should be "string" (crazy nonsense, compatible) not "int" (sensible, not compatible).

(New API uses sensible strings like "accepted" only.)

Test Plan

Called differential.query from web UI, saw "2" and similar statuses.

Diff Detail

Repository
rP Phabricator
Branch
ptype
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 18207
Build 24483: Run Core Tests
Build 24482: arc lint + arc unit