aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorcash <cash.costello@gmail.com>2011-11-18 17:44:45 -0500
committercash <cash.costello@gmail.com>2011-11-28 21:43:59 -0500
commita219beb80c90c377ed5b611a9a502f2dbc04d61a (patch)
tree485bebc41cf5cc9bca58568c4042285d94c6cbb5
parent337abf7ce0a68f4a2fe57036092c3188ba4f42f7 (diff)
downloadelgg-a219beb80c90c377ed5b611a9a502f2dbc04d61a.tar.gz
elgg-a219beb80c90c377ed5b611a9a502f2dbc04d61a.tar.bz2
Fixes #4115 returning bool from remove_user_from_access_collection()
-rw-r--r--engine/lib/access.php2
-rw-r--r--engine/tests/api/access_collections.php2
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);