jcheron / phalcon-jquery

Ajax/semantic/traits/SemanticHtmlCollectionsTrait.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
 	 * @param int $rowCount
 	 * @param int $colCount
 	 */
-	public function htmlTable($identifier, $rowCount, $colCount){
+	public function htmlTable($identifier, $rowCount, $colCount) {
 		return $this->addHtmlComponent(new HtmlTable($identifier, $rowCount, $colCount));
 	}
 }
\ No newline at end of file

Ajax/semantic/html/collections/form/traits/CheckboxTrait.php

Doc Comments +3 added lines

@@ -6,6 +6,9 @@
 
 trait CheckboxTrait {
 
+	/**
+	 * @param string $name
+	 */
 	public abstract function addToPropertyCtrl($name, $value, $typeCtrl);
 
 	public function setType($checkboxType) {

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

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
 class HtmlFormCheckbox extends HtmlFormField {
 	use CheckboxTrait;
 	public function __construct($identifier, $label=NULL, $value=NULL, $type=NULL) {
-		parent::__construct("field-".$identifier, new HtmlCheckbox($identifier,$label,$value,$type));
+		parent::__construct("field-".$identifier, new HtmlCheckbox($identifier, $label, $value, $type));
 	}
 
 	public static function slider($identifier, $label="", $value=NULL) {

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

Spacing +7 added lines, -7 removed lines

@@ -29,13 +29,13 @@ 
 				if (\is_string($end)) {
 					$label=$end;
 					\array_pop($fields);
-				} else
+				}else
 					$label=NULL;
 			}
 		}
 		if (isset($label))
 			$this->setLabel($label);
-		foreach ( $fields as $field ) {
+		foreach ($fields as $field) {
 			$this->addItem($field);
 		}
 		return $this;
@@ -65,7 +65,7 @@ 
 		if ($this->_equalWidth) {
 			$count=$this->count();
 			$this->addToProperty("class", Wide::getConstants()["W".$count]." fields");
-		} else
+		}else
 			$this->addToProperty("class", "fields");
 		return parent::compile($js, $view);
 	}
@@ -96,9 +96,9 @@ 
 	}
 
 	public static function radios($name, $items=array(), $label=NULL, $value=null, $type=NULL) {
-		$fields=array ();
+		$fields=array();
 		$i=0;
-		foreach ( $items as $val => $caption ) {
+		foreach ($items as $val => $caption) {
 			$itemO=new HtmlFormRadio($name."-".$i++, $name, $caption, $val, $type);
 			if ($val===$value) {
 				$itemO->getField()->getField()->setProperty("checked", "");
@@ -112,9 +112,9 @@ 
 	}
 
 	public static function checkeds($name, $items=array(), $label=NULL, $values=array(), $type=NULL) {
-		$fields=array ();
+		$fields=array();
 		$i=0;
-		foreach ( $items as $val => $caption ) {
+		foreach ($items as $val => $caption) {
 			$itemO=new HtmlFormCheckbox($name."-".$i++, $name, $caption, $val, $type);
 			if (\array_search($val, $values)!==false) {
 				$itemO->getField()->getField()->setProperty("checked", "");

Braces +15 added lines, -10 removed lines

@@ -29,12 +29,14 @@ 
 				if (\is_string($end)) {
 					$label=$end;
 					\array_pop($fields);
-				} else
-					$label=NULL;
+				} else {
+									$label=NULL;
+				}
 			}
 		}
-		if (isset($label))
-			$this->setLabel($label);
+		if (isset($label)) {
+					$this->setLabel($label);
+		}
 		foreach ( $fields as $field ) {
 			$this->addItem($field);
 		}
@@ -65,8 +67,9 @@ 
 		if ($this->_equalWidth) {
 			$count=$this->count();
 			$this->addToProperty("class", Wide::getConstants()["W".$count]." fields");
-		} else
-			$this->addToProperty("class", "fields");
+		} else {
+					$this->addToProperty("class", "fields");
+		}
 		return parent::compile($js, $view);
 	}
 
@@ -106,8 +109,9 @@ 
 			$fields[]=$itemO;
 		}
 		$radios=new HtmlFormFields("fields-".$name, $fields);
-		if (isset($label))
-			$radios->setLabel($label)->setProperty("for", $name);
+		if (isset($label)) {
+					$radios->setLabel($label)->setProperty("for", $name);
+		}
 		return $radios;
 	}
 
