Use TransactionEditor in Differential mail handling

Summary: Ref T2222. Moves this instance of CommentEditor to TransactionEditor.

Test Plan: Used `bin/mail receive-test` to test receiving comment mail and action mail.

Reviewers: btrahan

Reviewed By: btrahan

CC: aran

Maniphest Tasks: T2222

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

src/applications/differential/mail/DifferentialReplyHandler.php

@@ -129,28 +129,48 @@ class DifferentialReplyHandler extends PhabricatorMailReplyHandler {
 
     $body = $this->enhanceBodyWithAttachments($body, $attachments);
 
-    try {
+    $xactions = array();
-      $editor = new DifferentialCommentEditor(
-        $this->getMailReceiver(),
-        $command);
-      $editor->setActor($actor);
-      $editor->setExcludeMailRecipientPHIDs(
-        $this->getExcludeMailRecipientPHIDs());
 
-      // NOTE: We have to be careful about this because Facebook's
+    if ($command && ($command != DifferentialAction::ACTION_COMMENT)) {
-      // implementation jumps straight into handleAction() and will not have
+      $xactions[] = id(new DifferentialTransaction())
-      // a PhabricatorMetaMTAReceivedMail object.
+        ->setTransactionType(DifferentialTransaction::TYPE_ACTION)
-      if ($this->receivedMail) {
+        ->setNewValue($command);
-        $content_source = PhabricatorContentSource::newForSource(
+    }
-          PhabricatorContentSource::SOURCE_EMAIL,
+
-          array(
+    if (strlen($body)) {
-            'id' => $this->receivedMail->getID(),
+      $xactions[] = id(new DifferentialTransaction())
-          ));
+        ->setTransactionType(PhabricatorTransactions::TYPE_COMMENT)
-        $editor->setContentSource($content_source);
+        ->attachComment(
-        $editor->setParentMessageID($this->receivedMail->getMessageID());
+          id(new DifferentialTransactionComment())
-      }
+            ->setContent($body));
-      $editor->setMessage($body);
+    }
-      $editor->save();
+
+    $editor = id(new DifferentialTransactionEditor())
+      ->setActor($actor)
+      ->setExcludeMailRecipientPHIDs($this->getExcludeMailRecipientPHIDs())
+      ->setContinueOnMissingFields(true)
+      ->setContinueOnNoEffect(true);
+
+    // NOTE: We have to be careful about this because Facebook's
+    // implementation jumps straight into handleAction() and will not have
+    // a PhabricatorMetaMTAReceivedMail object.
+    if ($this->receivedMail) {
+      $content_source = PhabricatorContentSource::newForSource(
+        PhabricatorContentSource::SOURCE_EMAIL,
+        array(
+          'id' => $this->receivedMail->getID(),
+        ));
+      $editor->setContentSource($content_source);
+      $editor->setParentMessageID($this->receivedMail->getMessageID());
+    } else {
+      $content_source = PhabricatorContentSource::newForSource(
+        PhabricatorContentSource::SOURCE_LEGACY,
+        array());
+      $editor->setContentSource($content_source);
+    }
+
+    try {
+      $editor->applyTransactions($this->getMailReceiver(), $xactions);
     } catch (Exception $ex) {
       if ($this->receivedMail) {
         $error_body = $this->receivedMail->getRawTextBody();

src/applications/differential/mail/DifferentialRevisionMailReceiver.php

@@ -18,6 +18,7 @@ final class DifferentialRevisionMailReceiver
     $results = id(new DifferentialRevisionQuery())
       ->setViewer($viewer)
       ->withIDs(array($id))
+      ->needReviewerStatus(true)
       ->execute();
 
     return head($results);