Carried over modification made to file_prefs to distinguish

between a 0 value, and an unset value using ===.

Certain prefs were unsaveable after the XSS changes, which
cast zero values to ints (instead of strings), causing
them to pass the $set_to == '' check, and thus be treated
as an unset value, rather than a value of 0.

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@4280 7612ce4b-ef26-0410-bec9-ea0150e637f0

diff --git a/functions/db_prefs.php b/functions/db_prefs.php
index 3708441dac832583a3940c7b3b4e595e663ece04..ef96837d4a0c7915deadd1e6286461431b24b7ee 100644 (file)
--- a/functions/db_prefs.php
+++ b/functions/db_prefs.php
@@ -298,10 +298,10 @@ function setPref($data_dir, $username, $string, $set_to) {
     global $prefs_cache;
 
     if (isset($prefs_cache[$string]) && ($prefs_cache[$string] == $set_to)) {
-        return;
+       return;
     }
 
-    if ($set_to == '') {
+    if ($set_to === '') {
         removePref($data_dir, $username, $string);
         return;
     }