phpMv / phpMv-UI

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

Doc Comments +2 added lines, -1 removed lines

@@ -33,7 +33,7 @@ 
 	/**
 	 * Sets the menu type
 	 * @param string $type one of text,item
-	 * @return \Ajax\semantic\html\collections\HtmlMenu
+	 * @return HtmlMenu
 	 */
 	public function setType($type="") {
 		return $this->addToPropertyCtrl("class", $type, array ("","item","text" ));
@@ -81,6 +81,7 @@ 
 	 * {@inheritDoc}
 	 *
 	 * @see \Ajax\common\html\HtmlCollection::insertItem()
+	 * @param HtmlIcon $item
 	 */
 	public function insertItem($item, $position=0) {
 		$item=parent::insertItem($this->getItemToInsert($item), $position);

Braces +19 added lines, -15 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;
@@ -75,8 +75,9 @@ 
 	public function addItem($item) {
 		$number=$item;
 		$item=parent::addItem($this->getItemToInsert($item));
-		if(\is_int($number))
-			$item->setProperty("data-page", $number);
+		if(\is_int($number)) {
+					$item->setProperty("data-page", $number);
+		}
 		return $this->afterInsert($item);
 	}
 
@@ -149,10 +150,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");
+		}
 		return $this;
 	}
 
@@ -172,8 +174,9 @@ 
 		$this->apply(function (HtmlDoubleElement &$item) {
 			$item->setTagName("a");
 		});
-		if ($vertical === true)
-			$this->setVertical();
+		if ($vertical === true) {
+					$this->setVertical();
+		}
 		return $this->addToProperty("class", "tabular");
 	}
 
