From f7633b0f2e5ab47d8dc1b7f12955cbb2512bf3d1 Mon Sep 17 00:00:00 2001
From: skodak <skodak>
Date: Tue, 5 Feb 2008 11:45:05 +0000
Subject: [PATCH] MDL-12795 length specification in configtext constructor
 instead of css; merged from MOODLE_19_STABLE

---
 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 165978420a..78afad3a16 100644
--- a/lib/adminlib.php
+++ b/lib/adminlib.php
@@ -1708,6 +1708,7 @@ class admin_setting_heading extends admin_setting {
 class admin_setting_configtext extends admin_setting {
 
     var $paramtype;
+    var $size;
 
     /**
      * config text contructor
@@ -1716,9 +1717,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);
     }
 
@@ -1768,14 +1775,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);
     }
 }
@@ -1819,7 +1820,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='') {
@@ -1831,7 +1832,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);
     }
 }
@@ -1848,7 +1849,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='') {
@@ -1865,7 +1866,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);
     }
 }
@@ -1889,7 +1890,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);
     }
 }
@@ -1912,7 +1913,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 82a245148e..6724ff9c8c 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;
-- 
2.39.5