aboutsummaryrefslogtreecommitdiff
path: root/engine
diff options
context:
space:
mode:
authorcash <cash.costello@gmail.com>2011-11-18 17:50:36 -0500
committercash <cash.costello@gmail.com>2011-11-28 21:43:59 -0500
commit90baebb659b3e47dac24da969e232ff0cef94a1b (patch)
tree7ebc679dd5de987273c11aa7ac2c2369a1fd5ed1 /engine
parenta219beb80c90c377ed5b611a9a502f2dbc04d61a (diff)
downloadelgg-90baebb659b3e47dac24da969e232ff0cef94a1b.tar.gz
elgg-90baebb659b3e47dac24da969e232ff0cef94a1b.tar.bz2
Fixes #4118 returning bool from leave group functions
Diffstat (limited to 'engine')
-rw-r--r--engine/classes/ElggGroup.php6
-rw-r--r--engine/lib/relationships.php2
2 files changed, 4 insertions, 4 deletions
diff --git a/engine/classes/ElggGroup.php b/engine/classes/ElggGroup.php
index 49ba27204..5c16e978f 100644
--- a/engine/classes/ElggGroup.php
+++ b/engine/classes/ElggGroup.php
@@ -309,7 +309,7 @@ class ElggGroup extends ElggEntity
*
* @param ElggUser $user User
*
- * @return void
+ * @return bool
*/
public function leave(ElggUser $user) {
return leave_group($this->getGUID(), $user->getGUID());
@@ -322,7 +322,7 @@ class ElggGroup extends ElggEntity
*
* @param int $guid GUID of an ElggGroup entity
*
- * @return true
+ * @return bool
*/
protected function load($guid) {
// Test to see if we have the generic stuff
@@ -340,7 +340,7 @@ class ElggGroup extends ElggEntity
$row = get_group_entity_as_row($guid);
if (($row) && (!$this->isFullyLoaded())) {
// If $row isn't a cached copy then increment the counter
- $this->attributes['tables_loaded'] ++;
+ $this->attributes['tables_loaded']++;
}
// Now put these into the attributes array as core values
diff --git a/engine/lib/relationships.php b/engine/lib/relationships.php
index 3578f0cb8..5b7080b56 100644
--- a/engine/lib/relationships.php
+++ b/engine/lib/relationships.php
@@ -158,7 +158,7 @@ function remove_entity_relationship($guid_one, $relationship, $guid_two) {
and relationship='$relationship'
and guid_two=$guid_two";
- return delete_data($query);
+ return (bool)delete_data($query);
} else {
return false;
}