jcheron / phalcon-jquery

Ajax/bootstrap/html/HtmlPagination.php

Doc Comments +8 added lines

@@ -42,6 +42,7 @@ 
 
 	/**
 	 * @param string $identifier
+	 * @param integer $active
 	 */
 	public function __construct($identifier,$from=1,$to=1,$active=NULL,$countVisible=NULL){
 		parent::__construct($identifier,"ul");
@@ -57,6 +58,9 @@ 
 		$this->createContent();
 	}
 
+	/**
+	 * @param integer $num
+	 */
 	private function createElement($num,$content,$disabled=false,$current=false){
 		$count=sizeof($this->content)+1;
 		$elem=new HtmlBsDoubleElement("li-".$this->identifier."-".$count,"li");
@@ -168,6 +172,10 @@ 
 	public function getActive() {
 		return $this->active;
 	}
+
+	/**
+	 * @param integer $active
+	 */
 	public function setActive($active) {
 		$this->active = $active;
 		$this->createContent();

Spacing +37 added lines, -37 removed lines

@@ -43,70 +43,70 @@ 
 	/**
 	 * @param string $identifier
 	 */
-	public function __construct($identifier,$from=1,$to=1,$active=NULL,$countVisible=NULL){
-		parent::__construct($identifier,"ul");
+	public function __construct($identifier, $from=1, $to=1, $active=NULL, $countVisible=NULL) {
+		parent::__construct($identifier, "ul");
 		$this->setProperty("class", "pagination");
 		$this->active=$active;
 		$this->from=$from;
 		$this->to=$to;
 		$this->urlMask="%page%";
-		if(!isset($countVisible))
+		if (!isset($countVisible))
 			$this->countVisible=$to-$from+1;
 		else
 			$this->countVisible=$countVisible;
 		$this->createContent();
 	}
 
-	private function createElement($num,$content,$disabled=false,$current=false){
+	private function createElement($num, $content, $disabled=false, $current=false) {
 		$count=sizeof($this->content)+1;
-		$elem=new HtmlBsDoubleElement("li-".$this->identifier."-".$count,"li");
-		if($disabled){
+		$elem=new HtmlBsDoubleElement("li-".$this->identifier."-".$count, "li");
+		if ($disabled) {
 			$elem->setProperty("class", "disabled");
 		}
-		if($current){
+		if ($current) {
 			$content.="<span class='sr-only'>(current)</span>";
 			$elem->setProperty("class", "active");
 		}
-		if(!$disabled){
+		if (!$disabled) {
 			$url=$this->getUrl($num);
-			$href=new HtmlLink("a-".$this->identifier."-".$count,$url,$content);
+			$href=new HtmlLink("a-".$this->identifier."-".$count, $url, $content);
 			$href->setProperty($this->attr, $url);
 			$elem->setContent($href);
-		}else{
+		}else {
 			$elem->setContent($content);
 		}
 		$this->content[]=$elem;
 		return $this;
 	}
 
-	protected function createContent(){
+	protected function createContent() {
 		$this->content=array();
-		$this->createElement($this->active-1,"<span aria-hidden='true'>&laquo;</span>",$this->active===1);
+		$this->createElement($this->active-1, "<span aria-hidden='true'>&laquo;</span>", $this->active===1);
 		$start=$this->getStart();
-		$end=min($start+$this->countVisible-1,$this->to);
-		for($index=$start;$index<=$end;$index++){
-			$this->createElement($index,$index,false,$index===$this->active);
+		$end=min($start+$this->countVisible-1, $this->to);
+		for ($index=$start; $index<=$end; $index++) {
+			$this->createElement($index, $index, false, $index===$this->active);
 		}
-		$this->createElement($this->active+1,"<span aria-hidden='true'>&raquo;</span>",$this->active===$this->to);
+		$this->createElement($this->active+1, "<span aria-hidden='true'>&raquo;</span>", $this->active===$this->to);
 	}
 
-	protected function half(){
-		return (int)($this->countVisible/2);
+	protected function half() {
+		return (int) ($this->countVisible / 2);
 	}
 
-	protected function getStart(){
+	protected function getStart() {
 		$result=1;
-		if($this->countVisible!==$this->to-$this->from+1){
-			$result=max($this->active-$this->half(),$result);
+		if ($this->countVisible!==$this->to-$this->from+1) {
+			$result=max($this->active-$this->half(), $result);
 		}
 		return $result;
 	}
 
 	public function _addEvent($event, $jsCode) {
-		foreach ($this->content as $li){
+		foreach ($this->content as $li) {
 			$content=$li->getContent();
-			if($content instanceof BaseHtml)
-				$content->_addEvent($event,$jsCode);
+			if ($content instanceof BaseHtml)
+				$content->_addEvent($event, $jsCode);
 		}
 	}
 	/**
@@ -114,15 +114,15 @@ 
 	 * @param Dispatcher $dispatcher the request dispatcher
 	 * @return \Ajax\bootstrap\html\HtmlPagination
 	 */
-	public function fromDispatcher($dispatcher){
-		$items=array($dispatcher->getControllerName(),$dispatcher->getActionName());
-		$items=array_merge($items,$dispatcher->getParams());
+	public function fromDispatcher($dispatcher) {
+		$items=array($dispatcher->getControllerName(), $dispatcher->getActionName());
+		$items=array_merge($items, $dispatcher->getParams());
 		$url=implode("/", $items);
-		if($this->urlMask==="%page%"){
+		if ($this->urlMask==="%page%") {
 			$this->urlMask=preg_replace("/[0-9]/", "%page%", $url);
 		}
-		for($index=$this->from;$index<=$this->to;$index++){
-			if($this->getUrl($index)==$url){
+		for ($index=$this->from; $index<=$this->to; $index++) {
+			if ($this->getUrl($index)==$url) {
 				$this->setActive($index);
 				break;
 			}
@@ -130,7 +130,7 @@ 
 		return $this;
 	}
 
-	public function getUrl($index){
+	public function getUrl($index) {
 		return str_ireplace("%page%", $index, $this->urlMask);
 	}
 
@@ -144,7 +144,7 @@ 
 		if (is_int($size)) {
 			return $this->addToPropertyUnique("class", CssRef::sizes("pagination")[$size], CssRef::sizes("pagination"));
 		}
-		if(!PhalconUtils::startsWith($size, "pagination-") && $size!=="")
+		if (!PhalconUtils::startsWith($size, "pagination-") && $size!=="")
 			$size="pagination-".$size;
 		return $this->addToPropertyCtrl("class", $size, CssRef::sizes("pagination"));
 	}
@@ -153,7 +153,7 @@ 
 		return $this->from;
 	}
 	public function setFrom($from) {
-		$this->from = $from;
+		$this->from=$from;
 		$this->createContent();
 		return $this;
 	}
@@ -161,7 +161,7 @@ 
 		return $this->to;
 	}
 	public function setTo($to) {
-		$this->to = $to;
+		$this->to=$to;
 		$this->createContent();
 		return $this;
 	}
@@ -169,7 +169,7 @@ 
 		return $this->active;
 	}
 	public function setActive($active) {
-		$this->active = $active;
+		$this->active=$active;
 		$this->createContent();
 		return $this;
 	}
@@ -177,7 +177,7 @@ 
 		return $this->urlMask;
 	}
 	public function setUrlMask($urlMask) {
-		$this->urlMask = $urlMask;
+		$this->urlMask=$urlMask;
 		$this->createContent();
 		return $this;
 	}
@@ -185,7 +185,7 @@ 
 		return $this->countVisible;
 	}
 	public function setCountVisible($countVisible) {
-		$this->countVisible = $countVisible;
+		$this->countVisible=$countVisible;
 		$this->createContent();
 		return $this;
 	}

Ajax/common/html/BaseHtml.php

Doc Comments +31 added lines

@@ -48,6 +48,9 @@ 
 		return $this;
 	}
 
+	/**
+	 * @param string $name
+	 */
 	public function getProperty($name) {
 		if (array_key_exists($name, $this->properties))
 			return $this->properties [$name];
@@ -128,6 +131,10 @@ 
 		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)) {
@@ -152,6 +159,9 @@ 
 		return $this->addToProperty($name, $value);
 	}
 
+	/**
+	 * @param string $name
+	 */
 	public function addToPropertyCtrl($name, $value, $typeCtrl) {
 		// if($this->ctrl($name, $value, $typeCtrl)===true){
 		return $this->addToPropertyUnique($name, $value, $typeCtrl);
@@ -219,6 +229,9 @@ 
 
 	}
 
+	/**
+	 * @param string $before
+	 */
 	public function wrap($before, $after="") {
 		if(isset($before)){
 			$this->wrapBefore.=$before;
@@ -287,6 +300,9 @@ 
 		}
 	}
 
+	/**
+	 * @param string $operation
+	 */
 	public function _ajaxOn($operation, $event, $url, $responseElement="", $parameters=array()) {
 		$params=array (
 				"url" => $url,
@@ -297,14 +313,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);
@@ -314,6 +339,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);
@@ -363,6 +391,9 @@ 
 		return $value;
 	}
 
+	/**
+	 * @param string $jqueryCall
+	 */
 	public function jsDoJquery($jqueryCall, $param=""){
 		return "$('#".$this->identifier."').".$jqueryCall."(".$this->_prep_value($param).");";
 	}

Ajax/semantic/html/HtmlIcon.php

Doc Comments +1 added lines

@@ -40,6 +40,7 @@
 	/**
 	 * {@inheritDoc}
 	 * @see \Ajax\common\html\HtmlSingleElement::setSize()
+	 * @param string $size
 	 */
 	public function setSize($size) {
 		$this->setMemberCtrl($this->size, $size,IconSize::getConstants());

Spacing +15 added lines, -15 removed lines

@@ -17,7 +17,7 @@ 
 	protected $attributes;
 	protected $color;
 
-	public function __construct($identifier,$icon) {
+	public function __construct($identifier, $icon) {
 		parent::__construct($identifier, "i");
 		$this->icon=$icon;
 		$this->_template='<i class="%icon% icon %size% %attributes% %color%"></i>';
@@ -42,11 +42,11 @@ 
 	 * @see \Ajax\common\html\HtmlSingleElement::setSize()
 	 */
 	public function setSize($size) {
-		$this->setMemberCtrl($this->size, $size,IconSize::getConstants());
+		$this->setMemberCtrl($this->size, $size, IconSize::getConstants());
 		return $this;
 	}
 
-	public function setDisabled(){
+	public function setDisabled() {
 		return $this->addToMember($this->attributes, "disabled");
 	}
 
@@ -54,7 +54,7 @@ 
 	 *  Icon used as a simple loader
 	 * @return \Ajax\semantic\html\HtmlIcon
 	 */
-	public function asLoader(){
+	public function asLoader() {
 		return $this->addToMember($this->attributes, "loading");
 	}
 
@@ -62,7 +62,7 @@ 
 	 * An icon can be fitted, without any space to the left or right of it.
 	 * @return \Ajax\semantic\html\HtmlIcon
 	 */
-	public function setFitted(){
+	public function setFitted() {
 		return $this->addToMember($this->attributes, "fitted");
 	}
 
@@ -70,7 +70,7 @@ 
 	 * @param string $sens horizontally or vertically
 	 * @return \Ajax\semantic\html\HtmlIcon
 	 */
-	public function setFlipped($sens="horizontally"){
+	public function setFlipped($sens="horizontally") {
 		return $this->addToMember($this->attributes, "flipped ".$sens);
 	}
 
@@ -78,7 +78,7 @@ 
 	 * @param string $sens clockwise or counterclockwise
 	 * @return \Ajax\semantic\html\HtmlIcon
 	 */
-	public function setRotated($sens="clockwise"){
+	public function setRotated($sens="clockwise") {
 		return $this->addToMember($this->attributes, "rotated ".$sens);
 	}
 
@@ -86,13 +86,13 @@ 
 	 * icon formatted as a link
 	 * @return \Ajax\semantic\html\HtmlIcon
 	 */
-	public function asLink(){
+	public function asLink() {
 		return $this->addToMember($this->attributes, "link");
 	}
 
-	public function setCircular($inverted=false){
+	public function setCircular($inverted=false) {
 		$invertedStr="";
-		if($inverted)
+		if ($inverted)
 			$invertedStr=" inverted";
 		return $this->addToMember($this->attributes, "circular".$invertedStr);
 	}
@@ -100,7 +100,7 @@ 
 	/**
 	 * @return \Ajax\semantic\html\HtmlIcon
 	 */
-	public function setInverted(){
+	public function setInverted() {
 		return $this->addToMember($this->attributes, "inverted");
 	}
 
@@ -108,9 +108,9 @@ 
 	 * @param string $inverted
 	 * @return \Ajax\semantic\html\HtmlIcon
 	 */
-	public function setBordered($inverted=false){
+	public function setBordered($inverted=false) {
 		$invertedStr="";
-		if($inverted)
+		if ($inverted)
 			$invertedStr=" inverted";
 			return $this->addToMember($this->attributes, "bordered".$invertedStr);
 	}
@@ -119,14 +119,14 @@ 
 	 * @param string $color
 	 * @return \Ajax\semantic\html\HtmlIcon
 	 */
-	public function setColor($color){
+	public function setColor($color) {
 		return $this->setMemberCtrl($this->color, $color, Color::getConstants());
 	}
 
 	/**
 	 * @return \Ajax\semantic\html\HtmlIcon
 	 */
-	public function toCorner(){
+	public function toCorner() {
 		return $this->addToMember($this->attributes, "corner");
 	}
 }
\ No newline at end of file

Braces +6 added lines, -4 removed lines

@@ -92,8 +92,9 @@ 
 
 	public function setCircular($inverted=false){
 		$invertedStr="";
-		if($inverted)
-			$invertedStr=" inverted";
+		if($inverted) {
+					$invertedStr=" inverted";
+		}
 		return $this->addToMember($this->attributes, "circular".$invertedStr);
 	}
 
@@ -110,8 +111,9 @@ 
 	 */
 	public function setBordered($inverted=false){
 		$invertedStr="";
-		if($inverted)
-			$invertedStr=" inverted";
+		if($inverted) {
+					$invertedStr=" inverted";
+		}
 			return $this->addToMember($this->attributes, "bordered".$invertedStr);
 	}
 

Ajax/Semantic.php

Spacing +8 added lines, -8 removed lines

@@ -26,18 +26,18 @@
 		return $this->addHtmlComponent(new HtmlButton($identifier, $value, $cssStyle, $onClick));
 	}
 
-	public function htmlIcon($identifier,$icon){
+	public function htmlIcon($identifier, $icon) {
 		return $this->addHtmlComponent(new HtmlIcon($identifier, $icon));
 	}
 
-	public function htmlGroupIcons($identifier,$size="",$icons=array()){
-		$group=new HtmlGroupIcons($identifier,$size);
-		if(JArray::isAssociative($icons)){
-			foreach ($icons as $icon=>$size){
-				$group->addIcon($icon,$size);
+	public function htmlGroupIcons($identifier, $size="", $icons=array()) {
+		$group=new HtmlGroupIcons($identifier, $size);
+		if (JArray::isAssociative($icons)) {
+			foreach ($icons as $icon=>$size) {
+				$group->addIcon($icon, $size);
 			}
-		}else{
-			foreach ($icons as $icon){
+		}else {
+			foreach ($icons as $icon) {
 				$group->addIcon($icon);
 			}
 		}

Braces +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
 			foreach ($icons as $icon=>$size){
 				$group->addIcon($icon,$size);
 			}
-		}else{
+		} else{
 			foreach ($icons as $icon){
 				$group->addIcon($icon);
 			}

Ajax/bootstrap/html/HtmlProgressbar.php

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

Ajax/bootstrap/html/base/HtmlBsDoubleElement.php

Spacing +5 added lines, -5 removed lines

@@ -23,19 +23,19 @@
 		return $this;
 	}
 
-	public function addGlyph($glyphicon,$before=true){
+	public function addGlyph($glyphicon, $before=true) {
 		$glyph=new HtmlGlyphicon("");
 		$glyph->setGlyphicon($glyphicon);
-		$this->addContent($glyph,$before);
+		$this->addContent($glyph, $before);
 		return $this;
 	}
 
-	public function wrapContentWithGlyph($glyphBefore,$glyphAfter=""){
+	public function wrapContentWithGlyph($glyphBefore, $glyphAfter="") {
 		$before=HtmlGlyphicon::getGlyphicon($glyphBefore)." ";
 		$after="";
-		if($glyphAfter!==""){
+		if ($glyphAfter!=="") {
 			$after=" ".HtmlGlyphicon::getGlyphicon($glyphAfter);
 		}
-		return $this->wrapContent($before,$after);
+		return $this->wrapContent($before, $after);
 	}
 }
\ No newline at end of file

Ajax/bootstrap/html/HtmlBreadcrumbs.php

Spacing +38 added lines, -38 removed lines

@@ -41,15 +41,15 @@ 
 	 * @param boolean $autoActive sets the last element's class to <b>active</b> if true
 	 * @param function $hrefFunction the function who generates the href elements. default : function($e){return $e->getContent()}
 	 */
-	public function __construct($identifier,$elements=array(),$autoActive=true,$startIndex=0,$hrefFunction=NULL){
-		parent::__construct($identifier,"ol");
+	public function __construct($identifier, $elements=array(), $autoActive=true, $startIndex=0, $hrefFunction=NULL) {
+		parent::__construct($identifier, "ol");
 		$this->startIndex=$startIndex;
 		$this->setProperty("class", "breadcrumb");
 		$this->content=array();
 		$this->autoActive=$autoActive;
 		$this->absolutePaths;
-		$this->_hrefFunction=function ($e){return $e->getContent();};
-		if(isset($hrefFunction)){
+		$this->_hrefFunction=function($e) {return $e->getContent(); };
+		if (isset($hrefFunction)) {
 			$this->_hrefFunction=$hrefFunction;
 		}
 		$this->addElements($elements);
@@ -60,43 +60,43 @@ 
 	 * @param string $href
 	 * @return \Ajax\bootstrap\html\HtmlLink
 	 */
-	public function addElement($element,$href="",$glyph=NULL){
+	public function addElement($element, $href="", $glyph=NULL) {
 		$size=sizeof($this->content);
-		if(is_array($element)){
+		if (is_array($element)) {
 			$elm=new HtmlLink("lnk-".$this->identifier."-".$size);
 			$elm->fromArray($element);
-		}else if($element instanceof HtmlLink){
+		}else if ($element instanceof HtmlLink) {
 			$elm=$element;
-		}else{
-			$elm=new HtmlLink("lnk-".$this->identifier."-".$size,$href,$element);
-			if(isset($glyph)){
+		}else {
+			$elm=new HtmlLink("lnk-".$this->identifier."-".$size, $href, $element);
+			if (isset($glyph)) {
 				$elm->wrapContentWithGlyph($glyph);
 			}
 		}
-		$elm->wrap("<li>","</li>");
+		$elm->wrap("<li>", "</li>");
 		$this->content[]=$elm;
 		$elm->setProperty($this->attr, $this->getHref($size));
 		return $elm;
 	}
 
-	public function setActive($index=null){
-		if(!isset($index)){
+	public function setActive($index=null) {
+		if (!isset($index)) {
 			$index=sizeof($this->content)-1;
 		}
-		$li=new HtmlBsDoubleElement("","li");
+		$li=new HtmlBsDoubleElement("", "li");
 		$li->setClass("active");
 		$li->setContent($this->content[$index]->getContent());
 		$this->content[$index]=$li;
 	}
 
-	public function addElements($elements){
-		foreach ( $elements as $element ) {
+	public function addElements($elements) {
+		foreach ($elements as $element) {
 			$this->addElement($element);
 		}
 		return $this;
 	}
 
-	public function fromArray($array){
+	public function fromArray($array) {
 		$array=parent::fromArray($array);
 		$this->addElements($array);
 		return $array;
@@ -108,14 +108,14 @@ 
 	 * @param string $separator
 	 * @return string
 	 */
-	public function getHref($index=null,$separator="/"){
-		if(!isset($index)){
+	public function getHref($index=null, $separator="/") {
+		if (!isset($index)) {
 			$index=sizeof($this->content);
 		}
-		if($this->absolutePaths===true){
+		if ($this->absolutePaths===true) {
 			return $this->_hrefFunction($this->content[$index]);
-		}else{
-			return $this->root.implode($separator, array_slice(array_map(function($e){return $this->_hrefFunction($e);}, $this->content),$this->startIndex,$index+1));
+		}else {
+			return $this->root.implode($separator, array_slice(array_map(function($e) {return $this->_hrefFunction($e); }, $this->content), $this->startIndex, $index+1));
 		}
 	}
 
@@ -124,7 +124,7 @@ 
 	 * @see \Ajax\bootstrap\html\BaseHtml::compile()
 	 */
 	public function compile(JsUtils $js=NULL, View $view=NULL) {
-		if($this->autoActive){
+		if ($this->autoActive) {
 			$this->setActive();
 		}
 		return parent::compile($js, $view);
@@ -142,19 +142,19 @@ 
 	 * @see \Ajax\bootstrap\html\base\BaseHtml::on()
 	 */
 	public function on($event, $jsCode, $stopPropagation=false, $preventDefault=false) {
-		foreach ($this->content as $element){
-			$element->on($event,$jsCode,$stopPropagation,$preventDefault);
+		foreach ($this->content as $element) {
+			$element->on($event, $jsCode, $stopPropagation, $preventDefault);
 		}
 		return $this;
 	}
 
 	public function setAutoActive($autoActive) {
-		$this->autoActive = $autoActive;
+		$this->autoActive=$autoActive;
 		return $this;
 	}
 
 	public function _ajaxOn($operation, $event, $url, $responseElement="", $parameters=array()) {
-		foreach ($this->content as $element){
+		foreach ($this->content as $element) {
 			$element->_ajaxOn($operation, $event, $url, $responseElement, $parameters);
 		}
 		return $this;
@@ -167,18 +167,18 @@ 
 	 * @param string $attr the html attribute used to build the elements url
 	 * @return HtmlBreadcrumbs
 	 */
-	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(){
-		if($this->autoActive){
+	public function contentAsString() {
+		if ($this->autoActive) {
 			$this->setActive();
 		}
 		return parent::contentAsString();
 	}
 
-	public function getElement($index){
+	public function getElement($index) {
 		return $this->content[$index];
 	}
 
@@ -187,7 +187,7 @@ 
 	 * @param mixed $glyph
 	 * @param int $index
 	 */
-	public function addGlyph($glyph,$index=0){
+	public function addGlyph($glyph, $index=0) {
 		$elm=$this->getElement($index);
 		return $elm->wrapContentWithGlyph($glyph);
 	}
@@ -197,14 +197,14 @@ 
 	 * @param Dispatcher $dispatcher the request dispatcher
 	 * @return \Ajax\bootstrap\html\HtmlBreadcrumbs
 	 */
-	public function fromDispatcher($dispatcher){
+	public function fromDispatcher($dispatcher) {
 		$params=$dispatcher->getParams();
 		$action=$dispatcher->getActionName();
 		$items=array($dispatcher->getControllerName());
-		if(\sizeof($params)>0 || \strtolower($action)!="index" ){
+		if (\sizeof($params)>0 || \strtolower($action)!="index") {
 			$items[]=$action;
-			foreach ($params as $p){
-				if(\is_object($p)==false)
+			foreach ($params as $p) {
+				if (\is_object($p)==false)
 				$items[]=$p;
 			}
 		}
@@ -218,7 +218,7 @@ 
 	 * @return \Ajax\bootstrap\html\HtmlBreadcrumbs
 	 */
 	public function setHrefFunction($_hrefFunction) {
-		$this->_hrefFunction = $_hrefFunction;
+		$this->_hrefFunction=$_hrefFunction;
 		return $this;
 	}
 

Braces +6 added lines, -5 removed lines

@@ -65,9 +65,9 @@ 
 		if(is_array($element)){
 			$elm=new HtmlLink("lnk-".$this->identifier."-".$size);
 			$elm->fromArray($element);
-		}else if($element instanceof HtmlLink){
+		} else if($element instanceof HtmlLink){
 			$elm=$element;
-		}else{
+		} else{
 			$elm=new HtmlLink("lnk-".$this->identifier."-".$size,$href,$element);
 			if(isset($glyph)){
 				$elm->wrapContentWithGlyph($glyph);
@@ -114,7 +114,7 @@ 
 		}
 		if($this->absolutePaths===true){
 			return $this->_hrefFunction($this->content[$index]);
-		}else{
+		} else{
 			return $this->root.implode($separator, array_slice(array_map(function($e){return $this->_hrefFunction($e);}, $this->content),$this->startIndex,$index+1));
 		}
 	}
@@ -204,8 +204,9 @@ 
 		if(\sizeof($params)>0 || \strtolower($action)!="index" ){
 			$items[]=$action;
 			foreach ($params as $p){
-				if(\is_object($p)==false)
-				$items[]=$p;
+				if(\is_object($p)==false) {
+								$items[]=$p;
+				}
 			}
 		}
 		return $this->addElements($items);

Ajax/common/html/HtmlDoubleElement.php

Spacing +7 added lines, -7 removed lines

@@ -27,15 +27,15 @@ 
 		return $this->content;
 	}
 
-	public function addContent($content,$before=false) {
+	public function addContent($content, $before=false) {
 		if (is_array($this->content)===false) {
-			if(isset($this->content))
-				$this->content=array ($this->content);
+			if (isset($this->content))
+				$this->content=array($this->content);
 			else
 				$this->content=array();
 		}
-		if($before)
-			array_unshift($this->content,$content);
+		if ($before)
+			array_unshift($this->content, $content);
 		else
 			$this->content []=$content;
 		return $this;
@@ -49,8 +49,8 @@ 
 		parent::run($js);
 		if ($this->content instanceof HtmlDoubleElement) {
 			$this->content->run($js);
-		} else if (is_array($this->content)) {
-			foreach ( $this->content as $itemContent ) {
+		}else if (is_array($this->content)) {
+			foreach ($this->content as $itemContent) {
 				if ($itemContent instanceof HtmlDoubleElement) {
 					$itemContent->run($js);
 				}

Braces +10 added lines, -8 removed lines

@@ -29,15 +29,17 @@
 
 	public function addContent($content,$before=false) {
 		if (is_array($this->content)===false) {
-			if(isset($this->content))
-				$this->content=array ($this->content);
-			else
-				$this->content=array();
+			if(isset($this->content)) {
+							$this->content=array ($this->content);
+			} else {
+							$this->content=array();
+			}
+		}
+		if($before) {
+					array_unshift($this->content,$content);
+		} else {
+					$this->content []=$content;
 		}
-		if($before)
-			array_unshift($this->content,$content);
-		else
-			$this->content []=$content;
 		return $this;
 	}
 

Ajax/JsUtils.php

Spacing +59 added lines, -59 removed lines

@@ -50,7 +50,7 @@ 
 
 	protected function _setDi($di) {
 		$this->_di=$di;
-		if ($this->js!=null&&$di!=null)
+		if ($this->js!=null && $di!=null)
 			$this->js->setDi($di);
 	}
 
@@ -145,18 +145,18 @@ 
 	}
 
 	public function __construct($params=array()) {
-		$defaults=array (
+		$defaults=array(
 				'driver' => 'Jquery',
 				'debug' => true
 		);
-		foreach ( $defaults as $key => $val ) {
-			if (isset($params[$key])&&$params[$key]!=="") {
+		foreach ($defaults as $key => $val) {
+			if (isset($params[$key]) && $params[$key]!=="") {
 				$defaults[$key]=$params[$key];
 			}
 		}
 		extract($defaults);
 		$this->js=new Jquery($defaults);
-		$this->cdns=array ();
+		$this->cdns=array();
 	}
 
 	public function addToCompile($jsScript) {
@@ -408,7 +408,7 @@ 
 	 * @return string
 	 */
 	public function addClass($element='this', $class='', $immediatly=false) {
-		return $this->js->_genericCallValue('addClass',$element, $class, $immediatly);
+		return $this->js->_genericCallValue('addClass', $element, $class, $immediatly);
 	}
 
 	/**
@@ -418,8 +418,8 @@ 
 	 * @param boolean $immediatly defers the execution if set to false
 	 * @return string
 	 */
-	public function after($to, $element, $immediatly=false){
-		return $this->js->_genericCallElement('after',$to, $element, $immediatly);
+	public function after($to, $element, $immediatly=false) {
+		return $this->js->_genericCallElement('after', $to, $element, $immediatly);
 	}
 
 	/**
@@ -429,8 +429,8 @@ 
 	 * @param boolean $immediatly defers the execution if set to false
 	 * @return string
 	 */
-	public function before($to, $element, $immediatly=false){
-		return $this->js->_genericCallElement('before',$to, $element, $immediatly);
+	public function before($to, $element, $immediatly=false) {
+		return $this->js->_genericCallElement('before', $to, $element, $immediatly);
 	}
 
 	/**
@@ -450,8 +450,8 @@ 
 	 * @param string $value
 	 * @param boolean $immediatly defers the execution if set to false
 	 */
-	public function val($element='this',$value='',$immediatly=false){
-		return $this->js->_genericCallValue('val',$element,$value,$immediatly);
+	public function val($element='this', $value='', $immediatly=false) {
+		return $this->js->_genericCallValue('val', $element, $value, $immediatly);
 	}
 
 	/**
@@ -461,7 +461,7 @@ 
 	 * @param boolean $immediatly defers the execution if set to false
 	 */
 	public function html($element='this', $value='', $immediatly=false) {
-		return $this->js->_genericCallValue('html',$element, $value, $immediatly);
+		return $this->js->_genericCallValue('html', $element, $value, $immediatly);
 	}
 	// --------------------------------------------------------------------
 	/**
@@ -486,7 +486,7 @@ 
 	 * @return string
 	 */
 	public function append($to, $element, $immediatly=false) {
-		return $this->js->_genericCallElement('append',$to, $element, $immediatly);
+		return $this->js->_genericCallElement('append', $to, $element, $immediatly);
 	}
 
 	/**
@@ -497,7 +497,7 @@ 
 	 * @return string
 	 */
 	public function prepend($to, $element, $immediatly=false) {
-		return $this->js->_genericCallElement('prepend',$to, $element, $immediatly);
+		return $this->js->_genericCallElement('prepend', $to, $element, $immediatly);
 	}
 	// --------------------------------------------------------------------
 	/**
@@ -548,7 +548,7 @@ 
 	 * @return string
 	 */
 	public function removeClass($element='this', $class='', $immediatly=false) {
-		return $this->js->_genericCall('removeClass',$element, $class, $immediatly);
+		return $this->js->_genericCall('removeClass', $element, $class, $immediatly);
 	}
 	// --------------------------------------------------------------------
 	/**
@@ -609,7 +609,7 @@ 
 	 * @return string
 	 */
 	public function toggleClass($element='this', $class='', $immediatly=false) {
-		return $this->js->_genericCallValue('toggleClass',$element, $class, $immediatly);
+		return $this->js->_genericCallValue('toggleClass', $element, $class, $immediatly);
 	}
 
 	/**
@@ -656,11 +656,11 @@ 
 	 */
 	public function compile($view=NULL, $view_var='script_foot', $script_tags=TRUE) {
 		$bs=$this->_bootstrap;
-		if (isset($bs)&&isset($view)) {
+		if (isset($bs) && isset($view)) {
 			$bs->compileHtml($this, $view);
 		}
 		$sem=$this->_semantic;
-		if (isset($sem)&&isset($view)) {
+		if (isset($sem) && isset($view)) {
 			$sem->compileHtml($this, $view);
 		}
 		return $this->js->_compile($view, $view_var, $script_tags);
@@ -740,27 +740,27 @@ 
 				$json_result=$result->result_array();
 			} elseif (is_array($result)) {
 				$json_result=$result;
-			} else {
+			}else {
 				return $this->_prep_args($result);
 			}
-		} else {
+		}else {
 			return 'null';
 		}
 		return $this->_create_json($json_result, $match_array_type);
 	}
 
 	private function _create_json($json_result, $match_array_type) {
-		$json=array ();
+		$json=array();
 		$_is_assoc=TRUE;
-		if (!is_array($json_result)&&empty($json_result)) {
+		if (!is_array($json_result) && empty($json_result)) {
 			show_error("Generate JSON Failed - Illegal key, value pair.");
 		} elseif ($match_array_type) {
 			$_is_assoc=$this->_is_associative_array($json_result);
 		}
-		foreach ( $json_result as $k => $v ) {
+		foreach ($json_result as $k => $v) {
 			if ($_is_assoc) {
 				$json[]=$this->_prep_args($k, TRUE).':'.$this->generate_json($v, $match_array_type);
-			} else {
+			}else {
 				$json[]=$this->generate_json($v, $match_array_type);
 			}
 		}
@@ -775,7 +775,7 @@ 
 	 * @return type
 	 */
 	public function _is_associative_array($arr) {
-		foreach ( array_keys($arr) as $key => $val ) {
+		foreach (array_keys($arr) as $key => $val) {
 			if ($key!==$val) {
 				return TRUE;
 			}
@@ -794,11 +794,11 @@ 
 			return 'null';
 		} elseif (is_bool($result)) {
 			return ($result===TRUE) ? 'true' : 'false';
-		} elseif (is_string($result)||$is_key) {
-			return '"'.str_replace(array (
-					'\\',"\t","\n","\r",'"','/'
-			), array (
-					'\\\\','\\t','\\n',"\\r",'\"','\/'
+		} elseif (is_string($result) || $is_key) {
+			return '"'.str_replace(array(
+					'\\', "\t", "\n", "\r", '"', '/'
+			), array(
+					'\\\\', '\\t', '\\n', "\\r", '\"', '\/'
 			), $result).'"';
 		} elseif (is_scalar($result)) {
 			return $result;
@@ -813,8 +813,8 @@ 
 	 * @param string $jsCallback javascript code to execute after the request
 	 * @param boolean $hasLoader true for showing ajax loader. default : true
 	 */
-	public function get($url, $responseElement="", $params="{}", $jsCallback=NULL,$hasLoader=true) {
-		return $this->js->_get($url, $params, $responseElement, $jsCallback, NULL, $hasLoader,true);
+	public function get($url, $responseElement="", $params="{}", $jsCallback=NULL, $hasLoader=true) {
+		return $this->js->_get($url, $params, $responseElement, $jsCallback, NULL, $hasLoader, true);
 	}
 
 	/**
@@ -825,8 +825,8 @@ 
 	 * @param string $jsCallback javascript code to execute after the request
 	 * @param boolean $immediatly
 	 */
-	public function json($url, $method="get", $params="{}", $jsCallback=NULL, $attr="id", $context="document",$immediatly=false) {
-		return $this->js->_json($url, $method, $params, $jsCallback, $attr, $context,$immediatly);
+	public function json($url, $method="get", $params="{}", $jsCallback=NULL, $attr="id", $context="document", $immediatly=false) {
+		return $this->js->_json($url, $method, $params, $jsCallback, $attr, $context, $immediatly);
 	}
 
 	/**
@@ -836,8 +836,8 @@ 
 	 * @param string $url the request address
 	 * @param array $parameters default : array("preventDefault"=>true,"stopPropagation"=>true,"jsCallback"=>NULL,"attr"=>"id","params"=>"{}","method"=>"get","immediatly"=>true)
 	 */
-	public function jsonOn($event,$element, $url,$parameters=array()) {
-		return $this->js->_jsonOn($event, $element, $url,$parameters);
+	public function jsonOn($event, $element, $url, $parameters=array()) {
+		return $this->js->_jsonOn($event, $element, $url, $parameters);
 	}
 
 	/**
@@ -881,8 +881,8 @@ 
 	 * @param string $url the request url
 	 * @param array $parameters default : array("preventDefault"=>true,"stopPropagation"=>true,"jsCallback"=>NULL,"attr"=>"id","params"=>"{}","method"=>"get","immediatly"=>true)
 	 */
-	public function jsonArrayOn($event,$element,$maskSelector, $url,$parameters=array()) {
-		return $this->js->_jsonArrayOn($event,$element,$maskSelector, $url, $parameters);
+	public function jsonArrayOn($event, $element, $maskSelector, $url, $parameters=array()) {
+		return $this->js->_jsonArrayOn($event, $element, $maskSelector, $url, $parameters);
 	}
 
 	/**
@@ -894,7 +894,7 @@ 
 	 * @param string $jsCallback javascript code to execute after the request
 	 * @param string $attr the html attribute added to the request
 	 */
-	public function getDeferred($url, $responseElement="", $params="{}", $jsCallback=NULL,$attr="id") {
+	public function getDeferred($url, $responseElement="", $params="{}", $jsCallback=NULL, $attr="id") {
 		return $this->js->_get($url, $params, $responseElement, $jsCallback, $attr, false);
 	}
 
@@ -932,8 +932,8 @@ 
 	 * @param string $jsCallback javascript code to execute after the request
 	 * @param boolean $hasLoader true for showing ajax loader. default : true
 	 */
-	public function post($url, $responseElement="", $params="{}", $jsCallback=NULL,$hasLoader=true) {
-		return $this->js->_post($url, $params, $responseElement, $jsCallback, NULL, $hasLoader,true);
+	public function post($url, $responseElement="", $params="{}", $jsCallback=NULL, $hasLoader=true) {
+		return $this->js->_post($url, $params, $responseElement, $jsCallback, NULL, $hasLoader, true);
 	}
 
 	/**
@@ -946,8 +946,8 @@ 
 	 * @param string $jsCallback javascript code to execute after the request
 	 * @param boolean $hasLoader true for showing ajax loader. default : true
 	 */
-	public function postDeferred($url, $responseElement="", $params="{}", $jsCallback=NULL, $attr="id",$hasLoader=true) {
-		return $this->js->_post($url, $params, $responseElement, $jsCallback, $attr, $hasLoader,false);
+	public function postDeferred($url, $responseElement="", $params="{}", $jsCallback=NULL, $attr="id", $hasLoader=true) {
+		return $this->js->_post($url, $params, $responseElement, $jsCallback, $attr, $hasLoader, false);
 	}
 
 	/**
@@ -961,7 +961,7 @@ 
 	 * @param array $parameters default : array("preventDefault"=>true,"stopPropagation"=>true,"jsCallback"=>NULL,"attr"=>"id","hasLoader"=>true,"immediatly"=>true)
 	 */
 	public function postOn($event, $element, $url, $params="{}", $responseElement="", $parameters=array()) {
-		return $this->js->_postOn($event, $element,  $url, $params, $responseElement, $parameters);
+		return $this->js->_postOn($event, $element, $url, $params, $responseElement, $parameters);
 	}
 
 	/**
@@ -985,8 +985,8 @@ 
 	 * @param string $jsCallback javascript code to execute after the request
 	 * @param boolean $hasLoader true for showing ajax loader. default : true
 	 */
-	public function postForm($url, $form, $responseElement, $validation=false, $jsCallback=NULL,$hasLoader=true) {
-		return $this->js->_postForm($url, $form, $responseElement, $validation, $jsCallback, NULL, $hasLoader,true);
+	public function postForm($url, $form, $responseElement, $validation=false, $jsCallback=NULL, $hasLoader=true) {
+		return $this->js->_postForm($url, $form, $responseElement, $validation, $jsCallback, NULL, $hasLoader, true);
 	}
 
 	/**
@@ -999,8 +999,8 @@ 
 	 * @param string $attr the html attribute added to the request
 	 * @param boolean $hasLoader true for showing ajax loader. default : true
 	 */
-	public function postFormDeferred($url, $form, $responseElement, $validation=false, $jsCallback=NULL,$attr="id",$hasLoader=true) {
-		return $this->js->_postForm($url, $form, $responseElement, $validation, $jsCallback, $attr, $hasLoader,false);
+	public function postFormDeferred($url, $form, $responseElement, $validation=false, $jsCallback=NULL, $attr="id", $hasLoader=true) {
+		return $this->js->_postForm($url, $form, $responseElement, $validation, $jsCallback, $attr, $hasLoader, false);
 	}
 
 	/**
@@ -1014,7 +1014,7 @@ 
 	 * @param array $parameters default : array("preventDefault"=>true,"stopPropagation"=>true,"validation"=>false,"jsCallback"=>NULL,"attr"=>"id","hasLoader"=>true,"immediatly"=>true)
 	 */
 	public function postFormOn($event, $element, $url, $form, $responseElement="", $parameters=array()) {
-		return $this->js->_postFormOn($event,$element, $url, $form, $responseElement, $parameters);
+		return $this->js->_postFormOn($event, $element, $url, $form, $responseElement, $parameters);
 	}
 
 	/**
@@ -1069,7 +1069,7 @@ 
 		$preventDefault=false;
 		$immediatly=true;
 		extract($parameters);
-		return $this->js->_doJQueryOn($event, $element, $elementToModify, $jqueryCall, $param, $preventDefault, $stopPropagation, $jsCallback,$immediatly);
+		return $this->js->_doJQueryOn($event, $element, $elementToModify, $jqueryCall, $param, $preventDefault, $stopPropagation, $jsCallback, $immediatly);
 	}
 
 	/**
@@ -1096,7 +1096,7 @@ 
 		$preventDefault=false;
 		$immediatly=true;
 		extract($parameters);
-		$script=$this->js->_execOn($element, $event, $js, $preventDefault, $stopPropagation,$immediatly);
+		$script=$this->js->_execOn($element, $event, $js, $preventDefault, $stopPropagation, $immediatly);
 		return $script;
 	}
 
@@ -1106,7 +1106,7 @@ 
 
 	public function setCDNs($cdns) {
 		if (is_array($cdns)===false) {
-			$cdns=array (
+			$cdns=array(
 					$cdns
 			);
 		}
@@ -1117,9 +1117,9 @@ 
 		$hasJQuery=false;
 		$hasJQueryUI=false;
 		$hasBootstrap=false;
-		$result=array ();
-		foreach ( $this->cdns as $cdn ) {
-			switch(get_class($cdn)) {
+		$result=array();
+		foreach ($this->cdns as $cdn) {
+			switch (get_class($cdn)) {
 				case "Ajax\lib\CDNJQuery":
 					$hasJQuery=true;
 					$result[0]=$cdn;
@@ -1137,10 +1137,10 @@ 
 		if ($hasJQuery===false) {
 			$result[0]=new CDNJQuery("x");
 		}
-		if ($hasJQueryUI===false&&isset($this->_ui)) {
+		if ($hasJQueryUI===false && isset($this->_ui)) {
 			$result[1]=new CDNGuiGen("x", $template);
 		}
-		if ($hasBootstrap===false&&isset($this->_bootstrap)) {
+		if ($hasBootstrap===false && isset($this->_bootstrap)) {
 			$result[2]=new CDNBootstrap("x");
 		}
 		ksort($result);
@@ -1154,7 +1154,7 @@ 
 			$this->_setDi($di);
 		}
 	}
-} else {
+}else {
 	class JsUtils extends _JsUtils {
 
 		public function setDi(DiInterface $di) {