From: pdontthink <pdontthink@7612ce4b-ef26-0410-bec9-ea0150e637f0>
Date: Tue, 4 Mar 2008 08:44:14 +0000 (+0000)
Subject: Add convenience function; keep track of multi-value widget types in one place
X-Git-Url: https://vcs.fsf.org/?a=commitdiff_plain;h=5a42c1017d480d84a59ec0ec994db74f002cd340;p=squirrelmail.git

Add convenience function; keep track of multi-value widget types in one place

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

diff --git a/functions/options.php b/functions/options.php
index 036e43c7..f9001402 100644
--- a/functions/options.php
+++ b/functions/options.php
@@ -202,6 +202,14 @@ class SquirrelOption {
         }
     }
 
+    /** Convenience function that identifies which types of
+        widgets are stored as (serialized) array values. */
+    function is_multiple_valued() {
+        return ($this->type == SMOPT_TYPE_FLDRLIST_MULTI
+             || $this->type == SMOPT_TYPE_STRLIST_MULTI
+             || $this->type == SMOPT_TYPE_EDIT_LIST);
+    }
+
     /**
      * Set the value for this option.
      * @param mixed $value
@@ -673,8 +681,7 @@ function save_option($option) {
     // Certain option types need to be serialized because
     // they are not scalar
     //
-    } else if ($option->type == SMOPT_TYPE_FLDRLIST_MULTI
-            || $option->type == SMOPT_TYPE_STRLIST_MULTI)
+    } else if ($option->is_multiple_valued())
         setPref($data_dir, $username, $option->name, serialize($option->new_value));
 
     else