phpMv / phpMv-UI

Ajax/semantic/html/modules/checkbox/AbstractCheckbox.php

Braces +13 added lines, -9 removed lines

@@ -15,14 +15,15 @@ 
 		$field=new \Ajax\common\html\html5\HtmlInput($identifier, $inputType, $value);
 		$field->setProperty("name", $name);
 		$this->setField($field);
-		if (isset($label))
-			$this->setLabel($label,$value);
+		if (isset($label)) {
+					$this->setLabel($label,$value);
+		}
 	}
 
 	public function setChecked($value=true){
 		if($value===true){
 			$this->getField()->setProperty("checked", "checked");
-		}else{
+		} else{
 			$this->getField()->removeProperty("checked");
 		}
 		return $this;
@@ -38,8 +39,9 @@ 
 			$labelO=new HtmlSemDoubleElement("", "label", "");
 			$labelO->setContent($label);
 			$labelO->setProperty("for", $this->getField()->getIdentifier());
-			if(isset($value))
-				$labelO->setProperty("data-value", $value);
+			if(isset($value)) {
+							$labelO->setProperty("data-value", $value);
+			}
 		}
 		$this->content["label"]=$labelO;
 	}
@@ -53,8 +55,9 @@ 
 	 * @return mixed
 	 */
 	public function getLabel() {
-		if (\array_key_exists("label", $this->content))
-			return $this->content["label"];
+		if (\array_key_exists("label", $this->content)) {
+					return $this->content["label"];
+		}
 	}
 
 	/**
@@ -129,8 +132,9 @@ 
 	}
 
 	public function run(JsUtils $js) {
-		if(!isset($this->_bsComponent))
-			$this->_bsComponent=$js->semantic()->checkbox("#" . $this->identifier, $this->_params);
+		if(!isset($this->_bsComponent)) {
+					$this->_bsComponent=$js->semantic()->checkbox("#" . $this->identifier, $this->_params);
+		}
 		return parent::run($js);
 	}
 }

Spacing +13 added lines, -13 removed lines

@@ -16,13 +16,13 @@ 
 		$field->setProperty("name", $name);
 		$this->setField($field);
 		if (isset($label))
-			$this->setLabel($label,$value);
+			$this->setLabel($label, $value);
 	}
 
-	public function setChecked($value=true){
-		if($value===true){
+	public function setChecked($value=true) {
+		if ($value===true) {
 			$this->getField()->setProperty("checked", "checked");
-		}else{
+		} else {
 			$this->getField()->removeProperty("checked");
 		}
 		return $this;
@@ -32,13 +32,13 @@ 
 		return $this->addToPropertyCtrl("class", $checkboxType, CheckboxType::getConstants());
 	}
 
-	public function setLabel($label,$value=null) {
+	public function setLabel($label, $value=null) {
 		$labelO=$label;
 		if (\is_string($label)) {
 			$labelO=new HtmlSemDoubleElement("", "label", "");
 			$labelO->setContent($label);
 			$labelO->setProperty("for", $this->getField()->getIdentifier());
-			if(isset($value))
+			if (isset($value))
 				$labelO->setProperty("data-value", $value);
 		}
 		$this->content["label"]=$labelO;
@@ -86,7 +86,7 @@ 
 	 * @return AbstractCheckbox
 	 */
 	public function attachEvent($selector, $action=NULL) {
-		if (isset($action)||\is_numeric($action)===true) {
+		if (isset($action) || \is_numeric($action)===true) {
 			$js='$("#%identifier%").checkbox("attach events", "'.$selector.'", "'.$action.'");';
 		} else {
 			$js='$("#%identifier%").checkbox("attach events", "'.$selector.'");';
@@ -102,7 +102,7 @@ 
 	 */
 	public function attachEvents($events=array()) {
 		if (\is_array($events)) {
-			foreach ( $events as $action => $selector ) {
+			foreach ($events as $action => $selector) {
 				$this->attachEvent($selector, $action);
 			}
 		}
@@ -113,24 +113,24 @@ 
 		return $this->addToProperty("class", "fitted");
 	}
 
-	public function setOnChecked($jsCode){
+	public function setOnChecked($jsCode) {
 		$this->_params["onChecked"]=$jsCode;
 		return $this;
 	}
 
-	public function setOnUnchecked($jsCode){
+	public function setOnUnchecked($jsCode) {
 		$this->_params["onUnchecked"]=$jsCode;
 		return $this;
 	}
 
-	public function setOnChange($jsCode){
+	public function setOnChange($jsCode) {
 		$this->_params["onChange"]=$jsCode;
 		return $this;
 	}
 
 	public function run(JsUtils $js) {
-		if(!isset($this->_bsComponent))
-			$this->_bsComponent=$js->semantic()->checkbox("#" . $this->identifier, $this->_params);
+		if (!isset($this->_bsComponent))
+			$this->_bsComponent=$js->semantic()->checkbox("#".$this->identifier, $this->_params);
 		return parent::run($js);
 	}
 }

Ajax/Bootstrap.php

Spacing +11 added lines, -11 removed lines

@@ -172,8 +172,8 @@ 
 	 * @param string $tagName container tagName
 	 * @return HtmlListgroup
 	 */
-	public function htmlListgroup($identifier,$items=array(),$tagName="ul"){
-		$listGroup=new HtmlListgroup($identifier,$tagName);
+	public function htmlListgroup($identifier, $items=array(), $tagName="ul") {
+		$listGroup=new HtmlListgroup($identifier, $tagName);
 		$listGroup->addItems($items);
 		return $this->addHtmlComponent($listGroup);
 	}
@@ -198,8 +198,8 @@ 
 	 * @param string $onClick
 	 * @return HtmlSplitbutton
 	 */
-	public function htmlSplitbutton($identifier,$value="", $items=array(), $cssStyle="btn-default", $onClick=NULL) {
-		return $this->addHtmlComponent(new HtmlSplitbutton($identifier, $value, $items, $cssStyle,$onClick));
+	public function htmlSplitbutton($identifier, $value="", $items=array(), $cssStyle="btn-default", $onClick=NULL) {
+		return $this->addHtmlComponent(new HtmlSplitbutton($identifier, $value, $items, $cssStyle, $onClick));
 	}
 
 	/**
@@ -207,7 +207,7 @@ 
 	 * @param string $identifier
 	 * @return HtmlInputgroup
 	 */
-	public function htmlInputgroup($identifier){
+	public function htmlInputgroup($identifier) {
 		return $this->addHtmlComponent(new HtmlInputgroup($identifier));
 	}
 
@@ -219,8 +219,8 @@ 
 	 * @param callable $hrefFunction the function who generates the href elements. default : function($e){return $e->getContent()}
 	 * @return HtmlBreadcrumbs
 	 */
-	public function htmlBreadcrumbs($identifier,$elements=array(),$autoActive=true,$startIndex=0,$hrefFunction=NULL){
-		return $this->addHtmlComponent(new HtmlBreadcrumbs($identifier,$elements,$autoActive,$startIndex,$hrefFunction));
+	public function htmlBreadcrumbs($identifier, $elements=array(), $autoActive=true, $startIndex=0, $hrefFunction=NULL) {
+		return $this->addHtmlComponent(new HtmlBreadcrumbs($identifier, $elements, $autoActive, $startIndex, $hrefFunction));
 	}
 
 	/**
@@ -232,8 +232,8 @@ 
 	 * @param int $active The active page
 	 * @return HtmlPagination
 	 */
-	public function htmlPagination($identifier,$from=1,$to=1,$active=NULL,$countVisible=NULL){
-		return $this->addHtmlComponent(new HtmlPagination($identifier,$from,$to,$active,$countVisible));
+	public function htmlPagination($identifier, $from=1, $to=1, $active=NULL, $countVisible=NULL) {
+		return $this->addHtmlComponent(new HtmlPagination($identifier, $from, $to, $active, $countVisible));
 	}
 
 	/**
@@ -244,7 +244,7 @@ 
 	 * @param int $numCols
 	 * @return HtmlGridSystem
 	 */
-	public function htmlGridSystem($identifier,$numRows=1,$numCols=NULL){
-		return $this->addHtmlComponent(new HtmlGridSystem($identifier,$numRows,$numCols));
+	public function htmlGridSystem($identifier, $numRows=1, $numCols=NULL) {
+		return $this->addHtmlComponent(new HtmlGridSystem($identifier, $numRows, $numCols));
 	}
 }

Ajax/ui/components/DialogButton.php

Spacing +3 added lines, -3 removed lines

@@ -15,7 +15,7 @@ 
 
 	private function addFunction($jsCode) {
 		if (!JString::startsWith($jsCode, "function"))
-			$jsCode="%function(){" . $jsCode . "}%";
+			$jsCode="%function(){".$jsCode."}%";
 		return $jsCode;
 	}
 
@@ -50,7 +50,7 @@ 
 	 * @param array $parameters default : array("params"=>"{}","jsCallback"=>NULL,"attr"=>"id","hasLoader"=>true,"jqueryDone"=>"html","ajaxTransition"=>null,"jsCondition"=>NULL,"headers"=>null)
 	 * @return DialogButton
 	 */
-	public static function submitButton(JsUtils $js, $url, $form, $responseElement, $caption="Okay",$parameters=[]) {
-		return new DialogButton($caption, $js->postFormDeferred($url, $form, $responseElement,$parameters) . ";$( this ).dialog( 'close' );");
+	public static function submitButton(JsUtils $js, $url, $form, $responseElement, $caption="Okay", $parameters=[]) {
+		return new DialogButton($caption, $js->postFormDeferred($url, $form, $responseElement, $parameters).";$( this ).dialog( 'close' );");
 	}
 }

Braces +3 added lines, -2 removed lines

@@ -14,8 +14,9 @@
 class DialogButton extends BaseComponent {
 
 	private function addFunction($jsCode) {
-		if (!JString::startsWith($jsCode, "function"))
-			$jsCode="%function(){" . $jsCode . "}%";
+		if (!JString::startsWith($jsCode, "function")) {
+					$jsCode="%function(){" . $jsCode . "}%";
+		}
 		return $jsCode;
 	}
 

Ajax/bootstrap/html/HtmlPagination.php

Spacing +34 added lines, -34 removed lines

@@ -45,70 +45,70 @@ 
 	/**
 	 * @param string $identifier
 	 */
-	public function __construct($identifier,$from=1,$to=1,$active=NULL,$countVisible=NULL){
-		parent::__construct($identifier,"ul");
+	public function __construct($identifier, $from=1, $to=1, $active=NULL, $countVisible=NULL) {
+		parent::__construct($identifier, "ul");
 		$this->setProperty("class", "pagination");
 		$this->active=$active;
 		$this->from=$from;
 		$this->to=$to;
 		$this->urlMask="%page%";
-		if(!isset($countVisible))
+		if (!isset($countVisible))
 			$this->countVisible=$to-$from+1;
 		else
 			$this->countVisible=$countVisible;
 		$this->createContent();
 	}
 
-	private function createElement($num,$content,$disabled=false,$current=false){
+	private function createElement($num, $content, $disabled=false, $current=false) {
 		$count=sizeof($this->content)+1;
-		$elem=new HtmlBsDoubleElement("li-".$this->identifier."-".$count,"li");
-		if($disabled){
+		$elem=new HtmlBsDoubleElement("li-".$this->identifier."-".$count, "li");
+		if ($disabled) {
 			$elem->setProperty("class", "disabled");
 		}
-		if($current){
+		if ($current) {
 			$content.="<span class='sr-only'>(current)</span>";
 			$elem->setProperty("class", "active");
 		}
-		if(!$disabled){
+		if (!$disabled) {
 			$url=$this->getUrl($num);
-			$href=new HtmlLink("a-".$this->identifier."-".$count,$url,$content);
+			$href=new HtmlLink("a-".$this->identifier."-".$count, $url, $content);
 			$href->setProperty($this->attr, $url);
 			$elem->setContent($href);
-		}else{
+		} else {
 			$elem->setContent($content);
 		}
 		$this->content[]=$elem;
 		return $this;
 	}
 
-	protected function createContent(){
+	protected function createContent() {
 		$this->content=array();
-		$this->createElement($this->active-1,"<span aria-hidden='true'>&laquo;</span>",$this->active===1);
+		$this->createElement($this->active-1, "<span aria-hidden='true'>&laquo;</span>", $this->active===1);
 		$start=$this->getStart();
-		$end=min($start+$this->countVisible-1,$this->to);
-		for($index=$start;$index<=$end;$index++){
-			$this->createElement($index,$index,false,$index===$this->active);
+		$end=min($start+$this->countVisible-1, $this->to);
+		for ($index=$start; $index<=$end; $index++) {
+			$this->createElement($index, $index, false, $index===$this->active);
 		}
-		$this->createElement($this->active+1,"<span aria-hidden='true'>&raquo;</span>",$this->active===$this->to);
+		$this->createElement($this->active+1, "<span aria-hidden='true'>&raquo;</span>", $this->active===$this->to);
 	}
 
-	protected function half(){
+	protected function half() {
 		return (int)($this->countVisible/2);
 	}
 
-	protected function getStart(){
+	protected function getStart() {
 		$result=1;
-		if($this->countVisible!==$this->to-$this->from+1){
-			$result=max($this->active-$this->half(),$result);
+		if ($this->countVisible!==$this->to-$this->from+1) {
+			$result=max($this->active-$this->half(), $result);
 		}
 		return $result;
 	}
 
 	public function _addEvent($event, $jsCode) {
-		foreach ($this->content as $li){
+		foreach ($this->content as $li) {
 			$content=$li->getContent();
-			if($content instanceof BaseHtml)
-				$content->_addEvent($event,$jsCode);
+			if ($content instanceof BaseHtml)
+				$content->_addEvent($event, $jsCode);
 		}
 	}
 	/**
@@ -117,14 +117,14 @@ 
 	 * @param object $dispatcher the request dispatcher
 	 * @return \Ajax\bootstrap\html\HtmlPagination
 	 */
-	public function fromDispatcher(JsUtils $js,$dispatcher,$startIndex=0){
+	public function fromDispatcher(JsUtils $js, $dispatcher, $startIndex=0) {
 		$items=$js->fromDispatcher($dispatcher);
 		$url=implode("/", $items);
-		if($this->urlMask==="%page%"){
+		if ($this->urlMask==="%page%") {
 			$this->urlMask=preg_replace("/[0-9]/", "%page%", $url);
 		}
-		for($index=$this->from;$index<=$this->to;$index++){
-			if($this->getUrl($index)==$url){
+		for ($index=$this->from; $index<=$this->to; $index++) {
+			if ($this->getUrl($index)==$url) {
 				$this->setActive($index);
 				break;
 			}
@@ -132,7 +132,7 @@ 
 		return $this;
 	}
 
-	public function getUrl($index){
+	public function getUrl($index) {
 		return str_ireplace("%page%", $index, $this->urlMask);
 	}
 
@@ -146,7 +146,7 @@ 
 		if (is_int($size)) {
 			return $this->addToPropertyUnique("class", CssRef::sizes("pagination")[$size], CssRef::sizes("pagination"));
 		}
-		if(!JString::startsWith($size, "pagination-") && $size!=="")
+		if (!JString::startsWith($size, "pagination-") && $size!=="")
 			$size="pagination-".$size;
 		return $this->addToPropertyCtrl("class", $size, CssRef::sizes("pagination"));
 	}
@@ -155,7 +155,7 @@ 
 		return $this->from;
 	}
 	public function setFrom($from) {
-		$this->from = $from;
+		$this->from=$from;
 		$this->createContent();
 		return $this;
 	}
@@ -163,7 +163,7 @@ 
 		return $this->to;
 	}
 	public function setTo($to) {
-		$this->to = $to;
+		$this->to=$to;
 		$this->createContent();
 		return $this;
 	}
@@ -171,7 +171,7 @@ 
 		return $this->active;
 	}
 	public function setActive($active) {
-		$this->active = $active;
+		$this->active=$active;
 		$this->createContent();
 		return $this;
 	}
@@ -179,7 +179,7 @@ 
 		return $this->urlMask;
 	}
 	public function setUrlMask($urlMask) {
-		$this->urlMask = $urlMask;
+		$this->urlMask=$urlMask;
 		$this->createContent();
 		return $this;
 	}
@@ -187,7 +187,7 @@ 
 		return $this->countVisible;
 	}
 	public function setCountVisible($countVisible) {
-		$this->countVisible = $countVisible;
+		$this->countVisible=$countVisible;
 		$this->createContent();
 		return $this;
 	}

Braces +12 added lines, -9 removed lines

@@ -52,10 +52,11 @@ 
 		$this->from=$from;
 		$this->to=$to;
 		$this->urlMask="%page%";
-		if(!isset($countVisible))
-			$this->countVisible=$to-$from+1;
-		else
-			$this->countVisible=$countVisible;
+		if(!isset($countVisible)) {
+					$this->countVisible=$to-$from+1;
+		} else {
+					$this->countVisible=$countVisible;
+		}
 		$this->createContent();
 	}
 
@@ -74,7 +75,7 @@ 
 			$href=new HtmlLink("a-".$this->identifier."-".$count,$url,$content);
 			$href->setProperty($this->attr, $url);
 			$elem->setContent($href);
-		}else{
+		} else{
 			$elem->setContent($content);
 		}
 		$this->content[]=$elem;
@@ -107,8 +108,9 @@ 
 	public function _addEvent($event, $jsCode) {
 		foreach ($this->content as $li){
 			$content=$li->getContent();
-			if($content instanceof BaseHtml)
-				$content->_addEvent($event,$jsCode);
+			if($content instanceof BaseHtml) {
+							$content->_addEvent($event,$jsCode);
+			}
 		}
 	}
 	/**
@@ -146,8 +148,9 @@ 
 		if (is_int($size)) {
 			return $this->addToPropertyUnique("class", CssRef::sizes("pagination")[$size], CssRef::sizes("pagination"));
 		}
-		if(!JString::startsWith($size, "pagination-") && $size!=="")
-			$size="pagination-".$size;
+		if(!JString::startsWith($size, "pagination-") && $size!=="") {
+					$size="pagination-".$size;
+		}
 		return $this->addToPropertyCtrl("class", $size, CssRef::sizes("pagination"));
 	}
 

Ajax/common/html/html5/HtmlUtils.php

Spacing +2 added lines, -2 removed lines

@@ -3,11 +3,11 @@
 namespace Ajax\common\html\html5;
 
 class HtmlUtils {
-	public static function javascriptInclude($url){
+	public static function javascriptInclude($url) {
 		return '<script src="'.$url.'"></script>';
 	}
 	
-	public static function stylesheetInclude($url){
+	public static function stylesheetInclude($url) {
 		return '<link rel="stylesheet" type="text/css" href="'.$url.'">';
 	}
 }

Ajax/common/html/traits/BaseHtmlPropertiesTrait.php

Spacing +12 added lines, -12 removed lines

@@ -8,12 +8,12 @@ 
  * @author jc
  * @property BaseWidget $_self
  */
-trait BaseHtmlPropertiesTrait{
+trait BaseHtmlPropertiesTrait {
 
-	protected $properties=array ();
+	protected $properties=array();
 	abstract protected function ctrl($name, $value, $typeCtrl);
 	abstract protected function removeOldValues(&$oldValue, $allValues);
-	abstract protected function _getElementBy($callback,$elements);
+	abstract protected function _getElementBy($callback, $elements);
 	public function getProperties() {
 		return $this->_self->properties;
 	}
@@ -45,14 +45,14 @@ 
 	 */
 	public function addToProperty($name, $value, $separator=" ") {
 		if (\is_array($value)) {
-			foreach ( $value as $v ) {
+			foreach ($value as $v) {
 				$this->_self->addToProperty($name, $v, $separator);
 			}
-		} else if ($value !== "" && $this->_self->propertyContains($name, $value) === false) {
-			if(isset($this->_self->properties[$name])){
+		} else if ($value!=="" && $this->_self->propertyContains($name, $value)===false) {
+			if (isset($this->_self->properties[$name])) {
 				$v=$this->_self->properties[$name];
-				if (isset($v) && $v !== "")
-					$v=$v . $separator . $value;
+				if (isset($v) && $v!=="")
+					$v=$v.$separator.$value;
 					else
 						$v=$value;
 	
@@ -91,7 +91,7 @@ 
 	}
 
 	public function addToPropertyCtrlCheck($name, $value, $typeCtrl) {
-		if ($this->_self->ctrl($name, $value, $typeCtrl) === true) {
+		if ($this->_self->ctrl($name, $value, $typeCtrl)===true) {
 			return $this->_self->addToProperty($name, $value);
 		}
 		return $this;
@@ -112,12 +112,12 @@ 
 	}
 
 	protected function setPropertyCtrl($name, $value, $typeCtrl) {
-		if ($this->_self->ctrl($name, $value, $typeCtrl) === true)
+		if ($this->_self->ctrl($name, $value, $typeCtrl)===true)
 			return $this->_self->setProperty($name, $value);
 			return $this;
 	}
 
-	protected function getElementByPropertyValue($propertyName,$value, $elements) {
-		return $this->_self->_getElementBy(function(BaseHtml $element) use ($propertyName,$value){return $element->propertyContains($propertyName, $value) === true;}, $elements);
+	protected function getElementByPropertyValue($propertyName, $value, $elements) {
+		return $this->_self->_getElementBy(function(BaseHtml $element) use ($propertyName, $value){return $element->propertyContains($propertyName, $value)===true; }, $elements);
 	}
 }

Braces +17 added lines, -12 removed lines

@@ -33,8 +33,9 @@ 
 	}
 
 	public function getProperty($name) {
-		if (array_key_exists($name, $this->_self->properties))
-			return $this->_self->properties[$name];
+		if (array_key_exists($name, $this->_self->properties)) {
+					return $this->_self->properties[$name];
+		}
 	}
 
 	/**
@@ -51,10 +52,11 @@ 
 		} else if ($value !== "" && $this->_self->propertyContains($name, $value) === false) {
 			if(isset($this->_self->properties[$name])){
 				$v=$this->_self->properties[$name];
-				if (isset($v) && $v !== "")
-					$v=$v . $separator . $value;
-					else
-						$v=$value;
+				if (isset($v) && $v !== "") {
+									$v=$v . $separator . $value;
+				} else {
+											$v=$value;
+					}
 	
 				return $this->_self->setProperty($name, $v);
 			}
@@ -78,8 +80,9 @@ 
 	}
 
 	protected function addToPropertyUnique($name, $value, $typeCtrl) {
-		if (is_string($typeCtrl) && @class_exists($typeCtrl, true))
-			$typeCtrl=$typeCtrl::getConstants();
+		if (is_string($typeCtrl) && @class_exists($typeCtrl, true)) {
+					$typeCtrl=$typeCtrl::getConstants();
+		}
 			if (\is_array($typeCtrl)) {
 				$this->_self->removeOldValues($this->_self->properties[$name], $typeCtrl);
 			}
@@ -98,8 +101,9 @@ 
 	}
 
 	public function removeProperty($name) {
-		if (\array_key_exists($name, $this->_self->properties))
-			unset($this->_self->properties[$name]);
+		if (\array_key_exists($name, $this->_self->properties)) {
+					unset($this->_self->properties[$name]);
+		}
 			return $this;
 	}
 
@@ -112,8 +116,9 @@ 
 	}
 
 	protected function setPropertyCtrl($name, $value, $typeCtrl) {
-		if ($this->_self->ctrl($name, $value, $typeCtrl) === true)
-			return $this->_self->setProperty($name, $value);
+		if ($this->_self->ctrl($name, $value, $typeCtrl) === true) {
+					return $this->_self->setProperty($name, $value);
+		}
 			return $this;
 	}
 

Ajax/bootstrap/html/HtmlPanel.php

Braces +6 added lines, -4 removed lines

@@ -89,8 +89,9 @@ 
 	 * @return \Ajax\bootstrap\html\HtmlPanel default : "panel-default"
 	 */
 	public function setStyle($cssStyle) {
-		if (!JString::startsWith($cssStyle, "panel"))
-			$cssStyle="panel".$cssStyle;
+		if (!JString::startsWith($cssStyle, "panel")) {
+					$cssStyle="panel".$cssStyle;
+		}
 		return $this->addToPropertyCtrl("class", $cssStyle, CssRef::Styles("panel"));
 	}
 
@@ -132,7 +133,8 @@ 
 	 * @return $this default : false
 	 */
 	public function show($value) {
-		if ($this->_collapsable)
-			$this->_showOnStartup=$value;
+		if ($this->_collapsable) {
+					$this->_showOnStartup=$value;
+		}
 	}
 }

Ajax/common/Widget.php

Braces +7 added lines, -5 removed lines

@@ -250,8 +250,9 @@ 
 	 */
 	public function addItemInToolbar($caption,$icon=NULL,$callback=NULL){
 		$result=$this->addInToolbar($caption,$callback);
-		if(isset($icon) && method_exists($result, "addIcon"))
-			$result->addIcon($icon);
+		if(isset($icon) && method_exists($result, "addIcon")) {
+					$result->addIcon($icon);
+		}
 		return $result;
 	}
 
@@ -265,7 +266,7 @@ 
 			foreach ($items as $icon=>$item){
 				$this->addItemInToolbar($item,$icon,$callback);
 			}
-		}else{
+		} else{
 			foreach ($items as $item){
 				$this->addItemInToolbar($item,null,$callback);
 			}
@@ -416,8 +417,9 @@ 
 	protected function _compileForm(){
 		if(isset($this->_form)){
 			$noValidate="";
-			if(\sizeof($this->_form->getValidationParams())>0)
-				$noValidate="novalidate";
+			if(\sizeof($this->_form->getValidationParams())>0) {
+							$noValidate="novalidate";
+			}
 			$this->wrapContent("<form class='ui form' id='frm-".$this->identifier."' name='frm-".$this->identifier."' ".$noValidate.">","</form>");
 		}
 	}

Spacing +86 added lines, -86 removed lines

@@ -20,7 +20,7 @@ 
 use Ajax\semantic\html\modules\HtmlModal;
 
 abstract class Widget extends HtmlDoubleElement {
-	use FieldAsTrait,FormTrait;
+	use FieldAsTrait, FormTrait;
 
 	/**
 	 * @var string classname
@@ -53,17 +53,17 @@ 
 	protected $_generated;
 
 
-	public function __construct($identifier,$model,$modelInstance=NULL) {
+	public function __construct($identifier, $model, $modelInstance=NULL) {
 		parent::__construct($identifier);
 		$this->_template="%wrapContentBefore%%content%%wrapContentAfter%";
 		$this->setModel($model);
-		if(isset($modelInstance)){
+		if (isset($modelInstance)) {
 			$this->show($modelInstance);
 		}
 		$this->_generated=false;
 	}
 
-	protected function _init($instanceViewer,$contentKey,$content,$edition){
+	protected function _init($instanceViewer, $contentKey, $content, $edition) {
 		$this->_instanceViewer=$instanceViewer;
 		$this->content=[$contentKey=>$content];
 		$this->_self=$content;
@@ -75,31 +75,31 @@ 
 	 * @param int|string $fieldName
 	 * @return int|string|boolean
 	 */
-	protected function _getIndex($fieldName){
+	protected function _getIndex($fieldName) {
 		$index=$fieldName;
-		if(\is_string($fieldName)){
+		if (\is_string($fieldName)) {
 			$fields=$this->_instanceViewer->getVisibleProperties();
 			$index=\array_search($fieldName, $fields);
 		}
 		return $index;
 	}
 
-	protected function _getFieldIdentifier($prefix,$name=""){
+	protected function _getFieldIdentifier($prefix, $name="") {
 		return $this->identifier."-{$prefix}-".$this->_instanceViewer->getIdentifier();
 	}
 
-	protected function _getFieldName($index){
+	protected function _getFieldName($index) {
 		return $this->_instanceViewer->getFieldName($index);
 	}
 
-	protected function _getFieldCaption($index){
+	protected function _getFieldCaption($index) {
 		return $this->_instanceViewer->getCaption($index);
 	}
 
-	abstract protected  function _setToolbarPosition($table,$captions=NULL);
+	abstract protected  function _setToolbarPosition($table, $captions=NULL);
 
-	public function show($modelInstance){
-		if(\is_array($modelInstance)){
+	public function show($modelInstance) {
+		if (\is_array($modelInstance)) {
 			$modelInstance=\json_decode(\json_encode($modelInstance), FALSE);
 		}
 		$this->_modelInstance=$modelInstance;
@@ -125,7 +125,7 @@ 
 
 	abstract public function getHtmlComponent();
 
-	public function setAttached($value=true){
+	public function setAttached($value=true) {
 		return $this->getHtmlComponent()->setAttached($value);
 	}
 
@@ -136,65 +136,65 @@ 
 	 * @param callable $callback function called after the field compilation
 	 * @return Widget
 	 */
-	public function afterCompile($index,$callback){
+	public function afterCompile($index, $callback) {
 		$index=$this->_getIndex($index);
 		$this->_instanceViewer->afterCompile($index, $callback);
 		return $this;
 	}
 
-	public function setColor($color){
+	public function setColor($color) {
 		return $this->getHtmlComponent()->setColor($color);
 	}
 
 
-	public function setCaptions($captions){
+	public function setCaptions($captions) {
 		$this->_instanceViewer->setCaptions($captions);
 		return $this;
 	}
 
-	public function setCaption($index,$caption){
+	public function setCaption($index, $caption) {
 		$this->_instanceViewer->setCaption($this->_getIndex($index), $caption);
 		return $this;
 	}
 
-	public function setFields($fields){
+	public function setFields($fields) {
 		$this->_instanceViewer->setVisibleProperties($fields);
 		return $this;
 	}
 
-	public function addField($field,$key=null){
-		$this->_instanceViewer->addField($field,$key);
+	public function addField($field, $key=null) {
+		$this->_instanceViewer->addField($field, $key);
 		return $this;
 	}
 
-	public function addFields($fields){
+	public function addFields($fields) {
 		$this->_instanceViewer->addFields($fields);
 		return $this;
 	}
 	
-	public function countFields(){
+	public function countFields() {
 		return $this->_instanceViewer->visiblePropertiesCount();
 	}
 
-	public function addMessage($attributes=NULL,$fieldName="message"){
+	public function addMessage($attributes=NULL, $fieldName="message") {
 		$this->_instanceViewer->addField($fieldName);
 		$count=$this->_instanceViewer->visiblePropertiesCount();
-		return $this->fieldAsMessage($count-1,$attributes);
+		return $this->fieldAsMessage($count-1, $attributes);
 	}
 
-	public function addErrorMessage(){
-		return $this->addMessage(["error"=>true],"message");
+	public function addErrorMessage() {
+		return $this->addMessage(["error"=>true], "message");
 	}
 
-	public function insertField($index,$field,$key=null){
+	public function insertField($index, $field, $key=null) {
 		$index=$this->_getIndex($index);
-		$this->_instanceViewer->insertField($index, $field,$key);
+		$this->_instanceViewer->insertField($index, $field, $key);
 		return $this;
 	}
 
-	public function insertInField($index,$field,$key=null){
+	public function insertInField($index, $field, $key=null) {
 		$index=$this->_getIndex($index);
-		$this->_instanceViewer->insertInField($index, $field,$key);
+		$this->_instanceViewer->insertInField($index, $field, $key);
 		return $this;
 	}
 
@@ -204,13 +204,13 @@ 
 	 * @param callable $callback function parameters are : $value : the field value, $instance : the active instance of model, $fieldIndex : the field index, $rowIndex : the row index
 	 * @return Widget
 	 */
-	public function setValueFunction($index,$callback){
+	public function setValueFunction($index, $callback) {
 		$index=$this->_getIndex($index);
 		$this->_instanceViewer->setValueFunction($index, $callback);
 		return $this;
 	}
 
-	public function setIdentifierFunction($callback){
+	public function setIdentifierFunction($callback) {
 		$this->_instanceViewer->setIdentifierFunction($callback);
 		return $this;
 	}
@@ -218,8 +218,8 @@ 
 	/**
 	 * @return \Ajax\semantic\html\collections\menus\HtmlMenu
 	 */
-	public function getToolbar(){
-		if(isset($this->_toolbar)===false){
+	public function getToolbar() {
+		if (isset($this->_toolbar)===false) {
 			$this->_toolbar=new HtmlMenu("toolbar-".$this->identifier);
 		}
 		return $this->_toolbar;
@@ -231,15 +231,15 @@ 
 	 * @param callable $callback function to call on $element
 	 * @return \Ajax\common\html\HtmlDoubleElement
 	 */
-	public function addInToolbar($element,$callback=NULL){
+	public function addInToolbar($element, $callback=NULL) {
 		$tb=$this->getToolbar();
-		if($element instanceof BaseWidget){
-			if($element->getIdentifier()===""){
+		if ($element instanceof BaseWidget) {
+			if ($element->getIdentifier()==="") {
 				$element->setIdentifier("tb-item-".$this->identifier."-".$tb->count());
 			}
 		}
-		if(isset($callback)){
-			if(\is_callable($callback)){
+		if (isset($callback)) {
+			if (\is_callable($callback)) {
 				$callback($element);
 			}
 		}
@@ -252,9 +252,9 @@ 
 	 * @param callable $callback function($element)
 	 * @return \Ajax\common\html\HtmlDoubleElement
 	 */
-	public function addItemInToolbar($caption,$icon=NULL,$callback=NULL){
-		$result=$this->addInToolbar($caption,$callback);
-		if(isset($icon) && method_exists($result, "addIcon"))
+	public function addItemInToolbar($caption, $icon=NULL, $callback=NULL) {
+		$result=$this->addInToolbar($caption, $callback);
+		if (isset($icon) && method_exists($result, "addIcon"))
 			$result->addIcon($icon);
 		return $result;
 	}
@@ -264,14 +264,14 @@ 
 	 * @param callable $callback function($element)
 	 * @return \Ajax\common\Widget
 	 */
-	public function addItemsInToolbar(array $items,$callback=NULL){
-		if(JArray::isAssociative($items)){
-			foreach ($items as $icon=>$item){
-				$this->addItemInToolbar($item,$icon,$callback);
+	public function addItemsInToolbar(array $items, $callback=NULL) {
+		if (JArray::isAssociative($items)) {
+			foreach ($items as $icon=>$item) {
+				$this->addItemInToolbar($item, $icon, $callback);
 			}
-		}else{
-			foreach ($items as $item){
-				$this->addItemInToolbar($item,null,$callback);
+		} else {
+			foreach ($items as $item) {
+				$this->addItemInToolbar($item, null, $callback);
 			}
 		}
 		return $this;
@@ -283,12 +283,12 @@ 
 	 * @param callable $callback function($element)
 	 * @return \Ajax\common\html\HtmlDoubleElement
 	 */
-	public function addDropdownInToolbar($value,$items,$callback=NULL){
+	public function addDropdownInToolbar($value, $items, $callback=NULL) {
 		$dd=$value;
 		if (\is_string($value)) {
-			$dd=new HtmlDropdown("dropdown-". $this->identifier."-".$value, $value, $items);
+			$dd=new HtmlDropdown("dropdown-".$this->identifier."-".$value, $value, $items);
 		}
-		return $this->addInToolbar($dd,$callback);
+		return $this->addInToolbar($dd, $callback);
 	}
 
 	/**
@@ -297,9 +297,9 @@ 
 	 * @param callable $callback function($element)
 	 * @return \Ajax\common\html\HtmlDoubleElement
 	 */
-	public function addButtonInToolbar($caption,$cssStyle=null,$callback=NULL){
-		$bt=new HtmlButton("bt-".$caption,$caption,$cssStyle);
-		return $this->addInToolbar($bt,$callback);
+	public function addButtonInToolbar($caption, $cssStyle=null, $callback=NULL) {
+		$bt=new HtmlButton("bt-".$caption, $caption, $cssStyle);
+		return $this->addInToolbar($bt, $callback);
 	}
 
 	/**
@@ -308,9 +308,9 @@ 
 	 * @param callable $callback function($element)
 	 * @return \Ajax\common\html\HtmlDoubleElement
 	 */
-	public function addButtonsInToolbar(array $captions,$asIcon=false,$callback=NULL){
-		$bts=new HtmlButtonGroups("",$captions,$asIcon);
-		return $this->addInToolbar($bts,$callback);
+	public function addButtonsInToolbar(array $captions, $asIcon=false, $callback=NULL) {
+		$bts=new HtmlButtonGroups("", $captions, $asIcon);
+		return $this->addInToolbar($bts, $callback);
 	}
 
 	/**
@@ -320,15 +320,15 @@ 
 	 * @param boolean $labeled
 	 * @return \Ajax\common\html\HtmlDoubleElement
 	 */
-	public function addLabelledIconButtonInToolbar($caption,$icon,$before=true,$labeled=false){
-		$bt=new HtmlButton("",$caption);
-		$bt->addIcon($icon,$before,$labeled);
+	public function addLabelledIconButtonInToolbar($caption, $icon, $before=true, $labeled=false) {
+		$bt=new HtmlButton("", $caption);
+		$bt->addIcon($icon, $before, $labeled);
 		return $this->addInToolbar($bt);
 	}
 
-	public function addSubmitInToolbar($identifier,$value,$cssStyle=NULL,$url=NULL,$responseElement=NULL,$parameters=NULL){
-		$button=new HtmlButton($identifier,$value,$cssStyle);
-		$this->_buttonAsSubmit($button,"click",$url,$responseElement,$parameters);
+	public function addSubmitInToolbar($identifier, $value, $cssStyle=NULL, $url=NULL, $responseElement=NULL, $parameters=NULL) {
+		$button=new HtmlButton($identifier, $value, $cssStyle);
+		$this->_buttonAsSubmit($button, "click", $url, $responseElement, $parameters);
 		return $this->addInToolbar($button);
 	}
 
@@ -358,7 +358,7 @@ 
 	 * @param callable $defaultValueFunction function parameters are : $name : the field name, $value : the field value ,$index : the field index, $instance : the active instance of model
 	 * @return \Ajax\common\Widget
 	 */
-	public function setDefaultValueFunction($defaultValueFunction){
+	public function setDefaultValueFunction($defaultValueFunction) {
 		$this->_instanceViewer->setDefaultValueFunction($defaultValueFunction);
 		return $this;
 	}
@@ -366,7 +366,7 @@ 
 	/**
 	 * @return callable
 	 */
-	public function getDefaultValueFunction(){
+	public function getDefaultValueFunction() {
 		return $this->_instanceViewer->getDefaultValueFunction();
 	}
 
@@ -374,7 +374,7 @@ 
 	 * @param string|boolean $disable
 	 * @return string
 	 */
-	public function jsDisabled($disable=true){
+	public function jsDisabled($disable=true) {
 		return "$('#".$this->identifier." .ui.input,#".$this->identifier." .ui.dropdown,#".$this->identifier." .ui.checkbox').toggleClass('disabled',".$disable.");";
 	}
 
@@ -383,12 +383,12 @@ 
 	 * @param callable $callback function($element)
 	 * @return \Ajax\common\html\HtmlDoubleElement
 	 */
-	public function addEditButtonInToolbar($caption,$callback=NULL){
-		$bt=new HtmlButton($this->identifier."-editBtn",$caption);
+	public function addEditButtonInToolbar($caption, $callback=NULL) {
+		$bt=new HtmlButton($this->identifier."-editBtn", $caption);
 		$bt->setToggle();
 		$bt->setActive($this->_edition);
 		$bt->onClick($this->jsDisabled(Javascript::prep_value("!$(event.target).hasClass('active')")));
-		return $this->addInToolbar($bt,$callback);
+		return $this->addInToolbar($bt, $callback);
 	}
 
 	public function setToolbar(HtmlMenu $_toolbar) {
@@ -402,34 +402,34 @@ 
 	}
 
 	public function getForm() {
-		if(!isset($this->_form)){
+		if (!isset($this->_form)) {
 			$this->_form=new HtmlForm("frm-".$this->identifier);
 			$this->setEdition(true);
 		}
 		return $this->_form;
 	}
 
-	public function run(JsUtils $js){
+	public function run(JsUtils $js) {
 		parent::run($js);
-		if(isset($this->_form)){
+		if (isset($this->_form)) {
 			$this->runForm($js);
 		}
 	}
 
-	protected function runForm(JsUtils $js){
+	protected function runForm(JsUtils $js) {
 		$fields=$this->getContentInstances(HtmlFormField::class);
-		foreach ($fields as $field){
+		foreach ($fields as $field) {
 			$this->_form->addField($field);
 		}
 		return $this->_form->run($js);
 	}
 
-	protected function _compileForm(){
-		if(isset($this->_form)){
+	protected function _compileForm() {
+		if (isset($this->_form)) {
 			$noValidate="";
-			if(\sizeof($this->_form->getValidationParams())>0)
+			if (\sizeof($this->_form->getValidationParams())>0)
 				$noValidate="novalidate";
-			$this->wrapContent("<form class='ui form' id='frm-".$this->identifier."' name='frm-".$this->identifier."' ".$noValidate.">","</form>");
+			$this->wrapContent("<form class='ui form' id='frm-".$this->identifier."' name='frm-".$this->identifier."' ".$noValidate.">", "</form>");
 		}
 	}
 
@@ -444,32 +444,32 @@ 
 		return $this;
 	}
 
-	public function moveFieldTo($from,$to){
+	public function moveFieldTo($from, $to) {
 		return $this->_instanceViewer->moveFieldTo($from, $to);
 	}
 
-	public function swapFields($index1,$index2){
+	public function swapFields($index1, $index2) {
 		$index1=$this->_getIndex($index1);
 		$index2=$this->_getIndex($index2);
 		return $this->_instanceViewer->swapFields($index1, $index2);
 	}
 
-	public function removeField($index){
+	public function removeField($index) {
 		$index=$this->_getIndex($index);
 		$this->_instanceViewer->removeField($index);
 		return $this;
 	}
 
-	public function asModal($header=null){
-		$modal=new HtmlModal("modal-".$this->identifier,$header);
+	public function asModal($header=null) {
+		$modal=new HtmlModal("modal-".$this->identifier, $header);
 		$modal->setContent($this);
-		if(isset($this->_form)){
+		if (isset($this->_form)) {
 			$this->_form->onSuccess($modal->jsHide());
 		}
 		return $modal;
 	}
 
 	public function addToProperty($name, $value, $separator=" ") {
-		return $this->getHtmlComponent()->addToProperty($name,$value,$separator);
+		return $this->getHtmlComponent()->addToProperty($name, $value, $separator);
 	}
 }

Ajax/semantic/widgets/datatable/DataTableFieldAsTrait.php

Braces +12 added lines, -8 removed lines

@@ -63,15 +63,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;
 	}
 
@@ -88,8 +90,9 @@ 
 		return $this->_fieldAs(function($id,$name,$value,$caption) use ($url,$responseElement,$cssStyle,$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);
 	}
@@ -160,8 +163,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;
 	}
 

Spacing +68 added lines, -68 removed lines

@@ -14,15 +14,15 @@ 
  * @property boolean $_visibleHover
  * @property InstanceViewer $_instanceViewer
  */
-trait DataTableFieldAsTrait{
-	protected $_buttons=["display","edit","delete"];
+trait DataTableFieldAsTrait {
+	protected $_buttons=["display", "edit", "delete"];
 	protected $_buttonsColumn;
-	abstract public function addField($field,$key=null);
-	abstract public function insertField($index,$field,$key=null);
-	abstract public function insertInField($index,$field,$key=null);
-	abstract public function fieldAs($index,$type,$attributes=NULL);
+	abstract public function addField($field, $key=null);
+	abstract public function insertField($index, $field, $key=null);
+	abstract public function insertInField($index, $field, $key=null);
+	abstract public function fieldAs($index, $type, $attributes=NULL);
 	abstract protected function cleanIdentifier($id);
-	abstract protected function _fieldAs($elementCallback,&$index,$attributes=NULL,$prefix=null);
+	abstract protected function _fieldAs($elementCallback, &$index, $attributes=NULL, $prefix=null);
 	
 	/**
 	 * @param string $caption
@@ -30,8 +30,8 @@ 
 	 * @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);
 	}
 
 	/**
@@ -40,20 +40,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;
@@ -65,16 +65,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;
 	}
@@ -88,19 +88,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,$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, $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;");
 	}
 
 	/**
@@ -110,8 +110,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;
 	}
 
@@ -122,8 +122,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;
 	}
 
@@ -134,8 +134,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;
 	}
 
@@ -146,25 +146,25 @@ 
 	 * @param callable $callback
 	 * @return DataTable
 	 */
-	public function insertInFieldButton($index,$caption,$visibleHover=true,$callback=null,$key=null){
-		$this->insertInField($index, $this->getFieldButtonCallable($caption,$visibleHover,$callback),$key);
+	public function insertInFieldButton($index, $caption, $visibleHover=true, $callback=null, $key=null) {
+		$this->insertInField($index, $this->getFieldButtonCallable($caption, $visibleHover, $callback), $key);
 		return $this;
 	}
 
-	private function addDefaultButton($icon,$class=null,$visibleHover=true,$callback=null,$key=null){
-		$this->addField($this->getCallable("getDefaultButton",[$icon,$class,$visibleHover],$callback),$key);
+	private function addDefaultButton($icon, $class=null, $visibleHover=true, $callback=null, $key=null) {
+		$this->addField($this->getCallable("getDefaultButton", [$icon, $class, $visibleHover], $callback), $key);
 		return $this;
 	}
 
-	public function insertDefaultButtonIn($index,$icon,$class=null,$visibleHover=true,$callback=null,$key=null){
-		$this->insertInField($index,$this->getCallable("getDefaultButton",[$icon,$class,$visibleHover],$callback),$key);
+	public function insertDefaultButtonIn($index, $icon, $class=null, $visibleHover=true, $callback=null, $key=null) {
+		$this->insertInField($index, $this->getCallable("getDefaultButton", [$icon, $class, $visibleHover], $callback), $key);
 		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;
 	}
@@ -176,9 +176,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,"delete");
+		return $this->addDefaultButton("remove", "_delete red basic", $visibleHover, $callback, "delete");
 	}
 
 	/**
@@ -188,9 +188,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,"edit");
+		return $this->addDefaultButton("edit", "_edit basic", $visibleHover, $callback, "edit");
 	}
 	
 	/**
@@ -200,9 +200,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 addDisplayButton($visibleHover=true,$displayBehavior=[],$callback=null){
+	public function addDisplayButton($visibleHover=true, $displayBehavior=[], $callback=null) {
 		$this->_displayBehavior=$displayBehavior;
-		return $this->addDefaultButton("eye","_display basic",$visibleHover,$callback,"display");
+		return $this->addDefaultButton("eye", "_display basic", $visibleHover, $callback, "display");
 	}
 
 	/**
@@ -213,10 +213,10 @@ 
 	 * @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;
 	}
 	
@@ -229,35 +229,35 @@ 
 	 * @param callable $callbackDelete this function takes the following arguments : $object=>the delete button, $instance : the active instance of the object
 	 * @return DataTable
 	 */
-	public function addAllButtons($visibleHover=true,$behavior=[],$callbackDisplay=null,$callbackEdit=null,$callbackDelete=null){
-		$this->addDisplayButton($visibleHover,$behavior,$callbackDisplay);
+	public function addAllButtons($visibleHover=true, $behavior=[], $callbackDisplay=null, $callbackEdit=null, $callbackDelete=null) {
+		$this->addDisplayButton($visibleHover, $behavior, $callbackDisplay);
 		$index=$this->_instanceViewer->visiblePropertiesCount()-1;
 		$this->_buttonsColumn=$index;
-		$this->insertEditButtonIn($index,$visibleHover,$behavior,$callbackEdit);
-		$this->insertDeleteButtonIn($index,$visibleHover,$behavior,$callbackDelete);
+		$this->insertEditButtonIn($index, $visibleHover, $behavior, $callbackEdit);
+		$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,"delete");
+		return $this->insertDefaultButtonIn($index, "remove", "_delete red basic", $visibleHover, $callback, "delete");
 	}
 
-	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,"edit");
+		return $this->insertDefaultButtonIn($index, "edit", "_edit basic", $visibleHover, $callback, "edit");
 	}
 	
-	public function insertDisplayButtonIn($index,$visibleHover=true,$displayBehavior=[],$callback=null){
+	public function insertDisplayButtonIn($index, $visibleHover=true, $displayBehavior=[], $callback=null) {
 		$this->_displayBehavior=$displayBehavior;
-		return $this->insertDefaultButtonIn($index,"eye","_display basic",$visibleHover,$callback,"display");
+		return $this->insertDefaultButtonIn($index, "eye", "_display basic", $visibleHover, $callback, "display");
 	}
 	
 	/**
 	 * @param multitype:string  $_buttons
 	 */
 	public function setButtons($_buttons) {
-		$this->_buttons = $_buttons;
+		$this->_buttons=$_buttons;
 		return $this;
 	}
 }