phpMv / phpMv-UI

Ajax/semantic/widgets/datatable/DataTableFieldAsTrait.php

Doc Comments +34 added lines, -9 removed lines

@@ -15,15 +15,28 @@ 
  * @property InstanceViewer $_instanceViewer
  */
 trait DataTableFieldAsTrait{
+
+	/**
+	 * @param \Closure $field
+	 */
 	abstract public function addField($field);
+
+	/**
+	 * @param integer $index
+	 * @param \Closure $field
+	 */
 	abstract public function insertField($index,$field);
+
+	/**
+	 * @param \Closure $field
+	 */
 	abstract public function insertInField($index,$field);
 	abstract public function fieldAs($index,$type,$attributes=NULL);
 	/**
 	 * @param string $caption
 	 * @param callable $callback
 	 * @param boolean $visibleHover
-	 * @return callable
+	 * @return \Closure
 	 */
 	private function getFieldButtonCallable($caption,$visibleHover=true,$callback=null){
 		return $this->getCallable("getFieldButton",[$caption,$visibleHover],$callback);
@@ -33,7 +46,7 @@ 
 	 * @param callable $thisCallback
 	 * @param array $parameters
 	 * @param callable $callback
-	 * @return callable
+	 * @return \Closure
 	 */
 	private function getCallable($thisCallback,$parameters,$callback=null){
 		$result=function($instance) use($thisCallback,$parameters,$callback){
@@ -103,7 +116,7 @@ 
 	 * @param string $caption
 	 * @param callable $callback
 	 * @param boolean $visibleHover
-	 * @return DataTable
+	 * @return DataTableFieldAsTrait
 	 */
 	public function addFieldButton($caption,$visibleHover=true,$callback=null){
 		$this->addField($this->getCallable("getFieldButton",[$caption,$visibleHover],$callback));
@@ -115,7 +128,7 @@ 
 	 * @param array $buttons
 	 * @param callable $callback
 	 * @param boolean $visibleHover
-	 * @return DataTable
+	 * @return DataTableFieldAsTrait
 	 */
 	public function addFieldButtons($buttons,$visibleHover=true,$callback=null){
 		$this->addField($this->getCallable("getFieldButtons",[$buttons,$visibleHover],$callback));
@@ -127,7 +140,7 @@ 
 	 * @param int $index
 	 * @param string $caption
 	 * @param callable $callback
-	 * @return DataTable
+	 * @return DataTableFieldAsTrait
 	 */
 	public function insertFieldButton($index,$caption,$visibleHover=true,$callback=null){
 		$this->insertField($index, $this->getFieldButtonCallable($caption,$visibleHover,$callback));
@@ -139,18 +152,27 @@ 
 	 * @param int $index
 	 * @param string $caption
 	 * @param callable $callback
-	 * @return DataTable
+	 * @return DataTableFieldAsTrait
 	 */
 	public function insertInFieldButton($index,$caption,$visibleHover=true,$callback=null){
 		$this->insertInField($index, $this->getFieldButtonCallable($caption,$visibleHover,$callback));
 		return $this;
 	}
 
+	/**
+	 * @param string $icon
+	 * @param string $class
+	 * @param callable $callback
+	 */
 	private function addDefaultButton($icon,$class=null,$visibleHover=true,$callback=null){
 		$this->addField($this->getCallable("getDefaultButton",[$icon,$class,$visibleHover],$callback));
 		return $this;
 	}
 
+	/**
+	 * @param string $icon
+	 * @param string $class
+	 */
 	private function insertDefaultButtonIn($index,$icon,$class=null,$visibleHover=true,$callback=null){
 		$this->insertInField($index,$this->getCallable("getDefaultButton",[$icon,$class,$visibleHover],$callback));
 		return $this;
@@ -169,7 +191,7 @@ 
 	 * @param boolean $visibleHover
 	 * @param array $deleteBehavior default : array("preventDefault"=>true,"stopPropagation"=>true,"jsCallback"=>NULL,"attr"=>"data-ajax","params"=>"{}","method"=>"get")
 	 * @param callable $callback this function takes the following arguments : $object=>the delete button, $instance : the active instance of the object
-	 * @return DataTable
+	 * @return DataTableFieldAsTrait
 	 */
 	public function addDeleteButton($visibleHover=true,$deleteBehavior=[],$callback=null){
 		$this->_deleteBehavior=$deleteBehavior;
@@ -181,7 +203,7 @@ 
 	 * @param string $visibleHover
 	 * @param array $editBehavior default : array("preventDefault"=>true,"stopPropagation"=>true,"jsCallback"=>NULL,"attr"=>"data-ajax","params"=>"{}","method"=>"get")
 	 * @param callable $callback this function takes the following arguments : $object=>the delete button, $instance : the active instance of the object
-	 * @return DataTable
+	 * @return DataTableFieldAsTrait
 	 */
 	public function addEditButton($visibleHover=true,$editBehavior=[],$callback=null){
 		$this->_editBehavior=$editBehavior;
@@ -194,7 +216,7 @@ 
 	 * @param array $behavior default : array("preventDefault"=>true,"stopPropagation"=>true,"jsCallback"=>NULL,"attr"=>"data-ajax","params"=>"{}","method"=>"get")
 	 * @param callable $callbackEdit this function takes the following arguments : $object=>the delete button, $instance : the active instance of the object
 	 * @param callable $callbackDelete this function takes the following arguments : $object=>the delete button, $instance : the active instance of the object
-	 * @return DataTable
+	 * @return DataTableFieldAsTrait
 	 */
 	public function addEditDeleteButtons($visibleHover=true,$behavior=[],$callbackEdit=null,$callbackDelete=null){
 		$this->addEditButton($visibleHover,$behavior,$callbackEdit);
@@ -203,6 +225,9 @@ 
 		return $this;
 	}
 
+	/**
+	 * @param callable $callback
+	 */
 	public function insertDeleteButtonIn($index,$visibleHover=true,$deleteBehavior=[],$callback=null){
 		$this->_deleteBehavior=$deleteBehavior;
 		return $this->insertDefaultButtonIn($index,"remove","_delete red basic",$visibleHover,$callback);

Spacing +56 added lines, -56 removed lines

@@ -14,19 +14,19 @@ 
  * @property boolean $_visibleHover
  * @property InstanceViewer $_instanceViewer
  */
-trait DataTableFieldAsTrait{
+trait DataTableFieldAsTrait {
 	abstract public function addField($field);
-	abstract public function insertField($index,$field);
-	abstract public function insertInField($index,$field);
-	abstract public function fieldAs($index,$type,$attributes=NULL);
+	abstract public function insertField($index, $field);
+	abstract public function insertInField($index, $field);
+	abstract public function fieldAs($index, $type, $attributes=NULL);
 	/**
 	 * @param string $caption
 	 * @param callable $callback
 	 * @param boolean $visibleHover
 	 * @return callable
 	 */
-	private function getFieldButtonCallable($caption,$visibleHover=true,$callback=null){
-		return $this->getCallable("getFieldButton",[$caption,$visibleHover],$callback);
+	private function getFieldButtonCallable($caption, $visibleHover=true, $callback=null) {
+		return $this->getCallable("getFieldButton", [$caption, $visibleHover], $callback);
 	}
 
 	/**
@@ -35,20 +35,20 @@ 
 	 * @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,$this->_instanceViewer->count()+1);
+	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, $this->_instanceViewer->count()+1);
 				}
 			}
-			if($object instanceof HtmlSemDoubleElement){
+			if ($object instanceof HtmlSemDoubleElement) {
 				$id=$this->_instanceViewer->getIdentifier();
-				$object->setProperty("data-ajax",$id);
-				if($object->propertyContains("class","visibleover")){
+				$object->setProperty("data-ajax", $id);
+				if ($object->propertyContains("class", "visibleover")) {
 					$this->_visibleHover=true;
-					$object->setProperty("style","visibility:hidden;");
+					$object->setProperty("style", "visibility:hidden;");
 				}
 			}
 			return $object;
@@ -60,16 +60,16 @@ 
 	 * @param string $caption
 	 * @return HtmlButton
 	 */
-	private function getFieldButton($caption,$visibleHover=true){
-		$bt= new HtmlButton($this->cleanIdentifier($caption),$caption);
-		if($visibleHover)
+	private function getFieldButton($caption, $visibleHover=true) {
+		$bt=new HtmlButton($this->cleanIdentifier($caption), $caption);
+		if ($visibleHover)
 			$this->_visibleOver($bt);
 		return $bt;
 	}
 
-	private function getFieldButtons($buttons,$visibleHover=true){
-		$bts=new HtmlButtonGroups("",$buttons);
-		if($visibleHover)
+	private function getFieldButtons($buttons, $visibleHover=true) {
+		$bts=new HtmlButtonGroups("", $buttons);
+		if ($visibleHover)
 			$this->_visibleOver($bts);
 		return $bts;
 	}
@@ -83,19 +83,19 @@ 
 	 * @param array $attributes associative array (<b>ajax</b> key is for ajax post)
 	 * @return DataTable
 	 */
-	public function fieldAsSubmit($index,$cssStyle=NULL,$url=NULL,$responseElement=NULL,$attributes=NULL){
-		return $this->_fieldAs(function($id,$name,$value,$caption) use ($url,$responseElement,$cssStyle,$index,$attributes){
-			$button=new HtmlButton($id,$value,$cssStyle);
-			$button->postOnClick($url,"$(event.target).closest('tr').find(':input').serialize()",$responseElement,$attributes["ajax"]);
-			if(!isset($attributes["visibleHover"]) || $attributes["visibleHover"])
+	public function fieldAsSubmit($index, $cssStyle=NULL, $url=NULL, $responseElement=NULL, $attributes=NULL) {
+		return $this->_fieldAs(function($id, $name, $value, $caption) use ($url, $responseElement, $cssStyle, $index, $attributes){
+			$button=new HtmlButton($id, $value, $cssStyle);
+			$button->postOnClick($url, "$(event.target).closest('tr').find(':input').serialize()", $responseElement, $attributes["ajax"]);
+			if (!isset($attributes["visibleHover"]) || $attributes["visibleHover"])
 				$this->_visibleOver($button);
 				return $button;
-		}, $index,$attributes);
+		}, $index, $attributes);
 	}
 
-	protected function _visibleOver(BaseHtml $element){
+	protected function _visibleOver(BaseHtml $element) {
 		$this->_visibleHover=true;
-		return $element->addToProperty("class", "visibleover")->setProperty("style","visibility:hidden;");
+		return $element->addToProperty("class", "visibleover")->setProperty("style", "visibility:hidden;");
 	}
 
 	/**
@@ -105,8 +105,8 @@ 
 	 * @param boolean $visibleHover
 	 * @return DataTable
 	 */
-	public function addFieldButton($caption,$visibleHover=true,$callback=null){
-		$this->addField($this->getCallable("getFieldButton",[$caption,$visibleHover],$callback));
+	public function addFieldButton($caption, $visibleHover=true, $callback=null) {
+		$this->addField($this->getCallable("getFieldButton", [$caption, $visibleHover], $callback));
 		return $this;
 	}
 
@@ -117,8 +117,8 @@ 
 	 * @param boolean $visibleHover
 	 * @return DataTable
 	 */
-	public function addFieldButtons($buttons,$visibleHover=true,$callback=null){
-		$this->addField($this->getCallable("getFieldButtons",[$buttons,$visibleHover],$callback));
+	public function addFieldButtons($buttons, $visibleHover=true, $callback=null) {
+		$this->addField($this->getCallable("getFieldButtons", [$buttons, $visibleHover], $callback));
 		return $this;
 	}
 
@@ -129,8 +129,8 @@ 
 	 * @param callable $callback
 	 * @return DataTable
 	 */
-	public function insertFieldButton($index,$caption,$visibleHover=true,$callback=null){
-		$this->insertField($index, $this->getFieldButtonCallable($caption,$visibleHover,$callback));
+	public function insertFieldButton($index, $caption, $visibleHover=true, $callback=null) {
+		$this->insertField($index, $this->getFieldButtonCallable($caption, $visibleHover, $callback));
 		return $this;
 	}
 
@@ -141,25 +141,25 @@ 
 	 * @param callable $callback
 	 * @return DataTable
 	 */
-	public function insertInFieldButton($index,$caption,$visibleHover=true,$callback=null){
-		$this->insertInField($index, $this->getFieldButtonCallable($caption,$visibleHover,$callback));
+	public function insertInFieldButton($index, $caption, $visibleHover=true, $callback=null) {
+		$this->insertInField($index, $this->getFieldButtonCallable($caption, $visibleHover, $callback));
 		return $this;
 	}
 
-	private function addDefaultButton($icon,$class=null,$visibleHover=true,$callback=null){
-		$this->addField($this->getCallable("getDefaultButton",[$icon,$class,$visibleHover],$callback));
+	private function addDefaultButton($icon, $class=null, $visibleHover=true, $callback=null) {
+		$this->addField($this->getCallable("getDefaultButton", [$icon, $class, $visibleHover], $callback));
 		return $this;
 	}
 
-	private function insertDefaultButtonIn($index,$icon,$class=null,$visibleHover=true,$callback=null){
-		$this->insertInField($index,$this->getCallable("getDefaultButton",[$icon,$class,$visibleHover],$callback));
+	private function insertDefaultButtonIn($index, $icon, $class=null, $visibleHover=true, $callback=null) {
+		$this->insertInField($index, $this->getCallable("getDefaultButton", [$icon, $class, $visibleHover], $callback));
 		return $this;
 	}
 
-	private function getDefaultButton($icon,$class=null,$visibleHover=true){
-		$bt=$this->getFieldButton("",$visibleHover);
+	private function getDefaultButton($icon, $class=null, $visibleHover=true) {
+		$bt=$this->getFieldButton("", $visibleHover);
 		$bt->asIcon($icon);
-		if(isset($class))
+		if (isset($class))
 			$bt->addClass($class);
 		return $bt;
 	}
@@ -171,9 +171,9 @@ 
 	 * @param callable $callback this function takes the following arguments : $object=>the delete button, $instance : the active instance of the object
 	 * @return DataTable
 	 */
-	public function addDeleteButton($visibleHover=true,$deleteBehavior=[],$callback=null){
+	public function addDeleteButton($visibleHover=true, $deleteBehavior=[], $callback=null) {
 		$this->_deleteBehavior=$deleteBehavior;
-		return $this->addDefaultButton("remove","_delete red basic",$visibleHover,$callback);
+		return $this->addDefaultButton("remove", "_delete red basic", $visibleHover, $callback);
 	}
 
 	/**
@@ -183,9 +183,9 @@ 
 	 * @param callable $callback this function takes the following arguments : $object=>the delete button, $instance : the active instance of the object
 	 * @return DataTable
 	 */
-	public function addEditButton($visibleHover=true,$editBehavior=[],$callback=null){
+	public function addEditButton($visibleHover=true, $editBehavior=[], $callback=null) {
 		$this->_editBehavior=$editBehavior;
-		return $this->addDefaultButton("edit","_edit basic",$visibleHover,$callback);
+		return $this->addDefaultButton("edit", "_edit basic", $visibleHover, $callback);
 	}
 
 	/**
@@ -196,20 +196,20 @@ 
 	 * @param callable $callbackDelete this function takes the following arguments : $object=>the delete button, $instance : the active instance of the object
 	 * @return DataTable
 	 */
-	public function addEditDeleteButtons($visibleHover=true,$behavior=[],$callbackEdit=null,$callbackDelete=null){
-		$this->addEditButton($visibleHover,$behavior,$callbackEdit);
+	public function addEditDeleteButtons($visibleHover=true, $behavior=[], $callbackEdit=null, $callbackDelete=null) {
+		$this->addEditButton($visibleHover, $behavior, $callbackEdit);
 		$index=$this->_instanceViewer->visiblePropertiesCount()-1;
-		$this->insertDeleteButtonIn($index,$visibleHover,$behavior,$callbackDelete);
+		$this->insertDeleteButtonIn($index, $visibleHover, $behavior, $callbackDelete);
 		return $this;
 	}
 
-	public function insertDeleteButtonIn($index,$visibleHover=true,$deleteBehavior=[],$callback=null){
+	public function insertDeleteButtonIn($index, $visibleHover=true, $deleteBehavior=[], $callback=null) {
 		$this->_deleteBehavior=$deleteBehavior;
-		return $this->insertDefaultButtonIn($index,"remove","_delete red basic",$visibleHover,$callback);
+		return $this->insertDefaultButtonIn($index, "remove", "_delete red basic", $visibleHover, $callback);
 	}
 
-	public function insertEditButtonIn($index,$visibleHover=true,$editBehavior=[],$callback=null){
+	public function insertEditButtonIn($index, $visibleHover=true, $editBehavior=[], $callback=null) {
 		$this->_editBehavior=$editBehavior;
-		return $this->insertDefaultButtonIn($index,"edit","_edit basic",$visibleHover,$callback);
+		return $this->insertDefaultButtonIn($index, "edit", "_edit basic", $visibleHover, $callback);
 	}
 }

Braces +12 added lines, -8 removed lines

@@ -62,15 +62,17 @@ 
 	 */
 	private function getFieldButton($caption,$visibleHover=true){
 		$bt= new HtmlButton($this->cleanIdentifier($caption),$caption);
-		if($visibleHover)
-			$this->_visibleOver($bt);
+		if($visibleHover) {
+					$this->_visibleOver($bt);
+		}
 		return $bt;
 	}
 
 	private function getFieldButtons($buttons,$visibleHover=true){
 		$bts=new HtmlButtonGroups("",$buttons);
-		if($visibleHover)
-			$this->_visibleOver($bts);
+		if($visibleHover) {
+					$this->_visibleOver($bts);
+		}
 		return $bts;
 	}
 
@@ -87,8 +89,9 @@ 
 		return $this->_fieldAs(function($id,$name,$value,$caption) use ($url,$responseElement,$cssStyle,$index,$attributes){
 			$button=new HtmlButton($id,$value,$cssStyle);
 			$button->postOnClick($url,"$(event.target).closest('tr').find(':input').serialize()",$responseElement,$attributes["ajax"]);
-			if(!isset($attributes["visibleHover"]) || $attributes["visibleHover"])
-				$this->_visibleOver($button);
+			if(!isset($attributes["visibleHover"]) || $attributes["visibleHover"]) {
+							$this->_visibleOver($button);
+			}
 				return $button;
 		}, $index,$attributes);
 	}
@@ -159,8 +162,9 @@ 
 	private function getDefaultButton($icon,$class=null,$visibleHover=true){
 		$bt=$this->getFieldButton("",$visibleHover);
 		$bt->asIcon($icon);
-		if(isset($class))
-			$bt->addClass($class);
+		if(isset($class)) {
+					$bt->addClass($class);
+		}
 		return $bt;
 	}
 

Ajax/semantic/html/elements/HtmlButton.php

Doc Comments +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
 
 	/**
 	 * define the button style
-	 * @param string|int $cssStyle
+	 * @param string $cssStyle
 	 * @return HtmlButton default : ""
 	 */
 	public function setStyle($cssStyle) {

Spacing +23 added lines, -23 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;
@@ -104,17 +104,17 @@ 
 	 * @return HtmlLabel
 	 */
 	public function addLabel($label, $before=false, $icon=NULL) {
-		$this->tagName="div";$prefix="";
-		if($before)
+		$this->tagName="div"; $prefix="";
+		if ($before)
 			$prefix="left ";
 		$this->addToProperty("class", $prefix."labeled");
 		$isIcon=(isset($this->content[0]) && $this->content[0] instanceof HtmlIcon);
-		$this->content=new HtmlButton("button-" . $this->identifier, $this->content);
-		if($isIcon){
+		$this->content=new HtmlButton("button-".$this->identifier, $this->content);
+		if ($isIcon) {
 			$this->content->addClass("icon");
 		}
 		$this->content->setTagName("div");
-		$label=new HtmlLabel("label-" . $this->identifier, $label, $icon,"a");
+		$label=new HtmlLabel("label-".$this->identifier, $label, $icon, "a");
 		$label->setBasic();
 		$this->addContent($label, $before);
 		return $label;
@@ -126,7 +126,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;
@@ -140,10 +140,10 @@ 
 		return $this->addToProperty("class", "positive");
 	}
 
-	public function setColor($color){
-		if(\is_array($this->content)){
-			foreach ($this->content as $content){
-				if($content instanceof HtmlButton)
+	public function setColor($color) {
+		if (\is_array($this->content)) {
+			foreach ($this->content as $content) {
+				if ($content instanceof HtmlButton)
 					$content->setColor($color);
 			}
 		}
@@ -201,7 +201,7 @@ 
 	 * @return HtmlButton
 	 */
 	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);
@@ -238,8 +238,8 @@ 
 	 * {@inheritDoc}
 	 * @see HtmlSemDoubleElement::asLink()
 	 */
-	public function asLink($href=NULL,$target=NULL) {
-		parent::asLink($href,$target);
+	public function asLink($href=NULL, $target=NULL) {
+		parent::asLink($href, $target);
 		return $this;
 	}
 
@@ -252,10 +252,10 @@ 
 	 * @param string $icon
 	 * @return HtmlButtonGroups
 	 */
-	public static function dropdown($identifier,$value,$items=[],$asCombo=false,$icon=null){
-		$result=new HtmlButtonGroups($identifier,[$value]);
-		$result->addDropdown($items,$asCombo);
-		if(isset($icon))
+	public static function dropdown($identifier, $value, $items=[], $asCombo=false, $icon=null) {
+		$result=new HtmlButtonGroups($identifier, [$value]);
+		$result->addDropdown($items, $asCombo);
+		if (isset($icon))
 			$result->setIcon($icon);
 		return $result;
 	}

Braces +17 added lines, -13 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;
@@ -105,8 +105,9 @@ 
 	 */
 	public function addLabel($label, $before=false, $icon=NULL) {
 		$this->tagName="div";$prefix="";
-		if($before)
-			$prefix="left ";
+		if($before) {
+					$prefix="left ";
+		}
 		$this->addToProperty("class", $prefix."labeled");
 		$isIcon=(isset($this->content[0]) && $this->content[0] instanceof HtmlIcon);
 		$this->content=new HtmlButton("button-" . $this->identifier, $this->content);
@@ -143,12 +144,13 @@ 
 	public function setColor($color){
 		if(\is_array($this->content)){
 			foreach ($this->content as $content){
-				if($content instanceof HtmlButton)
-					$content->setColor($color);
+				if($content instanceof HtmlButton) {
+									$content->setColor($color);
+				}
 			}
+		} else {
+					parent::setColor($color);
 		}
-		else
-			parent::setColor($color);
 		return $this;
 	}
 
@@ -201,8 +203,9 @@ 
 	 * @return HtmlButton
 	 */
 	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());
@@ -255,8 +258,9 @@ 
 	public static function dropdown($identifier,$value,$items=[],$asCombo=false,$icon=null){
 		$result=new HtmlButtonGroups($identifier,[$value]);
 		$result->addDropdown($items,$asCombo);
-		if(isset($icon))
-			$result->setIcon($icon);
+		if(isset($icon)) {
+					$result->setIcon($icon);
+		}
 		return $result;
 	}
 }

Ajax/semantic/html/elements/HtmlStep.php

Spacing +23 added lines, -23 removed lines

@@ -9,12 +9,12 @@ 
 use Ajax\common\html\HtmlDoubleElement;
 use Ajax\semantic\html\base\constants\Side;
 
-class HtmlStep extends HtmlSemCollection{
+class HtmlStep extends HtmlSemCollection {
 	protected $_activeStep;
 	protected $_startStep;
 
-	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);
 	}
 
@@ -24,7 +24,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;
 	}
 
@@ -32,50 +32,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=$this->_startStep;$i<$count;$i++){
+		if (!$this->isOrdered()) {
+			for ($i=$this->_startStep; $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) && \is_numeric($this->_activeStep))
+		if (isset($this->_activeStep) && \is_numeric($this->_activeStep))
 			$this->defineActiveStep();
-		return parent::compile($js,$view);
+		return parent::compile($js, $view);
 	}
 
 	public function setActiveStep($_activeStep) {
@@ -83,15 +83,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

@@ -55,26 +55,29 @@
 			for($i=$this->_startStep;$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) && \is_numeric($this->_activeStep))
-			$this->defineActiveStep();
+		if(isset($this->_activeStep) && \is_numeric($this->_activeStep)) {
+					$this->defineActiveStep();
+		}
 		return parent::compile($js,$view);
 	}
 

Ajax/semantic/html/content/HtmlListItem.php

Spacing +17 added lines, -17 removed lines

@@ -9,15 +9,15 @@ 
 	protected $image;
 
 	public function __construct($identifier, $content=NULL) {
-		parent::__construct($identifier,"item",$content);
+		parent::__construct($identifier, "item", $content);
 	}
-	protected function initContent($content){
-		if(\is_array($content)){
-			if(JArray::isAssociative($content)===false){
+	protected function initContent($content) {
+		if (\is_array($content)) {
+			if (JArray::isAssociative($content)===false) {
 				$icon=@$content[0];
 				$title=@$content[1];
 				$desc=@$content[2];
-			}else{
+			} else {
 				$icon=@$content["icon"];
 				$image=@$content["image"];
 				$title=@$content["title"];
@@ -25,38 +25,38 @@ 
 				$desc=@$content["description"];
 				$items=@$content["items"];
 			}
-			if(isset($icon)){
+			if (isset($icon)) {
 				$this->setIcon($icon);
 			}
-			if(isset($image)){
+			if (isset($image)) {
 				$this->setImage($image);
 			}
-			if(isset($title)){
-				$this->setTitle($title,$desc);
-			}elseif (isset($header)){
-				$this->setTitle($header,$desc,"header");
+			if (isset($title)) {
+				$this->setTitle($title, $desc);
+			}elseif (isset($header)) {
+				$this->setTitle($header, $desc, "header");
 			}
-			if(isset($items)){
+			if (isset($items)) {
 				$this->addList($items);
 			}
-		}else{
+		} else {
 			$this->setContent($content);
 		}
 	}
-	public function addList($items=array(),$ordered=false) {
+	public function addList($items=array(), $ordered=false) {
 		$list=new HtmlList("", $items);
-		if($ordered)
+		if ($ordered)
 			$list->setOrdered();
 		$list->setClass("list");
 		$this->content["list"]=$list;
 		return $list;
 	}
 
-	public function getList(){
+	public function getList() {
 		return $this->content["list"];
 	}
 
-	public function getItem($index){
+	public function getItem($index) {
 		return $this->getList()->getItem($index);
 	}
 }

Braces +6 added lines, -5 removed lines

@@ -17,7 +17,7 @@ 
 				$icon=@$content[0];
 				$title=@$content[1];
 				$desc=@$content[2];
-			}else{
+			} else{
 				$icon=@$content["icon"];
 				$image=@$content["image"];
 				$title=@$content["title"];
@@ -33,20 +33,21 @@ 
 			}
 			if(isset($title)){
 				$this->setTitle($title,$desc);
-			}elseif (isset($header)){
+			} elseif (isset($header)){
 				$this->setTitle($header,$desc,"header");
 			}
 			if(isset($items)){
 				$this->addList($items);
 			}
-		}else{
+		} else{
 			$this->setContent($content);
 		}
 	}
 	public function addList($items=array(),$ordered=false) {
 		$list=new HtmlList("", $items);
-		if($ordered)
-			$list->setOrdered();
+		if($ordered) {
+					$list->setOrdered();
+		}
 		$list->setClass("list");
 		$this->content["list"]=$list;
 		return $list;

Ajax/semantic/html/modules/HtmlPopup.php

Spacing +15 added lines, -15 removed lines

@@ -11,7 +11,7 @@ 
 
 class HtmlPopup extends HtmlSemDoubleElement {
 	private $_container;
-	public function __construct(BaseHtml $container,$identifier, $content="") {
+	public function __construct(BaseHtml $container, $identifier, $content="") {
 		parent::__construct($identifier, "div");
 		$this->_container=$container;
 		$this->setClass("ui popup");
@@ -23,9 +23,9 @@ 
 	 * {@inheritDoc}
 	 * @see \Ajax\semantic\html\base\HtmlSemDoubleElement::addList()
 	 */
-	public function addList($items=array(),$header=NULL){
-		if(!$this->content instanceof HtmlGrid){
-			$this->content=new HtmlGrid("Grid-".$this->identifier,0);
+	public function addList($items=array(), $header=NULL) {
+		if (!$this->content instanceof HtmlGrid) {
+			$this->content=new HtmlGrid("Grid-".$this->identifier, 0);
 		}
 		$grid=$this->content;
 
@@ -33,12 +33,12 @@ 
 		$colCount++;
 		$grid->setColsCount($colCount);
 
-		$list=new HtmlList("",$items);
+		$list=new HtmlList("", $items);
 		$list->asLink();
-		if(isset($header)){
-			$list->addHeader(4,$header);
+		if (isset($header)) {
+			$list->addHeader(4, $header);
 		}
-		$grid->getCell(0,$colCount-1)->setContent($list);
+		$grid->getCell(0, $colCount-1)->setContent($list);
 		$grid->setDivided()->setRelaxed(true);
 		return $list;
 	}
@@ -46,14 +46,14 @@ 
 	/**
 	 * A popup can have no maximum width and continue to flow to fit its content
 	 */
-	public function setFlowing(){
+	public function setFlowing() {
 		return $this->addToProperty("class", "flowing");
 	}
 
 	/**
 	 * A popup can provide more basic formatting
 	 */
-	public function setBasic(){
+	public function setBasic() {
 		return $this->addToProperty("class", "basic");
 	}
 
@@ -61,23 +61,23 @@ 
 	 * {@inheritDoc}
 	 * @see \Ajax\semantic\html\base\HtmlSemDoubleElement::run()
 	 */
-	public function run(JsUtils $js){
+	public function run(JsUtils $js) {
 		parent::run($js);
 		$this->_params["popup"]="#".$this->identifier;
-		$js->semantic()->popup("#".$this->_container->getIdentifier(),$this->_params);
+		$js->semantic()->popup("#".$this->_container->getIdentifier(), $this->_params);
 	}
 
-	public function setOn($event="click"){
+	public function setOn($event="click") {
 		$this->_params["on"]=$event;
 		return $this;
 	}
 
-	public function setInline($value=true){
+	public function setInline($value=true) {
 		$this->_params["inline"]=$value;
 		return $this;
 	}
 
-	public function setPosition($position){
+	public function setPosition($position) {
 		$this->_params["position"]=$position;
 		return $this;
 	}

Ajax/semantic/html/collections/form/HtmlFormDropdown.php

Spacing +11 added lines, -11 removed lines

@@ -7,29 +7,29 @@ 
 
 class HtmlFormDropdown extends HtmlFormField {
 
-	public function __construct($identifier,$items=array(), $label=NULL,$value="",$multiple=false,$associative=true) {
-		parent::__construct("field-".$identifier, (new HtmlDropdown("dropdown-".$identifier,$value,$items,$associative))->asSelect($identifier,$multiple), $label);
+	public function __construct($identifier, $items=array(), $label=NULL, $value="", $multiple=false, $associative=true) {
+		parent::__construct("field-".$identifier, (new HtmlDropdown("dropdown-".$identifier, $value, $items, $associative))->asSelect($identifier, $multiple), $label);
 		$this->_identifier=$identifier;
 	}
 
-	public function setItems($items){
+	public function setItems($items) {
 		return $this->getField()->setItems($items);
 	}
-	public function addItem($item,$value=NULL,$image=NULL){
-		return $this->getField()->addItem($item,$value,$image);
+	public function addItem($item, $value=NULL, $image=NULL) {
+		return $this->getField()->addItem($item, $value, $image);
 	}
-	public static function multipleDropdown($identifier,$items=array(), $label=NULL,$value="",$associative=true){
-		return new HtmlFormDropdown($identifier,$items,$label,$value,true,$associative);
+	public static function multipleDropdown($identifier, $items=array(), $label=NULL, $value="", $associative=true) {
+		return new HtmlFormDropdown($identifier, $items, $label, $value, true, $associative);
 	}
 
 	/**
 	 * @return HtmlDropdown
 	 */
-	public function getDataField(){
+	public function getDataField() {
 		return $this->getField()->getInput();
 	}
-	public function asSelect($name=NULL,$multiple=false,$selection=true){
-		$this->getField()->asSelect($name,$multiple,$selection);
+	public function asSelect($name=NULL, $multiple=false, $selection=true) {
+		$this->getField()->asSelect($name, $multiple, $selection);
 		return $this;
 	}
 
@@ -37,7 +37,7 @@ 
 	 * @param boolean $floating
 	 * @return HtmlDropdown
 	 */
-	public function asButton($floating=false){
+	public function asButton($floating=false) {
 		$field=$this->content["field"];
 		$label=$this->content["label"];
 		$field->addContent($label);

Ajax/semantic/html/collections/HtmlMessage.php

Spacing +29 added lines, -29 removed lines

@@ -15,7 +15,7 @@ 
  * @version 1.001
  */
 class HtmlMessage extends HtmlSemDoubleElement {
-	use AttachedTrait,HasTimeoutTrait;
+	use AttachedTrait, HasTimeoutTrait;
 	protected $icon;
 	protected $close;
 
@@ -32,39 +32,39 @@ 
 	 * @param string|HtmlSemDoubleElement $header
 	 * @return \Ajax\semantic\html\collections\HtmlMessage
 	 */
-	public function addHeader($header){
+	public function addHeader($header) {
 		$headerO=$header;
-		if(\is_string($header)){
-			$headerO=new HtmlSemDoubleElement("header-".$this->identifier,"div");
+		if (\is_string($header)) {
+			$headerO=new HtmlSemDoubleElement("header-".$this->identifier, "div");
 			$headerO->setClass("header");
 			$headerO->setContent($header);
 		}
-		return $this->addContent($headerO,true);
+		return $this->addContent($headerO, true);
 	}
 
-	public function setHeader($header){
+	public function setHeader($header) {
 		return $this->addHeader($header);
 	}
 
-	public function setIcon($icon){
+	public function setIcon($icon) {
 		$this->addToProperty("class", "icon");
-		$this->wrapContent("<div class='content'>","</div>");
-		if(\is_string($icon)){
+		$this->wrapContent("<div class='content'>", "</div>");
+		if (\is_string($icon)) {
 			$this->icon=new HtmlIcon("icon-".$this->identifier, $icon);
-		}else{
+		} else {
 			$this->icon=$icon;
 		}
 		return $this;
 	}
 
-	public function addLoader($loaderIcon="notched circle"){
+	public function addLoader($loaderIcon="notched circle") {
 		$this->setIcon($loaderIcon);
 		$this->icon->addToIcon("loading");
 		return $this;
 	}
 
-	public function setDismissable($dismiss=true){
-		if($dismiss===true)
+	public function setDismissable($dismiss=true) {
+		if ($dismiss===true)
 			$this->close=new HtmlIcon("close-".$this->identifier, "close");
 		else
 			$this->close=NULL;
@@ -75,43 +75,43 @@ 
 	 * {@inheritDoc}
 	 * @see \Ajax\semantic\html\base\HtmlSemDoubleElement::run()
 	 */
-	public function run(JsUtils $js){
-		if(!isset($this->_bsComponent)){
-			if(isset($this->close)){
+	public function run(JsUtils $js) {
+		if (!isset($this->_bsComponent)) {
+			if (isset($this->close)) {
 				$js->execOn("click", "#".$this->identifier." .close", "$(this).closest('.message').transition({$this->_closeTransition})");
 			}
-			if(isset($this->_timeout)){
-				$js->exec("setTimeout(function() { $('#{$this->identifier}').transition({$this->_closeTransition}); }, {$this->_timeout});",true);
+			if (isset($this->_timeout)) {
+				$js->exec("setTimeout(function() { $('#{$this->identifier}').transition({$this->_closeTransition}); }, {$this->_timeout});", true);
 			}
 		}
 		return parent::run($js);
 	}
 
-	public function setState($visible=true){
-		$visible=($visible===true)?"visible":"hidden";
-		return $this->addToPropertyCtrl("class", $visible, array("visible","hidden"));
+	public function setState($visible=true) {
+		$visible=($visible===true) ? "visible" : "hidden";
+		return $this->addToPropertyCtrl("class", $visible, array("visible", "hidden"));
 	}
 
-	public function setVariation($value="floating"){
-		return $this->addToPropertyCtrl("class", $value, array("floating","compact"));
+	public function setVariation($value="floating") {
+		return $this->addToPropertyCtrl("class", $value, array("floating", "compact"));
 	}
 
-	public function setStyle($style){
+	public function setStyle($style) {
 		return $this->addToPropertyCtrl("class", $style, Style::getConstants());
 	}
 
-	public function setError(){
+	public function setError() {
 		return $this->setStyle("error");
 	}
 
-	public function setWarning(){
+	public function setWarning() {
 		return $this->setStyle("warning");
 	}
 
-	public function setMessage($message){
-		if(\is_array($this->content)){
+	public function setMessage($message) {
+		if (\is_array($this->content)) {
 			$this->content[\sizeof($this->content)-1]=$message;
-		}else
+		} else
 			$this->setContent($message);
 	}
 

Braces +9 added lines, -7 removed lines

@@ -51,7 +51,7 @@ 
 		$this->wrapContent("<div class='content'>","</div>");
 		if(\is_string($icon)){
 			$this->icon=new HtmlIcon("icon-".$this->identifier, $icon);
-		}else{
+		} else{
 			$this->icon=$icon;
 		}
 		return $this;
@@ -64,10 +64,11 @@ 
 	}
 
 	public function setDismissable($dismiss=true){
-		if($dismiss===true)
-			$this->close=new HtmlIcon("close-".$this->identifier, "close");
-		else
-			$this->close=NULL;
+		if($dismiss===true) {
+					$this->close=new HtmlIcon("close-".$this->identifier, "close");
+		} else {
+					$this->close=NULL;
+		}
 		return $this;
 	}
 
@@ -111,8 +112,9 @@ 
 	public function setMessage($message){
 		if(\is_array($this->content)){
 			$this->content[\sizeof($this->content)-1]=$message;
-		}else
-			$this->setContent($message);
+		} else {
+					$this->setContent($message);
+		}
 	}
 
 }

Ajax/service/Javascript.php

Spacing +4 added lines, -4 removed lines

@@ -6,8 +6,8 @@ 
 	public static $preventDefault="\nif(event && event.preventDefault) event.preventDefault();\n";
 	public static $stopPropagation="\nif(event && event.stopPropagation) event.stopPropagation();\n";
 
-	public static function containsCode($expression){
-		return strrpos($expression, 'this')!==false||strrpos($expression, 'event')!==false||strrpos($expression, 'self')!==false;
+	public static function containsCode($expression) {
+		return strrpos($expression, 'this')!==false || strrpos($expression, 'event')!==false || strrpos($expression, 'self')!==false;
 	}
 
 	/**
@@ -43,8 +43,8 @@ 
 		return $value;
 	}
 
-	public static function prep_jquery_selector($value){
-		if(JString::startswith($value, '$(')===false){
+	public static function prep_jquery_selector($value) {
+		if (JString::startswith($value, '$(')===false) {
 			return '$('.$value.')';
 		}
 		return $value;

Ajax/service/AjaxTransition.php

Spacing +8 added lines, -8 removed lines

@@ -3,26 +3,26 @@
 use Ajax\semantic\html\base\constants\Transition;
 
 class AjaxTransition {
-	public static function none($responseElement,$jqueryDone="html"){
+	public static function none($responseElement, $jqueryDone="html") {
 		return $responseElement.".".$jqueryDone."( data )";
 	}
 
-	public static function jqFade($responseElement,$jqueryDone="html"){
+	public static function jqFade($responseElement, $jqueryDone="html") {
 		return $responseElement.".hide().{$jqueryDone}( data ).fadeIn()";
 	}
 
-	public static function jqSlide($responseElement,$jqueryDone="html"){
+	public static function jqSlide($responseElement, $jqueryDone="html") {
 		return $responseElement.".hide().{$jqueryDone}( data ).slideDown()";
 	}
 
-	public static function random($responseElement,$jqueryDone="html"){
+	public static function random($responseElement, $jqueryDone="html") {
 		$transitions=Transition::getConstantValues();
-		$transition=$transitions[\rand(0,\sizeof($transitions)-1)];
-		return self::__callStatic($transition, [$responseElement,$jqueryDone]);
+		$transition=$transitions[\rand(0, \sizeof($transitions)-1)];
+		return self::__callStatic($transition, [$responseElement, $jqueryDone]);
 	}
 
-	public static function __callStatic($name, $arguments){
-		if(\sizeof($arguments)==2){
+	public static function __callStatic($name, $arguments) {
+		if (\sizeof($arguments)==2) {
 			$responseElement=$arguments[0];
 			$jqueryDone=$arguments[1];
 			$name=JString::camelCaseToSeparated($name);