jcheron / phalcon-jquery

Ajax/semantic/html/collections/form/HtmlFormCheckbox.php

Braces +3 added lines, -2 removed lines

@@ -21,8 +21,9 @@
 			$label->setProperty("tabindex",0);
 		}
 		$this->setClass("ui checkbox");
-		if(isset($type))
-			$this->setType($type);
+		if(isset($type)) {
+					$this->setType($type);
+		}
 		$this->wrap("<div class='field'>","</div>");
 	}
 

Spacing +7 added lines, -7 removed lines

@@ -10,18 +10,18 @@
  * @version 1.001
  */
 class HtmlFormCheckbox extends AbstractHtmlFormRadioCheckbox {
-	public function __construct($identifier, $label=NULL,$value=NULL,$type=NULL) {
-		parent::__construct($identifier, NULL,$label,$value,$type);
+	public function __construct($identifier, $label=NULL, $value=NULL, $type=NULL) {
+		parent::__construct($identifier, NULL, $label, $value, $type);
 		$this->_input->setClass("ui checkbox");
-		if(isset($type))
+		if (isset($type))
 			$this->setType($type);
 	}
 
-	public static function slider($identifier, $label=NULL,$value=NULL){
-		return new HtmlFormCheckbox($identifier,$label,$value,CheckboxType::SLIDER);
+	public static function slider($identifier, $label=NULL, $value=NULL) {
+		return new HtmlFormCheckbox($identifier, $label, $value, CheckboxType::SLIDER);
 	}
 
-	public static function toggle($identifier, $label=NULL,$value=NULL){
-		return new HtmlFormCheckbox($identifier,$label,$value,CheckboxType::TOGGLE);
+	public static function toggle($identifier, $label=NULL, $value=NULL) {
+		return new HtmlFormCheckbox($identifier, $label, $value, CheckboxType::TOGGLE);
 	}
 }
\ No newline at end of file

Ajax/semantic/html/base/constants/CheckboxType.php

Spacing +1 added lines, -1 removed lines

@@ -5,5 +5,5 @@
 use Ajax\common\BaseEnum;
 
 abstract class CheckboxType extends BaseEnum {
-	const STANDARD="",TOGGLE="toggle",SLIDER="slider";
+	const STANDARD="", TOGGLE="toggle", SLIDER="slider";
 }
\ No newline at end of file

Ajax/semantic/html/base/HtmlSemCollection.php

Spacing +3 added lines, -3 removed lines

@@ -9,10 +9,10 @@
  * @author jc
  * @version 1.001
  */
-abstract class HtmlSemCollection extends HtmlCollection{
+abstract class HtmlSemCollection extends HtmlCollection {
 	use BaseTrait;
-	public function __construct( $identifier, $tagName="div",$baseClass=""){
-		parent::__construct( $identifier, $tagName);
+	public function __construct($identifier, $tagName="div", $baseClass="") {
+		parent::__construct($identifier, $tagName);
 		$this->_baseClass=$baseClass;
 		$this->setClass($baseClass);
 	}

Ajax/common/html/html5/HtmlTextarea.php

Spacing +6 added lines, -6 removed lines

@@ -7,26 +7,26 @@
 
 class HtmlTextarea extends HtmlDoubleElement {
 
-	public function __construct($identifier,$value=NULL,$placeholder=NULL,$rows=NULL) {
+	public function __construct($identifier, $value=NULL, $placeholder=NULL, $rows=NULL) {
 		parent::__construct($identifier, "textarea");
 		$this->setValue($value);
 		$this->setPlaceholder($placeholder);
-		if(isset($rows))
+		if (isset($rows))
 			$this->setRows($rows);
 	}
 	public function setValue($value) {
-		if(isset($value))
+		if (isset($value))
 			$this->setContent($value);
 		return $this;
 	}
 
-	public function setPlaceholder($value){
-		if(JString::isNotNull($value))
+	public function setPlaceholder($value) {
+		if (JString::isNotNull($value))
 			$this->setProperty("placeholder", $value);
 		return $this;
 	}
 
-	public function setRows($count){
+	public function setRows($count) {
 		$this->setProperty("rows", $count);
 	}
 }
\ No newline at end of file

Braces +9 added lines, -6 removed lines

@@ -11,18 +11,21 @@
 		parent::__construct($identifier, "textarea");
 		$this->setValue($value);
 		$this->setPlaceholder($placeholder);
-		if(isset($rows))
-			$this->setRows($rows);
+		if(isset($rows)) {
+					$this->setRows($rows);
+		}
 	}
 	public function setValue($value) {
-		if(isset($value))
-			$this->setContent($value);
+		if(isset($value)) {
+					$this->setContent($value);
+		}
 		return $this;
 	}
 
 	public function setPlaceholder($value){
-		if(JString::isNotNull($value))
-			$this->setProperty("placeholder", $value);
+		if(JString::isNotNull($value)) {
+					$this->setProperty("placeholder", $value);
+		}
 		return $this;
 	}
 

Ajax/semantic/html/collections/form/HtmlFormTextarea.php

Spacing +4 added lines, -4 removed lines

@@ -8,17 +8,17 @@
 class HtmlFormTextarea extends HtmlFormField {
 	use TextFieldsTrait;
 
-	public function __construct($identifier, $label=NULL,$value=NULL,$placeholder=NULL,$rows=NULL) {
-		if(!isset($placeholder))
+	public function __construct($identifier, $label=NULL, $value=NULL, $placeholder=NULL, $rows=NULL) {
+		if (!isset($placeholder))
 			$placeholder=$label;
-		parent::__construct("field-".$identifier, new HtmlTextarea($identifier,$value,$placeholder,$rows), $label);
+		parent::__construct("field-".$identifier, new HtmlTextarea($identifier, $value, $placeholder, $rows), $label);
 	}
 
 	/**
 	 * Defines the textarea row count
 	 * @param int $count
 	 */
-	public function setRows($count){
+	public function setRows($count) {
 		$this->getField()->setRows($count);
 	}
 }
\ No newline at end of file

Braces +3 added lines, -2 removed lines

@@ -8,8 +8,9 @@
 class HtmlFormInput extends HtmlFormField {
 
 	public function __construct($identifier, $label=NULL,$type="text",$value=NULL,$placeholder=NULL) {
-		if(!isset($placeholder))
-			$placeholder=$label;
+		if(!isset($placeholder)) {
+					$placeholder=$label;
+		}
 		parent::__construct("field-".$identifier, new HtmlInput($identifier,$type,$value,$placeholder), $label);
 	}
 

Ajax/semantic/html/base/constants/State.php

Spacing +1 added lines, -1 removed lines

@@ -5,5 +5,5 @@
 use Ajax\common\BaseEnum;
 
 abstract class State extends BaseEnum {
-	const ACTIVE="active",DISABLED="disabled",ERROR="error",FOCUS="focus",LOADING="loading",SUCCESS="success",WARNING="warning";
+	const ACTIVE="active", DISABLED="disabled", ERROR="error", FOCUS="focus", LOADING="loading", SUCCESS="success", WARNING="warning";
 }
\ No newline at end of file

Ajax/Jquery.php

Doc Comments +5 added lines, -1 removed lines

@@ -207,7 +207,7 @@ 
 	 *
 	 * @param string $element
 	 * @param array $options
-	 * @return void
+	 * @return string
 	 */
 	public function sortable($element, $options=array()) {
 		if (count($options)>0) {
@@ -351,6 +351,7 @@ 
 	/**
 	 * A wrapper for writing document.ready()
 	 *
+	 * @param string $js
 	 * @return string
 	 */
 	public function _document_ready($js) {
@@ -420,6 +421,9 @@ 
 		return $speed;
 	}
 
+	/**
+	 * @param string $input
+	 */
 	private function minify($input) {
 	if(trim($input) === "") return $input;
 	return preg_replace(

Braces +23 added lines, -15 removed lines

@@ -173,10 +173,12 @@ 
 		if (isset($param)) {
 			$param=$this->_prep_value($param);
 			$str="$({$element}).{$jQueryCall}({$param});";
-		} else
-			$str="$({$element}).{$jQueryCall}();";
-			if ($immediatly)
-				$this->jquery_code_for_compile[]=$str;
+		} else {
+					$str="$({$element}).{$jQueryCall}();";
+		}
+			if ($immediatly) {
+							$this->jquery_code_for_compile[]=$str;
+			}
 			return $str;
 	}
 	/**
@@ -191,8 +193,9 @@ 
 		$to=$this->_prep_element($to);
 		$element=$this->_prep_element($element);
 		$str="$({$to}).{$jQueryCall}({$element});";
-		if ($immediatly)
-			$this->jquery_code_for_compile[]=$str;
+		if ($immediatly) {
+					$this->jquery_code_for_compile[]=$str;
+		}
 			return $str;
 	}
 	// --------------------------------------------------------------------
@@ -260,12 +263,14 @@ 
 		if ($stopPropagation===true) {
 			$js="event.stopPropagation();\n".$js;
 		}
-		if (array_search($event, $this->jquery_events)===false)
-			$event="\n\t$(".$this->_prep_element($element).").bind('{$event}',function(event){\n\t\t{$js}\n\t});\n";
-		else
-			$event="\n\t$(".$this->_prep_element($element).").{$event}(function(event){\n\t\t{$js}\n\t});\n";
-		if($immediatly)
-			$this->jquery_code_for_compile[]=$event;
+		if (array_search($event, $this->jquery_events)===false) {
+					$event="\n\t$(".$this->_prep_element($element).").bind('{$event}',function(event){\n\t\t{$js}\n\t});\n";
+		} else {
+					$event="\n\t$(".$this->_prep_element($element).").{$event}(function(event){\n\t\t{$js}\n\t});\n";
+		}
+		if($immediatly) {
+					$this->jquery_code_for_compile[]=$event;
+		}
 		return $event;
 	}
 
@@ -326,8 +331,9 @@ 
 		}
 		$output=($script_tags===FALSE) ? $script : $this->inline($script);
 
-		if ($view!=NULL)
-			$view->setVar($view_var, $output);
+		if ($view!=NULL) {
+					$view->setVar($view_var, $output);
+		}
 		return $output;
 	}
 
@@ -421,7 +427,9 @@ 
 	}
 
 	private function minify($input) {
-	if(trim($input) === "") return $input;
+	if(trim($input) === "") {
+		return $input;
+	}
 	return preg_replace(
 			array(
 					// Remove comment(s)

Spacing +24 added lines, -24 removed lines

@@ -17,21 +17,21 @@ 
  * jQuery Class
  */
 class Jquery {
-	use JqueryEventsTrait,JqueryAjaxTrait,JqueryActionsTrait;
+	use JqueryEventsTrait, JqueryAjaxTrait, JqueryActionsTrait;
 	protected $_di;
 	protected $_ui;
 	protected $_bootstrap;
 	protected $_semantic;
 	protected $libraryFile;
 	protected $_javascript_folder='js';
-	protected $jquery_code_for_load=array ();
-	protected $jquery_code_for_compile=array ();
+	protected $jquery_code_for_load=array();
+	protected $jquery_code_for_compile=array();
 	protected $jquery_corner_active=FALSE;
 	protected $jquery_table_sorter_active=FALSE;
 	protected $jquery_table_sorter_pager_active=FALSE;
 
-	protected $jquery_events=array (
-			"bind","blur","change","click","dblclick","delegate","die","error","focus","focusin","focusout","hover","keydown","keypress","keyup","live","load","mousedown","mousseenter","mouseleave","mousemove","mouseout","mouseover","mouseup","off","on","one","ready","resize","scroll","select","submit","toggle","trigger","triggerHandler","undind","undelegate","unload"
+	protected $jquery_events=array(
+			"bind", "blur", "change", "click", "dblclick", "delegate", "die", "error", "focus", "focusin", "focusout", "hover", "keydown", "keypress", "keyup", "live", "load", "mousedown", "mousseenter", "mouseleave", "mousemove", "mouseout", "mouseover", "mouseup", "off", "on", "one", "ready", "resize", "scroll", "select", "submit", "toggle", "trigger", "triggerHandler", "undind", "undelegate", "unload"
 	);
 
 	public function setDi($di) {
@@ -61,7 +61,7 @@ 
 
 	public function __construct($params) {
 		$this->params=array();
-		foreach ( $params as $key => $val ) {
+		foreach ($params as $key => $val) {
 				$this->params[$key]=$params[$key];
 		}
 	}
@@ -135,12 +135,12 @@ 
 	 */
 	public function _output($array_js='') {
 		if (!is_array($array_js)) {
-			$array_js=array (
+			$array_js=array(
 					$array_js
 			);
 		}
 
-		foreach ( $array_js as $js ) {
+		foreach ($array_js as $js) {
 			$this->jquery_code_for_compile[]="\t$js\n";
 		}
 	}
@@ -152,12 +152,12 @@ 
 	 * @param string $param
 	 * @param boolean $immediatly delayed if false
 	 */
-	public function _genericCallValue($jQueryCall,$element='this', $param="", $immediatly=false) {
+	public function _genericCallValue($jQueryCall, $element='this', $param="", $immediatly=false) {
 		$element=$this->_prep_element($element);
 		if (isset($param)) {
 			$param=$this->_prep_value($param);
 			$str="$({$element}).{$jQueryCall}({$param});";
-		} else
+		}else
 			$str="$({$element}).{$jQueryCall}();";
 			if ($immediatly)
 				$this->jquery_code_for_compile[]=$str;
@@ -171,7 +171,7 @@ 
 	 * @param boolean $immediatly delayed if false
 	 * @return string
 	 */
-	public function _genericCallElement($jQueryCall,$to='this', $element, $immediatly=false) {
+	public function _genericCallElement($jQueryCall, $to='this', $element, $immediatly=false) {
 		$to=$this->_prep_element($to);
 		$element=$this->_prep_element($element);
 		$str="$({$to}).{$jQueryCall}({$element});";
@@ -189,12 +189,12 @@ 
 	 */
 	public function sortable($element, $options=array()) {
 		if (count($options)>0) {
-			$sort_options=array ();
-			foreach ( $options as $k => $v ) {
+			$sort_options=array();
+			foreach ($options as $k => $v) {
 				$sort_options[]="\n\t\t".$k.': '.$v."";
 			}
 			$sort_options=implode(",", $sort_options);
-		} else {
+		}else {
 			$sort_options='';
 		}
 
@@ -222,7 +222,7 @@ 
 	 * @param boolean $stopPropagation Prevents the event from bubbling up the DOM tree, preventing any parent handlers from being notified of the event.
 	 * @return string
 	 */
-	public function _add_event($element, $js, $event, $preventDefault=false, $stopPropagation=false,$immediatly=true) {
+	public function _add_event($element, $js, $event, $preventDefault=false, $stopPropagation=false, $immediatly=true) {
 		if (is_array($js)) {
 			$js=implode("\n\t\t", $js);
 		}
@@ -236,7 +236,7 @@ 
 			$event="\n\t$(".$this->_prep_element($element).").bind('{$event}',function(event){\n\t\t{$js}\n\t});\n";
 		else
 			$event="\n\t$(".$this->_prep_element($element).").{$event}(function(event){\n\t\t{$js}\n\t});\n";
-		if($immediatly)
+		if ($immediatly)
 			$this->jquery_code_for_compile[]=$event;
 		return $event;
 	}
@@ -276,7 +276,7 @@ 
 
 		// External references
 		$external_scripts=implode('', $this->jquery_code_for_load);
-		extract(array (
+		extract(array(
 				'library_src' => $external_scripts
 		));
 
@@ -291,7 +291,7 @@ 
 		$script.='});';
 
 		$this->jquery_code_for_compile=array();
-		if($this->params["debug"]==false){
+		if ($this->params["debug"]==false) {
 			$script=$this->minify($script);
 		}
 		$output=($script_tags===FALSE) ? $script : $this->inline($script);
@@ -311,7 +311,7 @@ 
 	 * @return void
 	 */
 	public function _clear_compile() {
-		$this->jquery_code_for_compile=array ();
+		$this->jquery_code_for_compile=array();
 	}
 
 	/**
@@ -320,12 +320,12 @@ 
 	 */
 	public function _document_ready($js) {
 		if (!is_array($js)) {
-			$js=array (
+			$js=array(
 					$js
 			);
 		}
 
-		foreach ( $js as $script ) {
+		foreach ($js as $script) {
 			$this->jquery_code_for_compile[]=$script;
 		}
 	}
@@ -339,7 +339,7 @@ 
 	 * @return string
 	 */
 	public function _prep_element($element) {
-		if (strrpos($element, 'this')===false&&strrpos($element, 'event')===false&&strrpos($element, 'self')===false) {
+		if (strrpos($element, 'this')===false && strrpos($element, 'event')===false && strrpos($element, 'self')===false) {
 			$element='"'.addslashes($element).'"';
 		}
 		return $element;
@@ -357,14 +357,14 @@ 
 		if (is_array($value)) {
 			$value=implode(",", $value);
 		}
-		if (strrpos($value, 'this')===false&&strrpos($value, 'event')===false&&strrpos($value, 'self')===false) {
+		if (strrpos($value, 'this')===false && strrpos($value, 'event')===false && strrpos($value, 'self')===false) {
 			$value='"'.$value.'"';
 		}
 		return $value;
 	}
 
 	private function minify($input) {
-	if(trim($input) === "") return $input;
+	if (trim($input)==="") return $input;
 	return preg_replace(
 			array(
 					// Remove comment(s)

Ajax/Semantic.php

Doc Comments -1 removed lines

@@ -93,7 +93,6 @@
 	/**
 	 * Return a new Semantic Html Breadcrumb
 	 * @param string $identifier
-	 * @param array $elements
 	 * @param boolean $autoActive sets the last element's class to <b>active</b> if true. default : true
 	 * @param function $hrefFunction the function who generates the href elements. default : function($e){return $e->getContent()}
 	 * @return HtmlBreadcrumb

Spacing +23 added lines, -23 removed lines

@@ -21,7 +21,7 @@ 
 use Ajax\semantic\html\modules\HtmlSticky;
 
 class Semantic extends BaseGui {
-	use SemanticComponentsTrait,SemanticHtmlElementsTrait;
+	use SemanticComponentsTrait, SemanticHtmlElementsTrait;
 	public function __construct($autoCompile=true) {
 		parent::__construct($autoCompile=true);
 	}
@@ -32,24 +32,24 @@ 
 	 * @param array $items
 	 * @return Ajax\semantic\html\collections\HtmlMenu
 	 */
-	public function htmlMenu($identifier,$items=array()){
-		return $this->addHtmlComponent(new HtmlMenu($identifier,$items));
+	public function htmlMenu($identifier, $items=array()) {
+		return $this->addHtmlComponent(new HtmlMenu($identifier, $items));
 	}
 
 	/**Adds an icon menu
 	 * @param string $identifier
 	 * @param array $items icons
 	 */
-	public function htmlIconMenu($identifier,$items=array()){
-		return $this->addHtmlComponent(new HtmlIconMenu($identifier,$items));
+	public function htmlIconMenu($identifier, $items=array()) {
+		return $this->addHtmlComponent(new HtmlIconMenu($identifier, $items));
 	}
 
 	/**Adds an labeled icon menu
 	 * @param string $identifier
 	 * @param array $items icons
 	 */
-	public function htmlLabeledIconMenu($identifier,$items=array()){
-		return $this->addHtmlComponent(new HtmlLabeledIconMenu($identifier,$items));
+	public function htmlLabeledIconMenu($identifier, $items=array()) {
+		return $this->addHtmlComponent(new HtmlLabeledIconMenu($identifier, $items));
 	}
 
 	/**
@@ -57,8 +57,8 @@ 
 	 * @param string $value
 	 * @param array $items
 	 */
-	public function htmlDropdown($identifier, $value="", $items=array()){
-		return $this->addHtmlComponent(new HtmlDropdown($identifier,$value,$items));
+	public function htmlDropdown($identifier, $value="", $items=array()) {
+		return $this->addHtmlComponent(new HtmlDropdown($identifier, $value, $items));
 	}
 
 	/**
@@ -66,16 +66,16 @@ 
 	 * @param string $identifier
 	 * @param string $content
 	 */
-	public function htmlMessage($identifier, $content=""){
-		return $this->addHtmlComponent(new HtmlMessage($identifier,$content));
+	public function htmlMessage($identifier, $content="") {
+		return $this->addHtmlComponent(new HtmlMessage($identifier, $content));
 	}
 
 	/**
 	 * @param string $identifier
 	 * @param mixed $content
 	 */
-	public function htmlPopup(BaseHtml $container,$identifier,$content){
-		return $this->addHtmlComponent(new HtmlPopup($container,$identifier,$content));
+	public function htmlPopup(BaseHtml $container, $identifier, $content) {
+		return $this->addHtmlComponent(new HtmlPopup($container, $identifier, $content));
 	}
 
 	/**
@@ -85,8 +85,8 @@ 
 	 * @param boolean $createCols
 	 * @param boolean $implicitRows
 	 */
-	public function htmlGrid($identifier,$numRows=1,$numCols=NULL,$createCols=true,$implicitRows=false){
-		return $this->addHtmlComponent(new HtmlGrid($identifier,$numRows,$numCols,$createCols,$implicitRows));
+	public function htmlGrid($identifier, $numRows=1, $numCols=NULL, $createCols=true, $implicitRows=false) {
+		return $this->addHtmlComponent(new HtmlGrid($identifier, $numRows, $numCols, $createCols, $implicitRows));
 	}
 
 
@@ -99,8 +99,8 @@ 
 	 * @param function $hrefFunction the function who generates the href elements. default : function($e){return $e->getContent()}
 	 * @return HtmlBreadcrumb
 	 */
-	public function htmlBreadcrumb( $identifier,$items=array(),$autoActive=true,$startIndex=0,$hrefFunction=NULL){
-		return $this->addHtmlComponent(new HtmlBreadcrumb($identifier,$items,$autoActive,$startIndex,$hrefFunction));
+	public function htmlBreadcrumb($identifier, $items=array(), $autoActive=true, $startIndex=0, $hrefFunction=NULL) {
+		return $this->addHtmlComponent(new HtmlBreadcrumb($identifier, $items, $autoActive, $startIndex, $hrefFunction));
 	}
 
 	/**
@@ -117,8 +117,8 @@ 
 	 * @param string $identifier
 	 * @return HtmlAccordion
 	 */
-	public function htmlAccordionMenu($identifier,$items=array()) {
-		return $this->addHtmlComponent(new HtmlAccordionMenu($identifier,$items));
+	public function htmlAccordionMenu($identifier, $items=array()) {
+		return $this->addHtmlComponent(new HtmlAccordionMenu($identifier, $items));
 	}
 
 	/**
@@ -126,11 +126,11 @@ 
 	 * @param string $identifier
 	 * @param array $elements
 	 */
-	public function htmlForm($identifier,$elements=array()) {
-		return $this->addHtmlComponent(new HtmlForm($identifier,$elements));
+	public function htmlForm($identifier, $elements=array()) {
+		return $this->addHtmlComponent(new HtmlForm($identifier, $elements));
 	}
 
-	public function htmlSticky($identifier,$content=array()) {
-		return $this->addHtmlComponent(new HtmlSticky($identifier,$content));
+	public function htmlSticky($identifier, $content=array()) {
+		return $this->addHtmlComponent(new HtmlSticky($identifier, $content));
 	}
 }
\ No newline at end of file

Ajax/common/traits/JqueryActionsTrait.php

Braces +51 added lines, -34 removed lines

@@ -16,10 +16,12 @@ 
 		if (isset($value)) {
 			$value=$this->_prep_value($value);
 			$str="$({$element}).attr(\"$attributeName\",{$value});";
-		} else
-			$str="$({$element}).attr(\"$attributeName\");";
-			if ($immediatly)
-				$this->jquery_code_for_compile[]=$str;
+		} else {
+					$str="$({$element}).attr(\"$attributeName\");";
+		}
+			if ($immediatly) {
+							$this->jquery_code_for_compile[]=$str;
+			}
 				return $str;
 	}
 
@@ -34,8 +36,9 @@ 
 		$element=$this->_prep_element($element);
 		$value=$this->_prep_value($value);
 		$str="$({$element}).after({$value});";
-		if ($immediatly)
-			$this->jquery_code_for_compile[]=$str;
+		if ($immediatly) {
+					$this->jquery_code_for_compile[]=$str;
+		}
 			return $str;
 	}
 
@@ -71,8 +74,9 @@ 
 
 		$str="$({$element}).animate({\n$animations\n\t\t}".$speed.$extra.");";
 
-		if ($immediatly)
-			$this->jquery_code_for_compile[]=$str;
+		if ($immediatly) {
+					$this->jquery_code_for_compile[]=$str;
+		}
 			return $str;
 	}
 
@@ -97,8 +101,9 @@ 
 
 		$str="$({$element}).fadeIn({$speed}{$callback});";
 
-		if ($immediatly)
-			$this->jquery_code_for_compile[]=$str;
+		if ($immediatly) {
+					$this->jquery_code_for_compile[]=$str;
+		}
 			return $str;
 	}
 
@@ -123,8 +128,9 @@ 
 
 		$str="$({$element}).fadeOut({$speed}{$callback});";
 
-		if ($immediatly)
-			$this->jquery_code_for_compile[]=$str;
+		if ($immediatly) {
+					$this->jquery_code_for_compile[]=$str;
+		}
 			return $str;
 	}
 
@@ -149,8 +155,9 @@ 
 
 		$str="$({$element}).hide({$speed}{$callback});";
 
-		if ($immediatly)
-			$this->jquery_code_for_compile[]=$str;
+		if ($immediatly) {
+					$this->jquery_code_for_compile[]=$str;
+		}
 			return $str;
 	}
 
@@ -177,8 +184,9 @@ 
 
 		$str="$({$element}).slideUp({$speed}{$callback});";
 
-		if ($immediatly)
-			$this->jquery_code_for_compile[]=$str;
+		if ($immediatly) {
+					$this->jquery_code_for_compile[]=$str;
+		}
 			return $str;
 	}
 
@@ -203,8 +211,9 @@ 
 
 		$str="$({$element}).slideDown({$speed}{$callback});";
 
-		if ($immediatly)
-			$this->jquery_code_for_compile[]=$str;
+		if ($immediatly) {
+					$this->jquery_code_for_compile[]=$str;
+		}
 			return $str;
 	}
 
@@ -229,8 +238,9 @@ 
 
 		$str="$({$element}).slideToggle({$speed}{$callback});";
 
-		if ($immediatly)
-			$this->jquery_code_for_compile[]=$str;
+		if ($immediatly) {
+					$this->jquery_code_for_compile[]=$str;
+		}
 			return $str;
 	}
 
@@ -247,8 +257,9 @@ 
 		$element=$this->_prep_element($element);
 		$str="$({$element}).toggle();";
 
-		if ($immediatly)
-			$this->jquery_code_for_compile[]=$str;
+		if ($immediatly) {
+					$this->jquery_code_for_compile[]=$str;
+		}
 			return $str;
 	}
 
@@ -264,8 +275,9 @@ 
 		$element=$this->_prep_element($element);
 		$str="$({$element}).trigger(\"$event\");";
 
-		if ($immediatly)
-			$this->jquery_code_for_compile[]=$str;
+		if ($immediatly) {
+					$this->jquery_code_for_compile[]=$str;
+		}
 			return $str;
 	}
 
@@ -290,8 +302,9 @@ 
 
 		$str="$({$element}).show({$speed}{$callback});";
 
-		if ($immediatly)
-			$this->jquery_code_for_compile[]=$str;
+		if ($immediatly) {
+					$this->jquery_code_for_compile[]=$str;
+		}
 			return $str;
 	}
 
@@ -309,8 +322,9 @@ 
 			$str.="else{".$jsCodeIfFalse."}";
 		}
 
-		if ($immediatly)
-			$this->jquery_code_for_compile[]=$str;
+		if ($immediatly) {
+					$this->jquery_code_for_compile[]=$str;
+		}
 			return $str;
 	}
 
@@ -327,11 +341,13 @@ 
 	public function _doJQuery($element, $jqueryCall, $param="", $jsCallback="", $immediatly=false) {
 		$param=$this->_prep_value($param);
 		$callback="";
-		if ($jsCallback!="")
-			$callback=", function(event){\n{$jsCallback}\n}";
+		if ($jsCallback!="") {
+					$callback=", function(event){\n{$jsCallback}\n}";
+		}
 			$script="$(".$this->_prep_element($element).").".$jqueryCall."(".$param.$callback.");\n";
-			if ($immediatly)
-				$this->jquery_code_for_compile[]=$script;
+			if ($immediatly) {
+							$this->jquery_code_for_compile[]=$script;
+			}
 				return $script;
 	}
 
@@ -360,8 +376,9 @@ 
 	 */
 	public function _exec($js, $immediatly=false) {
 		$script=$js."\n";
-		if ($immediatly)
-			$this->jquery_code_for_compile[]=$script;
+		if ($immediatly) {
+					$this->jquery_code_for_compile[]=$script;
+		}
 			return $script;
 	}
 

Doc Comments +11 added lines, -1 removed lines

@@ -3,7 +3,17 @@ 
 namespace Ajax\common\traits;
 
 trait JqueryActionsTrait {
+
+	/**
+	 * @param string $element
+	 * @param string $js
+	 * @param string $event
+	 */
 	public abstract function _add_event($element, $js, $event, $preventDefault=false, $stopPropagation=false,$immediatly=true);
+
+	/**
+	 * @param string $element
+	 */
 	public abstract function _prep_element($element);
 	public abstract function _prep_value($value);
 
@@ -384,7 +394,7 @@ 
 
 	/**
 	 * Ensures the speed parameter is valid for jQuery
-	 * @param string|int $speed
+	 * @param string $speed
 	 * @return string
 	 */
 	private function _validate_speed($speed) {

Spacing +10 added lines, -10 removed lines

@@ -3,7 +3,7 @@ 
 namespace Ajax\common\traits;
 
 trait JqueryActionsTrait {
-	public abstract function _add_event($element, $js, $event, $preventDefault=false, $stopPropagation=false,$immediatly=true);
+	public abstract function _add_event($element, $js, $event, $preventDefault=false, $stopPropagation=false, $immediatly=true);
 	public abstract function _prep_element($element);
 	public abstract function _prep_value($value);
 
@@ -19,7 +19,7 @@ 
 		if (isset($value)) {
 			$value=$this->_prep_value($value);
 			$str="$({$element}).attr(\"$attributeName\",{$value});";
-		} else
+		}else
 			$str="$({$element}).attr(\"$attributeName\");";
 			if ($immediatly)
 				$this->jquery_code_for_compile[]=$str;
@@ -33,7 +33,7 @@ 
 	 * @param boolean $immediatly defers the execution if set to false
 	 * @return string
 	 */
-	public function after($element='this', $value='', $immediatly=false){
+	public function after($element='this', $value='', $immediatly=false) {
 		$element=$this->_prep_element($element);
 		$value=$this->_prep_value($value);
 		$str="$({$element}).after({$value});";
@@ -58,7 +58,7 @@ 
 
 		$animations="\t\t\t";
 		if (is_array($params)) {
-			foreach ( $params as $param => $value ) {
+			foreach ($params as $param => $value) {
 				$animations.=$param.': \''.$value.'\', ';
 			}
 		}
@@ -351,8 +351,8 @@ 
 	 * @param boolean $immediatly
 	 * @return string
 	 */
-	public function _doJQueryOn($event, $element, $elementToModify, $jqueryCall, $param="", $preventDefault=false, $stopPropagation=false, $jsCallback="",$immediatly=true) {
-		return $this->_add_event($element, $this->_doJQuery($elementToModify, $jqueryCall, $param, $jsCallback), $event, $preventDefault, $stopPropagation,$immediatly);
+	public function _doJQueryOn($event, $element, $elementToModify, $jqueryCall, $param="", $preventDefault=false, $stopPropagation=false, $jsCallback="", $immediatly=true) {
+		return $this->_add_event($element, $this->_doJQuery($elementToModify, $jqueryCall, $param, $jsCallback), $event, $preventDefault, $stopPropagation, $immediatly);
 	}
 
 	/**
@@ -378,8 +378,8 @@ 
 	 * @param boolean $immediatly
 	 * @return String
 	 */
-	public function _execOn($element, $event, $js, $preventDefault=false, $stopPropagation=false,$immediatly=true) {
-		return $this->_add_event($element, $this->_exec($js), $event, $preventDefault, $stopPropagation,$immediatly);
+	public function _execOn($element, $event, $js, $preventDefault=false, $stopPropagation=false, $immediatly=true) {
+		return $this->_add_event($element, $this->_exec($js), $event, $preventDefault, $stopPropagation, $immediatly);
 	}
 
 	/**
@@ -388,8 +388,8 @@ 
 	 * @return string
 	 */
 	private function _validate_speed($speed) {
-		if (in_array($speed, array (
-				'slow','normal','fast'
+		if (in_array($speed, array(
+				'slow', 'normal', 'fast'
 		))) {
 			$speed='"'.$speed.'"';
 		} elseif (preg_match("/[^0-9]/", $speed)) {