diff --git a/src/applications/badges/editor/PhabricatorBadgesEditor.php b/src/applications/badges/editor/PhabricatorBadgesEditor.php index 48057f66e6..a077f4c3ba 100644 --- a/src/applications/badges/editor/PhabricatorBadgesEditor.php +++ b/src/applications/badges/editor/PhabricatorBadgesEditor.php @@ -1,126 +1,121 @@ pht('Someone changes the badge\'s details.'), PhabricatorBadgesTransaction::MAILTAG_COMMENT => pht('Someone comments on a badge.'), PhabricatorBadgesTransaction::MAILTAG_OTHER => pht('Other badge activity not listed above occurs.'), ); } protected function shouldPublishFeedStory( PhabricatorLiskDAO $object, array $xactions) { return true; } protected function expandTransactions( PhabricatorLiskDAO $object, array $xactions) { $actor = $this->getActor(); $actor_phid = $actor->getPHID(); $results = parent::expandTransactions($object, $xactions); // Automatically subscribe the author when they create a badge. if ($this->getIsNewObject()) { if ($actor_phid) { $results[] = id(new PhabricatorBadgesTransaction()) ->setTransactionType(PhabricatorTransactions::TYPE_SUBSCRIBERS) ->setNewValue( array( '+' => array($actor_phid => $actor_phid), )); } } return $results; } protected function buildReplyHandler(PhabricatorLiskDAO $object) { return id(new PhabricatorBadgesReplyHandler()) ->setMailReceiver($object); } protected function buildMailTemplate(PhabricatorLiskDAO $object) { $name = $object->getName(); $id = $object->getID(); - $name = pht('Badge %d', $id); + $topic = pht('Badge %d', $id); + $subject = pht('Badge %d: %s', $id, $name); + return id(new PhabricatorMetaMTAMail()) - ->setSubject($name) - ->addHeader('Thread-Topic', $name); + ->setSubject($subject) + ->addHeader('Thread-Topic', $topic); } protected function getMailTo(PhabricatorLiskDAO $object) { return array( $object->getCreatorPHID(), $this->requireActor()->getPHID(), ); } protected function buildMailBody( PhabricatorLiskDAO $object, array $xactions) { - $description = $object->getDescription(); $body = parent::buildMailBody($object, $xactions); - if (strlen($description)) { - $body->addRemarkupSection( - pht('BADGE DESCRIPTION'), - $object->getDescription()); - } - $body->addLinkSection( pht('BADGE DETAIL'), PhabricatorEnv::getProductionURI('/badges/view/'.$object->getID().'/')); return $body; } protected function getMailSubjectPrefix() { return pht('[Badge]'); } }