Fix Search Application Config

Summary: Fix T12924. Looks like this melted in D17384, and nobody noticed yet.

Test Plan: Visit page, see fancy table.

Reviewers: epriestley, 20after4, #blessed_reviewers

Reviewed By: epriestley, 20after4, #blessed_reviewers

Subscribers: Korvin

Maniphest Tasks: T12924

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

src/applications/search/applicationpanel/PhabricatorSearchApplicationStorageEnginePanel.php

@@ -16,54 +16,61 @@ final class PhabricatorSearchApplicationStorageEnginePanel
     $viewer = $this->getViewer();
     $application = $this->getApplication();
 
-    $active_engine = PhabricatorFulltextStorageEngine::loadEngine();
+    $services = PhabricatorSearchService::getAllServices();
-    $engines = PhabricatorFulltextStorageEngine::loadAllEngines();
 
     $rows = array();
     $rowc = array();
 
-    foreach ($engines as $key => $engine) {
+    foreach ($services as $key => $service) {
       try {
-        $index_exists = $engine->indexExists() ? pht('Yes') : pht('No');
+        $name = $service->getDisplayName();
       } catch (Exception $ex) {
-        $index_exists = pht('N/A');
+        $name = phutil_tag('em', array(), pht('Error'));
       }
 
       try {
-        $index_is_sane = $engine->indexIsSane() ? pht('Yes') : pht('No');
+        $can_read = $service->isReadable() ? pht('Yes') : pht('No');
       } catch (Exception $ex) {
-        $index_is_sane = pht('N/A');
+        $can_read = pht('N/A');
       }
 
-      if ($engine == $active_engine) {
+      try {
-        $rowc[] = 'highlighted';
+        $can_write = $service->isWritable() ? pht('Yes') : pht('No');
-      } else {
+      } catch (Exception $ex) {
-        $rowc[] = null;
+        $can_write = pht('N/A');
       }
 
       $rows[] = array(
-        $key,
+        $name,
-        get_class($engine),
+        $can_read,
-        $index_exists,
+        $can_write,
-        $index_is_sane,
       );
     }
 
+    $instructions = pht(
+      'To configure the search engines, edit [[ %s | `%s` ]] configuration. '.
+      'See **[[ %s | %s ]]** for documentation.',
+      '/config/edit/cluster.search/',
+      'cluster.search',
+      PhabricatorEnv::getDoclink('Cluster: Search'),
+      pht('Cluster: Search'));
+
+
     $table = id(new AphrontTableView($rows))
       ->setNoDataString(pht('No search engines available.'))
+      ->setNotice(new PHUIRemarkupView($viewer, $instructions))
       ->setHeaders(
         array(
-          pht('Key'),
+          pht('Engine Name'),
-          pht('Class'),
+          pht('Writable'),
-          pht('Index Exists'),
+          pht('Readable'),
-          pht('Index Is Sane'),
         ))
       ->setRowClasses($rowc)
       ->setColumnClasses(
         array(
-          '',
           'wide',
           '',
+          '',
         ));
 
     $box = id(new PHUIObjectBoxView())