From 482f6ea56ba8ac47194ac12b626e4f0c93461313 Mon Sep 17 00:00:00 2001
From: epriestley <git@epriestley.com>
Date: Wed, 16 Jan 2013 15:06:19 -0800
Subject: [PATCH] Remove "Flags" from homepage

Summary: Flags have a large red count on the homepage now, which I think is a sufficient reminder of flagged stuff. This element was nice at first to raise awareness of the app, but it's fairly well integrated now and enjoys moderate use. This is also a sort of feeler for how much people use it / the homepage in general.

Test Plan: Looked at homepage, no flags.

Reviewers: chad, btrahan

Reviewed By: btrahan

CC: aran

Differential Revision: https://secure.phabricator.com/D4479
---
 .../PhabricatorDirectoryMainController.php    | 41 -------------------
 1 file changed, 41 deletions(-)

diff --git a/src/applications/directory/controller/PhabricatorDirectoryMainController.php b/src/applications/directory/controller/PhabricatorDirectoryMainController.php
index 82e8190795..ce1e7c100f 100644
--- a/src/applications/directory/controller/PhabricatorDirectoryMainController.php
+++ b/src/applications/directory/controller/PhabricatorDirectoryMainController.php
@@ -39,8 +39,6 @@ final class PhabricatorDirectoryMainController
       $tasks_panel = null;
     }
 
-    $flagged_panel = $this->buildFlaggedPanel();
-
     $jump_panel = $this->buildJumpPanel();
     $revision_panel = $this->buildRevisionPanel();
     $audit_panel = $this->buildAuditPanel();
@@ -52,7 +50,6 @@ final class PhabricatorDirectoryMainController
       $triage_panel,
       $revision_panel,
       $tasks_panel,
-      $flagged_panel,
       $audit_panel,
       $commit_panel,
     );
@@ -121,44 +118,6 @@ final class PhabricatorDirectoryMainController
     return $panel;
   }
 
-  private function buildFlaggedPanel() {
-    $user = $this->getRequest()->getUser();
-
-    $flag_query = id(new PhabricatorFlagQuery())
-      ->setViewer($user)
-      ->withOwnerPHIDs(array($user->getPHID()))
-      ->needHandles(true)
-      ->setLimit(10);
-
-    $flags = $flag_query->execute();
-
-    if (!$flags) {
-      return $this->renderMiniPanel(
-        'No Flags',
-        "You haven't flagged anything.");
-    }
-
-    $panel = new AphrontPanelView();
-    $panel->setHeader('Flagged Objects');
-    $panel->setCaption("Objects you've flagged.");
-
-    $flag_view = new PhabricatorFlagListView();
-    $flag_view->setFlags($flags);
-    $flag_view->setUser($user);
-    $panel->appendChild($flag_view);
-
-    $panel->addButton(
-      phutil_render_tag(
-        'a',
-        array(
-          'href'  => '/flag/',
-          'class' => 'grey button',
-        ),
-        "View All Flags \xC2\xBB"));
-
-    return $panel;
-  }
-
   private function buildNeedsTriagePanel(array $projects) {
     assert_instances_of($projects, 'PhabricatorProject');