]> gitweb.fluxo.info Git - lorea/elgg.git/commitdiff
Added checking for real entities to other private setting functions.
authorbrettp <brettp@36083f99-b078-4883-b0ff-0f9b5a30f544>
Sun, 2 Jan 2011 21:14:23 +0000 (21:14 +0000)
committerbrettp <brettp@36083f99-b078-4883-b0ff-0f9b5a30f544>
Sun, 2 Jan 2011 21:14:23 +0000 (21:14 +0000)
git-svn-id: http://code.elgg.org/elgg/trunk@7814 36083f99-b078-4883-b0ff-0f9b5a30f544

engine/lib/private_settings.php

index d21ee45e697cc5e8a368dd7d8cacf257ed17e8ac..366e8cb39023145d10f8198ea3a09724537198b3 100644 (file)
@@ -433,6 +433,11 @@ function get_private_setting($entity_guid, $name) {
        $entity_guid = (int) $entity_guid;
        $name = sanitise_string($name);
 
+       $entity = get_entity($entity_guid);
+       if (!$entity instanceof ElggEntity) {
+               return false;
+       }
+
        $query = "SELECT value from {$CONFIG->dbprefix}private_settings
                where name = '{$name}' and entity_guid = {$entity_guid}";
        $setting = get_data_row($query);
@@ -459,6 +464,10 @@ function get_all_private_settings($entity_guid) {
        global $CONFIG;
 
        $entity_guid = (int) $entity_guid;
+       $entity = get_entity($entity_guid);
+       if (!$entity instanceof ElggEntity) {
+               return false;
+       }
 
        $query = "SELECT * from {$CONFIG->dbprefix}private_settings where entity_guid = {$entity_guid}";
        $result = get_data($query);
@@ -527,6 +536,12 @@ function remove_private_setting($entity_guid, $name) {
        global $CONFIG;
 
        $entity_guid = (int) $entity_guid;
+
+       $entity = get_entity($entity_guid);
+       if (!$entity instanceof ElggEntity) {
+               return false;
+       }
+
        $name = sanitise_string($name);
 
        return delete_data("DELETE from {$CONFIG->dbprefix}private_settings
@@ -550,6 +565,12 @@ function remove_all_private_settings($entity_guid) {
        global $CONFIG;
 
        $entity_guid = (int) $entity_guid;
+
+       $entity = get_entity($entity_guid);
+       if (!$entity instanceof ElggEntity) {
+               return false;
+       }
+
        return delete_data("DELETE from {$CONFIG->dbprefix}private_settings
                where entity_guid = {$entity_guid}");
 }