jcheron / phalcon-jquery

Ajax/common/html/BaseHtml.php

Doc Comments +35 added lines

@@ -108,6 +108,10 @@ 
 		return true;
 	}
 
+	/**
+	 * @param string $propertyName
+	 * @param string $value
+	 */
 	protected function propertyContains($propertyName,$value){
 		$values=$this->getProperty($propertyName);
 		if(isset($values)){
@@ -142,12 +146,19 @@ 
 		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)) {
@@ -172,6 +183,9 @@ 
 		return $this->addToProperty($name, $value);
 	}
 
+	/**
+	 * @param string $name
+	 */
 	public function addToPropertyCtrl($name, $value, $typeCtrl) {
 		// if($this->ctrl($name, $value, $typeCtrl)===true){
 		return $this->addToPropertyUnique($name, $value, $typeCtrl);
@@ -239,6 +253,9 @@ 
 
 	}
 
+	/**
+	 * @param string $before
+	 */
 	public function wrap($before, $after="") {
 		if(isset($before)){
 			$this->wrapBefore.=$before;
@@ -307,6 +324,9 @@ 
 		}
 	}
 
+	/**
+	 * @param string $operation
+	 */
 	public function _ajaxOn($operation, $event, $url, $responseElement="", $parameters=array()) {
 		$params=array (
 				"url" => $url,
@@ -317,14 +337,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);
@@ -334,6 +363,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);
@@ -383,6 +415,9 @@ 
 		return $value;
 	}
 
+	/**
+	 * @param string $jqueryCall
+	 */
 	public function jsDoJquery($jqueryCall, $param=""){
 		return "$('#".$this->identifier."').".$jqueryCall."(".$this->_prep_value($param).");";
 	}

Spacing +39 added lines, -39 removed lines

@@ -16,8 +16,8 @@ 
 abstract class BaseHtml extends BaseWidget {
 	protected $_template;
 	protected $tagName;
-	protected $properties=array ();
-	protected $events=array ();
+	protected $properties=array();
+	protected $events=array();
 	protected $wrapBefore="";
 	protected $wrapAfter="";
 	protected $_bsComponent;
@@ -56,7 +56,7 @@ 
 
 	public function addToProperty($name, $value, $separator=" ") {
 		$v=@$this->properties[$name];
-		if (isset($v)&&$v!=="")
+		if (isset($v) && $v!=="")
 			$v=$v.$separator.$value;
 		else
 			$v=$value;
@@ -71,11 +71,11 @@ 
 
 	public function compile(JsUtils $js=NULL, View $view=NULL) {
 		$result=$this->getTemplate();
-		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);
@@ -87,7 +87,7 @@ 
 		if (isset($view)===true) {
 			$controls=$view->getVar("q");
 			if (isset($controls)===false) {
-				$controls=array ();
+				$controls=array();
 			}
 			$controls [$this->identifier]=$result;
 			$view->setVar("q", $controls);
@@ -100,7 +100,7 @@ 
 			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);
 			}
@@ -108,9 +108,9 @@ 
 		return true;
 	}
 
-	protected function propertyContains($propertyName,$value){
+	protected function propertyContains($propertyName, $value) {
 		$values=$this->getProperty($propertyName);
-		if(isset($values)){
+		if (isset($values)) {
 			return JString::contains($values, $value);
 		}
 		return false;
@@ -137,13 +137,13 @@ 
 		return $this;
 	}
 
-	protected function removePropertyValue($name,$value){
+	protected function removePropertyValue($name, $value) {
 		$this->properties[$name]=\str_replace($value, "", $this->properties[$name]);
 		return $this;
 	}
 
-	protected function removeProperty($name){
-		if(\array_key_exists($name, $this->properties))
+	protected function removeProperty($name) {
+		if (\array_key_exists($name, $this->properties))
 			unset($this->properties[$name]);
 		return $this;
 	}
@@ -196,28 +196,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);
 					$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 {
+			}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
 					}
 				}
@@ -226,21 +226,21 @@ 
 		return $array;
 	}
 
