Differential D19291

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

Authored by epriestley on Apr 3 2018, 5:04 PM.

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

Branch

bengine14

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 19978
Build 27091: Run Core Tests
Build 27090: arc lint + arc unit

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

Path

Size

src/

applications/

transactions/

storage/

PhabricatorModularTransaction.php

4 lines

Diff 46157

View Options

src/applications/transactions/storage/PhabricatorModularTransaction.php

Move PhabricatorModularTransaction slightly closer to having "final" methods againClosedPublicActions