More fixes for error handling.

horde · Oct 30, 2013 · 2ad5afe · 2ad5afe
1 parent af8caef
commit 2ad5afe
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/framework/Core/lib/Horde/ErrorHandler.php b/framework/Core/lib/Horde/ErrorHandler.php
@@ -28,7 +28,7 @@ static public function fatal($error)
             switch (get_class($error)) {
             case 'Horde_Exception_AuthenticationFailure':
                 if ($registry->isAuthenticated(array('app' => $error->application, 'notransparent' => true)) &&
-                    $registry->clearAuthApp($error->application)) {
+                    !$registry->clearAuthApp($error->application)) {
                     break;
                 }
 
@@ -50,6 +50,7 @@ static public function fatal($error)
                 switch ($error->getCode()) {
                 case Horde_Auth::REASON_MESSAGE:
                     $params['msg'] = $error->getMessage();
+                    $params['reason'] = $error->getCode();
                     break;
                 }
 
@@ -59,7 +60,7 @@ static public function fatal($error)
                  * issues on the login page since we would otherwise need
                  * to do session token checking (which might not be
                  * available, so logout won't happen, etc...) */
-                if (isset($params['app'])) {
+                if (array_key_exists($params, 'app')) {
                     $registry->clearAuth();
                 }