Paths

Table of Contentst

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

	Unknown Object (File)
	Thu, Nov 28, 3:29 PM

	Unknown Object (File)
	Wed, Nov 27, 10:44 PM

	Unknown Object (File)
	Tue, Nov 26, 7:43 PM

	Unknown Object (File)
	Tue, Nov 19, 5:02 AM

	Unknown Object (File)
	Oct 21 2024, 4:44 PM

	Unknown Object (File)
	Oct 20 2024, 5:56 PM

	Unknown Object (File)
	Oct 9 2024, 9:19 AM

	Unknown Object (File)
	Oct 9 2024, 9:19 AM

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

Branch

bengine13

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 19977
Build 27089: Run Core Tests
Build 27088: arc lint + arc unit

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

Path

Size

src/

applications/

differential/

editor/

DifferentialTransactionEditor.php

8 lines

Diff 46156

src/applications/differential/editor/DifferentialTransactionEditor.php

Loading...