phpMv / phpMv-UI

Ajax/semantic/html/views/HtmlCard.php

Braces +3 added lines, -2 removed lines

@@ -36,8 +36,9 @@
 
 	private function getPart($part, $index=NULL) {
 		if (\array_key_exists($part, $this->content)) {
-			if (isset($index))
-				return $this->content[$part][$index];
+			if (isset($index)) {
+							return $this->content[$part][$index];
+			}
 			return $this->content[$part];
 		}
 		return NULL;

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/html/views/HtmlCardGroups.php

Spacing +2 added lines, -2 removed lines

@@ -14,7 +14,7 @@ 
 	}
 
 	protected function createItem($value) {
-		$result=new HtmlCard("card-" . $this->count());
+		$result=new HtmlCard("card-".$this->count());
 		if (\is_array($value)) {
 			$header=JArray::getValue($value, "header", 0);
 			$metas=JArray::getValue($value, "metas", 1);
@@ -38,7 +38,7 @@ 
 	 * @param int $wide
 	 */
 	public function setWide($wide) {
-		$wide=Wide::getConstants()["W" . $wide];
+		$wide=Wide::getConstants()["W".$wide];
 		return $this->addToPropertyCtrl("class", $wide, Wide::getConstants());
 	}
 

Braces +3 added lines, -2 removed lines

@@ -28,8 +28,9 @@
 			if (isset($extra)) {
 				$result->addExtraContent($extra);
 			}
-		} else
-			$result->addCardContent($value);
+		} else {
+					$result->addCardContent($value);
+		}
 		return $result;
 	}
 

Ajax/Bootstrap.php

Spacing +11 added lines, -11 removed lines

@@ -176,8 +176,8 @@ 
 	 * @param string $tagName container tagName
 	 * @return HtmlListgroup
 	 */
