phpMv / phpMv-UI

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

Doc Comments +3 added lines, -3 removed lines

@@ -25,7 +25,7 @@ 
 	/**
 	 * @param string $placeholder
 	 * @param string $icon
-	 * @return \Ajax\semantic\html\content\HtmlDropdownItem|\Ajax\semantic\html\content\HtmlMenuItem
+	 * @return MenuItemTrait
 	 */
 	public function asSearchInput($placeholder=NULL,$icon=NULL){
 		$this->setClass("ui icon search input");
@@ -39,7 +39,7 @@ 
 	}
 
 	/**
-	 * @return \Ajax\semantic\html\content\HtmlDropdownItem|\Ajax\semantic\html\content\HtmlMenuItem
+	 * @return MenuItemTrait
 	 */
 	public function asDivider(){
 		$this->content=NULL;
@@ -51,7 +51,7 @@ 
 	/**
 	 * @param string $caption
 	 * @param string $icon
-	 * @return \Ajax\semantic\html\content\HtmlDropdownItem|\Ajax\semantic\html\content\HtmlMenuItem
+	 * @return MenuItemTrait
 	 */
 	public function asHeader($caption=NULL,$icon=NULL){
 		$this->setClass("header");

Spacing +15 added lines, -15 removed lines

@@ -8,16 +8,16 @@ 
 
 trait MenuItemTrait {
 
-	public function setContent($content){
-		if($content==="-"){
+	public function setContent($content) {
+		if ($content==="-") {
 			$this->asDivider();
-		}elseif($content==="-search-"){
-			$values=\explode(",",$content,-1);
-			$this->asSearchInput(JArray::getDefaultValue($values, 0, "Search..."),JArray::getDefaultValue($values, 1, "search"));
-		}elseif(JString::startswith($content, "-")){
-			$content=\ltrim($content,"-");
+		}elseif ($content==="-search-") {
+			$values=\explode(",", $content, -1);
+			$this->asSearchInput(JArray::getDefaultValue($values, 0, "Search..."), JArray::getDefaultValue($values, 1, "search"));
+		}elseif (JString::startswith($content, "-")) {
+			$content=\ltrim($content, "-");
 			$this->asHeader($content);
-		}else
+		} else
 			parent::setContent($content);
 		return $this;
 	}
@@ -27,13 +27,13 @@ 
 	 * @param string $icon
 	 * @return \Ajax\semantic\html\content\HtmlDropdownItem|\Ajax\semantic\html\content\HtmlMenuItem
 	 */
-	public function asSearchInput($placeholder=NULL,$icon=NULL){
+	public function asSearchInput($placeholder=NULL, $icon=NULL) {
 		$this->setClass("ui icon search input");
 		$input=new HtmlInput("search-".$this->identifier);
-		if(isset($placeholder))
+		if (isset($placeholder))
 			$input->setProperty("placeholder", $placeholder);
 			$this->content=$input;
-			if(isset($icon))
+			if (isset($icon))
 				$this->addIcon($icon);
 				return $this;
 	}
@@ -41,7 +41,7 @@ 
 	/**
 	 * @return \Ajax\semantic\html\content\HtmlDropdownItem|\Ajax\semantic\html\content\HtmlMenuItem
 	 */
-	public function asDivider(){
+	public function asDivider() {
 		$this->content=NULL;
 		$this->tagName="div";
 		$this->setClass("divider");
@@ -53,12 +53,12 @@ 
 	 * @param string $icon
 	 * @return \Ajax\semantic\html\content\HtmlDropdownItem|\Ajax\semantic\html\content\HtmlMenuItem
 	 */
-	public function asHeader($caption=NULL,$icon=NULL){
+	public function asHeader($caption=NULL, $icon=NULL) {
 		$this->setClass("header");
 		$this->tagName="div";
 		$this->content=$caption;
-		if(isset($icon))
-			$this->addIcon($icon,Direction::LEFT);
+		if (isset($icon))
+			$this->addIcon($icon, Direction::LEFT);
 			return $this;
 	}
 }
\ No newline at end of file

Braces +14 added lines, -10 removed lines

@@ -11,14 +11,15 @@ 
 	public function setContent($content){
 		if($content==="-"){
 			$this->asDivider();
-		}elseif($content==="-search-"){
+		} elseif($content==="-search-"){
 			$values=\explode(",",$content,-1);
 			$this->asSearchInput(JArray::getDefaultValue($values, 0, "Search..."),JArray::getDefaultValue($values, 1, "search"));
-		}elseif(JString::startswith($content, "-")){
+		} elseif(JString::startswith($content, "-")){
 			$content=\ltrim($content,"-");
 			$this->asHeader($content);
-		}else
-			parent::setContent($content);
+		} else {
+					parent::setContent($content);
+		}
 		return $this;
 	}
 
@@ -30,11 +31,13 @@ 
 	public function asSearchInput($placeholder=NULL,$icon=NULL){
 		$this->setClass("ui icon search input");
 		$input=new HtmlInput("search-".$this->identifier);
-		if(isset($placeholder))
-			$input->setProperty("placeholder", $placeholder);
+		if(isset($placeholder)) {
+					$input->setProperty("placeholder", $placeholder);
+		}
 			$this->content=$input;
-			if(isset($icon))
-				$this->addIcon($icon);
+			if(isset($icon)) {
+							$this->addIcon($icon);
+			}
 				return $this;
 	}
 
@@ -57,8 +60,9 @@ 
 		$this->setClass("header");
 		$this->tagName="div";
 		$this->content=$caption;
-		if(isset($icon))
-			$this->addIcon($icon,Direction::LEFT);
+		if(isset($icon)) {
+					$this->addIcon($icon,Direction::LEFT);
+		}
 			return $this;
 	}
 }
\ No newline at end of file

Ajax/semantic/html/collections/menus/HtmlMenu.php

Unused Use Statements -1 removed lines

@@ -11,7 +11,6 @@
 use Ajax\semantic\html\modules\HtmlDropdown;
 use Ajax\semantic\html\modules\HtmlPopup;
 use Ajax\semantic\html\elements\HtmlIcon;
-use Ajax\semantic\html\elements\html5\HtmlLink;
 use Ajax\semantic\html\elements\HtmlInput;
 use Ajax\semantic\html\elements\HtmlButton;
 use Ajax\semantic\html\base\traits\AttachedTrait;

Spacing +20 added lines, -20 removed lines

@@ -38,12 +38,12 @@ 
 	 * @return \Ajax\semantic\html\collections\HtmlMenu
 	 */
 	public function setType($type="") {
-		return $this->addToPropertyCtrl("class", $type, array ("","item","text" ));
+		return $this->addToPropertyCtrl("class", $type, array("", "item", "text"));
 	}
 
 	public function setActiveItem($index) {
 		$item=$this->getItem($index);
-		if ($item !== null) {
+		if ($item!==null) {
 			$item->addToProperty("class", "active");
 		}
 		return $this;
@@ -51,7 +51,7 @@ 
 
 	private function getItemToInsert($item) {
 		if ($item instanceof HtmlInput || $item instanceof HtmlImg || $item instanceof HtmlIcon || $item instanceof HtmlButton || ($item instanceof HtmlDropdown && $this->propertyContains("class", "vertical")===false)) {
-			$itemO=new HtmlMenuItem("item-" . $this->identifier, $item);
+			$itemO=new HtmlMenuItem("item-".$this->identifier, $item);
 			$item=$itemO;
 		}
 		return $item;
@@ -59,7 +59,7 @@ 
 
 	private function afterInsert($item) {
 		if (!$item instanceof HtmlMenu && $item->propertyContains("class", "header")===false)
-			$item->addToPropertyCtrl("class", "item", array ("item" ));
+			$item->addToPropertyCtrl("class", "item", array("item"));
 		else {
 			$this->setSecondary();
 		}
@@ -90,15 +90,15 @@ 
 
 	public function generateMenuAsItem($menu, $header=null) {
 		$count=$this->count();
-		$item=new HtmlSemDoubleElement("item-" . $this->identifier . "-" . $count, "div");
+		$item=new HtmlSemDoubleElement("item-".$this->identifier."-".$count, "div");
 		if (isset($header)) {
-			$headerItem=new HtmlSemDoubleElement("item-header-" . $this->identifier . "-" . $count, "div", "header");
+			$headerItem=new HtmlSemDoubleElement("item-header-".$this->identifier."-".$count, "div", "header");
 			$headerItem->setContent($header);
 			$item->addContent($headerItem);
 			$this->_itemHeader=$headerItem;
 		}
-		if(\is_array($menu)){
-			$menu=new HtmlMenu("menu-" . $this->identifier . "-" . $count,$menu);
+		if (\is_array($menu)) {
+			$menu=new HtmlMenu("menu-".$this->identifier."-".$count, $menu);
 		}
 		$menu->setClass("menu");
 		$item->addContent($menu);
@@ -113,7 +113,7 @@ 
 		$value=new HtmlSemDoubleElement($identifier, "a", "browse item", $value);
 		$value->addContent(new HtmlIcon("", "dropdown"));
 		$value=$this->addItem($value);
-		$popup=new HtmlPopup($value, "popup-" . $this->identifier . "-" . $this->count(), $content);
+		$popup=new HtmlPopup($value, "popup-".$this->identifier."-".$this->count(), $content);
 		$popup->setFlowing()->setPosition("bottom left")->setOn("click");
 		$this->wrap("", $popup);
 		return $popup;
@@ -122,7 +122,7 @@ 
 	public function addDropdownAsItem($value, $items=NULL) {
 		$dd=$value;
 		if (\is_string($value)) {
-			$dd=new HtmlDropdown("dropdown-" . $this->identifier . "-" . $this->count(), $value, $items);
+			$dd=new HtmlDropdown("dropdown-".$this->identifier."-".$this->count(), $value, $items);
 		}
 		$this->addItem($dd);
 		return $dd;
@@ -135,7 +135,7 @@ 
 	 * @see \Ajax\common\html\html5\HtmlCollection::createItem()
 	 */
 	protected function createItem($value) {
-		$itemO=new HtmlMenuItem($this->identifier."item" . \sizeof($this->content),"");
+		$itemO=new HtmlMenuItem($this->identifier."item".\sizeof($this->content), "");
 		$itemO->setTagName("a");
 		$itemO->setContent($value);
 		return $itemO;
@@ -146,35 +146,35 @@ 
 	}
 
 	public function setSecondary($value=true) {
-		if($value)
+		if ($value)
 			$this->addToProperty("class", "secondary");
 		else
 			$this->removePropertyValue("class", "secondary");
 	}
 
 	public function setVertical() {
-		return $this->addToPropertyCtrl("class", "vertical", array ("vertical" ));
+		return $this->addToPropertyCtrl("class", "vertical", array("vertical"));
 	}
 
 	public function setPosition($value="right") {
-		return $this->addToPropertyCtrl("class", $value, array ("right","left" ));
+		return $this->addToPropertyCtrl("class", $value, array("right", "left"));
 	}
 
 	public function setPointing($value=Direction::NONE) {
-		return $this->addToPropertyCtrl("class", $value . " pointing", Direction::getConstantValues("pointing"));
+		return $this->addToPropertyCtrl("class", $value." pointing", Direction::getConstantValues("pointing"));
 	}
 
 	public function asTab($vertical=false) {
-		$this->apply(function (HtmlDoubleElement &$item) {
+		$this->apply(function(HtmlDoubleElement&$item) {
 			$item->setTagName("a");
 		});
-		if ($vertical === true)
+		if ($vertical===true)
 			$this->setVertical();
 		return $this->addToProperty("class", "tabular");
 	}
 
 	public function asPagination() {
-		$this->apply(function (HtmlDoubleElement &$item) {
+		$this->apply(function(HtmlDoubleElement&$item) {
 			$item->setTagName("a");
 		});
 		return $this->addToProperty("class", "pagination");
@@ -211,10 +211,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", "item", array ("item" ));
+		return $this->addToPropertyCtrl("class", "item", array("item"));
 	}
 
 	public function addImage($identifier, $src="", $alt="") {

Braces +16 added lines, -13 removed lines

@@ -58,9 +58,9 @@ 
 	}
 
 	private function afterInsert($item) {
-		if (!$item instanceof HtmlMenu && $item->propertyContains("class", "header")===false)
-			$item->addToPropertyCtrl("class", "item", array ("item" ));
-		else {
+		if (!$item instanceof HtmlMenu && $item->propertyContains("class", "header")===false) {
+					$item->addToPropertyCtrl("class", "item", array ("item" ));
+		} else {
 			$this->setSecondary();
 		}
 		return $item;
@@ -146,10 +146,11 @@ 
 	}
 
 	public function setSecondary($value=true) {
-		if($value)
-			$this->addToProperty("class", "secondary");
-		else
-			$this->removePropertyValue("class", "secondary");
+		if($value) {
+					$this->addToProperty("class", "secondary");
+		} else {
+					$this->removePropertyValue("class", "secondary");
+		}
 	}
 
 	public function setVertical() {
@@ -168,8 +169,9 @@ 
 		$this->apply(function (HtmlDoubleElement &$item) {
 			$item->setTagName("a");
 		});
-		if ($vertical === true)
-			$this->setVertical();
+		if ($vertical === true) {
+					$this->setVertical();
+		}
 		return $this->addToProperty("class", "tabular");
 	}
 
@@ -198,10 +200,11 @@ 
 	 */
 	public function fromDatabaseObject($object, $function) {
 		$return=$function($object);
-		if (\is_array($return))
-			$this->addItems($return);
-		else
-			$this->addItem($return);
+		if (\is_array($return)) {
+					$this->addItems($return);
+		} else {
+					$this->addItem($return);
+		}
 	}
 
 	/**

Ajax/semantic/html/content/InternalPopup.php

Doc Comments +7 added lines

@@ -12,6 +12,10 @@ 
 	protected $variation;
 	protected $params;
 	protected $semElement;
+
+	/**
+	 * @param HtmlSemDoubleElement $semElement
+	 */
 	public function __construct($semElement,$title="",$content="",$variation=NULL,$params=array()){
 		$this->semElement=$semElement;
 		$this->title=$title;
@@ -19,6 +23,9 @@ 
 		$this->setAttributes($variation,$params);
 	}
 
+	/**
+	 * @param string $html
+	 */
 	public function setHtml($html) {
 		$this->html= $html;
 		return $this;

Spacing +16 added lines, -16 removed lines

@@ -12,58 +12,58 @@
 	protected $variation;
 	protected $params;
 	protected $semElement;
-	public function __construct($semElement,$title="",$content="",$variation=NULL,$params=array()){
+	public function __construct($semElement, $title="", $content="", $variation=NULL, $params=array()) {
 		$this->semElement=$semElement;
 		$this->title=$title;
 		$this->content=$content;
-		$this->setAttributes($variation,$params);
+		$this->setAttributes($variation, $params);
 	}
 
 	public function setHtml($html) {
-		$this->html= $html;
+		$this->html=$html;
 		return $this;
 	}
 
-	public function setAttributes($variation=NULL,$params=array()){
+	public function setAttributes($variation=NULL, $params=array()) {
 		$this->variation=$variation;
 		$this->params=$params;
 	}
 
-	public function onShow($jsCode){
+	public function onShow($jsCode) {
 		$this->params["onShow"]=$jsCode;
 	}
 
-	public function compile(JsUtils $js=NULL){
-		if(JString::isNotNull($this->title)){
+	public function compile(JsUtils $js=NULL) {
+		if (JString::isNotNull($this->title)) {
 			$this->semElement->addToProperty("data-title", $this->title);
 		}
-		if(JString::isNotNull($this->content)){
+		if (JString::isNotNull($this->content)) {
 			$this->semElement->addToProperty("data-content", $this->content);
 		}
-		if(JString::isNotNull($this->html)){
+		if (JString::isNotNull($this->html)) {
 			$html=$this->html;
-			if(\is_array($html)){
+			if (\is_array($html)) {
 				\array_walk($html, function(&$item) use($js){
-					if($item instanceof HtmlSemDoubleElement){
+					if ($item instanceof HtmlSemDoubleElement) {
 						$comp=$item->compile($js);
 						$bs=$item->run($js);
-						if(isset($bs))
+						if (isset($bs))
 							$this->params['onShow']=$bs->getScript();
 						$item=$comp;
 					}
 				});
-				$html=\implode("",$html);
+				$html=\implode("", $html);
 			}
 			$html=\str_replace("\"", "'", $html);
 			$this->semElement->addToProperty("data-html", $html);
 		}
-		if(JString::isNotNull($this->variation)){
+		if (JString::isNotNull($this->variation)) {
 			$this->semElement->addToProperty("data-variation", $this->variation);
 		}
 	}
 
-	public function run(JsUtils $js){
-		$js->semantic()->popup("#".$this->semElement->getIdentifier(),$this->params);
+	public function run(JsUtils $js) {
+		$js->semantic()->popup("#".$this->semElement->getIdentifier(), $this->params);
 	}
 
 }
\ No newline at end of file

Braces +3 added lines, -2 removed lines

@@ -47,8 +47,9 @@
 					if($item instanceof HtmlSemDoubleElement){
 						$comp=$item->compile($js);
 						$bs=$item->run($js);
-						if(isset($bs))
-							$this->params['onShow']=$bs->getScript();
+						if(isset($bs)) {
+													$this->params['onShow']=$bs->getScript();
+						}
 						$item=$comp;
 					}
 				});

Ajax/semantic/html/modules/HtmlDropdown.php

Doc Comments +10 added lines, -1 removed lines

@@ -22,6 +22,9 @@ 
 	protected $input;
 	protected $value;
 
+	/**
+	 * @param string $identifier
+	 */
 	public function __construct($identifier, $value="", $items=array()) {
 		parent::__construct($identifier, "div");
 		$this->_template=include dirname(__FILE__).'/../templates/tplDropdown.php';
@@ -49,7 +52,7 @@ 
 	/**
 	 * Insert an item at a position
 	 * @param mixed $item
-	 * @param number $position
+	 * @param integer $position
 	 * @return \Ajax\semantic\html\content\HtmlDropdownItem|unknown
 	 */
 	public function insertItem($item,$position=0){
@@ -240,6 +243,9 @@ 
 		return $this->addToPropertyCtrl("class", $value." pointing",Direction::getConstantValues("pointing"));
 	}
 
+	/**
+	 * @param string $value
+	 */
 	public function setValue($value){
 		$this->value=$value;
 		return $this;
@@ -274,6 +280,9 @@ 
 		return $this->addToPropertyCtrl("class", "compact", array("compact"));
 	}
 
+	/**
+	 * @param string $action
+	 */
 	public function setAction($action){
 		$this->_params["action"]=$action;
 	}

Spacing +74 added lines, -74 removed lines

@@ -17,8 +17,8 @@ 
 	}
 	protected $mClass="menu";
 	protected $mTagName="div";
-	protected $items=array ();
-	protected $_params=array("action"=>"nothing","on"=>"hover");
+	protected $items=array();
+	protected $_params=array("action"=>"nothing", "on"=>"hover");
 	protected $input;
 	protected $value;
 
@@ -26,23 +26,23 @@ 
 		parent::__construct($identifier, "div");
 		$this->_template=include dirname(__FILE__).'/../templates/tplDropdown.php';
 		$this->setProperty("class", "ui dropdown");
-		$content=new HtmlSemDoubleElement("text-".$this->identifier,"div");
+		$content=new HtmlSemDoubleElement("text-".$this->identifier, "div");
 		$content->setClass("text");
 		$this->setValue($value);
-		$content->wrap("",new HtmlIcon("", "dropdown"));
+		$content->wrap("", new HtmlIcon("", "dropdown"));
 		$this->content=array($content);
 		$this->tagName="div";
 		$this->addItems($items);
 	}
 
-	public function addItem($item,$value=NULL,$image=NULL,$description=NULL){
-		$itemO=$this->beforeAddItem($item,$value,$image,$description);
+	public function addItem($item, $value=NULL, $image=NULL, $description=NULL) {
+		$itemO=$this->beforeAddItem($item, $value, $image, $description);
 		$this->items[]=$itemO;
 		return $itemO;
 	}
 
-	public function addIcon($icon,$before=true,$labeled=false){
-		$this->addIconP($icon,$before,$labeled);
+	public function addIcon($icon, $before=true, $labeled=false) {
+		$this->addIconP($icon, $before, $labeled);
 		return $this->getElementById("text-".$this->identifier, $this->content)->setWrapAfter("");
 	}
 
@@ -52,26 +52,26 @@ 
 	 * @param number $position
 	 * @return \Ajax\semantic\html\content\HtmlDropdownItem|unknown
 	 */
-	public function insertItem($item,$position=0){
+	public function insertItem($item, $position=0) {
 		$itemO=$this->beforeAddItem($item);
-		 $start = array_slice($this->items, 0, $position);
-		 $end = array_slice($this->items, $position);
-		 $start[] = $item;
+		 $start=array_slice($this->items, 0, $position);
+		 $end=array_slice($this->items, $position);
+		 $start[]=$item;
 		 $this->items=array_merge($start, $end);
 		 return $itemO;
 	}
 
-	protected function beforeAddItem($item,$value=NULL,$image=NULL,$description=NULL){
+	protected function beforeAddItem($item, $value=NULL, $image=NULL, $description=NULL) {
 		$itemO=$item;
-		if(\is_array($item)){
+		if (\is_array($item)) {
 			$description=JArray::getValue($item, "description", 3);
 			$value=JArray::getValue($item, "value", 1);
 			$image=JArray::getValue($item, "image", 2);
 			$item=JArray::getValue($item, "item", 0);
 		}
-		if(!$item instanceof HtmlDropdownItem){
-			$itemO=new HtmlDropdownItem("dd-item-".$this->identifier."-".\sizeof($this->items),$item,$value,$image,$description);
-		}elseif($itemO instanceof HtmlDropdownItem){
+		if (!$item instanceof HtmlDropdownItem) {
+			$itemO=new HtmlDropdownItem("dd-item-".$this->identifier."-".\sizeof($this->items), $item, $value, $image, $description);
+		}elseif ($itemO instanceof HtmlDropdownItem) {
 			$this->addToProperty("class", "vertical");
 		}
 		return $itemO;
@@ -84,11 +84,11 @@ 
 		$this->addItem($function($object));
 	}
 
-	public function addInput($name){
-		if(!isset($name))
+	public function addInput($name) {
+		if (!isset($name))
 			$name="input-".$this->identifier;
 		$this->setAction("activate");
-		$this->input=new HtmlInput($name,"hidden");
+		$this->input=new HtmlInput($name, "hidden");
 	}
 
 	/**
@@ -97,15 +97,15 @@ 
 	 * @param string $icon
 	 * @return \Ajax\semantic\html\content\HtmlDropdownItem
 	 */
-	public function addSearchInputItem($placeHolder=NULL,$icon=NULL){
-		return $this->addItem(HtmlDropdownItem::searchInput($placeHolder,$icon));
+	public function addSearchInputItem($placeHolder=NULL, $icon=NULL) {
+		return $this->addItem(HtmlDropdownItem::searchInput($placeHolder, $icon));
 	}
 
 	/**
 	 * Adds a divider item
 	 * @return \Ajax\semantic\html\content\HtmlDropdownItem
 	 */
-	public function addDividerItem(){
+	public function addDividerItem() {
 		return $this->addItem(HtmlDropdownItem::divider());
 	}
 
@@ -115,8 +115,8 @@ 
 	 * @param string $icon
 	 * @return \Ajax\semantic\html\content\HtmlDropdownItem|unknown
 	 */
-	public function addHeaderItem($caption=NULL,$icon=NULL){
-		return $this->addItem(HtmlDropdownItem::header($caption,$icon));
+	public function addHeaderItem($caption=NULL, $icon=NULL) {
+		return $this->addItem(HtmlDropdownItem::header($caption, $icon));
 	}
 
 	/**
@@ -125,7 +125,7 @@ 
 	 * @param string $color
 	 * @return \Ajax\semantic\html\content\HtmlDropdownItem|unknown
 	 */
-	public function addCircularLabelItem($caption,$color){
+	public function addCircularLabelItem($caption, $color) {
 		return $this->addItem(HtmlDropdownItem::circular($caption, $color));
 	}
 
@@ -134,87 +134,87 @@ 
 	 * @param string $image
 	 * @return \Ajax\semantic\html\content\HtmlDropdownItem
 	 */
-	public function addMiniAvatarImageItem($caption,$image){
+	public function addMiniAvatarImageItem($caption, $image) {
 		return $this->addItem(HtmlDropdownItem::avatar($caption, $image));
 	}
 
-	public function addItems($items){
-		if(JArray::isAssociative($items)){
-			foreach ($items as $k=>$v){
+	public function addItems($items) {
+		if (JArray::isAssociative($items)) {
+			foreach ($items as $k=>$v) {
 				$this->addItem($v)->setData($k);
 			}
-		}else{
-			foreach ($items as $item){
+		} else {
+			foreach ($items as $item) {
 				$this->addItem($item);
 			}
 		}
 	}
 
-	public function getItem($index){
+	public function getItem($index) {
 		return $this->items[$index];
 	}
 
 	/**
 	 * @return int
 	 */
-	public function count(){
+	public function count() {
 		return \sizeof($this->items);
 	}
 	/**
 	 * @param boolean $dropdown
 	 */
-	public function asDropdown($dropdown){
-		if($dropdown===false){
+	public function asDropdown($dropdown) {
+		if ($dropdown===false) {
 			$this->_template=include dirname(__FILE__).'/../templates/tplDropdownMenu.php';
 			$dropdown="menu";
-		}else{
+		} else {
 			$dropdown="dropdown";
 			$this->mClass="menu";
 		}
-		return $this->addToPropertyCtrl("class", $dropdown,array("menu","dropdown"));
+		return $this->addToPropertyCtrl("class", $dropdown, array("menu", "dropdown"));
 	}
 
-	public function setVertical(){
-		return $this->addToPropertyCtrl("class", "vertical",array("vertical"));
+	public function setVertical() {
+		return $this->addToPropertyCtrl("class", "vertical", array("vertical"));
 	}
 
-	public function setInline(){
-		return $this->addToPropertyCtrl("class", "inline",["inline"]);
+	public function setInline() {
+		return $this->addToPropertyCtrl("class", "inline", ["inline"]);
 	}
 
-	public function setSimple(){
-		return $this->addToPropertyCtrl("class", "simple",array("simple"));
+	public function setSimple() {
+		return $this->addToPropertyCtrl("class", "simple", array("simple"));
 	}
 
-	public function asButton($floating=false){
-		if($floating)
+	public function asButton($floating=false) {
+		if ($floating)
 			$this->addToProperty("class", "floating");
 		$this->removePropertyValue("class", "selection");
 		return $this->addToProperty("class", "button");
 	}
 
-	public function asSelect($name=NULL,$multiple=false,$selection=true){
-		if(isset($name))
+	public function asSelect($name=NULL, $multiple=false, $selection=true) {
+		if (isset($name))
 			$this->addInput($name);
-		if($multiple)
+		if ($multiple)
 			$this->addToProperty("class", "multiple");
-		if ($selection){
-			if($this->propertyContains("class", "button")===false)
-				$this->addToPropertyCtrl("class", "selection",array("selection"));
+		if ($selection) {
+			if ($this->propertyContains("class", "button")===false)
+				$this->addToPropertyCtrl("class", "selection", array("selection"));
 		}
 		return $this;
 	}
 
-	public function asSearch($name=NULL,$multiple=false,$selection=true){
-		$this->asSelect($name,$multiple,$selection);
+	public function asSearch($name=NULL, $multiple=false, $selection=true) {
+		$this->asSelect($name, $multiple, $selection);
 		return $this->addToProperty("class", "search");
 	}
 
-	public function setSelect($name=NULL,$multiple=false){
-		if(!isset($name))
+	public function setSelect($name=NULL, $multiple=false) {
+		if (!isset($name))
 			$name="select-".$this->identifier;
 		$this->input=null;
-		if($multiple){
+		if ($multiple) {
 			$this->setProperty("multiple", true);
 			$this->addToProperty("class", "multiple");
 		}
@@ -222,37 +222,37 @@ 
 		$this->tagName="select";
 		$this->setProperty("name", $name);
 		$this->content=null;
-		foreach ($this->items as $item){
+		foreach ($this->items as $item) {
 			$item->asOption();
 		}
 		return $this;
 	}
 
-	public function asSubmenu($pointing=NULL){
+	public function asSubmenu($pointing=NULL) {
 		$this->setClass("ui dropdown link item");
-		if(isset($pointing)){
+		if (isset($pointing)) {
 			$this->setPointing($pointing);
 		}
 		return $this;
 	}
 
-	public function setPointing($value=Direction::NONE){
-		return $this->addToPropertyCtrl("class", $value." pointing",Direction::getConstantValues("pointing"));
+	public function setPointing($value=Direction::NONE) {
+		return $this->addToPropertyCtrl("class", $value." pointing", Direction::getConstantValues("pointing"));
 	}
 
-	public function setValue($value){
+	public function setValue($value) {
 		$this->value=$value;
 		return $this;
 	}
-	private function applyValue(){
+	private function applyValue() {
 		$value=$this->value;
-		if(isset($this->input) && isset($value)){
+		if (isset($this->input) && isset($value)) {
 			$this->input->setProperty("value", $value);
-		}else{
+		} else {
 			$this->setProperty("value", $value);
 		}
 			$textElement=$this->getElementById("text-".$this->identifier, $this->content);
-			if(isset($textElement))
+			if (isset($textElement))
 				$textElement->setContent($value);
 		return $this;
 	}
@@ -262,28 +262,28 @@ 
 	 * @see BaseHtml::run()
 	 */
 	public function run(JsUtils $js) {
-		if($this->propertyContains("class", "simple")===false){
-			if(isset($this->_bsComponent)===false)
-				$this->_bsComponent=$js->semantic()->dropdown("#".$this->identifier,$this->_params);
+		if ($this->propertyContains("class", "simple")===false) {
+			if (isset($this->_bsComponent)===false)
+				$this->_bsComponent=$js->semantic()->dropdown("#".$this->identifier, $this->_params);
 			$this->addEventsOnRun($js);
 			return $this->_bsComponent;
 		}
 	}
 
-	public function setCompact(){
+	public function setCompact() {
 		return $this->addToPropertyCtrl("class", "compact", array("compact"));
 	}
 
-	public function setAction($action){
+	public function setAction($action) {
 		$this->_params["action"]=$action;
 	}
 
-	public function setFullTextSearch($value){
+	public function setFullTextSearch($value) {
 		$this->_params["fullTextSearch"]=$value;
 	}
 
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
 		$this->applyValue();
-		return parent::compile($js,$view);
+		return parent::compile($js, $view);
 	}
 }
\ No newline at end of file

Braces +28 added lines, -20 removed lines

@@ -71,7 +71,7 @@ 
 		}
 		if(!$item instanceof HtmlDropdownItem){
 			$itemO=new HtmlDropdownItem("dd-item-".$this->identifier."-".\sizeof($this->items),$item,$value,$image,$description);
-		}elseif($itemO instanceof HtmlDropdownItem){
+		} elseif($itemO instanceof HtmlDropdownItem){
 			$this->addToProperty("class", "vertical");
 		}
 		return $itemO;
@@ -85,8 +85,9 @@ 
 	}
 
 	public function addInput($name){
-		if(!isset($name))
-			$name="input-".$this->identifier;
+		if(!isset($name)) {
+					$name="input-".$this->identifier;
+		}
 		$this->setAction("activate");
 		$this->input=new HtmlInput($name,"hidden");
 	}
@@ -143,7 +144,7 @@ 
 			foreach ($items as $k=>$v){
 				$this->addItem($v)->setData($k);
 			}
-		}else{
+		} else{
 			foreach ($items as $item){
 				$this->addItem($item);
 			}
@@ -167,7 +168,7 @@ 
 		if($dropdown===false){
 			$this->_template=include dirname(__FILE__).'/../templates/tplDropdownMenu.php';
 			$dropdown="menu";
-		}else{
+		} else{
 			$dropdown="dropdown";
 			$this->mClass="menu";
 		}
@@ -187,20 +188,24 @@ 
 	}
 
 	public function asButton($floating=false){
-		if($floating)
-			$this->addToProperty("class", "floating");
+		if($floating) {
+					$this->addToProperty("class", "floating");
+		}
 		$this->removePropertyValue("class", "selection");
 		return $this->addToProperty("class", "button");
 	}
 
 	public function asSelect($name=NULL,$multiple=false,$selection=true){
-		if(isset($name))
-			$this->addInput($name);
-		if($multiple)
-			$this->addToProperty("class", "multiple");
+		if(isset($name)) {
+					$this->addInput($name);
+		}
+		if($multiple) {
+					$this->addToProperty("class", "multiple");
+		}
 		if ($selection){
-			if($this->propertyContains("class", "button")===false)
-				$this->addToPropertyCtrl("class", "selection",array("selection"));
+			if($this->propertyContains("class", "button")===false) {
+							$this->addToPropertyCtrl("class", "selection",array("selection"));
+			}
 		}
 		return $this;
 	}
@@ -211,8 +216,9 @@ 
 	}
 
 	public function setSelect($name=NULL,$multiple=false){
-		if(!isset($name))
-			$name="select-".$this->identifier;
+		if(!isset($name)) {
+					$name="select-".$this->identifier;
+		}
 		$this->input=null;
 		if($multiple){
 			$this->setProperty("multiple", true);
@@ -248,12 +254,13 @@ 
 		$value=$this->value;
 		if(isset($this->input) && isset($value)){
 			$this->input->setProperty("value", $value);
-		}else{
+		} else{
 			$this->setProperty("value", $value);
 		}
 			$textElement=$this->getElementById("text-".$this->identifier, $this->content);
-			if(isset($textElement))
-				$textElement->setContent($value);
+			if(isset($textElement)) {
+							$textElement->setContent($value);
+			}
 		return $this;
 	}
 
@@ -263,8 +270,9 @@ 
 	 */
 	public function run(JsUtils $js) {
 		if($this->propertyContains("class", "simple")===false){
-			if(isset($this->_bsComponent)===false)
-				$this->_bsComponent=$js->semantic()->dropdown("#".$this->identifier,$this->_params);
+			if(isset($this->_bsComponent)===false) {
+							$this->_bsComponent=$js->semantic()->dropdown("#".$this->identifier,$this->_params);
+			}
 			$this->addEventsOnRun($js);
 			return $this->_bsComponent;
 		}

Ajax/semantic/html/views/HtmlCard.php

Doc Comments +10 added lines

@@ -27,6 +27,9 @@ 
 		return $result;
 	}
 
+	/**
+	 * @param string $key
+	 */
 	private function addElementInContent($key, $element) {
 		if (\array_key_exists($key, $this->content) === false) {
 			$this->content[$key]=array ();
@@ -35,6 +38,9 @@ 
 		return $element;
 	}
 
+	/**
+	 * @param string $part
+	 */
 	private function getPart($part, $index=NULL) {
 		if (\array_key_exists($part, $this->content)) {
 			if (isset($index))
@@ -59,6 +65,10 @@ 
 		return $this->addElementInContent("content", $image);
 	}
 
+	/**
+	 * @param HtmlImage $visibleContent
+	 * @param HtmlImage $hiddenContent
+	 */
 	public function addReveal($visibleContent, $hiddenContent=NULL, $type=RevealType::FADE, $attributeType=NULL) {
 		$reveal=$visibleContent;
 		if (!$visibleContent instanceof HtmlReveal) {

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use Ajax\semantic\html\base\HtmlSemDoubleElement;
 use Ajax\JsUtils;
 use Ajax\service\JArray;
-
 use Ajax\semantic\html\base\constants\State;
 
 class HtmlProgress extends HtmlSemDoubleElement {

Spacing +13 added lines, -13 removed lines

@@ -17,19 +17,19 @@ 
 class HtmlCard extends HtmlSemDoubleElement {
 
 	public function __construct($identifier) {
-		parent::__construct($identifier, "div", "ui card", array ());
+		parent::__construct($identifier, "div", "ui card", array());
 	}
 
 	private function createContent($content, $baseClass="content") {
 		$count=\sizeof($this->content);
-		$result=new HtmlCardContent("content-" . $count . "-" . $this->identifier, $content);
+		$result=new HtmlCardContent("content-".$count."-".$this->identifier, $content);
 		$result->setClass($baseClass);
 		return $result;
 	}
 
 	private function addElementInContent($key, $element) {
-		if (\array_key_exists($key, $this->content) === false) {
-			$this->content[$key]=array ();
+		if (\array_key_exists($key, $this->content)===false) {
+			$this->content[$key]=array();
 		}
 		$this->content[$key][]=$element;
 		return $element;
@@ -46,14 +46,14 @@ 
 
 	public function addHeader($header, $niveau=4, $type="page") {
 		if (!$header instanceof HtmlHeader) {
-			$header=new HtmlHeader("header-" . $this->identifier, $niveau, $header, $type);
+			$header=new HtmlHeader("header-".$this->identifier, $niveau, $header, $type);
 		}
 		$this->content["header"]=$this->createContent($header);
 	}
 
 	public function addImage($image, $title="") {
 		if (!$image instanceof HtmlImage) {
-			$image=new HtmlImage("image-" . $this->identifier, $image, $title);
+			$image=new HtmlImage("image-".$this->identifier, $image, $title);
 		}
 		$image->setClass("image");
 		return $this->addElementInContent("content", $image);
@@ -62,7 +62,7 @@ 
 	public function addReveal($visibleContent, $hiddenContent=NULL, $type=RevealType::FADE, $attributeType=NULL) {
 		$reveal=$visibleContent;
 		if (!$visibleContent instanceof HtmlReveal) {
-			$reveal=new HtmlReveal("reveral-" . $this->identifier, $visibleContent, $hiddenContent, $type, $attributeType);
+			$reveal=new HtmlReveal("reveral-".$this->identifier, $visibleContent, $hiddenContent, $type, $attributeType);
 		}
 		return $this->addElementInContent("content", $reveal);
 	}
@@ -91,8 +91,8 @@ 
 	 * @param string $asIcons
 	 * @return \Ajax\semantic\html\elements\HtmlButtonGroups
 	 */
-	public function addButtons($elements=array(), $asIcons=false){
-		$buttons=new HtmlButtonGroups("buttons-".$this->identifier,$elements,$asIcons);
+	public function addButtons($elements=array(), $asIcons=false) {
+		$buttons=new HtmlButtonGroups("buttons-".$this->identifier, $elements, $asIcons);
 		$this->addElementInContent("content", $buttons);
 		return $buttons;
 	}
@@ -101,12 +101,12 @@ 
 
 	public function addCardHeaderContent($header, $metas=array(), $description=NULL) {
 		$count=\sizeof($this->content);
-		return $this->addElementInContent("content", new HtmlCardHeaderContent("content-" . $count . "-" . $this->identifier, $header, $metas, $description));
+		return $this->addElementInContent("content", new HtmlCardHeaderContent("content-".$count."-".$this->identifier, $header, $metas, $description));
 	}
 
 	public function addCardContent($content=array()) {
 		$count=\sizeof($this->content);
-		return $this->addElementInContent("content", new HtmlCardContent("content-" . $count . "-" . $this->identifier, $content));
+		return $this->addElementInContent("content", new HtmlCardContent("content-".$count."-".$this->identifier, $content));
 	}
 
 	public function getCardContent($index=NULL) {
@@ -132,13 +132,13 @@ 
 	 * @see \Ajax\semantic\html\base\HtmlSemDoubleElement::compile()
 	 */
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
-		$this->content=JArray::sortAssociative($this->content, [ "header","image","content","extra-content" ]);
+		$this->content=JArray::sortAssociative($this->content, ["header", "image", "content", "extra-content"]);
 		return parent::compile($js, $view);
 	}
 
 	public function asLink($href="") {
 		$this->addToProperty("class", "link");
-		if ($href !== "") {
+		if ($href!=="") {
 			$this->setProperty("href", $href);
 		}
 		return $this;

Ajax/semantic/components/Popup.php

Spacing +5 added lines, -5 removed lines

@@ -27,11 +27,11 @@ 
 	 * @return $this
 	 */
 	public function setOnShow($jsCode) {
-		$jsCode=str_ireplace("\"","%quote%", $jsCode);
+		$jsCode=str_ireplace("\"", "%quote%", $jsCode);
 		return $this->setParam("onShow", "%function(){".$jsCode."}%");
 	}
 
-	public function setExclusive($value){
+	public function setExclusive($value) {
 		return $this->setParam("exclusive", $value);
 	}
 
@@ -40,15 +40,15 @@ 
 	 * @param string $popup the css selector of the popup
 	 * @return \Ajax\semantic\components\Popup
 	 */
-	public function setPopup($popup){
+	public function setPopup($popup) {
 		return $this->setParam("popup", $popup);
 	}
 
-	public function setInline($value){
+	public function setInline($value) {
 		return $this->setParam("inline", $value);
 	}
 
-	public function setPosition($value){
+	public function setPosition($value) {
 		return $this->setParam("position", $value);
 	}
 	//TODO other events implementation

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

Spacing +10 added lines, -10 removed lines

@@ -15,18 +15,18 @@ 
 	private $_hasIcon=false;
 
 	protected abstract function addToPropertyCtrl($name, $value, $typeCtrl);
-	public abstract function addContent($content,$before=false);
+	public abstract function addContent($content, $before=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,$direction===Direction::LEFT);
+			$this->addContent($iconO, $direction===Direction::LEFT);
 			$this->_hasIcon=true;
-		}else{
+		} else {
 			$iconO=$this->getIcon();
 			$iconO->setIcon($icon);
 			$this->addToPropertyCtrl("class", $direction." icon", Direction::getConstantValues("icon"));
@@ -34,10 +34,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;
 			}
 		}

Ajax/semantic/html/base/HtmlSemDoubleElement.php

Braces +15 added lines, -10 removed lines

@@ -32,8 +32,9 @@ 
 	}
 
 	public function setPopupAttributes($variation=NULL, $popupEvent=NULL) {
-		if (isset($this->_popup))
-			$this->_popup->setAttributes($variation, $popupEvent);
+		if (isset($this->_popup)) {
+					$this->_popup->setAttributes($variation, $popupEvent);
+		}
 	}
 
 	public function addPopup($title="", $content="", $variation=NULL, $params=array()) {
@@ -60,8 +61,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" ));
 		}
@@ -80,21 +82,24 @@ 
 	 * @return \Ajax\semantic\html\base\HtmlSemDoubleElement
 	 */
 	public function asLink($href=NULL) {
-		if (isset($href))
-			$this->setProperty("href", $href);
+		if (isset($href)) {
+					$this->setProperty("href", $href);
+		}
 		return $this->setTagName("a");
 	}
 
 	public function jsShowDimmer($show=true) {
 		$status="hide";
-		if ($show === true)
-			$status="show";
+		if ($show === true) {
+					$status="show";
+		}
 		return '$("#.' . $this->identifier . ').dimmer("' . $status . '");';
 	}
 
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
-	if (isset($this->_popup))
-			$this->_popup->compile($js);
+	if (isset($this->_popup)) {
+				$this->_popup->compile($js);
+	}
 		return parent::compile($js, $view);
 	}
 

Ajax/semantic/html/content/HtmlDropdownItem.php

Spacing +31 added lines, -31 removed lines

@@ -12,34 +12,34 @@ 
 use Ajax\semantic\html\base\traits\MenuItemTrait;
 
 class HtmlDropdownItem extends HtmlSemDoubleElement {
-	use IconTrait,MenuItemTrait;
-	public function __construct($identifier, $content="",$value=NULL,$image=NULL,$description=NULL) {
+	use IconTrait, MenuItemTrait;
+	public function __construct($identifier, $content="", $value=NULL, $image=NULL, $description=NULL) {
 		parent::__construct($identifier, "a");
 		$this->setClass("item");
 		$this->setContent($content);
-		if($value!==NULL)
+		if ($value!==NULL)
 			$this->setData($value);
-		if($image!==NULL)
+		if ($image!==NULL)
 			$this->asMiniAvatar($image);
-		if($description!==NULL)
+		if ($description!==NULL)
 			$this->setDescription($description);
 	}
 
-	public function setDescription($description){
-		$descO=new HtmlDoubleElement("desc-".$this->identifier,"span");
+	public function setDescription($description) {
+		$descO=new HtmlDoubleElement("desc-".$this->identifier, "span");
 		$descO->setClass("description");
 		$descO->setContent($description);
-		return $this->addContent($descO,true);
+		return $this->addContent($descO, true);
 	}
 
-	public function setData($value){
+	public function setData($value) {
 		$this->setProperty("data-value", $value);
 		return $this;
 	}
 
-	public function asOption(){
+	public function asOption() {
 		$this->tagName="option";
-		if($this->getProperty("data-value")!==null)
+		if ($this->getProperty("data-value")!==null)
 			$this->setProperty("value", $this->getProperty("data-value"));
 	}
 
@@ -47,11 +47,11 @@ 
 	 * @param string $image the image src
 	 * @return \Ajax\common\html\html5\HtmlImg
 	 */
-	public function asMiniAvatar($image){
+	public function asMiniAvatar($image) {
 		$this->tagName="div";
-		$img=new HtmlImg("image-".$this->identifier,$image);
+		$img=new HtmlImg("image-".$this->identifier, $image);
 		$img->setClass("ui mini avatar image");
-		$this->addContent($img,true);
+		$this->addContent($img, true);
 		return $this;
 	}
 
@@ -60,9 +60,9 @@ 
 	 * @param string $icon
 	 * @return \Ajax\semantic\html\content\HtmlDropdownItem
 	 */
-	public function asIcon($caption,$icon){
+	public function asIcon($caption, $icon) {
 		$this->setContent($caption);
-		$this->addContent(new HtmlIcon("", $icon),true);
+		$this->addContent(new HtmlIcon("", $icon), true);
 		return $this;
 	}
 
@@ -71,23 +71,23 @@ 
 	 * @param string $color
 	 * @return \Ajax\semantic\html\content\HtmlDropdownItem
 	 */
-	public function asCircularLabel($caption,$color){
+	public function asCircularLabel($caption, $color) {
 		$this->setContent($caption);
 		$lbl=new HtmlLabel("");
 		$lbl->setCircular()->setColor($color)->setEmpty();
-		$this->addContent($lbl,true);
+		$this->addContent($lbl, true);
 		return $this;
 	}
 
 
 
 	public function addMenuItem($items) {
-		$menu=new HtmlMenu("menu-" . $this->identifier, $items);
+		$menu=new HtmlMenu("menu-".$this->identifier, $items);
 		$content=$this->content;
 		$this->setTagName("div");
 		$this->setProperty("class", "item");
 		$icon=new HtmlIcon("", "dropdown");
-		$this->content=[$icon,new HtmlSemDoubleElement("","span","text",$content),$menu];
+		$this->content=[$icon, new HtmlSemDoubleElement("", "span", "text", $content), $menu];
 		return $menu;
 	}
 
@@ -96,14 +96,14 @@ 
 	 * @param string $icon
 	 * @return \Ajax\semantic\html\content\HtmlDropdownItem
 	 */
-	public static function searchInput($placeholder=NULL,$icon=NULL){
-		return (new HtmlDropdownItem(""))->asSearchInput($placeholder,$icon);
+	public static function searchInput($placeholder=NULL, $icon=NULL) {
+		return (new HtmlDropdownItem(""))->asSearchInput($placeholder, $icon);
 	}
 
 	/**
 	 * @return \Ajax\semantic\html\content\HtmlDropdownItem
 	 */
-	public static function divider(){
+	public static function divider() {
 		return (new HtmlDropdownItem(""))->asDivider();
 	}
 
@@ -112,8 +112,8 @@ 
 	 * @param string $icon
 	 * @return \Ajax\semantic\html\content\HtmlDropdownItem
 	 */
-	public static function header($caption=NULL,$icon=NULL){
-		return (new HtmlDropdownItem(""))->asHeader($caption,$icon);
+	public static function header($caption=NULL, $icon=NULL) {
+		return (new HtmlDropdownItem(""))->asHeader($caption, $icon);
 	}
 
 	/**
@@ -121,8 +121,8 @@ 
 	 * @param string $color
 	 * @return \Ajax\semantic\html\content\HtmlDropdownItem
 	 */
-	public static function circular($caption,$color){
-		return (new HtmlDropdownItem(""))->asCircularLabel($caption,$color);
+	public static function circular($caption, $color) {
+		return (new HtmlDropdownItem(""))->asCircularLabel($caption, $color);
 	}
 
 	/**
@@ -130,8 +130,8 @@ 
 	 * @param string $icon
 	 * @return \Ajax\semantic\html\content\HtmlDropdownItem
 	 */
-	public static function icon($caption,$icon){
-		return (new HtmlDropdownItem(""))->asIcon($caption,$icon);
+	public static function icon($caption, $icon) {
+		return (new HtmlDropdownItem(""))->asIcon($caption, $icon);
 	}
 
 	/**
@@ -139,7 +139,7 @@ 
 	 * @param unknown $image
 	 * @return \Ajax\semantic\html\content\HtmlDropdownItem
 	 */
-	public static function avatar($caption,$image){
-		return (new HtmlDropdownItem("",$caption))->asMiniAvatar($image);
+	public static function avatar($caption, $image) {
+		return (new HtmlDropdownItem("", $caption))->asMiniAvatar($image);
 	}
 }
\ No newline at end of file

Braces +12 added lines, -8 removed lines

@@ -17,12 +17,15 @@ 
 		parent::__construct($identifier, "a");
 		$this->setClass("item");
 		$this->setContent($content);
-		if($value!==NULL)
-			$this->setData($value);
-		if($image!==NULL)
-			$this->asMiniAvatar($image);
-		if($description!==NULL)
-			$this->setDescription($description);
+		if($value!==NULL) {
+					$this->setData($value);
+		}
+		if($image!==NULL) {
+					$this->asMiniAvatar($image);
+		}
+		if($description!==NULL) {
+					$this->setDescription($description);
+		}
 	}
 
 	public function setDescription($description){
@@ -39,8 +42,9 @@ 
 
 	public function asOption(){
 		$this->tagName="option";
-		if($this->getProperty("data-value")!==null)
-			$this->setProperty("value", $this->getProperty("data-value"));
+		if($this->getProperty("data-value")!==null) {
+					$this->setProperty("value", $this->getProperty("data-value"));
+		}
 	}
 
 	/**