jcheron / phalcon-jquery

Ajax/semantic/html/elements/HtmlButton.php

Doc Comments +3 added lines, -4 removed lines

@@ -38,7 +38,7 @@ 
 	/**
 	 * Set the button value
 	 * @param string $value
-	 * @return \Ajax\semantic\html\HtmlButton
+	 * @return HtmlButton
 	 */
 	public function setValue($value) {
 		$this->content=$value;
@@ -47,8 +47,8 @@ 
 
 	/**
 	 * define the button style
-	 * @param string|int $cssStyle
-	 * @return \Ajax\semantic\html\HtmlButton default : ""
+	 * @param string $cssStyle
+	 * @return HtmlButton default : ""
 	 */
 	public function setStyle($cssStyle) {
 		return $this->addToProperty("class", $cssStyle);
@@ -98,7 +98,6 @@ 
 
 	/**
 	 * Add and return a button label
-	 * @param string $caption
 	 * @param string $before
 	 * @param string $icon
 	 * @return \Ajax\semantic\html\elements\HtmlLabel

Spacing +12 added lines, -12 removed lines

@@ -55,7 +55,7 @@ 
 	}
 
 	public function setFocusable($value=true) {
-		if ($value === true)
+		if ($value===true)
 			$this->setProperty("tabindex", "0");
 		else {
 			$this->removeProperty("tabindex");
@@ -65,14 +65,14 @@ 
 
 	public function setAnimated($content, $animation="") {
 		$this->setTagName("div");
-		$this->addToProperty("class", "animated " . $animation);
-		$visible=new HtmlSemDoubleElement("visible-" . $this->identifier, "div");
+		$this->addToProperty("class", "animated ".$animation);
+		$visible=new HtmlSemDoubleElement("visible-".$this->identifier, "div");
 		$visible->setClass("visible content");
 		$visible->setContent($this->content);
-		$hidden=new HtmlSemDoubleElement("hidden-" . $this->identifier, "div");
+		$hidden=new HtmlSemDoubleElement("hidden-".$this->identifier, "div");
 		$hidden->setClass("hidden content");
 		$hidden->setContent($content);
-		$this->content=array ($visible,$hidden );
+		$this->content=array($visible, $hidden);
 		return $hidden;
 	}
 
@@ -84,7 +84,7 @@ 
 	public function asIcon($icon) {
 		$iconO=$icon;
 		if (\is_string($icon)) {
-			$iconO=new HtmlIcon("icon-" . $this->identifier, $icon);
+			$iconO=new HtmlIcon("icon-".$this->identifier, $icon);
 		}
 		$this->addToProperty("class", "icon");
 		$this->content=$iconO;
@@ -106,10 +106,10 @@ 
 	public function addLabel($label, $before=false, $icon=NULL) {
 		$this->tagName="div";
 		$this->addToProperty("class", "labeled");
-		$this->content=new HtmlButton("button-" . $this->identifier, $this->content);
+		$this->content=new HtmlButton("button-".$this->identifier, $this->content);
 		$this->content->setTagName("div");
-		$label=new HtmlLabel("label-" . $this->identifier, $label, "a");
-		if(isset($icon))
+		$label=new HtmlLabel("label-".$this->identifier, $label, "a");
+		if (isset($icon))
 			$label->addIcon($icon);
 		$label->setBasic();
 		$this->addContent($label, $before);
@@ -122,7 +122,7 @@ 
 	 */
 	public function fromArray($array) {
 		$array=parent::fromArray($array);
-		foreach ( $array as $key => $value ) {
+		foreach ($array as $key => $value) {
 			$this->setProperty($key, $value);
 		}
 		return $array;
@@ -185,7 +185,7 @@ 
 	}
 
 	public static function social($identifier, $social, $value=NULL) {
-		if ($value === NULL)
+		if ($value===NULL)
 			$value=\ucfirst($social);
 		$return=new HtmlButton($identifier, $value);
 		$return->addIcon($social);
@@ -205,7 +205,7 @@ 
 	}
 
 	public function asLink($href=NULL) {
-		$lnk=new HtmlLink("lnk-".$this->identifier,$href,$this->content);
+		$lnk=new HtmlLink("lnk-".$this->identifier, $href, $this->content);
 		$this->content=$lnk;
 		return $this;
 	}

Braces +9 added lines, -7 removed lines

@@ -55,9 +55,9 @@ 
 	}
 
 	public function setFocusable($value=true) {
-		if ($value === true)
-			$this->setProperty("tabindex", "0");
-		else {
+		if ($value === true) {
+					$this->setProperty("tabindex", "0");
+		} else {
 			$this->removeProperty("tabindex");
 		}
 		return $this;
@@ -109,8 +109,9 @@ 
 		$this->content=new HtmlButton("button-" . $this->identifier, $this->content);
 		$this->content->setTagName("div");
 		$label=new HtmlLabel("label-" . $this->identifier, $label, "a");
-		if(isset($icon))
-			$label->addIcon($icon);
+		if(isset($icon)) {
+					$label->addIcon($icon);
+		}
 		$label->setBasic();
 		$this->addContent($label, $before);
 		return $label;
@@ -185,8 +186,9 @@ 
 	}
 
 	public static function social($identifier, $social, $value=NULL) {
-		if ($value === NULL)
-			$value=\ucfirst($social);
+		if ($value === NULL) {
+					$value=\ucfirst($social);
+		}
 		$return=new HtmlButton($identifier, $value);
 		$return->addIcon($social);
 		return $return->addToPropertyCtrl("class", $social, Social::getConstants());

Ajax/semantic/html/elements/HtmlStep.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use Ajax\semantic\html\base\HtmlSemCollection;
 use Ajax\semantic\html\content\HtmlStepItem;
 use Ajax\JsUtils;
-
 use Ajax\common\html\HtmlDoubleElement;
 use Ajax\semantic\html\base\constants\Side;
 

Spacing +23 added lines, -23 removed lines

@@ -9,11 +9,11 @@ 
 use Ajax\common\html\HtmlDoubleElement;
 use Ajax\semantic\html\base\constants\Side;
 
-class HtmlStep extends HtmlSemCollection{
+class HtmlStep extends HtmlSemCollection {
 	protected $_activeStep;
 
-	public function __construct( $identifier,$steps=array()){
-		parent::__construct( $identifier,"div", "ui steps");
+	public function __construct($identifier, $steps=array()) {
+		parent::__construct($identifier, "div", "ui steps");
 		$this->addItems($steps);
 	}
 
@@ -23,7 +23,7 @@ 
 	 * @see \Ajax\common\html\html5\HtmlCollection::createItem()
 	 */
 	protected function createItem($value) {
-		$itemO=new HtmlStepItem("item-".\sizeof($this->content),$value);
+		$itemO=new HtmlStepItem("item-".\sizeof($this->content), $value);
 		return $itemO;
 	}
 
@@ -31,50 +31,50 @@ 
 	 * @param string|array $step
 	 * @return HtmlStepItem
 	 */
-	public function addStep($step){
+	public function addStep($step) {
 		return $this->addItem($step);
 	}
 
-	public function setOrdered(){
+	public function setOrdered() {
 		return $this->addToProperty("class", "ordered");
 	}
 
-	public function isOrdered(){
+	public function isOrdered() {
 		return $this->propertyContains("class", "ordered");
 	}
 
-	public function setVertical(){
+	public function setVertical() {
 		return $this->addToProperty("class", "vertical");
 	}
 
-	protected function defineActiveStep(){
+	protected function defineActiveStep() {
 		$activestep=$this->_activeStep;
 		$count=$this->count();
-		if($this->isOrdered()){
-			for($i=0;$i<$count;$i++){
+		if ($this->isOrdered()) {
+			for ($i=0; $i<$count; $i++) {
 				$step=$this->content[$i];
 				$step->removeStatus();
-				if($i<$activestep)
+				if ($i<$activestep)
 					$step->setCompleted();
 				elseif ($i===$activestep)
 					$step->setActive();
 				else
 					$step->setDisabled();
 			}
-		}else{
-			foreach ($this->content as $step){
+		}else {
+			foreach ($this->content as $step) {
 				$step->removeStatus();
 			}
-			if($activestep<$count)
+			if ($activestep<$count)
 				$this->content[$activestep]->setActive();
 		}
 		return $this;
 	}
 
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
-		if(isset($this->_activeStep)===true && \is_numeric($this->_activeStep))
+		if (isset($this->_activeStep)===true && \is_numeric($this->_activeStep))
 			$this->defineActiveStep();
-		return parent::compile($js,$view);
+		return parent::compile($js, $view);
 	}
 
 	public function setActiveStep($_activeStep) {
@@ -82,15 +82,15 @@ 
 		return $this;
 	}
 
-	public function setAttached($side="",HtmlDoubleElement $toElement=NULL){
-		if(isset($toElement)){
-			$toElement->addToPropertyCtrl("class", "attached",array("attached"));
+	public function setAttached($side="", HtmlDoubleElement $toElement=NULL) {
+		if (isset($toElement)) {
+			$toElement->addToPropertyCtrl("class", "attached", array("attached"));
 		}
-		return $this->addToPropertyCtrl("class", $side." attached",Side::getConstantValues("attached"));
+		return $this->addToPropertyCtrl("class", $side." attached", Side::getConstantValues("attached"));
 	}
 
-	public function asLink(){
-		foreach ($this->content as $step){
+	public function asLink() {
+		foreach ($this->content as $step) {
 			$step->asLink();
 		}
 		return $this;

Braces +14 added lines, -11 removed lines

@@ -54,26 +54,29 @@
 			for($i=0;$i<$count;$i++){
 				$step=$this->content[$i];
 				$step->removeStatus();
-				if($i<$activestep)
-					$step->setCompleted();
-				elseif ($i===$activestep)
-					$step->setActive();
-				else
-					$step->setDisabled();
+				if($i<$activestep) {
+									$step->setCompleted();
+				} elseif ($i===$activestep) {
+									$step->setActive();
+				} else {
+									$step->setDisabled();
+				}
 			}
-		}else{
+		} else{
 			foreach ($this->content as $step){
 				$step->removeStatus();
 			}
-			if($activestep<$count)
-				$this->content[$activestep]->setActive();
+			if($activestep<$count) {
+							$this->content[$activestep]->setActive();
+			}
 		}
 		return $this;
 	}
 
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
-		if(isset($this->_activeStep)===true && \is_numeric($this->_activeStep))
-			$this->defineActiveStep();
+		if(isset($this->_activeStep)===true && \is_numeric($this->_activeStep)) {
+					$this->defineActiveStep();
+		}
 		return parent::compile($js,$view);
 	}
 

Ajax/semantic/html/modules/HtmlProgress.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use Ajax\semantic\html\base\HtmlSemDoubleElement;
 use Ajax\JsUtils;
 use Ajax\service\JArray;
-
 use Ajax\semantic\html\base\constants\State;
 
 class HtmlProgress extends HtmlSemDoubleElement {

Spacing +15 added lines, -15 removed lines

@@ -9,26 +9,26 @@ 
 use Ajax\semantic\html\base\constants\State;
 
 class HtmlProgress extends HtmlSemDoubleElement {
-	private $_params=array ();
+	private $_params=array();
 
 	public function __construct($identifier, $value=NULL, $label=NULL, $attributes=array()) {
 		parent::__construct($identifier, "div", "ui progress");
-		if (isset($value) === true)
+		if (isset($value)===true)
 			$this->setProperty("data-percent", $value);
 		$this->createBar();
-		if (isset($label) === true)
+		if (isset($label)===true)
 			$this->setLabel($label);
-		$this->_states=[ State::SUCCESS,State::WARNING,State::ERROR,State::ACTIVE,State::DISABLED ];
+		$this->_states=[State::SUCCESS, State::WARNING, State::ERROR, State::ACTIVE, State::DISABLED];
 		$this->addToProperty("class", $attributes);
 	}
 
 	public function setLabel($label) {
-		$this->content["label"]=new HtmlSemDoubleElement("lbl-" . $this->identifier, "div", "label", $label);
+		$this->content["label"]=new HtmlSemDoubleElement("lbl-".$this->identifier, "div", "label", $label);
 		return $this;
 	}
 
 	private function createBar() {
-		$bar=new HtmlSemDoubleElement("bar-" . $this->identifier, "div", "bar", new HtmlSemDoubleElement("progress-" . $this->identifier, "div", "progress"));
+		$bar=new HtmlSemDoubleElement("bar-".$this->identifier, "div", "bar", new HtmlSemDoubleElement("progress-".$this->identifier, "div", "progress"));
 		$this->content["bar"]=$bar;
 		return $this;
 	}
@@ -68,20 +68,20 @@ 
 	 * @see \Ajax\semantic\html\base\HtmlSemDoubleElement::compile()
 	 */
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
-		$this->content=JArray::sortAssociative($this->content, [ "bar","label" ]);
+		$this->content=JArray::sortAssociative($this->content, ["bar", "label"]);
 		return parent::compile($js, $view);
 	}
 
 	public function jsSetValue($value) {
-		return '$("#' . $this->identifier . '").progress({value:' . $value . '});';
+		return '$("#'.$this->identifier.'").progress({value:'.$value.'});';
 	}
 
 	public function jsIncValue() {
-		return '$("#' . $this->identifier . '").progress("increment");';
+		return '$("#'.$this->identifier.'").progress("increment");';
 	}
 
 	public function jsDecValue() {
-		return '$("#' . $this->identifier . '").progress("decrement");';
+		return '$("#'.$this->identifier.'").progress("decrement");';
 	}
 
 	/**
@@ -95,7 +95,7 @@ 
 	 * @return HtmlProgress
 	 */
 	public function setTextValues($active=false, $error=false, $success=false, $warning=false, $percent="{percent}%", $ratio="{value} of {total}") {
-		if (\is_array($active) == true) {
+		if (\is_array($active)==true) {
 			$array=$active;
 			$active=JArray::getDefaultValue($array, "active", false);
 			$success=JArray::getDefaultValue($array, "success", $success);
@@ -103,12 +103,12 @@ 
 			$percent=JArray::getDefaultValue($array, "percent", $percent);
 			$ratio=JArray::getDefaultValue($array, "ratio", $ratio);
 		}
-		$this->_params["text"]="%{active  : " . \var_export($active, true) . ",error: " . \var_export($error, true) . ",success : " . \var_export($success, true) . ",warning : " . \var_export($warning, true) . ",percent : " . \var_export($percent, true) . ",ratio   : " . \var_export($ratio, true) . "}%";
+		$this->_params["text"]="%{active  : ".\var_export($active, true).",error: ".\var_export($error, true).",success : ".\var_export($success, true).",warning : ".\var_export($warning, true).",percent : ".\var_export($percent, true).",ratio   : ".\var_export($ratio, true)."}%";
 		return $this;
 	}
 
 	public function onChange($jsCode) {
-		return $this->_params["onChange"]="%function(percent, value, total){" . $jsCode . "}%";
+		return $this->_params["onChange"]="%function(percent, value, total){".$jsCode."}%";
 	}
 
 	/*
@@ -116,8 +116,8 @@ 
 	 * @see BaseHtml::run()
 	 */
 	public function run(JsUtils $js) {
-		if (isset($this->_bsComponent) === false)
-			$this->_bsComponent=$js->semantic()->progress("#" . $this->identifier, $this->_params);
+		if (isset($this->_bsComponent)===false)
+			$this->_bsComponent=$js->semantic()->progress("#".$this->identifier, $this->_params);
 		$this->addEventsOnRun($js);
 		return $this->_bsComponent;
 	}

Braces +9 added lines, -6 removed lines

@@ -13,11 +13,13 @@ 
 
 	public function __construct($identifier, $value=NULL, $label=NULL, $attributes=array()) {
 		parent::__construct($identifier, "div", "ui progress");
-		if (isset($value) === true)
-			$this->setProperty("data-percent", $value);
+		if (isset($value) === true) {
+					$this->setProperty("data-percent", $value);
+		}
 		$this->createBar();
-		if (isset($label) === true)
-			$this->setLabel($label);
+		if (isset($label) === true) {
+					$this->setLabel($label);
+		}
 		$this->_states=[ State::SUCCESS,State::WARNING,State::ERROR,State::ACTIVE,State::DISABLED ];
 		$this->addToProperty("class", $attributes);
 	}
@@ -116,8 +118,9 @@ 
 	 * @see BaseHtml::run()
 	 */
 	public function run(JsUtils $js) {
-		if (isset($this->_bsComponent) === false)
-			$this->_bsComponent=$js->semantic()->progress("#" . $this->identifier, $this->_params);
+		if (isset($this->_bsComponent) === false) {
+					$this->_bsComponent=$js->semantic()->progress("#" . $this->identifier, $this->_params);
+		}
 		$this->addEventsOnRun($js);
 		return $this->_bsComponent;
 	}

Ajax/common/BaseEnum.php

Spacing +9 added lines, -9 removed lines

@@ -12,8 +12,8 @@ 
 	private static $constCacheArray=NULL;
 
 	public static function getConstants() {
-		if (self::$constCacheArray == NULL) {
-			self::$constCacheArray=[ ];
+		if (self::$constCacheArray==NULL) {
+			self::$constCacheArray=[];
 		}
 		$calledClass=get_called_class();
 		if (!array_key_exists($calledClass, self::$constCacheArray)) {
@@ -23,16 +23,16 @@ 
 		return self::$constCacheArray[$calledClass];
 	}
 
-	public static function getConstantValues($postFix="",$prefixBefore=false) {
-		if ($postFix == "")
+	public static function getConstantValues($postFix="", $prefixBefore=false) {
+		if ($postFix=="")
 			return \array_values(self::getConstants());
 		else {
-			if($prefixBefore===false){
-				return \array_map(function ($elem) use($postFix) {
-					return $elem . " " . $postFix;
+			if ($prefixBefore===false) {
+				return \array_map(function($elem) use($postFix) {
+					return $elem." ".$postFix;
 				}, \array_values(self::getConstants()));
-			}else{
-				return \array_map(function ($elem) use($postFix) {
+			}else {
+				return \array_map(function($elem) use($postFix) {
 					return $postFix." ".$elem;
 				}, \array_values(self::getConstants()));
 			}

Braces +4 added lines, -4 removed lines

@@ -24,14 +24,14 @@
 	}
 
 	public static function getConstantValues($postFix="",$prefixBefore=false) {
-		if ($postFix == "")
-			return \array_values(self::getConstants());
-		else {
+		if ($postFix == "") {
+					return \array_values(self::getConstants());
+		} else {
 			if($prefixBefore===false){
 				return \array_map(function ($elem) use($postFix) {
 					return $elem . " " . $postFix;
 				}, \array_values(self::getConstants()));
-			}else{
+			} else{
 				return \array_map(function ($elem) use($postFix) {
 					return $postFix." ".$elem;
 				}, \array_values(self::getConstants()));

Ajax/common/html/html5/HtmlInput.php

Spacing +6 added lines, -6 removed lines

@@ -7,7 +7,7 @@ 
 
 class HtmlInput extends HtmlSingleElement {
 
-	public function __construct($identifier,$type="text",$value=NULL,$placeholder=NULL) {
+	public function __construct($identifier, $type="text", $value=NULL, $placeholder=NULL) {
 		parent::__construct($identifier, "input");
 		$this->setProperty("name", $identifier);
 		$this->setValue($value);
@@ -16,7 +16,7 @@ 
 	}
 
 	public function setValue($value) {
-		if(isset($value))
+		if (isset($value))
 		$this->setProperty("value", $value);
 		return $this;
 	}
@@ -25,12 +25,12 @@ 
 		return $this->setProperty("type", $value);
 	}
 
-	public function setPlaceholder($value){
-		if(JString::isNull($value)){
-			if(JString::isNotNull($this->identifier))
+	public function setPlaceholder($value) {
+		if (JString::isNull($value)) {
+			if (JString::isNotNull($this->identifier))
 				$value=\ucfirst($this->identifier);
 		}
-		if(JString::isNotNull($value))
+		if (JString::isNotNull($value))
 			$this->setProperty("placeholder", $value);
 		return $this;
 	}

Braces +9 added lines, -6 removed lines

@@ -16,8 +16,9 @@ 
 	}
 
 	public function setValue($value) {
-		if(isset($value))
-		$this->setProperty("value", $value);
+		if(isset($value)) {
+				$this->setProperty("value", $value);
+		}
 		return $this;
 	}
 
@@ -27,11 +28,13 @@ 
 
 	public function setPlaceholder($value){
 		if(JString::isNull($value)){
-			if(JString::isNotNull($this->identifier))
-				$value=\ucfirst($this->identifier);
+			if(JString::isNotNull($this->identifier)) {
+							$value=\ucfirst($this->identifier);
+			}
+		}
+		if(JString::isNotNull($value)) {
+					$this->setProperty("placeholder", $value);
 		}
-		if(JString::isNotNull($value))
-			$this->setProperty("placeholder", $value);
 		return $this;
 	}
 }
\ No newline at end of file

Ajax/semantic/traits/SemanticHtmlElementsTrait.php

Spacing +4 added lines, -4 removed lines

@@ -130,12 +130,12 @@
 	 * @param string $tagName
 	 * @return HtmlLabel
 	 */
-	public function htmlLabel($identifier, $content="", $icon=NULL,$tagName="div") {
-		return $this->addHtmlComponent(new HtmlLabel($identifier, $content,$icon, $tagName));
+	public function htmlLabel($identifier, $content="", $icon=NULL, $tagName="div") {
+		return $this->addHtmlComponent(new HtmlLabel($identifier, $content, $icon, $tagName));
 	}
 
-	public function htmlLabelGroups($identifier,$labels=array(),$attributes=array()){
-		return $this->addHtmlComponent(new HtmlLabelGroups($identifier,$labels,$attributes));
+	public function htmlLabelGroups($identifier, $labels=array(), $attributes=array()) {
+		return $this->addHtmlComponent(new HtmlLabelGroups($identifier, $labels, $attributes));
 	}
 
 	/**

Ajax/semantic/html/elements/HtmlLabelGroups.php

Spacing +4 added lines, -4 removed lines

@@ -15,9 +15,9 @@ 
  */
 class HtmlLabelGroups extends HtmlSemCollection {
 
-	public function __construct($identifier,$labels=array(), $attributes=array()) {
+	public function __construct($identifier, $labels=array(), $attributes=array()) {
 		parent::__construct($identifier, "div", "ui labels");
-		$this->_states=\array_merge(Size::getConstants(),Color::getConstants(),["tag","circular"]);
+		$this->_states=\array_merge(Size::getConstants(), Color::getConstants(), ["tag", "circular"]);
 		$this->addItems($labels);
 		$this->setStates($attributes);
 	}
@@ -31,12 +31,12 @@ 
 			$icon=JArray::getValue($value, "icon", 1);
 			$tagName=JArray::getValue($value, "tagName", 2);
 		}
-		$labelO=new HtmlLabel("label-" . $this->identifier, $caption,$icon,$tagName);
+		$labelO=new HtmlLabel("label-".$this->identifier, $caption, $icon, $tagName);
 		return $labelO;
 	}
 
 	protected function createCondition($value) {
-		return ($value instanceof HtmlLabel) === false;
+		return ($value instanceof HtmlLabel)===false;
 	}
 
 }
\ No newline at end of file

Ajax/semantic/html/elements/HtmlList.php

Spacing +15 added lines, -15 removed lines

@@ -18,17 +18,17 @@ 
 
 	protected function createItem($value) {
 		$count=$this->count();
-		$item=new HtmlListItem("item-" . $this->identifier . "-" . $count, $value);
+		$item=new HtmlListItem("item-".$this->identifier."-".$count, $value);
 		return $item;
 	}
 
 	public function addHeader($niveau, $content) {
-		$header=new HtmlHeader("header-" . $this->identifier, $niveau, $content, "page");
+		$header=new HtmlHeader("header-".$this->identifier, $niveau, $content, "page");
 		$this->wrap($header);
 		return $header;
 	}
 
-	public function getItemPart($index,$partName="header"){
+	public function getItemPart($index, $partName="header") {
 		return $this->getItem($index)->getPart($partName);
 	}
 
@@ -37,7 +37,7 @@ 
 	}
 
 	public function asLink() {
-		$this->addToPropertyCtrl("class", "link", array ("link" ));
+		$this->addToPropertyCtrl("class", "link", array("link"));
 		return $this->contentAs("a");
 	}
 
@@ -60,9 +60,9 @@ 
 	}
 
 	public function run(JsUtils $js) {
-		if ($this->_hasCheckedList === true) {
-			$jsCode=include dirname(__FILE__) . '/../../components/jsTemplates/tplCheckedList.php';
-			$jsCode=\str_replace("%identifier%", "#" . $this->identifier, $jsCode);
+		if ($this->_hasCheckedList===true) {
+			$jsCode=include dirname(__FILE__).'/../../components/jsTemplates/tplCheckedList.php';
+			$jsCode=\str_replace("%identifier%", "#".$this->identifier, $jsCode);
 			$this->executeOnRun($jsCode);
 		}
 		return parent::run($js);
@@ -86,28 +86,28 @@ 
 
 	public function addCheckedList($items=array(), $masterItem=NULL, $values=array()) {
 		$count=$this->count();
-		$identifier=$this->identifier . "-" . $count;
+		$identifier=$this->identifier."-".$count;
 		if (isset($masterItem)) {
-			$masterO=new HtmlFormCheckbox("master-" . $identifier, $masterItem);
+			$masterO=new HtmlFormCheckbox("master-".$identifier, $masterItem);
 			$masterO->getField()->addToProperty("class", "master");
 			$masterO->setClass("item");
 			$this->addItem($masterO);
 		}
-		$fields=array ();
+		$fields=array();
 		$i=0;
-		foreach ( $items as $val => $caption ) {
-			$itemO=new HtmlFormCheckbox($identifier . "-" . $i++, $caption, $val, "child");
-			if (\array_search($val, $values) !== false) {
+		foreach ($items as $val => $caption) {
+			$itemO=new HtmlFormCheckbox($identifier."-".$i++, $caption, $val, "child");
+			if (\array_search($val, $values)!==false) {
 				$itemO->getField()->getField()->setProperty("checked", "");
 			}
 			$itemO->setClass("item");
 			$fields[]=$itemO;
 		}
-		if (isset($masterO) === true) {
+		if (isset($masterO)===true) {
 			$list=new HtmlList("", $fields);
 			$list->setClass("list");
 			$masterO->addContent($list);
-		} else {
+		}else {
 			$this->addList($fields);
 		}
 		$this->_hasCheckedList=true;

Doc Comments +6 added lines

@@ -10,6 +10,9 @@ 
 class HtmlList extends HtmlSemCollection {
 	protected $_hasCheckedList;
 
+	/**
+	 * @param string $identifier
+	 */
 	public function __construct($identifier, $items=array()) {
 		parent::__construct($identifier, "div", "ui list");
 		$this->addItems($items);
@@ -22,6 +25,9 @@ 
 		return $item;
 	}
 
+	/**
+	 * @param integer $niveau
+	 */
 	public function addHeader($niveau, $content) {
 		$header=new HtmlHeader("header-" . $this->identifier, $niveau, $content, "page");
 		$this->wrap($header);

Ajax/semantic/html/elements/HtmlInput.php

Spacing +16 added lines, -16 removed lines

@@ -13,10 +13,10 @@ 
 	use IconTrait;
 
 	public function __construct($identifier, $type="text", $value="", $placeholder="") {
-		parent::__construct("div-" . $identifier, "div", "ui input");
-		$this->content=[ "field" => new \Ajax\common\html\html5\HtmlInput($identifier, $type, $value, $placeholder) ];
-		$this->_states=[ State::DISABLED,State::FOCUS,State::ERROR ];
-		$this->_variations=[ Variation::TRANSPARENT ];
+		parent::__construct("div-".$identifier, "div", "ui input");
+		$this->content=["field" => new \Ajax\common\html\html5\HtmlInput($identifier, $type, $value, $placeholder)];
+		$this->_states=[State::DISABLED, State::FOCUS, State::ERROR];
+		$this->_variations=[Variation::TRANSPARENT];
 	}
 
 	public function setFocus() {
@@ -24,38 +24,38 @@ 
 	}
 
 	public function addLoading() {
-		if ($this->_hasIcon === false) {
+		if ($this->_hasIcon===false) {
 			throw new \Exception("Input must have an icon for showing a loader, use addIcon before");
 		}
 		return $this->addToProperty("class", State::LOADING);
 	}
 
 	public function labeled($label, $direction=Direction::LEFT, $icon=NULL) {
-		$labelO=$this->addLabel($label,$direction===Direction::LEFT,$icon);
-		$this->addToProperty("class", $direction . " labeled");
+		$labelO=$this->addLabel($label, $direction===Direction::LEFT, $icon);
+		$this->addToProperty("class", $direction." labeled");
 		return $labelO;
 	}
 
 	public function labeledToCorner($icon, $direction=Direction::LEFT) {
-		return $this->labeled("", $direction . " corner", $icon)->toCorner($direction);
+		return $this->labeled("", $direction." corner", $icon)->toCorner($direction);
 	}
 
 	public function addAction($action, $direction=Direction::RIGHT, $icon=NULL, $labeled=false) {
 		$actionO=$action;
-		if (\is_object($action) === false) {
-			$actionO=new HtmlButton("action-" . $this->identifier, $action);
+		if (\is_object($action)===false) {
+			$actionO=new HtmlButton("action-".$this->identifier, $action);
 			if (isset($icon))
 				$actionO->addIcon($icon, true, $labeled);
 		}
-		$this->addToProperty("class", $direction . " action");
-		$this->addContent($actionO, \strstr($direction, Direction::LEFT) !== false);
+		$this->addToProperty("class", $direction." action");
+		$this->addContent($actionO, \strstr($direction, Direction::LEFT)!==false);
 		return $actionO;
 	}
 
-	public function addDropdown($label="", $items=array(),$direction=Direction::RIGHT){
-		$labelO=new HtmlDropdown("dd-".$this->identifier,$label,$items);
-		$labelO->asSelect("select-".$this->identifier,false,true);
-		return $this->labeled($labelO,$direction);
+	public function addDropdown($label="", $items=array(), $direction=Direction::RIGHT) {
+		$labelO=new HtmlDropdown("dd-".$this->identifier, $label, $items);
+		$labelO->asSelect("select-".$this->identifier, false, true);
+		return $this->labeled($labelO, $direction);
 	}
 
 	public function getField() {

Braces +3 added lines, -2 removed lines

@@ -44,8 +44,9 @@
 		$actionO=$action;
 		if (\is_object($action) === false) {
 			$actionO=new HtmlButton("action-" . $this->identifier, $action);
-			if (isset($icon))
-				$actionO->addIcon($icon, true, $labeled);
+			if (isset($icon)) {
+							$actionO->addIcon($icon, true, $labeled);
+			}
 		}
 		$this->addToProperty("class", $direction . " action");
 		$this->addContent($actionO, \strstr($direction, Direction::LEFT) !== false);