From d7541c70ddc9637675b58a29a8697880d1532c3f Mon Sep 17 00:00:00 2001
From: Joshua Spence <josh@joshuaspence.com>
Date: Wed, 18 Jun 2014 03:42:53 +1000
Subject: [PATCH] Convert `arc list` to use `PhutilConsoleTable`.

Summary: Similar to D9601 and D9602. Also added pretty colors.

Test Plan: {F167702}

Reviewers: #blessed_reviewers, epriestley

Reviewed By: #blessed_reviewers, epriestley

Subscribers: epriestley, Korvin

Differential Revision: https://secure.phabricator.com/D9603
---
 src/workflow/ArcanistListWorkflow.php | 46 +++++++++++++++++----------
 1 file changed, 30 insertions(+), 16 deletions(-)

diff --git a/src/workflow/ArcanistListWorkflow.php b/src/workflow/ArcanistListWorkflow.php
index 5a74b959..e443e700 100644
--- a/src/workflow/ArcanistListWorkflow.php
+++ b/src/workflow/ArcanistListWorkflow.php
@@ -39,6 +39,16 @@ EOTEXT
   }
 
   public function run() {
+    static $color_map = array(
+      'Closed'          => 'cyan',
+      'Needs Review'    => 'magenta',
+      'Needs Revision'  => 'red',
+      'Changes Planned' => 'red',
+      'Accepted'        => 'green',
+      'No Revision'     => 'blue',
+      'Abandoned'       => 'default',
+    );
+
     $revisions = $this->getConduit()->callMethodSynchronous(
       'differential.query',
       array(
@@ -54,40 +64,44 @@ EOTEXT
     $repository_api = $this->getRepositoryAPI();
 
     $info = array();
-
-    $status_len = 0;
     foreach ($revisions as $key => $revision) {
       $revision_path = Filesystem::resolvePath($revision['sourcePath']);
       $current_path  = Filesystem::resolvePath($repository_api->getPath());
       if ($revision_path == $current_path) {
-        $info[$key]['here'] = 1;
+        $info[$key]['exists'] = 1;
       } else {
-        $info[$key]['here'] = 0;
+        $info[$key]['exists'] = 0;
       }
       $info[$key]['sort'] = sprintf(
         '%d%04d%08d',
-        $info[$key]['here'],
+        $info[$key]['exists'],
         $revision['status'],
         $revision['id']);
       $info[$key]['statusName'] = $revision['statusName'];
-      $status_len = max(
-        $status_len,
-        strlen($info[$key]['statusName']));
+      $info[$key]['color'] = idx(
+        $color_map, $revision['statusName'], 'default');
     }
 
+    $table = id(new PhutilConsoleTable())
+      ->setShowHeader(false)
+      ->addColumn('exists', array('title' => ''))
+      ->addColumn('status', array('title' => 'Status'))
+      ->addColumn('title',  array('title' => 'Title'));
+
     $info = isort($info, 'sort');
     foreach ($info as $key => $spec) {
       $revision = $revisions[$key];
-      printf(
-        "%s %-".($status_len + 4)."s D%d: %s\n",
-        $spec['here']
-          ? phutil_console_format('**%s**', '*')
-          : ' ',
-        $spec['statusName'],
-        $revision['id'],
-        $revision['title']);
+
+      $table->addRow(array(
+        'exists' => $spec['exists'] ? phutil_console_format('**%s**', '*') : '',
+        'status' => phutil_console_format(
+          "<fg:{$spec['color']}>%s</fg>", $spec['statusName']),
+        'title'  => phutil_console_format(
+          '**D%d:** %s', $revision['id'], $revision['title']),
+      ));
     }
 
+    $table->draw();
     return 0;
   }
 }