Ongoing refactoring of Checkbox class.

Author: anuko

WEB-INF/lib/form/Checkbox.class.php

@@ -29,7 +29,7 @@
 import('form.FormElement');
 
 class Checkbox extends FormElement {
-    var $mChecked	= false;
+  var $checked = false;
     var $mOptions	= null;
 
   function __construct($name, $value = '') {
@@ -38,27 +38,28 @@ function __construct($name, $value = '') {
     $this->value = $value;
   }
 
-	function setChecked($value)	{ $this->mChecked = $value; }
-	function isChecked() { return $this->mChecked; }
+	function setChecked($value) { $this->checked = $value; }
+	function isChecked() { return $this->checked; }
 
 	function setData($value)	{ $this->mOptions = $value; }
 	function getData() { return $this->mOptions; }
-	
-	function getHtml() {
 
-	    if ($this->id=="") $this->id = $this->name;
-	    
-		$html = "\n\t<input type=\"checkbox\"";
-		$html .= " name=\"$this->name\" id=\"$this->id\"";
+  function getHtml() {
+    if ($this->id == '') $this->id = $this->name;
+
+    $html = "\n\t<input type=\"checkbox\"";
+    $html.= " id=\"$this->id\" name=\"$this->name\"";
+
+    if ($this->checked || (($this->value == $this->mOptions) && ($this->value != null)))
+      $html.= " checked=\"true\"";
 
 		if ($this->on_change!="")
 		   $html .= " onchange=\"$this->on_change\"";
 
 		if ($this->style!="")
 		   $html .= " style=\"$this->style\"";
 
-		if ($this->mChecked || (($this->value == $this->mOptions) && ($this->value != null)))
-		   $html .= " checked=\"true\"";
+
 
 		if (!$this->isEnabled())
 		   $html .= " disabled=\"disabled\"";

WEB-INF/lib/form/Form.class.php

@@ -26,13 +26,9 @@
 // | 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 {
   var $name = '';          // Form name.
   var $elements = array(); // An array of input controls in form.
@@ -89,8 +85,9 @@ function addInput($params) {
       case 'checkbox':
         import('form.Checkbox');
         $el = new Checkbox($params['name']);
+        if (isset($params['checked'])) $el->setChecked(true);
+
 // TODO: refactoring ongoing down from here.
-        if (@$params["checked"]) $el->setChecked(true);
 			    $el->setData(@$params["data"]);
 			    break;
 

WEB-INF/templates/footer.tpl

@@ -12,7 +12,7 @@
       <br>
       <table cellspacing="0" cellpadding="4" width="100%" border="0">
         <tr>
-          <td align="center">&nbsp;Anuko Time Tracker 1.10.38.3618 | Copyright &copy; <a href="https://www.anuko.com/lp/tt_3.htm" target="_blank">Anuko</a> |
+          <td align="center">&nbsp;Anuko Time Tracker 1.10.38.3619 | Copyright &copy; <a href="https://www.anuko.com/lp/tt_3.htm" target="_blank">Anuko</a> |
             <a href="https://www.anuko.com/lp/tt_4.htm" target="_blank">{$i18n.footer.credits}</a> |
             <a href="https://www.anuko.com/lp/tt_5.htm" target="_blank">{$i18n.footer.license}</a> |
             <a href="https://www.anuko.com/lp/tt_7.htm" target="_blank">{$i18n.footer.improve}</a>