phpMv / phpMv-UI

Ajax/semantic/widgets/datatable/DataTable.php

Braces +6 added lines, -4 removed lines

@@ -68,8 +68,9 @@ 
 
 		$table->setRowCount(0, \sizeof($captions));
 		$table->setHeaderValues($captions);
-		if(isset($this->_compileParts))
-			$table->setCompileParts($this->_compileParts);
+		if(isset($this->_compileParts)) {
+					$table->setCompileParts($this->_compileParts);
+		}
 		if(isset($this->_searchField) && isset($js)){
 			$this->_searchField->postOn("change", $this->_urls,"{'s':$(this).val()}","-#".$this->identifier." tbody",["preventDefault"=>false]);
 		}
@@ -275,8 +276,9 @@ 
 	private function getDefaultButton($icon,$class=null){
 		$bt=$this->getFieldButton("");
 		$bt->asIcon($icon);
-		if(isset($class))
-			$bt->addToProperty("class", $class);
+		if(isset($class)) {
+					$bt->addToProperty("class", $class);
+		}
 		return $bt;
 	}
 

Doc Comments +18 added lines, -3 removed lines

@@ -40,6 +40,10 @@ 
 		parent::run($js);
 	}
 
+	/**
+	 * @param string $identifier
+	 * @param string $model
+	 */
 	public function __construct($identifier,$model,$modelInstance=NULL) {
 		parent::__construct($identifier, $model,$modelInstance);
 		$this->_init(new InstanceViewer($identifier), "table", new HtmlTable($identifier, 0,0), false);
@@ -151,6 +155,9 @@ 
 		return $this;
 	}
 
