Fix notices from admin_setting_configmultiselect when no options are selected.

2007-01-12 18:30:27 +00:00
parent b317007212
commit d7933a55c8
1 changed files with 12 additions and 1 deletions
@@ -1491,10 +1491,21 @@ class admin_setting_configmultiselect extends admin_setting_configselect {

    function get_setting() {
        global $CFG;
-        return (isset($CFG->{$this->name}) ? explode(',', $CFG->{$this->name}) : NULL);;
+        if (isset($CFG->{$this->name})) {
+            if ($CFG->{$this->name}) {
+                return explode(',', $CFG->{$this->name});
+            } else {
+                return array();
+            }            
+        } else {
+            return NULL;
+        }
    }

    function write_setting($data) {
+        if (empty($data)) {
+            $data = array();
+        }
        foreach ($data as $datum) {
            if (! in_array($datum, array_keys($this->choices))) {
                return get_string('errorsetting', 'admin') . $this->visiblename . '<br />';