Improve more crumbs and cancel buttons for auth

Summary:
Ref T1536.

  - When linking accounts after initially failing, make the crumb say "Link Account" instead of "Login".
  - When on the LDAP failure form, show a "Cancel" button returning to start (if logging in) or settings (if linking accounts).
  - Allow providers to distinguish between "start", "login" and "link" rendering.

Test Plan: Linked and logged in with LDAP and other registration mechainsms.

Reviewers: chad, btrahan

Reviewed By: chad

CC: aran

Maniphest Tasks: T1536

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

src/applications/auth/controller/PhabricatorAuthLoginController.php

@@ -178,10 +178,19 @@ final class PhabricatorAuthLoginController
     $content) {
 
     $crumbs = $this->buildApplicationCrumbs();
+
+    if ($this->getRequest()->getUser()->isLoggedIn()) {
+      $crumbs->addCrumb(
+        id(new PhabricatorCrumbView())
+          ->setName(pht('Link Account'))
+          ->setHref($provider->getSettingsURI()));
+    } else {
       $crumbs->addCrumb(
         id(new PhabricatorCrumbView())
           ->setName(pht('Login'))
           ->setHref($this->getApplicationURI('start/')));
+    }
+
     $crumbs->addCrumb(
       id(new PhabricatorCrumbView())
         ->setName($provider->getProviderName()));

src/applications/auth/provider/PhabricatorAuthProvider.php

@@ -108,7 +108,7 @@ abstract class PhabricatorAuthProvider {
 
   public function buildLoginForm(
     PhabricatorAuthStartController $controller) {
-    return $this->renderLoginForm($controller->getRequest(), $is_link = false);
+    return $this->renderLoginForm($controller->getRequest(), $mode = 'start');
   }
 
   abstract public function processLoginRequest(
@@ -116,12 +116,12 @@ abstract class PhabricatorAuthProvider {
 
   public function buildLinkForm(
     PhabricatorAuthLinkController $controller) {
-    return $this->renderLoginForm($controller->getRequest(), $is_link = true);
+    return $this->renderLoginForm($controller->getRequest(), $mode = 'link');
   }
 
   protected function renderLoginForm(
     AphrontRequest $request,
-    $is_link) {
+    $mode) {
     throw new Exception("Not implemented!");
   }
 
@@ -213,16 +213,22 @@ abstract class PhabricatorAuthProvider {
     return $account;
   }
 
-  protected function getLoginURI() {
+  public function getLoginURI() {
     $app = PhabricatorApplication::getByClass('PhabricatorApplicationAuth');
     $uri = $app->getApplicationURI('/login/'.$this->getProviderKey().'/');
     return PhabricatorEnv::getURI($uri);
   }
 
-  protected function getCancelLinkURI() {
+  public function getSettingsURI() {
     return '/settings/panel/external/';
   }
 
+  public function getStartURI() {
+    $app = PhabricatorApplication::getByClass('PhabricatorApplicationAuth');
+    $uri = $app->getApplicationURI('/start/');
+    return $uri;
+  }
+
   public function isDefaultRegistrationProvider() {
     return false;
   }

src/applications/auth/provider/PhabricatorAuthProviderLDAP.php

@@ -63,23 +63,29 @@ final class PhabricatorAuthProviderLDAP
     return true;
   }
 
-  protected function renderLoginForm(AphrontRequest $request, $is_link) {
+  protected function renderLoginForm(AphrontRequest $request, $mode) {
     $viewer = $request->getUser();
 
     $dialog = id(new AphrontDialogView())
       ->setSubmitURI($this->getLoginURI())
       ->setUser($viewer);
 
-    if ($is_link) {
+    if ($mode == 'link') {
       $dialog->setTitle(pht('Link LDAP Account'));
       $dialog->addSubmitButton(pht('Link Accounts'));
-    } else if ($this->shouldAllowRegistration()) {
+      $dialog->addCancelButton($this->getSettingsURI());
+    } else {
+      if ($this->shouldAllowRegistration()) {
         $dialog->setTitle(pht('Login or Register with LDAP'));
         $dialog->addSubmitButton(pht('Login or Register'));
       } else {
         $dialog->setTitle(pht('Login with LDAP'));
         $dialog->addSubmitButton(pht('Login'));
       }
+      if ($mode == 'login') {
+        $dialog->addCancelButton($this->getStartURI());
+      }
+    }
 
     $v_user = $request->getStr('ldap_username');
 
@@ -137,7 +143,7 @@ final class PhabricatorAuthProviderLDAP
     if (!strlen($username) || !$has_password) {
       $response = $controller->buildProviderPageResponse(
         $this,
-        $this->renderLoginForm($request));
+        $this->renderLoginForm($request, 'login'));
       return array($account, $response);
     }
 

src/applications/auth/provider/PhabricatorAuthProviderOAuth.php

@@ -44,10 +44,10 @@ abstract class PhabricatorAuthProviderOAuth extends PhabricatorAuthProvider {
     return true;
   }
 
-  protected function renderLoginForm(AphrontRequest $request, $is_link) {
+  protected function renderLoginForm(AphrontRequest $request, $mode) {
     $viewer = $request->getUser();
 
-    if ($is_link) {
+    if ($mode == 'link') {
       $button_text = pht('Link External Account');
     } else if ($this->shouldAllowRegistration()) {
       $button_text = pht('Login or Register');