-	public function fromDatabaseObjects($objects,$function) {
-		if(isset($objects)){
-			foreach ($objects as $object){
-				$this->fromDatabaseObject($object,$function);
+	public function fromDatabaseObjects($objects, $function) {
+		if (isset($objects)) {
+			foreach ($objects as $object) {
+				$this->fromDatabaseObject($object, $function);
 			}
 		}
 		return $this;
 	}
 
-	public function fromDatabaseObject($object,$function){
+	public function fromDatabaseObject($object, $function) {
 
 	}
 
 	public function wrap($before, $after="") {
-		if(isset($before)){
+		if (isset($before)) {
 			$this->wrapBefore.=$before;
 		}
 		$this->wrapAfter=$after.$this->wrapAfter;
@@ -262,13 +262,13 @@ 
 		if (array_key_exists($event, $this->events)) {
 			if (is_array($this->events [$event])) {
 				$this->events [$event] []=$jsCode;
-			} else {
-				$this->events [$event]=array (
+			}else {
+				$this->events [$event]=array(
 						$this->events [$event],
 						$jsCode
 				);
 			}
-		} else {
+		}else {
 			$this->events [$event]=$jsCode;
 		}
 	}
@@ -287,14 +287,14 @@ 
 
 	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 {
+						}else {
 							$code.="\n".$jsC;
 						}
 					}
@@ -308,7 +308,7 @@ 
 	}
 
 	public function _ajaxOn($operation, $event, $url, $responseElement="", $parameters=array()) {
-		$params=array (
+		$params=array(
 				"url" => $url,
 				"responseElement" => $responseElement
 		);
@@ -347,7 +347,7 @@ 
 		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);
 				$index++;
@@ -361,7 +361,7 @@ 
 		return null;
 	}
 
-	public function __toString(){
+	public function __toString() {
 		return $this->compile();
 	}
 
@@ -377,25 +377,25 @@ 
 		if (is_array($value)) {
 			$value=implode(",", $value);
 		}
-		if (strrpos($value, 'this')===false&&strrpos($value, 'event')===false) {
+		if (strrpos($value, 'this')===false && strrpos($value, 'event')===false) {
 			$value='"'.$value.'"';
 		}
 		return $value;
 	}
 
-	public function jsDoJquery($jqueryCall, $param=""){
+	public function jsDoJquery($jqueryCall, $param="") {
 		return "$('#".$this->identifier."').".$jqueryCall."(".$this->_prep_value($param).");";
 	}
 
-	public function jsHtml($content=""){
-		return $this->jsDoJquery("html",$content);
+	public function jsHtml($content="") {
+		return $this->jsDoJquery("html", $content);
 	}
 
-	public function jsShow(){
+	public function jsShow() {
 		return $this->jsDoJquery("show");
 	}
 
-	public function jsHide(){
+	public function jsHide() {
 		return $this->jsDoJquery("hide");
 	}
 }
\ No newline at end of file

Braces +26 added lines, -18 removed lines

@@ -50,16 +50,18 @@ 
 	}
 
 	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=" ") {
 		$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);
 	}
@@ -117,8 +119,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;
 	}
 
@@ -143,8 +146,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;
 	}
 
@@ -164,8 +168,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);
 		}
@@ -348,15 +353,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;
 	}

Ajax/common/html/HtmlCollection.php

Doc Comments +6 added lines

@@ -87,6 +87,9 @@ 
 		$this->addItem($function($object));
 	}
 
