From 5df37f9934afeee9f4741d41f92c06cfc4b39ca9 Mon Sep 17 00:00:00 2001 From: Jan Schneider Date: Sun, 23 Aug 2009 23:11:47 +0200 Subject: [PATCH] This check doesn't make any sense to me, and it break guest application access. --- framework/Core/lib/Horde/Registry.php | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/framework/Core/lib/Horde/Registry.php b/framework/Core/lib/Horde/Registry.php index b8ee0bd83..7bcdf63fe 100644 --- a/framework/Core/lib/Horde/Registry.php +++ b/framework/Core/lib/Horde/Registry.php @@ -977,14 +977,8 @@ class Horde_Registry /* Always do isAuthenticated() check first. You can be an admin, but * application auth != Horde admin auth. */ if (!Horde_Auth::isAuthenticated(array('app' => $app))) { - /* There can *never* be non-SHOW guest access to an application - * that requires authentication. */ - if ($perms != PERMS_SHOW) { - return false; - } - - /* Otherwise, allow SHOW access for admins, for apps that do not - * have any explicit permissions, or for apps that allow SHOW. */ + /* Allow SHOW access for admins, for apps that do not have any + * explicit permissions, or for apps that allow SHOW. */ return Horde_Auth::isAdmin() || !$GLOBALS['perms']->exists($app) || $GLOBALS['perms']->hasPermission($app, Horde_Auth::getAuth(), $perms); -- 2.11.0