aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJerome Bakker <jeabakker@coldtrick.com>2013-06-17 14:47:49 +0200
committerJerome Bakker <jeabakker@coldtrick.com>2013-06-17 14:47:49 +0200
commite29554909dc098f163a9dc6de31c42cd749ab4ae (patch)
tree1a6fa412b52eaf5e34c80b9de4933b86ee0c800d
parentafb0cef118a59c86e649e0717748c299a05f2427 (diff)
downloadelgg-e29554909dc098f163a9dc6de31c42cd749ab4ae.tar.gz
elgg-e29554909dc098f163a9dc6de31c42cd749ab4ae.tar.bz2
fixed: wrong default behaviour in can_edit_entity_metadata
-rw-r--r--engine/lib/entities.php4
1 files changed, 2 insertions, 2 deletions
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)) {