From 02dab8d99ea0fa14899ccab713469e7561a0ed05 Mon Sep 17 00:00:00 2001
From: Michael M Slusarz <slusarz@curecanti.org>
Date: Tue, 12 Oct 2010 14:43:47 -0600
Subject: [PATCH] Convert horde/Group to using Horde_Session

---
 framework/Group/lib/Horde/Group/Contactlists.php |  8 ++++----
 framework/Group/lib/Horde/Group/Sql.php          | 15 +++++++++------
 2 files changed, 13 insertions(+), 10 deletions(-)

diff --git a/framework/Group/lib/Horde/Group/Contactlists.php b/framework/Group/lib/Horde/Group/Contactlists.php
index 9f27a0022..72fa758b7 100644
--- a/framework/Group/lib/Horde/Group/Contactlists.php
+++ b/framework/Group/lib/Horde/Group/Contactlists.php
@@ -562,8 +562,9 @@ class Horde_Group_Contactlists extends Horde_Group
      */
     public function userIsInGroup($user, $gid, $subgroups = true)
     {
-        if (isset($_SESSION['horde']['groups']['i'][$user][$subgroups][$gid])) {
-            return $_SESSION['horde']['groups']['i'][$user][$subgroups][$gid];
+        $id = implode('-', array($user, $subgroups, $gid);
+        if (isset($GLOBALS['session']['horde:groups_i/' . $id])) {
+            return $GLOBALS['session']['horde:groups_i/' . $id];
         }
 
         try {
@@ -573,8 +574,7 @@ class Horde_Group_Contactlists extends Horde_Group
             return false;
         }
 
-        $result = (bool)!empty($users[$user]);
-        $_SESSION['horde']['groups']['i'][$user][$subgroups][$gid] = $result;
+        $GLOBALS['session']['horde:groups_i/' . $id] = $result = (bool)!empty($users[$user]);
 
         return $result;
     }
diff --git a/framework/Group/lib/Horde/Group/Sql.php b/framework/Group/lib/Horde/Group/Sql.php
index 05db9624f..859daeadc 100644
--- a/framework/Group/lib/Horde/Group/Sql.php
+++ b/framework/Group/lib/Horde/Group/Sql.php
@@ -640,8 +640,9 @@ class Horde_Group_Sql extends Horde_Group
      */
     public function getGroupMemberships($user, $parentGroups = false)
     {
-        if (isset($_SESSION['horde']['groups']['m'][$user][$parentGroups])) {
-            return $_SESSION['horde']['groups']['m'][$user][$parentGroups];
+        $id = implode('|', array($user, $parentGroups));
+        if (isset($GLOBALS['session']['horde:groups_m/' . $id])) {
+            return $GLOBALS['session']['horde:groups_m/' . $id];
         }
 
         $sql = 'SELECT g.group_uid AS group_uid, g.group_name AS group_name FROM horde_groups g, horde_groups_members m '
@@ -663,7 +664,7 @@ class Horde_Group_Sql extends Horde_Group
             }
         }
 
-        $_SESSION['horde']['groups']['m'][$user][$parentGroups] = $groups;
+        $GLOBALS['session']['horde:groups_m/' . $id] = $groups;
 
         return $groups;
     }
@@ -680,8 +681,9 @@ class Horde_Group_Sql extends Horde_Group
      */
     public function userIsInGroup($user, $gid, $subgroups = true)
     {
-        if (isset($_SESSION['horde']['groups']['i'][$user][$subgroups][$gid])) {
-            return $_SESSION['horde']['groups']['i'][$user][$subgroups][$gid];
+        $id = implode('|', array($user, $subgroups, $gid));
+        if (isset($GLOBALS['session']['horde:groups_i/' . $id])) {
+            return $GLOBALS['session']['horde:groups_i/' . $id];
         }
 
         if ($subgroups) {
@@ -703,7 +705,8 @@ class Horde_Group_Sql extends Horde_Group
 
         }
 
-        $_SESSION['horde']['groups']['i'][$user][$subgroups][$gid] = (bool)$result;
+        $GLOBALS['session']['horde:groups_i/' . $id] = (bool)$result;
+
         return (bool)$result;
     }
 
-- 
2.11.0