@@ -122,8 +126,9 @@ 
 			$fields[]=$itemO;
 		}
 		$radios=new HtmlFormFields("fields-".$name, $fields);
-		if (isset($label))
-			$radios->setLabel($label)->setProperty("for", $name);
+		if (isset($label)) {
+					$radios->setLabel($label)->setProperty("for", $name);
+		}
 		return $radios;
 	}
 

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use Ajax\semantic\html\base\HtmlSemCollection;
 use Ajax\semantic\html\base\constants\Wide;
 use Ajax\JsUtils;
-
 use Ajax\semantic\html\base\HtmlSemDoubleElement;
 use Ajax\semantic\html\collections\form\traits\FieldsTrait;
 

Doc Comments +3 added lines

@@ -15,6 +15,9 @@
 	protected $_equalWidth;
 	protected $_name;
 
+	/**
+	 * @param string $identifier
+	 */
 	public function __construct($identifier, $fields=array(), $equalWidth=true) {
 		parent::__construct($identifier, "div");
 		$this->_equalWidth=$equalWidth;

Ajax/semantic/html/modules/checkbox/AbstractCheckbox.php

Spacing +4 added lines, -4 removed lines

@@ -6,7 +6,7 @@ 
 use Ajax\semantic\html\base\constants\CheckboxType;
 
 abstract class AbstractCheckbox extends HtmlSemDoubleElement {
-	protected $_params=array ();
+	protected $_params=array();
 
 	public function __construct($identifier, $name=NULL, $label=NULL, $value=NULL, $inputType="checkbox", $type="checkbox") {
 		parent::__construct("ck-".$identifier, "div", "ui ".$type);
@@ -73,9 +73,9 @@ 
 	 * @return \Ajax\semantic\html\collections\form\AbstractHtmlFormRadioCheckbox
 	 */
 	public function attachEvent($selector, $action=NULL) {
-		if (isset($action)!==false||\is_numeric($action)===true) {
+		if (isset($action)!==false || \is_numeric($action)===true) {
 			$js='$("#%identifier%").checkbox("attach events", "'.$selector.'", "'.$action.'");';
-		} else {
+		}else {
 			$js='$("#%identifier%").checkbox("attach events", "'.$selector.'");';
 		}
 		$js=\str_replace("%identifier%", $this->identifier, $js);
@@ -89,7 +89,7 @@ 
 	 */
 	public function attachEvents($events=array()) {
 		if (\is_array($events)) {
-			foreach ( $events as $action => $selector ) {
+			foreach ($events as $action => $selector) {
 				$this->attachEvent($selector, $action);
 			}
 		}

Braces +6 added lines, -4 removed lines

@@ -13,8 +13,9 @@ 
 		$field=new \Ajax\common\html\html5\HtmlInput($identifier, $inputType, $value);
 		$field->setProperty("name", $name);
 		$this->setField($field);
-		if (isset($label))
-			$this->setLabel($label);
+		if (isset($label)) {
+					$this->setLabel($label);
+		}
 	}
 
 	public function setType($checkboxType) {
@@ -40,8 +41,9 @@ 
 	 * @return mixed
 	 */
 	public function getLabel() {
-		if (\array_key_exists("label", $this->content))
-			return $this->content["label"];
+		if (\array_key_exists("label", $this->content)) {
+					return $this->content["label"];
+		}
 	}
 
 	/**

Ajax/semantic/html/collections/menus/HtmlMenu.php

Braces +11 added lines, -9 removed lines

@@ -55,9 +55,9 @@ 
 	}
 
 	private function afterInsert($item) {
-		if (!$item instanceof HtmlMenu)
-			$item->addToPropertyCtrl("class", "item", array ("item" ));
-		else {
+		if (!$item instanceof HtmlMenu) {
+					$item->addToPropertyCtrl("class", "item", array ("item" ));
+		} else {
 			$this->setSecondary();
 		}
 		return $item;
@@ -155,8 +155,9 @@ 
 		$this->apply(function (HtmlDoubleElement &$item) {
 			$item->setTagName("a");
 		});
-		if ($vertical === true)
-			$this->setVertical();
+		if ($vertical === true) {
+					$this->setVertical();
+		}
 		return $this->addToProperty("class", "tabular");
 	}
 
@@ -185,10 +186,11 @@ 
 	 */
 	public function fromDatabaseObject($object, $function) {
 		$return=$function($object);
-		if (\is_array($return))
-			$this->addItems($return);
-		else
-			$this->addItem($return);
+		if (\is_array($return)) {
+					$this->addItems($return);
+		} else {
+					$this->addItem($return);
+		}
 	}
 
 	/**

Doc Comments +2 added lines, -1 removed lines

@@ -33,7 +33,7 @@ 
 	/**
 	 * Sets the menu type
 	 * @param string $type one of text,item
-	 * @return \Ajax\semantic\html\collections\HtmlMenu
+	 * @return HtmlMenu
 	 */
 	public function setType($type="") {
 		return $this->addToPropertyCtrl("class", $type, array ("","item","text" ));
@@ -81,6 +81,7 @@ 
 	 * {@inheritDoc}
 	 *
 	 * @see \Ajax\common\html\HtmlCollection::insertItem()
+	 * @param HtmlIcon $item
 	 */
 	public function insertItem($item, $position=0) {
 		$item=parent::insertItem($this->getItemToInsert($item), $position);

Spacing +17 added lines, -17 removed lines

@@ -36,12 +36,12 @@ 
 	 * @return \Ajax\semantic\html\collections\HtmlMenu
 	 */
 	public function setType($type="") {
-		return $this->addToPropertyCtrl("class", $type, array ("","item","text" ));
+		return $this->addToPropertyCtrl("class", $type, array("", "item", "text"));
 	}
 
 	public function setActiveItem($index) {
 		$item=$this->getItem($index);
-		if ($item !== null) {
+		if ($item!==null) {
 			$item->addToProperty("class", "active");
 		}
 		return $this;
@@ -49,7 +49,7 @@ 
 
 	private function getItemToInsert($item) {
 		if ($item instanceof HtmlInput || $item instanceof HtmlImg || $item instanceof HtmlIcon || $item instanceof HtmlButton || ($item instanceof HtmlDropdown && $this->propertyContains("class", "vertical")===false)) {
-			$itemO=new HtmlSemDoubleElement("item-" . $this->identifier, "div", "");
+			$itemO=new HtmlSemDoubleElement("item-".$this->identifier, "div", "");
 			$itemO->setContent($item);
 			$item=$itemO;
 		}
@@ -58,7 +58,7 @@ 
 
 	private function afterInsert($item) {
 		if (!$item instanceof HtmlMenu)
-			$item->addToPropertyCtrl("class", "item", array ("item" ));
+			$item->addToPropertyCtrl("class", "item", array("item"));
 		else {
 			$this->setSecondary();
 		}
@@ -89,9 +89,9 @@ 
 
 	public function generateMenuAsItem($menu, $header=null) {
 		$count=$this->count();
-		$item=new HtmlSemDoubleElement("item-" . $this->identifier . "-" . $count, "div");
+		$item=new HtmlSemDoubleElement("item-".$this->identifier."-".$count, "div");
 		if (isset($header)) {
-			$headerItem=new HtmlSemDoubleElement("item-header-" . $this->identifier . "-" . $count, "div", "header");
+			$headerItem=new HtmlSemDoubleElement("item-header-".$this->identifier."-".$count, "div", "header");
 			$headerItem->setContent($header);
 			$item->addContent($headerItem);
 			$this->_itemHeader=$headerItem;
@@ -109,7 +109,7 @@ 
 		$value=new HtmlSemDoubleElement($identifier, "a", "browse item", $value);
 		$value->addContent(new HtmlIcon("", "dropdown"));
 		$value=$this->addItem($value);
-		$popup=new HtmlPopup($value, "popup-" . $this->identifier . "-" . $this->count(), $content);
+		$popup=new HtmlPopup($value, "popup-".$this->identifier."-".$this->count(), $content);
 		$popup->setFlowing()->setPosition("bottom left")->setOn("click");
 		$this->wrap("", $popup);
 		return $popup;
@@ -118,7 +118,7 @@ 
 	public function addDropdownAsItem($value, $items=NULL) {
 		$dd=$value;
 		if (\is_string($value)) {
-			$dd=new HtmlDropdown("dropdown-" . $this->identifier . "-" . $this->count(), $value, $items);
+			$dd=new HtmlDropdown("dropdown-".$this->identifier."-".$this->count(), $value, $items);
 		}
 		return $this->addItem($dd);
 	}
@@ -130,7 +130,7 @@ 
 	 * @see \Ajax\common\html\html5\HtmlCollection::createItem()
 	 */
 	protected function createItem($value) {
-		$itemO=new HtmlLink("item-" . \sizeof($this->content), "", $value);
+		$itemO=new HtmlLink("item-".\sizeof($this->content), "", $value);
 		return $itemO->setClass("item");
 	}
 
@@ -143,28 +143,28 @@ 
 	}
 
 	public function setVertical() {
-		return $this->addToPropertyCtrl("class", "vertical", array ("vertical" ));
+		return $this->addToPropertyCtrl("class", "vertical", array("vertical"));
 	}
 
 	public function setPosition($value="right") {
-		return $this->addToPropertyCtrl("class", $value, array ("right","left" ));
+		return $this->addToPropertyCtrl("class", $value, array("right", "left"));
 	}
 
 	public function setPointing($value=Direction::NONE) {
-		return $this->addToPropertyCtrl("class", $value . " pointing", Direction::getConstantValues("pointing"));
+		return $this->addToPropertyCtrl("class", $value." pointing", Direction::getConstantValues("pointing"));
 	}
 
 	public function asTab($vertical=false) {
-		$this->apply(function (HtmlDoubleElement &$item) {
+		$this->apply(function(HtmlDoubleElement & $item) {
 			$item->setTagName("a");
 		});
-		if ($vertical === true)
+		if ($vertical===true)
 			$this->setVertical();
 		return $this->addToProperty("class", "tabular");
 	}
 
 	public function asPagination() {
-		$this->apply(function (HtmlDoubleElement &$item) {
+		$this->apply(function(HtmlDoubleElement & $item) {
 			$item->setTagName("a");
 		});
 		return $this->addToProperty("class", "pagination");
@@ -201,10 +201,10 @@ 
 	 */
 	public function setWidth($width) {
 		if (\is_int($width)) {
-			$width=Wide::getConstants()["W" . $width];
+			$width=Wide::getConstants()["W".$width];
 		}
 		$this->addToPropertyCtrl("class", $width, Wide::getConstants());
-		return $this->addToPropertyCtrl("class", "item", array ("item" ));
+		return $this->addToPropertyCtrl("class", "item", array("item"));
 	}
 
 	public function addImage($identifier, $src="", $alt="") {

Ajax/semantic/html/content/HtmlGridCol.php

Spacing +2 added lines, -2 removed lines

@@ -30,10 +30,10 @@
 	 */
 	public function setWidth($width) {
 		if (\is_int($width)) {
-			$width=Wide::getConstants()["W" . $width];
+			$width=Wide::getConstants()["W".$width];
 		}
 		$this->addToPropertyCtrl("class", $width, Wide::getConstants());
-		return $this->addToPropertyCtrl("class", "wide", array ("wide" ));
+		return $this->addToPropertyCtrl("class", "wide", array("wide"));
 	}
 
 	public function setValue($value) {

Braces +8 added lines, -6 removed lines

@@ -20,8 +20,9 @@ 
 	public function __construct($identifier, $width=NULL) {
 		parent::__construct($identifier, "div");
 		$this->setClass("column");
-		if (isset($width))
-			$this->setWidth($width);
+		if (isset($width)) {
+					$this->setWidth($width);
+		}
 	}
 
 	/**
@@ -48,10 +49,11 @@ 
 
 	public function addDivider($vertical=true, $content=NULL) {
 		$divider=new HtmlDivider("", $content);
-		if ($vertical)
-			$divider->setVertical();
-		else
-			$divider->setHorizontal();
+		if ($vertical) {
+					$divider->setVertical();
+		} else {
+					$divider->setHorizontal();
+		}
 		$this->wrap("", $divider);
 		return $divider;
 	}

Ajax/semantic/html/base/traits/BaseTrait.php

Doc Comments +12 added lines

@@ -11,12 +11,24 @@
 	protected $_states=[ ];
 	protected $_baseClass;
 
+	/**
+	 * @param string $name
+	 */
 	protected abstract function setPropertyCtrl($name, $value, $typeCtrl);
 
+	/**
+	 * @param string $name
+	 */
 	protected abstract function addToPropertyCtrl($name, $value, $typeCtrl);
 
+	/**
+	 * @param string $name
+	 */
 	protected abstract function addToPropertyCtrlCheck($name, $value, $typeCtrl);
 
+	/**
+	 * @param string $name
+	 */
 	public abstract function addToProperty($name, $value, $separator=" ");
 
 	public function addVariation($variation) {

Braces +12 added lines, -8 removed lines

@@ -34,8 +34,9 @@ 
 
 	public function setVariations($variations) {
 		$this->setProperty("class", $this->_baseClass);
-		if (\is_string($variations))
-			$variations=\explode(" ", $variations);
+		if (\is_string($variations)) {
+					$variations=\explode(" ", $variations);
+		}
 		foreach ( $variations as $variation ) {
 			$this->addVariation($variation);
 		}
@@ -48,8 +49,9 @@ 
 	}
 
 	public function addVariations($variations=array()) {
-		if (\is_string($variations))
-			$variations=\explode(" ", $variations);
+		if (\is_string($variations)) {
+					$variations=\explode(" ", $variations);
+		}
 		foreach ( $variations as $variation ) {
 			$this->addVariation($variation);
 		}
@@ -57,8 +59,9 @@ 
 	}
 
 	public function addStates($states=array()) {
-		if (\is_string($states))
-			$states=\explode(" ", $states);
+		if (\is_string($states)) {
+					$states=\explode(" ", $states);
+		}
 		foreach ( $states as $state ) {
 			$this->addState($state);
 		}
@@ -67,8 +70,9 @@ 
 
 	public function setStates($states) {
 		$this->setProperty("class", $this->_baseClass);
-		if (\is_string($states))
-			$states=\explode(" ", $states);
+		if (\is_string($states)) {
+					$states=\explode(" ", $states);
+		}
 		foreach ( $states as $state ) {
 			$this->addState($state);
 		}

Spacing +8 added lines, -8 removed lines

@@ -8,8 +8,8 @@ 
 use Ajax\semantic\html\elements\HtmlIcon;
 
 trait BaseTrait {
-	protected $_variations=[ ];
-	protected $_states=[ ];
+	protected $_variations=[];
+	protected $_states=[];
 	protected $_baseClass;
 
 	protected abstract function setPropertyCtrl($name, $value, $typeCtrl);
@@ -37,7 +37,7 @@ 
 		$this->setProperty("class", $this->_baseClass);
 		if (\is_string($variations))
 			$variations=\explode(" ", $variations);
-		foreach ( $variations as $variation ) {
+		foreach ($variations as $variation) {
 			$this->addVariation($variation);
 		}
 		return $this;
@@ -51,7 +51,7 @@ 
 	public function addVariations($variations=array()) {
 		if (\is_string($variations))
 			$variations=\explode(" ", $variations);
-		foreach ( $variations as $variation ) {
+		foreach ($variations as $variation) {
 			$this->addVariation($variation);
 		}
 		return $this;
@@ -60,7 +60,7 @@ 
 	public function addStates($states=array()) {
 		if (\is_string($states))
 			$states=\explode(" ", $states);
-		foreach ( $states as $state ) {
+		foreach ($states as $state) {
 			$this->addState($state);
 		}
 		return $this;
@@ -70,14 +70,14 @@ 
 		$this->setProperty("class", $this->_baseClass);
 		if (\is_string($states))
 			$states=\explode(" ", $states);
-		foreach ( $states as $state ) {
+		foreach ($states as $state) {
 			$this->addState($state);
 		}
 		return $this;
 	}
 
 	public function addIcon($icon, $before=true) {
-		return $this->addContent(new HtmlIcon("icon-" . $this->identifier, $icon), $before);
+		return $this->addContent(new HtmlIcon("icon-".$this->identifier, $icon), $before);
 	}
 
 	/**
@@ -127,7 +127,7 @@ 
 	}
 
 	public function setFloated($direction="right") {
-		return $this->addToPropertyCtrl("class", $direction . " floated", Direction::getConstantValues("floated"));
+		return $this->addToPropertyCtrl("class", $direction." floated", Direction::getConstantValues("floated"));
 	}
 
 	public function floatRight() {

Ajax/service/JArray.php

Spacing +15 added lines, -15 removed lines

@@ -5,7 +5,7 @@ 
 class JArray {
 
 	public static function isAssociative($array) {
-		return (array_values($array) !== $array);
+		return (array_values($array)!==$array);
 		// return (array_keys($array)!==range(0, count($array)-1));
 	}
 
@@ -14,7 +14,7 @@ 
 			return $array[$key];
 		}
 		$values=array_values($array);
-		if ($pos < sizeof($values))
+		if ($pos<sizeof($values))
 			return $values[$pos];
 	}
 
@@ -22,12 +22,12 @@ 
 		$result=NULL;
 		if (array_key_exists($key, $array)) {
 			$result=$array[$key];
-			if ($condition($result) === true)
+			if ($condition($result)===true)
 				return $result;
 		}
 		$values=array_values($array);
-		foreach ( $values as $val ) {
-			if ($condition($val) === true)
+		foreach ($values as $val) {
+			if ($condition($val)===true)
 				return $val;
 		}
 		return $result;
@@ -36,7 +36,7 @@ 
 	public static function getDefaultValue($array, $key, $default=NULL) {
 		if (array_key_exists($key, $array)) {
 			return $array[$key];
-		} else
+		}else
 			return $default;
 	}
 
@@ -44,13 +44,13 @@ 
 		$result="";
 		if (\is_array($glue)) {
 			$size=\sizeof($pieces);
-			if ($size > 0) {
-				for($i=0; $i < $size - 1; $i++) {
-					$result.=$pieces[$i] . @$glue[$i];
+			if ($size>0) {
+				for ($i=0; $i<$size-1; $i++) {
+					$result.=$pieces[$i].@$glue[$i];
 				}
-				$result.=$pieces[$size - 1];
+				$result.=$pieces[$size-1];
 			}
-		} else {
+		}else {
 			$result=\implode($glue, $pieces);
 		}
 		return $result;
@@ -58,16 +58,16 @@ 
 
 	public static function dimension($array) {
 		if (is_array(reset($array))) {
-			$return=self::dimension(reset($array)) + 1;
-		} else {
+			$return=self::dimension(reset($array))+1;
+		}else {
 			$return=1;
 		}
 		return $return;
 	}
 
 	public static function sortAssociative($array, $sortedKeys=array()) {
-		$newArray=array ();
-		foreach ( $sortedKeys as $key ) {
+		$newArray=array();
+		foreach ($sortedKeys as $key) {
 			if (\array_key_exists($key, $array)) {
 				$newArray[$key]=$array[$key];
 			}

Braces +12 added lines, -8 removed lines

@@ -14,21 +14,24 @@ 
 			return $array[$key];
 		}
 		$values=array_values($array);
-		if ($pos < sizeof($values))
-			return $values[$pos];
+		if ($pos < sizeof($values)) {
+					return $values[$pos];
+		}
 	}
 
 	public static function getConditionalValue($array, $key, $condition) {
 		$result=NULL;
 		if (array_key_exists($key, $array)) {
 			$result=$array[$key];
-			if ($condition($result) === true)
-				return $result;
+			if ($condition($result) === true) {
+							return $result;
+			}
 		}
 		$values=array_values($array);
 		foreach ( $values as $val ) {
-			if ($condition($val) === true)
-				return $val;
+			if ($condition($val) === true) {
+							return $val;
+			}
 		}
 		return $result;
 	}
@@ -36,8 +39,9 @@ 
 	public static function getDefaultValue($array, $key, $default=NULL) {
 		if (array_key_exists($key, $array)) {
 			return $array[$key];
-		} else
-			return $default;
+		} else {
+					return $default;
+		}
 	}
 
 	public static function implode($glue, $pieces) {