Touch up Badges emails

Summary: Ref T12270. Adds the name of the badge to the subject, fixes the double description.

Test Plan: Edit lots of badges with and without descriptions, see good emails.

Reviewers: epriestley

Reviewed By: epriestley

Subscribers: Korvin

Maniphest Tasks: T12270

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

Author: Chad Little

src/applications/badges/editor/PhabricatorBadgesEditor.php

@@ -87,10 +87,12 @@ protected function buildReplyHandler(PhabricatorLiskDAO $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) {
@@ -104,15 +106,8 @@ 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().'/'));