From: cash <cash.costello@gmail.com>
Date: Fri, 18 Nov 2011 22:44:45 +0000 (-0500)
Subject: Fixes #4115 returning bool from remove_user_from_access_collection()
X-Git-Url: https://gitweb.fluxo.info/?a=commitdiff_plain;h=15f02eea51f0f0eb92e427b627324d58af166e5e;p=lorea%2Felgg.git

Fixes #4115 returning bool from remove_user_from_access_collection()
---

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);