From fc11bcffa66e2baaae52261a84de13802bb1f072 Mon Sep 17 00:00:00 2001 From: marcus Date: Tue, 30 Sep 2008 12:15:39 +0000 Subject: Closes #388: Additionally the site check in configuration_init() should be instanceof. Nice spot, thanks. git-svn-id: https://code.elgg.org/elgg/trunk@2151 36083f99-b078-4883-b0ff-0f9b5a30f544 --- actions/friends/remove.php | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) (limited to 'actions/friends') diff --git a/actions/friends/remove.php b/actions/friends/remove.php index 3d024b9d1..1f3cdcc5b 100644 --- a/actions/friends/remove.php +++ b/actions/friends/remove.php @@ -20,8 +20,14 @@ $errors = false; // Get the user - try{ - $_SESSION['user']->removeFriend($friend_guid) && get_class($friend) == "ElggUser"; + try{ + if ($friend instanceof ElggUser) + $_SESSION['user']->removeFriend($friend_guid); + else + { + register_error(sprintf(elgg_echo("friends:remove:failure"),$friend->name)); + $errors = true; + } } catch (Exception $e) { register_error(sprintf(elgg_echo("friends:remove:failure"),$friend->name)); $errors = true; -- cgit v1.2.3