From: Jan Schneider <jan@horde.org>
Date: Sun, 23 Aug 2009 21:11:47 +0000 (+0200)
Subject: This check doesn't make any sense to me, and it break guest application access.
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=5df37f9934afeee9f4741d41f92c06cfc4b39ca9;p=horde.git

This check doesn't make any sense to me, and it break guest application access.
---

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);