jcheron / phalcon-jquery

Ajax/semantic/html/modules/checkbox/AbstractCheckbox.php

Spacing +4 added lines, -4 removed lines

@@ -6,7 +6,7 @@ 
 use Ajax\semantic\html\base\constants\CheckboxType;
 
 abstract class AbstractCheckbox extends HtmlSemDoubleElement {
-	protected $_params=array ();
+	protected $_params=array();
 
 	public function __construct($identifier, $name=NULL, $label=NULL, $value=NULL, $inputType="checkbox", $type="checkbox") {
 		parent::__construct("ck-".$identifier, "div", "ui ".$type);
@@ -73,9 +73,9 @@ 
 	 * @return \Ajax\semantic\html\collections\form\AbstractHtmlFormRadioCheckbox
 	 */
 	public function attachEvent($selector, $action=NULL) {
-		if (isset($action)!==false||\is_numeric($action)===true) {
+		if (isset($action)!==false || \is_numeric($action)===true) {
 			$js='$("#%identifier%").checkbox("attach events", "'.$selector.'", "'.$action.'");';
-		} else {
+		}else {
 			$js='$("#%identifier%").checkbox("attach events", "'.$selector.'");';
 		}
 		$js=\str_replace("%identifier%", $this->identifier, $js);
@@ -89,7 +89,7 @@ 
 	 */
 	public function attachEvents($events=array()) {
 		if (\is_array($events)) {
-			foreach ( $events as $action => $selector ) {
+			foreach ($events as $action => $selector) {
 				$this->attachEvent($selector, $action);
 			}
 		}

Braces +6 added lines, -4 removed lines

@@ -13,8 +13,9 @@ 
 		$field=new \Ajax\common\html\html5\HtmlInput($identifier, $inputType, $value);
 		$field->setProperty("name", $name);
 		$this->setField($field);
-		if (isset($label))
-			$this->setLabel($label);
+		if (isset($label)) {
+					$this->setLabel($label);
+		}
 	}
 
 	public function setType($checkboxType) {
@@ -40,8 +41,9 @@ 
 	 * @return mixed
 	 */
 	public function getLabel() {
-		if (\array_key_exists("label", $this->content))
-			return $this->content["label"];
+		if (\array_key_exists("label", $this->content)) {
+					return $this->content["label"];
+		}
 	}
 
 	/**

Ajax/semantic/html/content/HtmlGridCol.php

Spacing +2 added lines, -2 removed lines

@@ -30,10 +30,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", "wide", array ("wide" ));
+		return $this->addToPropertyCtrl("class", "wide", array("wide"));
 	}
 
 	public function setValue($value) {

Braces +8 added lines, -6 removed lines

@@ -20,8 +20,9 @@ 
 	public function __construct($identifier, $width=NULL) {
 		parent::__construct($identifier, "div");
 		$this->setClass("column");
-		if (isset($width))
-			$this->setWidth($width);
+		if (isset($width)) {
+					$this->setWidth($width);
+		}
 	}
 
 	/**
@@ -48,10 +49,11 @@ 
 
 	public function addDivider($vertical=true, $content=NULL) {
 		$divider=new HtmlDivider("", $content);
-		if ($vertical)
-			$divider->setVertical();
-		else
-			$divider->setHorizontal();
+		if ($vertical) {
+					$divider->setVertical();
+		} else {
+					$divider->setHorizontal();
+		}
 		$this->wrap("", $divider);
 		return $divider;
 	}

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

Doc Comments +12 added lines

@@ -11,12 +11,24 @@
 	protected $_states=[ ];
 	protected $_baseClass;
 
+	/**
+	 * @param string $name
+	 */
 	protected abstract function setPropertyCtrl($name, $value, $typeCtrl);
 
+	/**
+	 * @param string $name
+	 */
 	protected abstract function addToPropertyCtrl($name, $value, $typeCtrl);
 
+	/**
+	 * @param string $name
+	 */
 	protected abstract function addToPropertyCtrlCheck($name, $value, $typeCtrl);
 
+	/**
+	 * @param string $name
+	 */
 	public abstract function addToProperty($name, $value, $separator=" ");
 
 	public function addVariation($variation) {

Braces +12 added lines, -8 removed lines

@@ -34,8 +34,9 @@ 
 
 	public function setVariations($variations) {
 		$this->setProperty("class", $this->_baseClass);
-		if (\is_string($variations))
-			$variations=\explode(" ", $variations);
+		if (\is_string($variations)) {
+					$variations=\explode(" ", $variations);
+		}
 		foreach ( $variations as $variation ) {
 			$this->addVariation($variation);
 		}
@@ -48,8 +49,9 @@ 
 	}
 
 	public function addVariations($variations=array()) {
-		if (\is_string($variations))
-			$variations=\explode(" ", $variations);
+		if (\is_string($variations)) {
+					$variations=\explode(" ", $variations);
+		}
 		foreach ( $variations as $variation ) {
 			$this->addVariation($variation);
 		}
@@ -57,8 +59,9 @@ 
 	}
 
 	public function addStates($states=array()) {
-		if (\is_string($states))
-			$states=\explode(" ", $states);
+		if (\is_string($states)) {
+					$states=\explode(" ", $states);
+		}
 		foreach ( $states as $state ) {
 			$this->addState($state);
 		}
@@ -67,8 +70,9 @@ 
 
 	public function setStates($states) {
 		$this->setProperty("class", $this->_baseClass);
-		if (\is_string($states))
-			$states=\explode(" ", $states);
+		if (\is_string($states)) {
+					$states=\explode(" ", $states);
+		}
 		foreach ( $states as $state ) {
 			$this->addState($state);
 		}

Spacing +8 added lines, -8 removed lines

@@ -8,8 +8,8 @@ 
 use Ajax\semantic\html\elements\HtmlIcon;
 
 trait BaseTrait {
-	protected $_variations=[ ];
-	protected $_states=[ ];
+	protected $_variations=[];
+	protected $_states=[];
 	protected $_baseClass;
 
 	protected abstract function setPropertyCtrl($name, $value, $typeCtrl);
@@ -37,7 +37,7 @@ 
 		$this->setProperty("class", $this->_baseClass);
 		if (\is_string($variations))
 			$variations=\explode(" ", $variations);
-		foreach ( $variations as $variation ) {
+		foreach ($variations as $variation) {
 			$this->addVariation($variation);
 		}
 		return $this;
@@ -51,7 +51,7 @@ 
 	public function addVariations($variations=array()) {
 		if (\is_string($variations))
 			$variations=\explode(" ", $variations);
-		foreach ( $variations as $variation ) {
+		foreach ($variations as $variation) {
 			$this->addVariation($variation);
 		}
 		return $this;
@@ -60,7 +60,7 @@ 
 	public function addStates($states=array()) {
 		if (\is_string($states))
 			$states=\explode(" ", $states);
-		foreach ( $states as $state ) {
+		foreach ($states as $state) {
 			$this->addState($state);
 		}
 		return $this;
@@ -70,14 +70,14 @@ 
 		$this->setProperty("class", $this->_baseClass);
 		if (\is_string($states))
 			$states=\explode(" ", $states);
-		foreach ( $states as $state ) {
+		foreach ($states as $state) {
 			$this->addState($state);
 		}
 		return $this;
 	}
 
 	public function addIcon($icon, $before=true) {
-		return $this->addContent(new HtmlIcon("icon-" . $this->identifier, $icon), $before);
+		return $this->addContent(new HtmlIcon("icon-".$this->identifier, $icon), $before);
 	}
 
 	/**
@@ -127,7 +127,7 @@ 
 	}
 
 	public function setFloated($direction="right") {
-		return $this->addToPropertyCtrl("class", $direction . " floated", Direction::getConstantValues("floated"));
+		return $this->addToPropertyCtrl("class", $direction." floated", Direction::getConstantValues("floated"));
 	}
 
 	public function floatRight() {

Ajax/service/JArray.php

Spacing +15 added lines, -15 removed lines

@@ -5,7 +5,7 @@ 
 class JArray {
 
 	public static function isAssociative($array) {
-		return (array_values($array) !== $array);
+		return (array_values($array)!==$array);
 		// return (array_keys($array)!==range(0, count($array)-1));
 	}
 
@@ -14,7 +14,7 @@ 
 			return $array[$key];
 		}
 		$values=array_values($array);
-		if ($pos < sizeof($values))
+		if ($pos<sizeof($values))
 			return $values[$pos];
 	}
 
@@ -22,12 +22,12 @@ 
 		$result=NULL;
 		if (array_key_exists($key, $array)) {
 			$result=$array[$key];
-			if ($condition($result) === true)
+			if ($condition($result)===true)
 				return $result;
 		}
 		$values=array_values($array);
-		foreach ( $values as $val ) {
-			if ($condition($val) === true)
+		foreach ($values as $val) {
+			if ($condition($val)===true)
 				return $val;
 		}
 		return $result;
@@ -36,7 +36,7 @@ 
 	public static function getDefaultValue($array, $key, $default=NULL) {
 		if (array_key_exists($key, $array)) {
 			return $array[$key];
-		} else
+		}else
 			return $default;
 	}
 
@@ -44,13 +44,13 @@ 
 		$result="";
 		if (\is_array($glue)) {
 			$size=\sizeof($pieces);
-			if ($size > 0) {
-				for($i=0; $i < $size - 1; $i++) {
-					$result.=$pieces[$i] . @$glue[$i];
+			if ($size>0) {
+				for ($i=0; $i<$size-1; $i++) {
+					$result.=$pieces[$i].@$glue[$i];
 				}
-				$result.=$pieces[$size - 1];
+				$result.=$pieces[$size-1];
 			}
-		} else {
+		}else {
 			$result=\implode($glue, $pieces);
 		}
 		return $result;
@@ -58,16 +58,16 @@ 
 
 	public static function dimension($array) {
 		if (is_array(reset($array))) {
-			$return=self::dimension(reset($array)) + 1;
-		} else {
+			$return=self::dimension(reset($array))+1;
+		}else {
 			$return=1;
 		}
 		return $return;
 	}
 
 	public static function sortAssociative($array, $sortedKeys=array()) {
-		$newArray=array ();
-		foreach ( $sortedKeys as $key ) {
+		$newArray=array();
+		foreach ($sortedKeys as $key) {
 			if (\array_key_exists($key, $array)) {
 				$newArray[$key]=$array[$key];
 			}

Braces +12 added lines, -8 removed lines

@@ -14,21 +14,24 @@ 
 			return $array[$key];
 		}
 		$values=array_values($array);
-		if ($pos < sizeof($values))
-			return $values[$pos];
+		if ($pos < sizeof($values)) {
+					return $values[$pos];
+		}
 	}
 
 	public static function getConditionalValue($array, $key, $condition) {
 		$result=NULL;
 		if (array_key_exists($key, $array)) {
 			$result=$array[$key];
-			if ($condition($result) === true)
-				return $result;
+			if ($condition($result) === true) {
+							return $result;
+			}
 		}
 		$values=array_values($array);
 		foreach ( $values as $val ) {
-			if ($condition($val) === true)
-				return $val;
+			if ($condition($val) === true) {
+							return $val;
+			}
 		}
 		return $result;
 	}
@@ -36,8 +39,9 @@ 
 	public static function getDefaultValue($array, $key, $default=NULL) {
 		if (array_key_exists($key, $array)) {
 			return $array[$key];
-		} else
-			return $default;
+		} else {
+					return $default;
+		}
 	}
 
 	public static function implode($glue, $pieces) {

Ajax/semantic/html/collections/HtmlTable.php

Spacing +12 added lines, -12 removed lines

@@ -19,9 +19,9 @@ 
 
 	public function __construct($identifier, $rowCount, $colCount) {
 		parent::__construct($identifier, "table", "ui table");
-		$this->content=array ();
+		$this->content=array();
 		$this->setRowCount($rowCount, $colCount);
-		$this->_variations=[ Variation::CELLED,Variation::PADDED,Variation::COMPACT ];
+		$this->_variations=[Variation::CELLED, Variation::PADDED, Variation::COMPACT];
 	}
 
 	/**
@@ -30,9 +30,9 @@ 
 	 * @return HtmlTableContent
 	 */
 	private function getPart($key) {
-		if (\array_key_exists($key, $this->content) === false) {
+		if (\array_key_exists($key, $this->content)===false) {
 			$this->content[$key]=new HtmlTableContent("", $key);
-			if ($key !== "tbody") {
+			if ($key!=="tbody") {
 				$this->content[$key]->setRowCount(1, $this->_colCount);
 			}
 		}
@@ -69,7 +69,7 @@ 
 	 * @return boolean
 	 */
 	public function hasPart($key) {
-		return \array_key_exists($key, $this->content) === true;
+		return \array_key_exists($key, $this->content)===true;
 	}
 
 	/**
@@ -148,10 +148,10 @@ 
 
 	private function colAlign($colIndex, $function) {
 		if (\is_array($colIndex)) {
-			foreach ( $colIndex as $cIndex ) {
+			foreach ($colIndex as $cIndex) {
 				$this->colAlign($cIndex, $function);
 			}
-		} else {
+		}else {
 			if ($this->hasPart("thead")) {
 				$this->getHeader()->$function($colIndex);
 			}
@@ -166,8 +166,8 @@ 
 
 	public function setBasic($very=false) {
 		if ($very)
-			$this->addToPropertyCtrl("class", "very", array ("very" ));
-		return $this->addToPropertyCtrl("class", "basic", array ("basic" ));
+			$this->addToPropertyCtrl("class", "very", array("very"));
+		return $this->addToPropertyCtrl("class", "basic", array("basic"));
 	}
 
 	public function setCollapsing() {
@@ -228,9 +228,9 @@ 
 	 * @see \Ajax\semantic\html\base\HtmlSemDoubleElement::compile()
 	 */
 	public function compile(JsUtils $js=NULL, View $view=NULL) {
-		$this->content=JArray::sortAssociative($this->content, [ "thead","tbody","tfoot" ]);
+		$this->content=JArray::sortAssociative($this->content, ["thead", "tbody", "tfoot"]);
 		if ($this->propertyContains("class", "sortable")) {
-			$this->addEvent("execute", "$('#" . $this->identifier . "').tablesort();");
+			$this->addEvent("execute", "$('#".$this->identifier."').tablesort();");
 		}
 		return parent::compile($js, $view);
 	}
@@ -245,7 +245,7 @@ 
 		$result=$function($object);
 		if (\is_array($result)) {
 			return $this->addRow($function($object));
-		} else {
+		}else {
 			return $this->getBody()->_addRow($result);
 		}
 	}

Braces +3 added lines, -2 removed lines

@@ -165,8 +165,9 @@
 	}
 
 	public function setBasic($very=false) {
-		if ($very)
-			$this->addToPropertyCtrl("class", "very", array ("very" ));
+		if ($very) {
+					$this->addToPropertyCtrl("class", "very", array ("very" ));
+		}
 		return $this->addToPropertyCtrl("class", "basic", array ("basic" ));
 	}
 

Doc Comments +9 added lines, -1 removed lines

@@ -17,6 +17,11 @@ 
 class HtmlTable extends HtmlSemDoubleElement {
 	private $_colCount;
 
+	/**
+	 * @param string $identifier
+	 * @param integer $rowCount
+	 * @param integer $colCount
+	 */
 	public function __construct($identifier, $rowCount, $colCount) {
 		parent::__construct($identifier, "table", "ui table");
 		$this->content=array ();
@@ -96,7 +101,7 @@ 
 	/**
 	 * Retuns the row at $rowIndex
 	 * @param int $rowIndex
-	 * @return \Ajax\semantic\html\content\HtmlTR
+	 * @return \Ajax\common\html\HtmlDoubleElement
 	 */
 	public function getRow($rowIndex) {
 		return $this->getBody()->getRow($rowIndex);
@@ -172,6 +177,9 @@ 
 		return $this->colAlign($colIndex, "colLeft");
 	}
 
+	/**
+	 * @param string $function
+	 */
 	private function colAlign($colIndex, $function) {
 		if (\is_array($colIndex)) {
 			foreach ( $colIndex as $cIndex ) {

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/semantic/html/content/table/HtmlTableContent.php

Spacing +21 added lines, -21 removed lines

@@ -11,7 +11,7 @@ 
  *
  */
 class HtmlTableContent extends HtmlSemCollection {
-	protected $_tdTagNames=[ "thead" => "th","tbody" => "td","tfoot" => "th" ];
+	protected $_tdTagNames=["thead" => "th", "tbody" => "td", "tfoot" => "th"];
 
 	/**
 	 *
@@ -34,7 +34,7 @@ 
 	 */
 	public function setRowCount($rowCount, $colCount) {
 		$count=$this->count();
-		for($i=$count; $i < $rowCount; $i++) {
+		for ($i=$count; $i<$rowCount; $i++) {
 			$this->addItem($colCount);
 		}
 		/*
@@ -62,7 +62,7 @@ 
 		$tr=new HtmlTR("", $value);
 		$tr->setContainer($this, $count);
 		$tr->setTdTagName($this->_tdTagNames[$this->tagName]);
-		if (isset($value) === true) {
+		if (isset($value)===true) {
 			$tr->setColCount($value);
 		}
 		return $tr;
@@ -112,7 +112,7 @@ 
 	 */
 	public function setCellValue($row, $col, $value="") {
 		$cell=$this->getCell($row, $col);
-		if (isset($cell) === true) {
+		if (isset($cell)===true) {
 			$cell->setValue($value);
 		}
 		return $this;
@@ -125,16 +125,16 @@ 
 	public function setValues($values=array()) {
 		$count=$this->count();
 		$isArray=true;
-		if (\is_array($values) === false) {
+		if (\is_array($values)===false) {
 			$values=\array_fill(0, $count, $values);
 			$isArray=false;
 		}
-		if (JArray::dimension($values) == 1 && $isArray)
-			$values=[ $values ];
+		if (JArray::dimension($values)==1 && $isArray)
+			$values=[$values];
 
 		$count=\min(\sizeof($values), $count);
 
-		for($i=0; $i < $count; $i++) {
+		for ($i=0; $i<$count; $i++) {
 			$row=$this->content[$i];
 			$row->setValues($values[$i]);
 		}
@@ -143,11 +143,11 @@ 
 
 	public function setColValues($colIndex, $values=array()) {
 		$count=$this->count();
-		if (\is_array($values) === false) {
+		if (\is_array($values)===false) {
 			$values=\array_fill(0, $count, $values);
 		}
 		$count=\min(\sizeof($values), $count);
-		for($i=0; $i < $count; $i++) {
+		for ($i=0; $i<$count; $i++) {
 			$this->getCell($i, $colIndex)->setValue($values[$i]);
 		}
 		return $this;
@@ -155,7 +155,7 @@ 
 
 	public function addColVariations($colIndex, $variations=array()) {
 		$count=$this->count();
-		for($i=0; $i < $count; $i++) {
+		for ($i=0; $i<$count; $i++) {
 			$this->getCell($i, $colIndex)->addVariations($variations);
 		}
 		return $this;
@@ -163,7 +163,7 @@ 
 
 	public function setRowValues($rowIndex, $values=array()) {
 		$count=$this->count();
-		if (\is_array($values) === false) {
+		if (\is_array($values)===false) {
 			$values=\array_fill(0, $count, $values);
 		}
 		$this->getItem($rowIndex)->setValues($values);
@@ -172,9 +172,9 @@ 
 
 	private function colAlign($colIndex, $function) {
 		$count=$this->count();
-		for($i=0; $i < $count; $i++) {
+		for ($i=0; $i<$count; $i++) {
 			$index=$this->content[$i]->getColPosition($colIndex);
-			if ($index !== NULL)
+			if ($index!==NULL)
 				$this->getCell($i, $index)->$function();
 		}
 		return $this;
@@ -206,7 +206,7 @@ 
 	 */
 	public function getColCount() {
 		$result=0;
-		if ($this->count() > 0)
+		if ($this->count()>0)
 			$result=$this->getItem(0)->getColCount();
 		return $result;
 	}
@@ -220,10 +220,10 @@ 
 	public function delete($rowIndex, $colIndex=NULL) {
 		if (isset($colIndex)) {
 			$row=$this->getItem($rowIndex);
-			if (isset($row) === true) {
+			if (isset($row)===true) {
 				$row->delete($colIndex);
 			}
-		} else {
+		}else {
 			$this->removeItem($rowIndex);
 		}
 		return $this;
@@ -252,7 +252,7 @@ 
 	 */
 	public function conditionalCellFormat($callback, $format) {
 		$rows=$this->content;
-		foreach ( $rows as $row ) {
+		foreach ($rows as $row) {
 			$row->conditionalCellFormat($callback, $format);
 		}
 		return $this;
@@ -265,7 +265,7 @@ 
 	 */
 	public function conditionalRowFormat($callback, $format) {
 		$rows=$this->content;
-		foreach ( $rows as $row ) {
+		foreach ($rows as $row) {
 			$row->conditionalRowFormat($callback, $format);
 		}
 		return $this;
@@ -277,7 +277,7 @@ 
 	 */
 	public function applyCells($callback) {
 		$rows=$this->content;
-		foreach ( $rows as $row ) {
+		foreach ($rows as $row) {
 			$row->applyCells($callback);
 		}
 		return $this;
@@ -289,7 +289,7 @@ 
 	 */
 	public function applyRows($callback) {
 		$rows=$this->content;
-		foreach ( $rows as $row ) {
+		foreach ($rows as $row) {
 			$row->apply($callback);
 		}
 		return $this;

Braces +12 added lines, -8 removed lines

@@ -22,8 +22,9 @@ 
 	 */
 	public function __construct($identifier, $tagName="tbody", $rowCount=NULL, $colCount=NULL) {
 		parent::__construct($identifier, $tagName, "");
-		if (isset($rowCount) && isset($colCount))
-			$this->setRowCount($rowCount, $colCount);
+		if (isset($rowCount) && isset($colCount)) {
+					$this->setRowCount($rowCount, $colCount);
+		}
 	}
 
 	/**
@@ -129,8 +130,9 @@ 
 			$values=\array_fill(0, $count, $values);
 			$isArray=false;
 		}
-		if (JArray::dimension($values) == 1 && $isArray)
-			$values=[ $values ];
+		if (JArray::dimension($values) == 1 && $isArray) {
+					$values=[ $values ];
+		}
 
 		$count=\min(\sizeof($values), $count);
 
@@ -174,8 +176,9 @@ 
 		$count=$this->count();
 		for($i=0; $i < $count; $i++) {
 			$index=$this->content[$i]->getColPosition($colIndex);
-			if ($index !== NULL)
-				$this->getCell($i, $index)->$function();
+			if ($index !== NULL) {
+							$this->getCell($i, $index)->$function();
+			}
 		}
 		return $this;
 	}
@@ -206,8 +209,9 @@ 
 	 */
 	public function getColCount() {
 		$result=0;
-		if ($this->count() > 0)
-			$result=$this->getItem(0)->getColCount();
+		if ($this->count() > 0) {
+					$result=$this->getItem(0)->getColCount();
+		}
 		return $result;
 	}
 

Doc Comments +16 added lines, -1 removed lines

@@ -68,10 +68,16 @@ 
 		return $tr;
 	}
 
+	/**
+	 * @param integer $value
+	 */
 	public function newRow($value) {
 		return $this->createItem($value);
 	}
 
+	/**
+	 * @param integer $colCount
+	 */
 	public function addRow($colCount) {
 		return $this->addItem($colCount);
 	}
@@ -97,7 +103,7 @@ 
 	/**
 	 *
 	 * @param int $index
-	 * @return \Ajax\semantic\html\content\HtmlTR
+	 * @return \Ajax\common\html\HtmlDoubleElement
 	 */
 	public function getRow($index) {
 		return $this->getItem($index);
@@ -141,6 +147,9 @@ 
 		return $this;
 	}
 
+	/**
+	 * @param integer $colIndex
+	 */
 	public function setColValues($colIndex, $values=array()) {
 		$count=$this->count();
 		if (\is_array($values) === false) {
@@ -161,6 +170,9 @@ 
 		return $this;
 	}
 
+	/**
+	 * @param integer $rowIndex
+	 */
 	public function setRowValues($rowIndex, $values=array()) {
 		$count=$this->count();
 		if (\is_array($values) === false) {
@@ -170,6 +182,9 @@ 
 		return $this;
 	}
 
+	/**
+	 * @param string $function
+	 */
 	private function colAlign($colIndex, $function) {
 		$count=$this->count();
 		for($i=0; $i < $count; $i++) {

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

Spacing +8 added lines, -8 removed lines

@@ -10,7 +10,7 @@ 
 use Ajax\semantic\html\elements\html5\HtmlLink;
 
 class HtmlTD extends HtmlSemDoubleElement {
-	use TextAlignmentTrait,TableElementTrait;
+	use TextAlignmentTrait, TableElementTrait;
 	private $_container;
 	private $_row;
 	private $_col;
@@ -23,8 +23,8 @@ 
 	 */
 	public function __construct($identifier, $content=NULL, $tagName="td") {
 		parent::__construct($identifier, $tagName, "", $content);
-		$this->_variations=[ Variation::COLLAPSING ];
-		$this->_states=[ State::ACTIVE,State::POSITIVE,State::NEGATIVE,State::WARNING,State::ERROR,State::DISABLED ];
+		$this->_variations=[Variation::COLLAPSING];
+		$this->_states=[State::ACTIVE, State::POSITIVE, State::NEGATIVE, State::WARNING, State::ERROR, State::DISABLED];
 	}
 
 	public function setContainer($container, $row, $col) {
@@ -39,8 +39,8 @@ 
 	}
 
 	public function setRowspan($rowspan) {
-		$to=min($this->_container->count(), $this->_row + $rowspan - 1);
-		for($i=$to; $i > $this->_row; $i--) {
+		$to=min($this->_container->count(), $this->_row+$rowspan-1);
+		for ($i=$to; $i>$this->_row; $i--) {
 			$this->_container->delete($i, $this->_col);
 		}
 		$this->setProperty("rowspan", $rowspan);
@@ -56,9 +56,9 @@ 
 	}
 
 	public function setColspan($colspan) {
-		$to=min($this->_container->getRow($this->_row)->count(), $this->_col + $colspan - 1);
-		for($i=$to; $i > $this->_col; $i--) {
-			$this->_container->delete($this->_row, $this->_col + 1);
+		$to=min($this->_container->getRow($this->_row)->count(), $this->_col+$colspan-1);
+		for ($i=$to; $i>$this->_col; $i--) {
+			$this->_container->delete($this->_row, $this->_col+1);
 		}
 		$this->setProperty("colspan", $colspan);
 		return $this->_container;

Braces +6 added lines, -4 removed lines

@@ -66,15 +66,17 @@
 
 	public function getColspan() {
 		$colspan=1;
-		if (\array_key_exists("colspan", $this->properties))
-			$colspan=$this->getProperty("colspan");
+		if (\array_key_exists("colspan", $this->properties)) {
+					$colspan=$this->getProperty("colspan");
+		}
 		return $colspan;
 	}
 
 	public function getRowspan() {
 		$rowspan=1;
-		if (\array_key_exists("rowspan", $this->properties))
-			$rowspan=$this->getProperty("rowspan");
+		if (\array_key_exists("rowspan", $this->properties)) {
+					$rowspan=$this->getProperty("rowspan");
+		}
 		return $rowspan;
 	}
 

Ajax/semantic/html/modules/HtmlProgress.php

Braces +9 added lines, -6 removed lines

@@ -12,11 +12,13 @@ 
 
 	public function __construct($identifier, $value=NULL, $label=NULL) {
 		parent::__construct($identifier, "div", "ui progress");
-		if (isset($value) === true)
-			$this->setProperty("data-percent", $value);
+		if (isset($value) === true) {
+					$this->setProperty("data-percent", $value);
+		}
 		$this->createBar();
-		if (isset($label) === true)
-			$this->setLabel($label);
+		if (isset($label) === true) {
+					$this->setLabel($label);
+		}
 	}
 
 	public function setLabel($label) {
@@ -93,8 +95,9 @@ 
 	 * @see BaseHtml::run()
 	 */
 	public function run(JsUtils $js) {
-		if (isset($this->_bsComponent) === false)
-			$this->_bsComponent=$js->semantic()->progress("#" . $this->identifier, $this->_params);
+		if (isset($this->_bsComponent) === false) {
+					$this->_bsComponent=$js->semantic()->progress("#" . $this->identifier, $this->_params);
+		}
 		$this->addEventsOnRun($js);
 		return $this->_bsComponent;
 	}

Spacing +15 added lines, -15 removed lines

@@ -9,26 +9,26 @@ 
 use Ajax\semantic\html\base\constants\State;
 
 class HtmlProgress extends HtmlSemDoubleElement {
-	private $_params=array ();
+	private $_params=array();
 
 	public function __construct($identifier, $value=NULL, $label=NULL, $attributes=array()) {
 		parent::__construct($identifier, "div", "ui progress");
-		if (isset($value) === true)
+		if (isset($value)===true)
 			$this->setProperty("data-percent", $value);
 		$this->createBar();
-		if (isset($label) === true)
+		if (isset($label)===true)
 			$this->setLabel($label);
-		$this->_states=[ State::SUCCESS,State::WARNING,State::ERROR,State::ACTIVE,State::DISABLED ];
+		$this->_states=[State::SUCCESS, State::WARNING, State::ERROR, State::ACTIVE, State::DISABLED];
 		$this->addToProperty("class", $attributes);
 	}
 
 	public function setLabel($label) {
-		$this->content["label"]=new HtmlSemDoubleElement("lbl-" . $this->identifier, "div", "label", $label);
+		$this->content["label"]=new HtmlSemDoubleElement("lbl-".$this->identifier, "div", "label", $label);
 		return $this;
 	}
 
 	private function createBar() {
-		$bar=new HtmlSemDoubleElement("bar-" . $this->identifier, "div", "bar", new HtmlSemDoubleElement("progress-" . $this->identifier, "div", "progress"));
+		$bar=new HtmlSemDoubleElement("bar-".$this->identifier, "div", "bar", new HtmlSemDoubleElement("progress-".$this->identifier, "div", "progress"));
 		$this->content["bar"]=$bar;
 		return $this;
 	}
@@ -68,20 +68,20 @@ 
 	 * @see \Ajax\semantic\html\base\HtmlSemDoubleElement::compile()
 	 */
 	public function compile(JsUtils $js=NULL, View $view=NULL) {
-		$this->content=JArray::sortAssociative($this->content, [ "bar","label" ]);
+		$this->content=JArray::sortAssociative($this->content, ["bar", "label"]);
 		return parent::compile($js, $view);
 	}
 
 	public function jsSetValue($value) {
-		return '$("#' . $this->identifier . '").progress({value:' . $value . '});';
+		return '$("#'.$this->identifier.'").progress({value:'.$value.'});';
 	}
 
 	public function jsIncValue() {
-		return '$("#' . $this->identifier . '").progress("increment");';
+		return '$("#'.$this->identifier.'").progress("increment");';
 	}
 
 	public function jsDecValue() {
-		return '$("#' . $this->identifier . '").progress("decrement");';
+		return '$("#'.$this->identifier.'").progress("decrement");';
 	}
 
 	/**
@@ -95,7 +95,7 @@ 
 	 * @return HtmlProgress
 	 */
 	public function setTextValues($active=false, $error=false, $success=false, $warning=false, $percent="{percent}%", $ratio="{value} of {total}") {
-		if (\is_array($active) == true) {
+		if (\is_array($active)==true) {
 			$array=$active;
 			$active=JArray::getDefaultValue($array, "active", false);
 			$success=JArray::getDefaultValue($array, "success", $success);
@@ -103,12 +103,12 @@ 
 			$percent=JArray::getDefaultValue($array, "percent", $percent);
 			$ratio=JArray::getDefaultValue($array, "ratio", $ratio);
 		}
-		$this->_params["text"]="%{active  : " . \var_export($active, true) . ",error: " . \var_export($error, true) . ",success : " . \var_export($success, true) . ",warning : " . \var_export($warning, true) . ",percent : " . \var_export($percent, true) . ",ratio   : " . \var_export($ratio, true) . "}%";
+		$this->_params["text"]="%{active  : ".\var_export($active, true).",error: ".\var_export($error, true).",success : ".\var_export($success, true).",warning : ".\var_export($warning, true).",percent : ".\var_export($percent, true).",ratio   : ".\var_export($ratio, true)."}%";
 		return $this;
 	}
 
 	public function onChange($jsCode) {
-		return $this->_params["onChange"]="%function(percent, value, total){" . $jsCode . "}%";
+		return $this->_params["onChange"]="%function(percent, value, total){".$jsCode."}%";
 	}
 
 	/*
@@ -116,8 +116,8 @@ 
 	 * @see BaseHtml::run()
 	 */
 	public function run(JsUtils $js) {
-		if (isset($this->_bsComponent) === false)
-			$this->_bsComponent=$js->semantic()->progress("#" . $this->identifier, $this->_params);
+		if (isset($this->_bsComponent)===false)
+			$this->_bsComponent=$js->semantic()->progress("#".$this->identifier, $this->_params);
 		$this->addEventsOnRun($js);
 		return $this->_bsComponent;
 	}

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/semantic/html/elements/HtmlButtonGroups.php

Spacing +11 added lines, -11 removed lines

@@ -14,8 +14,8 @@ 
 
 	public function __construct($identifier, $elements=array(), $asIcons=false) {
 		parent::__construct($identifier, "div", "ui buttons");
-		$this->content=array ();
-		if ($asIcons === true)
+		$this->content=array();
+		if ($asIcons===true)
 			$this->asIcons();
 		$this->addElements($elements, $asIcons);
 	}
@@ -24,25 +24,25 @@ 
 		$elementO=$element;
 		if (\is_string($element)) {
 			if ($asIcon) {
-				$elementO=new HtmlButton("button-" . $this->identifier . "-" . \sizeof($this->content));
+				$elementO=new HtmlButton("button-".$this->identifier."-".\sizeof($this->content));
 				$elementO->asIcon($element);
-			} else
-				$elementO=new HtmlButton("button-" . $this->identifier . "-" . \sizeof($this->content), $element);
+			}else
+				$elementO=new HtmlButton("button-".$this->identifier."-".\sizeof($this->content), $element);
 		}
 		$this->addContent($elementO);
 	}
 
 	public function addElements($elements, $asIcons=false) {
-		foreach ( $elements as $element ) {
+		foreach ($elements as $element) {
 			$this->addElement($element, $asIcons);
 		}
 		return $this;
 	}
 
 	public function insertOr($aferIndex=0, $or="or") {
-		$orElement=new HtmlSemDoubleElement("or-" . $this->identifier, "div", "or");
+		$orElement=new HtmlSemDoubleElement("or-".$this->identifier, "div", "or");
 		$orElement->setProperty("data-text", $or);
-		array_splice($this->content, $aferIndex + 1, 0, array ($orElement ));
+		array_splice($this->content, $aferIndex+1, 0, array($orElement));
 		return $this;
 	}
 
@@ -55,7 +55,7 @@ 
 	}
 
 	public function asIcons() {
-		foreach ( $this->content as $item ) {
+		foreach ($this->content as $item) {
 			$item->asIcon($item->getContent());
 		}
 		return $this->addToProperty("class", "icons");
@@ -93,7 +93,7 @@ 
 	 * @see \Ajax\bootstrap\html\base\BaseHtml::on()
 	 */
 	public function on($event, $jsCode, $stopPropagation=false, $preventDefault=false) {
-		foreach ( $this->content as $element ) {
+		foreach ($this->content as $element) {
 			$element->on($event, $jsCode, $stopPropagation, $preventDefault);
 		}
 		return $this;
@@ -105,7 +105,7 @@ 
 
 	public function addClasses($classes=array()) {
 		$i=0;
-		foreach ( $this->content as $button ) {
+		foreach ($this->content as $button) {
 			$button->addToProperty("class", $classes[$i++]);
 		}
 		return $this;

Braces +9 added lines, -7 removed lines

@@ -15,8 +15,9 @@ 
 	public function __construct($identifier, $elements=array(), $asIcons=false) {
 		parent::__construct($identifier, "div", "ui buttons");
 		$this->content=array ();
-		if ($asIcons === true)
-			$this->asIcons();
+		if ($asIcons === true) {
+					$this->asIcons();
+		}
 		$this->addElements($elements, $asIcons);
 	}
 
@@ -26,8 +27,9 @@ 
 			if ($asIcon) {
 				$elementO=new HtmlButton("button-" . $this->identifier . "-" . \sizeof($this->content));
 				$elementO->asIcon($element);
-			} else
-				$elementO=new HtmlButton("button-" . $this->identifier . "-" . \sizeof($this->content), $element);
+			} else {
+							$elementO=new HtmlButton("button-" . $this->identifier . "-" . \sizeof($this->content), $element);
+			}
 		}
 		$this->addContent($elementO);
 	}
@@ -75,9 +77,9 @@ 
 	 * @return HtmlButton
 	 */
 	public function getElement($index) {
-		if (is_int($index))
-			return $this->content[$index];
-		else {
+		if (is_int($index)) {
+					return $this->content[$index];
+		} else {
 			$elm=$this->getElementById($index, $this->content);
 			return $elm;
 		}