@@ -202,10 +205,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);
+		}
 	}
 
 	/**

Spacing +24 added lines, -24 removed lines

@@ -39,20 +39,20 @@ 
 	 * @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;
 	}
 
 	private function getItemToInsert($item) {
-		if ($item instanceof HtmlInput || $item instanceof HtmlImg || $item instanceof HtmlIcon || $item instanceof HtmlButtonGroups || $item instanceof HtmlButton ) {
-			$itemO=new HtmlMenuItem("item-" . $this->identifier . "-" . \sizeof($this->content) , $item);
+		if ($item instanceof HtmlInput || $item instanceof HtmlImg || $item instanceof HtmlIcon || $item instanceof HtmlButtonGroups || $item instanceof HtmlButton) {
+			$itemO=new HtmlMenuItem("item-".$this->identifier."-".\sizeof($this->content), $item);
 			$item=$itemO;
 		}
 		return $item;
@@ -60,7 +60,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();
 		}
@@ -76,7 +76,7 @@ 
 	public function addItem($item) {
 		$number=$item;
 		$item=parent::addItem($this->getItemToInsert($item));
-		if(\is_int($number))
+		if (\is_int($number))
 			$item->setProperty("data-page", $number);
 		return $this->afterInsert($item);
 	}
@@ -94,15 +94,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);
@@ -117,7 +117,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;
@@ -126,7 +126,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;
@@ -139,7 +139,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;
@@ -150,7 +150,7 @@ 
 	}
 
 	public function setSecondary($value=true) {
-		if($value)
+		if ($value)
 			$this->addToProperty("class", "secondary");
 		else
 			$this->removePropertyValue("class", "secondary");
@@ -158,28 +158,28 @@ 
 	}
 
 	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");
@@ -216,10 +216,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="") {
@@ -234,8 +234,8 @@ 
 		return $this->_itemHeader;
 	}
 
-	public function run(JsUtils $js){
-		$result= parent::run($js);
+	public function run(JsUtils $js) {
+		$result=parent::run($js);
 		return $result->setItemSelector(".item");
 	}
 }
\ No newline at end of file

Ajax/semantic/html/content/HtmlAbsractItem.php

Doc Comments +3 added lines

@@ -11,6 +11,9 @@
 
 abstract class HtmlAbsractItem extends HtmlSemDoubleElement {
 
+	/**
+	 * @param string $baseClass
+	 */
 	public function __construct($identifier, $baseClass,$content=NULL) {
 		parent::__construct($identifier, "div", $baseClass);
 		$this->content=array();

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use Ajax\semantic\html\base\HtmlSemDoubleElement;
 use Ajax\semantic\html\elements\HtmlIcon;
 use Ajax\JsUtils;
-
 use Ajax\service\JArray;
 use Ajax\semantic\html\elements\html5\HtmlImg;
 

Spacing +19 added lines, -19 removed lines

@@ -11,7 +11,7 @@ 
 
 abstract class HtmlAbsractItem extends HtmlSemDoubleElement {
 
-	public function __construct($identifier, $baseClass,$content=NULL) {
+	public function __construct($identifier, $baseClass, $content=NULL) {
 		parent::__construct($identifier, "div", $baseClass);
 		$this->content=array();
 		$this->initContent($content);
@@ -19,53 +19,53 @@ 
 
 	abstract protected function initContent($content);
 
-	public function setIcon($icon){
+	public function setIcon($icon) {
 		$this->content["icon"]=new HtmlIcon("icon-".$this->identifier, $icon);
 	}
 
-	public function setImage($image){
+	public function setImage($image) {
 		$image=new HtmlImg("icon-".$this->identifier, $image);
 		$image->asAvatar();
 		$this->content["image"]=$image;
 	}
 
-	private function createContent(){
-		$this->content["content"]=new HtmlSemDoubleElement("content-".$this->identifier,"div","content");
+	private function createContent() {
+		$this->content["content"]=new HtmlSemDoubleElement("content-".$this->identifier, "div", "content");
 		return $this->content["content"];
 	}
 
-	public function setTitle($title,$description=NULL,$baseClass="title"){
-		$title=new HtmlSemDoubleElement("","div",$baseClass,$title);
-		if(\array_key_exists("content", $this->content)===false){
+	public function setTitle($title, $description=NULL, $baseClass="title") {
+		$title=new HtmlSemDoubleElement("", "div", $baseClass, $title);
+		if (\array_key_exists("content", $this->content)===false) {
 			$this->createContent();
 		}
 		$this->content["content"]->addContent($title);
-		if(isset($description)){
-			$description=new HtmlSemDoubleElement("","div","description",$description);
+		if (isset($description)) {
+			$description=new HtmlSemDoubleElement("", "div", "description", $description);
 			$this->content["content"]->addContent($description);
 		}
 		return $this;
 	}
 
-	public function getPart($partName="header"){
-		$content=\array_merge($this->content["content"]->getContent(),array(@$this->content["icon"],@$this->content["image"]));
+	public function getPart($partName="header") {
+		$content=\array_merge($this->content["content"]->getContent(), array(@$this->content["icon"], @$this->content["image"]));
 		return $this->getElementByPropertyValue("class", $partName, $content);
 	}
 
-	public function setActive($value=true){
-		if($value){
+	public function setActive($value=true) {
+		if ($value) {
 			$this->setTagName("div");
 			$this->removeProperty("href");
 			$this->addToPropertyCtrl("class", "active", array("active"));
-		}else{
+		} else {
 			$this->removePropertyValue("class", "active");
 		}
 		return $this;
 	}
 
-	public function asLink($href=NULL,$part=NULL){
+	public function asLink($href=NULL, $part=NULL) {
 		$this->setTagName("a");
-		if(isset($href))
+		if (isset($href))
 			$this->setProperty("href", $href);
 		return $this;
 	}
@@ -77,8 +77,8 @@ 
 	 * @see \Ajax\semantic\html\base\HtmlSemDoubleElement::compile()
 	 */
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
-		if(\is_array($this->content) && JArray::isAssociative($this->content))
-			$this->content=JArray::sortAssociative($this->content, [ "icon","image","content" ]);
+		if (\is_array($this->content) && JArray::isAssociative($this->content))
+			$this->content=JArray::sortAssociative($this->content, ["icon", "image", "content"]);
 		return parent::compile($js, $view);
 	}
 }
\ No newline at end of file

Braces +7 added lines, -5 removed lines

@@ -57,7 +57,7 @@ 
 			$this->setTagName("div");
 			$this->removeProperty("href");
 			$this->addToPropertyCtrl("class", "active", array("active"));
-		}else{
+		} else{
 			$this->removePropertyValue("class", "active");
 		}
 		return $this;
@@ -65,8 +65,9 @@ 
 
 	public function asLink($href=NULL,$part=NULL){
 		$this->setTagName("a");
-		if(isset($href))
-			$this->setProperty("href", $href);
+		if(isset($href)) {
+					$this->setProperty("href", $href);
+		}
 		return $this;
 	}
 
@@ -77,8 +78,9 @@ 
 	 * @see \Ajax\semantic\html\base\HtmlSemDoubleElement::compile()
 	 */
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
-		if(\is_array($this->content) && JArray::isAssociative($this->content))
-			$this->content=JArray::sortAssociative($this->content, [ "icon","image","content" ]);
+		if(\is_array($this->content) && JArray::isAssociative($this->content)) {
+					$this->content=JArray::sortAssociative($this->content, [ "icon","image","content" ]);
+		}
 		return parent::compile($js, $view);
 	}
 }
\ No newline at end of file

Ajax/semantic/html/content/HtmlGridRow.php

Doc Comments +4 added lines, -1 removed lines

@@ -22,6 +22,9 @@ 
 	private $_colSize;
 	private $_implicite=false;
 
+	/**
+	 * @param string $identifier
+	 */
 	public function __construct( $identifier,$numCols=NULL,$colSizing=false,$implicite=false){
 		parent::__construct( $identifier,"div","row");
 		$this->_implicite=$implicite;
@@ -55,7 +58,7 @@ 
 	/**
 	 * return the col at $index
 	 * @param int $index
-	 * @return \Ajax\semantic\html\collections\HtmlGridCol
+	 * @return \Ajax\common\html\HtmlDoubleElement
 	 */
 	public function getCol($index){
 		return $this->getItem($index);

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use Ajax\semantic\html\base\constants\Wide;
 use Ajax\semantic\html\base\constants\Color;
 use Ajax\JsUtils;
-
 use Ajax\semantic\html\base\HtmlSemCollection;
 use Ajax\semantic\html\base\traits\TextAlignmentTrait;
 

Braces +5 added lines, -4 removed lines

@@ -29,10 +29,11 @@
 		if(isset($numCols)){
 			$numCols=min(16,$numCols);
 			$numCols=max(1,$numCols);
-			if($colSizing)
-				$width=(int)(16/$numCols);
-			else
-				$this->_colSize=16/$numCols;
+			if($colSizing) {
+							$width=(int)(16/$numCols);
+			} else {
+							$this->_colSize=16/$numCols;
+			}
 			for ($i=0;$i<$numCols;$i++){
 				$this->addItem($width);
 			}

Spacing +32 added lines, -32 removed lines

@@ -16,24 +16,24 @@ 
  * @author jc
  * @version 1.001
  */
-class HtmlGridRow extends HtmlSemCollection{
+class HtmlGridRow extends HtmlSemCollection {
 	use TextAlignmentTrait;
 
 	private $_colSize;
 	private $_implicite=false;
 
-	public function __construct( $identifier,$numCols=NULL,$colSizing=false,$implicite=false){
-		parent::__construct( $identifier,"div","row");
+	public function __construct($identifier, $numCols=NULL, $colSizing=false, $implicite=false) {
+		parent::__construct($identifier, "div", "row");
 		$this->_implicite=$implicite;
 		$width=null;
-		if(isset($numCols)){
-			$numCols=min(16,$numCols);
-			$numCols=max(1,$numCols);
-			if($colSizing)
+		if (isset($numCols)) {
+			$numCols=min(16, $numCols);
+			$numCols=max(1, $numCols);
+			if ($colSizing)
 				$width=(int)(16/$numCols);
 			else
 				$this->_colSize=16/$numCols;
-			for ($i=0;$i<$numCols;$i++){
+			for ($i=0; $i<$numCols; $i++) {
 				$this->addItem($width);
 			}
 		}
@@ -44,12 +44,12 @@ 
 	 * @param int $width
 	 * @return \Ajax\semantic\html\content\HtmlGridRow
 	 */
-	public function setWidth($width){
-		if(\is_int($width)){
+	public function setWidth($width) {
+		if (\is_int($width)) {
 			$width=Wide::getConstants()["W".$width];
 		}
 		$this->addToPropertyCtrl("class", $width, Wide::getConstants());
-		return $this->addToPropertyCtrl("class", "column",array("column"));
+		return $this->addToPropertyCtrl("class", "column", array("column"));
 	}
 
 	/**
@@ -57,29 +57,29 @@ 
 	 * @param int $index
 	 * @return \Ajax\semantic\html\collections\HtmlGridCol
 	 */
-	public function getCol($index){
+	public function getCol($index) {
 		return $this->getItem($index);
 	}
 
-	public function setColsCount($colsCount,$toCreate=true){
+	public function setColsCount($colsCount, $toCreate=true) {
 		$this->setWidth($colsCount);
-		if($toCreate===true){
+		if ($toCreate===true) {
 			$count=$this->count();
-			for($i=$count;$i<$colsCount;$i++){
+			for ($i=$count; $i<$colsCount; $i++) {
 				$this->addItem(null);
 			}
 		}
 		return $this;
 	}
 
-	public function addCols($colCount){
-		for($i=0;$i<$colCount;$i++){
+	public function addCols($colCount) {
+		for ($i=0; $i<$colCount; $i++) {
 			$this->addItem(null);
 		}
 		return $this;
 	}
 
-	public function addCol($width=NULL){
+	public function addCol($width=NULL) {
 		return $this->addItem($width);
 	}
 
@@ -87,7 +87,7 @@ 
 	 * stretch the row contents to take up the entire column height
 	 * @return \Ajax\semantic\html\content\HtmlGridRow
 	 */
-	public function setStretched(){
+	public function setStretched() {
 		return $this->addToProperty("class", "stretched");
 	}
 
@@ -95,20 +95,20 @@ 
 	 * @param string $color
 	 * @return \Ajax\semantic\html\content\HtmlGridRow
 	 */
-	public function setColor($color){
-		return $this->addToPropertyCtrl("class", $color,Color::getConstants());
+	public function setColor($color) {
+		return $this->addToPropertyCtrl("class", $color, Color::getConstants());
 	}
 
-	public function setValues($values,$force=false){
+	public function setValues($values, $force=false) {
 		$count=$this->count();
 		$valuesSize=\sizeof($values);
-		if($force===true){
-			for($i=$count;$i<$valuesSize;$i++){
-				$this->addItem(new HtmlGridCol($this->identifier."-col-".($this->count()+1),null));
+		if ($force===true) {
+			for ($i=$count; $i<$valuesSize; $i++) {
+				$this->addItem(new HtmlGridCol($this->identifier."-col-".($this->count()+1), null));
 			}
 		}
-		$count=\min(array($this->count(),$valuesSize));
-		for($i=0;$i<$count;$i++){
+		$count=\min(array($this->count(), $valuesSize));
+		for ($i=0; $i<$count; $i++) {
 			$this->content[$i]->setValue($values[$i]);
 		}
 		return $this;
@@ -118,15 +118,15 @@ 
 	 * {@inheritDoc}
 	 * @see \Ajax\common\html\HtmlCollection::createItem()
 	 */
-	protected function createItem($value){
-		$col=new HtmlGridCol($this->identifier."-col-".($this->count()+1),$value);
+	protected function createItem($value) {
+		$col=new HtmlGridCol($this->identifier."-col-".($this->count()+1), $value);
 		return $col;
 	}
 
-	public function compile(JsUtils $js=NULL,&$view=NULL){
-		if($this->_implicite===true){
+	public function compile(JsUtils $js=NULL, &$view=NULL) {
+		if ($this->_implicite===true) {
 			$this->_template="%wrapContentBefore%%content%%wrapContentAfter%";
 		}
-		return parent::compile($js,$view);
+		return parent::compile($js, $view);
 	}
 }
\ No newline at end of file

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

Doc Comments +3 added lines

@@ -18,6 +18,9 @@
 	private $_container;
 	private $_row;
 
+	/**
+	 * @param string $identifier
+	 */
 	public function __construct($identifier) {
 		parent::__construct($identifier, "tr", "");
 		$this->_states=[ State::ACTIVE,State::POSITIVE,State::NEGATIVE,State::WARNING,State::ERROR,State::DISABLED ];

Braces +12 added lines, -8 removed lines

@@ -105,17 +105,20 @@ 
 			$max=\min($colIndex, $rows[$i]->count());
 			for($j=0; $j < $max; $j++) {
 				$rowspan=$rows[$i]->getItem($j)->getRowspan();
-				if ($rowspan + $i > $this->_row)
-					$pos++;
+				if ($rowspan + $i > $this->_row) {
+									$pos++;
+				}
 			}
 		}
-		if ($pos > $colIndex)
-			return NULL;
+		if ($pos > $colIndex) {
+					return NULL;
+		}
 		$count=$this->count();
 		for($i=0; $i < $count; $i++) {
 			$pos+=$this->content[$i]->getColspan();
-			if ($pos >= $colIndex + 1)
-				return $i;
+			if ($pos >= $colIndex + 1) {
+							return $i;
+			}
 		}
 		return null;
 	}
@@ -138,8 +141,9 @@ 
 	public function containsStr($needle) {
 		$cells=$this->content;
 		foreach ( $cells as $cell ) {
-			if (\strpos($cell->getContent(), $needle) !== false)
-				return true;
+			if (\strpos($cell->getContent(), $needle) !== false) {
+							return true;
+			}
 		}
 		return false;
 	}

Spacing +14 added lines, -14 removed lines

@@ -20,12 +20,12 @@ 
 
 	public function __construct($identifier) {
 		parent::__construct($identifier, "tr", "");
-		$this->_states=[ State::ACTIVE,State::POSITIVE,State::NEGATIVE,State::WARNING,State::ERROR,State::DISABLED ];
+		$this->_states=[State::ACTIVE, State::POSITIVE, State::NEGATIVE, State::WARNING, State::ERROR, State::DISABLED];
 	}
 
 	public function setColCount($colCount) {
 		$count=$this->count();
-		for($i=$count; $i < $colCount; $i++) {
+		for ($i=$count; $i<$colCount; $i++) {
 			$item=$this->addItem(NULL);
 			$item->setTagName($this->_tdTagName);
 		}
@@ -68,13 +68,13 @@ 
 		if (!\is_array($values)) {
 			$values=\array_fill(0, $count, $values);
 		} else {
-			if (JArray::isAssociative($values) === true) {
+			if (JArray::isAssociative($values)===true) {
 				$values=\array_values($values);
 			}
 		}
 		$count=\min(\sizeof($values), $count);
 
-		for($i=0; $i < $count; $i++) {
+		for ($i=0; $i<$count; $i++) {
 			$cell=$this->content[$i];
 			$cell->setValue($values[$i]);
 		}
@@ -101,20 +101,20 @@ 
 	public function getColPosition($colIndex) {
 		$pos=0;
 		$rows=$this->_container->getContent();
-		for($i=0; $i < $this->_row; $i++) {
+		for ($i=0; $i<$this->_row; $i++) {
 			$max=\min($colIndex, $rows[$i]->count());
-			for($j=0; $j < $max; $j++) {
+			for ($j=0; $j<$max; $j++) {
 				$rowspan=$rows[$i]->getItem($j)->getRowspan();
-				if ($rowspan + $i > $this->_row)
+				if ($rowspan+$i>$this->_row)
 					$pos++;
 			}
 		}
-		if ($pos > $colIndex)
+		if ($pos>$colIndex)
 			return NULL;
 		$count=$this->count();
-		for($i=0; $i < $count; $i++) {
+		for ($i=0; $i<$count; $i++) {
 			$pos+=$this->content[$i]->getColspan();
-			if ($pos >= $colIndex + 1)
+			if ($pos>=$colIndex+1)
 				return $i;
 		}
 		return null;
@@ -122,7 +122,7 @@ 
 
 	public function conditionalCellFormat($callback, $format) {
 		$cells=$this->content;
-		foreach ( $cells as $cell ) {
+		foreach ($cells as $cell) {
 			$cell->conditionalCellFormat($callback, $format);
 		}
 		return $this;
@@ -137,8 +137,8 @@ 
 
 	public function containsStr($needle) {
 		$cells=$this->content;
-		foreach ( $cells as $cell ) {
-			if (\strpos($cell->getContent(), $needle) !== false)
+		foreach ($cells as $cell) {
+			if (\strpos($cell->getContent(), $needle)!==false)
 				return true;
 		}
 		return false;
@@ -151,7 +151,7 @@ 
 
 	public function applyCells($callback) {
 		$cells=$this->content;
-		foreach ( $cells as $cell ) {
+		foreach ($cells as $cell) {
 			$cell->apply($callback);
 		}
 		return $this;

Ajax/semantic/html/elements/HtmlIconGroups.php

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

Spacing +4 added lines, -4 removed lines

@@ -26,7 +26,7 @@ 
 			$icon=JArray::getValue($value, "icon", 0);
 			$size=JArray::getValue($value, "size", 1);
 		}
-		$iconO=new HtmlIcon("icon-" . $this->identifier, $icon);
+		$iconO=new HtmlIcon("icon-".$this->identifier, $icon);
 		if (isset($size)) {
 			$iconO->setSize($size);
 		}
@@ -34,15 +34,15 @@ 
 	}
 
 	protected function createCondition($value) {
-		return ($value instanceof HtmlIcon) === false;
+		return ($value instanceof HtmlIcon)===false;
 	}
 
 	public function getIcon($index) {
 		return $this->content[$index];
 	}
 
-	public function run(JsUtils $js){
-		$result= parent::run($js);
+	public function run(JsUtils $js) {
+		$result=parent::run($js);
 		return $result->setItemSelector("i");
 	}
 

Ajax/semantic/html/elements/HtmlList.php

Doc Comments +6 added lines

@@ -10,6 +10,9 @@ 
 class HtmlList extends HtmlSemCollection {
 	protected $_hasCheckedList;
 
+	/**
+	 * @param string $identifier
+	 */
 	public function __construct($identifier, $items=array()) {
 		parent::__construct($identifier, "div", "ui list");
 		$this->addItems($items);
@@ -22,6 +25,9 @@ 
 		return $item;
 	}
 
+	/**
+	 * @param integer $niveau
+	 */
 	public function addHeader($niveau, $content) {
 		$header=new HtmlHeader("header-" . $this->identifier, $niveau, $content, "page");
 		$this->wrap($header);

Spacing +14 added lines, -14 removed lines

@@ -18,17 +18,17 @@ 
 
 	protected function createItem($value) {
 		$count=$this->count();
-		$item=new HtmlListItem("item-" . $this->identifier . "-" . $count, $value);
+		$item=new HtmlListItem("item-".$this->identifier."-".$count, $value);
 		return $item;
 	}
 
 	public function addHeader($niveau, $content) {
-		$header=new HtmlHeader("header-" . $this->identifier, $niveau, $content, "page");
+		$header=new HtmlHeader("header-".$this->identifier, $niveau, $content, "page");
 		$this->wrap($header);
 		return $header;
 	}
 
-	public function getItemPart($index,$partName="header"){
+	public function getItemPart($index, $partName="header") {
 		return $this->getItem($index)->getPart($partName);
 	}
 
@@ -37,7 +37,7 @@ 
 	}
 
 	public function asLink() {
-		$this->addToPropertyCtrl("class", "link", array ("link" ));
+		$this->addToPropertyCtrl("class", "link", array("link"));
 		return $this->contentAs("a");
 	}
 
@@ -60,9 +60,9 @@ 
 	}
 
 	public function run(JsUtils $js) {
-		if ($this->_hasCheckedList === true) {
-			$jsCode=include dirname(__FILE__) . '/../../components/jsTemplates/tplCheckedList.php';
-			$jsCode=\str_replace("%identifier%", "#" . $this->identifier, $jsCode);
+		if ($this->_hasCheckedList===true) {
+			$jsCode=include dirname(__FILE__).'/../../components/jsTemplates/tplCheckedList.php';
+			$jsCode=\str_replace("%identifier%", "#".$this->identifier, $jsCode);
 			$this->executeOnRun($jsCode);
 		}
 		return parent::run($js);
@@ -86,24 +86,24 @@ 
 
 	public function addCheckedList($items=array(), $masterItem=NULL, $values=array()) {
 		$count=$this->count();
-		$identifier=$this->identifier . "-" . $count;
+		$identifier=$this->identifier."-".$count;
 		if (isset($masterItem)) {
-			$masterO=new HtmlFormCheckbox("master-" . $identifier, $masterItem);
+			$masterO=new HtmlFormCheckbox("master-".$identifier, $masterItem);
 			$masterO->getHtmlCk()->addToProperty("class", "master");
 			$masterO->setClass("item");
 			$this->addItem($masterO);
 		}
-		$fields=array ();
+		$fields=array();
 		$i=0;
-		foreach ( $items as $val => $caption ) {
-			$itemO=new HtmlFormCheckbox($identifier . "-" . $i++, $caption, $val, "child");
-			if (\array_search($val, $values) !== false) {
+		foreach ($items as $val => $caption) {
+			$itemO=new HtmlFormCheckbox($identifier."-".$i++, $caption, $val, "child");
+			if (\array_search($val, $values)!==false) {
 				$itemO->getField()->setProperty("checked", "");
 			}
 			$itemO->setClass("item");
 			$fields[]=$itemO;
 		}
-		if (isset($masterO) === true) {
+		if (isset($masterO)===true) {
 			$list=new HtmlList("", $fields);
 			$list->setClass("list");
 			$masterO->addContent($list);

Ajax/semantic/html/elements/HtmlStep.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use Ajax\semantic\html\base\HtmlSemCollection;
 use Ajax\semantic\html\content\HtmlStepItem;
 use Ajax\JsUtils;
-
 use Ajax\common\html\HtmlDoubleElement;
 use Ajax\semantic\html\base\constants\Side;
 

Spacing +23 added lines, -23 removed lines

@@ -9,11 +9,11 @@ 
 use Ajax\common\html\HtmlDoubleElement;
 use Ajax\semantic\html\base\constants\Side;
 
-class HtmlStep extends HtmlSemCollection{
+class HtmlStep extends HtmlSemCollection {
 	protected $_activeStep;
 
-	public function __construct( $identifier,$steps=array()){
-		parent::__construct( $identifier,"div", "ui steps");
+	public function __construct($identifier, $steps=array()) {
+		parent::__construct($identifier, "div", "ui steps");
 		$this->addItems($steps);
 	}
 
@@ -23,7 +23,7 @@ 
 	 * @see \Ajax\common\html\html5\HtmlCollection::createItem()
 	 */
 	protected function createItem($value) {
-		$itemO=new HtmlStepItem("item-".\sizeof($this->content),$value);
+		$itemO=new HtmlStepItem("item-".\sizeof($this->content), $value);
 		return $itemO;
 	}
 
@@ -31,50 +31,50 @@ 
 	 * @param string|array $step
 	 * @return HtmlStepItem
 	 */
-	public function addStep($step){
+	public function addStep($step) {
 		return $this->addItem($step);
 	}
 
-	public function setOrdered(){
+	public function setOrdered() {
 		return $this->addToProperty("class", "ordered");
 	}
 
-	public function isOrdered(){
+	public function isOrdered() {
 		return $this->propertyContains("class", "ordered");
 	}
 
-	public function setVertical(){
+	public function setVertical() {
 		return $this->addToProperty("class", "vertical");
 	}
 
-	protected function defineActiveStep(){
+	protected function defineActiveStep() {
 		$activestep=$this->_activeStep;
 		$count=$this->count();
-		if($this->isOrdered()){
-			for($i=0;$i<$count;$i++){
+		if ($this->isOrdered()) {
+			for ($i=0; $i<$count; $i++) {
 				$step=$this->content[$i];
 				$step->removeStatus();
-				if($i<$activestep)
+				if ($i<$activestep)
 					$step->setCompleted();
 				elseif ($i===$activestep)
 					$step->setActive();
 				else
 					$step->setDisabled();
 			}
-		}else{
-			foreach ($this->content as $step){
+		} else {
+			foreach ($this->content as $step) {
 				$step->removeStatus();
 			}
-			if($activestep<$count)
+			if ($activestep<$count)
 				$this->content[$activestep]->setActive();
 		}
 		return $this;
 	}
 
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
-		if(isset($this->_activeStep) && \is_numeric($this->_activeStep))
+		if (isset($this->_activeStep) && \is_numeric($this->_activeStep))
 			$this->defineActiveStep();
-		return parent::compile($js,$view);
+		return parent::compile($js, $view);
 	}
 
 	public function setActiveStep($_activeStep) {
@@ -82,15 +82,15 @@ 
 		return $this;
 	}
 
-	public function setAttached($side="",HtmlDoubleElement $toElement=NULL){
-		if(isset($toElement)){
-			$toElement->addToPropertyCtrl("class", "attached",array("attached"));
+	public function setAttached($side="", HtmlDoubleElement $toElement=NULL) {
+		if (isset($toElement)) {
+			$toElement->addToPropertyCtrl("class", "attached", array("attached"));
 		}
-		return $this->addToPropertyCtrl("class", $side." attached",Side::getConstantValues("attached"));
+		return $this->addToPropertyCtrl("class", $side." attached", Side::getConstantValues("attached"));
 	}
 
-	public function asLink(){
-		foreach ($this->content as $step){
+	public function asLink() {
+		foreach ($this->content as $step) {
 			$step->asLink();
 		}
 		return $this;

Braces +14 added lines, -11 removed lines

@@ -54,26 +54,29 @@
 			for($i=0;$i<$count;$i++){
 				$step=$this->content[$i];
 				$step->removeStatus();
-				if($i<$activestep)
-					$step->setCompleted();
-				elseif ($i===$activestep)
-					$step->setActive();
-				else
-					$step->setDisabled();
+				if($i<$activestep) {
+									$step->setCompleted();
+				} elseif ($i===$activestep) {
+									$step->setActive();
+				} else {
+									$step->setDisabled();
+				}
 			}
-		}else{
+		} else{
 			foreach ($this->content as $step){
 				$step->removeStatus();
 			}
-			if($activestep<$count)
-				$this->content[$activestep]->setActive();
+			if($activestep<$count) {
+							$this->content[$activestep]->setActive();
+			}
 		}
 		return $this;
 	}
 
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
-		if(isset($this->_activeStep) && \is_numeric($this->_activeStep))
-			$this->defineActiveStep();
+		if(isset($this->_activeStep) && \is_numeric($this->_activeStep)) {
+					$this->defineActiveStep();
+		}
 		return parent::compile($js,$view);
 	}
 

Doc Comments +3 added lines

@@ -12,6 +12,9 @@
 class HtmlStep extends HtmlSemCollection{
 	protected $_activeStep;
 
+	/**
+	 * @param string $identifier
+	 */
 	public function __construct( $identifier,$steps=array()){
 		parent::__construct( $identifier,"div", "ui steps");
 		$this->addItems($steps);

Ajax/semantic/html/modules/HtmlAccordion.php

Doc Comments +6 added lines

@@ -10,6 +10,9 @@ 
 
 	protected $params=array();
 
+	/**
+	 * @param string $identifier
+	 */
 	public function __construct( $identifier, $tagName="div", $baseClass="ui"){
 		parent::__construct( $identifier, "div", "ui accordion");
 	}
@@ -29,6 +32,9 @@ 
 		return ($value instanceof HtmlAccordionItem)===false;
 	}
 
+	/**
+	 * @param string $title
+	 */
 	public function addPanel($title,$content){
 		return $this->addItem([$title,$content]);
 	}

Spacing +19 added lines, -19 removed lines

@@ -6,31 +6,31 @@ 
 use Ajax\semantic\html\content\HtmlAccordionItem;
 use Ajax\JsUtils;
 
-class HtmlAccordion extends HtmlSemCollection{
+class HtmlAccordion extends HtmlSemCollection {
 
 	protected $params=array();
 
-	public function __construct( $identifier, $tagName="div", $baseClass="ui"){
-		parent::__construct( $identifier, "div", "ui accordion");
+	public function __construct($identifier, $tagName="div", $baseClass="ui") {
+		parent::__construct($identifier, "div", "ui accordion");
 	}
 
 
-	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];
 		}
-		return new HtmlAccordionItem("item-".$this->identifier."-".$count, $title,$content);
+		return new HtmlAccordionItem("item-".$this->identifier."-".$count, $title, $content);
 	}
 
-	protected function createCondition($value){
+	protected function createCondition($value) {
 		return ($value instanceof HtmlAccordionItem)===false;
 	}
 
-	public function addPanel($title,$content){
-		return $this->addItem([$title,$content]);
+	public function addPanel($title, $content) {
+		return $this->addItem([$title, $content]);
 	}
 
 	/**
@@ -42,8 +42,8 @@ 
 	 * @param string $action a Phalcon action
 	 * @param array $params
 	 */
-	public function forwardPanel(JsUtils $js,$title,$initialController,$controller,$action,$params=array()){
-		return $this->addPanel($title, $js->forward($initialController, $controller, $action,$params));
+	public function forwardPanel(JsUtils $js, $title, $initialController, $controller, $action, $params=array()) {
+		return $this->addPanel($title, $js->forward($initialController, $controller, $action, $params));
 	}
 
 	/**
@@ -54,30 +54,30 @@ 
 	 * @param string $viewName
 	 * @param $params The parameters to pass to the view
 	 */
-	public function renderViewPanel(JsUtils $js,$title,$initialController, $viewName, $params=array()) {
-		return $this->addPanel($title, $js->renderContent($initialController, $viewName,$params));
+	public function renderViewPanel(JsUtils $js, $title, $initialController, $viewName, $params=array()) {
+		return $this->addPanel($title, $js->renderContent($initialController, $viewName, $params));
 	}
 	/*
 	 * (non-PHPdoc)
 	 * @see BaseHtml::run()
 	 */
 	public function run(JsUtils $js) {
-		if(isset($this->_bsComponent)===false)
-			$this->_bsComponent=$js->semantic()->accordion("#".$this->identifier,$this->params);
+		if (isset($this->_bsComponent)===false)
+			$this->_bsComponent=$js->semantic()->accordion("#".$this->identifier, $this->params);
 			$this->addEventsOnRun($js);
 			return $this->_bsComponent;
 	}
 
-	public function setStyled(){
+	public function setStyled() {
 		return $this->addToProperty("class", "styled");
 	}
 
-	public function activate($index){
+	public function activate($index) {
 		$this->getItem($index)->setActive(true);
 		return $this;
 	}
 
-	public function setExclusive($value){
+	public function setExclusive($value) {
 		$this->params["exclusive"]=$value;
 	}
 }
\ No newline at end of file

Braces +3 added lines, -2 removed lines

@@ -34,8 +34,9 @@
 	 * @see BaseHtml::run()
 	 */
 	public function run(JsUtils $js) {
-		if(isset($this->_bsComponent)===false)
-			$this->_bsComponent=$js->semantic()->accordion("#".$this->identifier,$this->params);
+		if(isset($this->_bsComponent)===false) {
+					$this->_bsComponent=$js->semantic()->accordion("#".$this->identifier,$this->params);
+		}
 			$this->addEventsOnRun($js);
 			return $this->_bsComponent;
 	}

Ajax/ui/properties/Animation.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
 	 * @param string $value default : swing
 	 */
 	public function setEasing($value) {
-		$this->setParamCtrl("easing", $value, array (
+		$this->setParamCtrl("easing", $value, array(
 				"linear",
 				"swing",
 				"easeInQuad",