Page MenuHomePhabricator
Differential D19291

Move PhabricatorModularTransaction slightly closer to having "final" methods again
ClosedPublic
Actions

Authored by epriestley on Apr 3 2018, 5:04 PM.
Tags
None
Referenced Files
F18618360: D19291.diff
Sun, Sep 14, 11:54 PM
F18599446: D19291.id46173.diff
Sat, Sep 13, 7:26 AM
F18509505: D19291.id.diff
Fri, Sep 5, 3:26 AM
F18502561: D19291.diff
Thu, Sep 4, 10:31 PM
F18219324: D19291.id46157.diff
Aug 19 2025, 11:22 AM
F18201999: D19291.id.diff
Aug 18 2025, 9:13 AM
F18193815: D19291.diff
Aug 17 2025, 6:58 AM
F17829713: D19291.id46157.diff
Jul 26 2025, 10:18 AM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13110: Plans: Hefty Differential revisions, draft state transitions, and exotic interactions
Commits
rP592d72e00688: Move PhabricatorModularTransaction slightly closer to having "final" methods…
Summary

Depends on D19290. Ref T13110. Differential still has some hacks in place which require these methods to "very temporarily" be nonfinal, but the badness can be slightly reduced nowadays.

Test Plan

Loaded some pages, nothing fataled.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Apr 3 2018, 5:04 PM
Harbormaster completed remote builds in B19978: Diff 46157.Apr 3 2018, 5:05 PM
epriestley requested review of this revision.Apr 3 2018, 5:05 PM
This revision was not accepted when it landed; it landed in state Needs Review.Apr 3 2018, 6:14 PM
Closed by commit rP592d72e00688: Move PhabricatorModularTransaction slightly closer to having "final" methods… (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
transactions/
storage/
4 lines

Diff 46173

View Options

src/applications/transactions/storage/PhabricatorModularTransaction.php

Loading...
Phabricator · Built by Phacility