Page MenuHomePhabricator
Differential D12236

Lift common code for transaction-based reply handlers into parent class
ClosedPublic
Actions

Authored by epriestley on Apr 1 2015, 1:51 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Feb 8, 3:21 AM
Unknown Object (File)
Sat, Feb 8, 3:21 AM
Unknown Object (File)
Sat, Feb 8, 3:21 AM
Unknown Object (File)
Sat, Feb 8, 3:21 AM
Unknown Object (File)
Mon, Feb 3, 1:14 PM
Unknown Object (File)
Wed, Jan 29, 7:43 AM
Unknown Object (File)
Sun, Jan 26, 11:13 PM
Unknown Object (File)
Fri, Jan 24, 2:00 AM
View All Files
Subscribers
epriestley

Details

Reviewers
btrahan
Maniphest Tasks
T7199: Expand the power of "!action" commands in email
Commits
Restricted Diffusion Commit
rP161f936871b8: Lift common code for transaction-based reply handlers into parent class
Summary

Ref T7199. Essentially all of the reply handlers now apply transactions to something which implements PhabricatorApplicationTransactionInterface.

We can share code between them by lifting this stuff into a superclass.

First, convert paste. Also rename PasteMockMailReceiver to PasteMailReceiver (this got mis-copied from Pholio at some point, I think).

Test Plan

Used bin/mail receive-test to send comments + !unsubscribe to pastes.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 29395.Apr 1 2015, 1:51 AM
epriestley retitled this revision from to Lift common code for transaction-based reply handlers into parent class.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added a task: T7199: Expand the power of "!action" commands in email.
Herald added a subscriber: epriestley. · View Herald TranscriptApr 1 2015, 1:51 AM
btrahan accepted this revision.Apr 1 2015, 3:07 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Apr 1 2015, 3:07 PM
Closed by commit rP161f936871b8: Lift common code for transaction-based reply handlers into parent class (authored by epriestley, committed by epriestley). · Explain WhyApr 1 2015, 3:39 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
8 lines
applications/
paste/
mail/
PasteMailReceiver.php
PasteMockMailReceiver.php
2 lines
46 lines
transactions/
replyhandler/
92 lines

Diff 29406

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/paste/mail/PasteMailReceiver.php

Loading...
View Options

src/applications/paste/mail/PasteMockMailReceiver.php

Loading...
View Options

src/applications/paste/mail/PasteReplyHandler.php

Loading...
View Options

src/applications/transactions/replyhandler/PhabricatorApplicationTransactionReplyHandler.php

Loading...
Phabricator · Built by Phacility