-	public function htmlListgroup($identifier,$items=array(),$tagName="ul"){
-		$listGroup=new HtmlListgroup($identifier,$tagName);
+	public function htmlListgroup($identifier, $items=array(), $tagName="ul") {
+		$listGroup=new HtmlListgroup($identifier, $tagName);
 		$listGroup->addItems($items);
 		return $this->addHtmlComponent($listGroup);
 	}
@@ -202,8 +202,8 @@ 
 	 * @param string $onClick
 	 * @return HtmlSplitbutton
 	 */
-	public function htmlSplitbutton($identifier,$value="", $items=array(), $cssStyle="btn-default", $onClick=NULL) {
-		return $this->addHtmlComponent(new HtmlSplitbutton($identifier, $value, $items, $cssStyle,$onClick));
+	public function htmlSplitbutton($identifier, $value="", $items=array(), $cssStyle="btn-default", $onClick=NULL) {
+		return $this->addHtmlComponent(new HtmlSplitbutton($identifier, $value, $items, $cssStyle, $onClick));
 	}
 
 	/**
@@ -211,7 +211,7 @@ 
 	 * @param string $identifier
 	 * @return HtmlInputgroup
 	 */
-	public function htmlInputgroup($identifier){
+	public function htmlInputgroup($identifier) {
 		return $this->addHtmlComponent(new HtmlInputgroup($identifier));
 	}
 
@@ -223,8 +223,8 @@ 
 	 * @param function $hrefFunction the function who generates the href elements. default : function($e){return $e->getContent()}
 	 * @return HtmlBreadcrumbs
 	 */
-	public function htmlBreadcrumbs($identifier,$elements=array(),$autoActive=true,$startIndex=0,$hrefFunction=NULL){
-		return $this->addHtmlComponent(new HtmlBreadcrumbs($identifier,$elements,$autoActive,$startIndex,$hrefFunction));
+	public function htmlBreadcrumbs($identifier, $elements=array(), $autoActive=true, $startIndex=0, $hrefFunction=NULL) {
+		return $this->addHtmlComponent(new HtmlBreadcrumbs($identifier, $elements, $autoActive, $startIndex, $hrefFunction));
 	}
 
 	/**
@@ -236,8 +236,8 @@ 
 	 * @param int $active The active page
 	 * @return HtmlPagination
 	 */
-	public function htmlPagination($identifier,$from=1,$to=1,$active=NULL,$countVisible=NULL){
-		return $this->addHtmlComponent(new HtmlPagination($identifier,$from,$to,$active,$countVisible));
+	public function htmlPagination($identifier, $from=1, $to=1, $active=NULL, $countVisible=NULL) {
+		return $this->addHtmlComponent(new HtmlPagination($identifier, $from, $to, $active, $countVisible));
 	}
 
 	/**
@@ -248,7 +248,7 @@ 
 	 * @param int $numCols
 	 * @return HtmlGridSystem
 	 */
-	public function htmlGridSystem($identifier,$numRows=1,$numCols=NULL){
-		return $this->addHtmlComponent(new HtmlGridSystem($identifier,$numRows,$numCols));
+	public function htmlGridSystem($identifier, $numRows=1, $numCols=NULL) {
+		return $this->addHtmlComponent(new HtmlGridSystem($identifier, $numRows, $numCols));
 	}
 }
\ No newline at end of file

Ajax/bootstrap/components/Tabs.php

Spacing +2 added lines, -2 removed lines

@@ -19,7 +19,7 @@ 
 
 	public function __construct(JsUtils $js) {
 		parent::__construct($js);
-		$this->tabs=array ();
+		$this->tabs=array();
 	}
 
 	public function getTabs() {
@@ -36,7 +36,7 @@ 
 	}
 
 	public function getTab($index) {
-		if ($index>0&&$index<sizeof($this->tabs))
+		if ($index>0 && $index<sizeof($this->tabs))
 			return $this->tabs [$index];
 	}
 

Braces +15 added lines, -10 removed lines

@@ -36,8 +36,9 @@ 
 	}
 
 	public function getTab($index) {
-		if ($index>0&&$index<sizeof($this->tabs))
-			return $this->tabs [$index];
+		if ($index>0&&$index<sizeof($this->tabs)) {
+					return $this->tabs [$index];
+		}
 	}
 
 	public function show($index) {
@@ -53,8 +54,9 @@ 
 	 */
 	public function onShow($index, $jsCode) {
 		$tab=$this->getTab($index);
-		if (isset($tab))
-			return $tab->onShow($jsCode);
+		if (isset($tab)) {
+					return $tab->onShow($jsCode);
+		}
 	}
 
 	/**
@@ -65,8 +67,9 @@ 
 	 */
 	public function onShown($index, $jsCode) {
 		$tab=$this->getTab($index);
-		if (isset($tab))
-			return $tab->onShown($jsCode);
+		if (isset($tab)) {
+					return $tab->onShown($jsCode);
+		}
 	}
 
 	/**
@@ -77,8 +80,9 @@ 
 	 */
 	public function onHide($index, $jsCode) {
 		$tab=$this->getTab($index);
-		if (isset($tab))
-			return $tab->onShow($jsCode);
+		if (isset($tab)) {
+					return $tab->onShow($jsCode);
+		}
 	}
 
 	/**
@@ -89,7 +93,8 @@ 
 	 */
 	public function onHidden($index, $jsCode) {
 		$tab=$this->getTab($index);
-		if (isset($tab))
-			return $tab->onShow($jsCode);
+		if (isset($tab)) {
+					return $tab->onShow($jsCode);
+		}
 	}
 }
\ No newline at end of file

Ajax/bootstrap/components/Splitbutton.php

Spacing +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
 	 * @see \Ajax\common\SimpleComponent::compileEvents()
 	 */
 	protected function compileEvents() {
-		foreach ( $this->events as $event => $jsCode ) {
+		foreach ($this->events as $event => $jsCode) {
 			if ($event==="buttonClick") {
 				$this->jquery_code_for_compile []="$( \"#split-".preg_replace('/[^a-zA-Z0-9\-.]/s', '', $this->attachTo)."\" ).on(\"click\" , function( event, data ) {".$jsCode."});";
 			} else {

Ajax/bootstrap/html/HtmlInput.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
 		}
 	}
 
-	public function setPlaceHolder($value){
+	public function setPlaceHolder($value) {
 		$this->setProperty("placeholder", $value);
 	}
 

Ajax/bootstrap/html/HtmlBreadcrumbs.php

Spacing +35 added lines, -35 removed lines

@@ -41,15 +41,15 @@ 
 	 * @param boolean $autoActive sets the last element's class to <b>active</b> if true
 	 * @param function $hrefFunction the function who generates the href elements. default : function($e){return $e->getContent()}
 	 */
-	public function __construct($identifier,$elements=array(),$autoActive=true,$startIndex=0,$hrefFunction=NULL){
-		parent::__construct($identifier,"ol");
+	public function __construct($identifier, $elements=array(), $autoActive=true, $startIndex=0, $hrefFunction=NULL) {
+		parent::__construct($identifier, "ol");
 		$this->startIndex=$startIndex;
 		$this->setProperty("class", "breadcrumb");
 		$this->content=array();
 		$this->autoActive=$autoActive;
 		$this->absolutePaths;
-		$this->_hrefFunction=function ($e){return $e->getContent();};
-		if(isset($hrefFunction)){
+		$this->_hrefFunction=function($e) {return $e->getContent(); };
+		if (isset($hrefFunction)) {
 			$this->_hrefFunction=$hrefFunction;
 		}
 		$this->addElements($elements);
@@ -60,43 +60,43 @@ 
 	 * @param string $href
 	 * @return \Ajax\bootstrap\html\HtmlLink
 	 */
-	public function addElement($element,$href="",$glyph=NULL){
+	public function addElement($element, $href="", $glyph=NULL) {
 		$size=sizeof($this->content);
-		if(is_array($element)){
+		if (is_array($element)) {
 			$elm=new HtmlLink("lnk-".$this->identifier."-".$size);
 			$elm->fromArray($element);
-		}else if($element instanceof HtmlLink){
+		} else if ($element instanceof HtmlLink) {
 			$elm=$element;
-		}else{
-			$elm=new HtmlLink("lnk-".$this->identifier."-".$size,$href,$element);
-			if(isset($glyph)){
+		} else {
+			$elm=new HtmlLink("lnk-".$this->identifier."-".$size, $href, $element);
+			if (isset($glyph)) {
 				$elm->wrapContentWithGlyph($glyph);
 			}
 		}
-		$elm->wrap("<li>","</li>");
+		$elm->wrap("<li>", "</li>");
 		$this->content[]=$elm;
 		$elm->setProperty($this->attr, $this->getHref($size));
 		return $elm;
 	}
 
-	public function setActive($index=null){
-		if(!isset($index)){
+	public function setActive($index=null) {
+		if (!isset($index)) {
 			$index=sizeof($this->content)-1;
 		}
-		$li=new HtmlBsDoubleElement("","li");
+		$li=new HtmlBsDoubleElement("", "li");
 		$li->setClass("active");
 		$li->setContent($this->content[$index]->getContent());
 		$this->content[$index]=$li;
 	}
 
-	public function addElements($elements){
-		foreach ( $elements as $element ) {
+	public function addElements($elements) {
+		foreach ($elements as $element) {
 			$this->addElement($element);
 		}
 		return $this;
 	}
 
-	public function fromArray($array){
+	public function fromArray($array) {
 		$array=parent::fromArray($array);
 		$this->addElements($array);
 		return $array;
@@ -108,14 +108,14 @@ 
 	 * @param string $separator
 	 * @return string
 	 */
-	public function getHref($index=null,$separator="/"){
-		if(!isset($index)){
+	public function getHref($index=null, $separator="/") {
+		if (!isset($index)) {
 			$index=sizeof($this->content);
 		}
-		if($this->absolutePaths===true){
+		if ($this->absolutePaths===true) {
 			return $this->_hrefFunction($this->content[$index]);
-		}else{
-			return $this->root.implode($separator, array_slice(array_map(function($e){return $this->_hrefFunction($e);}, $this->content),$this->startIndex,$index+1));
+		} else {
+			return $this->root.implode($separator, array_slice(array_map(function($e) {return $this->_hrefFunction($e); }, $this->content), $this->startIndex, $index+1));
 		}
 	}
 
@@ -124,7 +124,7 @@ 
 	 * @see \Ajax\bootstrap\html\BaseHtml::compile()
 	 */
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
-		if($this->autoActive){
+		if ($this->autoActive) {
 			$this->setActive();
 		}
 		return parent::compile($js, $view);
@@ -142,19 +142,19 @@ 
 	 * @see \Ajax\bootstrap\html\base\BaseHtml::on()
 	 */
 	public function on($event, $jsCode, $stopPropagation=false, $preventDefault=false) {
-		foreach ($this->content as $element){
-			$element->on($event,$jsCode,$stopPropagation,$preventDefault);
+		foreach ($this->content as $element) {
+			$element->on($event, $jsCode, $stopPropagation, $preventDefault);
 		}
 		return $this;
 	}
 
 	public function setAutoActive($autoActive) {
-		$this->autoActive = $autoActive;
+		$this->autoActive=$autoActive;
 		return $this;
 	}
 
 	public function _ajaxOn($operation, $event, $url, $responseElement="", $parameters=array()) {
-		foreach ($this->content as $element){
+		foreach ($this->content as $element) {
 			$element->_ajaxOn($operation, $event, $url, $responseElement, $parameters);
 		}
 		return $this;
@@ -166,18 +166,18 @@ 
 	 * @param string $targetSelector the target of the get
 	 * @return HtmlBreadcrumbs
 	 */
-	public function autoGetOnClick($targetSelector){
-		return $this->getOnClick($this->root, $targetSelector,array("attr"=>$this->attr));
+	public function autoGetOnClick($targetSelector) {
+		return $this->getOnClick($this->root, $targetSelector, array("attr"=>$this->attr));
 	}
 
-	public function contentAsString(){
-		if($this->autoActive){
+	public function contentAsString() {
+		if ($this->autoActive) {
 			$this->setActive();
 		}
 		return parent::contentAsString();
 	}
 
-	public function getElement($index){
+	public function getElement($index) {
 		return $this->content[$index];
 	}
 
@@ -186,7 +186,7 @@ 
 	 * @param mixed $glyph
 	 * @param int $index
 	 */
-	public function addGlyph($glyph,$index=0){
+	public function addGlyph($glyph, $index=0) {
 		$elm=$this->getElement($index);
 		return $elm->wrapContentWithGlyph($glyph);
 	}
@@ -197,7 +197,7 @@ 
 	 * @param Dispatcher $dispatcher the request dispatcher
 	 * @return \Ajax\bootstrap\html\HtmlBreadcrumbs
 	 */
-	public function fromDispatcher(JsUtils $js,$dispatcher,$startIndex=0){
+	public function fromDispatcher(JsUtils $js, $dispatcher, $startIndex=0) {
 		$this->startIndex=$startIndex;
 		return $this->addElements($js->fromDispatcher($dispatcher));
 	}
@@ -209,7 +209,7 @@ 
 	 * @return \Ajax\bootstrap\html\HtmlBreadcrumbs
 	 */
 	public function setHrefFunction($_hrefFunction) {
-		$this->_hrefFunction = $_hrefFunction;
+		$this->_hrefFunction=$_hrefFunction;
 		return $this;
 	}
 

Braces +3 added lines, -3 removed lines

@@ -65,9 +65,9 @@ 
 		if(is_array($element)){
 			$elm=new HtmlLink("lnk-".$this->identifier."-".$size);
 			$elm->fromArray($element);
-		}else if($element instanceof HtmlLink){
+		} else if($element instanceof HtmlLink){
 			$elm=$element;
-		}else{
+		} else{
 			$elm=new HtmlLink("lnk-".$this->identifier."-".$size,$href,$element);
 			if(isset($glyph)){
 				$elm->wrapContentWithGlyph($glyph);
@@ -114,7 +114,7 @@ 
 		}
 		if($this->absolutePaths===true){
 			return $this->_hrefFunction($this->content[$index]);
-		}else{
+		} else{
 			return $this->root.implode($separator, array_slice(array_map(function($e){return $this->_hrefFunction($e);}, $this->content),$this->startIndex,$index+1));
 		}
 	}

Doc Comments +3 added lines

@@ -177,6 +177,9 @@
 		return parent::contentAsString();
 	}
 
+	/**
+	 * @param integer $index
+	 */
 	public function getElement($index){
 		return $this->content[$index];
 	}

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 {

Ajax/bootstrap/html/HtmlCollapse.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
 		if (!isset($id))
 			if ($this->element->getTagName()==="a")
 				$id=$this->element->getProperty("href");
-		if (!isset($id)||$id==="#") {
+		if (!isset($id) || $id==="#") {
 			$id="collapse-".$this->element->getIdentifier();
 			$this->attachTo($id);
 		}

Braces +11 added lines, -8 removed lines

@@ -24,17 +24,19 @@ 
 
 	public function attachTo($identifier) {
 		$this->element->setProperty("aria-controls", $identifier);
-		if ($this->element->getTagName()==="a")
-			$this->element->setProperty("href", "#".$identifier);
-		else
-			$this->element->setProperty("data-target", "#".$identifier);
+		if ($this->element->getTagName()==="a") {
+					$this->element->setProperty("href", "#".$identifier);
+		} else {
+					$this->element->setProperty("data-target", "#".$identifier);
+		}
 	}
 
 	public function getAttachedZone() {
 		$id=$this->element->getProperty("aria-controls");
-		if (!isset($id))
-			if ($this->element->getTagName()==="a")
+		if (!isset($id)) {
+					if ($this->element->getTagName()==="a")
 				$id=$this->element->getProperty("href");
+		}
 		if (!isset($id)||$id==="#") {
 			$id="collapse-".$this->element->getIdentifier();
 			$this->attachTo($id);
@@ -49,8 +51,9 @@ 
 	}
 
 	public function createCollapsedZone($content="", $attachTo=NULL) {
-		if (isset($attachTo))
-			$this->attachTo($attachTo);
+		if (isset($attachTo)) {
+					$this->attachTo($attachTo);
+		}
 		$collapsedZone=new HtmlBsDoubleElement($this->getAttachedZone());
 		$collapsedZone->setProperty("class", "collapse");
 		$collapsedZone->setContent($content);

Ajax/bootstrap/html/HtmlTabs.php

Spacing +16 added lines, -16 removed lines

@@ -16,7 +16,7 @@ 
 
 
 class HtmlTabs extends HtmlBsDoubleElement {
-	protected $tabs=array ();
+	protected $tabs=array();
 	protected $_tabsType="tabs";
 	protected $stacked="";
 
@@ -27,11 +27,11 @@ 
 	}
 
 	protected function addTab_($tab, $index=null) {
-		if($tab instanceof HtmlDropdown){
+		if ($tab instanceof HtmlDropdown) {
 			$tab->setMTagName("li");
 		}
 		if (isset($index)) {
-			$inserted=array (
+			$inserted=array(
 					$tab
 			);
 			array_splice($this->tabs, $index, 0, $inserted);
@@ -39,13 +39,13 @@ 
 			$this->tabs []=$tab;
 	}
 
-	public function setActive($index){
-		for ($i=0;$i<sizeof($this->tabs);$i++){
+	public function setActive($index) {
+		for ($i=0; $i<sizeof($this->tabs); $i++) {
 			$this->tabs[$i]->setActive($i==$index);
 		}
 	}
 
-	public function disable($index){
+	public function disable($index) {
 		$this->tabs[$index]->disable();
 	}
 
@@ -76,7 +76,7 @@ 
 	}
 
 	public function addTabs($tabs) {
-		foreach ( $tabs as $tab ) {
+		foreach ($tabs as $tab) {
 			$this->addTab($tab);
 		}
 		return $this;
@@ -106,7 +106,7 @@ 
 	 */
 	public function run(JsUtils $js) {
 		$this->_bsComponent=new Tabs($js);
-		foreach ( $this->tabs as $tab ) {
+		foreach ($this->tabs as $tab) {
 			$this->_bsComponent->addTab($tab->run($js));
 		}
 		$this->addEventsOnRun($js);
@@ -115,11 +115,11 @@ 
 
 	public function createTabContents() {
 		$tabContent=new HtmlTabContent("tabcontent-".$this->identifier);
-		foreach ( $this->tabs as $tab ) {
+		foreach ($this->tabs as $tab) {
 			if ($tab instanceof HtmlTabItem)
 				$tabContent->addTabItem($tab->getHref());
 			elseif ($tab instanceof HtmlDropdown) {
-				foreach ( $tab->getItems() as $dropdownItem ) {
+				foreach ($tab->getItems() as $dropdownItem) {
 					$tabContent->addTabItem($dropdownItem->getHref());
 				}
 			}
@@ -154,21 +154,21 @@ 
 		if (sizeof($this->content->getTabItems())>0) {
 			$this->content->getTabItem(0)->addToProperty("class", "fade in");
 			$size=sizeof($this->tabs);
-			for($index=0; $index<$size; $index++) {
+			for ($index=0; $index<$size; $index++) {
 				$this->content->getTabItem($index)->addToProperty("class", "fade");
 			}
 		}
 	}
 	
-	public function on($event, $jsCode,$stopPropagation=false,$preventDefault=false){
-		foreach ($this->tabs as $tab){
-			$tab->on($event,$jsCode,$stopPropagation,$preventDefault);
+	public function on($event, $jsCode, $stopPropagation=false, $preventDefault=false) {
+		foreach ($this->tabs as $tab) {
+			$tab->on($event, $jsCode, $stopPropagation, $preventDefault);
 		}
 		return $this;
 	}
 
-	public function setStacked($stacked=true){
-		if($stacked)
+	public function setStacked($stacked=true) {
+		if ($stacked)
 			$this->stacked="nav-stacked";
 		else $this->stacked="";
 	}

Braces +17 added lines, -12 removed lines

@@ -35,8 +35,9 @@ 
 					$tab
 			);
 			array_splice($this->tabs, $index, 0, $inserted);
-		} else
-			$this->tabs []=$tab;
+		} else {
+					$this->tabs []=$tab;
+		}
 	}
 
 	public function setActive($index){
@@ -116,9 +117,9 @@ 
 	public function createTabContents() {
 		$tabContent=new HtmlTabContent("tabcontent-".$this->identifier);
 		foreach ( $this->tabs as $tab ) {
-			if ($tab instanceof HtmlTabItem)
-				$tabContent->addTabItem($tab->getHref());
-			elseif ($tab instanceof HtmlDropdown) {
+			if ($tab instanceof HtmlTabItem) {
+							$tabContent->addTabItem($tab->getHref());
+			} elseif ($tab instanceof HtmlDropdown) {
 				foreach ( $tab->getItems() as $dropdownItem ) {
 					$tabContent->addTabItem($dropdownItem->getHref());
 				}
@@ -137,8 +138,9 @@ 
 
 	public function setContentToTab($index, $text) {
 		$tabContentItem=$this->content->getTabItem($index);
-		if (isset($tabContentItem))
-			$tabContentItem->setContent($text);
+		if (isset($tabContentItem)) {
+					$tabContentItem->setContent($text);
+		}
 	}
 
 	public function countTabs() {
@@ -146,8 +148,9 @@ 
 	}
 
 	public function getTabItem($index) {
-		if ($index<sizeof($this->content->get))
-			return $this->content;
+		if ($index<sizeof($this->content->get)) {
+					return $this->content;
+		}
 	}
 
 	public function fadeEffect() {
@@ -168,9 +171,11 @@ 
 	}
 
 	public function setStacked($stacked=true){
-		if($stacked)
-			$this->stacked="nav-stacked";
-		else $this->stacked="";
+		if($stacked) {
+					$this->stacked="nav-stacked";
+		} else {
+			$this->stacked="";
+		}
 	}
 
 	/* (non-PHPdoc)

Doc Comments +3 added lines

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