From 44942f3378a911f8162166a7ecdb82a076a51901 Mon Sep 17 00:00:00 2001
From: Bob Trahan <btrahan@phacility.com>
Date: Fri, 28 Mar 2014 13:41:19 -0700
Subject: [PATCH] Fix help menu links for folks with diviner uninstalled

Summary: move code to uninstallable help app rather than diviner. Fixes T4690.

Test Plan: uninstalled diviner, noted no links, then moved the code and suddenly helpful help links showed up once more.

Reviewers: epriestley

Reviewed By: epriestley

Subscribers: epriestley, Korvin

Maniphest Tasks: T4690

Differential Revision: https://secure.phabricator.com/D8638
---
 .../PhabricatorApplicationDiviner.php         | 25 -------------------
 .../PhabricatorApplicationHelp.php            | 24 ++++++++++++++++++
 2 files changed, 24 insertions(+), 25 deletions(-)

diff --git a/src/applications/diviner/application/PhabricatorApplicationDiviner.php b/src/applications/diviner/application/PhabricatorApplicationDiviner.php
index 3362906487..524786fe9d 100644
--- a/src/applications/diviner/application/PhabricatorApplicationDiviner.php
+++ b/src/applications/diviner/application/PhabricatorApplicationDiviner.php
@@ -45,29 +45,4 @@ final class PhabricatorApplicationDiviner extends PhabricatorApplication {
     );
   }
 
-  public function buildMainMenuItems(
-    PhabricatorUser $user,
-    PhabricatorController $controller = null) {
-
-    $items = array();
-
-    $application = null;
-    if ($controller) {
-      $application = $controller->getCurrentApplication();
-    }
-
-    if ($application && $application->getHelpURI()) {
-      $item = id(new PHUIListItemView())
-        ->setName(pht('%s Help', $application->getName()))
-        ->addClass('core-menu-item')
-        ->setIcon('info-sm')
-        ->setOrder(200)
-        ->setHref($application->getHelpURI());
-      $items[] = $item;
-    }
-
-    return $items;
-  }
-
-
 }
diff --git a/src/applications/help/application/PhabricatorApplicationHelp.php b/src/applications/help/application/PhabricatorApplicationHelp.php
index fe8a5f176c..7ba399cdbf 100644
--- a/src/applications/help/application/PhabricatorApplicationHelp.php
+++ b/src/applications/help/application/PhabricatorApplicationHelp.php
@@ -19,4 +19,28 @@ final class PhabricatorApplicationHelp extends PhabricatorApplication {
     );
   }
 
+  public function buildMainMenuItems(
+    PhabricatorUser $user,
+    PhabricatorController $controller = null) {
+
+    $items = array();
+
+    $application = null;
+    if ($controller) {
+      $application = $controller->getCurrentApplication();
+    }
+
+    if ($application && $application->getHelpURI()) {
+      $item = id(new PHUIListItemView())
+        ->setName(pht('%s Help', $application->getName()))
+        ->addClass('core-menu-item')
+        ->setIcon('info-sm')
+        ->setOrder(200)
+        ->setHref($application->getHelpURI());
+      $items[] = $item;
+    }
+
+    return $items;
+  }
+
 }