+	/**
+	 * @param PositionInTable $part
+	 */
 	private function addToolbarRow($part,$table,$captions){
 		$row=$table->getPart($part)->addRow(\sizeof($captions));
 		$row->mergeCol();
@@ -190,7 +197,7 @@ 
 	/**
 	 * @param string $caption
 	 * @param callable $callback
-	 * @return callable
+	 * @return \Closure
 	 */
 	private function getFieldButtonCallable($caption,$callback=null){
 		return $this->getCallable("getFieldButton",[$caption],$callback);
@@ -198,9 +205,9 @@ 
 
 	/**
 	 * @param callable $thisCallback
-	 * @param array $parameters
+	 * @param string[] $parameters
 	 * @param callable $callback
-	 * @return callable
+	 * @return \Closure
 	 */
 	private function getCallable($thisCallback,$parameters,$callback=null){
 		$result=function($instance) use($thisCallback,$parameters,$callback){
@@ -261,11 +268,19 @@ 
 		return $this;
 	}
 
+	/**
+	 * @param string $icon
+	 * @param string $class
+	 */
 	private function addDefaultButton($icon,$class=null,$callback=null){
 		$this->addField($this->getCallable("getDefaultButton",[$icon,$class],$callback));
 		return $this;
 	}
 
+	/**
+	 * @param string $icon
+	 * @param string $class
+	 */
 	private function insertDefaultButtonIn($index,$icon,$class=null,$callback=null){
 		$this->insertInField($index,$this->getCallable("getDefaultButton",[$icon,$class],$callback));
 		return $this;

Spacing +78 added lines, -78 removed lines

@@ -30,8 +30,8 @@ 
 	protected $_hasCheckboxes;
 	protected $_compileParts;
 
-	public function run(JsUtils $js){
-		if($this->_hasCheckboxes && isset($js)){
+	public function run(JsUtils $js) {
+		if ($this->_hasCheckboxes && isset($js)) {
 			$js->execOn("change", "#".$this->identifier." [name='selection[]']", "
 		var \$parentCheckbox=\$('#ck-main-ck-{$this->identifier}'),\$checkbox=\$('#{$this->identifier} [name=\"selection[]\"]'),allChecked=true,allUnchecked=true;
 		\$checkbox.each(function() {if($(this).prop('checked')){allUnchecked = false;}else{allChecked = false;}});
@@ -40,9 +40,9 @@ 
 		parent::run($js);
 	}
 
-	public function __construct($identifier,$model,$modelInstance=NULL) {
-		parent::__construct($identifier, $model,$modelInstance);
-		$this->_init(new InstanceViewer($identifier), "table", new HtmlTable($identifier, 0,0), false);
+	public function __construct($identifier, $model, $modelInstance=NULL) {
+		parent::__construct($identifier, $model, $modelInstance);
+		$this->_init(new InstanceViewer($identifier), "table", new HtmlTable($identifier, 0, 0), false);
 	}
 
 	/**
@@ -54,61 +54,61 @@ 
 	}
 
 
-	public function compile(JsUtils $js=NULL,&$view=NULL){
+	public function compile(JsUtils $js=NULL, &$view=NULL) {
 		$this->_instanceViewer->setInstance($this->_model);
 		$captions=$this->_instanceViewer->getCaptions();
 
 		$table=$this->content["table"];
 
-		if($this->_hasCheckboxes){
+		if ($this->_hasCheckboxes) {
 			$this->_generateMainCheckbox($captions);
 		}
 
 		$table->setRowCount(0, \sizeof($captions));
 		$table->setHeaderValues($captions);
-		if(isset($this->_compileParts))
+		if (isset($this->_compileParts))
 			$table->setCompileParts($this->_compileParts);
-		if(isset($this->_searchField) && isset($js)){
-			$this->_searchField->postOn("change", $this->_urls,"{'s':$(this).val()}","-#".$this->identifier." tbody",["preventDefault"=>false]);
+		if (isset($this->_searchField) && isset($js)) {
+			$this->_searchField->postOn("change", $this->_urls, "{'s':$(this).val()}", "-#".$this->identifier." tbody", ["preventDefault"=>false]);
 		}
 
 		$this->_generateContent($table);
 
-		if($this->_hasCheckboxes && $table->hasPart("thead")){
-				$table->getHeader()->getCell(0, 0)->addToProperty("class","no-sort");
+		if ($this->_hasCheckboxes && $table->hasPart("thead")) {
+				$table->getHeader()->getCell(0, 0)->addToProperty("class", "no-sort");
 		}
 
-		if(isset($this->_pagination) && $this->_pagination->getVisible()){
+		if (isset($this->_pagination) && $this->_pagination->getVisible()) {
 			$this->_generatePagination($table);
 		}
-		if(isset($this->_toolbar)){
+		if (isset($this->_toolbar)) {
 			$this->_setToolbarPosition($table, $captions);
 		}
-		$this->content=JArray::sortAssociative($this->content, [PositionInTable::BEFORETABLE,"table",PositionInTable::AFTERTABLE]);
-		return parent::compile($js,$view);
+		$this->content=JArray::sortAssociative($this->content, [PositionInTable::BEFORETABLE, "table", PositionInTable::AFTERTABLE]);
+		return parent::compile($js, $view);
 	}
 
-	private function _generateMainCheckbox(&$captions){
-		$ck=new HtmlCheckbox("main-ck-".$this->identifier,"");
+	private function _generateMainCheckbox(&$captions) {
+		$ck=new HtmlCheckbox("main-ck-".$this->identifier, "");
 		$ck->setOnChecked("$('#".$this->identifier." [name=%quote%selection[]%quote%]').prop('checked',true);");
 		$ck->setOnUnchecked("$('#".$this->identifier." [name=%quote%selection[]%quote%]').prop('checked',false);");
 		\array_unshift($captions, $ck);
 	}
 
-	protected function _generateContent($table){
+	protected function _generateContent($table) {
 		$objects=$this->_modelInstance;
-		if(isset($this->_pagination)){
+		if (isset($this->_pagination)) {
 			$objects=$this->_pagination->getObjects($this->_modelInstance);
 		}
 		InstanceViewer::setIndex(0);
-		$table->fromDatabaseObjects($objects, function($instance){
+		$table->fromDatabaseObjects($objects, function($instance) {
 			$this->_instanceViewer->setInstance($instance);
 			InstanceViewer::$index++;
-			$result= $this->_instanceViewer->getValues();
-			if($this->_hasCheckboxes){
-				$ck=new HtmlCheckbox("ck-".$this->identifier,"");
+			$result=$this->_instanceViewer->getValues();
+			if ($this->_hasCheckboxes) {
+				$ck=new HtmlCheckbox("ck-".$this->identifier, "");
 				$field=$ck->getField();
-				$field->setProperty("value",$this->_instanceViewer->getIdentifier());
+				$field->setProperty("value", $this->_instanceViewer->getIdentifier());
 				$field->setProperty("name", "selection[]");
 				\array_unshift($result, $ck);
 			}
@@ -116,28 +116,28 @@ 
 		});
 	}
 
-	private function _generatePagination($table){
+	private function _generatePagination($table) {
 		$footer=$table->getFooter();
 		$footer->mergeCol();
-		$menu=new HtmlPaginationMenu("pagination-".$this->identifier,$this->_pagination->getPagesNumbers());
+		$menu=new HtmlPaginationMenu("pagination-".$this->identifier, $this->_pagination->getPagesNumbers());
 		$menu->floatRight();
 		$menu->setActiveItem($this->_pagination->getPage()-1);
 		$footer->setValues($menu);
-		$menu->postOnClick($this->_urls,"{'p':$(this).attr('data-page')}","-#".$this->identifier." tbody",["preventDefault"=>false]);
+		$menu->postOnClick($this->_urls, "{'p':$(this).attr('data-page')}", "-#".$this->identifier." tbody", ["preventDefault"=>false]);
 	}
 
-	protected function _setToolbarPosition($table,$captions=NULL){
-		switch ($this->_toolbarPosition){
+	protected function _setToolbarPosition($table, $captions=NULL) {
+		switch ($this->_toolbarPosition) {
 			case PositionInTable::BEFORETABLE:
 			case PositionInTable::AFTERTABLE:
-				if(isset($this->_compileParts)===false){
+				if (isset($this->_compileParts)===false) {
 					$this->content[$this->_toolbarPosition]=$this->_toolbar;
 				}
 				break;
 			case PositionInTable::HEADER:
 			case PositionInTable::FOOTER:
 			case PositionInTable::BODY:
-				$this->addToolbarRow($this->_toolbarPosition,$table, $captions);
+				$this->addToolbarRow($this->_toolbarPosition, $table, $captions);
 				break;
 		}
 	}
@@ -149,18 +149,18 @@ 
 	 * @param callable $callback function called after the field compilation
 	 * @return \Ajax\semantic\widgets\datatable\DataTable
 	 */
-	public function afterCompile($index,$callback){
-		$this->_instanceViewer->afterCompile($index,$callback);
+	public function afterCompile($index, $callback) {
+		$this->_instanceViewer->afterCompile($index, $callback);
 		return $this;
 	}
 
-	private function addToolbarRow($part,$table,$captions){
+	private function addToolbarRow($part, $table, $captions) {
 		$row=$table->getPart($part)->addRow(\sizeof($captions));
 		$row->mergeCol();
 		$row->setValues([$this->_toolbar]);
 	}
 
-	public function getHtmlComponent(){
+	public function getHtmlComponent() {
 		return $this->content["table"];
 	}
 
@@ -173,8 +173,8 @@ 
 		return $this;
 	}
 
-	public function paginate($items_per_page=10,$page=1){
-		$this->_pagination=new Pagination($items_per_page,4,$page);
+	public function paginate($items_per_page=10, $page=1) {
+		$this->_pagination=new Pagination($items_per_page, 4, $page);
 	}
 
 	public function getHasCheckboxes() {
@@ -186,7 +186,7 @@ 
 		return $this;
 	}
 
-	public function refresh($compileParts=["tbody"]){
+	public function refresh($compileParts=["tbody"]) {
 		$this->_compileParts=$compileParts;
 		return $this;
 	}
@@ -195,8 +195,8 @@ 
 	 * @param callable $callback
 	 * @return callable
 	 */
-	private function getFieldButtonCallable($caption,$callback=null){
-		return $this->getCallable("getFieldButton",[$caption],$callback);
+	private function getFieldButtonCallable($caption, $callback=null) {
+		return $this->getCallable("getFieldButton", [$caption], $callback);
 	}
 
 	/**
@@ -205,16 +205,16 @@ 
 	 * @param callable $callback
 	 * @return callable
 	 */
-	private function getCallable($thisCallback,$parameters,$callback=null){
-		$result=function($instance) use($thisCallback,$parameters,$callback){
-			$object=call_user_func_array(array($this,$thisCallback), $parameters);
-			if(isset($callback)){
-				if(\is_callable($callback)){
-					$callback($object,$instance);
+	private function getCallable($thisCallback, $parameters, $callback=null) {
+		$result=function($instance) use($thisCallback, $parameters, $callback){
+			$object=call_user_func_array(array($this, $thisCallback), $parameters);
+			if (isset($callback)) {
+				if (\is_callable($callback)) {
+					$callback($object, $instance);
 				}
 			}
-			if($object instanceof HtmlSemDoubleElement){
-				$object->setProperty("data-ajax",$this->_instanceViewer->getIdentifier());
+			if ($object instanceof HtmlSemDoubleElement) {
+				$object->setProperty("data-ajax", $this->_instanceViewer->getIdentifier());
 			}
 			return $object;
 		};
@@ -225,8 +225,8 @@ 
 	 * @param string $caption
 	 * @return HtmlButton
 	 */
-	private function getFieldButton($caption){
-		return new HtmlButton("",$caption);
+	private function getFieldButton($caption) {
+		return new HtmlButton("", $caption);
 	}
 
 	/**
@@ -235,8 +235,8 @@ 
 	 * @param callable $callback
 	 * @return \Ajax\semantic\widgets\datatable\DataTable
 	 */
-	public function addFieldButton($caption,$callback=null){
-		$this->addField($this->getCallable("getFieldButton",[$caption],$callback));
+	public function addFieldButton($caption, $callback=null) {
+		$this->addField($this->getCallable("getFieldButton", [$caption], $callback));
 		return $this;
 	}
 
@@ -247,8 +247,8 @@ 
 	 * @param callable $callback
 	 * @return \Ajax\semantic\widgets\datatable\DataTable
 	 */
-	public function insertFieldButton($index,$caption,$callback=null){
-		$this->insertField($index, $this->getFieldButtonCallable($caption,$callback));
+	public function insertFieldButton($index, $caption, $callback=null) {
+		$this->insertField($index, $this->getFieldButtonCallable($caption, $callback));
 		return $this;
 	}
 
@@ -259,60 +259,60 @@ 
 	 * @param callable $callback
 	 * @return \Ajax\semantic\widgets\datatable\DataTable
 	 */
-	public function insertInFieldButton($index,$caption,$callback=null){
-		$this->insertInField($index, $this->getFieldButtonCallable($caption,$callback));
+	public function insertInFieldButton($index, $caption, $callback=null) {
+		$this->insertInField($index, $this->getFieldButtonCallable($caption, $callback));
 		return $this;
 	}
 
-	private function addDefaultButton($icon,$class=null,$callback=null){
-		$this->addField($this->getCallable("getDefaultButton",[$icon,$class],$callback));
+	private function addDefaultButton($icon, $class=null, $callback=null) {
+		$this->addField($this->getCallable("getDefaultButton", [$icon, $class], $callback));
 		return $this;
 	}
 
-	private function insertDefaultButtonIn($index,$icon,$class=null,$callback=null){
-		$this->insertInField($index,$this->getCallable("getDefaultButton",[$icon,$class],$callback));
+	private function insertDefaultButtonIn($index, $icon, $class=null, $callback=null) {
+		$this->insertInField($index, $this->getCallable("getDefaultButton", [$icon, $class], $callback));
 		return $this;
 	}
 
-	private function getDefaultButton($icon,$class=null){
+	private function getDefaultButton($icon, $class=null) {
 		$bt=$this->getFieldButton("");
 		$bt->asIcon($icon);
-		if(isset($class))
+		if (isset($class))
 			$bt->addToProperty("class", $class);
 		return $bt;
 	}
 
-	public function addDeleteButton($callback=null){
-		return $this->addDefaultButton("remove","delete red basic",$callback);
+	public function addDeleteButton($callback=null) {
+		return $this->addDefaultButton("remove", "delete red basic", $callback);
 	}
 
-	public function addEditButton($callback=null){
-		return $this->addDefaultButton("edit","edit basic",$callback);
+	public function addEditButton($callback=null) {
+		return $this->addDefaultButton("edit", "edit basic", $callback);
 	}
 
-	public function addEditDeleteButtons($callbackEdit=null,$callbackDelete=null){
+	public function addEditDeleteButtons($callbackEdit=null, $callbackDelete=null) {
 		$this->addEditButton($callbackEdit);
 		$index=$this->_instanceViewer->visiblePropertiesCount()-1;
-		$this->insertDeleteButtonIn($index,$callbackDelete);
+		$this->insertDeleteButtonIn($index, $callbackDelete);
 		return $this;
 	}
 
-	public function insertDeleteButtonIn($index,$callback=null){
-		return $this->insertDefaultButtonIn($index,"remove","delete red basic",$callback);
+	public function insertDeleteButtonIn($index, $callback=null) {
+		return $this->insertDefaultButtonIn($index, "remove", "delete red basic", $callback);
 	}
 
-	public function insertEditButtonIn($index,$callback=null){
-		return $this->insertDefaultButtonIn($index,"edit","edit basic",$callback);
+	public function insertEditButtonIn($index, $callback=null) {
+		return $this->insertDefaultButtonIn($index, "edit", "edit basic", $callback);
 	}
 
-	public function addSearchInToolbar($position=Direction::RIGHT){
+	public function addSearchInToolbar($position=Direction::RIGHT) {
 		return $this->addInToolbar($this->getSearchField())->setPosition($position);
 	}
 
-	public function getSearchField(){
-		if(isset($this->_searchField)===false){
-			$this->_searchField=new HtmlInput("search-".$this->identifier,"search","","Search...");
-			$this->_searchField->addIcon("search",Direction::RIGHT);
+	public function getSearchField() {
+		if (isset($this->_searchField)===false) {
+			$this->_searchField=new HtmlInput("search-".$this->identifier, "search", "", "Search...");
+			$this->_searchField->addIcon("search", Direction::RIGHT);
 		}
 		return $this->_searchField;
 	}

Ajax/common/html/traits/BaseHtmlEventsTrait.php

Braces +4 added lines, -3 removed lines

@@ -52,7 +52,7 @@ 
 	public function onCreate($jsCode){
 		if(isset($this->_events["_create"])){
 			$this->_events["_create"][]=$jsCode;
-		}else{
+		} else{
 			$this->_events["_create"]=[$jsCode];
 		}
 		return $this;
@@ -87,8 +87,9 @@ 
 			if(\is_array($create)){
 				$create=\implode("", $create);
 			}
-			if(isset($js) && $create!=="")
-				$js->exec($create,true);
+			if(isset($js) && $create!=="") {
+							$js->exec($create,true);
+			}
 			unset($this->_events["_create"]);
 		}
 	}

Spacing +22 added lines, -22 removed lines

@@ -11,9 +11,9 @@ 
  * @property SimpleExtComponent $_bsComponent
  * @property string identifier
  */
-trait BaseHtmlEventsTrait{
+trait BaseHtmlEventsTrait {
 
-	protected $_events=array ();
+	protected $_events=array();
 
 	/**
 	 * @param string $event
@@ -23,11 +23,11 @@ 
 	 * @return \Ajax\common\html\BaseHtml
 	 */
 	public function addEvent($event, $jsCode, $stopPropagation=false, $preventDefault=false) {
-		if ($stopPropagation === true) {
-			$jsCode="event.stopPropagation();" . $jsCode;
+		if ($stopPropagation===true) {
+			$jsCode="event.stopPropagation();".$jsCode;
 		}
-		if ($preventDefault === true) {
-			$jsCode="event.preventDefault();" . $jsCode;
+		if ($preventDefault===true) {
+			$jsCode="event.preventDefault();".$jsCode;
 		}
 		return $this->_addEvent($event, $jsCode);
 	}
@@ -42,7 +42,7 @@ 
 			if (\is_array($this->_events[$event])) {
 				$this->_events[$event][]=$jsCode;
 			} else {
-				$this->_events[$event]=array ($this->_events[$event],$jsCode );
+				$this->_events[$event]=array($this->_events[$event], $jsCode);
 			}
 		} else {
 			$this->_events[$event]=$jsCode;
@@ -69,10 +69,10 @@ 
 		return $this->onClick($jsCode);
 	}
 
-	public function onCreate($jsCode){
-		if(isset($this->_events["_create"])){
+	public function onCreate($jsCode) {
+		if (isset($this->_events["_create"])) {
 			$this->_events["_create"][]=$jsCode;
-		}else{
+		} else {
 			$this->_events["_create"]=[$jsCode];
 		}
 		return $this;
@@ -81,15 +81,15 @@ 
 	public function addEventsOnRun(JsUtils $js=NULL) {
 		$this->_eventsOnCreate($js);
 		if (isset($this->_bsComponent)) {
-			foreach ( $this->_events as $event => $jsCode ) {
+			foreach ($this->_events as $event => $jsCode) {
 				$code=$jsCode;
 				if (\is_array($jsCode)) {
 					$code="";
-					foreach ( $jsCode as $jsC ) {
+					foreach ($jsCode as $jsC) {
 						if ($jsC instanceof AjaxCall) {
-							$code.="\n" . $jsC->compile($js);
+							$code.="\n".$jsC->compile($js);
 						} else {
-							$code.="\n" . $jsC;
+							$code.="\n".$jsC;
 						}
 					}
 				} elseif ($jsCode instanceof AjaxCall) {
@@ -97,18 +97,18 @@ 
 				}
 				$this->_bsComponent->addEvent($event, $code);
 			}
-			$this->_events=array ();
+			$this->_events=array();
 		}
 	}
 
-	protected function _eventsOnCreate(JsUtils $js=NULL){
-		if(isset($this->_events["_create"])){
+	protected function _eventsOnCreate(JsUtils $js=NULL) {
+		if (isset($this->_events["_create"])) {
 			$create=$this->_events["_create"];
-			if(\is_array($create)){
+			if (\is_array($create)) {
 				$create=\implode("", $create);
 			}
-			if(isset($js) && $create!=="")
-				$js->exec($create,true);
+			if (isset($js) && $create!=="")
+				$js->exec($create, true);
 			unset($this->_events["_create"]);
 		}
 	}
@@ -122,7 +122,7 @@ 
 	 * @return BaseHtml
 	 */
 	public function _ajaxOn($operation, $event, $url, $responseElement="", $parameters=array()) {
-		$params=array ("url" => $url,"responseElement" => $responseElement );
+		$params=array("url" => $url, "responseElement" => $responseElement);
 		$params=array_merge($params, $parameters);
 		$this->_addEvent($event, new AjaxCall($operation, $params));
 		return $this;
@@ -155,7 +155,7 @@ 
 	}
 
 	public function jsDoJquery($jqueryCall, $param="") {
-		return "$('#" . $this->identifier . "')." . $jqueryCall . "(" . Javascript::prep_value($param) . ");";
+		return "$('#".$this->identifier."').".$jqueryCall."(".Javascript::prep_value($param).");";
 	}
 
 	public function executeOnRun($jsCode) {

Ajax/common/html/html5/HtmlLinkTrait.php

Spacing +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
 
 trait HtmlLinkTrait {
 
-	abstract public function setProperty($name,$value);
+	abstract public function setProperty($name, $value);
 	abstract public function getProperty($name);
 
 	public function setHref($value) {

Ajax/semantic/html/base/traits/AttachedTrait.php

Spacing +2 added lines, -2 removed lines

@@ -14,8 +14,8 @@
 	 */
 	public function setAttachment($toElement, $side=Side::BOTH) {
 		if (isset($toElement)) {
-			$toElement->addToPropertyCtrl("class", "attached", array ("attached" ));
+			$toElement->addToPropertyCtrl("class", "attached", array("attached"));
 		}
-		return $this->addToPropertyCtrl("class", $side . " attached", Side::getConstantValues("attached"));
+		return $this->addToPropertyCtrl("class", $side." attached", Side::getConstantValues("attached"));
 	}
 }
\ No newline at end of file

Ajax/semantic/html/base/traits/LabeledIconTrait.php

Spacing +6 added lines, -6 removed lines

@@ -13,7 +13,7 @@ 
 trait LabeledIconTrait {
 
 	abstract public function addToProperty($name, $value, $separator=" ");
-	abstract public function addContent($content,$before=false);
+	abstract public function addContent($content, $before=false);
 
 	/**
 	 * Adds an icon before or after
@@ -22,17 +22,17 @@ 
 	 * @param boolean $labeled
 	 * @return \Ajax\semantic\html\elements\HtmlIcon
 	 */
-	public function addIcon($icon,$before=true,$labeled=false){
+	public function addIcon($icon, $before=true, $labeled=false) {
 		$iconO=$icon;
-		if(\is_string($icon)){
+		if (\is_string($icon)) {
 			$iconO=new HtmlIcon("icon-".$this->identifier, $icon);
 		}
-		if($labeled!==false){
-			$direction=($before===true)?Direction::LEFT:Direction::RIGHT;
+		if ($labeled!==false) {
+			$direction=($before===true) ? Direction::LEFT : Direction::RIGHT;
 			$this->addToProperty("class", $direction." labeled icon");
 			$this->tagName="div";
 		}
-		$this->addContent($iconO,$before);
+		$this->addContent($iconO, $before);
 		return $iconO;
 	}
 }
\ No newline at end of file

Ajax/semantic/html/elements/html5/HtmlImg.php

Braces +3 added lines, -2 removed lines

@@ -14,8 +14,9 @@
 	}
 
 	public function asAvatar($caption=NULL) {
-		if (isset($caption))
-			$this->wrap("", $caption);
+		if (isset($caption)) {
+					$this->wrap("", $caption);
+		}
 			return $this->addToProperty("class", "avatar");
 	}
 

Doc Comments +3 added lines

@@ -7,6 +7,9 @@
 class HtmlImg extends \Ajax\common\html\html5\HtmlImg {
 	use BaseTrait;
 
+	/**
+	 * @param string $identifier
+	 */
 	public function __construct($identifier, $src="", $alt="") {
 		parent::__construct($identifier, $src, $alt);
 		$this->_baseClass="ui image";

Ajax/semantic/html/modules/HtmlDimmer.php

Spacing +5 added lines, -5 removed lines

@@ -18,14 +18,14 @@ 
 	}
 
 	public function setContent($content) {
-		$this->content=new HtmlSemDoubleElement("content-" . $this->identifier, "div", "content", new HtmlSemDoubleElement("", "div", "center", $content));
+		$this->content=new HtmlSemDoubleElement("content-".$this->identifier, "div", "content", new HtmlSemDoubleElement("", "div", "center", $content));
 		return $this;
 	}
 
 	public function asIcon($icon, $title, $subHeader=NULL) {
-		$header=new HtmlHeader("header-" . $this->identifier);
+		$header=new HtmlHeader("header-".$this->identifier);
 		$header->asIcon($icon, $title, $subHeader);
-		if ($this->_inverted === false)
+		if ($this->_inverted===false)
 			$header->setInverted();
 		return $this->setContent($header);
 	}
@@ -42,13 +42,13 @@ 
 
 	public function run(JsUtils $js) {
 		if ($this->_container instanceof HtmlSingleElement)
-			$this->_bsComponent=$js->semantic()->dimmer("#" . $this->_container->getIdentifier(), $this->_params);
+			$this->_bsComponent=$js->semantic()->dimmer("#".$this->_container->getIdentifier(), $this->_params);
 		return parent::run($js);
 	}
 
 	public function jsShow() {
 		if (isset($this->_container))
-			return '$("#.' . $this->_container->getIdentifier() . ').dimmer("show");';
+			return '$("#.'.$this->_container->getIdentifier().').dimmer("show");';
 	}
 
 	public function setBlurring() {

Braces +9 added lines, -6 removed lines

@@ -25,8 +25,9 @@ 
 	public function asIcon($icon, $title, $subHeader=NULL) {
 		$header=new HtmlHeader("header-" . $this->identifier);
 		$header->asIcon($icon, $title, $subHeader);
-		if ($this->_inverted === false)
-			$header->setInverted();
+		if ($this->_inverted === false) {
+					$header->setInverted();
+		}
 		return $this->setContent($header);
 	}
 
@@ -41,14 +42,16 @@ 
 	}
 
 	public function run(JsUtils $js) {
-		if ($this->_container instanceof HtmlSingleElement)
-			$this->_bsComponent=$js->semantic()->dimmer("#" . $this->_container->getIdentifier(), $this->_params);
+		if ($this->_container instanceof HtmlSingleElement) {
+					$this->_bsComponent=$js->semantic()->dimmer("#" . $this->_container->getIdentifier(), $this->_params);
+		}
 		return parent::run($js);
 	}
 
 	public function jsShow() {
-		if (isset($this->_container))
-			return '$("#.' . $this->_container->getIdentifier() . ').dimmer("show");';
+		if (isset($this->_container)) {
+					return '$("#.' . $this->_container->getIdentifier() . ').dimmer("show");';
+		}
 	}
 
 	public function setBlurring() {

Ajax/semantic/html/modules/HtmlSearch.php

Spacing +11 added lines, -11 removed lines

@@ -8,12 +8,12 @@ 
 use Ajax\semantic\html\base\constants\Direction;
 
 class HtmlSearch extends HtmlSemDoubleElement {
-	private $_elements=array ();
-	private $_searchFields=array ("title" );
+	private $_elements=array();
+	private $_searchFields=array("title");
 	private $_local=false;
 
 	public function __construct($identifier, $placeholder=NULL, $icon=NULL) {
-		parent::__construct("search-" . $identifier, "div", "ui search", array ());
+		parent::__construct("search-".$identifier, "div", "ui search", array());
 		$this->createField($placeholder, $icon);
 		$this->createResult();
 		$this->_params["type"]="standard";
@@ -32,7 +32,7 @@ 
 	}
 
 	private function createResult() {
-		$this->content["result"]=new HtmlSemDoubleElement("results-" . $this->identifier, "div", "results");
+		$this->content["result"]=new HtmlSemDoubleElement("results-".$this->identifier, "div", "results");
 		return $this->content["result"];
 	}
 
@@ -49,7 +49,7 @@ 
 	}
 
 	public function setUrl($url) {
-		$this->_params["apiSettings"]="%{url: %quote%" . $url . "%quote%}%";
+		$this->_params["apiSettings"]="%{url: %quote%".$url."%quote%}%";
 		return $this;
 	}
 
@@ -72,16 +72,16 @@ 
 	}
 
 	public function run(JsUtils $js) {
-		$this->_params["onSelect"]='%function(result,response){$(%quote%#' . $this->identifier . '%quote%).trigger(%quote%onSelect%quote%, {%quote%result%quote%: result, %quote%response%quote%:response} );}%';
+		$this->_params["onSelect"]='%function(result,response){$(%quote%#'.$this->identifier.'%quote%).trigger(%quote%onSelect%quote%, {%quote%result%quote%: result, %quote%response%quote%:response} );}%';
 		$searchFields=\json_encode($this->_searchFields);
 		$searchFields=str_ireplace("\"", "%quote%", $searchFields);
-		$this->_params["searchFields"]="%" . $searchFields . "%";
-		if ($this->_local === true) {
+		$this->_params["searchFields"]="%".$searchFields."%";
+		if ($this->_local===true) {
 			$this->_params["source"]="%content%";
-			$this->addEvent("beforeExecute", "var content=" . $this->resultsToJson() . ";");
+			$this->addEvent("beforeExecute", "var content=".$this->resultsToJson().";");
 		}
-		if (isset($this->_bsComponent) === false) {
-			$this->_bsComponent=$js->semantic()->search("#" . $this->identifier, $this->_params);
+		if (isset($this->_bsComponent)===false) {
+			$this->_bsComponent=$js->semantic()->search("#".$this->identifier, $this->_params);
 		}
 		$this->addEventsOnRun($js);
 		return $this->_bsComponent;

Braces +6 added lines, -4 removed lines

@@ -21,10 +21,12 @@
 
 	private function createField($placeholder=NULL, $icon=NULL) {
 		$field=new HtmlInput($this->identifier);
-		if (isset($placeholder))
-			$field->setPlaceholder($placeholder);
-		if (isset($icon))
-			$field->addIcon($icon, Direction::RIGHT);
+		if (isset($placeholder)) {
+					$field->setPlaceholder($placeholder);
+		}
+		if (isset($icon)) {
+					$field->addIcon($icon, Direction::RIGHT);
+		}
 		//TODO check getField
 		$field->getDataField()->setClass("prompt");
 		$this->content["field"]=$field;

Ajax/semantic/html/modules/HtmlProgress.php

Braces +9 added lines, -6 removed lines

@@ -12,11 +12,13 @@ 
 
 	public function __construct($identifier, $value=NULL, $label=NULL, $attributes=array()) {
 		parent::__construct($identifier, "div", "ui progress");
-		if (isset($value))
-			$this->setProperty("data-percent", $value);
+		if (isset($value)) {
+					$this->setProperty("data-percent", $value);
+		}
 		$this->createBar();
-		if (isset($label))
-			$this->setLabel($label);
+		if (isset($label)) {
+					$this->setLabel($label);
+		}
 		$this->_states=[ State::SUCCESS,State::WARNING,State::ERROR,State::ACTIVE,State::DISABLED ];
 		$this->addToProperty("class", $attributes);
 	}
@@ -111,8 +113,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;
 	}

Spacing +11 added lines, -11 removed lines

@@ -17,17 +17,17 @@ 
 		$this->createBar();
 		if (isset($label))
 			$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;
 	}
@@ -63,20 +63,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");';
 	}
 
 	/**
@@ -98,12 +98,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."}%";
 	}
 
 	/*
@@ -111,8 +111,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;
 	}