phpMv / phpMv-UI

Ajax/bootstrap/html/base/HtmlNavElement.php

Doc Comments +4 added lines, -1 removed lines

@@ -19,6 +19,10 @@ 
 	protected $attr;
 
 
+	/**
+	 * @param string $identifier
+	 * @param string $tagName
+	 */
 	public function __construct($identifier,$tagName){
 		parent::__construct($identifier,$tagName);
 		$this->root="";
@@ -29,7 +33,6 @@ 
 	 * Associate an ajax get to the elements, displayed in $targetSelector
 	 * $attr member is used to build each element url
 	 * @param string $targetSelector the target of the get
-	 * @param string $attr the html attribute used to build the elements url
 	 * @return HtmlNavElement
 	 */
 	public function autoGetOnClick($targetSelector){

Spacing +11 added lines, -11 removed lines

@@ -19,8 +19,8 @@ 
 	protected $attr;
 
 
-	public function __construct($identifier,$tagName){
-		parent::__construct($identifier,$tagName);
+	public function __construct($identifier, $tagName) {
+		parent::__construct($identifier, $tagName);
 		$this->root="";
 		$this->attr="data-ajax";
 	}
@@ -32,11 +32,11 @@ 
 	 * @param string $attr the html attribute used to build the elements url
 	 * @return HtmlNavElement
 	 */
-	public function autoGetOnClick($targetSelector){
-		return $this->getOnClick($this->root, $targetSelector,array("attr"=>$this->attr));
+	public function autoGetOnClick($targetSelector) {
+		return $this->getOnClick($this->root, $targetSelector, array("attr"=>$this->attr));
 	}
 
-	public function contentAsString(){
+	public function contentAsString() {
 		return implode("", $this->content);
 	}
 
@@ -44,15 +44,15 @@ 
 	 * Generate the jquery script to set the elements to the HtmlNavElement
 	 * @param JsUtils $jsUtils
 	 */
-	public function jsSetContent(JsUtils $jsUtils){
-		$jsUtils->html("#".$this->identifier,str_replace("\"","'", $this->contentAsString()),true);
+	public function jsSetContent(JsUtils $jsUtils) {
+		$jsUtils->html("#".$this->identifier, str_replace("\"", "'", $this->contentAsString()), true);
 	}
 
 	public function getRoot() {
 		return $this->root;
 	}
 	public function setRoot($root) {
-		$this->root = $root;
+		$this->root=$root;
 		return $this;
 	}
 	public function getAttr() {
@@ -65,12 +65,12 @@ 
 	 * @return HtmlNavElement
 	 */
 	public function setAttr($attr) {
-		$this->attr = $attr;
+		$this->attr=$attr;
 		return $this;
 	}
 
 	public function __call($method, $args) {
-		if(isset($this->$method) && is_callable($this->$method)) {
+		if (isset($this->$method) && is_callable($this->$method)) {
 			return call_user_func_array(
 					$this->$method,
 					$args
@@ -78,6 +78,6 @@ 
 		}
 	}
 
-	public abstract function fromDispatcher(JsUtils $js,$dispatcher,$startIndex=0);
+	public abstract function fromDispatcher(JsUtils $js, $dispatcher, $startIndex=0);
 
 }
\ No newline at end of file

Ajax/bootstrap/html/content/HtmlGridRow.php

Doc Comments +10 added lines

@@ -14,6 +14,11 @@ 
  */
 class HtmlGridRow extends HtmlBsDoubleElement {
 	private $cols;
+
+	/**
+	 * @param string $identifier
+	 * @param integer $numCols
+	 */
 	public function __construct($identifier,$numCols=NULL){
 		parent::__construct($identifier,"div");
 		$this->setProperty("class", "row");
@@ -86,6 +91,11 @@ 
 			$this->delete($size,$start+1, $width);
 		}
 	}
+
+	/**
+	 * @param integer $start
+	 * @param integer $width
+	 */
 	public function delete($size=CssSize::SIZE_MD,$start,$width){
 		while($start<sizeof($this->cols)+1 && $width>0){
 			$col=$this->getColAt($start,false);

Spacing +37 added lines, -37 removed lines

@@ -14,89 +14,89 @@
  */
 class HtmlGridRow extends HtmlBsDoubleElement {
 	private $cols;
-	public function __construct($identifier,$numCols=NULL){
-		parent::__construct($identifier,"div");
+	public function __construct($identifier, $numCols=NULL) {
+		parent::__construct($identifier, "div");
 		$this->setProperty("class", "row");
 		$this->cols=array();
-		if(isset($numCols)){
-			$numCols=min(12,$numCols);
-			$numCols=max(1,$numCols);
+		if (isset($numCols)) {
+			$numCols=min(12, $numCols);
+			$numCols=max(1, $numCols);
 			$width=12/$numCols;
-			for ($i=0;$i<$numCols;$i++){
-				$this->addCol(CssSize::SIZE_MD,$width);
+			for ($i=0; $i<$numCols; $i++) {
+				$this->addCol(CssSize::SIZE_MD, $width);
 			}
 		}
 	}
 
-	public function addCol($size=CssSize::SIZE_MD,$width=1){
-		$col=new HtmlGridCol($this->identifier."-col-".(sizeof($this->cols)+1),$size,$width);
+	public function addCol($size=CssSize::SIZE_MD, $width=1) {
+		$col=new HtmlGridCol($this->identifier."-col-".(sizeof($this->cols)+1), $size, $width);
 		$this->cols[]=$col;
 		return $col;
 	}
 
-	public function addColAt($size=CssSize::SIZE_MD,$width=1,$offset=1){
-		$col=$this->addCol($size,$width);
-		return $col->setOffset($size, max($offset,sizeof($this->cols)+1));
+	public function addColAt($size=CssSize::SIZE_MD, $width=1, $offset=1) {
+		$col=$this->addCol($size, $width);
+		return $col->setOffset($size, max($offset, sizeof($this->cols)+1));
 	}
 
-	public function getCol($index,$force=true){
+	public function getCol($index, $force=true) {
 		$result=null;
-		if($index<sizeof($this->cols)+1){
+		if ($index<sizeof($this->cols)+1) {
 			$result=$this->cols[$index-1];
-		}else if ($force){
-			$result=$this->addColAt(CssSize::SIZE_MD,1,$index);
+		} else if ($force) {
+			$result=$this->addColAt(CssSize::SIZE_MD, 1, $index);
 		}
 		return $result;
 	}
 
-	public function getColAt($offset,$force=true){
+	public function getColAt($offset, $force=true) {
 		$result=null;
-		foreach ($this->cols as $col){
+		foreach ($this->cols as $col) {
 			$offsets=$col->getOffsets();
-			if($result=array_search($offset, $offsets)){
+			if ($result=array_search($offset, $offsets)) {
 				break;
 			}
 		}
-		if(!$result || isset($result)==false){
-			$result=$this->getCol($offset,$force);
+		if (!$result || isset($result)==false) {
+			$result=$this->getCol($offset, $force);
 		}
 		return $result;
 	}
 
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
 
-		foreach ($this->cols as $col){
+		foreach ($this->cols as $col) {
 			$this->addContent($col);
 		}
-		return parent::compile($js,$view);
+		return parent::compile($js, $view);
 	}
 	public function getCols() {
 		return $this->cols;
 	}
 
-	public function setContentForAll($content){
-		foreach ($this->cols as $col){
+	public function setContentForAll($content) {
+		foreach ($this->cols as $col) {
 			$col->setContent($content);
 		}
 	}
-	public function merge($size=CssSize::SIZE_MD,$start,$width){
-		$col=$this->getColAt($start,false);
-		if(isset($col)){
-			$col->setWidth($size,$width+1);
-			$this->delete($size,$start+1, $width);
+	public function merge($size=CssSize::SIZE_MD, $start, $width) {
+		$col=$this->getColAt($start, false);
+		if (isset($col)) {
+			$col->setWidth($size, $width+1);
+			$this->delete($size, $start+1, $width);
 		}
 	}
-	public function delete($size=CssSize::SIZE_MD,$start,$width){
-		while($start<sizeof($this->cols)+1 && $width>0){
-			$col=$this->getColAt($start,false);
-			if(isset($col)){
+	public function delete($size=CssSize::SIZE_MD, $start, $width) {
+		while ($start<sizeof($this->cols)+1 && $width>0) {
+			$col=$this->getColAt($start, false);
+			if (isset($col)) {
 				$widthCol=$col->getWidth($size);
-				if($widthCol<=$width){
+				if ($widthCol<=$width) {
 					unset($this->cols[$start-1]);
-					$this->cols = array_values($this->cols);
+					$this->cols=array_values($this->cols);
 					$width=$width-$widthCol;
 				}
-			}else{
+			} else {
 				$width=0;
 			}
 		}

Braces +2 added lines, -2 removed lines

@@ -43,7 +43,7 @@ 
 		$result=null;
 		if($index<sizeof($this->cols)+1){
 			$result=$this->cols[$index-1];
-		}else if ($force){
+		} else if ($force){
 			$result=$this->addColAt(CssSize::SIZE_MD,1,$index);
 		}
 		return $result;
@@ -96,7 +96,7 @@ 
 					$this->cols = array_values($this->cols);
 					$width=$width-$widthCol;
 				}
-			}else{
+			} else{
 				$width=0;
 			}
 		}

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use Ajax\semantic\html\base\HtmlSemDoubleElement;
 use Ajax\JsUtils;
 use Ajax\service\JArray;
-
 use Ajax\semantic\html\base\constants\State;
 
 class HtmlProgress extends HtmlSemDoubleElement {

Ajax/bootstrap/html/HtmlButtongroups.php

Doc Comments +3 added lines

@@ -49,6 +49,9 @@
 			$element->setStyle($value);
 	}
 
+	/**
+	 * @param HtmlDropdown $bt
+	 */
 	private function dropdownAsButton($bt) {
 		$this->addExistingDropDown($bt);
 		$bt->setTagName("button");

Spacing +6 added lines, -6 removed lines

@@ -35,7 +35,7 @@ 
 	 * @return HtmlButtongroups default : ""
 	 */
 	public function setSize($size) {
-		foreach ( $this->elements as $element ) {
+		foreach ($this->elements as $element) {
 			$element->setSize($size);
 		}
 		if (is_int($size)) {
@@ -45,7 +45,7 @@ 
 	}
 
 	public function setStyle($value) {
-		foreach ( $this->elements as $element )
+		foreach ($this->elements as $element)
 			$element->setStyle($value);
 	}
 
@@ -65,7 +65,7 @@ 
 	public function addElement($element) {
 		$result=$element;
 		$iid=sizeof($this->elements)+1;
-		if (($element instanceof HtmlDropdown)||($element instanceof HtmlSplitbutton)) {
+		if (($element instanceof HtmlDropdown) || ($element instanceof HtmlSplitbutton)) {
 			$this->addExistingDropDown($element);
 			$this->elements[]=$element;
 		} elseif ($element instanceof HtmlButton) {
@@ -94,7 +94,7 @@ 
 	}
 
 	public function addElements($elements) {
-		foreach ( $elements as $element ) {
+		foreach ($elements as $element) {
 			$this->addElement($element);
 		}
 		return $this;
@@ -114,7 +114,7 @@ 
 		} else
 			$value="btn-group-".$value;
 		if (strstr($value, "justified")) {
-			foreach ( $this->elements as $element ) {
+			foreach ($this->elements as $element) {
 				$element->wrap('<div class="btn-group" role="group">', '</div>');
 			}
 		}
@@ -145,7 +145,7 @@ 
 	 * @see \Ajax\bootstrap\html\base\BaseHtml::on()
 	 */
 	public function on($event, $jsCode, $stopPropagation=false, $preventDefault=false) {
-		foreach ( $this->elements as $element ) {
+		foreach ($this->elements as $element) {
 			$element->on($event, $jsCode, $stopPropagation, $preventDefault);
 		}
 		return $this;

Braces +20 added lines, -16 removed lines

@@ -45,8 +45,9 @@ 
 	}
 
 	public function setStyle($value) {
-		foreach ( $this->elements as $element )
-			$element->setStyle($value);
+		foreach ( $this->elements as $element ) {
+					$element->setStyle($value);
+		}
 	}
 
 	private function dropdownAsButton($bt) {
@@ -71,16 +72,18 @@ 
 		} elseif ($element instanceof HtmlButton) {
 			$this->elements[]=$element;
 		} elseif (is_array($element)) {
-			if (array_key_exists("glyph", $element))
-				$bt=new HtmlGlyphButton($this->identifier."-button-".$iid);
-			elseif (array_key_exists("btnCaption", $element)) {
-				if (array_key_exists("split", $element))
-					$bt=new HtmlSplitbutton($this->identifier."-dropdown-".$iid);
-				else
-					$bt=new HtmlDropdown($this->identifier."-dropdown-".$iid);
+			if (array_key_exists("glyph", $element)) {
+							$bt=new HtmlGlyphButton($this->identifier."-button-".$iid);
+			} elseif (array_key_exists("btnCaption", $element)) {
+				if (array_key_exists("split", $element)) {
+									$bt=new HtmlSplitbutton($this->identifier."-dropdown-".$iid);
+				} else {
+									$bt=new HtmlDropdown($this->identifier."-dropdown-".$iid);
+				}
 				$this->dropdownAsButton($bt);
-			} else
-				$bt=new HtmlButton($this->identifier."-button-".$iid);
+			} else {
+							$bt=new HtmlButton($this->identifier."-button-".$iid);
+			}
 			$bt->fromArray($element);
 			$this->elements[]=$bt;
 			$result=$bt;
@@ -111,8 +114,9 @@ 
 	public function setAlignment($value) {
 		if (is_int($value)) {
 			$value=CssRef::alignment("btn-group")[$value];
-		} else
-			$value="btn-group-".$value;
+		} else {
+					$value="btn-group-".$value;
+		}
 		if (strstr($value, "justified")) {
 			foreach ( $this->elements as $element ) {
 				$element->wrap('<div class="btn-group" role="group">', '</div>');
@@ -127,9 +131,9 @@ 
 	 * @return HtmlButton
 	 */
 	public function getElement($index) {
-		if (is_int($index))
-			return $this->elements[$index];
-		else {
+		if (is_int($index)) {
+					return $this->elements[$index];
+		} else {
 			$elm=$this->getElementById($index, $this->elements);
 			return $elm;
 		}

Ajax/bootstrap/html/HtmlButtontoolbar.php

Doc Comments +6 added lines, -1 removed lines

@@ -12,6 +12,11 @@ 
 
 class HtmlButtontoolbar extends HtmlButtongroups {
 
+	/**
+	 * @param string $identifier
+	 * @param string $cssStyle
+	 * @param string $size
+	 */
 	public function __construct($identifier, $elements=array(), $cssStyle=NULL, $size=NULL, $tagName="div") {
 		parent::__construct($identifier, $elements, $cssStyle, $size, $tagName);
 		$this->setClass("btn-toolbar");
@@ -57,7 +62,7 @@ 
 
 	/**
 	 * return the Buttongroups at position $index
-	 * @return \Ajax\bootstrap\html\HtmlButtongroups
+	 * @return HtmlButton
 	 */
 	public function getGroup($index) {
 		return parent::getElement($index);

Spacing +3 added lines, -3 removed lines

@@ -79,15 +79,15 @@
 		$element=null;
 		$i=0;
 		if (is_int($index)) {
-			$elements=array ();
-			foreach ( $this->elements as $group ) {
+			$elements=array();
+			foreach ($this->elements as $group) {
 				$elements=array_merge($elements, $group->getElements());
 			}
 			if ($index<sizeof($elements)) {
 				$element=$elements [$index];
 			}
 		} else {
-			while ( $element==null&&$i<sizeof($this->elements) ) {
+			while ($element==null && $i<sizeof($this->elements)) {
 				$element=$this->elements [$i]->getElement($index);
 				$i++;
 			}

Braces +6 added lines, -5 removed lines

@@ -46,9 +46,9 @@ 
 	 */
 	private function getLastButtonGroup() {
 		$nb=sizeof($this->elements);
-		if ($nb>0)
-			$bg=$this->elements [$nb-1];
-		else {
+		if ($nb>0) {
+					$bg=$this->elements [$nb-1];
+		} else {
 			$bg=new HtmlButtongroups($this->identifier."-buttongroups-".$nb);
 			$this->elements []=$bg;
 		}
@@ -66,8 +66,9 @@ 
 	public function getLastGroup() {
 		$bg=null;
 		$nb=sizeof($this->elements);
-		if ($nb>0)
-			$bg=$this->elements [$nb-1];
+		if ($nb>0) {
+					$bg=$this->elements [$nb-1];
+		}
 		return $bg;
 	}
 

Ajax/bootstrap/html/HtmlForm.php

Doc Comments +3 added lines

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

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

@@ -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 {

Spacing +9 added lines, -9 removed lines

@@ -15,8 +15,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();
 	}
 
 	/*
@@ -26,10 +26,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");
@@ -47,7 +47,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");
 						//$control->setLabel($this->getPart($futureElement));
@@ -62,7 +62,7 @@ 
 				$this->addElement($control);
 			}
 		}
-		foreach ( $this->formGroups as $group ) {
+		foreach ($this->formGroups as $group) {
 			$this->addContent($group);
 		}
 		return parent::compile($js, $view);
@@ -90,7 +90,7 @@ 
 
 	private function getPrefix($element) {
 		$result="input_text";
-		foreach ( $this->formElementsPrefix as $k => $v ) {
+		foreach ($this->formElementsPrefix as $k => $v) {
 			if (Text::startsWith($element, $k)) {
 				$result=$v;
 				break;
@@ -120,7 +120,7 @@ 
 
 	public function getElement($name) {
 		$element=null;
-		foreach ( $this->formGroups as $group ) {
+		foreach ($this->formGroups as $group) {
 		}
 		return $element;
 	}
@@ -130,7 +130,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;
 			}

Ajax/bootstrap/html/HtmlGridsystem.php

Doc Comments +4 added lines

@@ -16,6 +16,10 @@
 class HtmlGridSystem extends HtmlBsDoubleElement {
 	private $rows;
 	
+	/**
+	 * @param string $identifier
+	 * @param integer $numCols
+	 */
 	public function __construct($identifier,$numRows=1,$numCols=NULL){
 		parent::__construct($identifier,"div");
 		$this->setProperty("class", "container-fluid");

Spacing +22 added lines, -22 removed lines

@@ -16,11 +16,11 @@ 
 class HtmlGridSystem extends HtmlBsDoubleElement {
 	private $rows;
 	
-	public function __construct($identifier,$numRows=1,$numCols=NULL){
-		parent::__construct($identifier,"div");
+	public function __construct($identifier, $numRows=1, $numCols=NULL) {
+		parent::__construct($identifier, "div");
 		$this->setProperty("class", "container-fluid");
 		$this->rows=array();
-		$this->setNumRows($numRows,$numCols);
+		$this->setNumRows($numRows, $numCols);
 	}
 	
 	/**
@@ -28,8 +28,8 @@ 
 	 * @param int $numCols 
 	 * @return \Ajax\bootstrap\html\content\HtmlGridRow
 	 */
-	public function addRow($numCols=NULL){
-		$row=new HtmlGridRow($this->identifier."-row-".(sizeof($this->rows)+1),$numCols);
+	public function addRow($numCols=NULL) {
+		$row=new HtmlGridRow($this->identifier."-row-".(sizeof($this->rows)+1), $numCols);
 		$this->rows[]=$row;
 		return $row;
 	}
@@ -40,10 +40,10 @@ 
 	 * @param boolean $force add the row at $index if true
 	 * @return \Ajax\bootstrap\html\content\HtmlGridRow
 	 */
-	public function getRow($index,$force=true){
-		if($index<sizeof($this->rows)){
+	public function getRow($index, $force=true) {
+		if ($index<sizeof($this->rows)) {
 			$result=$this->rows[$index-1];
-		}else if ($force){
+		} else if ($force) {
 			$this->setNumRows($index);
 			$result=$this->rows[$index-1];
 		}
@@ -56,8 +56,8 @@ 
 	 * @param int $numCols
 	 * @return \Ajax\bootstrap\html\HtmlGridSystem
 	 */
-	public function setNumRows($numRows,$numCols=NULL){
-		for($i=sizeof($this->rows);$i<$numRows;$i++){
+	public function setNumRows($numRows, $numCols=NULL) {
+		for ($i=sizeof($this->rows); $i<$numRows; $i++) {
 			$this->addRow($numCols);
 		}
 		return $this;
@@ -69,10 +69,10 @@ 
 	 * @param $force add the cell at $row,$col if true
 	 * @return HtmlGridCol
 	 */
-	public function getCell($row,$col,$force=true){
-		$row=$this->getRow($row,$force);
-		if(isset($row)){
-			$col=$row->getCol($col,$force);
+	public function getCell($row, $col, $force=true) {
+		$row=$this->getRow($row, $force);
+		if (isset($row)) {
+			$col=$row->getCol($col, $force);
 		}
 		return $col;
 	}
@@ -82,22 +82,22 @@ 
 	 * @param int $col
 	 * @return HtmlGridCol
 	 */
-	public function getCellAt($row,$col,$force=true){
-		$row=$this->getRow($row,$force);
-		if(isset($row)){
-			$col=$row->getColAt($col,$force);
+	public function getCellAt($row, $col, $force=true) {
+		$row=$this->getRow($row, $force);
+		if (isset($row)) {
+			$col=$row->getColAt($col, $force);
 		}
 		return $col;
 	}
 	
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
-		foreach ($this->rows as $row){
+		foreach ($this->rows as $row) {
 			$this->addContent($row);
 		}
-		return parent::compile($js,$view);
+		return parent::compile($js, $view);
 	}
-	public function setContentForAll($content){
-		foreach ($this->rows as $row){
+	public function setContentForAll($content) {
+		foreach ($this->rows as $row) {
 			$row->setContentForAll($content);
 		}
 	}

Braces +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
 	public function getRow($index,$force=true){
 		if($index<sizeof($this->rows)){
 			$result=$this->rows[$index-1];
-		}else if ($force){
+		} else if ($force){
 			$this->setNumRows($index);
 			$result=$this->rows[$index-1];
 		}

Ajax/bootstrap/html/HtmlModal.php

Doc Comments +1 added lines, -2 removed lines

@@ -121,7 +121,6 @@ 
 	/**
 	 * render the content of $controller::$action and set the response to the modal content
 	 * @param JsUtils $js
-	 * @param string $title The panel title
 	 * @param Controller $initialControllerInstance
 	 * @param string $controllerName the controller name
 	 * @param string $actionName the action name
@@ -198,7 +197,7 @@ 
 	 * Includes a modal-backdrop element.
 	 * Alternatively, specify static for a backdrop which doesn't close the modal on click.
 	 * @param Boolean $value default : true
-	 * @return HtmlModal
+	 * @return boolean
 	 */
 	public function setBackdrop($value) {
 		return $this->backdrop=$value;

Spacing +10 added lines, -10 removed lines

@@ -14,7 +14,7 @@ 
 class HtmlModal extends BaseHtml {
 	protected $title="Titre de ma boîte";
 	protected $content="";
-	protected $buttons=array ();
+	protected $buttons=array();
 	protected $showOnStartup=false;
 	protected $draggable=false;
 	protected $validCondition=NULL;
@@ -27,10 +27,10 @@ 
 	public function __construct($identifier, $title="", $content="", $buttonCaptions=array()) {
 		parent::__construct($identifier);
 		$this->_template=include 'templates/tplModal.php';
-		$this->buttons=array ();
+		$this->buttons=array();
 		$this->title=$title;
 		$this->content=$content;
-		foreach ( $buttonCaptions as $button ) {
+		foreach ($buttonCaptions as $button) {
 			$this->addButton($button);
 		}
 	}
@@ -65,7 +65,7 @@ 
 	 * @param string $value
 	 * @return HtmlButton
 	 */
-	public function addOkayButton($value="Okay",$jsCode="") {
+	public function addOkayButton($value="Okay", $jsCode="") {
 		$btn=$this->addButton($value, "btn-primary");
 		$btn->onClick("if(".$this->getValidCondition()."){ ".$jsCode."$('#".$this->identifier."').modal('hide');}");
 		return $btn;
@@ -114,8 +114,8 @@ 
 	 * @param string $viewName
 	 * @param $params The parameters to pass to the view
 	 */
-	public function renderView(JsUtils $js,$initialController,$viewName, $params=array()) {
-		$this->content=$js->renderContent($initialController, $viewName,$params);
+	public function renderView(JsUtils $js, $initialController, $viewName, $params=array()) {
+		$this->content=$js->renderContent($initialController, $viewName, $params);
 	}
 
 	/**
@@ -126,8 +126,8 @@ 
 	 * @param string $controllerName the controller name
 	 * @param string $actionName the action name
 	 */
-	public function forward(JsUtils $js,$initialControllerInstance,$controllerName,$actionName,$params=NULL){
-		$this->content=$js->forward($initialControllerInstance, $controllerName, $actionName,$params);
+	public function forward(JsUtils $js, $initialControllerInstance, $controllerName, $actionName, $params=NULL) {
+		$this->content=$js->forward($initialControllerInstance, $controllerName, $actionName, $params);
 	}
 
 	/*
@@ -135,10 +135,10 @@ 
 	 * @see BaseHtml::run()
 	 */
 	public function run(JsUtils $js) {
-		if($this->content instanceof BaseHtml){
+		if ($this->content instanceof BaseHtml) {
 			$this->content->run($js);
 		}
-		$this->_bsComponent=$js->bootstrap()->modal("#".$this->identifier, array (
+		$this->_bsComponent=$js->bootstrap()->modal("#".$this->identifier, array(
 				"show" => $this->showOnStartup
 		));
 		if ($this->draggable)

Braces +11 added lines, -8 removed lines

@@ -141,18 +141,20 @@ 
 		$this->_bsComponent=$js->bootstrap()->modal("#".$this->identifier, array (
 				"show" => $this->showOnStartup
 		));
-		if ($this->draggable)
-			$this->_bsComponent->setDraggable(true);
+		if ($this->draggable) {
+					$this->_bsComponent->setDraggable(true);
+		}
 		$this->_bsComponent->setBackdrop($this->backdrop);
 		$this->addEventsOnRun($js);
 		return $this->_bsComponent;
 	}
 
 	public function getButton($index) {
-		if (is_int($index))
-			return $this->buttons [$index];
-		else
-			return $this->getElementById($index, $this->buttons);
+		if (is_int($index)) {
+					return $this->buttons [$index];
+		} else {
+					return $this->getElementById($index, $this->buttons);
+		}
 	}
 
 	public function showOnCreate() {
@@ -178,8 +180,9 @@ 
 
 	public function jsHideButton($index) {
 		$btn=$this->getButton($index);
-		if ($btn)
-			return "$('#".$btn->getIdentifier()."').hide();";
+		if ($btn) {
+					return "$('#".$btn->getIdentifier()."').hide();";
+		}
 	}
 
 	/**

Ajax/bootstrap/html/HtmlProgressbar.php

Doc Comments +16 added lines, -1 removed lines

@@ -19,6 +19,9 @@ 
 	protected $style="";
 	protected $styleLimits=null;
 
+	/**
+	 * @param string $identifier
+	 */
 	public function __construct($identifier, $style="info", $value=0, $max=100, $min=0) {
 		parent::__construct($identifier);
 		$this->_template=include 'templates/tplProgressbar.php';
@@ -28,6 +31,9 @@ 
 		$this->setStyle($style);
 	}
 
+	/**
+	 * @param boolean $value
+	 */
 	public function setActive($value) {
 		if(is_array($this->content)){
 			foreach ($this->content as $pb){
@@ -42,6 +48,9 @@ 
 		return $this;
 	}
 
+	/**
+	 * @param boolean $value
+	 */
 	public function setStriped($value) {
 		if(is_array($this->content)){
 			foreach ($this->content as $pb){
@@ -56,6 +65,9 @@ 
 		return $this;
 	}
 
+	/**
+	 * @param boolean $value
+	 */
 	public function showCaption($value) {
 		if(is_array($this->content)){
 			foreach ($this->content as $pb){
@@ -113,6 +125,9 @@ 
 		return $this->isStacked;
 	}
 
+	/**
+	 * @param boolean $isStacked
+	 */
 	public function setIsStacked($isStacked) {
 		$this->isStacked=$isStacked;
 		return $this;
@@ -121,7 +136,7 @@ 
 	/**
 	 * define the progressbar style
 	 * avaible values : "success","info","warning","danger"
-	 * @param string|int $cssStyle
+	 * @param string $cssStyle
 	 * @return \Ajax\bootstrap\html\HtmlProgressbar default : ""
 	 */
 	public function setStyle($cssStyle) {

Spacing +14 added lines, -14 removed lines

@@ -29,11 +29,11 @@ 
 	}
 
 	public function setActive($value) {
-		if(is_array($this->content)){
-			foreach ($this->content as $pb){
+		if (is_array($this->content)) {
+			foreach ($this->content as $pb) {
 				$pb->setActive($value);
 			}
-		}else{
+		} else {
 			if ($value===true)
 				$this->active="active";
 			else
@@ -43,11 +43,11 @@ 
 	}
 
 	public function setStriped($value) {
-		if(is_array($this->content)){
-			foreach ($this->content as $pb){
+		if (is_array($this->content)) {
+			foreach ($this->content as $pb) {
 				$pb->setStriped($value);
 			}
-		}else{
+		} else {
 			if ($value===true)
 				$this->striped="progress-bar-striped";
 			else
@@ -57,11 +57,11 @@ 
 	}
 
 	public function showCaption($value) {
-		if(is_array($this->content)){
-			foreach ($this->content as $pb){
+		if (is_array($this->content)) {
+			foreach ($this->content as $pb) {
 				$pb->showCaption($value);
 			}
-		}else{
+		} else {
 			if ($value===true)
 				$this->caption="%value%%";
 			else
@@ -81,7 +81,7 @@ 
 		$progressBar->setStriped($this->striped!=="" || $progressBar->isStriped());
 		$progressBar->setActive($this->active==="active" || $progressBar->isActive());
 		if (is_array($this->content)===false) {
-			$this->content=array ();
+			$this->content=array();
 		}
 		$this->content []=$progressBar;
 	}
@@ -125,7 +125,7 @@ 
 	 * @return \Ajax\bootstrap\html\HtmlProgressbar default : ""
 	 */
 	public function setStyle($cssStyle) {
-		return $this->setMemberCtrl($this->style,CssRef::getStyle($cssStyle, "progress-bar"), CssRef::Styles("progress-bar"));
+		return $this->setMemberCtrl($this->style, CssRef::getStyle($cssStyle, "progress-bar"), CssRef::Styles("progress-bar"));
 	}
 
 	/*
@@ -134,10 +134,10 @@ 
 	 */
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
 		$actualStyle=$this->style;
-		if(isset($this->styleLimits)&& JArray::isAssociative($this->styleLimits)){
-			foreach ($this->styleLimits as $k=>$v){
+		if (isset($this->styleLimits) && JArray::isAssociative($this->styleLimits)) {
+			foreach ($this->styleLimits as $k=>$v) {
 				$actualStyle=$k;
-				if($v>$this->value)
+				if ($v>$this->value)
 					break;
 			}
 		}

Braces +21 added lines, -17 removed lines

@@ -33,11 +33,12 @@ 
 			foreach ($this->content as $pb){
 				$pb->setActive($value);
 			}
-		}else{
-			if ($value===true)
-				$this->active="active";
-			else
-				$this->active="";
+		} else{
+			if ($value===true) {
+							$this->active="active";
+			} else {
+							$this->active="";
+			}
 		}
 		return $this;
 	}
@@ -47,11 +48,12 @@ 
 			foreach ($this->content as $pb){
 				$pb->setStriped($value);
 			}
-		}else{
-			if ($value===true)
-				$this->striped="progress-bar-striped";
-			else
-				$this->striped="";
+		} else{
+			if ($value===true) {
+							$this->striped="progress-bar-striped";
+			} else {
+							$this->striped="";
+			}
 		}
 		return $this;
 	}
@@ -61,11 +63,12 @@ 
 			foreach ($this->content as $pb){
 				$pb->showCaption($value);
 			}
-		}else{
-			if ($value===true)
-				$this->caption="%value%%";
-			else
-				$this->caption='<span class="sr-only">%value%% Complete (%style%)</span>';
+		} else{
+			if ($value===true) {
+							$this->caption="%value%%";
+			} else {
+							$this->caption='<span class="sr-only">%value%% Complete (%style%)</span>';
+			}
 		}
 		return $this;
 	}
@@ -137,8 +140,9 @@ 
 		if(isset($this->styleLimits)&& JArray::isAssociative($this->styleLimits)){
 			foreach ($this->styleLimits as $k=>$v){
 				$actualStyle=$k;
-				if($v>$this->value)
-					break;
+				if($v>$this->value) {
+									break;
+				}
 			}
 		}
 		$this->style=$actualStyle;

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/common/html/BaseHtml.php

Doc Comments +42 added lines

@@ -124,6 +124,10 @@ 
 		return $this;
 	}
 
+	/**
+	 * @param string $name
+	 * @param string[] $typeCtrl
+	 */
 	protected function setMemberCtrl(&$name, $value, $typeCtrl) {
 		if ($this->ctrl($name, $value, $typeCtrl) === true) {
 			return $name=$value;
@@ -139,22 +143,35 @@ 
 		return $this;
 	}
 
+	/**
+	 * @param string $name
+	 */
 	protected function removePropertyValue($name, $value) {
 		$this->properties[$name]=\str_replace($value, "", $this->properties[$name]);
 		return $this;
 	}
 
+	/**
+	 * @param string $name
+	 */
 	protected function removePropertyValues($name, $values) {
 		$this->removeOldValues($this->properties[$name], $values);
 		return $this;
 	}
 
+	/**
+	 * @param string $name
+	 */
 	public function removeProperty($name) {
 		if (\array_key_exists($name, $this->properties))
 			unset($this->properties[$name]);
 		return $this;
 	}
 
+	/**
+	 * @param string $name
+	 * @param string[] $typeCtrl
+	 */
 	protected function addToMemberCtrl(&$name, $value, $typeCtrl, $separator=" ") {
 		if ($this->ctrl($name, $value, $typeCtrl) === true) {
 			if (is_array($typeCtrl)) {
@@ -165,6 +182,9 @@ 
 		return $this;
 	}
 
+	/**
+	 * @param string $name
+	 */
 	protected function addToMember(&$name, $value, $separator=" ") {
 		$name=str_ireplace($value, "", $name) . $separator . $value;
 		return $this;
@@ -179,6 +199,9 @@ 
 		return $this->addToProperty($name, $value);
 	}
 
+	/**
+	 * @param string $name
+	 */
 	public function addToPropertyCtrl($name, $value, $typeCtrl) {
 		return $this->addToPropertyUnique($name, $value, $typeCtrl);
 	}
@@ -327,14 +350,23 @@ 
 		return $this;
 	}
 
+	/**
+	 * @param string $event
+	 */
 	public function getOn($event, $url, $responseElement="", $parameters=array()) {
 		return $this->_ajaxOn("get", $event, $url, $responseElement, $parameters);
 	}
 
+	/**
+	 * @param string $url
+	 */
 	public function getOnClick($url, $responseElement="", $parameters=array()) {
 		return $this->getOn("click", $url, $responseElement, $parameters);
 	}
 
+	/**
+	 * @param string $event
+	 */
 	public function postOn($event, $url, $params="{}", $responseElement="", $parameters=array()) {
 		$parameters["params"]=$params;
 		return $this->_ajaxOn("post", $event, $url, $responseElement, $parameters);
@@ -344,6 +376,9 @@ 
 		return $this->postOn("click", $url, $params, $responseElement, $parameters);
 	}
 
+	/**
+	 * @param string $event
+	 */
 	public function postFormOn($event, $url, $form, $responseElement="", $parameters=array()) {
 		$parameters["form"]=$form;
 		return $this->_ajaxOn("postForm", $event, $url, $responseElement, $parameters);
@@ -371,6 +406,10 @@ 
 		return null;
 	}
 
+	/**
+	 * @param string $propertyName
+	 * @param string $value
+	 */
 	protected function getElementByPropertyValue($propertyName,$value, $elements) {
 		if (is_array($elements)) {
 			$flag=false;
@@ -411,6 +450,9 @@ 
 		return $value;
 	}
 
+	/**
+	 * @param string $jqueryCall
+	 */
 	public function jsDoJquery($jqueryCall, $param="") {
 		return "$('#" . $this->identifier . "')." . $jqueryCall . "(" . $this->_prep_value($param) . ");";
 	}

Spacing +62 added lines, -62 removed lines

@@ -16,10 +16,10 @@ 
 abstract class BaseHtml extends BaseWidget {
 	protected $_template;
 	protected $tagName;
-	protected $properties=array ();
-	protected $_events=array ();
-	protected $_wrapBefore=array ();
-	protected $_wrapAfter=array ();
+	protected $properties=array();
+	protected $_events=array();
+	protected $_wrapBefore=array();
+	protected $_wrapAfter=array();
 	protected $_bsComponent;
 
 	public function getBsComponent() {
@@ -32,7 +32,7 @@ 
 	}
 
 	protected function getTemplate(JsUtils $js=NULL) {
-		return PropertyWrapper::wrap($this->_wrapBefore, $js) . $this->_template . PropertyWrapper::wrap($this->_wrapAfter, $js);
+		return PropertyWrapper::wrap($this->_wrapBefore, $js).$this->_template.PropertyWrapper::wrap($this->_wrapAfter, $js);
 	}
 
 	public function getProperties() {
@@ -56,13 +56,13 @@ 
 
 	public function addToProperty($name, $value, $separator=" ") {
 		if (\is_array($value)) {
-			foreach ( $value as $v ) {
+			foreach ($value as $v) {
 				$this->addToProperty($name, $v, $separator);
 			}
-		} else if ($value !== "" && $this->propertyContains($name, $value) === false) {
+		} else if ($value!=="" && $this->propertyContains($name, $value)===false) {
 			$v=@$this->properties[$name];
-			if (isset($v) && $v !== "")
-				$v=$v . $separator . $value;
+			if (isset($v) && $v!=="")
+				$v=$v.$separator.$value;
 			else
 				$v=$value;
 
@@ -78,19 +78,19 @@ 
 
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
 		$result=$this->getTemplate($js);
-		foreach ( $this as $key => $value ) {
-			if (JString::startswith($key, "_") === false && $key !== "events") {
+		foreach ($this as $key => $value) {
+			if (JString::startswith($key, "_")===false && $key!=="events") {
 				if (is_array($value)) {
 					$v=PropertyWrapper::wrap($value, $js);
 				} else {
 					$v=$value;
 				}
-				$result=str_ireplace("%" . $key . "%", $v, $result);
+				$result=str_ireplace("%".$key."%", $v, $result);
 			}
 		}
 		if (isset($js)===true) {
 			$this->run($js);
-			if (isset($view) === true) {
+			if (isset($view)===true) {
 				$js->addViewElement($this->identifier, $result, $view);
 			}
 		}
@@ -99,12 +99,12 @@ 
 
 	protected function ctrl($name, $value, $typeCtrl) {
 		if (is_array($typeCtrl)) {
-			if (array_search($value, $typeCtrl) === false) {
-				throw new \Exception("La valeur passée `" . $value . "` à la propriété `" . $name . "` ne fait pas partie des valeurs possibles : {" . implode(",", $typeCtrl) . "}");
+			if (array_search($value, $typeCtrl)===false) {
+				throw new \Exception("La valeur passée `".$value."` à la propriété `".$name."` ne fait pas partie des valeurs possibles : {".implode(",", $typeCtrl)."}");
 			}
 		} else {
 			if (!$typeCtrl($value)) {
-				throw new \Exception("La fonction " . $typeCtrl . " a retourné faux pour l'affectation de la propriété " . $name);
+				throw new \Exception("La fonction ".$typeCtrl." a retourné faux pour l'affectation de la propriété ".$name);
 			}
 		}
 		return true;
@@ -119,13 +119,13 @@ 
 	}
 
 	protected function setPropertyCtrl($name, $value, $typeCtrl) {
-		if ($this->ctrl($name, $value, $typeCtrl) === true)
+		if ($this->ctrl($name, $value, $typeCtrl)===true)
 			return $this->setProperty($name, $value);
 		return $this;
 	}
 
 	protected function setMemberCtrl(&$name, $value, $typeCtrl) {
-		if ($this->ctrl($name, $value, $typeCtrl) === true) {
+		if ($this->ctrl($name, $value, $typeCtrl)===true) {
 			return $name=$value;
 		}
 		return $this;
@@ -134,7 +134,7 @@ 
 	protected function addToMemberUnique(&$name, $value, $typeCtrl, $separator=" ") {
 		if (is_array($typeCtrl)) {
 			$this->removeOldValues($name, $typeCtrl);
-			$name.=$separator . $value;
+			$name.=$separator.$value;
 		}
 		return $this;
 	}
@@ -156,17 +156,17 @@ 
 	}
 
 	protected function addToMemberCtrl(&$name, $value, $typeCtrl, $separator=" ") {
-		if ($this->ctrl($name, $value, $typeCtrl) === true) {
+		if ($this->ctrl($name, $value, $typeCtrl)===true) {
 			if (is_array($typeCtrl)) {
 				$this->removeOldValues($name, $typeCtrl);
 			}
-			$name.=$separator . $value;
+			$name.=$separator.$value;
 		}
 		return $this;
 	}
 
 	protected function addToMember(&$name, $value, $separator=" ") {
-		$name=str_ireplace($value, "", $name) . $separator . $value;
+		$name=str_ireplace($value, "", $name).$separator.$value;
 		return $this;
 	}
 
@@ -184,7 +184,7 @@ 
 	}
 
 	public function addToPropertyCtrlCheck($name, $value, $typeCtrl) {
-		if ($this->ctrl($name, $value, $typeCtrl) === true) {
+		if ($this->ctrl($name, $value, $typeCtrl)===true) {
 			return $this->addToProperty($name, $value);
 		}
 		return $this;
@@ -212,28 +212,28 @@ 
 	}
 
 	public function fromArray($array) {
-		foreach ( $this as $key => $value ) {
+		foreach ($this as $key => $value) {
 			if (array_key_exists($key, $array) && !JString::startswith($key, "_")) {
-				$setter="set" . ucfirst($key);
+				$setter="set".ucfirst($key);
 				$this->$setter($array[$key]);
 				unset($array[$key]);
 			}
 		}
-		foreach ( $array as $key => $value ) {
+		foreach ($array as $key => $value) {
 			if (method_exists($this, $key)) {
 				try {
 					$this->$key($value);
 					unset($array[$key]);
-				} catch ( \Exception $e ) {
+				}catch (\Exception $e) {
 					// Nothing to do
 				}
 			} else {
-				$setter="set" . ucfirst($key);
+				$setter="set".ucfirst($key);
 				if (method_exists($this, $setter)) {
 					try {
 						$this->$setter($value);
 						unset($array[$key]);
-					} catch ( \Exception $e ) {
+					}catch (\Exception $e) {
 						// Nothing to do
 					}
 				}
@@ -244,7 +244,7 @@ 
 
 	public function fromDatabaseObjects($objects, $function) {
 		if (isset($objects)) {
-			foreach ( $objects as $object ) {
+			foreach ($objects as $object) {
 				$this->fromDatabaseObject($object, $function);
 			}
 		}
@@ -264,11 +264,11 @@ 
 	}
 
 	public function addEvent($event, $jsCode, $stopPropagation=false, $preventDefault=false) {
-		if ($stopPropagation === true) {
-			$jsCode="event.stopPropagation();" . $jsCode;
+		if ($stopPropagation===true) {
+			$jsCode="event.stopPropagation();".$jsCode;
 		}
-		if ($preventDefault === true) {
-			$jsCode="event.preventDefault();" . $jsCode;
+		if ($preventDefault===true) {
+			$jsCode="event.preventDefault();".$jsCode;
 		}
 		return $this->_addEvent($event, $jsCode);
 	}
@@ -278,7 +278,7 @@ 
 			if (is_array($this->_events[$event])) {
 				$this->_events[$event][]=$jsCode;
 			} else {
-				$this->_events[$event]=array ($this->_events[$event],$jsCode );
+				$this->_events[$event]=array($this->_events[$event], $jsCode);
 			}
 		} else {
 			$this->_events[$event]=$jsCode;
@@ -298,35 +298,35 @@ 
 		return $this->onClick($jsCode);
 	}
 
-	public function onCreate($jsCode){
-		if(isset($this->_events["_create"])){
+	public function onCreate($jsCode) {
+		if (isset($this->_events["_create"])) {
 			$this->_events["_create"][]=$jsCode;
-		}else{
+		} else {
 			$this->_events["_create"]=[$jsCode];
 		}
 		return $this;
 	}
 
 	public function addEventsOnRun(JsUtils $js) {
-		if(isset($this->_events["_create"])){
+		if (isset($this->_events["_create"])) {
 			$create=$this->_events["_create"];
-			if(\is_array($create)){
+			if (\is_array($create)) {
 				$create=\implode("", $create);
 			}
-			if($create!=="")
-				$js->exec($create,true);
+			if ($create!=="")
+				$js->exec($create, true);
 			unset($this->_events["_create"]);
 		}
 		if (isset($this->_bsComponent)) {
-			foreach ( $this->_events as $event => $jsCode ) {
+			foreach ($this->_events as $event => $jsCode) {
 				$code=$jsCode;
 				if (is_array($jsCode)) {
 					$code="";
-					foreach ( $jsCode as $jsC ) {
+					foreach ($jsCode as $jsC) {
 						if ($jsC instanceof AjaxCall) {
-							$code.="\n" . $jsC->compile($js);
+							$code.="\n".$jsC->compile($js);
 						} else {
-							$code.="\n" . $jsC;
+							$code.="\n".$jsC;
 						}
 					}
 				} elseif ($jsCode instanceof AjaxCall) {
@@ -334,12 +334,12 @@ 
 				}
 				$this->_bsComponent->addEvent($event, $code);
 			}
-			$this->_events=array ();
+			$this->_events=array();
 		}
 	}
 
 	public function _ajaxOn($operation, $event, $url, $responseElement="", $parameters=array()) {
-		$params=array ("url" => $url,"responseElement" => $responseElement );
+		$params=array("url" => $url, "responseElement" => $responseElement);
 		$params=array_merge($params, $parameters);
 		$this->_addEvent($event, new AjaxCall($operation, $params));
 		return $this;
@@ -375,33 +375,33 @@ 
 		if (is_array($elements)) {
 			$flag=false;
 			$index=0;
-			while ( !$flag && $index < sizeof($elements) ) {
+			while (!$flag && $index<sizeof($elements)) {
 				if ($elements[$index] instanceof BaseHtml)
-					$flag=($elements[$index]->getIdentifier() === $identifier);
+					$flag=($elements[$index]->getIdentifier()===$identifier);
 				$index++;
 			}
-			if ($flag === true)
-				return $elements[$index - 1];
+			if ($flag===true)
+				return $elements[$index-1];
 		} elseif ($elements instanceof BaseHtml) {
-			if ($elements->getIdentifier() === $identifier)
+			if ($elements->getIdentifier()===$identifier)
 				return $elements;
 		}
 		return null;
 	}
 
-	protected function getElementByPropertyValue($propertyName,$value, $elements) {
+	protected function getElementByPropertyValue($propertyName, $value, $elements) {
 		if (is_array($elements)) {
 			$flag=false;
 			$index=0;
-			while ( !$flag && $index < sizeof($elements) ) {
+			while (!$flag && $index<sizeof($elements)) {
 				if ($elements[$index] instanceof BaseHtml)
-					$flag=($elements[$index]->propertyContains($propertyName, $value) === true);
+					$flag=($elements[$index]->propertyContains($propertyName, $value)===true);
 					$index++;
 			}
-			if ($flag === true)
-				return $elements[$index - 1];
+			if ($flag===true)
+				return $elements[$index-1];
 		} elseif ($elements instanceof BaseHtml) {
-			if ($elements->propertyContains($propertyName, $value) === true)
+			if ($elements->propertyContains($propertyName, $value)===true)
 				return $elements;
 		}
 		return null;
@@ -423,14 +423,14 @@ 
 		if (is_array($value)) {
 			$value=implode(",", $value);
 		}
-		if (strrpos($value, 'this') === false && strrpos($value, 'event') === false) {
-			$value='"' . $value . '"';
+		if (strrpos($value, 'this')===false && strrpos($value, 'event')===false) {
+			$value='"'.$value.'"';
 		}
 		return $value;
 	}
 
 	public function jsDoJquery($jqueryCall, $param="") {
-		return "$('#" . $this->identifier . "')." . $jqueryCall . "(" . $this->_prep_value($param) . ");";
+		return "$('#".$this->identifier."').".$jqueryCall."(".$this->_prep_value($param).");";
 	}
 
 	public function executeOnRun($jsCode) {

Braces +39 added lines, -27 removed lines

@@ -50,8 +50,9 @@ 
 	}
 
 	public function getProperty($name) {
-		if (array_key_exists($name, $this->properties))
-			return $this->properties[$name];
+		if (array_key_exists($name, $this->properties)) {
+					return $this->properties[$name];
+		}
 	}
 
 	public function addToProperty($name, $value, $separator=" ") {
@@ -61,10 +62,11 @@ 
 			}
 		} else if ($value !== "" && $this->propertyContains($name, $value) === false) {
 			$v=@$this->properties[$name];
-			if (isset($v) && $v !== "")
-				$v=$v . $separator . $value;
-			else
-				$v=$value;
+			if (isset($v) && $v !== "") {
+							$v=$v . $separator . $value;
+			} else {
+							$v=$value;
+			}
 
 			return $this->setProperty($name, $v);
 		}
@@ -119,8 +121,9 @@ 
 	}
 
 	protected function setPropertyCtrl($name, $value, $typeCtrl) {
-		if ($this->ctrl($name, $value, $typeCtrl) === true)
-			return $this->setProperty($name, $value);
+		if ($this->ctrl($name, $value, $typeCtrl) === true) {
+					return $this->setProperty($name, $value);
+		}
 		return $this;
 	}
 
@@ -150,8 +153,9 @@ 
 	}
 
 	public function removeProperty($name) {
-		if (\array_key_exists($name, $this->properties))
-			unset($this->properties[$name]);
+		if (\array_key_exists($name, $this->properties)) {
+					unset($this->properties[$name]);
+		}
 		return $this;
 	}
 
@@ -171,8 +175,9 @@ 
 	}
 
 	protected function addToPropertyUnique($name, $value, $typeCtrl) {
-		if (@class_exists($typeCtrl, true))
-			$typeCtrl=$typeCtrl::getConstants();
+		if (@class_exists($typeCtrl, true)) {
+					$typeCtrl=$typeCtrl::getConstants();
+		}
 		if (is_array($typeCtrl)) {
 			$this->removeOldValues($this->properties[$name], $typeCtrl);
 		}
@@ -301,7 +306,7 @@ 
 	public function onCreate($jsCode){
 		if(isset($this->_events["_create"])){
 			$this->_events["_create"][]=$jsCode;
-		}else{
+		} else{
 			$this->_events["_create"]=[$jsCode];
 		}
 		return $this;
@@ -313,8 +318,9 @@ 
 			if(\is_array($create)){
 				$create=\implode("", $create);
 			}
-			if($create!=="")
-				$js->exec($create,true);
+			if($create!=="") {
+							$js->exec($create,true);
+			}
 			unset($this->_events["_create"]);
 		}
 		if (isset($this->_bsComponent)) {
@@ -376,15 +382,18 @@ 
 			$flag=false;
 			$index=0;
 			while ( !$flag && $index < sizeof($elements) ) {
-				if ($elements[$index] instanceof BaseHtml)
-					$flag=($elements[$index]->getIdentifier() === $identifier);
+				if ($elements[$index] instanceof BaseHtml) {
+									$flag=($elements[$index]->getIdentifier() === $identifier);
+				}
 				$index++;
 			}
-			if ($flag === true)
-				return $elements[$index - 1];
+			if ($flag === true) {
+							return $elements[$index - 1];
+			}
 		} elseif ($elements instanceof BaseHtml) {
-			if ($elements->getIdentifier() === $identifier)
-				return $elements;
+			if ($elements->getIdentifier() === $identifier) {
+							return $elements;
+			}
 		}
 		return null;
 	}
@@ -394,15 +403,18 @@ 
 			$flag=false;
 			$index=0;
 			while ( !$flag && $index < sizeof($elements) ) {
-				if ($elements[$index] instanceof BaseHtml)
-					$flag=($elements[$index]->propertyContains($propertyName, $value) === true);
+				if ($elements[$index] instanceof BaseHtml) {
+									$flag=($elements[$index]->propertyContains($propertyName, $value) === true);
+				}
 					$index++;
 			}
-			if ($flag === true)
-				return $elements[$index - 1];
+			if ($flag === true) {
+							return $elements[$index - 1];
+			}
 		} elseif ($elements instanceof BaseHtml) {
-			if ($elements->propertyContains($propertyName, $value) === true)
-				return $elements;
+			if ($elements->propertyContains($propertyName, $value) === true) {
+							return $elements;
+			}
 		}
 		return null;
 	}