phpMv / phpMv-UI

Ajax/bootstrap/components/Modal.php

Doc Comments +3 added lines

@@ -54,6 +54,9 @@
 		return $this->setParam("keyboard", $value);
 	}
 
+	/**
+	 * @param boolean $value
+	 */
 	public function setDraggable($value) {
 		if ($value) {
 			$this->jsCodes ["draggable"]=new Draggable();

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

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 
 use Ajax\bootstrap\html\base\CssSize;
 use Ajax\JsUtils;
-
 use Ajax\bootstrap\html\base\HtmlBsDoubleElement;
 
 /**

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

Ajax/bootstrap/html/HtmlAccordion.php

Doc Comments +6 added lines

@@ -14,6 +14,9 @@ 
  */
 class HtmlAccordion extends HtmlBsDoubleElement {
 
+	/**
+	 * @param string $identifier
+	 */
 	public function __construct($identifier, $tagName="div") {
 		parent::__construct($identifier, $tagName);
 		$this->setClass("panel-group");
@@ -22,6 +25,9 @@ 
 		$this->content=array ();
 	}
 
+	/**
+	 * @param string $title
+	 */
 	public function addPanel($title, $content) {
 		$nb=sizeof($this->content)+1;
 		$panel=new HtmlPanel("panel-".$this->identifier."-".$nb);

Indentation +8 added lines, -8 removed lines

@@ -50,14 +50,14 @@
 	}
 
 		/**
-	 * render the content of $controller::$action and set the response to a new panel
-	 * @param JsUtils $js
-	 * @param string $title The panel title
-	 * @param Controller $initialController
-	 * @param string $controller a Phalcon controller
-	 * @param string $action a Phalcon action
-	 * @param array $params
-	 */
+		 * render the content of $controller::$action and set the response to a new panel
+		 * @param JsUtils $js
+		 * @param string $title The panel title
+		 * @param Controller $initialController
+		 * @param string $controller a Phalcon controller
+		 * @param string $action a Phalcon action
+		 * @param array $params
+		 */
 	public function forwardPanel(JsUtils $js,$title,$initialController,$controller,$action,$params=array()){
 		return $this->addPanel($title, $js->forward($initialController, $controller, $action,$params));
 	}

Spacing +6 added lines, -6 removed lines

@@ -19,7 +19,7 @@ 
 		$this->setClass("panel-group");
 		$this->setRole("tablist");
 		$this->setProperty("aria-multiselectable", "true");
-		$this->content=array ();
+		$this->content=array();
 	}
 
 	public function addPanel($title, $content) {
@@ -45,8 +45,8 @@ 
 	 * @param string $viewName
 	 * @param $params The parameters to pass to the view
 	 */
-	public function renderViewPanel(JsUtils $js,$title,$initialController, $viewName, $params=array()) {
-		return $this->addPanel($title, $js->renderContent($initialController, $viewName,$params));
+	public function renderViewPanel(JsUtils $js, $title, $initialController, $viewName, $params=array()) {
+		return $this->addPanel($title, $js->renderContent($initialController, $viewName, $params));
 	}
 
 		/**
@@ -58,12 +58,12 @@ 
 	 * @param string $action a Phalcon action
 	 * @param array $params
 	 */
-	public function forwardPanel(JsUtils $js,$title,$initialController,$controller,$action,$params=array()){
-		return $this->addPanel($title, $js->forward($initialController, $controller, $action,$params));
+	public function forwardPanel(JsUtils $js, $title, $initialController, $controller, $action, $params=array()) {
+		return $this->addPanel($title, $js->forward($initialController, $controller, $action, $params));
 	}
 
 	public function run(JsUtils $js) {
-		foreach ( $this->content as $content ) {
+		foreach ($this->content as $content) {
 			$content->run($js);
 		}
 	}

Braces +3 added lines, -2 removed lines

@@ -69,7 +69,8 @@
 	}
 
 	public function getPanel($index) {
-		if ($index<sizeof($this->content))
-			return $this->content [$index];
+		if ($index<sizeof($this->content)) {
+					return $this->content [$index];
+		}
 	}
 }
\ No newline at end of file

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/HtmlCarousel.php

Doc Comments +4 added lines, -1 removed lines

@@ -23,6 +23,9 @@ 
 	protected $_base="";
 	protected $_glyphs=array ();
 
+	/**
+	 * @param string $identifier
+	 */
 	public function __construct($identifier, $images=NULL) {
 		parent::__construct($identifier);
 		$this->_template=include 'templates/tplCarousel.php';
@@ -61,7 +64,7 @@ 
 	 *
 	 * @param string $caption
 	 * @param string $sens
-	 * @return HtmlCarouselControl|string
+	 * @return HtmlCarouselControl
 	 */
 	private function createControl($caption="next", $sens="left") {
 		$control=new HtmlCarouselControl($sens."-ctrl-".$this->identifier);

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 use Ajax\bootstrap\html\base\CssRef;
 use Ajax\bootstrap\html\base\HtmlBsDoubleElement;
 use Ajax\bootstrap\html\content\HtmlCarouselItem;
-
 use Ajax\common\html\BaseHtml;
 /**
  * Composant Twitter Bootstrap Carousel

Spacing +8 added lines, -8 removed lines

@@ -16,12 +16,12 @@ 
  * @version 1.001
  */
 class HtmlCarousel extends BaseHtml {
-	protected $indicators=array ();
-	protected $slides=array ();
+	protected $indicators=array();
+	protected $slides=array();
 	protected $leftControl="";
 	protected $rightControl="";
 	protected $_base="";
-	protected $_glyphs=array ();
+	protected $_glyphs=array();
 
 	public function __construct($identifier, $images=NULL) {
 		parent::__construct($identifier);
@@ -38,7 +38,7 @@ 
 	}
 
 	public function setBase($_base) {
-		foreach ($this->slides as $slide){
+		foreach ($this->slides as $slide) {
 			$imgSrc=$slide->getImageSrc();
 			$slide->setImageSrc(str_replace($this->_base.$imgSrc, $_base.$imgSrc, $imgSrc));
 		}
@@ -93,9 +93,9 @@ 
 	}
 
 	public function addImage($imageSrc, $imageAlt="", $caption=NULL, $description=NULL) {
-		if(is_array($imageSrc)){
-			$this->addImage($imageSrc[0],@$imageSrc[1],@$imageSrc[2],@$imageSrc[3]);
-		}else{
+		if (is_array($imageSrc)) {
+			$this->addImage($imageSrc[0], @$imageSrc[1], @$imageSrc[2], @$imageSrc[3]);
+		} else {
 			$image=new HtmlCarouselItem("item-".$this->identifier);
 			$image->setImageSrc($this->_base.$imageSrc);
 			$image->setImageAlt($imageAlt);
@@ -118,7 +118,7 @@ 
 	 */
 	public function fromArray($array) {
 		if (is_array($array) && sizeof($array)>0) {
-			foreach ( $array as $value ) {
+			foreach ($array as $value) {
 				if (is_array($value)) {
 					$this->addImage($value ["src"], @$value ["alt"], @$value ["caption"], @$value ["description"]);
 				} else {

Braces +10 added lines, -7 removed lines

@@ -75,27 +75,30 @@
 	}
 
 	private function getGlyph($sens="left") {
-		if (array_key_exists($sens, $this->_glyphs))
-			return $this->_glyphs [$sens];
+		if (array_key_exists($sens, $this->_glyphs)) {
+					return $this->_glyphs [$sens];
+		}
 		return "glyphicon-chevron-".$sens;
 	}
 
 	public function setRightGlyph($glyphicon) {
 		$glyphs=CssRef::glyphIcons();
-		if (array_search($glyphicon, $glyphs)!==false)
-			$this->_glyphs ["right"]=$glyphicon;
+		if (array_search($glyphicon, $glyphs)!==false) {
+					$this->_glyphs ["right"]=$glyphicon;
+		}
 	}
 
 	public function setLeftGlyph($glyphicon) {
 		$glyphs=CssRef::glyphIcons();
-		if (array_search($glyphicon, $glyphs)!==false)
-			$this->_glyphs ["left"]=$glyphicon;
+		if (array_search($glyphicon, $glyphs)!==false) {
+					$this->_glyphs ["left"]=$glyphicon;
+		}
 	}
 
 	public function addImage($imageSrc, $imageAlt="", $caption=NULL, $description=NULL) {
 		if(is_array($imageSrc)){
 			$this->addImage($imageSrc[0],@$imageSrc[1],@$imageSrc[2],@$imageSrc[3]);
-		}else{
+		} else{
 			$image=new HtmlCarouselItem("item-".$this->identifier);
 			$image->setImageSrc($this->_base.$imageSrc);
 			$image->setImageAlt($imageAlt);

Ajax/bootstrap/html/HtmlDropdown.php

Doc Comments +12 added lines, -1 removed lines

@@ -22,6 +22,7 @@ 
 	/**
 	 *
 	 * @param string $identifier the id
+	 * @param string $cssStyle
 	 */
 	public function __construct($identifier, $value="", $items=array(), $cssStyle=null, $onClick=null) {
 		parent::__construct($identifier, "", $cssStyle, $onClick);
@@ -45,7 +46,7 @@ 
 	/**
 	 * define the button style
 	 * avaible values : "btn-default","btn-primary","btn-success","btn-info","btn-warning","btn-danger"
-	 * @param string|int $cssStyle
+	 * @param string $cssStyle
 	 * @return \Ajax\bootstrap\html\HtmlDropdown default : "btn-default"
 	 */
 	public function setStyle($cssStyle) {
@@ -149,10 +150,16 @@ 
 		return $this->items [$index];
 	}
 
+	/**
+	 * @param string $value
+	 */
 	public function setBtnClass($value) {
 		$this->class=$value;
 	}
 
+	/**
+	 * @param string $value
+	 */
 	public function setMClass($value) {
 		$this->mClass=$value;
 	}
@@ -183,6 +190,7 @@ 
 	/**
 	 * Sets the tagName's dropdown
 	 * @see \Ajax\bootstrap\html\BaseHtml::setTagName()
+	 * @param string $tagName
 	 */
 	public function setTagName($tagName) {
 		if ($tagName=="button")
@@ -194,6 +202,9 @@ 
 		return $this->compile();
 	}
 
+	/**
+	 * @param string $btnCaption
+	 */
 	public function setBtnCaption($btnCaption) {
 		$this->btnCaption=$btnCaption;
 		return $this;

Spacing +9 added lines, -9 removed lines

@@ -17,7 +17,7 @@ 
 	protected $class="dropdown-toggle";
 	protected $mClass="dropdown";
 	protected $mTagName="div";
-	protected $items=array ();
+	protected $items=array();
 
 	/**
 	 *
@@ -87,9 +87,9 @@ 
 	 * @return HtmlDropdownItem
 	 */
 	public function addItem($caption, $href="#") {
-		if($caption instanceof HtmlDropdownItem){
+		if ($caption instanceof HtmlDropdownItem) {
 			$item=$caption;
-		}else{
+		} else {
 			$iid=$this->getItemsCount()+1;
 			$item=new HtmlDropdownItem($this->identifier."-dropdown-item-".$iid);
 			$item->setCaption($caption)->setHref($href);
@@ -109,7 +109,7 @@ 
 	public function addItems($items) {
 		$iid=$this->getItemsCount()+1;
 		if (is_array($items)) {
-			foreach ( $items as $item ) {
+			foreach ($items as $item) {
 				if (is_string($item)) {
 					$this->addItem($item);
 				} else if (is_array($item)) {
@@ -136,7 +136,7 @@ 
 	}
 
 	public function setItems($items) {
-		$this->items=array ();
+		$this->items=array();
 		$this->addItems($items);
 	}
 
@@ -171,7 +171,7 @@ 
 	 */
 	public function run(JsUtils $js) {
 		if ($this->getProperty("role")==="nav") {
-			foreach ( $this->items as $dropdownItem ) {
+			foreach ($this->items as $dropdownItem) {
 				$dropdownItem->runNav($js);
 			}
 		}
@@ -263,9 +263,9 @@ 
 	/* (non-PHPdoc)
 	 * @see \Ajax\bootstrap\html\base\BaseHtml::on()
 	 */
-	public function on($event, $jsCode, $stopPropagation = false, $preventDefault = false) {
-		foreach ($this->items as $item){
-			$item->on($event, $jsCode,$stopPropagation,$preventDefault);
+	public function on($event, $jsCode, $stopPropagation=false, $preventDefault=false) {
+		foreach ($this->items as $item) {
+			$item->on($event, $jsCode, $stopPropagation, $preventDefault);
 		}
 	}
 

Braces +12 added lines, -9 removed lines

@@ -89,7 +89,7 @@ 
 	public function addItem($caption, $href="#") {
 		if($caption instanceof HtmlDropdownItem){
 			$item=$caption;
-		}else{
+		} else{
 			$iid=$this->getItemsCount()+1;
 			$item=new HtmlDropdownItem($this->identifier."-dropdown-item-".$iid);
 			$item->setCaption($caption)->setHref($href);
@@ -129,10 +129,11 @@ 
 	 * @see BaseHtml::fromArray()
 	 */
 	public function fromArray($array) {
-		if (array_keys($array)!==range(0, count($array)-1))
-			return parent::fromArray($array);
-		else
-			return $this->addItems($array);
+		if (array_keys($array)!==range(0, count($array)-1)) {
+					return parent::fromArray($array);
+		} else {
+					return $this->addItems($array);
+		}
 	}
 
 	public function setItems($items) {
@@ -185,8 +186,9 @@ 
 	 * @see \Ajax\bootstrap\html\BaseHtml::setTagName()
 	 */
 	public function setTagName($tagName) {
-		if ($tagName=="button")
-			$this->class="btn dropdown-toggle";
+		if ($tagName=="button") {
+					$this->class="btn dropdown-toggle";
+		}
 		return parent::setTagName($tagName);
 	}
 
@@ -204,8 +206,9 @@ 
 	}
 
 	public function setAlignment($alignment) {
-		if (is_int($alignment))
-			$alignment="dropdown-menu-".CssRef::alignment()[$alignment];
+		if (is_int($alignment)) {
+					$alignment="dropdown-menu-".CssRef::alignment()[$alignment];
+		}
 		return $this->addToMemberCtrl($this->class, $alignment, CssRef::alignment());
 	}
 

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

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;
 
 class HtmlForm extends HtmlBsDoubleElement {

Spacing +8 added lines, -8 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");
@@ -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 @@ 
 
 	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;
@@ -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;
 			}

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