Merge pull request #10995 from jdalsem/setting-arrays

chore(plugins): return false if usersetting value is an array
Elgg · Jul 21, 2017 · 823df94 · 823df94
2 parents aeeb8f2 + b1fa2c1
commit 823df94
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/engine/classes/ElggPlugin.php b/engine/classes/ElggPlugin.php
@@ -543,7 +543,7 @@ public function setUserSetting($name, $value, $user_guid = 0) {
 
 		if (is_array($value)) {
 			elgg_log('Plugin user settings cannot store arrays.', 'ERROR');
-			$result = false;
+			return false;
 		}
 
 		// set the namespaced name.