phpMv / phpMv-UI

Ajax/semantic/html/elements/HtmlLabel.php

Indentation +6 added lines, -6 removed lines

@@ -91,12 +91,12 @@
 	}
 
 	/**
-	*  Adds an image
-	* @param string $src
-	* @param string $alt
-	* @param boolean $before
-	* @return \Ajax\semantic\html\elements\html5\HtmlImg
-	*/
+	 *  Adds an image
+	 * @param string $src
+	 * @param string $alt
+	 * @param boolean $before
+	 * @return \Ajax\semantic\html\elements\html5\HtmlImg
+	 */
 	public function addImage($src, $alt="", $before=true) {
 		$img=new HtmlImg("image-" . $this->identifier, $src, $alt);
 		$img->setClass("");

Braces +13 added lines, -10 removed lines

@@ -14,8 +14,9 @@ 
 	public function __construct($identifier, $caption="", $icon=NULL, $tagName="div") {
 		parent::__construct($identifier, $tagName, "ui label");
 		$this->content=$caption;
-		if (isset($icon))
-			$this->addIcon($icon);
+		if (isset($icon)) {
+					$this->addIcon($icon);
+		}
 	}
 
 	/**
@@ -24,10 +25,11 @@ 
 	 * @return \Ajax\semantic\html\elements\HtmlLabel
 	 */
 	public function setPointing($value=Direction::NONE) {
-		if($value==="left" || $value==="right")
-			return $this->addToPropertyCtrl("class", $value." pointing", Direction::getConstantValues("pointing"));
-		else
-			return $this->addToPropertyCtrl("class", "pointing ".$value, Direction::getConstantValues("pointing",true));
+		if($value==="left" || $value==="right") {
+					return $this->addToPropertyCtrl("class", $value." pointing", Direction::getConstantValues("pointing"));
+		} else {
+					return $this->addToPropertyCtrl("class", "pointing ".$value, Direction::getConstantValues("pointing",true));
+		}
 	}
 
 	/**
@@ -121,10 +123,11 @@ 
 	}
 
 	public function setAttached($side=Side::TOP,$direction=Direction::NONE){
-		if($direction!==Direction::NONE)
-			return $this->addToPropertyCtrl("class", $side." ".$direction." attached",Side::getConstantValues($direction." attached"));
-		else
-			return $this->addToPropertyCtrl("class", $side." attached",Side::getConstantValues("attached"));
+		if($direction!==Direction::NONE) {
+					return $this->addToPropertyCtrl("class", $side." ".$direction." attached",Side::getConstantValues($direction." attached"));
+		} else {
+					return $this->addToPropertyCtrl("class", $side." attached",Side::getConstantValues("attached"));
+		}
 	}
 
 	public static function ribbon($identifier, $caption) {

Spacing +19 added lines, -19 removed lines

@@ -10,7 +10,7 @@ 
 use Ajax\semantic\html\base\traits\HasTimeoutTrait;
 
 class HtmlLabel extends HtmlSemDoubleElement {
-	use LabeledIconTrait,HasTimeoutTrait;
+	use LabeledIconTrait, HasTimeoutTrait;
 
 	public function __construct($identifier, $caption="", $icon=NULL, $tagName="div") {
 		parent::__construct($identifier, $tagName, "ui label");
@@ -25,10 +25,10 @@ 
 	 * @return \Ajax\semantic\html\elements\HtmlLabel
 	 */
 	public function setPointing($value=Direction::NONE) {
-		if($value==="left" || $value==="right")
+		if ($value==="left" || $value==="right")
 			return $this->addToPropertyCtrl("class", $value." pointing", Direction::getConstantValues("pointing"));
 		else
-			return $this->addToPropertyCtrl("class", "pointing ".$value, Direction::getConstantValues("pointing",true));
+			return $this->addToPropertyCtrl("class", "pointing ".$value, Direction::getConstantValues("pointing", true));
 	}
 
 	/**
@@ -37,15 +37,15 @@ 
 	 * @return \Ajax\semantic\html\elements\HtmlLabel
 	 */
 	public function toCorner($side="left") {
-		return $this->addToPropertyCtrl("class", $side . " corner", array ("right corner","left corner" ));
+		return $this->addToPropertyCtrl("class", $side." corner", array("right corner", "left corner"));
 	}
 
-	public function setHorizontal(){
-		return $this->addToPropertyCtrl("class", "hozizontal",array("horizontal"));
+	public function setHorizontal() {
+		return $this->addToPropertyCtrl("class", "hozizontal", array("horizontal"));
 	}
 
-	public function setFloating(){
-		return $this->addToPropertyCtrl("class", "floating",array("floating"));
+	public function setFloating() {
+		return $this->addToPropertyCtrl("class", "floating", array("floating"));
 	}
 
 	/**
@@ -56,9 +56,9 @@ 
 		return $this->addToProperty("class", "tag");
 	}
 
-	public function setEmpty(){
+	public function setEmpty() {
 		$this->content=NULL;
-		return $this->addToPropertyCtrl("class", "empty",array("empty"));
+		return $this->addToPropertyCtrl("class", "empty", array("empty"));
 	}
 
 	public function setBasic() {
@@ -74,7 +74,7 @@ 
 	 */
 	public function addEmphasisImage($src, $alt="", $before=true) {
 		$this->addToProperty("class", "image");
-		return $this->addImage($src,$alt,$before);
+		return $this->addImage($src, $alt, $before);
 	}
 
 	/**
@@ -85,7 +85,7 @@ 
 	 * @return \Ajax\semantic\html\elements\html5\HtmlImg
 	 */
 	public function addAvatarImage($src, $alt="", $before=true) {
-		$img=$this->addImage($src,$alt,$before);
+		$img=$this->addImage($src, $alt, $before);
 		$img->setClass("ui image");
 		$img->asAvatar();
 		return $img;
@@ -99,7 +99,7 @@ 
 	* @return \Ajax\semantic\html\elements\html5\HtmlImg
 	*/
 	public function addImage($src, $alt="", $before=true) {
-		$img=new HtmlImg("image-" . $this->identifier, $src, $alt);
+		$img=new HtmlImg("image-".$this->identifier, $src, $alt);
 		$img->setClass("");
 		$this->addContent($img, $before);
 		return $img;
@@ -111,21 +111,21 @@ 
 	 * @return \Ajax\common\html\HtmlDoubleElement
 	 */
 	public function addDetail($detail) {
-		$div=new HtmlSemDoubleElement("detail-" . $this->identifier, $this->tagName,"detail");
+		$div=new HtmlSemDoubleElement("detail-".$this->identifier, $this->tagName, "detail");
 		$div->setContent($detail);
 		$this->addContent($div);
 		return $div;
 	}
 
 	public function asRibbon($direction=Direction::NONE) {
-		return $this->addToPropertyCtrl("class", $direction." ribbon", array ("ribbon","right ribbon","left ribbon" ));
+		return $this->addToPropertyCtrl("class", $direction." ribbon", array("ribbon", "right ribbon", "left ribbon"));
 	}
 
-	public function setAttached($side=Side::TOP,$direction=Direction::NONE){
-		if($direction!==Direction::NONE)
-			return $this->addToPropertyCtrl("class", $side." ".$direction." attached",Side::getConstantValues($direction." attached"));
+	public function setAttached($side=Side::TOP, $direction=Direction::NONE) {
+		if ($direction!==Direction::NONE)
+			return $this->addToPropertyCtrl("class", $side." ".$direction." attached", Side::getConstantValues($direction." attached"));
 		else
-			return $this->addToPropertyCtrl("class", $side." attached",Side::getConstantValues("attached"));
+			return $this->addToPropertyCtrl("class", $side." attached", Side::getConstantValues("attached"));
 	}
 
 	public static function ribbon($identifier, $caption) {

Ajax/semantic/html/elements/HtmlReveal.php

Spacing +22 added lines, -22 removed lines

@@ -10,64 +10,64 @@
 
 class HtmlReveal extends HtmlSemDoubleElement {
 
-	public function __construct($identifier, $visibleContent,$hiddenContent,$type=RevealType::FADE,$attributeType=NULL) {
+	public function __construct($identifier, $visibleContent, $hiddenContent, $type=RevealType::FADE, $attributeType=NULL) {
 		parent::__construct($identifier, "div", "ui reveal");
 		$this->setElement(0, $visibleContent);
 		$this->setElement(1, $hiddenContent);
-		$this->setType($type,$attributeType);
+		$this->setType($type, $attributeType);
 	}
 
-	private function setElement($index,$content){
-		if(!$content instanceof HtmlSingleElement){
-			$content=new HtmlLabel("",$content);
+	private function setElement($index, $content) {
+		if (!$content instanceof HtmlSingleElement) {
+			$content=new HtmlLabel("", $content);
 		}
-		if($content instanceof HtmlSemDoubleElement){
-			$content=new HtmlSemDoubleElement($this->identifier."-".$index,"div","",$content);
-		}elseif ($content instanceof HtmlImg){
+		if ($content instanceof HtmlSemDoubleElement) {
+			$content=new HtmlSemDoubleElement($this->identifier."-".$index, "div", "", $content);
+		}elseif ($content instanceof HtmlImg) {
 			$this->addToPropertyCtrl("class", "image", array("image"));
 		}
-		$content->addToProperty("class",(($index===0)?"visible":"hidden")." content");
+		$content->addToProperty("class", (($index===0) ? "visible" : "hidden")." content");
 		$this->content[$index]=$content;
 		return $this;
 	}
 
-	public function setVisibleContent($visibleContent){
+	public function setVisibleContent($visibleContent) {
 		return $this->setElement(0, $visibleContent);
 	}
 
-	public function setHiddenContent($hiddenContent){
+	public function setHiddenContent($hiddenContent) {
 		return $this->setElement(1, $hiddenContent);
 	}
 
-	public function getVisibleContent(){
+	public function getVisibleContent() {
 		return $this->content[0];
 	}
 
-	public function getHiddenContent(){
+	public function getHiddenContent() {
 		return $this->content[1];
 	}
 
-	public function setType($type,$attribute=NULL){
+	public function setType($type, $attribute=NULL) {
 		$this->addToPropertyCtrl("class", $type, RevealType::getConstants());
-		if(isset($attribute)){
+		if (isset($attribute)) {
 			$this->addToPropertyCtrl("class", $attribute, Direction::getConstants());
 		}
 		return $this;
 	}
 
-	public function setFade($attribute=NULL){
-		return $this->setType(RevealType::FADE,$attribute);
+	public function setFade($attribute=NULL) {
+		return $this->setType(RevealType::FADE, $attribute);
 	}
 
-	public function setMove($attribute=NULL){
-		return $this->setType(RevealType::MOVE,$attribute);
+	public function setMove($attribute=NULL) {
+		return $this->setType(RevealType::MOVE, $attribute);
 	}
 
-	public function setRotate($attribute=NULL){
-		return $this->setType(RevealType::ROTATE,$attribute);
+	public function setRotate($attribute=NULL) {
+		return $this->setType(RevealType::ROTATE, $attribute);
 	}
 
-	public function setCircular(){
+	public function setCircular() {
 		return $this->addToProperty("class", "circular");
 	}
 

Braces +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
 		}
 		if($content instanceof HtmlSemDoubleElement){
 			$content=new HtmlSemDoubleElement($this->identifier."-".$index,"div","",$content);
-		}elseif ($content instanceof HtmlImg){
+		} elseif ($content instanceof HtmlImg){
 			$this->addToPropertyCtrl("class", "image", array("image"));
 		}
 		$content->addToProperty("class",(($index===0)?"visible":"hidden")." content");

Ajax/semantic/html/elements/HtmlContainer.php

Spacing +2 added lines, -2 removed lines

@@ -13,11 +13,11 @@
 class HtmlContainer extends HtmlSemDoubleElement {
 	use TextAlignmentTrait;
 	public function __construct($identifier, $content="") {
-		parent::__construct($identifier, "div","ui container");
+		parent::__construct($identifier, "div", "ui container");
 		$this->content=$content;
 	}
 
-	public function asText(){
+	public function asText() {
 		return $this->addToProperty("class", "text");
 	}
 }
\ No newline at end of file

Ajax/semantic/html/elements/HtmlHeader.php

Braces +3 added lines, -2 removed lines

@@ -19,8 +19,9 @@
 		if (isset($type)) {
 			if ($type == "page") {
 				$this->asPageHeader($niveau);
-			} else
-				$this->asContentHeader($niveau);
+			} else {
+							$this->asContentHeader($niveau);
+			}
 		}
 		$this->content=$content;
 	}

Spacing +7 added lines, -7 removed lines

@@ -10,14 +10,14 @@ 
 use Ajax\semantic\html\base\traits\AttachedTrait;
 
 class HtmlHeader extends HtmlSemDoubleElement {
-	use TextAlignmentTrait,AttachedTrait;
+	use TextAlignmentTrait, AttachedTrait;
 	protected $image;
 
 	public function __construct($identifier, $niveau=1, $content=NULL, $type="page") {
 		parent::__construct($identifier, "div", "ui header");
 		$this->_template="<%tagName% %properties%>%image%%wrapContentBefore%%content%%wrapContentAfter%</%tagName%>";
 		if (isset($type)) {
-			if ($type == "page") {
+			if ($type=="page") {
 				$this->asPageHeader($niveau);
 			} else
 				$this->asContentHeader($niveau);
@@ -26,7 +26,7 @@ 
 	}
 
 	public function asPageHeader($niveau) {
-		$this->tagName="h" . $niveau;
+		$this->tagName="h".$niveau;
 	}
 
 	public function asContentHeader($niveau) {
@@ -39,25 +39,25 @@ 
 
 	public function asIcon($icon, $title, $subHeader=NULL) {
 		$this->addToProperty("class", "icon");
-		$this->image=new HtmlIcon("icon-" . $this->identifier, $icon);
+		$this->image=new HtmlIcon("icon-".$this->identifier, $icon);
 		return $this->asTitle($title, $subHeader);
 	}
 
 	public function asImage($src, $title, $subHeader=NULL) {
-		$this->image=new HtmlImg("img-" . $this->identifier, $src, $title);
+		$this->image=new HtmlImg("img-".$this->identifier, $src, $title);
 		return $this->asTitle($title, $subHeader);
 	}
 
 	public function asTitle($title, $subHeader=NULL) {
 		if (!\is_object($title)) {
-			$this->content=new HtmlDoubleElement("content-" . $this->identifier, "div");
+			$this->content=new HtmlDoubleElement("content-".$this->identifier, "div");
 			$this->content->setContent($title);
 		} else {
 			$this->content=$title;
 		}
 		$this->content->setClass("content");
 		if (isset($subHeader)) {
-			$sub=new HtmlDoubleElement("subheader-" . $this->identifier, "div");
+			$sub=new HtmlDoubleElement("subheader-".$this->identifier, "div");
 			$sub->setClass("sub header");
 			$sub->setContent($subHeader);
 			$this->content->addContent($sub);

Ajax/semantic/html/content/HtmlAccordionMenuItem.php

Spacing +3 added lines, -3 removed lines

@@ -4,11 +4,11 @@
 
 
 class HtmlAccordionMenuItem extends HtmlMenuItem {
-	public function __construct($identifier,$title,$content) {
-		parent::__construct($identifier, new HtmlAccordionItem("accordion-".$identifier, $title,$content));
+	public function __construct($identifier, $title, $content) {
+		parent::__construct($identifier, new HtmlAccordionItem("accordion-".$identifier, $title, $content));
 	}
 
-	public function setActive($value=true){
+	public function setActive($value=true) {
 		$this->content->setActive($value);
 		return $this;
 	}

Doc Comments +4 added lines

@@ -4,6 +4,10 @@
 
 
 class HtmlAccordionMenuItem extends HtmlMenuItem {
+
+	/**
+	 * @param string $identifier
+	 */
 	public function __construct($identifier,$title,$content) {
 		parent::__construct($identifier, new HtmlAccordionItem("accordion-".$identifier, $title,$content));
 	}

Ajax/semantic/html/content/HtmlAccordionItem.php

Spacing +10 added lines, -10 removed lines

@@ -20,30 +20,30 @@
 		$this->_title=$title;
 	}
 
-	public function setTitle($title){
+	public function setTitle($title) {
 		$this->_title=$title;
 	}
 
-	public function setIcon($icon){
+	public function setIcon($icon) {
 		$this->_icon=$icon;
 	}
 
-	protected function createTitleElement(){
-		$element=new HtmlSemDoubleElement("title-".$this->identifier,"div","title");
-		$element->setContent(array(new HtmlIcon("", $this->_icon),$this->_title));
-		if($this->_active===true)
+	protected function createTitleElement() {
+		$element=new HtmlSemDoubleElement("title-".$this->identifier, "div", "title");
+		$element->setContent(array(new HtmlIcon("", $this->_icon), $this->_title));
+		if ($this->_active===true)
 			$element->addToProperty("class", "active");
 		return $element;
 	}
 
-	public function compile(JsUtils $js=NULL, &$view=NULL){
+	public function compile(JsUtils $js=NULL, &$view=NULL) {
 		$this->titleElement=$this->createTitleElement();
-		return parent::compile($js,$view);
+		return parent::compile($js, $view);
 	}
 
-	public function setActive($value=true){
+	public function setActive($value=true) {
 		$this->_active=$value;
-		if($value===true)
+		if ($value===true)
 			$this->addToPropertyCtrl("class", "active", array("active"));
 		else
 			$this->removePropertyValue("class", "active");

Braces +8 added lines, -6 removed lines

@@ -31,8 +31,9 @@ 
 	protected function createTitleElement(){
 		$element=new HtmlSemDoubleElement("title-".$this->identifier,"div","title");
 		$element->setContent(array(new HtmlIcon("", $this->_icon),$this->_title));
-		if($this->_active===true)
-			$element->addToProperty("class", "active");
+		if($this->_active===true) {
+					$element->addToProperty("class", "active");
+		}
 		return $element;
 	}
 
@@ -43,10 +44,11 @@ 
 
 	public function setActive($value=true){
 		$this->_active=$value;
-		if($value===true)
-			$this->addToPropertyCtrl("class", "active", array("active"));
-		else
-			$this->removePropertyValue("class", "active");
+		if($value===true) {
+					$this->addToPropertyCtrl("class", "active", array("active"));
+		} else {
+					$this->removePropertyValue("class", "active");
+		}
 		return $this;
 	}
 }
\ No newline at end of file

Doc Comments +3 added lines

@@ -14,6 +14,9 @@
 	protected $_title;
 	protected $_active;
 
+	/**
+	 * @param string $identifier
+	 */
 	public function __construct($identifier, $title, $content=NULL) {
 		parent::__construct($identifier, "div", "content", $content);
 		$this->_template="%titleElement%".$this->_template;

Ajax/semantic/html/content/card/HtmlCardContent.php

Spacing +8 added lines, -8 removed lines

@@ -15,16 +15,16 @@ 
 
 	private function addElement($content, $baseClass) {
 		$count=\sizeof($this->content);
-		$result=new HtmlSemDoubleElement("element-" . $count . "-" . $this->identifier, "div", $baseClass, $content);
+		$result=new HtmlSemDoubleElement("element-".$count."-".$this->identifier, "div", $baseClass, $content);
 		$this->addContent($result);
 		return $result;
 	}
 
 	public function addMeta($value, $direction=Direction::LEFT) {
-		if (\array_key_exists("meta", $this->content) === false) {
-			$this->content["meta"]=new HtmlSemDoubleElement("meta-" . $this->identifier, "div", "meta", array ());
+		if (\array_key_exists("meta", $this->content)===false) {
+			$this->content["meta"]=new HtmlSemDoubleElement("meta-".$this->identifier, "div", "meta", array());
 		}
-		if ($direction === Direction::RIGHT) {
+		if ($direction===Direction::RIGHT) {
 			$value=new HtmlSemDoubleElement("", "span", "", $value);
 			$value->setFloated($direction);
 		}
@@ -42,7 +42,7 @@ 
 
 	public function addMetas($metas) {
 		if (\is_array($metas)) {
-			foreach ( $metas as $meta ) {
+			foreach ($metas as $meta) {
 				$this->addMeta($meta);
 			}
 		} else
@@ -51,7 +51,7 @@ 
 	}
 
 	public function addContentIcon($icon, $caption=NULL, $direction=Direction::LEFT) {
-		if ($direction === Direction::RIGHT) {
+		if ($direction===Direction::RIGHT) {
 			if (isset($caption)) {
 				$result=new HtmlSemDoubleElement("", "span", "", $caption);
 				$result->addIcon($icon);
@@ -69,7 +69,7 @@ 
 	}
 
 	public function addContentText($caption, $direction=Direction::LEFT) {
-		if ($direction === Direction::RIGHT) {
+		if ($direction===Direction::RIGHT) {
 			$result=new HtmlSemDoubleElement("", "span", "", $caption);
 			$this->addContent($result);
 			$result->setFloated($direction);
@@ -79,7 +79,7 @@ 
 	}
 
 	public function addContentIcons($icons, $direction=Direction::LEFT) {
-		foreach ( $icons as $icon ) {
+		foreach ($icons as $icon) {
 			$this->addContentIcon($icon, NULL, $direction);
 		}
 		return $this;

Braces +9 added lines, -6 removed lines

@@ -34,8 +34,9 @@ 
 
 	public function addImage($src="", $alt="", $size=NULL) {
 		$image=new HtmlImg("img-", $src, $alt);
-		if (isset($size))
-			$image->setSize($size);
+		if (isset($size)) {
+					$image->setSize($size);
+		}
 		$this->content["image"]=$image;
 		return $image;
 	}
@@ -45,8 +46,9 @@ 
 			foreach ( $metas as $meta ) {
 				$this->addMeta($meta);
 			}
-		} else
-			$this->addMeta($metas);
+		} else {
+					$this->addMeta($metas);
+		}
 		return $this;
 	}
 
@@ -73,8 +75,9 @@ 
 			$result=new HtmlSemDoubleElement("", "span", "", $caption);
 			$this->addContent($result);
 			$result->setFloated($direction);
-		} else
-			$result=$this->addContent($caption);
+		} else {
+					$result=$this->addContent($caption);
+		}
 		return $result;
 	}
 

Ajax/semantic/html/content/card/HtmlCardHeaderContent.php

Spacing +4 added lines, -4 removed lines

@@ -10,7 +10,7 @@ 
 class HtmlCardHeaderContent extends HtmlCardContent {
 
 	public function __construct($identifier, $header=NULL, $metas=array(), $description=NULL) {
-		parent::__construct($identifier, array ());
+		parent::__construct($identifier, array());
 		if (isset($header)) {
 			$this->setHeader($header);
 		}
@@ -21,11 +21,11 @@ 
 	}
 
 	public function setDescription($value) {
-		$this->content["description"]=new HtmlSemDoubleElement("description-" . $this->identifier, "div", "description", $value);
+		$this->content["description"]=new HtmlSemDoubleElement("description-".$this->identifier, "div", "description", $value);
 	}
 
 	public function setHeader($value) {
-		$this->content["header"]=new HtmlSemDoubleElement("header-" . $this->identifier, "a", "header", $value);
+		$this->content["header"]=new HtmlSemDoubleElement("header-".$this->identifier, "a", "header", $value);
 	}
 
 	/**
@@ -35,7 +35,7 @@ 
 	 * @see \Ajax\semantic\html\base\HtmlSemDoubleElement::compile()
 	 */
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
-		$this->content=JArray::sortAssociative($this->content, [ "image","header","meta","description" ]);
+		$this->content=JArray::sortAssociative($this->content, ["image", "header", "meta", "description"]);
 		return parent::compile($js, $view);
 	}
 }
\ No newline at end of file

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

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

Doc Comments +4 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 ];
@@ -78,6 +81,7 @@ 
 	/**
 	 * Sets or adds values to the row cols
 	 * @param mixed $values
+	 * @param \Closure $callback
 	 */
 	protected function _addOrSetValues($values,$callback) {
 		$count=$this->count();

Spacing +19 added lines, -19 removed lines

@@ -20,19 +20,19 @@ 
 
 	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);
 		}
 		return $this;
 	}
 
-	public function getColCount(){
+	public function getColCount() {
 		return $this->count();
 	}
 
@@ -68,7 +68,7 @@ 
 	 * @param mixed $values
 	 */
 	public function setValues($values=array()) {
-		return $this->_addOrSetValues($values, function(&$cell,$value){$cell->setValue($value);});
+		return $this->_addOrSetValues($values, function(&$cell, $value) {$cell->setValue($value); });
 	}
 
 	/**
@@ -76,27 +76,27 @@ 
 	 * @param mixed $values
 	 */
 	public function addValues($values=array()) {
-		return $this->_addOrSetValues($values, function(&$cell,$value){$cell->addValue($value);});
+		return $this->_addOrSetValues($values, function(&$cell, $value) {$cell->addValue($value); });
 	}
 
 	/**
 	 * Sets or adds values to the row cols
 	 * @param mixed $values
 	 */
-	protected function _addOrSetValues($values,$callback) {
+	protected function _addOrSetValues($values, $callback) {
 		$count=$this->count();
 		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];
-			$callback($cell,$values[$i]);
+			$callback($cell, $values[$i]);
 		}
 		return $this;
 	}
@@ -122,20 +122,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;
@@ -143,7 +143,7 @@ 
 
 	public function conditionalCellFormat($callback, $format) {
 		$cells=$this->content;
-		foreach ( $cells as $cell ) {
+		foreach ($cells as $cell) {
 			$cell->conditionalCellFormat($callback, $format);
 		}
 		return $this;
@@ -158,8 +158,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;
@@ -172,7 +172,7 @@ 
 
 	public function applyCells($callback) {
 		$cells=$this->content;
-		foreach ( $cells as $cell ) {
+		foreach ($cells as $cell) {
 			$cell->apply($callback);
 		}
 		return $this;