Move "unlisted" apps to Query, use Query for app preferences

Summary:
  - Demagic "unlisted" apps.
  - Respect policies in application visibility settings.

Test Plan: Viewed applications, settings.

Reviewers: btrahan

Reviewed By: btrahan

CC: aran

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

src/applications/base/PhabricatorApplication.php

@@ -82,6 +82,13 @@ abstract class PhabricatorApplication
     return false;
   }
 
+  /**
+   * Return true if this application should not appear in application lists in
+   * the UI. Primarily intended for unit test applications or other
+   * pseudo-applications.
+   *
+   * @return bool True to remove application from UI lists.
+   */
   public function isUnlisted() {
     return false;
   }
@@ -89,6 +96,8 @@ abstract class PhabricatorApplication
   /**
    * Returns true if an application is first-party (developed by Phacility)
    * and false otherwise.
+   *
+   * @return bool True if this application is developed by Phacility.
    */
   final public function isFirstParty() {
     $where = id(new ReflectionClass($this))->getFileName();

src/applications/meta/controller/PhabricatorApplicationsListController.php

@@ -30,10 +30,6 @@ final class PhabricatorApplicationsListController
     $applications = msort($applications, 'getName');
 
     foreach ($applications as $application) {
-      if ($application->isUnlisted()) {
-        continue;
-      }
-
       $item = id(new PHUIObjectItemView())
         ->setHeader($application->getName())
         ->setHref('/applications/view/'.get_class($application).'/')

src/applications/meta/query/PhabricatorAppSearchEngine.php

@@ -21,7 +21,8 @@ final class PhabricatorAppSearchEngine
 
   public function buildQueryFromSavedQuery(PhabricatorSavedQuery $saved) {
     $query = id(new PhabricatorApplicationQuery())
-      ->setOrder(PhabricatorApplicationQuery::ORDER_NAME);
+      ->setOrder(PhabricatorApplicationQuery::ORDER_NAME)
+      ->withUnlisted(false);
 
     $name = $saved->getParameter('name');
     if (strlen($name)) {

src/applications/meta/query/PhabricatorApplicationQuery.php

@@ -7,6 +7,7 @@ final class PhabricatorApplicationQuery
   private $beta;
   private $firstParty;
   private $nameContains;
+  private $unlisted;
   private $classes;
   private $phids;
 
@@ -35,6 +36,11 @@ final class PhabricatorApplicationQuery
     return $this;
   }
 
+  public function withUnlisted($unlisted) {
+    $this->unlisted = $unlisted;
+    return $this;
+  }
+
   public function withClasses(array $classes) {
     $this->classes = $classes;
     return $this;
@@ -103,6 +109,14 @@ final class PhabricatorApplicationQuery
       }
     }
 
+    if ($this->unlisted !== null) {
+      foreach ($apps as $key => $app) {
+        if ($app->isUnlisted() != $this->unlisted) {
+          unset($apps[$key]);
+        }
+      }
+    }
+
     switch ($this->order) {
       case self::ORDER_NAME:
         $apps = msort($apps, 'getName');

src/applications/settings/panel/PhabricatorSettingsPanelHomePreferences.php

@@ -21,7 +21,11 @@ final class PhabricatorSettingsPanelHomePreferences
 
     require_celerity_resource('phabricator-settings-css');
 
-    $apps = PhabricatorApplication::getAllInstalledApplications();
+    $apps = id(new PhabricatorApplicationQuery())
+      ->setViewer($user)
+      ->withUnlisted(false)
+      ->execute();
+
     $pref_tiles = PhabricatorUserPreferences::PREFERENCE_APP_TILES;
     $tiles = $preferences->getPreference($pref_tiles, array());
 
@@ -181,9 +185,7 @@ final class PhabricatorSettingsPanelHomePreferences
         ->appendChild($table)
         ->setNoBackground();
 
-
       $output[] = $panel;
-
     }
 
     $form
@@ -205,9 +207,7 @@ final class PhabricatorSettingsPanelHomePreferences
       ->setFormError($error_view)
       ->setForm($form);
 
-    return array(
+    return $form_box;
-      $form_box,
-    );
   }
 }