Move group id session caching into group_ids()

gallery · Sep 29, 2009 · 18221f1 · 18221f1
1 parent 416ffe8
commit 18221f1
Showing 1 changed file with 9 additions and 8 deletions.
diff --git a/modules/user/libraries/drivers/UserGroupStorage/Gallery3.php b/modules/user/libraries/drivers/UserGroupStorage/Gallery3.php
@@ -20,7 +20,15 @@
 
 class UserGroupStorage_Gallery3_Driver extends UserGroupStorage_Driver {
   public function group_ids() {
-    return Session::instance()->get("group_ids", array(1));
+    $session = Session::instance();
+    if (!($ids = $session->get("group_ids"))) {
+      $ids = array();
+      foreach ($user->groups as $group) {
+        $ids[] = $group->id;
+      }
+      $session->set("group_ids", $ids);
+    }
+    return $ids;
   }
 
   public function active_user() {
@@ -40,13 +48,6 @@ public function active_user() {
         $session->set("user", $user);
       }
 
-      if (!$session->get("group_ids")) {
-        $ids = array();
-        foreach ($user->groups as $group) {
-          $ids[] = $group->id;
-        }
-        $session->set("group_ids", $ids);
-      }
     }
     return $user;
   }