jcheron / phalcon-jquery

Ajax/semantic/html/elements/HtmlInput.php

Braces +6 added lines, -4 removed lines

@@ -33,8 +33,9 @@ 
 		$labelO=$label;
 		if (\is_object($label)===false) {
 			$labelO=new HtmlLabel("label-".$this->identifier, $label);
-			if (isset($icon))
-				$labelO->addIcon($icon);
+			if (isset($icon)) {
+							$labelO->addIcon($icon);
+			}
 		} else {
 			$labelO->addToPropertyCtrl("class", "label", array ("label" ));
 		}
@@ -51,8 +52,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);

Spacing +16 added lines, -16 removed lines

@@ -12,10 +12,10 @@ 
 	use IconTrait;
 
 	public function __construct($identifier, $type="text", $value="", $placeholder="") {
-		parent::__construct("div-" . $identifier, "div", "ui input");
-		$this->content=[ "field" => new \Ajax\common\html\html5\HtmlInput($identifier, $type, $value, $placeholder) ];
-		$this->_states=[ State::DISABLED,State::FOCUS,State::ERROR ];
-		$this->_variations=[ Variation::TRANSPARENT ];
+		parent::__construct("div-".$identifier, "div", "ui input");
+		$this->content=["field" => new \Ajax\common\html\html5\HtmlInput($identifier, $type, $value, $placeholder)];
+		$this->_states=[State::DISABLED, State::FOCUS, State::ERROR];
+		$this->_variations=[Variation::TRANSPARENT];
 	}
 
 	public function setFocus() {
@@ -23,7 +23,7 @@ 
 	}
 
 	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);
@@ -31,31 +31,31 @@ 
 
 	public function labeled($label, $direction=Direction::LEFT, $icon=NULL) {
 		$labelO=$label;
-		if (\is_object($label) === false) {
-			$labelO=new HtmlLabel("label-" . $this->identifier, $label);
+		if (\is_object($label)===false) {
+			$labelO=new HtmlLabel("label-".$this->identifier, $label);
 			if (isset($icon))
 				$labelO->addIcon($icon);
-		} else {
-			$labelO->addToPropertyCtrl("class", "label", array ("label" ));
+		}else {
+			$labelO->addToPropertyCtrl("class", "label", array("label"));
 		}
-		$this->addToProperty("class", $direction . " labeled");
-		$this->addContent($labelO, \strstr($direction, Direction::LEFT) !== false);
+		$this->addToProperty("class", $direction." labeled");
+		$this->addContent($labelO, \strstr($direction, Direction::LEFT)!==false);
 		return $labelO;
 	}
 
 	public function labeledToCorner($label, $direction=Direction::LEFT, $icon=NULL) {
-		return $this->labeled($label, $direction . " corner", $icon)->toCorner($direction);
+		return $this->labeled($label, $direction." corner", $icon)->toCorner($direction);
 	}
 
 	public function addAction($action, $direction=Direction::LEFT, $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;
 	}
 

Ajax/common/html/BaseHtml.php

Doc Comments +35 added lines

@@ -141,22 +141,36 @@ 
 		return $this;
 	}
 
+	/**
+	 * @param string $name
+	 * @param string $value
+	 */
 	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
+	 */
 	protected 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)) {
@@ -181,6 +195,9 @@ 
 		return $this->addToProperty($name, $value);
 	}
 
+	/**
+	 * @param string $name
+	 */
 	public function addToPropertyCtrl($name, $value, $typeCtrl) {
 		return $this->addToPropertyUnique($name, $value, $typeCtrl);
 	}
@@ -321,6 +338,9 @@ 
 		}
 	}
 
+	/**
+	 * @param string $operation
+	 */
 	public function _ajaxOn($operation, $event, $url, $responseElement="", $parameters=array()) {
 		$params=array ("url" => $url,"responseElement" => $responseElement );
 		$params=array_merge($params, $parameters);
@@ -328,14 +348,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);
@@ -345,6 +374,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);
@@ -394,6 +426,9 @@ 
 		return $value;
 	}
 
+	/**
+	 * @param string $jqueryCall
+	 */
 	public function jsDoJquery($jqueryCall, $param="") {
 		return "$('#" . $this->identifier . "')." . $jqueryCall . "(" . $this->_prep_value($param) . ");";
 	}

Braces +26 added lines, -18 removed lines

@@ -51,17 +51,19 @@ 
 	}
 
 	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=" ") {
 		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);
 		}
@@ -121,8 +123,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;
 	}
 
