Refactoring - removed not used default parameters from constructors.

[timetracker.git] / WEB-INF / lib / form / Form.class.php

diff --git a/WEB-INF/lib/form/Form.class.php b/WEB-INF/lib/form/Form.class.php
index 7814877f47023f0e54cec0210ecea849b67a3f28..e871743d0bb1e1af5141c3d07db317702d5a9538 100644 (file)
--- a/WEB-INF/lib/form/Form.class.php
+++ b/WEB-INF/lib/form/Form.class.php
@@ -26,15 +26,10 @@
 // | https://www.anuko.com/time_tracker/credits.htm
 // +----------------------------------------------------------------------+
 
 // | https://www.anuko.com/time_tracker/credits.htm
 // +----------------------------------------------------------------------+
 

-// +----------------------------------------------------------------------+
-// |
-// | Class generates elements of specification HTML 4.01
-// | http://www.w3.org/TR/1999/REC-html401-19991224
-// |
-// +----------------------------------------------------------------------+
-
+// Form class is a container for HTML forms we use in the application.
+// It contains an array of $elements - which are individual input controls
+// belonging to a form.

 class Form {
 class Form {

-

   var $name = '';          // Form name.
   var $elements = array(); // An array of input controls in form.
 
   var $name = '';          // Form name.
   var $elements = array(); // An array of input controls in form.
 


@@ -79,19 +74,20 @@ class Form {
         if (isset($params['format'])) $el->setFormat($params['format']);
         break;
 
         if (isset($params['format'])) $el->setFormat($params['format']);
         break;
 

+      case 'textarea':
+        import('form.TextArea');
+        $el = new TextArea($params['name']);
+        if (isset($params['cols'])) $el->setColumns($params['cols']);
+        if (isset($params['rows'])) $el->setRows($params['rows']);
+        if (isset($params['maxlength'])) $el->setMaxLength($params['maxlength']);
+        break;
+
+      case 'checkbox':
+        import('form.Checkbox');
+        $el = new Checkbox($params['name']);
+        if (isset($params['checked'])) $el->setChecked($params['checked']);
+

 // TODO: refactoring ongoing down from here.
 // TODO: refactoring ongoing down from here.

-               case "textarea":
-                           import('form.TextArea');
-                           $el = new TextArea($params["name"]);
-                           $el->setColumns(@$params["cols"]);
-                           $el->setRows(@$params["rows"]);
-                           if (isset($params["maxlength"])) $el->setMaxLength($params["maxlength"]);
-                           break;
-                           
-                       case "checkbox":
-                           import('form.Checkbox');
-                           $el = new Checkbox($params["name"]);
-                           if (@$params["checked"]) $el->setChecked(true);

                            $el->setData(@$params["data"]);
                            break;
                            
                            $el->setData(@$params["data"]);
                            break;