jcheron / phalcon-jquery

Ajax/bootstrap/html/HtmlButtongroups.php

Doc Comments +3 added lines

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

Braces +20 added lines, -16 removed lines

@@ -44,8 +44,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) {
@@ -70,16 +71,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;
@@ -110,8 +113,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>');
@@ -126,9 +130,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

@@ -11,6 +11,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");
@@ -56,7 +61,7 @@ 
 
 	/**
 	 * return the Buttongroups at position $index
-	 * @return \Ajax\bootstrap\html\HtmlButtongroups
+	 * @return HtmlButton
 	 */
 	public function getGroup($index) {
 		return parent::getElement($index);

Braces +6 added lines, -5 removed lines

@@ -45,9 +45,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;
 		}
@@ -65,8 +65,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);

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/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 HtmlDoubleElement($identifier);
 		$group->setTagName("div");
 		$group->setClass("form-group");

Ajax/bootstrap/html/HtmlModal.php

Doc Comments +1 added lines, -1 removed lines

@@ -208,7 +208,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;

Braces +11 added lines, -8 removed lines

@@ -151,18 +151,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() {
@@ -188,8 +190,9 @@ 
 
 	public function jsHideButton($index) {
 		$btn=$this->getButton($index);
-		if ($btn)
-			return "$('#".$btn->getIdentifier()."').hide();";
+		if ($btn) {
+					return "$('#".$btn->getIdentifier()."').hide();";
+		}
 	}
 
 	/**

Ajax/bootstrap/html/HtmlPanel.php

Doc Comments +8 added lines

@@ -21,6 +21,11 @@ 
 	protected $collapseEnd;
 	protected $_showOnStartup;
 
+	/**
+	 * @param string $identifier
+	 * @param string $header
+	 * @param string $footer
+	 */
 	public function __construct($identifier, $content=NULL, $header=NULL, $footer=NULL) {
 		parent::__construct($identifier, "div");
 		$this->_template=include 'templates/tplPanel.php';
@@ -109,6 +114,9 @@ 
 		return $this->_bsComponent;
 	}
 
+	/**
+	 * @param boolean $_collapsable
+	 */
 	public function setCollapsable($_collapsable) {
 		$this->_collapsable=$_collapsable;
 		if ($_collapsable) {

Braces +6 added lines, -4 removed lines

@@ -89,8 +89,9 @@ 
 	 * @return \Ajax\bootstrap\html\HtmlPanel default : "panel-default"
 	 */
 	public function setStyle($cssStyle) {
-		if (!PhalconUtils::startsWith($cssStyle, "panel"))
-			$cssStyle="panel".$cssStyle;
+		if (!PhalconUtils::startsWith($cssStyle, "panel")) {
+					$cssStyle="panel".$cssStyle;
+		}
 		return $this->addToPropertyCtrl("class", $cssStyle, CssRef::Styles("panel"));
 	}
 
@@ -132,7 +133,8 @@ 
 	 * @return $this default : false
 	 */
 	public function show($value) {
-		if ($this->_collapsable)
-			$this->_showOnStartup=$value;
+		if ($this->_collapsable) {
+					$this->_showOnStartup=$value;
+		}
 	}
 }
\ No newline at end of file

Ajax/bootstrap/html/HtmlTabs.php

Doc Comments +3 added lines

@@ -20,6 +20,9 @@
 	protected $_tabsType="tabs";
 	protected $stacked="";
 
+	/**
+	 * @param string $identifier
+	 */
 	public function __construct($identifier, $tagName="ul") {
 		parent::__construct($identifier, $tagName);
 		$this->_template="<%tagName% %properties%>%tabs%</%tagName%>%content%";

Braces +17 added lines, -12 removed lines

@@ -35,8 +35,9 @@ 
 					$tab
 			);
 			array_splice($this->tabs, $index, 0, $inserted);
-		} else
-			$this->tabs []=$tab;
+		} else {
+					$this->tabs []=$tab;
+		}
 	}
 
 	public function setActive($index){
@@ -116,9 +117,9 @@ 
 	public function createTabContents() {
 		$tabContent=new HtmlTabContent("tabcontent-".$this->identifier);
 		foreach ( $this->tabs as $tab ) {
-			if ($tab instanceof HtmlTabItem)
-				$tabContent->addTabItem($tab->getHref());
-			elseif ($tab instanceof HtmlDropdown) {
+			if ($tab instanceof HtmlTabItem) {
+							$tabContent->addTabItem($tab->getHref());
+			} elseif ($tab instanceof HtmlDropdown) {
 				foreach ( $tab->getItems() as $dropdownItem ) {
 					$tabContent->addTabItem($dropdownItem->getHref());
 				}
@@ -137,8 +138,9 @@ 
 
 	public function setContentToTab($index, $text) {
 		$tabContentItem=$this->content->getTabItem($index);
-		if (isset($tabContentItem))
-			$tabContentItem->setContent($text);
+		if (isset($tabContentItem)) {
+					$tabContentItem->setContent($text);
+		}
 	}
 
 	public function countTabs() {
@@ -146,8 +148,9 @@ 
 	}
 
 	public function getTabItem($index) {
-		if ($index<sizeof($this->content->get))
-			return $this->content;
+		if ($index<sizeof($this->content->get)) {
+					return $this->content;
+		}
 	}
 
 	public function fadeEffect() {
@@ -168,9 +171,11 @@ 
 	}
 
 	public function setStacked($stacked=true){
-		if($stacked)
-			$this->stacked="nav-stacked";
-		else $this->stacked="";
+		if($stacked) {
+					$this->stacked="nav-stacked";
+		} else {
+			$this->stacked="";
+		}
 	}
 
 	/* (non-PHPdoc)

Ajax/JsUtils.php

Doc Comments +6 added lines, -3 removed lines

@@ -676,6 +676,9 @@ 
 		return $this->_create_json($json_result, $match_array_type);
 	}
 
+	/**
+	 * @param boolean $match_array_type
+	 */
 	private function _create_json($json_result, $match_array_type) {
 		$json=array ();
 		$_is_assoc=TRUE;
@@ -699,7 +702,7 @@ 
 	 * Checks for an associative array
 	 *
 	 * @param type
-	 * @return type
+	 * @return boolean
 	 */
 	public function _is_associative_array($arr) {
 		foreach ( array_keys($arr) as $key => $val ) {
@@ -939,7 +942,7 @@ 
 	 * @param string $jqueryCall the JQuery callback
 	 * @param mixed $param array or string parameters
 	 * @param string $jsCallback javascript code to execute after the jquery call
-	 * @return mixed
+	 * @return string
 	 */
 	public function doJQuery($element, $jqueryCall, $param="", $jsCallback="") {
 		return $this->js->_doJQuery($element, $jqueryCall, $param, $jsCallback, true);
@@ -951,7 +954,7 @@ 
 	 * @param string $jqueryCall the JQuery callback
 	 * @param mixed $param array or string parameters
 	 * @param string $jsCallback javascript code to execute after the jquery call
-	 * @return mixed
+	 * @return string
 	 */
 	public function doJQueryDeferred($element, $jqueryCall, $param="", $jsCallback="") {
 		return $this->js->_doJQuery($element, $jqueryCall, $param, $jsCallback, false);

Braces +3 added lines, -2 removed lines

@@ -45,8 +45,9 @@
 
 	protected function _setDi($di) {
 		$this->_di=$di;
-		if ($this->js!=null&&$di!=null)
-			$this->js->setDi($di);
+		if ($this->js!=null&&$di!=null) {
+					$this->js->setDi($di);
+		}
 	}
 
 	/**

Ajax/bootstrap/components/Tabs.php

Braces +15 added lines, -10 removed lines

@@ -35,8 +35,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) {
@@ -52,8 +53,9 @@ 
 	 */
 	public function onShow($index, $jsCode) {
 		$tab=$this->getTab($index);
-		if (isset($tab))
-			return $tab->onShow($jsCode);
+		if (isset($tab)) {
+					return $tab->onShow($jsCode);
+		}
 	}
 
 	/**
@@ -64,8 +66,9 @@ 
 	 */
 	public function onShown($index, $jsCode) {
 		$tab=$this->getTab($index);
-		if (isset($tab))
-			return $tab->onShown($jsCode);
+		if (isset($tab)) {
+					return $tab->onShown($jsCode);
+		}
 	}
 
 	/**
@@ -76,8 +79,9 @@ 
 	 */
 	public function onHide($index, $jsCode) {
 		$tab=$this->getTab($index);
-		if (isset($tab))
-			return $tab->onShow($jsCode);
+		if (isset($tab)) {
+					return $tab->onShow($jsCode);
+		}
 	}
 
 	/**
@@ -88,7 +92,8 @@ 
 	 */
 	public function onHidden($index, $jsCode) {
 		$tab=$this->getTab($index);
-		if (isset($tab))
-			return $tab->onShow($jsCode);
+		if (isset($tab)) {
+					return $tab->onShow($jsCode);
+		}
 	}
 }
\ No newline at end of file