added a catch for configuration names longer than 32 characters

git-svn-id: http://code.elgg.org/elgg/trunk@7753 36083f99-b078-4883-b0ff-0f9b5a30f544

diff --git a/engine/lib/configuration.php b/engine/lib/configuration.php
index c79aff2d6d5afc2dd25cd56e41f96ec26c2a48e1..4bbed775c7edea391fc8f1844c31e4f91bbe6504 100644 (file)
--- a/engine/lib/configuration.php
+++ b/engine/lib/configuration.php
@@ -120,7 +120,7 @@ function elgg_set_config($name, $value) {
 /**
  * Save a configuration setting
  *
- * @param string $name      Configuration name
+ * @param string $name      Configuration name (cannot be greater than 32 characters)
  * @param mixed  $value     Configuration value. Should be string for installation setting
  * @param int    $site_guid NULL for installation setting, 0 for default site
  *
@@ -132,6 +132,11 @@ function elgg_save_config($name, $value, $site_guid = 0) {
 
        $name = trim($name);
 
+       if (strlen($name) > 32) {
+               elgg_log("The name length for configuration variables cannot be greater than 32", "ERROR");
+               return false;
+       }
+
        elgg_set_config($name, $value);
 
        if ($site_guid === NULL) {