phpMv / phpMv-UI

Ajax/semantic/components/Form.php

Doc Comments +3 added lines

@@ -13,6 +13,9 @@
  */
 class HtmlIconGroups extends HtmlSemCollection {
 
+	/**
+	 * @param string $identifier
+	 */
 	public function __construct($identifier, $icons=array(), $size="") {
 		parent::__construct($identifier, "i", "icons");
 		$this->addItems($icons);

Spacing +10 added lines, -10 removed lines

@@ -20,15 +20,15 @@ 
 		$this->params["fields"]=[];
 	}
 
-	public function addField($identifier){
+	public function addField($identifier) {
 		$this->params["fields"][$identifier]=new FieldValidation($identifier);
 	}
 
-	public function setInline($value){
+	public function setInline($value) {
 		return $this->setParam("inline", true);
 	}
 
-	public function setOn($value){
+	public function setOn($value) {
 		return $this->setParam("on", $value);
 	}
 
@@ -40,33 +40,33 @@ 
 	 * @param mixed $value
 	 * @param string|NULL $prompt
 	 */
-	public function addFieldRule($identifier,$type,$prompt=NULL,$value=NULL){
-		if(isset($this->params["fields"][$identifier])===false){
+	public function addFieldRule($identifier, $type, $prompt=NULL, $value=NULL) {
+		if (isset($this->params["fields"][$identifier])===false) {
 			$this->addField($identifier);
 		}
-		$this->params["fields"][$identifier]->addRule($type,$prompt,$value);
+		$this->params["fields"][$identifier]->addRule($type, $prompt, $value);
 	}
 
 	/**
 	 * @param FieldValidation $fieldValidation
 	 */
-	public function addFieldValidation($fieldValidation){
+	public function addFieldValidation($fieldValidation) {
 		$this->params["fields"][$fieldValidation->getIdentifier()]=$fieldValidation;
 	}
 
-	public function setJs(JsUtils $js){
+	public function setJs(JsUtils $js) {
 		$this->js=$js;
 	}
 
 	public function getScript() {
 		$allParams=$this->params;
-		$this->jquery_code_for_compile=array ();
+		$this->jquery_code_for_compile=array();
 		$this->jquery_code_for_compile []="$( \"".$this->attachTo."\" ).{$this->uiName}(".$this->getParamsAsJSON($allParams).");";
 		$this->compileEvents();
 		return $this->compileJQueryCode();
 	}
 
-	public function onValid($jsCode){
+	public function onValid($jsCode) {
 		$this->addComponentEvent("onValid", $jsCode);
 	}
 }
\ No newline at end of file

Ajax/common/html/BaseHtml.php

Spacing +62 added lines, -62 removed lines

@@ -16,10 +16,10 @@ 
 abstract class BaseHtml extends BaseWidget {
 	protected $_template;
 	protected $tagName;
-	protected $properties=array ();
-	protected $_events=array ();
-	protected $_wrapBefore=array ();
-	protected $_wrapAfter=array ();
+	protected $properties=array();
+	protected $_events=array();
+	protected $_wrapBefore=array();
+	protected $_wrapAfter=array();
 	protected $_bsComponent;
 
 	public function getBsComponent() {
@@ -32,7 +32,7 @@ 
 	}
 
 	protected function getTemplate(JsUtils $js=NULL) {
-		return PropertyWrapper::wrap($this->_wrapBefore, $js) . $this->_template . PropertyWrapper::wrap($this->_wrapAfter, $js);
+		return PropertyWrapper::wrap($this->_wrapBefore, $js).$this->_template.PropertyWrapper::wrap($this->_wrapAfter, $js);
 	}
 
 	public function getProperties() {
@@ -56,13 +56,13 @@ 
 
 	public function addToProperty($name, $value, $separator=" ") {
 		if (\is_array($value)) {
-			foreach ( $value as $v ) {
+			foreach ($value as $v) {
 				$this->addToProperty($name, $v, $separator);
 			}
-		} else if ($value !== "" && $this->propertyContains($name, $value) === false) {
+		} else if ($value!=="" && $this->propertyContains($name, $value)===false) {
 			$v=@$this->properties[$name];
-			if (isset($v) && $v !== "")
-				$v=$v . $separator . $value;
+			if (isset($v) && $v!=="")
+				$v=$v.$separator.$value;
 			else
 				$v=$value;
 
@@ -78,19 +78,19 @@ 
 
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
 		$result=$this->getTemplate($js);
-		foreach ( $this as $key => $value ) {
-			if (JString::startswith($key, "_") === false && $key !== "events") {
+		foreach ($this as $key => $value) {
+			if (JString::startswith($key, "_")===false && $key!=="events") {
 				if (is_array($value)) {
 					$v=PropertyWrapper::wrap($value, $js);
 				} else {
 					$v=$value;
 				}
-				$result=str_ireplace("%" . $key . "%", $v, $result);
+				$result=str_ireplace("%".$key."%", $v, $result);
 			}
 		}
 		if (isset($js)===true) {
 			$this->run($js);
-			if (isset($view) === true) {
+			if (isset($view)===true) {
 				$js->addViewElement($this->identifier, $result, $view);
 			}
 		}
@@ -99,12 +99,12 @@ 
 
 	protected function ctrl($name, $value, $typeCtrl) {
 		if (is_array($typeCtrl)) {
-			if (array_search($value, $typeCtrl) === false) {
-				throw new \Exception("La valeur passée `" . $value . "` à la propriété `" . $name . "` ne fait pas partie des valeurs possibles : {" . implode(",", $typeCtrl) . "}");
+			if (array_search($value, $typeCtrl)===false) {
+				throw new \Exception("La valeur passée `".$value."` à la propriété `".$name."` ne fait pas partie des valeurs possibles : {".implode(",", $typeCtrl)."}");
 			}
 		} else {
 			if (!$typeCtrl($value)) {
-				throw new \Exception("La fonction " . $typeCtrl . " a retourné faux pour l'affectation de la propriété " . $name);
+				throw new \Exception("La fonction ".$typeCtrl." a retourné faux pour l'affectation de la propriété ".$name);
 			}
 		}
 		return true;
@@ -119,13 +119,13 @@ 
 	}
 
 	protected function setPropertyCtrl($name, $value, $typeCtrl) {
-		if ($this->ctrl($name, $value, $typeCtrl) === true)
+		if ($this->ctrl($name, $value, $typeCtrl)===true)
 			return $this->setProperty($name, $value);
 		return $this;
 	}
 
 	protected function setMemberCtrl(&$name, $value, $typeCtrl) {
-		if ($this->ctrl($name, $value, $typeCtrl) === true) {
+		if ($this->ctrl($name, $value, $typeCtrl)===true) {
 			return $name=$value;
 		}
 		return $this;
@@ -134,7 +134,7 @@ 
 	protected function addToMemberUnique(&$name, $value, $typeCtrl, $separator=" ") {
 		if (is_array($typeCtrl)) {
 			$this->removeOldValues($name, $typeCtrl);
-			$name.=$separator . $value;
+			$name.=$separator.$value;
 		}
 		return $this;
 	}
@@ -156,17 +156,17 @@ 
 	}
 
 	protected function addToMemberCtrl(&$name, $value, $typeCtrl, $separator=" ") {
-		if ($this->ctrl($name, $value, $typeCtrl) === true) {
+		if ($this->ctrl($name, $value, $typeCtrl)===true) {
 			if (is_array($typeCtrl)) {
 				$this->removeOldValues($name, $typeCtrl);
 			}
-			$name.=$separator . $value;
+			$name.=$separator.$value;
 		}
 		return $this;
 	}
 
 	protected function addToMember(&$name, $value, $separator=" ") {
-		$name=str_ireplace($value, "", $name) . $separator . $value;
+		$name=str_ireplace($value, "", $name).$separator.$value;
 		return $this;
 	}
 
@@ -184,7 +184,7 @@ 
 	}
 
 	public function addToPropertyCtrlCheck($name, $value, $typeCtrl) {
-		if ($this->ctrl($name, $value, $typeCtrl) === true) {
+		if ($this->ctrl($name, $value, $typeCtrl)===true) {
 			return $this->addToProperty($name, $value);
 		}
 		return $this;
@@ -212,28 +212,28 @@ 
 	}
 
 	public function fromArray($array) {
-		foreach ( $this as $key => $value ) {
+		foreach ($this as $key => $value) {
 			if (array_key_exists($key, $array) && !JString::startswith($key, "_")) {
-				$setter="set" . ucfirst($key);
+				$setter="set".ucfirst($key);
 				$this->$setter($array[$key]);
 				unset($array[$key]);
 			}
 		}
-		foreach ( $array as $key => $value ) {
+		foreach ($array as $key => $value) {
 			if (method_exists($this, $key)) {
 				try {
 					$this->$key($value);
 					unset($array[$key]);
-				} catch ( \Exception $e ) {
+				}catch (\Exception $e) {
 					// Nothing to do
 				}
 			} else {
-				$setter="set" . ucfirst($key);
+				$setter="set".ucfirst($key);
 				if (method_exists($this, $setter)) {
 					try {
 						$this->$setter($value);
 						unset($array[$key]);
-					} catch ( \Exception $e ) {
+					}catch (\Exception $e) {
 						// Nothing to do
 					}
 				}
@@ -244,7 +244,7 @@ 
 
 	public function fromDatabaseObjects($objects, $function) {
 		if (isset($objects)) {
-			foreach ( $objects as $object ) {
+			foreach ($objects as $object) {
 				$this->fromDatabaseObject($object, $function);
 			}
 		}
@@ -264,11 +264,11 @@ 
 	}
 
 	public function addEvent($event, $jsCode, $stopPropagation=false, $preventDefault=false) {
-		if ($stopPropagation === true) {
-			$jsCode="event.stopPropagation();" . $jsCode;
+		if ($stopPropagation===true) {
+			$jsCode="event.stopPropagation();".$jsCode;
 		}
-		if ($preventDefault === true) {
-			$jsCode="event.preventDefault();" . $jsCode;
+		if ($preventDefault===true) {
+			$jsCode="event.preventDefault();".$jsCode;
 		}
 		return $this->_addEvent($event, $jsCode);
 	}
@@ -278,7 +278,7 @@ 
 			if (is_array($this->_events[$event])) {
 				$this->_events[$event][]=$jsCode;
 			} else {
-				$this->_events[$event]=array ($this->_events[$event],$jsCode );
+				$this->_events[$event]=array($this->_events[$event], $jsCode);
 			}
 		} else {
 			$this->_events[$event]=$jsCode;
@@ -298,35 +298,35 @@ 
 		return $this->onClick($jsCode);
 	}
 
-	public function onCreate($jsCode){
-		if(isset($this->_events["_create"])){
+	public function onCreate($jsCode) {
+		if (isset($this->_events["_create"])) {
 			$this->_events["_create"][]=$jsCode;
-		}else{
+		} else {
 			$this->_events["_create"]=[$jsCode];
 		}
 		return $this;
 	}
 
 	public function addEventsOnRun(JsUtils $js) {
-		if(isset($this->_events["_create"])){
+		if (isset($this->_events["_create"])) {
 			$create=$this->_events["_create"];
-			if(\is_array($create)){
+			if (\is_array($create)) {
 				$create=\implode("", $create);
 			}
-			if($create!=="")
-				$js->exec($create,true);
+			if ($create!=="")
+				$js->exec($create, true);
 			unset($this->_events["_create"]);
 		}
 		if (isset($this->_bsComponent)) {
-			foreach ( $this->_events as $event => $jsCode ) {
+			foreach ($this->_events as $event => $jsCode) {
 				$code=$jsCode;
 				if (is_array($jsCode)) {
 					$code="";
-					foreach ( $jsCode as $jsC ) {
+					foreach ($jsCode as $jsC) {
 						if ($jsC instanceof AjaxCall) {
-							$code.="\n" . $jsC->compile($js);
+							$code.="\n".$jsC->compile($js);
 						} else {
-							$code.="\n" . $jsC;
+							$code.="\n".$jsC;
 						}
 					}
 				} elseif ($jsCode instanceof AjaxCall) {
@@ -334,12 +334,12 @@ 
 				}
 				$this->_bsComponent->addEvent($event, $code);
 			}
-			$this->_events=array ();
+			$this->_events=array();
 		}
 	}
 
 	public function _ajaxOn($operation, $event, $url, $responseElement="", $parameters=array()) {
-		$params=array ("url" => $url,"responseElement" => $responseElement );
+		$params=array("url" => $url, "responseElement" => $responseElement);
 		$params=array_merge($params, $parameters);
 		$this->_addEvent($event, new AjaxCall($operation, $params));
 		return $this;
@@ -375,33 +375,33 @@ 
 		if (is_array($elements)) {
 			$flag=false;
 			$index=0;
-			while ( !$flag && $index < sizeof($elements) ) {
+			while (!$flag && $index<sizeof($elements)) {
 				if ($elements[$index] instanceof BaseHtml)
-					$flag=($elements[$index]->getIdentifier() === $identifier);
+					$flag=($elements[$index]->getIdentifier()===$identifier);
 				$index++;
 			}
-			if ($flag === true)
-				return $elements[$index - 1];
+			if ($flag===true)
+				return $elements[$index-1];
 		} elseif ($elements instanceof BaseHtml) {
-			if ($elements->getIdentifier() === $identifier)
+			if ($elements->getIdentifier()===$identifier)
 				return $elements;
 		}
 		return null;
 	}
 
-	protected function getElementByPropertyValue($propertyName,$value, $elements) {
+	protected function getElementByPropertyValue($propertyName, $value, $elements) {
 		if (is_array($elements)) {
 			$flag=false;
 			$index=0;
-			while ( !$flag && $index < sizeof($elements) ) {
+			while (!$flag && $index<sizeof($elements)) {
 				if ($elements[$index] instanceof BaseHtml)
-					$flag=($elements[$index]->propertyContains($propertyName, $value) === true);
+					$flag=($elements[$index]->propertyContains($propertyName, $value)===true);
 					$index++;
 			}
-			if ($flag === true)
-				return $elements[$index - 1];
+			if ($flag===true)
+				return $elements[$index-1];
 		} elseif ($elements instanceof BaseHtml) {
-			if ($elements->propertyContains($propertyName, $value) === true)
+			if ($elements->propertyContains($propertyName, $value)===true)
 				return $elements;
 		}
 		return null;
@@ -423,14 +423,14 @@ 
 		if (is_array($value)) {
 			$value=implode(",", $value);
 		}
-		if (strrpos($value, 'this') === false && strrpos($value, 'event') === false) {
-			$value='"' . $value . '"';
+		if (strrpos($value, 'this')===false && strrpos($value, 'event')===false) {
+			$value='"'.$value.'"';
 		}
 		return $value;
 	}
 
 	public function jsDoJquery($jqueryCall, $param="") {
-		return "$('#" . $this->identifier . "')." . $jqueryCall . "(" . $this->_prep_value($param) . ");";
+		return "$('#".$this->identifier."').".$jqueryCall."(".$this->_prep_value($param).");";
 	}
 
 	public function executeOnRun($jsCode) {

Braces +39 added lines, -27 removed lines

@@ -50,8 +50,9 @@ 
 	}
 
 	public function getProperty($name) {
-		if (array_key_exists($name, $this->properties))
-			return $this->properties[$name];
+		if (array_key_exists($name, $this->properties)) {
+					return $this->properties[$name];
+		}
 	}
 
 	public function addToProperty($name, $value, $separator=" ") {
@@ -61,10 +62,11 @@ 
 			}
 		} else if ($value !== "" && $this->propertyContains($name, $value) === false) {
 			$v=@$this->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->setProperty($name, $v);
 		}
@@ -119,8 +121,9 @@ 
 	}
 
 	protected function setPropertyCtrl($name, $value, $typeCtrl) {
-		if ($this->ctrl($name, $value, $typeCtrl) === true)
-			return $this->setProperty($name, $value);
+		if ($this->ctrl($name, $value, $typeCtrl) === true) {
+					return $this->setProperty($name, $value);
+		}
 		return $this;
 	}
 
@@ -150,8 +153,9 @@ 
 	}
 
 	public function removeProperty($name) {
-		if (\array_key_exists($name, $this->properties))
-			unset($this->properties[$name]);
+		if (\array_key_exists($name, $this->properties)) {
+					unset($this->properties[$name]);
+		}
 		return $this;
 	}
 
@@ -171,8 +175,9 @@ 
 	}
 
 	protected function addToPropertyUnique($name, $value, $typeCtrl) {
-		if (@class_exists($typeCtrl, true))
-			$typeCtrl=$typeCtrl::getConstants();
+		if (@class_exists($typeCtrl, true)) {
+					$typeCtrl=$typeCtrl::getConstants();
+		}
 		if (is_array($typeCtrl)) {
 			$this->removeOldValues($this->properties[$name], $typeCtrl);
 		}
@@ -301,7 +306,7 @@ 
 	public function onCreate($jsCode){
 		if(isset($this->_events["_create"])){
 			$this->_events["_create"][]=$jsCode;
-		}else{
+		} else{
 			$this->_events["_create"]=[$jsCode];
 		}
 		return $this;
@@ -313,8 +318,9 @@ 
 			if(\is_array($create)){
 				$create=\implode("", $create);
 			}
-			if($create!=="")
-				$js->exec($create,true);
+			if($create!=="") {
+							$js->exec($create,true);
+			}
 			unset($this->_events["_create"]);
 		}
 		if (isset($this->_bsComponent)) {
@@ -376,15 +382,18 @@ 
 			$flag=false;
 			$index=0;
 			while ( !$flag && $index < sizeof($elements) ) {
-				if ($elements[$index] instanceof BaseHtml)
-					$flag=($elements[$index]->getIdentifier() === $identifier);
+				if ($elements[$index] instanceof BaseHtml) {
+									$flag=($elements[$index]->getIdentifier() === $identifier);
+				}
 				$index++;
 			}
-			if ($flag === true)
-				return $elements[$index - 1];
+			if ($flag === true) {
+							return $elements[$index - 1];
+			}
 		} elseif ($elements instanceof BaseHtml) {
-			if ($elements->getIdentifier() === $identifier)
-				return $elements;
+			if ($elements->getIdentifier() === $identifier) {
+							return $elements;
+			}
 		}
 		return null;
 	}
@@ -394,15 +403,18 @@ 
 			$flag=false;
 			$index=0;
 			while ( !$flag && $index < sizeof($elements) ) {
-				if ($elements[$index] instanceof BaseHtml)
-					$flag=($elements[$index]->propertyContains($propertyName, $value) === true);
+				if ($elements[$index] instanceof BaseHtml) {
+									$flag=($elements[$index]->propertyContains($propertyName, $value) === true);
+				}
 					$index++;
 			}
-			if ($flag === true)
-				return $elements[$index - 1];
+			if ($flag === true) {
+							return $elements[$index - 1];
+			}
 		} elseif ($elements instanceof BaseHtml) {
-			if ($elements->propertyContains($propertyName, $value) === true)
-				return $elements;
+			if ($elements->propertyContains($propertyName, $value) === true) {
+							return $elements;
+			}
 		}
 		return null;
 	}

Doc Comments +45 added lines

@@ -124,6 +124,10 @@ 
 		return $this;
 	}
 
+	/**
+	 * @param string $name
+	 * @param string[] $typeCtrl
+	 */
 	protected function setMemberCtrl(&$name, $value, $typeCtrl) {
 		if ($this->ctrl($name, $value, $typeCtrl) === true) {
 			return $name=$value;
@@ -139,22 +143,35 @@ 
 		return $this;
 	}
 
+	/**
+	 * @param string $name
+	 */
 	protected function removePropertyValue($name, $value) {
 		$this->properties[$name]=\str_replace($value, "", $this->properties[$name]);
 		return $this;
 	}
 
+	/**
+	 * @param string $name
+	 */
 	protected function removePropertyValues($name, $values) {
 		$this->removeOldValues($this->properties[$name], $values);
 		return $this;
 	}
 
+	/**
+	 * @param string $name
+	 */
 	public function removeProperty($name) {
 		if (\array_key_exists($name, $this->properties))
 			unset($this->properties[$name]);
 		return $this;
 	}
 
+	/**
+	 * @param string $name
+	 * @param string[] $typeCtrl
+	 */
 	protected function addToMemberCtrl(&$name, $value, $typeCtrl, $separator=" ") {
 		if ($this->ctrl($name, $value, $typeCtrl) === true) {
 			if (is_array($typeCtrl)) {
@@ -165,6 +182,9 @@ 
 		return $this;
 	}
 
+	/**
+	 * @param string $name
+	 */
 	protected function addToMember(&$name, $value, $separator=" ") {
 		$name=str_ireplace($value, "", $name) . $separator . $value;
 		return $this;
@@ -179,6 +199,9 @@ 
 		return $this->addToProperty($name, $value);
 	}
 
+	/**
+	 * @param string $name
+	 */
 	public function addToPropertyCtrl($name, $value, $typeCtrl) {
 		return $this->addToPropertyUnique($name, $value, $typeCtrl);
 	}
@@ -242,6 +265,9 @@ 
 		return $array;
 	}
 
+	/**
+	 * @param \Closure $function
+	 */
 	public function fromDatabaseObjects($objects, $function) {
 		if (isset($objects)) {
 			foreach ( $objects as $object ) {
@@ -345,14 +371,23 @@ 
 		return $this;
 	}
 
+	/**
+	 * @param string $event
+	 */
 	public function getOn($event, $url, $responseElement="", $parameters=array()) {
 		return $this->_ajaxOn("get", $event, $url, $responseElement, $parameters);
 	}
 
+	/**
+	 * @param string $url
+	 */
 	public function getOnClick($url, $responseElement="", $parameters=array()) {
 		return $this->getOn("click", $url, $responseElement, $parameters);
 	}
 
+	/**
+	 * @param string $event
+	 */
 	public function postOn($event, $url, $params="{}", $responseElement="", $parameters=array()) {
 		$parameters["params"]=$params;
 		return $this->_ajaxOn("post", $event, $url, $responseElement, $parameters);
@@ -362,6 +397,9 @@ 
 		return $this->postOn("click", $url, $params, $responseElement, $parameters);
 	}
 
+	/**
+	 * @param string $event
+	 */
 	public function postFormOn($event, $url, $form, $responseElement="", $parameters=array()) {
 		$parameters["form"]=$form;
 		return $this->_ajaxOn("postForm", $event, $url, $responseElement, $parameters);
@@ -389,6 +427,10 @@ 
 		return null;
 	}
 
+	/**
+	 * @param string $propertyName
+	 * @param string $value
+	 */
 	protected function getElementByPropertyValue($propertyName,$value, $elements) {
 		if (is_array($elements)) {
 			$flag=false;
@@ -429,6 +471,9 @@ 
 		return $value;
 	}
 
+	/**
+	 * @param string $jqueryCall
+	 */
 	public function jsDoJquery($jqueryCall, $param="") {
 		return "$('#" . $this->identifier . "')." . $jqueryCall . "(" . $this->_prep_value($param) . ");";
 	}

Ajax/semantic/traits/SemanticHtmlCollectionsTrait.php

Spacing +4 added lines, -4 removed lines

@@ -18,7 +18,7 @@ 
 	 * @param int $colCount
 	 * @return HtmlTable
 	 */
-	public function htmlTable($identifier, $rowCount, $colCount){
+	public function htmlTable($identifier, $rowCount, $colCount) {
 		return $this->addHtmlComponent(new HtmlTable($identifier, $rowCount, $colCount));
 	}
 
@@ -29,9 +29,9 @@ 
 	 * @param $styles string|array|NULL
 	 * @return HtmlMessage
 	 */
-	public function htmlMessage($identifier, $content="",$styles=NULL) {
-		$msg= $this->addHtmlComponent(new HtmlMessage($identifier, $content));
-		if(isset($msg))
+	public function htmlMessage($identifier, $content="", $styles=NULL) {
+		$msg=$this->addHtmlComponent(new HtmlMessage($identifier, $content));
+		if (isset($msg))
 			$msg->setStyle($styles);
 		return $msg;
 	}

Braces +3 added lines, -2 removed lines

@@ -31,8 +31,9 @@
 	 */
 	public function htmlMessage($identifier, $content="",$styles=NULL) {
 		$msg= $this->addHtmlComponent(new HtmlMessage($identifier, $content));
-		if(isset($msg))
-			$msg->setStyle($styles);
+		if(isset($msg)) {
+					$msg->setStyle($styles);
+		}
 		return $msg;
 	}
 

Ajax/semantic/traits/SemanticComponentsTrait.php

Spacing +5 added lines, -5 removed lines

@@ -127,8 +127,8 @@ 
 	 * @param string|array $params
 	 * @return Modal
 	 */
-	public function modal($attachTo=NULL, $params=NULL,$paramsParts=NULL) {
-		$result= $this->addComponent(new Modal($this->js), $attachTo, $params);
+	public function modal($attachTo=NULL, $params=NULL, $paramsParts=NULL) {
+		$result=$this->addComponent(new Modal($this->js), $attachTo, $params);
 		$result->setParamParts($paramsParts);
 		return $result;
 	}
@@ -139,7 +139,7 @@ 
 	 * @return Tab
 	 */
 	public function tab($attachTo=NULL, $params=NULL) {
-		$result= $this->addComponent(new Tab($this->js), $attachTo, $params);
+		$result=$this->addComponent(new Tab($this->js), $attachTo, $params);
 		return $result;
 	}
 
@@ -149,7 +149,7 @@ 
 	 * @return Shape
 	 */
 	public function shape($attachTo=NULL, $params=NULL) {
-		$result= $this->addComponent(new Shape($this->js), $attachTo, $params);
+		$result=$this->addComponent(new Shape($this->js), $attachTo, $params);
 		return $result;
 	}
 
@@ -159,7 +159,7 @@ 
 	 * @return Form
 	 */
 	public function form($attachTo=NULL, $params=NULL) {
-		$result= $this->addComponent(new Form($this->js), $attachTo, $params);
+		$result=$this->addComponent(new Form($this->js), $attachTo, $params);
 		return $result;
 	}
 

Ajax/semantic/components/SimpleSemExtComponent.php

Spacing +6 added lines, -6 removed lines

@@ -12,9 +12,9 @@ 
 		$this->paramParts=array();
 	}
 
-	protected function generateParamParts(){
+	protected function generateParamParts() {
 		$results=[];
-		foreach ($this->paramParts as $paramPart){
+		foreach ($this->paramParts as $paramPart) {
 			$results[]="{$this->uiName}(".\implode(",", $paramPart).")";
 		}
 		return \implode(".", $results);
@@ -22,9 +22,9 @@ 
 
 	public function getScript() {
 		$allParams=$this->params;
-		$this->jquery_code_for_compile=array ();
+		$this->jquery_code_for_compile=array();
 		$paramParts="";
-		if(\sizeof($this->paramParts)>0){
+		if (\sizeof($this->paramParts)>0) {
 			$paramParts=".".$this->generateParamParts();
 		}
 		$this->jquery_code_for_compile []="$( \"".$this->attachTo."\" ).{$this->uiName}(".$this->getParamsAsJSON($allParams).")".$paramParts.";";
@@ -37,8 +37,8 @@ 
 		return $this;
 	}
 
-	public function addComponentEvent($event,$jsCode){
-		$jsCode=str_ireplace("\"","%quote%", $jsCode);
+	public function addComponentEvent($event, $jsCode) {
+		$jsCode=str_ireplace("\"", "%quote%", $jsCode);
 		return $this->setParam($event, "%function(){".$jsCode."}%");
 	}
 

Ajax/semantic/components/validation/FieldValidation.php

Spacing +10 added lines, -10 removed lines

@@ -7,7 +7,7 @@ 
  * @version 1.001
  * Generates a JSON field validator
  */
-class FieldValidation implements \JsonSerializable{
+class FieldValidation implements \JsonSerializable {
 	/**
 	 * @var string
 	 */
@@ -24,7 +24,7 @@ 
 
 	protected $optional;
 
-	public function __construct($identifier){
+	public function __construct($identifier) {
 		$this->identifier=$identifier;
 		$this->rules=[];
 	}
@@ -42,20 +42,20 @@ 
 		return $this->rules;
 	}
 
-	public function addRule($type,$prompt=NULL,$value=NULL){
-		if($type instanceof  Rule)
+	public function addRule($type, $prompt=NULL, $value=NULL) {
+		if ($type instanceof  Rule)
 			$this->rules[]=$type;
-		else if(\is_array($type)){
+		else if (\is_array($type)) {
 			$value=JArray::getValue($type, "value", 2);
 			$prompt=JArray::getValue($type, "prompt", 1);
 			$type=JArray::getValue($type, "type", 0);
-			$this->rules[]=new Rule($type,$prompt,$value);
-		}else
-			$this->rules[]=new Rule($type,$prompt,$value);
+			$this->rules[]=new Rule($type, $prompt, $value);
+		} else
+			$this->rules[]=new Rule($type, $prompt, $value);
 	}
 
-	public function jsonSerialize(){
-		return ["identifier"=>$this->identifier,"rules"=>$this->rules];
+	public function jsonSerialize() {
+		return ["identifier"=>$this->identifier, "rules"=>$this->rules];
 	}
 
 	public function setDepends($depends) {

Braces +6 added lines, -5 removed lines

@@ -43,15 +43,16 @@
 	}
 
 	public function addRule($type,$prompt=NULL,$value=NULL){
-		if($type instanceof  Rule)
-			$this->rules[]=$type;
-		else if(\is_array($type)){
+		if($type instanceof  Rule) {
+					$this->rules[]=$type;
+		} else if(\is_array($type)){
 			$value=JArray::getValue($type, "value", 2);
 			$prompt=JArray::getValue($type, "prompt", 1);
 			$type=JArray::getValue($type, "type", 0);
 			$this->rules[]=new Rule($type,$prompt,$value);
-		}else
-			$this->rules[]=new Rule($type,$prompt,$value);
+		} else {
+					$this->rules[]=new Rule($type,$prompt,$value);
+		}
 	}
 
 	public function jsonSerialize(){

Ajax/semantic/components/validation/Rule.php

Spacing +47 added lines, -47 removed lines

@@ -5,7 +5,7 @@ 
  * @version 1.001
  * Generates a JSON Rule for the validation of a field
  */
-class Rule implements \JsonSerializable{
+class Rule implements \JsonSerializable {
 	/**
 	 * @var string
 	 */
@@ -20,7 +20,7 @@ 
 	 */
 	private $value;
 
-	public function __construct($type,$prompt=NULL,$value=NULL){
+	public function __construct($type, $prompt=NULL, $value=NULL) {
 		$this->type=$type;
 		$this->prompt=$prompt;
 		$this->value=$value;
@@ -54,10 +54,10 @@ 
 	}
 
 	public function jsonSerialize() {
-		$result= ["type"=>$this->type];
-		if(isset($this->prompt))
+		$result=["type"=>$this->type];
+		if (isset($this->prompt))
 			$result["prompt"]=$this->prompt;
-		if(isset($this->value))
+		if (isset($this->value))
 			$result["value"]=$this->value;
 		return $result;
 	}
@@ -68,8 +68,8 @@ 
 	 * @param string $prompt
 	 * @return \Ajax\semantic\components\validation\Rule
 	 */
-	public static function match($name,$prompt=NULL){
-		return new Rule("match[".$name."]",$prompt);
+	public static function match($name, $prompt=NULL) {
+		return new Rule("match[".$name."]", $prompt);
 	}
 
 	/**
@@ -78,8 +78,8 @@ 
 	 * @param string $prompt
 	 * @return \Ajax\semantic\components\validation\Rule
 	 */
-	public static function different($name,$prompt=NULL){
-		return new Rule("different[".$name."]",$prompt);
+	public static function different($name, $prompt=NULL) {
+		return new Rule("different[".$name."]", $prompt);
 	}
 
 	/**
@@ -89,78 +89,78 @@ 
 	 * @param string $prompt
 	 * @return \Ajax\semantic\components\validation\Rule
 	 */
-	public static function integer($min=NULL,$max=NULL,$prompt=NULL){
-		if(\is_int($min) && \is_int($max))
-			return new Rule("integer[{$min}..{$max}]",$prompt);
-		return new Rule("integer",$prompt);
+	public static function integer($min=NULL, $max=NULL, $prompt=NULL) {
+		if (\is_int($min) && \is_int($max))
+			return new Rule("integer[{$min}..{$max}]", $prompt);
+		return new Rule("integer", $prompt);
 	}
 
-	public static function decimal($prompt=NULL){
-		return new Rule("decimal",$prompt);
+	public static function decimal($prompt=NULL) {
+		return new Rule("decimal", $prompt);
 	}
 
-	public static function number($prompt=NULL){
-		return new Rule("number",$prompt);
+	public static function number($prompt=NULL) {
+		return new Rule("number", $prompt);
 	}
 
-	public static function is($value,$prompt=NULL){
-		return new Rule("is[".$value."]",$prompt);
+	public static function is($value, $prompt=NULL) {
+		return new Rule("is[".$value."]", $prompt);
 	}
 
-	public static function isExactly($value,$prompt=NULL){
-		return new Rule("isExactly[".$value."]",$prompt);
+	public static function isExactly($value, $prompt=NULL) {
+		return new Rule("isExactly[".$value."]", $prompt);
 	}
 
-	public static function not($value,$prompt=NULL){
-		return new Rule("not[".$value."]",$prompt);
+	public static function not($value, $prompt=NULL) {
+		return new Rule("not[".$value."]", $prompt);
 	}
 
-	public static function notExactly($value,$prompt=NULL){
-		return new Rule("notExactly[".$value."]",$prompt);
+	public static function notExactly($value, $prompt=NULL) {
+		return new Rule("notExactly[".$value."]", $prompt);
 	}
 
-	public static function contains($value,$prompt=NULL){
-		return new Rule("contains[".$value."]",$prompt);
+	public static function contains($value, $prompt=NULL) {
+		return new Rule("contains[".$value."]", $prompt);
 	}
 
-	public static function containsExactly($value,$prompt=NULL){
-		return new Rule("containsExactly[".$value."]",$prompt);
+	public static function containsExactly($value, $prompt=NULL) {
+		return new Rule("containsExactly[".$value."]", $prompt);
 	}
 
-	public static function doesntContain($value,$prompt=NULL){
-		return new Rule("doesntContain[".$value."]",$prompt);
+	public static function doesntContain($value, $prompt=NULL) {
+		return new Rule("doesntContain[".$value."]", $prompt);
 	}
 
-	public static function doesntContainExactly($value,$prompt=NULL){
-		return new Rule("doesntContainExactly[".$value."]",$prompt);
+	public static function doesntContainExactly($value, $prompt=NULL) {
+		return new Rule("doesntContainExactly[".$value."]", $prompt);
 	}
 
-	public static function minCount($value,$prompt=NULL){
-		return new Rule("minCount[".$value."]",$prompt);
+	public static function minCount($value, $prompt=NULL) {
+		return new Rule("minCount[".$value."]", $prompt);
 	}
 
-	public static function maxCount($value,$prompt=NULL){
-		return new Rule("maxCount[".$value."]",$prompt);
+	public static function maxCount($value, $prompt=NULL) {
+		return new Rule("maxCount[".$value."]", $prompt);
 	}
 
-	public static function exactCount($value,$prompt=NULL){
-		return new Rule("exactCount[".$value."]",$prompt);
+	public static function exactCount($value, $prompt=NULL) {
+		return new Rule("exactCount[".$value."]", $prompt);
 	}
 
-	public static function email($prompt=NULL){
-		return new Rule("email",$prompt);
+	public static function email($prompt=NULL) {
+		return new Rule("email", $prompt);
 	}
 
-	public static function url($prompt=NULL){
-		return new Rule("url",$prompt);
+	public static function url($prompt=NULL) {
+		return new Rule("url", $prompt);
 	}
 
-	public static function regExp($value,$prompt=NULL){
-		return new Rule("regExp",$prompt,$value);
+	public static function regExp($value, $prompt=NULL) {
+		return new Rule("regExp", $prompt, $value);
 	}
 
-	public static function custom($name,$jsFunction){
-		return "$.fn.form.settings.rules.".$name." =".$jsFunction ;
+	public static function custom($name, $jsFunction) {
+		return "$.fn.form.settings.rules.".$name." =".$jsFunction;
 	}
 
 }
\ No newline at end of file

Braces +9 added lines, -6 removed lines

@@ -55,10 +55,12 @@ 
 
 	public function jsonSerialize() {
 		$result= ["type"=>$this->type];
-		if(isset($this->prompt))
-			$result["prompt"]=$this->prompt;
-		if(isset($this->value))
-			$result["value"]=$this->value;
+		if(isset($this->prompt)) {
+					$result["prompt"]=$this->prompt;
+		}
+		if(isset($this->value)) {
+					$result["value"]=$this->value;
+		}
 		return $result;
 	}
 
@@ -90,8 +92,9 @@ 
 	 * @return \Ajax\semantic\components\validation\Rule
 	 */
 	public static function integer($min=NULL,$max=NULL,$prompt=NULL){
-		if(\is_int($min) && \is_int($max))
-			return new Rule("integer[{$min}..{$max}]",$prompt);
+		if(\is_int($min) && \is_int($max)) {
+					return new Rule("integer[{$min}..{$max}]",$prompt);
+		}
 		return new Rule("integer",$prompt);
 	}
 

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

Braces +27 added lines, -18 removed lines

@@ -47,8 +47,9 @@ 
 	 */
 	public function addHeader($title, $niveau=1, $dividing=true) {
 		$header=new HtmlHeader("", $niveau, $title);
-		if ($dividing)
-			$header->setDividing();
+		if ($dividing) {
+					$header->setDividing();
+		}
 		return $this->addItem($header);
 	}
 
@@ -69,14 +70,16 @@ 
 					if (\is_string($end)) {
 						$label=$end;
 						\array_pop($fields);
-					} else
-						$label=NULL;
+					} else {
+											$label=NULL;
+					}
 				}
 				$this->_fields=\array_merge($this->_fields, $fields);
 				$fields=new HtmlFormFields("fields-" . $this->identifier . "-" . $this->count(), $fields);
 			}
-			if (isset($label))
-				$fields=new HtmlFormField("", $fields, $label);
+			if (isset($label)) {
+							$fields=new HtmlFormField("", $fields, $label);
+			}
 		} else {
 			$fields=new HtmlFormFields("fields-" . $this->identifier . "-" . $this->count());
 		}
@@ -145,12 +148,15 @@ 
 	 */
 	public function addMessage($identifier, $content, $header=NULL, $icon=NULL, $type=NULL) {
 		$message=new HtmlMessage($identifier, $content);
-		if (isset($header))
-			$message->addHeader($header);
-		if (isset($icon))
-			$message->setIcon($icon);
-		if (isset($type))
-			$message->setStyle($type);
+		if (isset($header)) {
+					$message->addHeader($header);
+		}
+		if (isset($icon)) {
+					$message->setIcon($icon);
+		}
+		if (isset($type)) {
+					$message->setStyle($type);
+		}
 		return $this->addItem($message);
 	}
 
@@ -169,15 +175,17 @@ 
 	public function run(JsUtils $js) {
 		$compo=NULL;
 		foreach ($this->_fields as $field){
-			if($field instanceof HtmlFormField)
-				$compo=$this->addCompoValidation($js, $compo, $field);
+			if($field instanceof HtmlFormField) {
+							$compo=$this->addCompoValidation($js, $compo, $field);
+			}
 		}
 		foreach ($this->content as $field){
 			if($field instanceof HtmlFormFields){
 				$items=$field->getItems();
 				foreach ($items as $_field){
-					if($_field instanceof HtmlFormField)
-						$compo=$this->addCompoValidation($js, $compo, $_field);
+					if($_field instanceof HtmlFormField) {
+											$compo=$this->addCompoValidation($js, $compo, $_field);
+					}
 				}
 			}
 		}
@@ -230,8 +238,9 @@ 
 
 	public function addSubmit($identifier,$value,$CssStyle=NULL,$url=NULL,$responseElement=NULL){
 		$bt=$this->addButton($identifier, $value,$CssStyle);
-		if(isset($url) && isset($responseElement))
-			$this->submitOnClick($identifier, $url, $responseElement);
+		if(isset($url) && isset($responseElement)) {
+					$this->submitOnClick($identifier, $url, $responseElement);
+		}
 		return $bt;
 	}
 

Spacing +31 added lines, -31 removed lines

@@ -32,9 +32,9 @@ 
 
 	public function __construct($identifier, $elements=array()) {
 		parent::__construct($identifier, "form", "ui form");
-		$this->_states=[ State::ERROR,State::SUCCESS,State::WARNING,State::DISABLED ];
+		$this->_states=[State::ERROR, State::SUCCESS, State::WARNING, State::DISABLED];
 		$this->setProperty("name", $this->identifier);
-		$this->_fields=array ();
+		$this->_fields=array();
 		$this->_validationParams=[];
 		$this->addItems($elements);
 	}
@@ -56,14 +56,14 @@ 
 	 * @param string $caption
 	 * @return \Ajax\semantic\html\collections\form\HtmlForm
 	 */
-	public function addDivider($caption=NULL){
-		return $this->addContent(new HtmlDivider("",$caption));
+	public function addDivider($caption=NULL) {
+		return $this->addContent(new HtmlDivider("", $caption));
 	}
 
 	public function addFields($fields=NULL, $label=NULL) {
 		if (isset($fields)) {
 			if (!$fields instanceof HtmlFormFields) {
-				if (\is_array($fields) === false) {
+				if (\is_array($fields)===false) {
 					$fields=\func_get_args();
 					$end=\end($fields);
 					if (\is_string($end)) {
@@ -73,12 +73,12 @@ 
 						$label=NULL;
 				}
 				$this->_fields=\array_merge($this->_fields, $fields);
-				$fields=new HtmlFormFields("fields-" . $this->identifier . "-" . $this->count(), $fields);
+				$fields=new HtmlFormFields("fields-".$this->identifier."-".$this->count(), $fields);
 			}
 			if (isset($label))
 				$fields=new HtmlFormField("", $fields, $label);
 		} else {
-			$fields=new HtmlFormFields("fields-" . $this->identifier . "-" . $this->count());
+			$fields=new HtmlFormFields("fields-".$this->identifier."-".$this->count());
 		}
 		$this->addItem($fields);
 		return $fields;
@@ -86,7 +86,7 @@ 
 
 	public function addItem($item) {
 		$item=parent::addItem($item);
-		if (\is_subclass_of($item, HtmlFormField::class) === true) {
+		if (\is_subclass_of($item, HtmlFormField::class)===true) {
 			$this->_fields[]=$item;
 		}
 		return $item;
@@ -138,10 +138,10 @@ 
 		return $this->addItem($message);
 	}
 
-	private function addCompoValidation($js,$compo,$field){
+	private function addCompoValidation($js, $compo, $field) {
 		$validation=$field->getValidation();
-		if(isset($validation)){
-			if(isset($compo)===false){
+		if (isset($validation)) {
+			if (isset($compo)===false) {
 				$compo=$js->semantic()->form("#".$this->identifier);
 			}
 			$validation->setIdentifier($field->getDataField()->getIdentifier());
@@ -152,26 +152,26 @@ 
 
 	public function run(JsUtils $js) {
 		$compo=NULL;
-		foreach ($this->_fields as $field){
-			if($field instanceof HtmlFormField)
+		foreach ($this->_fields as $field) {
+			if ($field instanceof HtmlFormField)
 				$compo=$this->addCompoValidation($js, $compo, $field);
 		}
-		foreach ($this->content as $field){
-			if($field instanceof HtmlFormFields){
+		foreach ($this->content as $field) {
+			if ($field instanceof HtmlFormFields) {
 				$items=$field->getItems();
-				foreach ($items as $_field){
-					if($_field instanceof HtmlFormField)
+				foreach ($items as $_field) {
+					if ($_field instanceof HtmlFormField)
 						$compo=$this->addCompoValidation($js, $compo, $_field);
 				}
 			}
 		}
-		if(isset($compo)===false){
+		if (isset($compo)===false) {
 			return parent::run($js);
 		}
-		if(isset($this->_validationParams["_ajaxSubmit"])){
-			if($this->_validationParams["_ajaxSubmit"] instanceof AjaxCall){
+		if (isset($this->_validationParams["_ajaxSubmit"])) {
+			if ($this->_validationParams["_ajaxSubmit"] instanceof AjaxCall) {
 				$compilation=$this->_validationParams["_ajaxSubmit"]->compile($js);
-				$compilation=str_ireplace("\"","%quote%", $compilation);
+				$compilation=str_ireplace("\"", "%quote%", $compilation);
 				$this->onSuccess($compilation);
 				unset($this->_validationParams["_ajaxSubmit"]);
 			}
@@ -186,7 +186,7 @@ 
 		return $this->addToProperty("class", "loading");
 	}
 
-	public function addErrorMessage(){
+	public function addErrorMessage() {
 		return $this->addContent((new HtmlMessage(""))->setError());
 	}
 
@@ -199,22 +199,22 @@ 
 		return $this;
 	}
 
-	public function submitOn($event,$identifier,$url,$responseElement){
+	public function submitOn($event, $identifier, $url, $responseElement) {
 		$elem=$this->getElementById($identifier, $this->content);
-		if(isset($elem)){
+		if (isset($elem)) {
 			$elem->addEvent($event, "$('#".$this->identifier."').form('validate form');");
-			$this->_validationParams["_ajaxSubmit"]=new AjaxCall("postForm", ["form"=>$this->identifier,"responseElement"=>$responseElement,"url"=>$url]);
+			$this->_validationParams["_ajaxSubmit"]=new AjaxCall("postForm", ["form"=>$this->identifier, "responseElement"=>$responseElement, "url"=>$url]);
 		}
 		return $this;
 	}
 
-	public function submitOnClick($identifier,$url,$responseElement){
+	public function submitOnClick($identifier, $url, $responseElement) {
 		return $this->submitOn("click", $identifier, $url, $responseElement);
 	}
 
-	public function addSubmit($identifier,$value,$CssStyle=NULL,$url=NULL,$responseElement=NULL){
-		$bt=$this->addButton($identifier, $value,$CssStyle);
-		if(isset($url) && isset($responseElement))
+	public function addSubmit($identifier, $value, $CssStyle=NULL, $url=NULL, $responseElement=NULL) {
+		$bt=$this->addButton($identifier, $value, $CssStyle);
+		if (isset($url) && isset($responseElement))
 			$this->submitOnClick($identifier, $url, $responseElement);
 		return $bt;
 	}
@@ -224,7 +224,7 @@ 
 	 * @param string $jsCode
 	 * @return \Ajax\semantic\html\collections\form\HtmlForm
 	 */
-	public function onValid($jsCode){
+	public function onValid($jsCode) {
 		$this->_validationParams["onValid"]="%function(){".$jsCode."}%";
 		return $this;
 	}
@@ -234,7 +234,7 @@ 
 	 * @param string $jsCode can use event and fields parameters
 	 * @return HtmlForm
 	 */
-	public function onSuccess($jsCode){
+	public function onSuccess($jsCode) {
 		$this->_validationParams["onSuccess"]="%function(evt,fields){".$jsCode."}%";
 		return $this;
 	}

Ajax/semantic/html/collections/form/traits/FieldTrait.php

Doc Comments +3 added lines

@@ -6,6 +6,9 @@
 
 trait CheckboxTrait {
 
+	/**
+	 * @param string $name
+	 */
 	public abstract function addToPropertyCtrl($name, $value, $typeCtrl);
 
 	public function setType($checkboxType) {

Spacing +14 added lines, -14 removed lines

@@ -11,51 +11,51 @@
 
 	public abstract function addToProperty($name, $value, $separator=" ");
 	public abstract function addLabel($caption, $style="label-default", $leftSeparator=" ");
-	public abstract function addContent($content,$before=false);
+	public abstract function addContent($content, $before=false);
 	public function setFocus() {
 		$this->addToProperty("class", State::FOCUS);
 	}
 
 	public function addLoading() {
-		if ($this->_hasIcon === false) {
+		if ($this->_hasIcon===false) {
 			throw new \Exception("Input must have an icon for showing a loader, use addIcon before");
 		}
 		return $this->addToProperty("class", State::LOADING);
 	}
 
 	public function labeled($label, $direction=Direction::LEFT, $icon=NULL) {
-		$labelO=$this->addLabel($label,$direction===Direction::LEFT,$icon);
-		$this->addToProperty("class", $direction . " labeled");
+		$labelO=$this->addLabel($label, $direction===Direction::LEFT, $icon);
+		$this->addToProperty("class", $direction." labeled");
 		return $labelO;
 	}
 
 	public function labeledToCorner($icon, $direction=Direction::LEFT) {
-		return $this->labeled("", $direction . " corner", $icon)->toCorner($direction);
+		return $this->labeled("", $direction." corner", $icon)->toCorner($direction);
 	}
 
 	public function addAction($action, $direction=Direction::RIGHT, $icon=NULL, $labeled=false) {
 		$actionO=$action;
-		if (\is_object($action) === false) {
-			$actionO=new HtmlButton("action-" . $this->identifier, $action);
+		if (\is_object($action)===false) {
+			$actionO=new HtmlButton("action-".$this->identifier, $action);
 			if (isset($icon))
 				$actionO->addIcon($icon, true, $labeled);
 		}
-		$this->addToProperty("class", $direction . " action");
-		$this->addContent($actionO, \strstr($direction, Direction::LEFT) !== false);
+		$this->addToProperty("class", $direction." action");
+		$this->addContent($actionO, \strstr($direction, Direction::LEFT)!==false);
 		return $actionO;
 	}
 
-	public function addDropdown($label="", $items=array(),$direction=Direction::RIGHT){
-		$labelO=new HtmlDropdown("dd-".$this->identifier,$label,$items);
-		$labelO->asSelect("select-".$this->identifier,false,true);
-		return $this->labeled($labelO,$direction);
+	public function addDropdown($label="", $items=array(), $direction=Direction::RIGHT) {
+		$labelO=new HtmlDropdown("dd-".$this->identifier, $label, $items);
+		$labelO->asSelect("select-".$this->identifier, false, true);
+		return $this->labeled($labelO, $direction);
 	}
 
 	public function setTransparent() {
 		return $this->addToProperty("class", "transparent");
 	}
 
-	public function setReadonly(){
+	public function setReadonly() {
 		$this->getField()->setProperty("readonly", "");
 	}
 

Braces +3 added lines, -2 removed lines

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