+	/**
+	 * @param \Closure $callBack
+	 */
 	public function apply($callBack){
 		foreach ($this->content as $item){
 			$callBack($item);
@@ -121,6 +124,9 @@ 
 		return $this->_bsComponent;
 	}
 
+	/**
+	 * @param string $tagName
+	 */
 	protected function contentAs($tagName){
 		foreach ($this->content as $item){
 			$item->setTagName($tagName);

Spacing +19 added lines, -19 removed lines

@@ -13,29 +13,29 @@ 
  */
 abstract class HtmlCollection extends HtmlDoubleElement {
 
-	public function __construct($identifier,$tagName="div"){
-		parent::__construct($identifier,$tagName);
+	public function __construct($identifier, $tagName="div") {
+		parent::__construct($identifier, $tagName);
 		$this->content=array();
 	}
 
-	public function addItems($items){
-		if(JArray::isAssociative($items)){
-			foreach ($items as $k=>$v){
-				$this->addItem([$k,$v]);
+	public function addItems($items) {
+		if (JArray::isAssociative($items)) {
+			foreach ($items as $k=>$v) {
+				$this->addItem([$k, $v]);
 			}
-		}else{
-			foreach ($items as $item){
+		}else {
+			foreach ($items as $item) {
 				$this->addItem($item);
 			}
 		}
 	}
 
-	public function setItems($items){
+	public function setItems($items) {
 		$this->content=$items;
 		return $this;
 	}
 
-	public function getItems(){
+	public function getItems() {
 		return $this->content;
 	}
 
@@ -44,9 +44,9 @@ 
 	 * @param HtmlDoubleElement|string $item
 	 * @return \Ajax\common\html\HtmlDoubleElement
 	 */
-	public function addItem($item){
+	public function addItem($item) {
 		$itemO=$item;
-		if($this->createCondition($item)===true){
+		if ($this->createCondition($item)===true) {
 			$itemO=$this->createItem($item);
 		}
 		$this->addContent($itemO);
@@ -72,11 +72,11 @@ 
 		return $this;
 	}
 
-	public function removeItem($index){
+	public function removeItem($index) {
 		return array_splice($this->content, $index, 1);
 	}
 
-	public function count(){
+	public function count() {
 		return \sizeof($this->content);
 	}
 
@@ -87,8 +87,8 @@ 
 		$this->addItem($function($object));
 	}
 
-	public function apply($callBack){
-		foreach ($this->content as $item){
+	public function apply($callBack) {
+		foreach ($this->content as $item) {
 			$callBack($item);
 		}
 	}
@@ -106,7 +106,7 @@ 
 	 */
 	protected abstract function createItem($value);
 
-	protected function createCondition($value){
+	protected function createCondition($value) {
 		return \is_object($value)===false;
 	}
 
@@ -121,8 +121,8 @@ 
 		return $this->_bsComponent;
 	}
 
-	protected function contentAs($tagName){
-		foreach ($this->content as $item){
+	protected function contentAs($tagName) {
+		foreach ($this->content as $item) {
 			$item->setTagName($tagName);
 		}
 	}

Ajax/semantic/html/base/HtmlSemNavElement.php

Doc Comments +10 added lines, -3 removed lines

@@ -22,6 +22,11 @@ 
 	protected $_contentSeparator="";
 
 
+	/**
+	 * @param string $identifier
+	 * @param string $tagName
+	 * @param string $baseClass
+	 */
 	public function __construct($identifier,$tagName,$baseClass){
 		parent::__construct($identifier,$tagName,$baseClass);
 		$this->root="";
@@ -32,8 +37,7 @@ 
 	 * Associate an ajax get to the elements, displayed in $targetSelector
 	 * $attr member is used to build each element url
 	 * @param string $targetSelector the target of the get
-	 * @param string $attr the html attribute used to build the elements url
-	 * @return HtmlNavElement
+	 * @return HtmlSemNavElement
 	 */
 	public function autoGetOnClick($targetSelector){
 		return $this->getOnClick($this->root, $targetSelector,array("attr"=>$this->attr));
@@ -65,7 +69,7 @@ 
 	/**
 	 * Define the html attribute for each element url in ajax
 	 * @param string $attr html attribute
-	 * @return HtmlNavElement
+	 * @return HtmlSemNavElement
 	 */
 	public function setAttr($attr) {
 		$this->attr = $attr;
@@ -105,6 +109,9 @@ 
 		return $this;
 	}
 
+	/**
+	 * @param integer $index
+	 */
 	protected function getContentDivider($index){
 		if(\is_array($this->_contentSeparator)===true){
 			return @$this->_contentSeparator[$index];

Spacing +20 added lines, -20 removed lines

@@ -22,8 +22,8 @@ 
 	protected $_contentSeparator="";
 
 
-	public function __construct($identifier,$tagName,$baseClass){
-		parent::__construct($identifier,$tagName,$baseClass);
+	public function __construct($identifier, $tagName, $baseClass) {
+		parent::__construct($identifier, $tagName, $baseClass);
 		$this->root="";
 		$this->attr="data-ajax";
 	}
@@ -35,11 +35,11 @@ 
 	 * @param string $attr the html attribute used to build the elements url
 	 * @return HtmlNavElement
 	 */
-	public function autoGetOnClick($targetSelector){
-		return $this->getOnClick($this->root, $targetSelector,array("attr"=>$this->attr));
+	public function autoGetOnClick($targetSelector) {
+		return $this->getOnClick($this->root, $targetSelector, array("attr"=>$this->attr));
 	}
 
-	public function contentAsString(){
+	public function contentAsString() {
 		return JArray::implode($this->_contentSeparator, $this->content);
 	}
 
@@ -47,15 +47,15 @@ 
 	 * Generate the jquery script to set the elements to the HtmlNavElement
 	 * @param JsUtils $jsUtils
 	 */
-	public function jsSetContent(JsUtils $jsUtils){
-		$jsUtils->html("#".$this->identifier,str_replace("\"","'", $this->contentAsString()),true);
+	public function jsSetContent(JsUtils $jsUtils) {
+		$jsUtils->html("#".$this->identifier, str_replace("\"", "'", $this->contentAsString()), true);
 	}
 
 	public function getRoot() {
 		return $this->root;
 	}
 	public function setRoot($root) {
-		$this->root = $root;
+		$this->root=$root;
 		return $this;
 	}
 	public function getAttr() {
@@ -68,12 +68,12 @@ 
 	 * @return HtmlNavElement
 	 */
 	public function setAttr($attr) {
-		$this->attr = $attr;
+		$this->attr=$attr;
 		return $this;
 	}
 
 	public function __call($method, $args) {
-		if(isset($this->$method) && is_callable($this->$method)) {
+		if (isset($this->$method) && is_callable($this->$method)) {
 			return call_user_func_array(
 					$this->$method,
 					$args
@@ -81,32 +81,32 @@ 
 		}
 	}
 
-	public abstract function fromDispatcher($dispatcher,$startIndex=0);
+	public abstract function fromDispatcher($dispatcher, $startIndex=0);
 
 
-	public function setContentDivider($divider,$index=NULL) {
+	public function setContentDivider($divider, $index=NULL) {
 		$divider="<div class='divider'> {$divider} </div>";
 		return $this->setDivider($divider, $index);
 	}
 
-	public function setIconContentDivider($iconContentDivider,$index=NULL) {
+	public function setIconContentDivider($iconContentDivider, $index=NULL) {
 		$contentDivider="<i class='".$iconContentDivider." icon divider'></i>";
 		return $this->setDivider($contentDivider, $index);
 	}
 
-	protected function setDivider($divider,$index){
-		if(isset($index)){
-			if(\is_array($this->_contentSeparator)===false)
-				$this->_contentSeparator=array_fill (0, $this->count()-1,$this->_contentSeparator);
+	protected function setDivider($divider, $index) {
+		if (isset($index)) {
+			if (\is_array($this->_contentSeparator)===false)
+				$this->_contentSeparator=array_fill(0, $this->count()-1, $this->_contentSeparator);
 			$this->_contentSeparator[$index]=$divider;
-		}else{
+		}else {
 			$this->_contentSeparator=$divider;
 		}
 		return $this;
 	}
 
-	protected function getContentDivider($index){
-		if(\is_array($this->_contentSeparator)===true){
+	protected function getContentDivider($index) {
+		if (\is_array($this->_contentSeparator)===true) {
 			return @$this->_contentSeparator[$index];
 		}
 		return $this->_contentSeparator;

Braces +4 added lines, -3 removed lines

@@ -96,10 +96,11 @@
 
 	protected function setDivider($divider,$index){
 		if(isset($index)){
-			if(\is_array($this->_contentSeparator)===false)
-				$this->_contentSeparator=array_fill (0, $this->count()-1,$this->_contentSeparator);
+			if(\is_array($this->_contentSeparator)===false) {
+							$this->_contentSeparator=array_fill (0, $this->count()-1,$this->_contentSeparator);
+			}
 			$this->_contentSeparator[$index]=$divider;
-		}else{
+		} else{
 			$this->_contentSeparator=$divider;
 		}
 		return $this;

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

Doc Comments +3 added lines

@@ -7,6 +7,9 @@
 trait IconTrait {
 	private $_hasIcon=false;
 
+	/**
+	 * @param boolean $direction
+	 */
 	public function addIcon($icon,$direction=Direction::LEFT){
 		if($this->_hasIcon===false){
 			$iconO=$icon;

Spacing +9 added lines, -9 removed lines

@@ -7,16 +7,16 @@ 
 trait IconTrait {
 	private $_hasIcon=false;
 
-	public function addIcon($icon,$direction=Direction::LEFT){
-		if($this->_hasIcon===false){
+	public function addIcon($icon, $direction=Direction::LEFT) {
+		if ($this->_hasIcon===false) {
 			$iconO=$icon;
-			if(\is_string($icon)){
+			if (\is_string($icon)) {
 				$iconO=new HtmlIcon("icon-".$this->identifier, $icon);
 			}
 			$this->addToPropertyCtrl("class", $direction." icon", Direction::getConstantValues("icon"));
-			$this->addContent($iconO,false);
+			$this->addContent($iconO, false);
 			$this->_hasIcon=true;
-		}else{
+		}else {
 			$iconO=$this->getIcon();
 			$iconO->setIcon($icon);
 			$this->addToPropertyCtrl("class", $direction." icon", Direction::getConstantValues("icon"));
@@ -24,10 +24,10 @@ 
 		return $iconO;
 	}
 
-	public function getIcon(){
-		if(\is_array($this->content)){
-			foreach ($this->content as $item){
-				if($item instanceof HtmlIcon)
+	public function getIcon() {
+		if (\is_array($this->content)) {
+			foreach ($this->content as $item) {
+				if ($item instanceof HtmlIcon)
 					return $item;
 			}
 		}

Braces +4 added lines, -3 removed lines

@@ -16,7 +16,7 @@ 
 			$this->addToPropertyCtrl("class", $direction." icon", Direction::getConstantValues("icon"));
 			$this->addContent($iconO,false);
 			$this->_hasIcon=true;
-		}else{
+		} else{
 			$iconO=$this->getIcon();
 			$iconO->setIcon($icon);
 			$this->addToPropertyCtrl("class", $direction." icon", Direction::getConstantValues("icon"));
@@ -27,8 +27,9 @@ 
 	public function getIcon(){
 		if(\is_array($this->content)){
 			foreach ($this->content as $item){
-				if($item instanceof HtmlIcon)
-					return $item;
+				if($item instanceof HtmlIcon) {
+									return $item;
+				}
 			}
 		}
 	}

Ajax/semantic/html/content/HtmlDropdownItem.php

Doc Comments +7 added lines

@@ -11,6 +11,10 @@ 
 
 class HtmlDropdownItem extends HtmlSemDoubleElement {
 	use IconTrait;
+
+	/**
+	 * @param string $identifier
+	 */
 	public function __construct($identifier, $content="",$value=NULL,$image=NULL) {
 		parent::__construct($identifier, "a");
 		$this->setClass("item");
@@ -61,6 +65,9 @@ 
 		return $this;
 	}
 
+	/**
+	 * @param string $content
+	 */
 	public function setContent($content){
 		if($content==="-"){
 			$this->asDivider();

Ajax/semantic/html/collections/HtmlBreadcrumb.php

Spacing +37 added lines, -37 removed lines

@@ -14,7 +14,7 @@ 
  * @author jc
  * @version 1.001
  */
-class HtmlBreadcrumb extends HtmlSemNavElement{
+class HtmlBreadcrumb extends HtmlSemNavElement {
 	/**
 	 * @var integer the start index for href generation
 	 */
@@ -40,13 +40,13 @@ 
 	 * @param boolean $autoActive sets the last element's class to <b>active</b> if true
 	 * @param function $hrefFunction the function who generates the href elements. default : function($e){return $e->getContent()}
 	 */
-	public function __construct( $identifier,$items=array(),$autoActive=true,$startIndex=0,$hrefFunction=NULL){
-		parent::__construct( $identifier, "div", "ui breadcrumb");
+	public function __construct($identifier, $items=array(), $autoActive=true, $startIndex=0, $hrefFunction=NULL) {
+		parent::__construct($identifier, "div", "ui breadcrumb");
 		$this->startIndex=$startIndex;
 		$this->autoActive=$autoActive;
 		$this->_contentSeparator="<div class='divider'> / </div>";
-		$this->_hrefFunction=function ($e){return $e->getContent();};
-		if(isset($hrefFunction)){
+		$this->_hrefFunction=function($e) {return $e->getContent(); };
+		if (isset($hrefFunction)) {
 			$this->_hrefFunction=$hrefFunction;
 		}
 		$this->addItems($items);
@@ -59,23 +59,23 @@ 
 	 * @param string $attr the html attribute used to build the elements url
 	 * @return HtmlBreadcrumbs
 	 */
-	public function autoGetOnClick($targetSelector){
-		return $this->getOnClick($this->root, $targetSelector,array("attr"=>$this->attr));
+	public function autoGetOnClick($targetSelector) {
+		return $this->getOnClick($this->root, $targetSelector, array("attr"=>$this->attr));
 	}
 
-	public function contentAsString(){
-		if($this->autoActive){
+	public function contentAsString() {
+		if ($this->autoActive) {
 			$this->setActive();
 		}
 		return parent::contentAsString();
 	}
 
-	public function setActive($index=null){
-		if(!isset($index)){
+	public function setActive($index=null) {
+		if (!isset($index)) {
 			$index=sizeof($this->content)-1;
 		}
 		$activeItem=$this->content[$index];
-		$activeItem->addToProperty("class","active");
+		$activeItem->addToProperty("class", "active");
 		$activeItem->setTagName("div");
 	}
 
@@ -84,15 +84,15 @@ 
 	 * @param Dispatcher $dispatcher the request dispatcher
 	 * @return \Ajax\bootstrap\html\HtmlBreadcrumbs
 	 */
-	public function fromDispatcher($dispatcher,$startIndex=0){
+	public function fromDispatcher($dispatcher, $startIndex=0) {
 		$this->startIndex=$startIndex;
 		$params=$dispatcher->getParams();
 		$action=$dispatcher->getActionName();
 		$items=array($dispatcher->getControllerName());
-		if(\sizeof($params)>0 || \strtolower($action)!="index" ){
+		if (\sizeof($params)>0 || \strtolower($action)!="index") {
 			$items[]=$action;
-			foreach ($params as $p){
-				if(\is_object($p)===false)
+			foreach ($params as $p) {
+				if (\is_object($p)===false)
 					$items[]=$p;
 			}
 		}
@@ -105,14 +105,14 @@ 
 	 * @param string $separator
 	 * @return string
 	 */
-	public function getHref($index=null,$separator="/"){
-		if(!isset($index)){
+	public function getHref($index=null, $separator="/") {
+		if (!isset($index)) {
 			$index=sizeof($this->content);
 		}
-		if($this->absolutePaths===true){
+		if ($this->absolutePaths===true) {
 			return $this->_hrefFunction($this->content[$index]);
-		}else{
-			return $this->root.implode($separator, array_slice(array_map(function($e){return $this->_hrefFunction($e);}, $this->content),$this->startIndex,$index+1));
+		}else {
+			return $this->root.implode($separator, array_slice(array_map(function($e) {return $this->_hrefFunction($e); }, $this->content), $this->startIndex, $index+1));
 		}
 	}
 
@@ -122,7 +122,7 @@ 
 	 * @return \Ajax\bootstrap\html\HtmlBreadcrumbs
 	 */
 	public function setHrefFunction($_hrefFunction) {
-		$this->_hrefFunction = $_hrefFunction;
+		$this->_hrefFunction=$_hrefFunction;
 		return $this;
 	}
 
@@ -132,7 +132,7 @@ 
 	}
 
 	public function setAutoActive($autoActive) {
-		$this->autoActive = $autoActive;
+		$this->autoActive=$autoActive;
 		return $this;
 	}
 
@@ -141,11 +141,11 @@ 
 	 * @see \Ajax\bootstrap\html\BaseHtml::compile()
 	 */
 	public function compile(JsUtils $js=NULL, View $view=NULL) {
-		if($this->autoActive){
+		if ($this->autoActive) {
 			$this->setActive();
 		}
 		$count=$this->count();
-		for($i=1;$i<$count;$i++){
+		for ($i=1; $i<$count; $i++) {
 			$this->content[$i]->wrap($this->getContentDivider($i-1));
 		}
 		return parent::compile($js, $view);
@@ -156,16 +156,16 @@ 
 	 * @see \Ajax\bootstrap\html\base\BaseHtml::on()
 	 */
 	public function on($event, $jsCode, $stopPropagation=false, $preventDefault=false) {
-		foreach ($this->content as $element){
-				$element->on($event,$jsCode,$stopPropagation,$preventDefault);
+		foreach ($this->content as $element) {
+				$element->on($event, $jsCode, $stopPropagation, $preventDefault);
 		}
 		return $this;
 	}
 
 
 	public function _ajaxOn($operation, $event, $url, $responseElement="", $parameters=array()) {
-		foreach ($this->content as $element){
-			if($element->getProperty($this->attr)!=NULL)
+		foreach ($this->content as $element) {
+			if ($element->getProperty($this->attr)!=NULL)
 				$element->_ajaxOn($operation, $event, $url, $responseElement, $parameters);
 		}
 		return $this;
@@ -177,35 +177,35 @@ 
 	 */
 	protected function createItem($value) {
 		$count=$this->count();
-		$itemO=new HtmlSemDoubleElement("item-".$this->identifier."-".$count,"a","section");
-		if(\is_array($value))
+		$itemO=new HtmlSemDoubleElement("item-".$this->identifier."-".$count, "a", "section");
+		if (\is_array($value))
 			$itemO->fromArray($value);
-		else{
+		else {
 			$itemO->setContent($value);
 			$itemO->setProperty($this->attr, $this->getHref($count));
 		}
 		return $itemO;
 	}
 
-	public function addIcon($icon,$index){
+	public function addIcon($icon, $index) {
 		$item=$this->getItem($index);
-		if(isset($item)){
+		if (isset($item)) {
 			$icon=new HtmlIcon("icon-".$this->identifier, $icon);
 			$item->wrapContent($icon);
 		}
 	}
 
-	public function addItem($item){
+	public function addItem($item) {
 		$itemO=parent::addItem($item);
 		$this->addToPropertyCtrl("class", "section", array("section"));
 		return $itemO;
 	}
 
-	public function asLinks(){
+	public function asLinks() {
 		$this->contentAs("a");
 	}
 
-	public function asTexts(){
+	public function asTexts() {
 		$this->contentAs("div");
 	}
 

Braces +10 added lines, -8 removed lines

@@ -92,8 +92,9 @@ 
 		if(\sizeof($params)>0 || \strtolower($action)!="index" ){
 			$items[]=$action;
 			foreach ($params as $p){
-				if(\is_object($p)===false)
-					$items[]=$p;
+				if(\is_object($p)===false) {
+									$items[]=$p;
+				}
 			}
 		}
 		return $this->addItems($items);
@@ -111,7 +112,7 @@ 
 		}
 		if($this->absolutePaths===true){
 			return $this->_hrefFunction($this->content[$index]);
-		}else{
+		} else{
 			return $this->root.implode($separator, array_slice(array_map(function($e){return $this->_hrefFunction($e);}, $this->content),$this->startIndex,$index+1));
 		}
 	}
@@ -165,8 +166,9 @@ 
 
 	public function _ajaxOn($operation, $event, $url, $responseElement="", $parameters=array()) {
 		foreach ($this->content as $element){
-			if($element->getProperty($this->attr)!=NULL)
-				$element->_ajaxOn($operation, $event, $url, $responseElement, $parameters);
+			if($element->getProperty($this->attr)!=NULL) {
+							$element->_ajaxOn($operation, $event, $url, $responseElement, $parameters);
+			}
 		}
 		return $this;
 	}
@@ -178,9 +180,9 @@ 
 	protected function createItem($value) {
 		$count=$this->count();
 		$itemO=new HtmlSemDoubleElement("item-".$this->identifier."-".$count,"a","section");
-		if(\is_array($value))
-			$itemO->fromArray($value);
-		else{
+		if(\is_array($value)) {
+					$itemO->fromArray($value);
+		} else{
 			$itemO->setContent($value);
 			$itemO->setProperty($this->attr, $this->getHref($count));
 		}

Ajax/semantic/html/elements/HtmlButton.php

Spacing +22 added lines, -22 removed lines

@@ -22,7 +22,7 @@ 
 	 * @param string $onClick JS Code for click event
 	 */
 	public function __construct($identifier, $value="", $cssStyle=null, $onClick=null) {
-		parent::__construct($identifier, "button","ui button");
+		parent::__construct($identifier, "button", "ui button");
 		$this->content=$value;
 		if (isset($cssStyle)) {
 			$this->setStyle($cssStyle);
@@ -48,25 +48,25 @@ 
 	 * @return \Ajax\semantic\html\HtmlButton default : ""
 	 */
 	public function setStyle($cssStyle) {
-		return $this->addToProperty("class",$cssStyle);
+		return $this->addToProperty("class", $cssStyle);
 	}
 
-	public function setFocusable($value=true){
-		if($value===true)
+	public function setFocusable($value=true) {
+		if ($value===true)
 			$this->setProperty("tabindex", "0");
-		else{
+		else {
 			$this->removeProperty("tabindex");
 		}
 		return $this;
 	}
 
-	public function setAnimated($content,$animation=""){
+	public function setAnimated($content, $animation="") {
 		$this->setTagName("div");
 		$this->addToProperty("class", "animated ".$animation);
-		$visible=new HtmlSemDoubleElement("visible-".$this->identifier,"div");
+		$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=$visible.$hidden;
@@ -76,9 +76,9 @@ 
 	 * @param string|HtmlIcon $icon
 	 * @return \Ajax\semantic\html\elements\HtmlButton
 	 */
-	public function asIcon($icon){
+	public function asIcon($icon) {
 		$iconO=$icon;
-		if(\is_string($icon)){
+		if (\is_string($icon)) {
 			$iconO=new HtmlIcon("icon-".$this->identifier, $icon);
 		}
 		$this->addToProperty("class", "icon");
@@ -92,14 +92,14 @@ 
 	 * @param string $before
 	 * @return \Ajax\semantic\html\elements\HtmlLabel
 	 */
-	public function addLabel($caption,$before=false){
+	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);
+		$this->addContent($label, $before);
 		return $label;
 	}
 	/*
@@ -108,7 +108,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;
@@ -118,7 +118,7 @@ 
 	 *  show it is currently the active user selection
 	 * @return \Ajax\semantic\html\elements\HtmlButton
 	 */
-	public function setActive(){
+	public function setActive() {
 		return $this->addToProperty("class", "active");
 	}
 
@@ -126,7 +126,7 @@ 
 	 * hint towards a positive consequence
 	 * @return \Ajax\semantic\html\elements\HtmlButton
 	 */
-	public function setPositive(){
+	public function setPositive() {
 		return $this->addToProperty("class", "positive");
 	}
 
@@ -134,7 +134,7 @@ 
 	 * hint towards a negative consequence
 	 * @return \Ajax\semantic\html\elements\HtmlButton
 	 */
-	public function setNegative(){
+	public function setNegative() {
 		return $this->addToProperty("class", "negative");
 	}
 
@@ -142,14 +142,14 @@ 
 	 * formatted to toggle on/off
 	 * @return \Ajax\semantic\html\elements\HtmlButton
 	 */
-	public function setToggle(){
+	public function setToggle() {
 		return $this->addToProperty("class", "toggle");
 	}
 
 	/**
 	 * @return \Ajax\semantic\html\elements\HtmlButton
 	 */
-	public function setCircular(){
+	public function setCircular() {
 		return $this->addToProperty("class", "circular");
 	}
 
@@ -157,11 +157,11 @@ 
 	 *  button is less pronounced
 	 * @return \Ajax\semantic\html\elements\HtmlButton
 	 */
-	public function setBasic(){
+	public function setBasic() {
 		return $this->addToProperty("class", "basic");
 	}
 
-	public function setEmphasis($value){
+	public function setEmphasis($value) {
 		return $this->addToPropertyCtrl("class", $value, Emphasis::getConstants());
 	}
 }
\ No newline at end of file

Braces +3 added lines, -3 removed lines

@@ -52,9 +52,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;

Ajax/semantic/html/elements/HtmlInput.php

Spacing +22 added lines, -22 removed lines

@@ -8,53 +8,53 @@
 use Ajax\semantic\html\base\constants\Variation;
 use Ajax\semantic\html\base\traits\IconTrait;
 
-class HtmlInput extends HtmlSemDoubleElement{
+class HtmlInput extends HtmlSemDoubleElement {
 	use IconTrait;
-	public function __construct($identifier,$type="text",$value="",$placeholder=""){
-		parent::__construct("div-".$identifier,"div","ui input");
-		$this->content=new \Ajax\common\html\html5\HtmlInput($identifier,$type,$value,$placeholder);
-		$this->_states=[State::DISABLED,State::FOCUS,State::ERROR];
+	public function __construct($identifier, $type="text", $value="", $placeholder="") {
+		parent::__construct("div-".$identifier, "div", "ui input");
+		$this->content=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(){
+	public function setFocus() {
 		$this->addToProperty("class", State::FOCUS);
 	}
 
-	public function addLoading(){
-		if($this->_hasIcon===false){
+	public function addLoading() {
+		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 addLabel($label,$direction=Direction::LEFT,$icon=NULL){
+	public function addLabel($label, $direction=Direction::LEFT, $icon=NULL) {
 		$labelO=$label;
-		if(\is_object($label)===false){
-			$labelO=new HtmlLabel("label-".$this->identifier,$label);
-			if(isset($icon))
+		if (\is_object($label)===false) {
+			$labelO=new HtmlLabel("label-".$this->identifier, $label);
+			if (isset($icon))
 				$labelO->addIcon($icon);
-		}else{
+		}else {
 			$labelO->addToPropertyCtrl("class", "label", array("label"));
 		}
 		$this->addToProperty("class", $direction." labeled");
-		$this->addContent($labelO,\strstr($direction,Direction::LEFT)!==false);
+		$this->addContent($labelO, \strstr($direction, Direction::LEFT)!==false);
 		return $labelO;
 	}
 
-	public function addLabelToCorner($label,$direction=Direction::LEFT,$icon=NULL){
-		return $this->addLabel($label,$direction." corner",$icon)->toCorner($direction);
+	public function addLabelToCorner($label, $direction=Direction::LEFT, $icon=NULL) {
+		return $this->addLabel($label, $direction." corner", $icon)->toCorner($direction);
 	}
 
-	public function addAction($action,$direction=Direction::LEFT,$icon=NULL,$labeled=false){
+	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(isset($icon))
-				$actionO->addIcon($icon,true,$labeled);
+		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->addContent($actionO, \strstr($direction, Direction::LEFT)!==false);
 		return $actionO;
 	}
 

Braces +7 added lines, -5 removed lines

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

Ajax/semantic/html/base/constants/Emphasis.php

Spacing +1 added lines, -1 removed lines

@@ -5,5 +5,5 @@
 use Ajax\common\BaseEnum;
 
 abstract class Emphasis extends BaseEnum {
-	const PRIMARY="primary",SECONDARY="secondary",TERTIARY="tertiary";
+	const PRIMARY="primary", SECONDARY="secondary", TERTIARY="tertiary";
 }
\ No newline at end of file