Page MenuHomePhabricator
Differential D19290

Clean up an old transaction state flag
ClosedPublic
Actions

Authored by epriestley on Apr 3 2018, 5:00 PM.
Tags
None
Referenced Files
F19495680: D19290.id46172.diff
Thu, Jan 8, 10:34 AM
F19362924: D19290.diff
Fri, Dec 26, 10:51 AM
F19019711: D19290.diff
Nov 23 2025, 3:42 PM
F19004110: D19290.id.diff
Nov 21 2025, 10:30 AM
F18759099: D19290.id46172.diff
Oct 6 2025, 1:42 AM
F18756421: D19290.diff
Oct 5 2025, 11:23 AM
F18755919: D19290.id46156.diff
Oct 5 2025, 8:14 AM
F18749106: D19290.diff
Oct 4 2025, 2:24 AM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13110: Plans: Hefty Differential revisions, draft state transitions, and exotic interactions
Commits
rP6f520e053462: Clean up an old transaction state flag
Summary

Depends on D19289. Ref T13110. This flag has been obsolete for some time and has no callers.

Test Plan

Grepped for hasReviewTransaction, no hits.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Apr 3 2018, 5:00 PM
Harbormaster completed remote builds in B19977: Diff 46156.Apr 3 2018, 5:01 PM
epriestley requested review of this revision.Apr 3 2018, 5:01 PM
epriestley added a child revision: D19291: Move PhabricatorModularTransaction slightly closer to having "final" methods again.Apr 3 2018, 5:04 PM
This revision was not accepted when it landed; it landed in state Needs Review.Apr 3 2018, 6:13 PM
Closed by commit rP6f520e053462: Clean up an old transaction state flag (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
differential/
editor/
8 lines

Diff 46172

View Options

src/applications/differential/editor/DifferentialTransactionEditor.php

Loading...
Phabricator · Built by Phacility