[SM-CVS] SF.net SVN: squirrelmail: [13004] trunk/squirrelmail

[<pdo...@us...>]

Revision: 13004
          http://squirrelmail.svn.sourceforge.net/squirrelmail/?rev=13004&view=rev
Author:   pdontthink
Date:     2008-03-05 13:44:14 -0800 (Wed, 05 Mar 2008)

Log Message:
-----------
Boolean option widgets now default to checkboxes

Modified Paths:
--------------
    trunk/squirrelmail/functions/options.php
    trunk/squirrelmail/templates/default/options.tpl

Modified: trunk/squirrelmail/functions/options.php
===================================================================
--- trunk/squirrelmail/functions/options.php	2008-03-05 21:07:41 UTC (rev 13003)
+++ trunk/squirrelmail/functions/options.php	2008-03-05 21:44:14 UTC (rev 13004)
@@ -549,12 +549,12 @@
      *                          constructed as a checkbox,
      *                          otherwise it will be a set of
      *                          Yes/No radio buttons (OPTIONAL;
-     *                          default is FALSE (radio buttons)).
+     *                          default is TRUE (checkbox)).
      *
      * @return string html formated boolean widget
      *
      */
-    function createWidget_Boolean($checkbox=FALSE) {
+    function createWidget_Boolean($checkbox=TRUE) {
 
         global $oTemplate, $nbsp;
 

Modified: trunk/squirrelmail/templates/default/options.tpl
===================================================================
--- trunk/squirrelmail/templates/default/options.tpl	2008-03-05 21:07:41 UTC (rev 13003)
+++ trunk/squirrelmail/templates/default/options.tpl	2008-03-05 21:44:14 UTC (rev 13004)
@@ -46,10 +46,8 @@
             echo   "<tr>\n" .
                    " <td class=\"optionName\">\n  ";
 
-//FIXME: use the following two lines instead if we make the default boolean type checkbox
-            //if ($opt->type == SMOPT_TYPE_BOOLEAN_CHECKBOX
-            // || $opt->type == SMOPT_TYPE_BOOLEAN) {
-            if ($opt->type == SMOPT_TYPE_BOOLEAN_CHECKBOX) {
+            if ($opt->type == SMOPT_TYPE_BOOLEAN_CHECKBOX
+             || $opt->type == SMOPT_TYPE_BOOLEAN) {
                 echo '<label for="new_' . $opt->name . '">'
                    . $opt->caption . '</label>';
             } else {


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.