From 6708fa8b981f9164cb58cb9363d6763181a1a4d4 Mon Sep 17 00:00:00 2001
From: skodak <skodak>
Date: Tue, 5 Feb 2008 11:42:51 +0000
Subject: [PATCH] MDL-12795 length specification in configtext constructor
 instead of css

---
 lib/adminlib.php                 | 29 +++++++++++++++--------------
 theme/standard/styles_layout.css | 17 -----------------
 2 files changed, 15 insertions(+), 31 deletions(-)

diff --git a/lib/adminlib.php b/lib/adminlib.php
index b9187241e8f..21c52ff2265 100644
--- a/lib/adminlib.php
+++ b/lib/adminlib.php
@@ -1659,6 +1659,7 @@ class admin_setting_heading extends admin_setting {
 class admin_setting_configtext extends admin_setting {
 
     var $paramtype;
+    var $size;
 
     /**
      * config text contructor
@@ -1667,9 +1668,15 @@ class admin_setting_configtext extends admin_setting {
      * @param string $description long localised info
      * @param string $defaultsetting
      * @param mixed $paramtype int means PARAM_XXX type, string is a allowed format in regex
+     * @param int $size default field size
      */
-    function admin_setting_configtext($name, $visiblename, $description, $defaultsetting, $paramtype=PARAM_RAW) {
+    function admin_setting_configtext($name, $visiblename, $description, $defaultsetting, $paramtype=PARAM_RAW, $size=null) {
         $this->paramtype = $paramtype;
+        if (!is_null($size)) {
+            $this->size  = $size;
+        } else {
+            $this->size  = ($paramtype == PARAM_INT) ? 5 : 30;
+        }
         parent::admin_setting($name, $visiblename, $description, $defaultsetting);
     }
 
@@ -1719,14 +1726,8 @@ class admin_setting_configtext extends admin_setting {
     function output_html($data, $query='') {
         $default = $this->get_defaultsetting();
 
-        if ($this->paramtype === PARAM_INT) {
-            $paramclass = 'class="number"';
-        } else {
-            $paramclass = '';
-        }
-
         return format_admin_setting($this, $this->visiblename,
-                '<div class="form-text defaultsnext"><input type="text" '.$paramclass.' id="'.$this->get_id().'" name="'.$this->get_full_name().'" value="'.s($data).'" /></div>',
+                '<div class="form-text defaultsnext"><input type="text" size="'.$this->size.'" id="'.$this->get_id().'" name="'.$this->get_full_name().'" value="'.s($data).'" /></div>',
                 $this->description, true, '', $default, $query);
     }
 }
@@ -1770,7 +1771,7 @@ class admin_setting_configpasswordunmask extends admin_setting_configtext {
      * @param string $defaultsetting default password
      */
     function admin_setting_configpasswordunmask($name, $visiblename, $description, $defaultsetting) {
-        parent::admin_setting_configtext($name, $visiblename, $description, $defaultsetting, PARAM_RAW);
+        parent::admin_setting_configtext($name, $visiblename, $description, $defaultsetting, PARAM_RAW, 30);
     }
 
     function output_html($data, $query='') {
@@ -1782,7 +1783,7 @@ document.write(\'<span class="unmask"><input id="'.$id.'unmask" value="1" type="
 //]]>
 </script>';
         return format_admin_setting($this, $this->visiblename,
-                '<div class="form-password"><input type="password" id="'.$this->get_id().'" name="'.$this->get_full_name().'" value="'.s($data).'" />'.$unmaskjs.'</div>',
+                '<div class="form-password"><input type="password" size="'.$this->size.'" id="'.$this->get_id().'" name="'.$this->get_full_name().'" value="'.s($data).'" />'.$unmaskjs.'</div>',
                 $this->description, true, '', NULL, $query);
     }
 }
@@ -1799,7 +1800,7 @@ class admin_setting_configfile extends admin_setting_configtext {
      * @param string $defaultdirectory default directory location
      */
     function admin_setting_configfile($name, $visiblename, $description, $defaultdirectory) {
-        parent::admin_setting_configtext($name, $visiblename, $description, $defaultdirectory, PARAM_RAW);
+        parent::admin_setting_configtext($name, $visiblename, $description, $defaultdirectory, PARAM_RAW, 50);
     }
 
     function output_html($data, $query='') {
@@ -1816,7 +1817,7 @@ class admin_setting_configfile extends admin_setting_configtext {
         }
 
         return format_admin_setting($this, $this->visiblename,
-                '<div class="form-file defaultsnext"><input type="text" id="'.$this->get_id().'" name="'.$this->get_full_name().'" value="'.s($data).'" />'.$executable.'</div>',
+                '<div class="form-file defaultsnext"><input type="text" size="'.$this->size.'" id="'.$this->get_id().'" name="'.$this->get_full_name().'" value="'.s($data).'" />'.$executable.'</div>',
                 $this->description, true, '', $default, $query);
     }
 }
@@ -1840,7 +1841,7 @@ class admin_setting_configexecutable extends admin_setting_configfile {
         }
 
         return format_admin_setting($this, $this->visiblename,
-                '<div class="form-file defaultsnext"><input type="text" id="'.$this->get_id().'" name="'.$this->get_full_name().'" value="'.s($data).'" />'.$executable.'</div>',
+                '<div class="form-file defaultsnext"><input type="text" size="'.$this->size.'" id="'.$this->get_id().'" name="'.$this->get_full_name().'" value="'.s($data).'" />'.$executable.'</div>',
                 $this->description, true, '', $default, $query);
     }
 }
@@ -1863,7 +1864,7 @@ class admin_setting_configdirectory extends admin_setting_configfile {
         }
 
         return format_admin_setting($this, $this->visiblename,
-                '<div class="form-file defaultsnext"><input type="text" id="'.$this->get_id().'" name="'.$this->get_full_name().'" value="'.s($data).'" />'.$executable.'</div>',
+                '<div class="form-file defaultsnext"><input type="text" size="'.$this->size.'" id="'.$this->get_id().'" name="'.$this->get_full_name().'" value="'.s($data).'" />'.$executable.'</div>',
                 $this->description, true, '', $default, $query);
     }
 }
diff --git a/theme/standard/styles_layout.css b/theme/standard/styles_layout.css
index 69e52edb13f..0d96029b49a 100644
--- a/theme/standard/styles_layout.css
+++ b/theme/standard/styles_layout.css
@@ -1079,23 +1079,6 @@ body#admin-modules table.generaltable td.c0
   margin:0;
 }
 
-#adminsettings .form-text input {
-  width: 20em;
-}
-
-#adminsettings .form-password input[type="text"],
-#adminsettings .form-password input[type="password"] {
-    width: 20em;
-}
-
-#adminsettings .form-text input.number {
-  width: 5em;
-}
-
-#adminsettings .form-file input {
-    width: 30em;
-}
-
 #adminsettings .form-item .pathok,
 #adminsettings .form-item .patherror {
   margin-left: 0.5em;