diff options
| author | cash <cash.costello@gmail.com> | 2011-11-18 17:44:45 -0500 |
|---|---|---|
| committer | cash <cash.costello@gmail.com> | 2011-11-28 21:43:59 -0500 |
| commit | a219beb80c90c377ed5b611a9a502f2dbc04d61a (patch) | |
| tree | 485bebc41cf5cc9bca58568c4042285d94c6cbb5 | |
| parent | 337abf7ce0a68f4a2fe57036092c3188ba4f42f7 (diff) | |
| download | elgg-a219beb80c90c377ed5b611a9a502f2dbc04d61a.tar.gz elgg-a219beb80c90c377ed5b611a9a502f2dbc04d61a.tar.bz2 | |
Fixes #4115 returning bool from remove_user_from_access_collection()
| -rw-r--r-- | engine/lib/access.php | 2 | ||||
| -rw-r--r-- | engine/tests/api/access_collections.php | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/engine/lib/access.php b/engine/lib/access.php index ba3820f46..08b9283cd 100644 --- a/engine/lib/access.php +++ b/engine/lib/access.php @@ -717,7 +717,7 @@ function remove_user_from_access_collection($user_guid, $collection_id) { WHERE access_collection_id = {$collection_id} AND user_guid = {$user_guid}"; - return delete_data($q); + return (bool)delete_data($q); } /** diff --git a/engine/tests/api/access_collections.php b/engine/tests/api/access_collections.php index 7875b1409..bea995a6e 100644 --- a/engine/tests/api/access_collections.php +++ b/engine/tests/api/access_collections.php @@ -88,7 +88,7 @@ class ElggCoreAccessCollectionsTest extends ElggCoreUnitTest { if ($result) { $result = remove_user_from_access_collection($this->user->guid, $acl_id); - $this->assertTrue($result); + $this->assertIdentical(true, $result); } delete_access_collection($acl_id); |