I think these addFlag values were reversed; agreement/notices should stop the
authorChuck Hagenbuch <chuck@horde.org>
Sun, 29 Nov 2009 15:43:43 +0000 (10:43 -0500)
committerChuck Hagenbuch <chuck@horde.org>
Sun, 29 Nov 2009 15:43:43 +0000 (10:43 -0500)
display of other tasks, not the other way around.

framework/LoginTasks/lib/Horde/LoginTasks/Tasklist.php

index 3a7c34b..8c9d5ad 100644 (file)
@@ -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) {