I think these addFlag values were reversed; agreement/notices should stop the

display of other tasks, not the other way around.

diff --git a/framework/LoginTasks/lib/Horde/LoginTasks/Tasklist.php b/framework/LoginTasks/lib/Horde/LoginTasks/Tasklist.php
index 3a7c34b..8c9d5ad 100644 (file)
--- a/framework/LoginTasks/lib/Horde/LoginTasks/Tasklist.php
+++ b/framework/LoginTasks/lib/Horde/LoginTasks/Tasklist.php
@@ -69,11 +69,11 @@ class Horde_LoginTasks_Tasklist
         if (($task->display == Horde_LoginTasks::DISPLAY_AGREE) ||
             ($task->display == Horde_LoginTasks::DISPLAY_NOTICE)) {
             $tmp['display'] = true;
-            $this->_addFlag = false;
+            $this->_addFlag = true;
         } elseif (($task->display != Horde_LoginTasks::DISPLAY_NONE) &&
                   !$this->_addFlag) {
             $tmp['display'] = true;
-            $this->_addFlag = true;
+            $this->_addFlag = false;
         }
 
         switch ($task->priority) {