Follow on to 1057436b7c483c60b3c128fab993a3b78fac7093 -- cache the

users and groups as an array so that multiple calls will not call
ORM_Iterator->current() repeatedly.

1 files changed, 3 insertions, 3 deletions

diff --git a/modules/user/models/group.php b/modules/user/models/group.php
index 66c9aafc..8f8e218d 100644
--- a/modules/user/models/group.php
+++ b/modules/user/models/group.php
@@ -29,12 +29,12 @@ class Group_Model_Core extends ORM implements Group_Definition {
     module::event("group_before_delete", $this);
     parent::delete($id);
     module::event("group_deleted", $old);
-    unset($this->users_cache);
+    $this->users_cache = null;
   }
 
   public function users() {
     if (!$this->users_cache) {
-      $this->users_cache = $this->users->find_all();
+      $this->users_cache = $this->users->find_all()->as_array();
     }
     return $this->users_cache;
   }
@@ -65,7 +65,7 @@ class Group_Model_Core extends ORM implements Group_Definition {
       module::event("group_updated", $original, $this);
     }
 
-    unset($this->users_cache);
+    $this->users_cache = null;
     return $this;
   }