diff options
| author | Bharat Mediratta <bharat@menalto.com> | 2008-12-04 06:40:35 +0000 |
|---|---|---|
| committer | Bharat Mediratta <bharat@menalto.com> | 2008-12-04 06:40:35 +0000 |
| commit | bae905a4cb6c86acf2a887b57779492d964ecba1 (patch) | |
| tree | 3f616001ac60a593f62feb4e7acfdb1176b68c31 /core/helpers/access.php | |
| parent | 1ae3ed7e3b796bae6e16e687c29f3fcf0df5fbfe (diff) | |
Improvements to access helper.
o Rename access::remove_group() to access::delete_group() for
consistency.
o Wrote more unit tests
o Tests found a bug in access::remove_item() .. yay!
Diffstat (limited to 'core/helpers/access.php')
| -rw-r--r-- | core/helpers/access.php | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/core/helpers/access.php b/core/helpers/access.php index 96f4d375..95b9819c 100644 --- a/core/helpers/access.php +++ b/core/helpers/access.php @@ -197,7 +197,7 @@ class access_Core { * @param Group_Model $group * @return void */ - public static function remove_group($group) { + public static function delete_group($group) { foreach (ORM::factory("permission")->find_all() as $perm) { self::_drop_columns($perm->name, $group->id); } @@ -236,9 +236,9 @@ class access_Core { * @param Item_Model $item * @return void */ - public static function remove_item($item) { - ORM::factory("access_intent")->where("item_id", $item->id)->delete(); - ORM::factory("access_cache")->where("item_id", $item->id)->delete(); + public static function delete_item($item) { + ORM::factory("access_intent")->where("item_id", $item->id)->find()->delete(); + ORM::factory("access_cache")->where("item_id", $item->id)->find()->delete(); } /** |