jcheron / phalcon-jquery

Ajax/ui/components/Dialog.php

Spacing +7 added lines, -7 removed lines

@@ -12,11 +12,11 @@ 
  */
 class Dialog extends SimpleComponent {
 	protected $attachTo;
-	protected $buttons=array ();
+	protected $buttons=array();
 
 	public function __construct(JsUtils $js) {
 		parent::__construct($js);
-		$this->params=array (
+		$this->params=array(
 				"dialogClass" => "no-close" 
 		);
 		$this->addCancelBtn("Annuler");
@@ -24,12 +24,12 @@ 
 
 	public function getScript() {
 		$allParams=$this->params;
-		$jsonButtons=array ();
-		foreach ( $this->buttons as $button ) {
+		$jsonButtons=array();
+		foreach ($this->buttons as $button) {
 			$jsonButtons []=$button->getParams();
 		}
 		$allParams ["buttons"]=$jsonButtons;
-		$this->jquery_code_for_compile []="$( '" . $this->attachTo . "' ).dialog(" . $this->getParamsAsJSON($allParams) . ");";
+		$this->jquery_code_for_compile []="$( '".$this->attachTo."' ).dialog(".$this->getParamsAsJSON($allParams).");";
 		$result=implode("", $this->jquery_code_for_compile);
 		$result=str_ireplace("\"%", "", $result);
 		$result=str_ireplace("%\"", "", $result);
@@ -59,9 +59,9 @@ 
 	}
 
 	private function insertBtn($insert, $position=NULL) {
-		if ($position != NULL) {
+		if ($position!=NULL) {
 			$this->buttons=array_splice($this->buttons, $position, 0, $insert);
-		} else {
+		}else {
 			$this->buttons []=$insert;
 		}
 	}

Ajax/ui/components/DialogButton.php

Spacing +2 added lines, -2 removed lines

@@ -16,7 +16,7 @@ 
 
 	private function addFunction($jsCode) {
 		if (!Text::startsWith($jsCode, "function"))
-			$jsCode="%function(){" . $jsCode . "}%";
+			$jsCode="%function(){".$jsCode."}%";
 		return $jsCode;
 	}
 
@@ -43,6 +43,6 @@ 
 	}
 
 	public static function submitButton(JsUtils $js, $url, $form, $responseElement, $caption="Okay") {
-		return new DialogButton($caption, $js->postForm($url, $form, $responseElement) . ";$( this ).dialog( 'close' );");
+		return new DialogButton($caption, $js->postForm($url, $form, $responseElement).";$( this ).dialog( 'close' );");
 	}
 }
\ No newline at end of file

Braces +3 added lines, -2 removed lines

@@ -15,8 +15,9 @@
 class DialogButton extends BaseComponent {
 
 	private function addFunction($jsCode) {
-		if (!Text::startsWith($jsCode, "function"))
-			$jsCode="%function(){" . $jsCode . "}%";
+		if (!Text::startsWith($jsCode, "function")) {
+					$jsCode="%function(){" . $jsCode . "}%";
+		}
 		return $jsCode;
 	}
 

Ajax/ui/components/Progressbar.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 
 	public function __construct(JsUtils $js) {
 		parent::__construct($js);
-		$this->params=array (
+		$this->params=array(
 				"value" => 50 
 		);
 		$this->uiName="progressbar";

Ajax/ui/properties/Animation.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
 	 * @param string $value default : swing
 	 */
 	public function setEasing($value) {
-		$this->setParamCtrl("easing", $value, array (
+		$this->setParamCtrl("easing", $value, array(
 				"linear",
 				"swing",
 				"easeInQuad",

Ajax/ui/properties/Position.php

Spacing +1 added lines, -1 removed lines

@@ -74,7 +74,7 @@
 	protected function setParamCtrl($key, $value, $typeCtrl) {
 		if (!$typeCtrl($value)) {
 			throw new \Exception("La fonction ".$typeCtrl." a retourné faux pour l'affectation de la propriété ".$key." à la position");
-		} else
+		}else
 			$this->setParam($key, $value);
 	}
 

Braces +3 added lines, -2 removed lines

@@ -74,8 +74,9 @@
 	protected function setParamCtrl($key, $value, $typeCtrl) {
 		if (!$typeCtrl($value)) {
 			throw new \Exception("La fonction ".$typeCtrl." a retourné faux pour l'affectation de la propriété ".$key." à la position");
-		} else
-			$this->setParam($key, $value);
+		} else {
+					$this->setParam($key, $value);
+		}
 	}
 
 	/*

Ajax/bootstrap/html/base/HtmlNavElement.php

Doc Comments +4 added lines, -1 removed lines

@@ -20,6 +20,10 @@ 
 	protected $attr;
 	
 	
+	/**
+	 * @param string $identifier
+	 * @param string $tagName
+	 */
 	public function __construct($identifier,$tagName){
 		parent::__construct($identifier,$tagName);
 		$this->root="";
@@ -30,7 +34,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/HtmlGlyphicon.php

Spacing +1 added lines, -1 removed lines

@@ -149,7 +149,7 @@
 		$glyph->setGlyphicon($glyphicon);
 		if ($left) {
 			$this->content=$glyph->compile().$separator.$this->content;
-		} else {
+		}else {
 			$this->content.=$separator.$glyph->compile();
 		}
 		return $this;

Ajax/bootstrap/html/HtmlPagination.php

Braces +12 added lines, -9 removed lines

@@ -52,10 +52,11 @@ 
 		$this->from=$from;
 		$this->to=$to;
 		$this->urlMask="%page%";
-		if(!isset($countVisible))
-			$this->countVisible=$to-$from+1;
-		else
-			$this->countVisible=$countVisible;
+		if(!isset($countVisible)) {
+					$this->countVisible=$to-$from+1;
+		} else {
+					$this->countVisible=$countVisible;
+		}
 		$this->createContent();
 	}
 	
@@ -74,7 +75,7 @@ 
 			$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;
@@ -107,8 +108,9 @@ 
 	public function _addEvent($event, $jsCode) {
 		foreach ($this->content as $li){
 			$content=$li->getContent();
-			if($content instanceof BaseHtml)
-				$content->_addEvent($event,$jsCode);
+			if($content instanceof BaseHtml) {
+							$content->_addEvent($event,$jsCode);
+			}
 		}
 	}
 	/**
@@ -146,8 +148,9 @@ 
 		if (is_int($size)) {
 			return $this->addToPropertyUnique("class", CssRef::sizes("pagination")[$size], CssRef::sizes("pagination"));
 		}
-		if(!PhalconUtils::startsWith($size, "pagination-") && $size!=="")
-			$size="pagination-".$size;
+		if(!PhalconUtils::startsWith($size, "pagination-") && $size!=="") {
+					$size="pagination-".$size;
+		}
 		return $this->addToPropertyCtrl("class", $size, CssRef::sizes("pagination"));
 	}
 	

Spacing +35 added lines, -35 removed lines

@@ -45,70 +45,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);
 		}
 	}
 	/**
@@ -117,14 +117,14 @@ 
 	 * @param Dispatcher $dispatcher the request dispatcher
 	 * @return \Ajax\bootstrap\html\HtmlPagination
 	 */
-	public function fromDispatcher(JsUtils $js,$dispatcher,$startIndex=0){
+	public function fromDispatcher(JsUtils $js, $dispatcher, $startIndex=0) {
 		$items=$js->fromDispatcher($dispatcher);
 		$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;
 			}
@@ -132,7 +132,7 @@ 
 		return $this;
 	}
 
-	public function getUrl($index){
+	public function getUrl($index) {
 		return str_ireplace("%page%", $index, $this->urlMask);
 	}
 
@@ -146,7 +146,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"));
 	}
@@ -155,7 +155,7 @@ 
 		return $this->from;
 	}
 	public function setFrom($from) {
-		$this->from = $from;
+		$this->from=$from;
 		$this->createContent();
 		return $this;
 	}
@@ -163,7 +163,7 @@ 
 		return $this->to;
 	}
 	public function setTo($to) {
-		$this->to = $to;
+		$this->to=$to;
 		$this->createContent();
 		return $this;
 	}
@@ -171,7 +171,7 @@ 
 		return $this->active;
 	}
 	public function setActive($active) {
-		$this->active = $active;
+		$this->active=$active;
 		$this->createContent();
 		return $this;
 	}
@@ -179,7 +179,7 @@ 
 		return $this->urlMask;
 	}
 	public function setUrlMask($urlMask) {
-		$this->urlMask = $urlMask;
+		$this->urlMask=$urlMask;
 		$this->createContent();
 		return $this;
 	}
@@ -187,7 +187,7 @@ 
 		return $this->countVisible;
 	}
 	public function setCountVisible($countVisible) {
-		$this->countVisible = $countVisible;
+		$this->countVisible=$countVisible;
 		$this->createContent();
 		return $this;
 	}

Ajax/bootstrap/html/content/HtmlGridRow.php

Doc Comments +10 added lines

@@ -14,6 +14,11 @@ 
  */
 class HtmlGridRow extends HtmlDoubleElement {
 	private $cols;
+
+	/**
+	 * @param string $identifier
+	 * @param integer $numCols
+	 */
 	public function __construct($identifier,$numCols=NULL){
 		parent::__construct($identifier,"div");
 		$this->setProperty("class", "row");
@@ -85,6 +90,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);

Braces +2 added lines, -2 removed lines

@@ -42,7 +42,7 @@ 
 	public function getCol($index,$force=true){
 		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;
@@ -95,7 +95,7 @@ 
 					$this->cols = array_values($this->cols);
 					$width=$width-$widthCol;
 				}
-			}else{
+			} else{
 				$width=0;
 			}
 		}

Spacing +38 added lines, -38 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);
-			$width=12/$numCols;
-			for ($i=0;$i<$numCols;$i++){
-				$this->addCol(CssSize::SIZE_MD,$width);
+		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);
 			}
 		}
 	}
 
-	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 $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;
 			}
 		}

Unused Use Statements -1 removed lines

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