X-Git-Url: http://wagnertech.de/git?a=blobdiff_plain;f=WEB-INF%2Flib%2Fform%2FCheckbox.class.php;h=c4acbc5f51ee0aeaa011cd54ff39d1a474ca92d4;hb=fcb232f2236dd0105525b0239fa7b212aaf72a2a;hp=8c20288ae29c88e53198f6fe85549ad8016603ba;hpb=ac5189157ef60820dd125400baee9f408a9ba2ea;p=timetracker.git
diff --git a/WEB-INF/lib/form/Checkbox.class.php b/WEB-INF/lib/form/Checkbox.class.php
index 8c20288a..c4acbc5f 100644
--- a/WEB-INF/lib/form/Checkbox.class.php
+++ b/WEB-INF/lib/form/Checkbox.class.php
@@ -29,28 +29,20 @@
import('form.FormElement');
class Checkbox extends FormElement {
- var $mChecked = false;
- var $mOptions = null;
- var $class = 'Checkbox';
- function __construct($name,$value="")
- {
- $this->name = $name;
- $this->value = $value;
- }
+ function __construct($name) {
+ $this->class = 'Checkbox';
+ $this->name = $name;
+ }
+
+ function getHtml() {
+ if ($this->id == '') $this->id = $this->name;
- function setChecked($value) { $this->mChecked = $value; }
- function isChecked() { return $this->mChecked; }
-
- function setData($value) { $this->mOptions = $value; }
- function getData() { return $this->mOptions; }
-
- function toStringControl() {
+ $html = "\n\tid\" name=\"$this->name\"";
- if ($this->id=="") $this->id = $this->name;
-
- $html = "\n\tname\" id=\"$this->id\"";
+ if ($this->value)
+ $html.= " checked=\"true\"";
if ($this->on_change!="")
$html .= " onchange=\"$this->on_change\"";
@@ -58,13 +50,14 @@ class Checkbox extends FormElement {
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\"";
-
- $html .= " value=\"".htmlspecialchars($this->mOptions)."\"";
+
+ // Provide a value so that we pass "1" for set checkboxes on form submit.
+ // Otherwise the default is "on" string, which is not what we want.
+ $html .= " value=\"1\"";
$html .= "/>\n";