From: Michael M Slusarz <slusarz@curecanti.org>
Date: Wed, 22 Jul 2009 06:43:59 +0000 (-0600)
Subject: Don't run tasks if not authenticated
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=27c39c9fd6393c499b42f4ddbb6881a4f7c264f4;p=horde.git

Don't run tasks if not authenticated
---

diff --git a/framework/LoginTasks/lib/Horde/LoginTasks.php b/framework/LoginTasks/lib/Horde/LoginTasks.php
index 43dc18652..10b69e62c 100644
--- a/framework/LoginTasks/lib/Horde/LoginTasks.php
+++ b/framework/LoginTasks/lib/Horde/LoginTasks.php
@@ -91,12 +91,14 @@ class Horde_LoginTasks
     {
         $this->_app = $app;
 
-        /* Retrieves a cached tasklist or make sure one is created. */
-        if (isset($_SESSION['horde_logintasks'][$app])) {
-            $this->_tasklist = unserialize($_SESSION['horde_logintasks'][$app]);
-        } else {
-            $this->_createTaskList($url);
-            $this->_init = true;
+        if (Horde_Auth::getAuth()) {
+            /* Retrieves a cached tasklist or make sure one is created. */
+            if (isset($_SESSION['horde_logintasks'][$app])) {
+                $this->_tasklist = unserialize($_SESSION['horde_logintasks'][$app]);
+            } else {
+                $this->_createTaskList($url);
+                $this->_init = true;
+            }
         }
     }
 
@@ -105,7 +107,9 @@ class Horde_LoginTasks
      */
     public function __destruct()
     {
-        $_SESSION['horde_logintasks'][$this->_app] = serialize($this->_tasklist);
+        if (isset($this->_tasklist)) {
+            $_SESSION['horde_logintasks'][$this->_app] = serialize($this->_tasklist);
+        }
     }
 
     /**
@@ -217,7 +221,8 @@ class Horde_LoginTasks
      */
     public function runTasks($confirmed = false)
     {
-        if ($this->_tasklist === true) {
+        if (!isset($this->_tasklist) ||
+            ($this->_tasklist === true)) {
             return;
         }