@@ -152,8 +155,9 @@ 
 	}
 
 	protected 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;
 	}
 
@@ -173,8 +177,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);
 		}
@@ -359,15 +364,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;
 	}

Spacing +57 added lines, -57 removed lines

@@ -17,10 +17,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() {
@@ -33,7 +33,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() {
@@ -57,13 +57,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;
 			
@@ -79,23 +79,23 @@ 
 
 	public function compile(JsUtils $js=NULL, View $view=NULL) {
 		$result=$this->getTemplate($js);
-		foreach ( $this as $key => $value ) {
-			if (PhalconUtils::startsWith($key, "_") === false && $key !== "events") {
+		foreach ($this as $key => $value) {
+			if (PhalconUtils::startsWith($key, "_")===false && $key!=="events") {
 				if (is_array($value)) {
 					$v=PropertyWrapper::wrap($value, $js);
-				} else {
+				}else {
 					$v=$value;
 				}
-				$result=str_ireplace("%" . $key . "%", $v, $result);
+				$result=str_ireplace("%".$key."%", $v, $result);
 			}
 		}
 		if (isset($js)) {
 			$this->run($js);
 		}
-		if (isset($view) === true) {
+		if (isset($view)===true) {
 			$controls=$view->getVar("q");
-			if (isset($controls) === false) {
-				$controls=array ();
+			if (isset($controls)===false) {
+				$controls=array();
 			}
 			$controls[$this->identifier]=$result;
 			$view->setVar("q", $controls);
@@ -105,12 +105,12 @@ 
 
 	protected function ctrl($name, $value, $typeCtrl) {
 		if (is_array($typeCtrl)) {
-			if (array_search($value, $typeCtrl) === false) {
-				throw new \Exception("La valeur passée a 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 a propriété `".$name."` ne fait pas partie des valeurs possibles : {".implode(",", $typeCtrl)."}");
 			}
-		} else {
+		}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;
@@ -125,13 +125,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;
@@ -140,7 +140,7 @@ 
 	protected function addToMemberUnique(&$name, $value, $typeCtrl, $separator=" ") {
 		if (is_array($typeCtrl)) {
 			$this->removeOldValues($name, $typeCtrl);
-			$name.=$separator . $value;
+			$name.=$separator.$value;
 		}
 		return $this;
 	}
@@ -162,17 +162,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;
 	}
 
@@ -190,7 +190,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;
@@ -218,28 +218,28 @@ 
 	}
 
 	public function fromArray($array) {
-		foreach ( $this as $key => $value ) {
+		foreach ($this as $key => $value) {
 			if (array_key_exists($key, $array) && !PhalconUtils::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);
+			}else {
+				$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
 					}
 				}
@@ -250,7 +250,7 @@ 
 
 	public function fromDatabaseObjects($objects, $function) {
 		if (isset($objects)) {
-			foreach ( $objects as $object ) {
+			foreach ($objects as $object) {
 				$this->fromDatabaseObject($object, $function);
 			}
 		}
@@ -269,11 +269,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);
 	}
@@ -282,10 +282,10 @@ 
 		if (array_key_exists($event, $this->_events)) {
 			if (is_array($this->_events[$event])) {
 				$this->_events[$event][]=$jsCode;
-			} else {
-				$this->_events[$event]=array ($this->_events[$event],$jsCode );
+			}else {
+				$this->_events[$event]=array($this->_events[$event], $jsCode);
 			}
-		} else {
+		}else {
 			$this->_events[$event]=$jsCode;
 		}
 		return $this;
@@ -305,15 +305,15 @@ 
 
 	public function addEventsOnRun(JsUtils $js) {
 		if (isset($this->_bsComponent)) {
-			foreach ( $this->_events as $event => $jsCode ) {
+			foreach ($this->_events as $event => $jsCode) {
 				$code=$jsCode;
 				if (is_array($jsCode)) {
 					$code="";
-					foreach ( $jsCode as $jsC ) {
+					foreach ($jsCode as $jsC) {
 						if ($jsC instanceof AjaxCall) {
-							$code.="\n" . $jsC->compile($js);
-						} else {
-							$code.="\n" . $jsC;
+							$code.="\n".$jsC->compile($js);
+						}else {
+							$code.="\n".$jsC;
 						}
 					}
 				} elseif ($jsCode instanceof AjaxCall) {
@@ -321,12 +321,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;
@@ -362,15 +362,15 @@ 
 		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;
@@ -392,14 +392,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) {

Ajax/semantic/html/elements/HtmlSegment.php

Spacing +4 added lines, -4 removed lines

@@ -17,12 +17,12 @@ 
  * @version 1.001
  */
 class HtmlSegment extends HtmlSemDoubleElement {
-	use AttachedTrait,TextAlignmentTrait;
+	use AttachedTrait, TextAlignmentTrait;
 
 	public function __construct($identifier, $content="") {
 		parent::__construct($identifier, "div", "ui segment");
-		$this->_variations=\array_merge($this->_variations, [ Variation::PADDED,Variation::COMPACT ]);
-		$this->_states=\array_merge($this->_states, [ State::LOADING ]);
+		$this->_variations=\array_merge($this->_variations, [Variation::PADDED, Variation::COMPACT]);
+		$this->_states=\array_merge($this->_states, [State::LOADING]);
 		$this->content=$content;
 	}
 
@@ -36,7 +36,7 @@ 
 	}
 
 	public function setSens($sens="vertical") {
-		return $this->addToPropertyCtrl("class", $sens, array ("vertical","horizontal" ));
+		return $this->addToPropertyCtrl("class", $sens, array("vertical", "horizontal"));
 	}
 
 	public function setEmphasis($value=Emphasis::SECONDARY) {

Ajax/semantic/html/elements/HtmlButton.php

Spacing +10 added lines, -10 removed lines

@@ -54,7 +54,7 @@ 
 	}
 
 	public function setFocusable($value=true) {
-		if ($value === true)
+		if ($value===true)
 			$this->setProperty("tabindex", "0");
 		else {
 			$this->removeProperty("tabindex");
@@ -64,14 +64,14 @@ 
 
 	public function setAnimated($content, $animation="") {
 		$this->setTagName("div");
-		$this->addToProperty("class", "animated " . $animation);
-		$visible=new HtmlSemDoubleElement("visible-" . $this->identifier, "div");
+		$this->addToProperty("class", "animated ".$animation);
+		$visible=new HtmlSemDoubleElement("visible-".$this->identifier, "div");
 		$visible->setClass("visible content");
 		$visible->setContent($this->content);
-		$hidden=new HtmlSemDoubleElement("hidden-" . $this->identifier, "div");
+		$hidden=new HtmlSemDoubleElement("hidden-".$this->identifier, "div");
 		$hidden->setClass("hidden content");
 		$hidden->setContent($content);
-		$this->content=array ($visible,$hidden );
+		$this->content=array($visible, $hidden);
 		return $hidden;
 	}
 
@@ -83,7 +83,7 @@ 
 	public function asIcon($icon) {
 		$iconO=$icon;
 		if (\is_string($icon)) {
-			$iconO=new HtmlIcon("icon-" . $this->identifier, $icon);
+			$iconO=new HtmlIcon("icon-".$this->identifier, $icon);
 		}
 		$this->addToProperty("class", "icon");
 		$this->content=$iconO;
@@ -104,9 +104,9 @@ 
 	public function addLabel($caption, $before=false) {
 		$this->tagName="div";
 		$this->addToProperty("class", "labeled");
-		$this->content=new HtmlButton("button-" . $this->identifier, $this->content);
+		$this->content=new HtmlButton("button-".$this->identifier, $this->content);
 		$this->content->setTagName("div");
-		$label=new HtmlLabel("label-" . $this->identifier, $caption, "a");
+		$label=new HtmlLabel("label-".$this->identifier, $caption, "a");
 		$label->setBasic();
 		$this->addContent($label, $before);
 		return $label;
@@ -118,7 +118,7 @@ 
 	 */
 	public function fromArray($array) {
 		$array=parent::fromArray($array);
-		foreach ( $array as $key => $value ) {
+		foreach ($array as $key => $value) {
 			$this->setProperty($key, $value);
 		}
 		return $array;
@@ -181,7 +181,7 @@ 
 	}
 
 	public static function social($identifier, $social, $value=NULL) {
-		if ($value === NULL)
+		if ($value===NULL)
 			$value=\ucfirst($social);
 		$return=new HtmlButton($identifier, $value);
 		$return->addIcon($social);

Braces +6 added lines, -5 removed lines

@@ -54,9 +54,9 @@ 
 	}
 
 	public function setFocusable($value=true) {
-		if ($value === true)
-			$this->setProperty("tabindex", "0");
-		else {
+		if ($value === true) {
+					$this->setProperty("tabindex", "0");
+		} else {
 			$this->removeProperty("tabindex");
 		}
 		return $this;
@@ -181,8 +181,9 @@ 
 	}
 
 	public static function social($identifier, $social, $value=NULL) {
-		if ($value === NULL)
-			$value=\ucfirst($social);
+		if ($value === NULL) {
+					$value=\ucfirst($social);
+		}
 		$return=new HtmlButton($identifier, $value);
 		$return->addIcon($social);
 		return $return->addToPropertyCtrl("class", $social, Social::getConstants());

Doc Comments +3 added lines, -3 removed lines

@@ -37,7 +37,7 @@ 
 	/**
 	 * Set the button value
 	 * @param string $value
-	 * @return \Ajax\semantic\html\HtmlButton
+	 * @return HtmlButton
 	 */
 	public function setValue($value) {
 		$this->content=$value;
@@ -46,8 +46,8 @@ 
 
 	/**
 	 * define the button style
-	 * @param string|int $cssStyle
-	 * @return \Ajax\semantic\html\HtmlButton default : ""
+	 * @param string $cssStyle
+	 * @return HtmlButton default : ""
 	 */
 	public function setStyle($cssStyle) {
 		return $this->addToProperty("class", $cssStyle);

Ajax/semantic/html/content/HtmlGridCol.php

Spacing +2 added lines, -2 removed lines

@@ -30,10 +30,10 @@
 	 */
 	public function setWidth($width) {
 		if (\is_int($width)) {
-			$width=Wide::getConstants()["W" . $width];
+			$width=Wide::getConstants()["W".$width];
 		}
 		$this->addToPropertyCtrl("class", $width, Wide::getConstants());
-		return $this->addToPropertyCtrl("class", "wide", array ("wide" ));
+		return $this->addToPropertyCtrl("class", "wide", array("wide"));
 	}
 
 	public function setValue($value) {

Braces +3 added lines, -2 removed lines

@@ -19,8 +19,9 @@
 	public function __construct($identifier, $width=NULL) {
 		parent::__construct($identifier, "div");
 		$this->setClass("column");
-		if (isset($width))
-			$this->setWidth($width);
+		if (isset($width)) {
+					$this->setWidth($width);
+		}
 	}
 
 	/**

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

Doc Comments +12 added lines

@@ -11,12 +11,24 @@
 	protected $_states=[ ];
 	protected $_baseClass;
 
+	/**
+	 * @param string $name
+	 */
 	protected abstract function setPropertyCtrl($name, $value, $typeCtrl);
 
+	/**
+	 * @param string $name
+	 */
 	protected abstract function addToPropertyCtrl($name, $value, $typeCtrl);
 
+	/**
+	 * @param string $name
+	 */
 	protected abstract function addToPropertyCtrlCheck($name, $value, $typeCtrl);
 
+	/**
+	 * @param string $name
+	 */
 	public abstract function addToProperty($name, $value, $separator=" ");
 
 	public function addVariation($variation) {

Spacing +7 added lines, -7 removed lines

@@ -7,8 +7,8 @@ 
 use Ajax\semantic\html\base\constants\Direction;
 
 trait BaseTrait {
-	protected $_variations=[ ];
-	protected $_states=[ ];
+	protected $_variations=[];
+	protected $_states=[];
 	protected $_baseClass;
 
 	protected abstract function setPropertyCtrl($name, $value, $typeCtrl);
@@ -36,7 +36,7 @@ 
 		$this->setProperty("class", $this->_baseClass);
 		if (\is_string($variations))
 			$variations=\explode(" ", $variations);
-		foreach ( $variations as $variation ) {
+		foreach ($variations as $variation) {
 			$this->addVariation($variation);
 		}
 		return $this;
@@ -50,7 +50,7 @@ 
 	public function addVariations($variations=array()) {
 		if (\is_string($variations))
 			$variations=\explode(" ", $variations);
-		foreach ( $variations as $variation ) {
+		foreach ($variations as $variation) {
 			$this->addVariation($variation);
 		}
 		return $this;
@@ -59,7 +59,7 @@ 
 	public function addStates($states=array()) {
 		if (\is_string($states))
 			$states=\explode(" ", $states);
-		foreach ( $states as $state ) {
+		foreach ($states as $state) {
 			$this->addState($state);
 		}
 		return $this;
@@ -69,7 +69,7 @@ 
 		$this->setProperty("class", $this->_baseClass);
 		if (\is_string($states))
 			$states=\explode(" ", $states);
-		foreach ( $states as $state ) {
+		foreach ($states as $state) {
 			$this->addState($state);
 		}
 		return $this;
@@ -122,7 +122,7 @@ 
 	}
 
 	public function setFloated($direction="right") {
-		return $this->addToPropertyCtrl("class", $direction . " floated", Direction::getConstantValues("floated"));
+		return $this->addToPropertyCtrl("class", $direction." floated", Direction::getConstantValues("floated"));
 	}
 
 	public function floatRight() {

Braces +12 added lines, -8 removed lines

@@ -34,8 +34,9 @@ 
 
 	public function setVariations($variations) {
 		$this->setProperty("class", $this->_baseClass);
-		if (\is_string($variations))
-			$variations=\explode(" ", $variations);
+		if (\is_string($variations)) {
+					$variations=\explode(" ", $variations);
+		}
 		foreach ( $variations as $variation ) {
 			$this->addVariation($variation);
 		}
@@ -48,8 +49,9 @@ 
 	}
 
 	public function addVariations($variations=array()) {
-		if (\is_string($variations))
-			$variations=\explode(" ", $variations);
+		if (\is_string($variations)) {
+					$variations=\explode(" ", $variations);
+		}
 		foreach ( $variations as $variation ) {
 			$this->addVariation($variation);
 		}
@@ -57,8 +59,9 @@ 
 	}
 
 	public function addStates($states=array()) {
-		if (\is_string($states))
-			$states=\explode(" ", $states);
+		if (\is_string($states)) {
+					$states=\explode(" ", $states);
+		}
 		foreach ( $states as $state ) {
 			$this->addState($state);
 		}
@@ -67,8 +70,9 @@ 
 
 	public function setStates($states) {
 		$this->setProperty("class", $this->_baseClass);
-		if (\is_string($states))
-			$states=\explode(" ", $states);
+		if (\is_string($states)) {
+					$states=\explode(" ", $states);
+		}
 		foreach ( $states as $state ) {
 			$this->addState($state);
 		}

Ajax/semantic/html/collections/HtmlTable.php

Doc Comments +8 added lines

@@ -17,6 +17,11 @@ 
 class HtmlTable extends HtmlSemDoubleElement {
 	private $_colCount;
 
+	/**
+	 * @param string $identifier
+	 * @param integer $rowCount
+	 * @param integer $colCount
+	 */
 	public function __construct($identifier, $rowCount, $colCount) {
 		parent::__construct($identifier, "table", "ui table");
 		$this->content=array ();
@@ -146,6 +151,9 @@ 
 		return $this->colAlign($colIndex, "colLeft");
 	}
 
+	/**
+	 * @param string $function
+	 */
 	private function colAlign($colIndex, $function) {
 		if (\is_array($colIndex)) {
 			foreach ( $colIndex as $cIndex ) {

Spacing +12 added lines, -12 removed lines

@@ -19,9 +19,9 @@ 
 
 	public function __construct($identifier, $rowCount, $colCount) {
 		parent::__construct($identifier, "table", "ui table");
-		$this->content=array ();
+		$this->content=array();
 		$this->setRowCount($rowCount, $colCount);
-		$this->_variations=[ Variation::CELLED,Variation::PADDED,Variation::COMPACT ];
+		$this->_variations=[Variation::CELLED, Variation::PADDED, Variation::COMPACT];
 	}
 
 	/**
@@ -30,9 +30,9 @@ 
 	 * @return HtmlTableContent
 	 */
 	private function getPart($key) {
-		if (\array_key_exists($key, $this->content) === false) {
+		if (\array_key_exists($key, $this->content)===false) {
 			$this->content[$key]=new HtmlTableContent("", $key);
-			if ($key !== "tbody") {
+			if ($key!=="tbody") {
 				$this->content[$key]->setRowCount(1, $this->_colCount);
 			}
 		}
@@ -69,7 +69,7 @@ 
 	 * @return boolean
 	 */
 	public function hasPart($key) {
-		return \array_key_exists($key, $this->content) === true;
+		return \array_key_exists($key, $this->content)===true;
 	}
 
 	/**
@@ -148,10 +148,10 @@ 
 
 	private function colAlign($colIndex, $function) {
 		if (\is_array($colIndex)) {
-			foreach ( $colIndex as $cIndex ) {
+			foreach ($colIndex as $cIndex) {
 				$this->colAlign($cIndex, $function);
 			}
-		} else {
+		}else {
 			if ($this->hasPart("thead")) {
 				$this->getHeader()->$function($colIndex);
 			}
@@ -166,8 +166,8 @@ 
 
 	public function setBasic($very=false) {
 		if ($very)
-			$this->addToPropertyCtrl("class", "very", array ("very" ));
-		return $this->addToPropertyCtrl("class", "basic", array ("basic" ));
+			$this->addToPropertyCtrl("class", "very", array("very"));
+		return $this->addToPropertyCtrl("class", "basic", array("basic"));
 	}
 
 	public function setCollapsing() {
@@ -228,9 +228,9 @@ 
 	 * @see \Ajax\semantic\html\base\HtmlSemDoubleElement::compile()
 	 */
 	public function compile(JsUtils $js=NULL, View $view=NULL) {
-		$this->content=JArray::sortAssociative($this->content, [ "thead","tbody","tfoot" ]);
+		$this->content=JArray::sortAssociative($this->content, ["thead", "tbody", "tfoot"]);
 		if ($this->propertyContains("class", "sortable")) {
-			$this->addEvent("execute", "$('#" . $this->identifier . "').tablesort();");
+			$this->addEvent("execute", "$('#".$this->identifier."').tablesort();");
 		}
 		return parent::compile($js, $view);
 	}
@@ -245,7 +245,7 @@ 
 		$result=$function($object);
 		if (\is_array($result)) {
 			return $this->addRow($function($object));
-		} else {
+		}else {
 			return $this->getBody()->_addRow($result);
 		}
 	}

Braces +3 added lines, -2 removed lines

@@ -165,8 +165,9 @@
 	}
 
 	public function setBasic($very=false) {
-		if ($very)
-			$this->addToPropertyCtrl("class", "very", array ("very" ));
+		if ($very) {
+					$this->addToPropertyCtrl("class", "very", array ("very" ));
+		}
 		return $this->addToPropertyCtrl("class", "basic", array ("basic" ));
 	}
 

Ajax/semantic/html/content/table/HtmlTableContent.php

Doc Comments +10 added lines, -1 removed lines

@@ -68,10 +68,16 @@ 
 		return $tr;
 	}
 
+	/**
+	 * @param integer $value
+	 */
 	public function newRow($value) {
 		return $this->createItem($value);
 	}
 
+	/**
+	 * @param integer $colCount
+	 */
 	public function addRow($colCount) {
 		return $this->addItem($colCount);
 	}
@@ -97,7 +103,7 @@ 
 	/**
 	 *
 	 * @param int $index
-	 * @return \Ajax\semantic\html\content\HtmlTR
+	 * @return \Ajax\common\html\HtmlDoubleElement
 	 */
 	public function getRow($index) {
 		return $this->getItem($index);
@@ -170,6 +176,9 @@ 
 		return $this;
 	}
 
+	/**
+	 * @param string $function
+	 */
 	private function colAlign($colIndex, $function) {
 		$count=$this->count();
 		for($i=0; $i < $count; $i++) {

Spacing +21 added lines, -21 removed lines

@@ -11,7 +11,7 @@ 
  *
  */
 class HtmlTableContent extends HtmlSemCollection {
-	protected $_tdTagNames=[ "thead" => "th","tbody" => "td","tfoot" => "th" ];
+	protected $_tdTagNames=["thead" => "th", "tbody" => "td", "tfoot" => "th"];
 
 	/**
 	 *
@@ -34,7 +34,7 @@ 
 	 */
 	public function setRowCount($rowCount, $colCount) {
 		$count=$this->count();
-		for($i=$count; $i < $rowCount; $i++) {
+		for ($i=$count; $i<$rowCount; $i++) {
 			$this->addItem($colCount);
 		}
 		/*
@@ -62,7 +62,7 @@ 
 		$tr=new HtmlTR("", $value);
 		$tr->setContainer($this, $count);
 		$tr->setTdTagName($this->_tdTagNames[$this->tagName]);
-		if (isset($value) === true) {
+		if (isset($value)===true) {
 			$tr->setColCount($value);
 		}
 		return $tr;
@@ -112,7 +112,7 @@ 
 	 */
 	public function setCellValue($row, $col, $value="") {
 		$cell=$this->getCell($row, $col);
-		if (isset($cell) === true) {
+		if (isset($cell)===true) {
 			$cell->setValue($value);
 		}
 		return $this;
@@ -125,16 +125,16 @@ 
 	public function setValues($values=array()) {
 		$count=$this->count();
 		$isArray=true;
-		if (\is_array($values) === false) {
+		if (\is_array($values)===false) {
 			$values=\array_fill(0, $count, $values);
 			$isArray=false;
 		}
-		if (JArray::dimension($values) == 1 && $isArray)
-			$values=[ $values ];
+		if (JArray::dimension($values)==1 && $isArray)
+			$values=[$values];
 
 		$count=\min(\sizeof($values), $count);
 
-		for($i=0; $i < $count; $i++) {
+		for ($i=0; $i<$count; $i++) {
 			$row=$this->content[$i];
 			$row->setValues($values[$i]);
 		}
@@ -143,11 +143,11 @@ 
 
 	public function setColValues($colIndex, $values=array()) {
 		$count=$this->count();
-		if (\is_array($values) === false) {
+		if (\is_array($values)===false) {
 			$values=\array_fill(0, $count, $values);
 		}
 		$count=\min(\sizeof($values), $count);
-		for($i=0; $i < $count; $i++) {
+		for ($i=0; $i<$count; $i++) {
 			$this->getCell($i, $colIndex)->setValue($values[$i]);
 		}
 		return $this;
@@ -155,7 +155,7 @@ 
 
 	public function addColVariations($colIndex, $variations=array()) {
 		$count=$this->count();
-		for($i=0; $i < $count; $i++) {
+		for ($i=0; $i<$count; $i++) {
 			$this->getCell($i, $colIndex)->addVariations($variations);
 		}
 		return $this;
@@ -163,7 +163,7 @@ 
 
 	public function setRowValues($rowIndex, $values=array()) {
 		$count=$this->count();
-		if (\is_array($values) === false) {
+		if (\is_array($values)===false) {
 			$values=\array_fill(0, $count, $values);
 		}
 		$this->getItem($rowIndex)->setValues($values);
@@ -172,9 +172,9 @@ 
 
 	private function colAlign($colIndex, $function) {
 		$count=$this->count();
-		for($i=0; $i < $count; $i++) {
+		for ($i=0; $i<$count; $i++) {
 			$index=$this->content[$i]->getColPosition($colIndex);
-			if ($index !== NULL)
+			if ($index!==NULL)
 				$this->getCell($i, $index)->$function();
 		}
 		return $this;
@@ -206,7 +206,7 @@ 
 	 */
 	public function getColCount() {
 		$result=0;
-		if ($this->count() > 0)
+		if ($this->count()>0)
 			$result=$this->getItem(0)->getColCount();
 		return $result;
 	}
@@ -220,10 +220,10 @@ 
 	public function delete($rowIndex, $colIndex=NULL) {
 		if (isset($colIndex)) {
 			$row=$this->getItem($rowIndex);
-			if (isset($row) === true) {
+			if (isset($row)===true) {
 				$row->delete($colIndex);
 			}
-		} else {
+		}else {
 			$this->removeItem($rowIndex);
 		}
 		return $this;
@@ -252,7 +252,7 @@ 
 	 */
 	public function conditionalCellFormat($callback, $format) {
 		$rows=$this->content;
-		foreach ( $rows as $row ) {
+		foreach ($rows as $row) {
 			$row->conditionalCellFormat($callback, $format);
 		}
 		return $this;
@@ -265,7 +265,7 @@ 
 	 */
 	public function conditionalRowFormat($callback, $format) {
 		$rows=$this->content;
-		foreach ( $rows as $row ) {
+		foreach ($rows as $row) {
 			$row->conditionalRowFormat($callback, $format);
 		}
 		return $this;
@@ -277,7 +277,7 @@ 
 	 */
 	public function applyCells($callback) {
 		$rows=$this->content;
-		foreach ( $rows as $row ) {
+		foreach ($rows as $row) {
 			$row->applyCells($callback);
 		}
 		return $this;
@@ -289,7 +289,7 @@ 
 	 */
 	public function applyRows($callback) {
 		$rows=$this->content;
-		foreach ( $rows as $row ) {
+		foreach ($rows as $row) {
 			$row->apply($callback);
 		}
 		return $this;

Braces +12 added lines, -8 removed lines

@@ -22,8 +22,9 @@ 
 	 */
 	public function __construct($identifier, $tagName="tbody", $rowCount=NULL, $colCount=NULL) {
 		parent::__construct($identifier, $tagName, "");
-		if (isset($rowCount) && isset($colCount))
-			$this->setRowCount($rowCount, $colCount);
+		if (isset($rowCount) && isset($colCount)) {
+					$this->setRowCount($rowCount, $colCount);
+		}
 	}
 
 	/**
@@ -129,8 +130,9 @@ 
 			$values=\array_fill(0, $count, $values);
 			$isArray=false;
 		}
-		if (JArray::dimension($values) == 1 && $isArray)
-			$values=[ $values ];
+		if (JArray::dimension($values) == 1 && $isArray) {
+					$values=[ $values ];
+		}
 
 		$count=\min(\sizeof($values), $count);
 
@@ -174,8 +176,9 @@ 
 		$count=$this->count();
 		for($i=0; $i < $count; $i++) {
 			$index=$this->content[$i]->getColPosition($colIndex);
-			if ($index !== NULL)
-				$this->getCell($i, $index)->$function();
+			if ($index !== NULL) {
+							$this->getCell($i, $index)->$function();
+			}
 		}
 		return $this;
 	}
@@ -206,8 +209,9 @@ 
 	 */
 	public function getColCount() {
 		$result=0;
-		if ($this->count() > 0)
-			$result=$this->getItem(0)->getColCount();
+		if ($this->count() > 0) {
+					$result=$this->getItem(0)->getColCount();
+		}
 		return $result;
 	}
 

Ajax/service/JArray.php

Spacing +15 added lines, -15 removed lines

@@ -5,7 +5,7 @@ 
 class JArray {
 
 	public static function isAssociative($array) {
-		return (array_values($array) !== $array);
+		return (array_values($array)!==$array);
 		// return (array_keys($array)!==range(0, count($array)-1));
 	}
 
@@ -14,7 +14,7 @@ 
 			return $array[$key];
 		}
 		$values=array_values($array);
-		if ($pos < sizeof($values))
+		if ($pos<sizeof($values))
 			return $values[$pos];
 	}
 
@@ -22,12 +22,12 @@ 
 		$result=NULL;
 		if (array_key_exists($key, $array)) {
 			$result=$array[$key];
-			if ($condition($result) === true)
+			if ($condition($result)===true)
 				return $result;
 		}
 		$values=array_values($array);
-		foreach ( $values as $val ) {
-			if ($condition($val) === true)
+		foreach ($values as $val) {
+			if ($condition($val)===true)
 				return $val;
 		}
 		return $result;
@@ -36,7 +36,7 @@ 
 	public static function getDefaultValue($array, $key, $default=NULL) {
 		if (array_key_exists($key, $array)) {
 			return $array[$key];
-		} else
+		}else
 			return $default;
 	}
 
@@ -44,13 +44,13 @@ 
 		$result="";
 		if (\is_array($glue)) {
 			$size=\sizeof($pieces);
-			if ($size > 0) {
-				for($i=0; $i < $size - 1; $i++) {
-					$result.=$pieces[$i] . @$glue[$i];
+			if ($size>0) {
+				for ($i=0; $i<$size-1; $i++) {
+					$result.=$pieces[$i].@$glue[$i];
 				}
-				$result.=$pieces[$size - 1];
+				$result.=$pieces[$size-1];
 			}
-		} else {
+		}else {
 			$result=\implode($glue, $pieces);
 		}
 		return $result;
@@ -58,16 +58,16 @@ 
 
 	public static function dimension($array) {
 		if (is_array(reset($array))) {
-			$return=self::dimension(reset($array)) + 1;
-		} else {
+			$return=self::dimension(reset($array))+1;
+		}else {
 			$return=1;
 		}
 		return $return;
 	}
 
 	public static function sortAssociative($array, $sortedKeys=array()) {
-		$newArray=array ();
-		foreach ( $sortedKeys as $key ) {
+		$newArray=array();
+		foreach ($sortedKeys as $key) {
 			if (\array_key_exists($key, $array)) {
 				$newArray[$key]=$array[$key];
 			}

Braces +12 added lines, -8 removed lines

@@ -14,21 +14,24 @@ 
 			return $array[$key];
 		}
 		$values=array_values($array);
-		if ($pos < sizeof($values))
-			return $values[$pos];
+		if ($pos < sizeof($values)) {
+					return $values[$pos];
+		}
 	}
 
 	public static function getConditionalValue($array, $key, $condition) {
 		$result=NULL;
 		if (array_key_exists($key, $array)) {
 			$result=$array[$key];
-			if ($condition($result) === true)
-				return $result;
+			if ($condition($result) === true) {
+							return $result;
+			}
 		}
 		$values=array_values($array);
 		foreach ( $values as $val ) {
-			if ($condition($val) === true)
-				return $val;
+			if ($condition($val) === true) {
+							return $val;
+			}
 		}
 		return $result;
 	}
@@ -36,8 +39,9 @@ 
 	public static function getDefaultValue($array, $key, $default=NULL) {
 		if (array_key_exists($key, $array)) {
 			return $array[$key];
-		} else
-			return $default;
+		} else {
+					return $default;
+		}
 	}
 
 	public static function implode($glue, $pieces) {