Fix issue not allowing save of boolean prefs that default to SMPREF_ON

[squirrelmail.git] / functions / options.php

diff --git a/functions/options.php b/functions/options.php
index 0bca18e608ad5943b9ffaec1b6979ffb772e757a..fb0d8aa68f1f12a31672676af15b59b1f749de2d 100644 (file)
--- a/functions/options.php
+++ b/functions/options.php
@@ -784,6 +784,14 @@ function save_option($option) {
     } else if ($option->is_multiple_valued())
         setPref($data_dir, $username, $option->name, serialize($option->new_value));
 
+    // Checkboxes, when unchecked, don't submit anything in
+    // the POST, so set to SMPREF_OFF if not found
+    //
+    else if (($option->type == SMOPT_TYPE_BOOLEAN
+           || $option->type == SMOPT_TYPE_BOOLEAN_CHECKBOX)
+          && empty($option->new_value)) 
+        setPref($data_dir, $username, $option->name, SMPREF_OFF);
+
     else
         setPref($data_dir, $username, $option->name, $option->new_value);