phpMv / phpMv-UI

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

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;

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

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 +20 added lines, -20 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(HtmlTD &$cell,$value){$cell->setValue($value);});
+		return $this->_addOrSetValues($values, function(HtmlTD&$cell, $value) {$cell->setValue($value); });
 	}
 
 	/**
@@ -76,27 +76,27 @@ 
 	 * @param mixed $values
 	 */
 	public function addValues($values=array()) {
-		return $this->_addOrSetValues($values, function(HtmlTD &$cell,$value){$cell->addValue($value);});
+		return $this->_addOrSetValues($values, function(HtmlTD&$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,13 +172,13 @@ 
 
 	public function applyCells($callback) {
 		$cells=$this->content;
-		foreach ( $cells as $cell ) {
+		foreach ($cells as $cell) {
 			$cell->apply($callback);
 		}
 		return $this;
 	}
 
-	public function toDelete($colIndex){
+	public function toDelete($colIndex) {
 		$this->getItem($colIndex)->toDelete();
 		return $this;
 	}

Ajax/semantic/html/content/HtmlGridRow.php

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

Ajax/bootstrap/components/Tabs.php

Spacing +2 added lines, -2 removed lines

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

Braces +15 added lines, -10 removed lines

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

Ajax/bootstrap/components/Collapse.php

Spacing +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
 	}
 
 	protected function compileEvents() {
-		foreach ( $this->events as $event => $jsCode ) {
+		foreach ($this->events as $event => $jsCode) {
 			$this->jquery_code_for_compile []="$( \"".$this->collapsed."\" ).on(\"".$event."\" , function (e) {".$jsCode."});";
 		}
 	}

Ajax/bootstrap/components/Splitbutton.php

Spacing +1 added lines, -1 removed lines

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

Ajax/bootstrap/html/HtmlInput.php

Spacing +1 added lines, -1 removed lines

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

Ajax/bootstrap/html/HtmlForm.php

Braces +3 added lines, -2 removed lines

@@ -99,8 +99,9 @@
 	}
 
 	public function addGroup($identifier="") {
-		if ($identifier==="")
-			$identifier="form-".$this->identifier;
+		if ($identifier==="") {
+					$identifier="form-".$this->identifier;
+		}
 		$group=new HtmlBsDoubleElement($identifier);
 		$group->setTagName("div");
 		$group->setClass("form-group");

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use Ajax\bootstrap\html\base\HtmlBsDoubleElement;
 use Ajax\JsUtils;
-
 use Ajax\common\html\html5\HtmlSelect;
 use Ajax\service\JString;
 

Doc Comments +3 added lines

@@ -87,6 +87,9 @@
 		return $result;
 	}
 
+	/**
+	 * @return string
+	 */
 	protected function getPrefix($element) {
 		$result="input_text";
 		foreach ( $this->formElementsPrefix as $k => $v ) {

Spacing +9 added lines, -9 removed lines

@@ -16,8 +16,8 @@ 
 	public function __construct($identifier) {
 		parent::__construct($identifier, "form");
 		$this->_template='<form id="%identifier%" name="%identifier%" %properties%>%content%</form>';
-		$this->futureElements=array ();
-		$this->formGroups=array ();
+		$this->futureElements=array();
+		$this->formGroups=array();
 	}
 
 	/*
@@ -27,10 +27,10 @@ 
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
 		if (isset($js)) {
 			$this->formElementsPrefix=$js->config()->getVar("formElementsPrefix");
-			foreach ( $this->futureElements as $futureElement ) {
+			foreach ($this->futureElements as $futureElement) {
 				$futureElementValue=$this->getPrefix($futureElement);
 				$futureElementValues=explode("_", $futureElementValue);
-				switch($futureElementValues [0]) {
+				switch ($futureElementValues [0]) {
 					case "input":
 						$control=new HtmlInput($futureElement);
 						$control->setClass("form-control");
@@ -48,7 +48,7 @@ 
 						$control->setLabel($label, false);
 						break;
 					case "select":
-						$control=new HtmlSelect($futureElement,"");
+						$control=new HtmlSelect($futureElement, "");
 						$control->setProperty("size", $futureElementValues [1]);
 						$control->setClass("form-control");
 						break;
@@ -61,7 +61,7 @@ 
 				$this->addElement($control);
 			}
 		}
-		foreach ( $this->formGroups as $group ) {
+		foreach ($this->formGroups as $group) {
 			$this->addContent($group);
 		}
 		return parent::compile($js, $view);
@@ -89,7 +89,7 @@ 
 
 	protected function getPrefix($element) {
 		$result="input_text";
-		foreach ( $this->formElementsPrefix as $k => $v ) {
+		foreach ($this->formElementsPrefix as $k => $v) {
 			if (JString::startswith($element, $k)) {
 				$result=$v;
 				break;
@@ -119,7 +119,7 @@ 
 
 	public function getElement($name) {
 		$element=null;
-		foreach ( $this->formGroups as $group ) {
+		foreach ($this->formGroups as $group) {
 		}
 		return $element;
 	}
@@ -129,7 +129,7 @@ 
 	 * @see \Ajax\bootstrap\html\base\HtmlSingleElement::fromArray()
 	 */
 	public function fromArray($array) {
-		foreach ( $array as $value ) {
+		foreach ($array as $value) {
 			if (is_string($value)) {
 				$this->futureElements []=$value;
 			}