Allow ApplicationTransactionEditor to figure out whether TYPE_COMMENT is supported or not
ClosedPublic
Actions

Authored by epriestley on May 3 2017, 5:43 PM.

Details

Reviewers

Commits

rP60a19e36463b: Allow ApplicationTransactionEditor to figure out whether TYPE_COMMENT is…

Summary

See D17812, etc. We can figure this out by looking at the object carefully. We don't need to go delete all the old TYPE_COMMENT (it doesn't hurt anything) but can nuke it when we see it.

Test Plan

Made a comment in Slowvote (supports commenting).
Viewed an Almanac device (does not support commenting).

Diff Detail

Repository

rP Phabricator

Branch

comment1

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 16777
Build 22387: Run Core Tests
Build 22386: arc lint + arc unit

Event Timeline

epriestley created this revision.May 3 2017, 5:43 PM

Harbormaster completed remote builds in B16777: Diff 42851.May 3 2017, 5:44 PM

chad accepted this revision.May 3 2017, 5:48 PM

This revision is now accepted and ready to land.May 3 2017, 5:48 PM

Macro diff assumes this will land

Closed by commit rP60a19e36463b: Allow ApplicationTransactionEditor to figure out whether TYPE_COMMENT is… (authored by epriestley, committed by epriestley). · Explain WhyMay 3 2017, 6:21 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

slowvote/

editor/

PhabricatorSlowvoteEditor.php

1 line

transactions/

editor/

PhabricatorApplicationTransactionEditor.php

12 lines

Diff 42851

View Options

src/applications/slowvote/editor/PhabricatorSlowvoteEditor.php