From: Jerome Bakker <jeabakker@coldtrick.com>
Date: Mon, 17 Jun 2013 12:47:49 +0000 (+0200)
Subject: fixed: wrong default behaviour in can_edit_entity_metadata
X-Git-Url: https://gitweb.fluxo.info/?a=commitdiff_plain;h=e29554909dc098f163a9dc6de31c42cd749ab4ae;p=lorea%2Felgg.git

fixed: wrong default behaviour in can_edit_entity_metadata
---

diff --git a/engine/lib/entities.php b/engine/lib/entities.php
index 072b26805..226cf5c6c 100644
--- a/engine/lib/entities.php
+++ b/engine/lib/entities.php
@@ -761,7 +761,7 @@ function get_entity($guid) {
 	// @todo We need a single Memcache instance with a shared pool of namespace wrappers. This function would pull an instance from the pool.
 	static $shared_cache;
 
-	// We could also use: if (!(int) $guid) { return FALSE }, 
+	// We could also use: if (!(int) $guid) { return FALSE },
 	// but that evaluates to a false positive for $guid = TRUE.
 	// This is a bit slower, but more thorough.
 	if (!is_numeric($guid) || $guid === 0 || $guid === '0') {
@@ -2126,7 +2126,7 @@ function can_edit_entity_metadata($entity_guid, $user_guid = 0, $metadata = null
 
 		$return = null;
 
-		if ($metadata->owner_guid == 0) {
+		if ($metadata && ($metadata->owner_guid == 0)) {
 			$return = true;
 		}
 		if (is_null($return)) {