Route reply handler Maniphest writes through new transaction code

Summary: Ref T2217. Mail writes go through the new code now.

Test Plan: Shipped a bunch of mail in with `./bin/mail`, got reasonable edits as a result.

Reviewers: btrahan

Reviewed By: btrahan

CC: aran

Maniphest Tasks: T2217

Differential Revision: https://secure.phabricator.com/D7086

src/applications/maniphest/mail/ManiphestReplyHandler.php

@@ -56,9 +56,7 @@ final class ManiphestReplyHandler extends PhabricatorMailReplyHandler {
         'id' => $mail->getID(),
       ));
 
-    $template = new ManiphestTransaction();
-    $template->setContentSource($content_source);
-    $template->setAuthorPHID($user->getPHID());
+    $template = new ManiphestTransactionPro();
 
     $is_unsub = false;
     if ($is_new_task) {
@@ -112,14 +110,24 @@ final class ManiphestReplyHandler extends PhabricatorMailReplyHandler {
           break;
       }
 
+      if ($ttype != PhabricatorTransactions::TYPE_COMMENT) {
         $xaction = clone $template;
         $xaction->setTransactionType($ttype);
         $xaction->setNewValue($new_value);
-      $xaction->setComments($body);
-
         $xactions[] = $xaction;
       }
 
+      if (strlen($body)) {
+        $xaction = clone $template;
+        $xaction->setTransactionType(PhabricatorTransactions::TYPE_COMMENT);
+        $xaction->attachComment(
+          id(new ManiphestTransactionComment())
+            ->setContent($body));
+        $xactions[] = $xaction;
+      }
+
+    }
+
     $ccs = $mail->loadCCPHIDs();
     $old_ccs = $task->getCCPHIDs();
     $new_ccs = array_merge($old_ccs, $ccs);
@@ -149,13 +157,14 @@ final class ManiphestReplyHandler extends PhabricatorMailReplyHandler {
     $task = $event->getValue('task');
     $xactions = $event->getValue('transactions');
 
-
-    $editor = new ManiphestTransactionEditor();
-    $editor->setActor($user);
-    $editor->setParentMessageID($mail->getMessageID());
-    $editor->setExcludeMailRecipientPHIDs(
-      $this->getExcludeMailRecipientPHIDs());
-    $editor->applyTransactions($task, $xactions);
+    $editor = id(new ManiphestTransactionEditorPro())
+      ->setActor($user)
+      ->setParentMessageID($mail->getMessageID())
+      ->setExcludeMailRecipientPHIDs($this->getExcludeMailRecipientPHIDs())
+      ->setContinueOnNoEffect(true)
+      ->setContinueOnMissingFields(true)
+      ->setContentSource($content_source)
+      ->applyTransactions($task, $xactions);
 
     $event = new PhabricatorEvent(
       PhabricatorEventType::TYPE_MANIPHEST_DIDEDITTASK,