Page MenuHomePhabricator

Reduce STATUS_CLOSED (now internally "Published") revision status callsites
ClosedPublic

Authored by epriestley on Aug 4 2017, 3:23 PM.
Tags
None
Referenced Files
F18871271: D18341.id44160.diff
Tue, Nov 4, 9:56 PM
F18850750: D18341.id.diff
Thu, Oct 30, 7:07 PM
F18845155: D18341.diff
Wed, Oct 29, 10:11 AM
F18820201: D18341.id.diff
Oct 22 2025, 1:15 PM
F18812594: D18341.id44094.diff
Oct 20 2025, 10:42 AM
F18720452: D18341.id.diff
Sep 29 2025, 6:55 PM
F18705189: D18341.diff
Sep 28 2025, 10:26 AM
F18568152: D18341.id44160.diff
Sep 9 2025, 11:04 PM
Subscribers
None

Details

Summary

Ref T2543. Add isPublished() to mean: exactly the status 'closed', which is now interally called 'published', but still shown as 'closed' to users.

We have some callsites which are about "exactly that status", vs "any 'closed' status", e.g. including "abandoned".

This also introduces isChangePlanned(), which felt less awkward than isChangesPlanned() but more consistent than hasChangesPlanned() or isStatusChangesPlanned() or similar.

Test Plan

grep, loaded revisions, requested review.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable