From df29af4750c1613771be05b3477a038ca420fc82 Mon Sep 17 00:00:00 2001
From: marcus <marcus@36083f99-b078-4883-b0ff-0f9b5a30f544>
Date: Fri, 25 Jul 2008 20:07:44 +0000
Subject: User test negative non-fatal on permissions check

git-svn-id: https://code.elgg.org/elgg/trunk@1548 36083f99-b078-4883-b0ff-0f9b5a30f544
---
 engine/lib/entities.php | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

(limited to 'engine')

diff --git a/engine/lib/entities.php b/engine/lib/entities.php
index 236aa880e..663ac9c4e 100644
--- a/engine/lib/entities.php
+++ b/engine/lib/entities.php
@@ -1588,11 +1588,15 @@
 			$user = get_entity($user_guid);
 		}
 
-		if (($entity = get_entity($entity_guid)) && (!is_null($user))) {
+		if ($entity = get_entity($entity_guid)) {
 			
-			if ($entity->getOwner() == $user->getGUID()) return true;
-			if ($entity->type == "user" && $entity->getGUID() == $user->getGUID()) return true;
-		
+			// Test user if possible - should default to false unless a plugin hook says otherwise
+			if (!is_null($user))
+			{
+				if ($entity->getOwner() == $user->getGUID()) return true;
+				if ($entity->type == "user" && $entity->getGUID() == $user->getGUID()) return true;
+			}
+				
 			return trigger_plugin_hook('permissions_check',$entity->type,array('entity' => $entity, 'user' => $user),false);
 		
 		} else {		
-- 
cgit v1.2.3