phpMv / phpMv-UI

Ajax/common/html/HtmlCollection.php

Doc Comments +6 added lines

@@ -84,6 +84,9 @@ 
 		return $this;
 	}
 
+	/**
+	 * @param integer $index
+	 */
 	public function removeItem($index){
 		return array_splice($this->content, $index, 1);
 	}
@@ -99,6 +102,9 @@ 
 		return $this->addItem($function($object));
 	}
 
+	/**
+	 * @param \Closure $callBack
+	 */
 	public function apply($callBack){
 		foreach ($this->content as $item){
 			$callBack($item);

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use Ajax\common\html\HtmlDoubleElement;
 use Ajax\service\JArray;
-use Ajax\JsUtils;
 
 /**
  * Base class for Html collections

Spacing +22 added lines, -22 removed lines

@@ -13,36 +13,36 @@ 
  */
 abstract class HtmlCollection extends HtmlDoubleElement {
 
-	public function __construct($identifier,$tagName="div"){
-		parent::__construct($identifier,$tagName);
+	public function __construct($identifier, $tagName="div") {
+		parent::__construct($identifier, $tagName);
 		$this->content=array();
 	}
 
-	public function addItems($items){
-		if(JArray::isAssociative($items)){
-			foreach ($items as $k=>$v){
-				$this->addItem([$k,$v]);
+	public function addItems($items) {
+		if (JArray::isAssociative($items)) {
+			foreach ($items as $k=>$v) {
+				$this->addItem([$k, $v]);
 			}
-		}else{
-			foreach ($items as $item){
+		} else {
+			foreach ($items as $item) {
 				$this->addItem($item);
 			}
 		}
 		return $this;
 	}
 
-	public function setItems($items){
+	public function setItems($items) {
 		$this->content=$items;
 		return $this;
 	}
 
-	public function getItems(){
+	public function getItems() {
 		return $this->content;
 	}
 
-	protected function getItemToAdd($item){
+	protected function getItemToAdd($item) {
 		$itemO=$item;
-		if($this->createCondition($item)===true){
+		if ($this->createCondition($item)===true) {
 			$itemO=$this->createItem($item);
 		}
 		return $itemO;
@@ -53,15 +53,15 @@ 
 	 * @param HtmlDoubleElement|string $item
 	 * @return \Ajax\common\html\HtmlDoubleElement
 	 */
-	public function addItem($item){
+	public function addItem($item) {
 		$itemO=$this->getItemToAdd($item);
 		$this->addContent($itemO);
 		return $itemO;
 	}
 
-	public function insertItem($item,$position=0){
+	public function insertItem($item, $position=0) {
 		$itemO=$this->getItemToAdd($item);
-		\array_splice( $this->content, $position, 0, array($itemO));
+		\array_splice($this->content, $position, 0, array($itemO));
 		return $itemO;
 	}
 
@@ -84,11 +84,11 @@ 
 		return $this;
 	}
 
-	public function removeItem($index){
+	public function removeItem($index) {
 		return array_splice($this->content, $index, 1);
 	}
 
-	public function count(){
+	public function count() {
 		return \sizeof($this->content);
 	}
 
@@ -99,8 +99,8 @@ 
 		return $this->addItem($function($object));
 	}
 
-	public function apply($callBack){
-		foreach ($this->content as $item){
+	public function apply($callBack) {
+		foreach ($this->content as $item) {
 			$callBack($item);
 		}
 		return $this;
@@ -120,12 +120,12 @@ 
 	 */
 	protected abstract function createItem($value);
 
-	protected function createCondition($value){
+	protected function createCondition($value) {
 		return \is_object($value)===false;
 	}
 
-	protected function contentAs($tagName){
-		foreach ($this->content as $item){
+	protected function contentAs($tagName) {
+		foreach ($this->content as $item) {
 			$item->setTagName($tagName);
 		}
 		return $this;

Braces +4 added lines, -4 removed lines

@@ -23,7 +23,7 @@ 
 			foreach ($items as $k=>$v){
 				$this->addItem([$k,$v]);
 			}
-		}else{
+		} else{
 			foreach ($items as $item){
 				$this->addItem($item);
 			}
@@ -71,9 +71,9 @@ 
 	 * @return \Ajax\common\html\HtmlDoubleElement
 	 */
 	public function getItem($index) {
-		if (is_int($index))
-			return $this->content[$index];
-		else {
+		if (is_int($index)) {
+					return $this->content[$index];
+		} else {
 			$elm=$this->getElementById($index, $this->content);
 			return $elm;
 		}

Ajax/php/yii/URI.php

Doc Comments +1 added lines, -1 removed lines

@@ -429,7 +429,7 @@
 	 *
 	 * @access public
 	 * @param array an associative array of key/values
-	 * @return array
+	 * @return string
 	 *
 	 */
 	function assoc_to_uri($array) {

Spacing +41 added lines, -41 removed lines

@@ -10,7 +10,7 @@ 
 	 * @access public
 	 *
 	 */
-	var $keyval=array ();
+	var $keyval=array();
 	/**
 	 * Current uri string
 	 *
@@ -26,7 +26,7 @@ 
 	 * @access public
 	 *
 	 */
-	var $segments=array ();
+	var $segments=array();
 	/**
 	 * Re-indexed list of uri segments
 	 * Starts at 1 instead of 0
@@ -35,7 +35,7 @@ 
 	 * @access public
 	 *
 	 */
-	var $rsegments=array ();
+	var $rsegments=array();
 	var $permitted_uri_chars="a-z 0-9~%.:_\-";
 	var $enable_query_strings=false;
 
@@ -75,9 +75,9 @@ 
 	 */
 	function _fetch_uri_string() {
 		$uri="";
-		if ($this->uri_protocol == 'AUTO') {
+		if ($this->uri_protocol=='AUTO') {
 			// Is the request coming from the command line?
-			if (php_sapi_name() == 'cli' or defined('STDIN')) {
+			if (php_sapi_name()=='cli' or defined('STDIN')) {
 				$this->_set_uri_string($this->_parse_cli_args());
 				return;
 			}
@@ -91,20 +91,20 @@ 
 			// Is there a PATH_INFO variable?
 			// Note: some servers seem to have trouble with getenv() so we'll test it two ways
 			$path=(isset($_SERVER['PATH_INFO'])) ? $_SERVER['PATH_INFO'] : @getenv('PATH_INFO');
-			if (trim($path, '/') != '' && $path != "/" . SELF) {
+			if (trim($path, '/')!='' && $path!="/".SELF) {
 				$this->_set_uri_string($path);
 				return;
 			}
 
 			// No PATH_INFO?... What about QUERY_STRING?
 			$path=(isset($_SERVER['QUERY_STRING'])) ? $_SERVER['QUERY_STRING'] : @getenv('QUERY_STRING');
-			if (trim($path, '/') != '') {
+			if (trim($path, '/')!='') {
 				$this->_set_uri_string($path);
 				return;
 			}
 
 			// As a last ditch effort lets try using the $_GET array
-			if (is_array($_GET) && count($_GET) == 1 && trim(key($_GET), '/') != '') {
+			if (is_array($_GET) && count($_GET)==1 && trim(key($_GET), '/')!='') {
 				$this->_set_uri_string(key($_GET));
 				return;
 			}
@@ -114,7 +114,7 @@ 
 			return;
 		}
 
-		if ($uri == 'REQUEST_URI') {
+		if ($uri=='REQUEST_URI') {
 			$this->_set_uri_string($this->_detect_uri());
 			return;
 		}
@@ -138,7 +138,7 @@ 
 		// $str = remove_invisible_characters($str, FALSE);
 
 		// If the URI contains only a slash we'll kill it
-		$this->uri_string=($str == '/') ? '' : $str;
+		$this->uri_string=($str=='/') ? '' : $str;
 	}
 
 	// --------------------------------------------------------------------
@@ -159,15 +159,15 @@ 
 		}
 
 		$uri=$_SERVER['REQUEST_URI'];
-		if (strpos($uri, $_SERVER['SCRIPT_NAME']) === 0) {
+		if (strpos($uri, $_SERVER['SCRIPT_NAME'])===0) {
 			$uri=substr($uri, strlen($_SERVER['SCRIPT_NAME']));
-		} elseif (strpos($uri, dirname($_SERVER['SCRIPT_NAME'])) === 0) {
+		} elseif (strpos($uri, dirname($_SERVER['SCRIPT_NAME']))===0) {
 			$uri=substr($uri, strlen(dirname($_SERVER['SCRIPT_NAME'])));
 		}
 
 		// This section ensures that even on servers that require the URI to be in the query string (Nginx) a correct
 		// URI is found, and also fixes the QUERY_STRING server var and $_GET array.
-		if (strncmp($uri, '?/', 2) === 0) {
+		if (strncmp($uri, '?/', 2)===0) {
 			$uri=substr($uri, 2);
 		}
 		$parts=preg_split('#\?#i', $uri, 2);
@@ -177,17 +177,17 @@ 
 			parse_str($_SERVER['QUERY_STRING'], $_GET);
 		} else {
 			$_SERVER['QUERY_STRING']='';
-			$_GET=array ();
+			$_GET=array();
 		}
 
-		if ($uri == '/' || empty($uri)) {
+		if ($uri=='/' || empty($uri)) {
 			return '/';
 		}
 
 		$uri=parse_url($uri, PHP_URL_PATH);
 
 		// Do some final cleaning of the URI and return it
-		return str_replace(array ('//','../' ), '/', trim($uri, '/'));
+		return str_replace(array('//', '../'), '/', trim($uri, '/'));
 	}
 
 	// --------------------------------------------------------------------
@@ -204,7 +204,7 @@ 
 	private function _parse_cli_args() {
 		$args=array_slice($_SERVER['argv'], 1);
 
-		return $args ? '/' . implode('/', $args) : '';
+		return $args ? '/'.implode('/', $args) : '';
 	}
 
 	// --------------------------------------------------------------------
@@ -218,17 +218,17 @@ 
 	 *
 	 */
 	function _filter_uri($str) {
-		if ($str != '' && $this->permitted_uri_chars != '' && $this->enable_query_strings == FALSE) {
+		if ($str!='' && $this->permitted_uri_chars!='' && $this->enable_query_strings==FALSE) {
 			// preg_quote() in PHP 5.3 escapes -, so the str_replace() and addition of - to preg_quote() is to maintain backwards
 			// compatibility as many are unaware of how characters in the permitted_uri_chars will be parsed as a regex pattern
-			if (!preg_match("|^[" . str_replace(array ('\\-','\-' ), '-', preg_quote($this->permitted_uri_chars, '-')) . "]+$|i", $str)) {
+			if (!preg_match("|^[".str_replace(array('\\-', '\-'), '-', preg_quote($this->permitted_uri_chars, '-'))."]+$|i", $str)) {
 				show_error('The URI you submitted has disallowed characters.', 400);
 			}
 		}
 
 		// Convert programatic characters to entities
-		$bad=array ('$','(',')','%28','%29' );
-		$good=array ('$','(',')','(',')' );
+		$bad=array('$', '(', ')', '%28', '%29');
+		$good=array('$', '(', ')', '(', ')');
 
 		return str_replace($bad, $good, $str);
 	}
@@ -243,11 +243,11 @@ 
 	 *
 	 */
 	function _explode_segments() {
-		foreach ( explode("/", preg_replace("|/*(.+?)/*$|", "\\1", $this->uri_string)) as $val ) {
+		foreach (explode("/", preg_replace("|/*(.+?)/*$|", "\\1", $this->uri_string)) as $val) {
 			// Filter segments for security
 			$val=trim($this->_filter_uri($val));
 
-			if ($val != '') {
+			if ($val!='') {
 				$this->segments[]=$val;
 			}
 		}
@@ -364,7 +364,7 @@ 
 	 *
 	 */
 	function _uri_to_assoc($n=3, $default=array(), $which='segment') {
-		if ($which == 'segment') {
+		if ($which=='segment') {
 			$total_segments='total_segments';
 			$segment_array='segment_array';
 		} else {
@@ -380,25 +380,25 @@ 
 			return $this->keyval[$n];
 		}
 
-		if ($this->$total_segments() < $n) {
-			if (count($default) == 0) {
-				return array ();
+		if ($this->$total_segments()<$n) {
+			if (count($default)==0) {
+				return array();
 			}
 
-			$retval=array ();
-			foreach ( $default as $val ) {
+			$retval=array();
+			foreach ($default as $val) {
 				$retval[$val]=FALSE;
 			}
 			return $retval;
 		}
 
-		$segments=array_slice($this->$segment_array(), ($n - 1));
+		$segments=array_slice($this->$segment_array(), ($n-1));
 
 		$i=0;
 		$lastval='';
-		$retval=array ();
-		foreach ( $segments as $seg ) {
-			if ($i % 2) {
+		$retval=array();
+		foreach ($segments as $seg) {
+			if ($i%2) {
 				$retval[$lastval]=$seg;
 			} else {
 				$retval[$seg]=FALSE;
@@ -408,8 +408,8 @@ 
 			$i++;
 		}
 
-		if (count($default) > 0) {
-			foreach ( $default as $val ) {
+		if (count($default)>0) {
+			foreach ($default as $val) {
 				if (!array_key_exists($val, $retval)) {
 					$retval[$val]=FALSE;
 				}
@@ -433,8 +433,8 @@ 
 	 *
 	 */
 	function assoc_to_uri($array) {
-		$temp=array ();
-		foreach ( ( array ) $array as $key => $val ) {
+		$temp=array();
+		foreach ((array)$array as $key => $val) {
 			$temp[]=$key;
 			$temp[]=$val;
 		}
@@ -488,13 +488,13 @@ 
 		$leading='/';
 		$trailing='/';
 
-		if ($where == 'trailing') {
+		if ($where=='trailing') {
 			$leading='';
-		} elseif ($where == 'leading') {
+		} elseif ($where=='leading') {
 			$trailing='';
 		}
 
-		return $leading . $this->$which($n) . $trailing;
+		return $leading.$this->$which($n).$trailing;
 	}
 
 	// --------------------------------------------------------------------
@@ -572,7 +572,7 @@ 
 	 *
 	 */
 	function ruri_string() {
-		return '/' . implode('/', $this->rsegment_array());
+		return '/'.implode('/', $this->rsegment_array());
 	}
 }
 // END URI Class
\ 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/HtmlBreadcrumb.php

Doc Comments +3 added lines, -4 removed lines

@@ -64,8 +64,7 @@ 
 	 * Associate an ajax get to the breadcrumb 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 HtmlBreadcrumbs
+	 * @return HtmlBreadcrumb
 	 */
 	public function autoGetOnClick($targetSelector) {
 		return $this->getOnClick($this->root, $targetSelector, array ("attr" => $this->attr ));
@@ -91,7 +90,7 @@ 
 	 * Add new elements in breadcrumbs corresponding to request dispatcher : controllerName, actionName, parameters
 	 * @param JsUtils $js
 	 * @param Dispatcher $dispatcher the request dispatcher
-	 * @return \Ajax\bootstrap\html\HtmlBreadcrumbs
+	 * @return HtmlBreadcrumb
 	 */
 	public function fromDispatcher(JsUtils $js,$dispatcher, $startIndex=0) {
 		return $this->addItems($js->fromDispatcher($dispatcher));
@@ -120,7 +119,7 @@ 
 	 * sets the function who generates the href elements.
 	 * default : function($element){return $element->getContent()}
 	 * @param function $_hrefFunction
-	 * @return \Ajax\bootstrap\html\HtmlBreadcrumbs
+	 * @return HtmlBreadcrumb
 	 */
 	public function setHrefFunction($_hrefFunction) {
 		$this->_hrefFunction=$_hrefFunction;

Braces +3 added lines, -3 removed lines

@@ -181,9 +181,9 @@
 	protected function createItem($value) {
 		$count=$this->count();
 		$itemO=new HtmlSemDoubleElement("item-" . $this->identifier . "-" . $count, "a", "section");
-		if (\is_array($value))
-			$itemO->fromArray($value);
-		else {
+		if (\is_array($value)) {
+					$itemO->fromArray($value);
+		} else {
 			$itemO->setContent($value);
 		}
 		return $itemO;

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 {

Spacing +16 added lines, -16 removed lines

@@ -51,7 +51,7 @@ 
 		$this->startIndex=$startIndex;
 		$this->autoActive=$autoActive;
 		$this->_contentSeparator="<div class='divider'> / </div>";
-		$this->_hrefFunction=function ($e) {
+		$this->_hrefFunction=function($e) {
 			return $e->getContent();
 		};
 		if (isset($hrefFunction)) {
@@ -68,7 +68,7 @@ 
 	 * @return HtmlBreadcrumbs
 	 */
 	public function autoGetOnClick($targetSelector) {
-		return $this->getOnClick($this->root, $targetSelector, array ("attr" => $this->attr ));
+		return $this->getOnClick($this->root, $targetSelector, array("attr" => $this->attr));
 	}
 
 	public function contentAsString() {
@@ -80,7 +80,7 @@ 
 
 	public function setActive($index=null) {
 		if (!isset($index)) {
-			$index=sizeof($this->content) - 1;
+			$index=sizeof($this->content)-1;
 		}
 		$activeItem=$this->content[$index];
 		$activeItem->addToProperty("class", "active");
@@ -93,7 +93,7 @@ 
 	 * @param Dispatcher $dispatcher the request dispatcher
 	 * @return \Ajax\bootstrap\html\HtmlBreadcrumbs
 	 */
-	public function fromDispatcher(JsUtils $js,$dispatcher, $startIndex=0) {
+	public function fromDispatcher(JsUtils $js, $dispatcher, $startIndex=0) {
 		return $this->addItems($js->fromDispatcher($dispatcher));
 	}
 
@@ -107,12 +107,12 @@ 
 		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->root.implode($separator, array_slice(array_map(function($e) {
 				return $this->_hrefFunction($e);
-			}, $this->content), $this->startIndex, $index + 1));
+			}, $this->content), $this->startIndex, $index+1));
 		}
 	}
 
@@ -146,8 +146,8 @@ 
 			$this->setActive();
 		}
 		$count=$this->count();
-		for($i=1; $i < $count; $i++) {
-			$this->content[$i]->wrap($this->getContentDivider($i - 1));
+		for ($i=1; $i<$count; $i++) {
+			$this->content[$i]->wrap($this->getContentDivider($i-1));
 		}
 		return parent::compile($js, $view);
 	}
@@ -157,15 +157,15 @@ 
 	 * @see \Ajax\bootstrap\html\base\BaseHtml::on()
 	 */
 	public function on($event, $jsCode, $stopPropagation=false, $preventDefault=false) {
-		foreach ( $this->content as $element ) {
+		foreach ($this->content as $element) {
 			$element->on($event, $jsCode, $stopPropagation, $preventDefault);
 		}
 		return $this;
 	}
 
 	public function _ajaxOn($operation, $event, $url, $responseElement="", $parameters=array()) {
-		foreach ( $this->content as $element ) {
-			if ($element->getProperty($this->attr) != NULL){
+		foreach ($this->content as $element) {
+			if ($element->getProperty($this->attr)!=NULL) {
 				$element->_ajaxOn($operation, $event, $url, $responseElement, $parameters);
 			}
 		}
@@ -180,7 +180,7 @@ 
 	 */
 	protected function createItem($value) {
 		$count=$this->count();
-		$itemO=new HtmlSemDoubleElement("item-" . $this->identifier . "-" . $count, "a", "section");
+		$itemO=new HtmlSemDoubleElement("item-".$this->identifier."-".$count, "a", "section");
 		if (\is_array($value))
 			$itemO->fromArray($value);
 		else {
@@ -192,7 +192,7 @@ 
 	public function addIconAt($icon, $index) {
 		$item=$this->getItem($index);
 		if (isset($item)) {
-			$icon=new HtmlIcon("icon-" . $this->identifier, $icon);
+			$icon=new HtmlIcon("icon-".$this->identifier, $icon);
 			$item->wrapContent($icon);
 		}
 	}
@@ -200,7 +200,7 @@ 
 	public function addItem($item) {
 		$count=$this->count();
 		$itemO=parent::addItem($item);
-		$this->addToPropertyCtrl("class", "section", array ("section" ));
+		$this->addToPropertyCtrl("class", "section", array("section"));
 		$itemO->setProperty($this->attr, $this->getHref($count));
 		return $itemO;
 	}
@@ -216,7 +216,7 @@ 
 	public function setAbsolutePaths($absolutePaths) {
 		$this->absolutePaths=$absolutePaths;
 		$size=\sizeof($this->content);
-		for($i=0;$i<$size;$i++){
+		for ($i=0; $i<$size; $i++) {
 			$this->content[$i]->setProperty($this->attr, $this->getHref($i));
 		}
 		return $this;

Ajax/semantic/html/collections/HtmlGrid.php

Doc Comments +5 added lines, -2 removed lines

@@ -22,6 +22,9 @@ 
 	private $_colSizing=true;
 	private $_implicitRows=false;
 
+	/**
+	 * @param string $identifier
+	 */
 	public function __construct($identifier, $numRows=1, $numCols=NULL, $createCols=true, $implicitRows=false) {
 		parent::__construct($identifier, "div", "ui grid");
 		$this->_implicitRows=$implicitRows;
@@ -158,7 +161,7 @@ 
 	/**
 	 * return the row at $index
 	 * @param int $index
-	 * @return \Ajax\semantic\html\collections\HtmlGridRow
+	 * @return \Ajax\common\html\HtmlDoubleElement
 	 */
 	public function getRow($index) {
 		return $this->getItem($index);
@@ -297,7 +300,7 @@ 
 
 	/**
 	 * stretch the row contents to take up the entire column height
-	 * @return \Ajax\semantic\html\content\HtmlGridRow
+	 * @return HtmlGrid
 	 */
 	public function setStretched() {
 		return $this->addToProperty("class", "stretched");

Spacing +39 added lines, -39 removed lines

@@ -36,11 +36,11 @@ 
 	}
 
 	public function asSegment() {
-		return $this->addToPropertyCtrl("class", "segment", array ("segment" ));
+		return $this->addToPropertyCtrl("class", "segment", array("segment"));
 	}
 
 	public function asContainer() {
-		return $this->addToPropertyCtrl("class", "container", array ("container" ));
+		return $this->addToPropertyCtrl("class", "container", array("container"));
 	}
 
 	/**
@@ -48,9 +48,9 @@ 
 	 * @param int $wide
 	 */
 	public function setWide($wide) {
-		$wide=Wide::getConstants()["W" . $wide];
+		$wide=Wide::getConstants()["W".$wide];
 		$this->addToPropertyCtrl("class", $wide, Wide::getConstants());
-		return $this->addToPropertyCtrl("class", "column", array ("column" ));
+		return $this->addToPropertyCtrl("class", "column", array("column"));
 	}
 
 	/**
@@ -68,9 +68,9 @@ 
 	 * @return mixed
 	 */
 	public function addRow($colsCount=NULL) {
-		$rowCount=$this->rowCount() + 1;
+		$rowCount=$this->rowCount()+1;
 		$this->setRowsCount($rowCount, $colsCount, true);
-		return $this->content[$rowCount - 1];
+		return $this->content[$rowCount-1];
 	}
 
 	/**
@@ -79,10 +79,10 @@ 
 	 * @return mixed|\Ajax\semantic\html\collections\HtmlGrid
 	 */
 	public function addCol($width=NULL) {
-		$colCount=$this->colCount() + 1;
+		$colCount=$this->colCount()+1;
 		$this->setColsCount($colCount, true, $width);
 		if ($this->hasOnlyCols($this->count()))
-			return $this->content[$colCount - 1];
+			return $this->content[$colCount-1];
 		return $this;
 	}
 
@@ -92,7 +92,7 @@ 
 	 * @return \Ajax\semantic\html\collections\HtmlGrid
 	 */
 	public function addCols($sizes=array()) {
-		foreach ( $sizes as $size ) {
+		foreach ($sizes as $size) {
 			$this->addCol($size);
 		}
 		return $this;
@@ -106,19 +106,19 @@ 
 	 */
 	public function setRowsCount($rowsCount, $colsCount=NULL, $force=false) {
 		$count=$this->count();
-		if ($rowsCount < 2 && $force === false) {
-			for($i=$count; $i < $colsCount; $i++) {
-				$this->addItem(new HtmlGridCol("col-" . $this->identifier . "-" . $i));
+		if ($rowsCount<2 && $force===false) {
+			for ($i=$count; $i<$colsCount; $i++) {
+				$this->addItem(new HtmlGridCol("col-".$this->identifier."-".$i));
 			}
 		} else {
 			if ($this->hasOnlyCols($count)) {
 				$tmpContent=$this->content;
 				$item=$this->addItem($colsCount);
 				$item->setContent($tmpContent);
-				$this->content=array ();
+				$this->content=array();
 				$count=1;
 			}
-			for($i=$count; $i < $rowsCount; $i++) {
+			for ($i=$count; $i<$rowsCount; $i++) {
 				$this->addItem($colsCount);
 			}
 		}
@@ -126,7 +126,7 @@ 
 	}
 
 	protected function hasOnlyCols($count) {
-		return $count > 0 && $this->content[0] instanceof HtmlGridCol;
+		return $count>0 && $this->content[0] instanceof HtmlGridCol;
 	}
 
 	/**
@@ -140,14 +140,14 @@ 
 		/*if (isset($width)==false) {
 			$this->setWide($numCols);
 		}*/
-		if ($toCreate == true) {
+		if ($toCreate==true) {
 			$count=$this->count();
-			if ($count == 0 || $this->hasOnlyCols($count)) {
-				for($i=$count; $i < $numCols; $i++) {
-					$this->addItem(new HtmlGridCol("col-" . $this->identifier . "-" . $i, $width));
+			if ($count==0 || $this->hasOnlyCols($count)) {
+				for ($i=$count; $i<$numCols; $i++) {
+					$this->addItem(new HtmlGridCol("col-".$this->identifier."-".$i, $width));
 				}
 			} else {
-				for($i=0; $i < $count; $i++) {
+				for ($i=0; $i<$count; $i++) {
 					$this->getItem($i)->setColsCount($numCols);
 				}
 			}
@@ -183,7 +183,7 @@ 
 		$count=$this->count();
 		if ($this->hasOnlyCols($count))
 			return $count;
-		if ($count > 0)
+		if ($count>0)
 			return $this->getItem(0)->count();
 		return 0;
 	}
@@ -195,7 +195,7 @@ 
 	 * @return \Ajax\semantic\html\collections\HtmlGridCol
 	 */
 	public function getCell($row, $col) {
-		if ($row < 2 && $this->hasOnlyCols($this->count()))
+		if ($row<2 && $this->hasOnlyCols($this->count()))
 			return $this->getItem($col);
 		$row=$this->getItem($row);
 		if (isset($row)) {
@@ -210,8 +210,8 @@ 
 	 * @return \Ajax\semantic\html\collections\HtmlGrid
 	 */
 	public function setDivided($vertically=false) {
-		$value=($vertically === true) ? "vertically divided" : "divided";
-		return $this->addToPropertyCtrl("class", $value, array ("divided" ));
+		$value=($vertically===true) ? "vertically divided" : "divided";
+		return $this->addToPropertyCtrl("class", $value, array("divided"));
 	}
 
 	/**
@@ -220,15 +220,15 @@ 
 	 * @return \Ajax\semantic\html\collections\HtmlGrid
 	 */
 	public function setCelled($internally=false) {
-		$value=($internally === true) ? "internally celled" : "celled";
-		return $this->addToPropertyCtrl("class", $value, array ("celled","internally celled" ));
+		$value=($internally===true) ? "internally celled" : "celled";
+		return $this->addToPropertyCtrl("class", $value, array("celled", "internally celled"));
 	}
 
 	/**
 	 * A grid can have its columns centered
 	 */
 	public function setCentered() {
-		return $this->addToPropertyCtrl("class", "centered", array ("centered" ));
+		return $this->addToPropertyCtrl("class", "centered", array("centered"));
 	}
 
 	/**
@@ -246,7 +246,7 @@ 
 	 */
 	public function setPadded($value=NULL) {
 		if (isset($value))
-			$this->addToPropertyCtrl("class", $value, array ("vertically","horizontally" ));
+			$this->addToPropertyCtrl("class", $value, array("vertically", "horizontally"));
 		return $this->addToProperty("class", "padded");
 	}
 
@@ -256,12 +256,12 @@ 
 	 * @return \Ajax\semantic\html\collections\HtmlGrid
 	 */
 	public function setRelaxed($very=false) {
-		$value=($very === true) ? "very relaxed" : "relaxed";
-		return $this->addToPropertyCtrl("class", $value, array ("relaxed","very relaxed" ));
+		$value=($very===true) ? "very relaxed" : "relaxed";
+		return $this->addToPropertyCtrl("class", $value, array("relaxed", "very relaxed"));
 	}
 
 	public function setVerticalAlignment($value=VerticalAlignment::MIDDLE) {
-		return $this->addToPropertyCtrl("class", $value . " aligned", VerticalAlignment::getConstantValues("aligned"));
+		return $this->addToPropertyCtrl("class", $value." aligned", VerticalAlignment::getConstantValues("aligned"));
 	}
 
 	/**
@@ -271,9 +271,9 @@ 
 	 * @see \Ajax\common\html\HtmlCollection::createItem()
 	 */
 	protected function createItem($value) {
-		if ($this->_createCols === false)
+		if ($this->_createCols===false)
 			$value=null;
-		$item=new HtmlGridRow($this->identifier . "-row-" . ($this->count() + 1), $value, $this->_colSizing, $this->_implicitRows);
+		$item=new HtmlGridRow($this->identifier."-row-".($this->count()+1), $value, $this->_colSizing, $this->_implicitRows);
 		return $item;
 	}
 
@@ -283,15 +283,15 @@ 
 	 */
 	public function setValues($values, $force=true) {
 		$count=$this->count();
-		if ($this->_createCols === false || $force === true) {
-			for($i=$count; $i < \sizeof($values); $i++) {
+		if ($this->_createCols===false || $force===true) {
+			for ($i=$count; $i<\sizeof($values); $i++) {
 				$colSize=\sizeof($values[$i]);
-				$this->addItem(new HtmlGridRow($this->identifier . "-row-" . ($this->count() + 1), $colSize, $this->_colSizing, $this->_implicitRows));
+				$this->addItem(new HtmlGridRow($this->identifier."-row-".($this->count()+1), $colSize, $this->_colSizing, $this->_implicitRows));
 			}
 		}
-		$count=\min(array ($this->count(),\sizeof($values) ));
-		for($i=0; $i < $count; $i++) {
-			$this->content[$i]->setValues($values[$i], $this->_createCols === false);
+		$count=\min(array($this->count(), \sizeof($values)));
+		for ($i=0; $i<$count; $i++) {
+			$this->content[$i]->setValues($values[$i], $this->_createCols===false);
 		}
 	}
 

Braces +21 added lines, -14 removed lines

@@ -81,8 +81,9 @@ 
 	public function addCol($width=NULL) {
 		$colCount=$this->colCount() + 1;
 		$this->setColsCount($colCount, true, $width);
-		if ($this->hasOnlyCols($this->count()))
-			return $this->content[$colCount - 1];
+		if ($this->hasOnlyCols($this->count())) {
+					return $this->content[$colCount - 1];
+		}
 		return $this;
 	}
 
@@ -170,8 +171,9 @@ 
 	 */
 	public function rowCount() {
 		$count=$this->count();
-		if ($this->hasOnlyCols($count))
-			return 0;
+		if ($this->hasOnlyCols($count)) {
+					return 0;
+		}
 		return $count;
 	}
 
@@ -181,10 +183,12 @@ 
 	 */
 	public function colCount() {
 		$count=$this->count();
-		if ($this->hasOnlyCols($count))
-			return $count;
-		if ($count > 0)
-			return $this->getItem(0)->count();
+		if ($this->hasOnlyCols($count)) {
+					return $count;
+		}
+		if ($count > 0) {
+					return $this->getItem(0)->count();
+		}
 		return 0;
 	}
 
@@ -195,8 +199,9 @@ 
 	 * @return \Ajax\semantic\html\collections\HtmlGridCol
 	 */
 	public function getCell($row, $col) {
-		if ($row < 2 && $this->hasOnlyCols($this->count()))
-			return $this->getItem($col);
+		if ($row < 2 && $this->hasOnlyCols($this->count())) {
+					return $this->getItem($col);
+		}
 		$row=$this->getItem($row);
 		if (isset($row)) {
 			$col=$row->getItem($col);
@@ -245,8 +250,9 @@ 
 	 * @return \Ajax\semantic\html\collections\HtmlGrid
 	 */
 	public function setPadded($value=NULL) {
-		if (isset($value))
-			$this->addToPropertyCtrl("class", $value, array ("vertically","horizontally" ));
+		if (isset($value)) {
+					$this->addToPropertyCtrl("class", $value, array ("vertically","horizontally" ));
+		}
 		return $this->addToProperty("class", "padded");
 	}
 
@@ -271,8 +277,9 @@ 
 	 * @see \Ajax\common\html\HtmlCollection::createItem()
 	 */
 	protected function createItem($value) {
-		if ($this->_createCols === false)
-			$value=null;
+		if ($this->_createCols === false) {
+					$value=null;
+		}
 		$item=new HtmlGridRow($this->identifier . "-row-" . ($this->count() + 1), $value, $this->_colSizing, $this->_implicitRows);
 		return $item;
 	}

Ajax/semantic/html/collections/HtmlTable.php

Doc Comments +9 added lines, -1 removed lines

@@ -17,6 +17,11 @@ 
 class HtmlTable extends HtmlSemDoubleElement {
 	private $_colCount;
 
+	/**
+	 * @param string $identifier
+	 * @param integer $rowCount
+	 * @param integer $colCount
+	 */
 	public function __construct($identifier, $rowCount, $colCount) {
 		parent::__construct($identifier, "table", "ui table");
 		$this->content=array ();
@@ -96,7 +101,7 @@ 
 	/**
 	 * Retuns the row at $rowIndex
 	 * @param int $rowIndex
-	 * @return \Ajax\semantic\html\content\HtmlTR
+	 * @return \Ajax\common\html\HtmlDoubleElement
 	 */
 	public function getRow($rowIndex) {
 		return $this->getBody()->getRow($rowIndex);
@@ -172,6 +177,9 @@ 
 		return $this->colAlign($colIndex, "colLeft");
 	}
 
+	/**
+	 * @param string $function
+	 */
 	private function colAlign($colIndex, $function) {
 		if (\is_array($colIndex)) {
 			foreach ( $colIndex as $cIndex ) {

Spacing +10 added lines, -10 removed lines

@@ -19,9 +19,9 @@ 
 
 	public function __construct($identifier, $rowCount, $colCount) {
 		parent::__construct($identifier, "table", "ui table");
-		$this->content=array ();
+		$this->content=array();
 		$this->setRowCount($rowCount, $colCount);
-		$this->_variations=[ Variation::CELLED,Variation::PADDED,Variation::COMPACT ];
+		$this->_variations=[Variation::CELLED, Variation::PADDED, Variation::COMPACT];
 	}
 
 	/**
@@ -30,9 +30,9 @@ 
 	 * @return HtmlTableContent
 	 */
 	private function getPart($key) {
-		if (\array_key_exists($key, $this->content) === false) {
+		if (\array_key_exists($key, $this->content)===false) {
 			$this->content[$key]=new HtmlTableContent("", $key);
-			if ($key !== "tbody") {
+			if ($key!=="tbody") {
 				$this->content[$key]->setRowCount(1, $this->_colCount);
 			}
 		}
@@ -69,7 +69,7 @@ 
 	 * @return boolean
 	 */
 	public function hasPart($key) {
-		return \array_key_exists($key, $this->content) === true;
+		return \array_key_exists($key, $this->content)===true;
 	}
 
 	/**
@@ -174,7 +174,7 @@ 
 
 	private function colAlign($colIndex, $function) {
 		if (\is_array($colIndex)) {
-			foreach ( $colIndex as $cIndex ) {
+			foreach ($colIndex as $cIndex) {
 				$this->colAlign($cIndex, $function);
 			}
 		} else {
@@ -192,8 +192,8 @@ 
 
 	public function setBasic($very=false) {
 		if ($very)
-			$this->addToPropertyCtrl("class", "very", array ("very" ));
-		return $this->addToPropertyCtrl("class", "basic", array ("basic" ));
+			$this->addToPropertyCtrl("class", "very", array("very"));
+		return $this->addToPropertyCtrl("class", "basic", array("basic"));
 	}
 
 	public function setCollapsing() {
@@ -254,9 +254,9 @@ 
 	 * @see \Ajax\semantic\html\base\HtmlSemDoubleElement::compile()
 	 */
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
-		$this->content=JArray::sortAssociative($this->content, [ "thead","tbody","tfoot" ]);
+		$this->content=JArray::sortAssociative($this->content, ["thead", "tbody", "tfoot"]);
 		if ($this->propertyContains("class", "sortable")) {
-			$this->addEvent("execute", "$('#" . $this->identifier . "').tablesort();");
+			$this->addEvent("execute", "$('#".$this->identifier."').tablesort();");
 		}
 		return parent::compile($js, $view);
 	}

Braces +3 added lines, -2 removed lines

@@ -191,8 +191,9 @@
 	}
 
 	public function setBasic($very=false) {
-		if ($very)
-			$this->addToPropertyCtrl("class", "very", array ("very" ));
+		if ($very) {
+					$this->addToPropertyCtrl("class", "very", array ("very" ));
+		}
 		return $this->addToPropertyCtrl("class", "basic", array ("basic" ));
 	}
 

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 {

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

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

Braces +11 added lines, -9 removed lines

@@ -57,9 +57,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;
@@ -158,8 +158,9 @@ 
 		$this->apply(function (HtmlDoubleElement &$item) {
 			$item->setTagName("a");
 		});
-		if ($vertical === true)
-			$this->setVertical();
+		if ($vertical === true) {
+					$this->setVertical();
+		}
 		return $this->addToProperty("class", "tabular");
 	}
 
@@ -188,10 +189,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);
+		}
 	}
 
 	/**

Spacing +19 added lines, -19 removed lines

@@ -37,12 +37,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;
@@ -50,7 +50,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;
 		}
@@ -59,7 +59,7 @@ 
 
 	private function afterInsert($item) {
 		if (!$item instanceof HtmlMenu)
-			$item->addToPropertyCtrl("class", "item", array ("item" ));
+			$item->addToPropertyCtrl("class", "item", array("item"));
 		else {
 			$this->setSecondary();
 		}
@@ -90,15 +90,15 @@ 
 
 	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;
 		}
-		if(\is_array($menu)){
-			$menu=new HtmlMenu("menu-" . $this->identifier . "-" . $count,$menu);
+		if (\is_array($menu)) {
+			$menu=new HtmlMenu("menu-".$this->identifier."-".$count, $menu);
 		}
 		$menu->setClass("menu");
 		$item->addContent($menu);
@@ -113,7 +113,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;
@@ -122,7 +122,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);
 	}
@@ -134,7 +134,7 @@ 
 	 * @see \Ajax\common\html\html5\HtmlCollection::createItem()
 	 */
 	protected function createItem($value) {
-		$itemO=new HtmlLink($this->identifier."item" . \sizeof($this->content), "", $value);
+		$itemO=new HtmlLink($this->identifier."item".\sizeof($this->content), "", $value);
 		return $itemO->setClass("item");
 	}
 
@@ -147,28 +147,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");
@@ -205,10 +205,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/HtmlAbsractItem.php

Doc Comments +3 added lines

@@ -11,6 +11,9 @@
 
 abstract class HtmlAbsractItem extends HtmlSemDoubleElement {
 
+	/**
+	 * @param string $baseClass
+	 */
 	public function __construct($identifier, $baseClass,$content=NULL) {
 		parent::__construct($identifier, "div", $baseClass);
 		$this->content=array();

Braces +6 added lines, -4 removed lines

@@ -60,8 +60,9 @@ 
 
 	public function asLink($href=NULL,$part=NULL){
 		$this->setTagName("a");
-		if(isset($href))
-			$this->setProperty("href", $href);
+		if(isset($href)) {
+					$this->setProperty("href", $href);
+		}
 		return $this;
 	}
 
@@ -72,8 +73,9 @@ 
 	 * @see \Ajax\semantic\html\base\HtmlSemDoubleElement::compile()
 	 */
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
-		if(\is_array($this->content))
-			$this->content=JArray::sortAssociative($this->content, [ "icon","image","content" ]);
+		if(\is_array($this->content)) {
+					$this->content=JArray::sortAssociative($this->content, [ "icon","image","content" ]);
+		}
 		return parent::compile($js, $view);
 	}
 }
\ No newline at end of file

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 {

Spacing +18 added lines, -18 removed lines

@@ -11,7 +11,7 @@ 
 
 abstract class HtmlAbsractItem extends HtmlSemDoubleElement {
 
-	public function __construct($identifier, $baseClass,$content=NULL) {
+	public function __construct($identifier, $baseClass, $content=NULL) {
 		parent::__construct($identifier, "div", $baseClass);
 		$this->content=array();
 		$this->initContent($content);
@@ -19,41 +19,41 @@ 
 
 	protected abstract function initContent($content);
 
-	public function setIcon($icon){
+	public function setIcon($icon) {
 		$this->content["icon"]=new HtmlIcon("icon-".$this->identifier, $icon);
 	}
 
-	public function setImage($image){
+	public function setImage($image) {
 		$image=new HtmlImg("icon-".$this->identifier, $image);
 		$image->asAvatar();
 		$this->content["image"]=$image;
 	}
 
-	private function createContent(){
-		$this->content["content"]=new HtmlSemDoubleElement("content-".$this->identifier,"div","content");
+	private function createContent() {
+		$this->content["content"]=new HtmlSemDoubleElement("content-".$this->identifier, "div", "content");
 		return $this->content["content"];
 	}
 
-	public function setTitle($title,$description=NULL,$baseClass="title"){
-		$title=new HtmlSemDoubleElement("","div",$baseClass,$title);
-		if(\array_key_exists("content", $this->content)===false){
+	public function setTitle($title, $description=NULL, $baseClass="title") {
+		$title=new HtmlSemDoubleElement("", "div", $baseClass, $title);
+		if (\array_key_exists("content", $this->content)===false) {
 			$this->createContent();
 		}
 		$this->content["content"]->addContent($title);
-		if(isset($description)){
-			$description=new HtmlSemDoubleElement("","div","description",$description);
+		if (isset($description)) {
+			$description=new HtmlSemDoubleElement("", "div", "description", $description);
 			$this->content["content"]->addContent($description);
 		}
 		return $this;
 	}
 
-	public function getPart($partName="header"){
-		$content=\array_merge($this->content["content"]->getContent(),array(@$this->content["icon"],@$this->content["image"]));
+	public function getPart($partName="header") {
+		$content=\array_merge($this->content["content"]->getContent(), array(@$this->content["icon"], @$this->content["image"]));
 		return $this->getElementByPropertyValue("class", $partName, $content);
 	}
 
-	public function setActive($value=true){
-		if($value){
+	public function setActive($value=true) {
+		if ($value) {
 			$this->setTagName("div");
 			$this->removeProperty("href");
 			$this->addToPropertyCtrl("class", "active", array("active"));
@@ -61,9 +61,9 @@ 
 		return $this;
 	}
 
-	public function asLink($href=NULL,$part=NULL){
+	public function asLink($href=NULL, $part=NULL) {
 		$this->setTagName("a");
-		if(isset($href))
+		if (isset($href))
 			$this->setProperty("href", $href);
 		return $this;
 	}
@@ -75,8 +75,8 @@ 
 	 * @see \Ajax\semantic\html\base\HtmlSemDoubleElement::compile()
 	 */
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
-		if(\is_array($this->content))
-			$this->content=JArray::sortAssociative($this->content, [ "icon","image","content" ]);
+		if (\is_array($this->content))
+			$this->content=JArray::sortAssociative($this->content, ["icon", "image", "content"]);
 		return parent::compile($js, $view);
 	}
 }
\ No newline at end of file

Ajax/semantic/html/content/HtmlDropdownItem.php

Doc Comments +7 added lines

@@ -11,6 +11,10 @@ 
 
 class HtmlDropdownItem extends HtmlSemDoubleElement {
 	use IconTrait;
+
+	/**
+	 * @param string $identifier
+	 */
 	public function __construct($identifier, $content="",$value=NULL,$image=NULL) {
 		parent::__construct($identifier, "a");
 		$this->setClass("item");
@@ -61,6 +65,9 @@ 
 		return $this;
 	}
 
+	/**
+	 * @param string $content
+	 */
 	public function setContent($content){
 		if($content==="-"){
 			$this->asDivider();

Spacing +30 added lines, -30 removed lines

@@ -11,30 +11,30 @@ 
 
 class HtmlDropdownItem extends HtmlSemDoubleElement {
 	use IconTrait;
-	public function __construct($identifier, $content="",$value=NULL,$image=NULL) {
+	public function __construct($identifier, $content="", $value=NULL, $image=NULL) {
 		parent::__construct($identifier, "a");
 		$this->setClass("item");
 		$this->setContent($content);
-		if($value!==NULL)
+		if ($value!==NULL)
 			$this->setData($value);
-		if($image!==NULL)
+		if ($image!==NULL)
 			$this->asMiniAvatar($image);
 	}
 
-	public function setDescription($description){
-		$descO=new HtmlDoubleElement("desc-".$this->identifier,"span");
+	public function setDescription($description) {
+		$descO=new HtmlDoubleElement("desc-".$this->identifier, "span");
 		$descO->setClass("description");
 		$descO->setContent($description);
-		return $this->addContent($descO,true);
+		return $this->addContent($descO, true);
 	}
 
-	public function setData($value){
+	public function setData($value) {
 		$this->setProperty("data-value", $value);
 	}
 
-	public function asOption(){
+	public function asOption() {
 		$this->tagName="option";
-		if($this->getProperty("data-value")!==null)
+		if ($this->getProperty("data-value")!==null)
 			$this->setProperty("value", $this->getProperty("data-value"));
 	}
 
@@ -42,58 +42,58 @@ 
 	 * @param string $image the image src
 	 * @return \Ajax\common\html\html5\HtmlImg
 	 */
-	public function asMiniAvatar($image){
+	public function asMiniAvatar($image) {
 		$this->tagName="div";
-		$img=new HtmlImg("image-".$this->identifier,$image);
+		$img=new HtmlImg("image-".$this->identifier, $image);
 		$img->setClass("ui mini avatar image");
-		$this->addContent($img,true);
+		$this->addContent($img, true);
 		return $img;
 	}
 
-	public function asSearchInput($placeholder=NULL,$icon=NULL){
+	public function asSearchInput($placeholder=NULL, $icon=NULL) {
 		$this->setClass("ui icon search input");
 		$input=new HtmlInput("search-".$this->identifier);
-		if(isset($placeholder))
+		if (isset($placeholder))
 			$input->setProperty("placeholder", $placeholder);
 		$this->content=$input;
-		if(isset($icon))
+		if (isset($icon))
 			$this->addIcon($icon);
 		return $this;
 	}
 
-	public function setContent($content){
-		if($content==="-"){
+	public function setContent($content) {
+		if ($content==="-") {
 			$this->asDivider();
-		}elseif($content==="-search-"){
-			$values=\explode(",",$content,-1);
-			$this->asSearchInput(JArray::getDefaultValue($values, 0, "Search..."),JArray::getDefaultValue($values, 1, "search"));
-		}else
+		}elseif ($content==="-search-") {
+			$values=\explode(",", $content, -1);
+			$this->asSearchInput(JArray::getDefaultValue($values, 0, "Search..."), JArray::getDefaultValue($values, 1, "search"));
+		} else
 			parent::setContent($content);
 		return $this;
 	}
 
-	public function asDivider(){
+	public function asDivider() {
 		$this->content=NULL;
 		$this->setClass("divider");
 	}
 
-	public function asHeader($caption=NULL,$icon=NULL){
+	public function asHeader($caption=NULL, $icon=NULL) {
 		$this->setClass("header");
 		$this->content=$caption;
-		if(isset($icon))
-			$this->addIcon($icon,true);
+		if (isset($icon))
+			$this->addIcon($icon, true);
 		return $this;
 	}
 
-	public static function searchInput($placeholder=NULL,$icon=NULL){
-		return (new HtmlDropdownItem(""))->asSearchInput($placeholder,$icon);
+	public static function searchInput($placeholder=NULL, $icon=NULL) {
+		return (new HtmlDropdownItem(""))->asSearchInput($placeholder, $icon);
 	}
 
-	public static function divider($placeholder=NULL,$icon=NULL){
+	public static function divider($placeholder=NULL, $icon=NULL) {
 		return (new HtmlDropdownItem(""))->asDivider();
 	}
 
-	public static function header($caption=NULL,$icon=NULL){
-		return (new HtmlDropdownItem(""))->asHeader($caption,$icon);
+	public static function header($caption=NULL, $icon=NULL) {
+		return (new HtmlDropdownItem(""))->asHeader($caption, $icon);
 	}
 }
\ No newline at end of file

Braces +22 added lines, -15 removed lines

@@ -15,10 +15,12 @@ 
 		parent::__construct($identifier, "a");
 		$this->setClass("item");
 		$this->setContent($content);
-		if($value!==NULL)
-			$this->setData($value);
-		if($image!==NULL)
-			$this->asMiniAvatar($image);
+		if($value!==NULL) {
+					$this->setData($value);
+		}
+		if($image!==NULL) {
+					$this->asMiniAvatar($image);
+		}
 	}
 
 	public function setDescription($description){
@@ -34,8 +36,9 @@ 
 
 	public function asOption(){
 		$this->tagName="option";
-		if($this->getProperty("data-value")!==null)
-			$this->setProperty("value", $this->getProperty("data-value"));
+		if($this->getProperty("data-value")!==null) {
+					$this->setProperty("value", $this->getProperty("data-value"));
+		}
 	}
 
 	/**
@@ -53,22 +56,25 @@ 
 	public function asSearchInput($placeholder=NULL,$icon=NULL){
 		$this->setClass("ui icon search input");
 		$input=new HtmlInput("search-".$this->identifier);
-		if(isset($placeholder))
-			$input->setProperty("placeholder", $placeholder);
+		if(isset($placeholder)) {
+					$input->setProperty("placeholder", $placeholder);
+		}
 		$this->content=$input;
-		if(isset($icon))
-			$this->addIcon($icon);
+		if(isset($icon)) {
+					$this->addIcon($icon);
+		}
 		return $this;
 	}
 
 	public function setContent($content){
 		if($content==="-"){
 			$this->asDivider();
-		}elseif($content==="-search-"){
+		} elseif($content==="-search-"){
 			$values=\explode(",",$content,-1);
 			$this->asSearchInput(JArray::getDefaultValue($values, 0, "Search..."),JArray::getDefaultValue($values, 1, "search"));
-		}else
-			parent::setContent($content);
+		} else {
+					parent::setContent($content);
+		}
 		return $this;
 	}
 
@@ -80,8 +86,9 @@ 
 	public function asHeader($caption=NULL,$icon=NULL){
 		$this->setClass("header");
 		$this->content=$caption;
-		if(isset($icon))
-			$this->addIcon($icon,true);
+		if(isset($icon)) {
+					$this->addIcon($icon,true);
+		}
 		return $this;
 	}