phpMv / phpMv-UI

Ajax/php/micro/JsUtils.php

Spacing +12 added lines, -12 removed lines

@@ -4,34 +4,34 @@
 
 use micro\controllers\Startup;
 use micro\utils\RequestUtils;
-class JsUtils extends \Ajax\JsUtils{
+class JsUtils extends \Ajax\JsUtils {
 
-	public function getUrl($url){
+	public function getUrl($url) {
 		return RequestUtils::getUrl($url);
 	}
 
-	public function addViewElement($identifier,$content,&$view){
+	public function addViewElement($identifier, $content, &$view) {
 		$controls=$view->getVar("q");
-		if (isset($controls) === false) {
-			$controls=array ();
+		if (isset($controls)===false) {
+			$controls=array();
 		}
 		$controls[$identifier]=$content;
 		$view->setVar("q", $controls);
 	}
 
-	public function createScriptVariable(&$view,$view_var, $output){
-		$view->setVar($view_var,$output);
+	public function createScriptVariable(&$view, $view_var, $output) {
+		$view->setVar($view_var, $output);
 	}
 
-	public function forward($initialController,$controller,$action,$params=array()){
-		return $initialController->forward($controller,$action,$params,true,true,true);
+	public function forward($initialController, $controller, $action, $params=array()) {
+		return $initialController->forward($controller, $action, $params, true, true, true);
 	}
 
-	public function renderContent($initialControllerInstance,$viewName, $params=NULL) {
-		return $initialControllerInstance->loadView($viewName,$params,true);
+	public function renderContent($initialControllerInstance, $viewName, $params=NULL) {
+		return $initialControllerInstance->loadView($viewName, $params, true);
 	}
 
-	public function fromDispatcher($dispatcher){
+	public function fromDispatcher($dispatcher) {
 		return Startup::$urlParts;
 	}
 }
\ No newline at end of file

Ajax/semantic/html/content/HtmlStepItem.php

Spacing +15 added lines, -15 removed lines

@@ -8,52 +8,52 @@
 class HtmlStepItem extends HtmlAbsractItem {
 
 	public function __construct($identifier, $content) {
-		parent::__construct($identifier,"step",$content);
+		parent::__construct($identifier, "step", $content);
 	}
-	protected function initContent($content){
-		if(\is_array($content)){
-			if(JArray::isAssociative($content)===false){
+	protected function initContent($content) {
+		if (\is_array($content)) {
+			if (JArray::isAssociative($content)===false) {
 				$icon=@$content[0];
 				$title=@$content[1];
 				$desc=@$content[2];
 				$status=@$content[3];
-			}else{
+			} else {
 				$icon=@$content["icon"];
 				$title=@$content["title"];
 				$desc=@$content["description"];
 				$status=@$content["status"];
 			}
-			if(isset($icon)===true){
+			if (isset($icon)===true) {
 				$this->setIcon($icon);
 			}
-			if(isset($status)===true){
+			if (isset($status)===true) {
 				$this->setStatus($status);
 			}
-			if(isset($title)===true){
-				$this->setTitle($title,$desc);
+			if (isset($title)===true) {
+				$this->setTitle($title, $desc);
 			}
-		}else{
+		} else {
 			$this->setContent($content);
 		}
 	}
 
-	public function setActive($value=true){
-		if($value)
+	public function setActive($value=true) {
+		if ($value)
 			$this->setStatus(StepStatus::ACTIVE);
 		else
 			$this->setStatus(StepStatus::NONE);
 		return $this;
 	}
 
-	public function setCompleted(){
+	public function setCompleted() {
 		return $this->setStatus(StepStatus::COMPLETED);
 	}
 
-	public function setStatus($status){
+	public function setStatus($status) {
 		return $this->addToPropertyCtrl("class", $status, StepStatus::getConstants());
 	}
 
-	public function removeStatus(){
+	public function removeStatus() {
 		$this->removePropertyValues("class", StepStatus::getConstants());
 	}
 }
\ No newline at end of file

Braces +7 added lines, -6 removed lines

@@ -17,7 +17,7 @@ 
 				$title=@$content[1];
 				$desc=@$content[2];
 				$status=@$content[3];
-			}else{
+			} else{
 				$icon=@$content["icon"];
 				$title=@$content["title"];
 				$desc=@$content["description"];
@@ -32,16 +32,17 @@ 
 			if(isset($title)===true){
 				$this->setTitle($title,$desc);
 			}
-		}else{
+		} else{
 			$this->setContent($content);
 		}
 	}
 
 	public function setActive($value=true){
-		if($value)
-			$this->setStatus(StepStatus::ACTIVE);
-		else
-			$this->setStatus(StepStatus::NONE);
+		if($value) {
+					$this->setStatus(StepStatus::ACTIVE);
+		} else {
+					$this->setStatus(StepStatus::NONE);
+		}
 		return $this;
 	}
 

Ajax/semantic/html/modules/HtmlTab.php

Spacing +44 added lines, -44 removed lines

@@ -14,14 +14,14 @@ 
  * @author jc
  * @version 1.001
  */
-class HtmlTab extends HtmlSemCollection{
+class HtmlTab extends HtmlSemCollection {
 
 	protected $params=array("debug"=>true);
 
-	public function __construct( $identifier, $tabs=array()){
-		parent::__construct( $identifier, "div", "");
+	public function __construct($identifier, $tabs=array()) {
+		parent::__construct($identifier, "div", "");
 		$menu=new HtmlMenu("menu".$this->identifier);
-		$menu->asTab(false)->setAttachment(NULL,Side::TOP);
+		$menu->asTab(false)->setAttachment(NULL, Side::TOP);
 		$this->content["menu"]=$menu;
 		$this->addItems($tabs);
 	}
@@ -31,12 +31,12 @@ 
 	 * @see \Ajax\common\html\HtmlCollection::createItem()
 	 * @return HtmlSegment
 	 */
-	protected function createItem($value){
+	protected function createItem($value) {
 		$count=$this->count();
 		$title=$value;
 		$content=NULL;
-		if(\is_array($value)){
-			$title=@$value[0];$content=@$value[1];
+		if (\is_array($value)) {
+			$title=@$value[0]; $content=@$value[1];
 		}
 		$menuItem=$this->content["menu"]->addItem($title);
 		$menuItem->addToProperty("data-tab", $menuItem->getIdentifier());
@@ -50,9 +50,9 @@ 
 	 * @param string $datatab
 	 * @return \Ajax\semantic\html\elements\HtmlSegment
 	 */
-	private function createSegment($count,$content,$datatab){
+	private function createSegment($count, $content, $datatab) {
 		$segment=new HtmlSegment("item-".$this->identifier."-".$count, $content);
-		$segment->setAttachment(NULL,Side::BOTTOM)->addToProperty("class", "tab")->addToProperty("data-tab",$datatab);
+		$segment->setAttachment(NULL, Side::BOTTOM)->addToProperty("class", "tab")->addToProperty("data-tab", $datatab);
 		return $segment;
 	}
 
@@ -62,12 +62,12 @@ 
 	 * @param String $content new content
 	 * @return \Ajax\semantic\html\modules\HtmlTab
 	 */
-	public function setTabContent($index,$content){
+	public function setTabContent($index, $content) {
 		$menu=$this->content["menu"];
-		if($index<$menu->count()){
-			if(isset($this->content[$index])===false){
+		if ($index<$menu->count()) {
+			if (isset($this->content[$index])===false) {
 				$this->content[$index]=$this->createSegment($index, $content, $menu->getItem($index)->getIdentifier());
-			}else
+			} else
 				$this->content[$index]->setContent($content);
 		}
 		return $this;
@@ -78,8 +78,8 @@ 
 	 * @param array $contents
 	 * @return \Ajax\semantic\html\modules\HtmlTab
 	 */
-	public function setTabsContent($contents){
-		for($i=0;$i<\sizeof($contents);$i++){
+	public function setTabsContent($contents) {
+		for ($i=0; $i<\sizeof($contents); $i++) {
 			$this->setTabContent($i, $contents[$i]);
 		}
 		return $this;
@@ -90,7 +90,7 @@ 
 	 * @param int $index
 	 * @return \Ajax\semantic\html\modules\HtmlTab
 	 */
-	public function activate($index){
+	public function activate($index) {
 		$this->content["menu"]->getItem($index)->setActive(true);
 		$this->content[$index]->setActive(true);
 		return $this;
@@ -102,8 +102,8 @@ 
 	 * @param string $content
 	 * @return \Ajax\semantic\html\elements\HtmlSegment
 	 */
-	public function addTab($title,$content){
-		return $this->addItem([$title,$content]);
+	public function addTab($title, $content) {
+		return $this->addItem([$title, $content]);
 	}
 
 	/**
@@ -117,13 +117,13 @@ 
 	 * @param array $params
 	 * @return \Ajax\semantic\html\elements\HtmlSegment
 	 */
-	public function forwardTab($index,JsUtils $js,$title,$initialController,$controller,$action,$params=array()){
-		if(\array_key_exists($index, $this->content)){
-			$this->content[$index]=$js->forward($initialController, $controller, $action,$params);
+	public function forwardTab($index, JsUtils $js, $title, $initialController, $controller, $action, $params=array()) {
+		if (\array_key_exists($index, $this->content)) {
+			$this->content[$index]=$js->forward($initialController, $controller, $action, $params);
 			return $this->content[$index];
 		}
 
-		return $this->addAndForwardTab($js, $title, $initialController, $controller, $action,$params);
+		return $this->addAndForwardTab($js, $title, $initialController, $controller, $action, $params);
 	}
 
 	/**
@@ -136,12 +136,12 @@ 
 	 * @param $params The parameters to pass to the view
 	 * @return \Ajax\semantic\html\elements\HtmlSegment
 	 */
-	public function renderViewTab($index,JsUtils $js,$title,$initialController, $viewName, $params=array()) {
-		if(\array_key_exists($index, $this->content)){
-			$this->content[$index]=$js->renderContent($initialController, $viewName,$params);
+	public function renderViewTab($index, JsUtils $js, $title, $initialController, $viewName, $params=array()) {
+		if (\array_key_exists($index, $this->content)) {
+			$this->content[$index]=$js->renderContent($initialController, $viewName, $params);
 			return $this->content[$index];
 		}
-		return $this->addAndRenderViewTab($js, $title, $initialController, $viewName,$params);
+		return $this->addAndRenderViewTab($js, $title, $initialController, $viewName, $params);
 	}
 
 
@@ -155,8 +155,8 @@ 
 	 * @param array $params
 	 * @return \Ajax\semantic\html\elements\HtmlSegment
 	 */
-	public function addAndForwardTab(JsUtils $js,$title,$initialController,$controller,$action,$params=array()){
-		return $this->addTab($title, $js->forward($initialController, $controller, $action,$params));
+	public function addAndForwardTab(JsUtils $js, $title, $initialController, $controller, $action, $params=array()) {
+		return $this->addTab($title, $js->forward($initialController, $controller, $action, $params));
 	}
 
 	/**
@@ -168,8 +168,8 @@ 
 	 * @param $params The parameters to pass to the view
 	 * @return \Ajax\semantic\html\elements\HtmlSegment
 	 */
-	public function addAndRenderViewTab(JsUtils $js,$title,$initialController, $viewName, $params=array()) {
-		return $this->addTab($title, $js->renderContent($initialController, $viewName,$params));
+	public function addAndRenderViewTab(JsUtils $js, $title, $initialController, $viewName, $params=array()) {
+		return $this->addTab($title, $js->renderContent($initialController, $viewName, $params));
 	}
 
 	public function setPointing($value=Direction::NONE) {
@@ -185,7 +185,7 @@ 
 	 * @param int $index
 	 * @return Ajax\semantic\html\content\HtmlMenuItem
 	 */
-	public function getMenuTab($index){
+	public function getMenuTab($index) {
 		return $this->content["menu"]->getItem($index);
 	}
 
@@ -194,7 +194,7 @@ 
 	 * @param int $index
 	 * @return HtmlSegment
 	 */
-	public function getTab($index){
+	public function getTab($index) {
 		return $this->content[$index];
 	}
 
@@ -203,20 +203,20 @@ 
 	 * @param HtmlMenu $menu
 	 * @return \Ajax\semantic\html\modules\HtmlTab
 	 */
-	public function setMenu($menu){
-		for($i=0;$i<\sizeof($this->content);$i++){
-			if($menu->getItem($i)!==NULL){
-				if(isset($this->content[$i])===true){
-					$menu->getItem($i)->addToProperty("data-tab",$this->content[$i]->getProperty("data-tab"));
+	public function setMenu($menu) {
+		for ($i=0; $i<\sizeof($this->content); $i++) {
+			if ($menu->getItem($i)!==NULL) {
+				if (isset($this->content[$i])===true) {
+					$menu->getItem($i)->addToProperty("data-tab", $this->content[$i]->getProperty("data-tab"));
 				}
 			}
 		}
-		for($i=0;$i<$menu->count();$i++){
+		for ($i=0; $i<$menu->count(); $i++) {
 			$menu->getItem($i)->removeProperty("href");
-			if(isset($this->content[$i])===false){
+			if (isset($this->content[$i])===false) {
 				$this->content[$i]=$this->createSegment($i, "New content", $menu->getItem($i)->getIdentifier());
 			}
-			$menu->getItem($i)->addToProperty("data-tab",$this->content[$i]->getProperty("data-tab"));
+			$menu->getItem($i)->addToProperty("data-tab", $this->content[$i]->getProperty("data-tab"));
 		}
 
 		$this->content["menu"]=$menu;
@@ -228,15 +228,15 @@ 
 	 * @see BaseHtml::run()
 	 */
 	public function run(JsUtils $js) {
-		if(isset($this->_bsComponent)===false)
-			$this->_bsComponent=$js->semantic()->tab("#".$this->identifier." .item",$this->params);
+		if (isset($this->_bsComponent)===false)
+			$this->_bsComponent=$js->semantic()->tab("#".$this->identifier." .item", $this->params);
 			$this->addEventsOnRun($js);
 			return $this->_bsComponent;
 	}
 
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
-		if($this->content["menu"]->count()>0 && \sizeof($this->content)>1)
+		if ($this->content["menu"]->count()>0 && \sizeof($this->content)>1)
 			$this->activate(0);
-		return parent::compile($js,$view);
+		return parent::compile($js, $view);
 	}
 }
\ No newline at end of file

Braces +9 added lines, -6 removed lines

@@ -67,8 +67,9 @@ 
 		if($index<$menu->count()){
 			if(isset($this->content[$index])===false){
 				$this->content[$index]=$this->createSegment($index, $content, $menu->getItem($index)->getIdentifier());
-			}else
-				$this->content[$index]->setContent($content);
+			} else {
+							$this->content[$index]->setContent($content);
+			}
 		}
 		return $this;
 	}
@@ -228,15 +229,17 @@ 
 	 * @see BaseHtml::run()
 	 */
 	public function run(JsUtils $js) {
-		if(isset($this->_bsComponent)===false)
-			$this->_bsComponent=$js->semantic()->tab("#".$this->identifier." .item",$this->params);
+		if(isset($this->_bsComponent)===false) {
+					$this->_bsComponent=$js->semantic()->tab("#".$this->identifier." .item",$this->params);
+		}
 			$this->addEventsOnRun($js);
 			return $this->_bsComponent;
 	}
 
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
-		if($this->content["menu"]->count()>0 && \sizeof($this->content)>1)
-			$this->activate(0);
+		if($this->content["menu"]->count()>0 && \sizeof($this->content)>1) {
+					$this->activate(0);
+		}
 		return parent::compile($js,$view);
 	}
 }
\ No newline at end of file

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 +17 added lines, -17 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,16 +53,16 @@ 
 	 * @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;
 	}
 
-	public function setPosition($direction){
-		$this->addToProperty("class",$direction);
+	public function setPosition($direction) {
+		$this->addToProperty("class", $direction);
 	}
 }
\ 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;
 	}
 

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/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/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);
 	}
 

Spacing +11 added lines, -11 removed lines

@@ -21,7 +21,7 @@ 
 	use BaseTrait;
 	protected $_popup=NULL;
 	protected $_dimmer=NULL;
-	protected $_params=array ();
+	protected $_params=array();
 
 
 	public function __construct($identifier, $tagName="p", $baseClass="ui", $content=NULL) {
@@ -51,7 +51,7 @@ 
 	}
 
 	public function addDimmer($params=array(), $content=NULL) {
-		$dimmer=new HtmlDimmer("dimmer-" . $this->identifier, $content);
+		$dimmer=new HtmlDimmer("dimmer-".$this->identifier, $content);
 		$dimmer->setParams($params);
 		$dimmer->setContainer($this);
 		$this->addContent($dimmer);
@@ -60,20 +60,20 @@ 
 
 	public function addLabel($label, $before=false, $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" ));
+			$labelO->addToPropertyCtrl("class", "label", array("label"));
 		}
 		$this->addContent($labelO, $before);
 		return $labelO;
 	}
 
-	public function attachLabel($label,$side,$direction=Direction::NONE,$icon=NULL){
-		$label=$this->addLabel($label,true,$icon);
-		$label->setAttached($side,$direction);
+	public function attachLabel($label, $side, $direction=Direction::NONE, $icon=NULL) {
+		$label=$this->addLabel($label, true, $icon);
+		$label->setAttached($side, $direction);
 		return $this;
 	}
 
@@ -89,9 +89,9 @@ 
 
 	public function jsShowDimmer($show=true) {
 		$status="hide";
-		if ($show === true)
+		if ($show===true)
 			$status="show";
-		return '$("#.' . $this->identifier . ').dimmer("' . $status . '");';
+		return '$("#.'.$this->identifier.').dimmer("'.$status.'");';
 	}
 
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
@@ -101,7 +101,7 @@ 
 	}
 
 	public function run(JsUtils $js) {
-		$this->_bsComponent=$js->semantic()->generic("#" . $this->identifier);
+		$this->_bsComponent=$js->semantic()->generic("#".$this->identifier);
 		parent::run($js);
 		$this->addEventsOnRun($js);
 		if (isset($this->_popup)) {

Ajax/semantic/html/modules/HtmlDropdown.php

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;
 		}

Doc Comments +7 added lines, -1 removed lines

@@ -49,7 +49,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 +240,9 @@ 
 		return $this->addToPropertyCtrl("class", $value." pointing",Direction::getConstantValues("pointing"));
 	}
 
+	/**
+	 * @param string $value
+	 */
 	public function setValue($value){
 		$this->value=$value;
 		return $this;
@@ -274,6 +277,9 @@ 
 		return $this->addToPropertyCtrl("class", "compact", array("compact"));
 	}
 
+	/**
+	 * @param string $action
+	 */
 	public function setAction($action){
 		$this->_params["action"]=$action;
 	}

Spacing +76 added lines, -76 removed lines

@@ -13,13 +13,13 @@ 
 use Ajax\semantic\html\collections\form\traits\FieldTrait;
 
 class HtmlDropdown extends HtmlSemDoubleElement {
-	use FieldTrait,LabeledIconTrait {
+	use FieldTrait, LabeledIconTrait {
 		addIcon as addIconP;
 	}
 	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;
 
@@ -27,27 +27,27 @@ 
 		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 getField(){
+	public function getField() {
 		return $this->input;
 	}
 
-	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("");
 	}
 
@@ -57,26 +57,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;
@@ -89,11 +89,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");
 	}
 
 	/**
@@ -102,15 +102,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());
 	}
 
@@ -120,8 +120,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));
 	}
 
 	/**
@@ -130,7 +130,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));
 	}
 
@@ -139,87 +139,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(\is_array($items) && JArray::isAssociative($items)){
-			foreach ($items as $k=>$v){
+	public function addItems($items) {
+		if (\is_array($items) && 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");
 		}
@@ -227,37 +227,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;
 	}
@@ -267,29 +267,29 @@ 
 	 * @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);
 	}
 
 	public function getInput() {

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"));
+		}
 	}
 
 	/**