Adding pro tips to Differential commit message

Summary: Randomly displaying tips about different features of arc.

Test Plan: Will test by running arc diff and seeing if the tips appear in text-based UI of arc

Reviewers: epriestley, AnhNhan

Reviewed By: AnhNhan

CC: aran, Korvin, AnhNhan

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

Author: afaquejam

src/applications/differential/conduit/ConduitAPI_differential_getcommitmessage_Method.php

@@ -48,9 +48,12 @@ protected function execute(ConduitAPIRequest $request) {
     $aux_fields = DifferentialFieldSelector::newSelector()
       ->getFieldSpecifications();
 
+    $pro_tips = array();
+
     foreach ($aux_fields as $key => $aux_field) {
       $aux_field->setUser($request->getUser());
       $aux_field->setRevision($revision);
+      $pro_tips[] = $aux_field->getCommitMessageTips();
       if (!$aux_field->shouldAppearOnCommitMessage()) {
         unset($aux_fields[$key]);
       }
@@ -125,6 +128,26 @@ protected function execute(ConduitAPIRequest $request) {
         }
       }
     }
+
+    if ($is_edit) {
+      $pro_tips = array_mergev($pro_tips);
+
+      if (!empty($pro_tips)) {
+        shuffle($pro_tips);
+        $pro_tip = "Tip: ".$pro_tips[0];
+        $pro_tip = wordwrap($pro_tip, 78, "\n", true);
+
+        $lines = explode("\n", $pro_tip);
+
+        foreach ($lines as $key => $line) {
+          $lines[$key] = "# ".$line;
+        }
+
+        $pro_tip = implode("\n", $lines);
+        $commit_message[] = $pro_tip;
+      }
+    }
+
     $commit_message = implode("\n\n", $commit_message);
 
     return $commit_message;

src/applications/differential/field/specification/DifferentialDependsOnFieldSpecification.php

@@ -47,4 +47,11 @@ public function getKeyForConduit() {
     return 'phabricator:depends-on';
   }
 
+  public function getCommitMessageTips() {
+    return array(
+      'Use "Depends on D123" in your summary to mark '.
+      'a dependency between revisions.'
+      );
+  }
+
 }

src/applications/differential/field/specification/DifferentialFieldSpecification.php

@@ -667,6 +667,10 @@ public function didParseCommit(
     return;
   }
 
+  public function getCommitMessageTips() {
+      return array();
+  }
+
 
 /* -(  Loading Additional Data  )-------------------------------------------- */
 

src/applications/differential/field/specification/DifferentialManiphestTasksFieldSpecification.php

@@ -173,4 +173,11 @@ public function renderValueForMail($phase) {
     return implode("\n", $body);
   }
 
+  public function getCommitMessageTips() {
+    return array(
+      'Use "Fixes T123" in your summary to mark that the current '.
+      'revision completes a given task.'
+      );
+  }
+
 }