From ca17e2283d92e9f49ed0ef2870895766e82827a3 Mon Sep 17 00:00:00 2001 From: Chad Little Date: Tue, 25 Jul 2017 13:25:57 -0700 Subject: [PATCH] Have Maniphest use create transactions when using email Summary: Fixes T12929. Sets a create transaction if new. Test Plan: test a new task over email via command line Reviewers: epriestley Reviewed By: epriestley Subscribers: Korvin Maniphest Tasks: T12929 Differential Revision: https://secure.phabricator.com/D18279 --- src/applications/maniphest/mail/ManiphestReplyHandler.php | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/applications/maniphest/mail/ManiphestReplyHandler.php b/src/applications/maniphest/mail/ManiphestReplyHandler.php index bbcfe86551..cc2bdff9d0 100644 --- a/src/applications/maniphest/mail/ManiphestReplyHandler.php +++ b/src/applications/maniphest/mail/ManiphestReplyHandler.php @@ -24,6 +24,10 @@ final class ManiphestReplyHandler $xactions = array(); if ($is_new) { + $xactions[] = $this->newTransaction() + ->setTransactionType(PhabricatorTransactions::TYPE_CREATE) + ->setNewValue(true); + $xactions[] = $this->newTransaction() ->setTransactionType(ManiphestTaskTitleTransaction::TRANSACTIONTYPE) ->setNewValue(nonempty($mail->getSubject(), pht('Untitled Task')));