jcheron / phalcon-jquery

Ajax/bootstrap/html/HtmlModal.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use Phalcon\Mvc\View;
 use Ajax\JsUtils;
 use Ajax\bootstrap\html\base\BaseHtml;
-use Phalcon\Mvc\Dispatcher;
 use Phalcon\Mvc\Controller;
 
 /**

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

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

Ajax/bootstrap/html/HtmlAccordion.php

Braces +3 added lines, -2 removed lines

@@ -77,7 +77,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

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

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

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

Braces +11 added lines, -8 removed lines

@@ -24,17 +24,19 @@ 
 
 	public function attachTo($identifier) {
 		$this->element->setProperty("aria-controls", $identifier);
-		if ($this->element->getTagName()==="a")
-			$this->element->setProperty("href", "#".$identifier);
-		else
-			$this->element->setProperty("data-target", "#".$identifier);
+		if ($this->element->getTagName()==="a") {
+					$this->element->setProperty("href", "#".$identifier);
+		} else {
+					$this->element->setProperty("data-target", "#".$identifier);
+		}
 	}
 
 	public function getAttachedZone() {
 		$id=$this->element->getProperty("aria-controls");
-		if (!isset($id))
-			if ($this->element->getTagName()==="a")
+		if (!isset($id)) {
+					if ($this->element->getTagName()==="a")
 				$id=$this->element->getProperty("href");
+		}
 		if (!isset($id)||$id==="#") {
 			$id="collapse-".$this->element->getIdentifier();
 			$this->attachTo($id);
@@ -49,8 +51,9 @@ 
 	}
 
 	public function createCollapsedZone($content="", $attachTo=NULL) {
-		if (isset($attachTo))
-			$this->attachTo($attachTo);
+		if (isset($attachTo)) {
+					$this->attachTo($attachTo);
+		}
 		$collapsedZone=new HtmlDoubleElement($this->getAttachedZone());
 		$collapsedZone->setProperty("class", "collapse");
 		$collapsedZone->setContent($content);

Ajax/bootstrap/html/HtmlDropdown.php

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