From 68c6bb6b3448bda9506c86a80df7f0444e7b209c Mon Sep 17 00:00:00 2001 From: Chuck Hagenbuch Date: Sun, 29 Nov 2009 10:43:43 -0500 Subject: [PATCH] I think these addFlag values were reversed; agreement/notices should stop the display of other tasks, not the other way around. --- framework/LoginTasks/lib/Horde/LoginTasks/Tasklist.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/framework/LoginTasks/lib/Horde/LoginTasks/Tasklist.php b/framework/LoginTasks/lib/Horde/LoginTasks/Tasklist.php index 3a7c34be3..8c9d5ad1d 100644 --- 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) { -- 2.11.0