phpMv / phpMv-UI

Ajax/common/html/traits/BaseHtmlPropertiesTrait.php

Braces +17 added lines, -12 removed lines

@@ -34,8 +34,9 @@ 
 	}
 
 	public function getProperty($name) {
-		if (array_key_exists($name, $this->_self->properties))
-			return $this->_self->properties[$name];
+		if (array_key_exists($name, $this->_self->properties)) {
+					return $this->_self->properties[$name];
+		}
 	}
 
 	public function addToProperty($name, $value, $separator=" ") {
@@ -45,10 +46,11 @@ 
 			}
 		} else if ($value !== "" && $this->_self->propertyContains($name, $value) === false) {
 			$v=@$this->_self->properties[$name];
-			if (isset($v) && $v !== "")
-				$v=$v . $separator . $value;
-				else
-					$v=$value;
+			if (isset($v) && $v !== "") {
+							$v=$v . $separator . $value;
+			} else {
+									$v=$value;
+				}
 
 					return $this->_self->setProperty($name, $v);
 		}
@@ -71,8 +73,9 @@ 
 	}
 
 	protected function addToPropertyUnique($name, $value, $typeCtrl) {
-		if (@class_exists($typeCtrl, true))
-			$typeCtrl=$typeCtrl::getConstants();
+		if (@class_exists($typeCtrl, true)) {
+					$typeCtrl=$typeCtrl::getConstants();
+		}
 			if (\is_array($typeCtrl)) {
 				$this->_self->removeOldValues($this->_self->properties[$name], $typeCtrl);
 			}
@@ -91,8 +94,9 @@ 
 	}
 
 	public function removeProperty($name) {
-		if (\array_key_exists($name, $this->_self->properties))
-			unset($this->_self->properties[$name]);
+		if (\array_key_exists($name, $this->_self->properties)) {
+					unset($this->_self->properties[$name]);
+		}
 			return $this;
 	}
 
@@ -105,8 +109,9 @@ 
 	}
 
 	protected function setPropertyCtrl($name, $value, $typeCtrl) {
-		if ($this->_self->ctrl($name, $value, $typeCtrl) === true)
-			return $this->_self->setProperty($name, $value);
+		if ($this->_self->ctrl($name, $value, $typeCtrl) === true) {
+					return $this->_self->setProperty($name, $value);
+		}
 			return $this;
 	}
 

Spacing +11 added lines, -11 removed lines

@@ -9,12 +9,12 @@ 
  * @author jc
  * @property BaseWidget $_self
  */
-trait BaseHtmlPropertiesTrait{
+trait BaseHtmlPropertiesTrait {
 
-	protected $properties=array ();
+	protected $properties=array();
 	abstract protected function ctrl($name, $value, $typeCtrl);
 	abstract protected function removeOldValues(&$oldValue, $allValues);
-	abstract protected function _getElementBy($callback,$elements);
+	abstract protected function _getElementBy($callback, $elements);
 	public function getProperties() {
 		return $this->_self->properties;
 	}
@@ -40,13 +40,13 @@ 
 
 	public function addToProperty($name, $value, $separator=" ") {
 		if (\is_array($value)) {
-			foreach ( $value as $v ) {
+			foreach ($value as $v) {
 				$this->_self->addToProperty($name, $v, $separator);
 			}
-		} else if ($value !== "" && $this->_self->propertyContains($name, $value) === false) {
+		} else if ($value!=="" && $this->_self->propertyContains($name, $value)===false) {
 			$v=@$this->_self->properties[$name];
-			if (isset($v) && $v !== "")
-				$v=$v . $separator . $value;
+			if (isset($v) && $v!=="")
+				$v=$v.$separator.$value;
 				else
 					$v=$value;
 
@@ -84,7 +84,7 @@ 
 	}
 
 	public function addToPropertyCtrlCheck($name, $value, $typeCtrl) {
-		if ($this->_self->ctrl($name, $value, $typeCtrl) === true) {
+		if ($this->_self->ctrl($name, $value, $typeCtrl)===true) {
 			return $this->_self->addToProperty($name, $value);
 		}
 		return $this;
@@ -105,12 +105,12 @@ 
 	}
 
 	protected function setPropertyCtrl($name, $value, $typeCtrl) {
-		if ($this->_self->ctrl($name, $value, $typeCtrl) === true)
+		if ($this->_self->ctrl($name, $value, $typeCtrl)===true)
 			return $this->_self->setProperty($name, $value);
 			return $this;
 	}
 
-	protected function getElementByPropertyValue($propertyName,$value, $elements) {
-		return $this->_self->_getElementBy(function(BaseHtml $element) use ($propertyName,$value){return $element->propertyContains($propertyName, $value) === true;}, $elements);
+	protected function getElementByPropertyValue($propertyName, $value, $elements) {
+		return $this->_self->_getElementBy(function(BaseHtml $element) use ($propertyName, $value){return $element->propertyContains($propertyName, $value)===true; }, $elements);
 	}
 }

Ajax/semantic/widgets/dataform/DataForm.php

Doc Comments +11 added lines

@@ -72,6 +72,9 @@ 
 		}
 	}
 
+	/**
+	 * @param HtmlForm $form
+	 */
 	protected function _generateFields($form,$values,$headers,$sepFirst,$wrappers){
 		$wrapper=null;
 		if(isset($headers[$sepFirst+1]))
@@ -136,12 +139,20 @@ 
 		$this->content[$this->_toolbarPosition]=$this->_toolbar;
 	}
 
+	/**
+	 * @param integer $index
+	 * @param string $title
+	 */
 	public function addDividerBefore($index,$title){
 		$index=$this->_getIndex($index);
 		$this->_instanceViewer->addHeaderDividerBefore($index, $title);
 		return $this;
 	}
 
+	/**
+	 * @param string $index
+	 * @param string $contentAfter
+	 */
 	public function addWrapper($index,$contentBefore,$contentAfter=null){
 		$index=$this->_getIndex($index);
 		$this->_instanceViewer->addWrapper($index, $contentBefore,$contentAfter);

Braces +8 added lines, -6 removed lines

@@ -63,7 +63,7 @@ 
 				$this->_generateFields($form, [$v], $headers, $i, $wrappers);
 				$i++;
 			}
-		}else{
+		} else{
 			$separators[]=$count;
 			for($i=0;$i<$size;$i++){
 				$fields=\array_slice($values, $separators[$i]+1,$separators[$i+1]-$separators[$i]);
@@ -74,17 +74,19 @@ 
 
 	protected function _generateFields($form,$values,$headers,$sepFirst,$wrappers){
 		$wrapper=null;
-		if(isset($headers[$sepFirst+1]))
-			$form->addHeader($headers[$sepFirst+1],4,true);
+		if(isset($headers[$sepFirst+1])) {
+					$form->addHeader($headers[$sepFirst+1],4,true);
+		}
 		if(isset($wrappers[$sepFirst+1])){
 			$wrapper=$wrappers[$sepFirst+1];
 		}
 		if(\sizeof($values)===1){
 			$added=$form->addField($values[0]);
-		}elseif(\sizeof($values)>1){
+		} elseif(\sizeof($values)>1){
 			$added=$form->addFields($values);
-		}else
-			return;
+		} else {
+					return;
+		}
 		if(isset($wrapper)){
 			$added->wrap($wrapper[0],$wrapper[1]);
 		}

Spacing +33 added lines, -33 removed lines

@@ -21,83 +21,83 @@ 
 	use BaseTrait;
 
 	public function __construct($identifier, $modelInstance=NULL) {
-		parent::__construct($identifier, null,$modelInstance);
+		parent::__construct($identifier, null, $modelInstance);
 		$this->_form=new HtmlForm($identifier);
 		$this->_init(new FormInstanceViewer($identifier), "form", $this->_form, true);
 	}
 
-	protected function _getFieldIdentifier($prefix,$name=""){
+	protected function _getFieldIdentifier($prefix, $name="") {
 		return $this->identifier."-{$name}-".$this->_instanceViewer->getIdentifier();
 	}
 
-	public function compile(JsUtils $js=NULL,&$view=NULL){
-		if(!$this->_generated){
+	public function compile(JsUtils $js=NULL, &$view=NULL) {
+		if (!$this->_generated) {
 			$this->_instanceViewer->setInstance($this->_modelInstance);
 
 			$form=$this->content["form"];
 			$this->_generateContent($form);
 
-			if(isset($this->_toolbar)){
+			if (isset($this->_toolbar)) {
 				$this->_setToolbarPosition($form);
 			}
-			$this->content=JArray::sortAssociative($this->content, [PositionInTable::BEFORETABLE,"form",PositionInTable::AFTERTABLE]);
+			$this->content=JArray::sortAssociative($this->content, [PositionInTable::BEFORETABLE, "form", PositionInTable::AFTERTABLE]);
 			$this->_generated=true;
 		}
-		return parent::compile($js,$view);
+		return parent::compile($js, $view);
 	}
 
 	/**
 	 * @param HtmlForm $form
 	 */
-	protected function _generateContent($form){
-		$values= $this->_instanceViewer->getValues();
+	protected function _generateContent($form) {
+		$values=$this->_instanceViewer->getValues();
 		$count=$this->_instanceViewer->count();
 		$separators=$this->_instanceViewer->getSeparators();
 		$headers=$this->_instanceViewer->getHeaders();
 		$wrappers=$this->_instanceViewer->getWrappers();
 		\sort($separators);
 		$size=\sizeof($separators);
-		if($size===1){
+		if ($size===1) {
 			$i=-1;
-			foreach ($values as $v){
+			foreach ($values as $v) {
 				$this->_generateFields($form, [$v], $headers, $i, $wrappers);
 				$i++;
 			}
-		}else{
+		} else {
 			$separators[]=$count;
-			for($i=0;$i<$size;$i++){
-				$fields=\array_slice($values, $separators[$i]+1,$separators[$i+1]-$separators[$i]);
+			for ($i=0; $i<$size; $i++) {
+				$fields=\array_slice($values, $separators[$i]+1, $separators[$i+1]-$separators[$i]);
 				$this->_generateFields($form, $fields, $headers, $separators[$i], $wrappers);
 			}
 		}
 	}
 
-	protected function _generateFields($form,$values,$headers,$sepFirst,$wrappers){
+	protected function _generateFields($form, $values, $headers, $sepFirst, $wrappers) {
 		$wrapper=null;
-		if(isset($headers[$sepFirst+1]))
-			$form->addHeader($headers[$sepFirst+1],4,true);
-		if(isset($wrappers[$sepFirst+1])){
+		if (isset($headers[$sepFirst+1]))
+			$form->addHeader($headers[$sepFirst+1], 4, true);
+		if (isset($wrappers[$sepFirst+1])) {
 			$wrapper=$wrappers[$sepFirst+1];
 		}
-		if(\sizeof($values)===1){
+		if (\sizeof($values)===1) {
 			$added=$form->addField($values[0]);
-		}elseif(\sizeof($values)>1){
+		}elseif (\sizeof($values)>1) {
 			$added=$form->addFields($values);
-		}else
+		} else
 			return;
-		if(isset($wrapper)){
-			$added->wrap($wrapper[0],$wrapper[1]);
+		if (isset($wrapper)) {
+			$added->wrap($wrapper[0], $wrapper[1]);
 		}
 	}
 
 	/**
 	 * @return HtmlForm
 	 */
-	public function getForm(){
+	public function getForm() {
 		return $this->content["form"];
 	}
 
-	public function addSeparatorAfter($fieldNum){
+	public function addSeparatorAfter($fieldNum) {
 		$fieldNum=$this->_getIndex($fieldNum);
 		$this->_instanceViewer->addSeparatorAfter($fieldNum);
 		return $this;
@@ -112,12 +112,12 @@ 
 		return $this;
 	}
 
-	public function fieldAsReset($index,$cssStyle=NULL,$attributes=NULL){
-		return $this->_fieldAs(function($id,$name,$value,$caption) use ($cssStyle){
-			$button=new HtmlButton($id,$value,$cssStyle);
+	public function fieldAsReset($index, $cssStyle=NULL, $attributes=NULL) {
+		return $this->_fieldAs(function($id, $name, $value, $caption) use ($cssStyle){
+			$button=new HtmlButton($id, $value, $cssStyle);
 			$button->setProperty("type", "reset");
 			return $button;
-		}, $index,$attributes);
+		}, $index, $attributes);
 	}
 
 	/**
@@ -136,19 +136,19 @@ 
 		$this->content[$this->_toolbarPosition]=$this->_toolbar;
 	}
 
-	public function addDividerBefore($index,$title){
+	public function addDividerBefore($index, $title) {
 		$index=$this->_getIndex($index);
 		$this->_instanceViewer->addHeaderDividerBefore($index, $title);
 		return $this;
 	}
 
-	public function addWrapper($index,$contentBefore,$contentAfter=null){
+	public function addWrapper($index, $contentBefore, $contentAfter=null) {
 		$index=$this->_getIndex($index);
-		$this->_instanceViewer->addWrapper($index, $contentBefore,$contentAfter);
+		$this->_instanceViewer->addWrapper($index, $contentBefore, $contentAfter);
 		return $this;
 	}
 
-	public function run(JsUtils $js){
+	public function run(JsUtils $js) {
 		return parent::run($js);
 	}
 }

Ajax/semantic/traits/SemanticHtmlViewsTrait.php

Doc Comments -1 removed lines

@@ -21,7 +21,6 @@
 
 	/**
 	 * @param string $identifier
-	 * @param array $items
 	 * @return HtmlCardGroups
 	 */
 	public function htmlCardGroups($identifier, $cards=array()) {

Ajax/semantic/html/views/HtmlCardGroups.php

Spacing +3 added lines, -3 removed lines

@@ -8,11 +8,11 @@
 class HtmlCardGroups extends HtmlViewGroups {
 
 	public function __construct($identifier, $cards=array()) {
-		parent::__construct($identifier, "ui cards",$cards);
+		parent::__construct($identifier, "ui cards", $cards);
 	}
 
-	protected function createElement(){
-		return new HtmlCard("card-" . $this->count());
+	protected function createElement() {
+		return new HtmlCard("card-".$this->count());
 	}
 
 	public function newItem($identifier) {

Ajax/semantic/html/content/view/HtmlViewGroups.php

Braces +3 added lines, -2 removed lines

@@ -30,8 +30,9 @@
 			if (isset($extra)) {
 				$result->addExtraContent($extra);
 			}
-		} else
-			$result->addItemContent($value);
+		} else {
+					$result->addItemContent($value);
+		}
 		return $result;
 	}
 

Spacing +3 added lines, -3 removed lines

@@ -9,7 +9,7 @@ 
 
 abstract class HtmlViewGroups extends HtmlSemCollection {
 
-	public function __construct($identifier, $uiClass,$items=array()) {
+	public function __construct($identifier, $uiClass, $items=array()) {
 		parent::__construct($identifier, "div", $uiClass);
 		$this->addItems($items);
 	}
@@ -41,7 +41,7 @@ 
 	 * @param int $wide
 	 */
 	public function setWide($wide) {
-		$wide=Wide::getConstants()["W" . $wide];
+		$wide=Wide::getConstants()["W".$wide];
 		return $this->addToPropertyCtrl("class", $wide, Wide::getConstants());
 	}
 
@@ -59,7 +59,7 @@ 
 		return $this->addItem($function($object));
 	}
 
-	public function run(JsUtils $js){
+	public function run(JsUtils $js) {
 		$result=parent::run($js);
 		return $result->setItemSelector(".item");
 	}

Ajax/semantic/html/views/HtmlItems.php

Spacing +3 added lines, -3 removed lines

@@ -9,11 +9,11 @@
 class HtmlItems extends HtmlViewGroups {
 
 	public function __construct($identifier, $items=[]) {
-		parent::__construct($identifier, "ui items",$items);
+		parent::__construct($identifier, "ui items", $items);
 	}
 
-	protected function createElement(){
-		return new HtmlItem("item-" . $this->count());
+	protected function createElement() {
+		return new HtmlItem("item-".$this->count());
 	}
 
 	public function newItem($identifier) {

Ajax/semantic/html/content/view/ContentPartTrait.php

Doc Comments +5 added lines

@@ -7,6 +7,11 @@
  * @property mixed $content
  */
 trait ContentPartTrait{
+
+	/**
+	 * @param \Ajax\semantic\html\elements\HtmlButtonGroups $element
+	 * @param string $partKey
+	 */
 	public function addElementInPart($element,$partKey,$before=false,$force=false){
 		$part=$this->getPart($partKey,null,$force);
 		if($part instanceof  HtmlSemDoubleElement){

Spacing +8 added lines, -8 removed lines

@@ -6,24 +6,24 @@
  * @author jc
  * @property mixed $content
  */
-trait ContentPartTrait{
-	public function addElementInPart($element,$partKey,$before=false,$force=false){
-		$part=$this->getPart($partKey,null,$force);
-		if($part instanceof  HtmlSemDoubleElement){
+trait ContentPartTrait {
+	public function addElementInPart($element, $partKey, $before=false, $force=false) {
+		$part=$this->getPart($partKey, null, $force);
+		if ($part instanceof  HtmlSemDoubleElement) {
 			$this->content[$partKey]=$part;
-			$part->addContent($element,$before);
+			$part->addContent($element, $before);
 		}
 		return $this;
 	}
 
-	public function getPart($partKey, $index=NULL,$force=false) {
+	public function getPart($partKey, $index=NULL, $force=false) {
 		if (\array_key_exists($partKey, $this->content)) {
 			if (isset($index))
 				return $this->content[$partKey][$index];
 			return $this->content[$partKey];
 		}
-		if($force){
-			return new HtmlSemDoubleElement($partKey."-".$this->identifier,"div",$partKey);
+		if ($force) {
+			return new HtmlSemDoubleElement($partKey."-".$this->identifier, "div", $partKey);
 		}
 		return NULL;
 	}

Braces +3 added lines, -2 removed lines

@@ -18,8 +18,9 @@
 
 	public function getPart($partKey, $index=NULL,$force=false) {
 		if (\array_key_exists($partKey, $this->content)) {
-			if (isset($index))
-				return $this->content[$partKey][$index];
+			if (isset($index)) {
+							return $this->content[$partKey][$index];
+			}
 			return $this->content[$partKey];
 		}
 		if($force){

Ajax/semantic/html/content/view/HtmlViewContent.php

Doc Comments +4 added lines

@@ -12,6 +12,10 @@
 
 class HtmlViewContent extends HtmlSemDoubleElement {
 	use ContentPartTrait;
+
+	/**
+	 * @param string $identifier
+	 */
 	public function __construct($identifier, $content=array()) {
 		parent::__construct($identifier, "div", "content",[]);
 		$this->setContent($content);

Spacing +20 added lines, -20 removed lines

@@ -13,11 +13,11 @@ 
 class HtmlViewContent extends HtmlSemDoubleElement {
 	use ContentPartTrait;
 	public function __construct($identifier, $content=array()) {
-		parent::__construct($identifier, "div", "content",[]);
+		parent::__construct($identifier, "div", "content", []);
 		$this->setContent($content);
 	}
 
-	public function setContent($value){
+	public function setContent($value) {
 		if (\is_array($value)) {
 			$header=JArray::getValue($value, "header", 0);
 			$metas=JArray::getValue($value, "metas", 1);
@@ -27,24 +27,24 @@ 
 			if (isset($image)) {
 				$this->addImage($image);
 			}
-			$this->addHeaderContent($header, $metas, $description,$extra);
+			$this->addHeaderContent($header, $metas, $description, $extra);
 		} else
 			$this->addContent($value);
 	}
 
-	public function addElement($content, $baseClass="",$part=NULL) {
+	public function addElement($content, $baseClass="", $part=NULL) {
 		$count=\sizeof($this->content);
-		$result=new HtmlViewContent("element-" . $count . "-" . $this->identifier, $content);
+		$result=new HtmlViewContent("element-".$count."-".$this->identifier, $content);
 		$result->setClass($baseClass);
 		$this->addContent($result);
 		return $result;
 	}
 
 	public function addMeta($value, $direction=Direction::LEFT) {
-		if (\array_key_exists("meta", $this->content) === false) {
-			$this->content["meta"]=new HtmlSemDoubleElement("meta-" . $this->identifier, "div", "meta", array ());
+		if (\array_key_exists("meta", $this->content)===false) {
+			$this->content["meta"]=new HtmlSemDoubleElement("meta-".$this->identifier, "div", "meta", array());
 		}
-		if ($direction === Direction::RIGHT) {
+		if ($direction===Direction::RIGHT) {
 			$value=new HtmlSemDoubleElement("", "span", "", $value);
 			$value->setFloated($direction);
 		}
@@ -53,8 +53,8 @@ 
 	}
 
 	public function addExtra($value) {
-		if (\array_key_exists("extra", $this->content) === false) {
-			$this->content["extra"]=new HtmlSemDoubleElement("extra-" . $this->identifier, "div", "extra", array ());
+		if (\array_key_exists("extra", $this->content)===false) {
+			$this->content["extra"]=new HtmlSemDoubleElement("extra-".$this->identifier, "div", "extra", array());
 		}
 		$this->content["extra"]->addContent($value);
 		return $this->content["extra"];
@@ -75,15 +75,15 @@ 
 	 * @param boolean $before
 	 * @return HtmlButtonGroups
 	 */
-	public function addContentButtons($elements=array(), $asIcons=false,$part="extra",$before=false){
-		$buttons=new HtmlButtonGroups("buttons-".$this->identifier,$elements,$asIcons);
-		$this->addElementInPart($buttons,$part, $before,true);
+	public function addContentButtons($elements=array(), $asIcons=false, $part="extra", $before=false) {
+		$buttons=new HtmlButtonGroups("buttons-".$this->identifier, $elements, $asIcons);
+		$this->addElementInPart($buttons, $part, $before, true);
 		return $buttons;
 	}
 
 	public function addMetas($metas) {
 		if (\is_array($metas)) {
-			foreach ( $metas as $meta ) {
+			foreach ($metas as $meta) {
 				$this->addMeta($meta);
 			}
 		} else
@@ -92,7 +92,7 @@ 
 	}
 
 	public function addContentIcon($icon, $caption=NULL, $direction=Direction::LEFT) {
-		if ($direction === Direction::RIGHT) {
+		if ($direction===Direction::RIGHT) {
 			if (isset($caption)) {
 				$result=new HtmlSemDoubleElement("", "span", "", $caption);
 				$result->addIcon($icon);
@@ -110,7 +110,7 @@ 
 	}
 
 	public function addContentText($caption, $direction=Direction::LEFT) {
-		if ($direction === Direction::RIGHT) {
+		if ($direction===Direction::RIGHT) {
 			$result=new HtmlSemDoubleElement("", "span", "", $caption);
 			$this->addContent($result);
 			$result->setFloated($direction);
@@ -120,20 +120,20 @@ 
 	}
 
 	public function addContentIcons($icons, $direction=Direction::LEFT) {
-		foreach ( $icons as $icon ) {
+		foreach ($icons as $icon) {
 			$this->addContentIcon($icon, NULL, $direction);
 		}
 		return $this;
 	}
 
-	public function addHeaderContent($header, $metas=array(), $description=NULL,$extra=NULL) {
-		if(isset($header))
+	public function addHeaderContent($header, $metas=array(), $description=NULL, $extra=NULL) {
+		if (isset($header))
 			$this->addElement($header, "header");
 		$this->addMetas($metas);
 		if (isset($description)) {
 			$this->addElement($description, "description");
 		}
-		if(isset($extra)){
+		if (isset($extra)) {
 			$this->addExtra($extra);
 		}
 		return $this;

Braces +15 added lines, -10 removed lines

@@ -28,8 +28,9 @@ 
 				$this->addImage($image);
 			}
 			$this->addHeaderContent($header, $metas, $description,$extra);
-		} else
-			$this->addContent($value);
+		} else {
+					$this->addContent($value);
+		}
 	}
 
 	public function addElement($content, $baseClass="",$part=NULL) {
@@ -62,8 +63,9 @@ 
 
 	public function addImage($src="", $alt="", $size=NULL) {
 		$image=new HtmlImg("img-", $src, $alt);
-		if (isset($size))
-			$image->setSize($size);
+		if (isset($size)) {
+					$image->setSize($size);
+		}
 		$this->content['image']=$image;
 		return $image;
 	}
@@ -86,8 +88,9 @@ 
 			foreach ( $metas as $meta ) {
 				$this->addMeta($meta);
 			}
-		} else
-			$this->addMeta($metas);
+		} else {
+					$this->addMeta($metas);
+		}
 		return $this;
 	}
 
@@ -114,8 +117,9 @@ 
 			$result=new HtmlSemDoubleElement("", "span", "", $caption);
 			$this->addContent($result);
 			$result->setFloated($direction);
-		} else
-			$result=$this->addContent($caption);
+		} else {
+					$result=$this->addContent($caption);
+		}
 		return $result;
 	}
 
@@ -127,8 +131,9 @@ 
 	}
 
 	public function addHeaderContent($header, $metas=array(), $description=NULL,$extra=NULL) {
-		if(isset($header))
-			$this->addElement($header, "header");
+		if(isset($header)) {
+					$this->addElement($header, "header");
+		}
 		$this->addMetas($metas);
 		if (isset($description)) {
 			$this->addElement($description, "description");

Ajax/semantic/html/content/view/HtmlViewItem.php

Braces +17 added lines, -12 removed lines

@@ -19,8 +19,9 @@ 
 	public function __construct($identifier,$baseClass,$content=NULL) {
 		parent::__construct($identifier, "div", $baseClass);
 		$this->content=["content"=>new HtmlViewContent("content-".$this->identifier)];
-		if(isset($content))
-			$this->setContent($content);
+		if(isset($content)) {
+					$this->setContent($content);
+		}
 	}
 
 	public function setContent($value){
@@ -31,10 +32,12 @@ 
 			if (isset($image)) {
 				$this->addImage($image);
 			}
-			if(isset($content))
-				$this->content["content"]->setContent($content);
-			if(isset($extra))
-				$this->addExtraContent($extra);
+			if(isset($content)) {
+							$this->content["content"]->setContent($content);
+			}
+			if(isset($extra)) {
+							$this->addExtraContent($extra);
+			}
 		}
 	}
 
@@ -49,10 +52,11 @@ 
 		if (\array_key_exists($key, $this->content) === false) {
 			$this->content[$key]=[];
 		}
-		if($this->content[$key] instanceof HtmlViewContent)
-			$this->content[$key]->addElement($element);
-		else
-			$this->content[$key][]=$element;
+		if($this->content[$key] instanceof HtmlViewContent) {
+					$this->content[$key]->addElement($element);
+		} else {
+					$this->content[$key][]=$element;
+		}
 		return $element;
 	}
 
@@ -153,8 +157,9 @@ 
 		$this->addToProperty("class", "link");
 		if ($href !== "") {
 			$this->setProperty("href", $href);
-			if (isset($target))
-				$this->setProperty("target", $target);
+			if (isset($target)) {
+							$this->setProperty("target", $target);
+			}
 		}
 		return $this;
 	}

Spacing +28 added lines, -28 removed lines

@@ -17,14 +17,14 @@ 
 abstract class HtmlViewItem extends HtmlSemDoubleElement {
 	use ContentPartTrait;
 
-	public function __construct($identifier,$baseClass,$content=NULL) {
+	public function __construct($identifier, $baseClass, $content=NULL) {
 		parent::__construct($identifier, "div", $baseClass);
 		$this->content=["content"=>new HtmlViewContent("content-".$this->identifier)];
-		if(isset($content))
+		if (isset($content))
 			$this->setContent($content);
 	}
 
-	public function setContent($value){
+	public function setContent($value) {
 		if (\is_array($value)) {
 			$image=JArray::getValue($value, "image", 0);
 			$content=JArray::getValue($value, "content", 1);
@@ -32,57 +32,57 @@ 
 			if (isset($image)) {
 				$this->addImage($image);
 			}
-			if(isset($content))
+			if (isset($content))
 				$this->content["content"]->setContent($content);
-			if(isset($extra))
+			if (isset($extra))
 				$this->addExtraContent($extra);
 		}
 	}
 
 	private function createContent($content, $baseClass="content") {
 		$count=\sizeof($this->content);
-		$result=new HtmlViewContent("content-" . $count . "-" . $this->identifier, $content);
+		$result=new HtmlViewContent("content-".$count."-".$this->identifier, $content);
 		$result->setClass($baseClass);
 		return $result;
 	}
 
 	private function addElementIn($key, $element) {
-		if (\array_key_exists($key, $this->content) === false) {
+		if (\array_key_exists($key, $this->content)===false) {
 			$this->content[$key]=[];
 		}
-		if($this->content[$key] instanceof HtmlViewContent)
+		if ($this->content[$key] instanceof HtmlViewContent)
 			$this->content[$key]->addElement($element);
 		else
 			$this->content[$key][]=$element;
 		return $element;
 	}
 
-	public function addIcon($icon,$before=true){
-		return $this->addElementIn("icon",new HtmlIcon("icon-" . $this->identifier, $icon));
+	public function addIcon($icon, $before=true) {
+		return $this->addElementIn("icon", new HtmlIcon("icon-".$this->identifier, $icon));
 	}
 
 
 	public function addHeader($header, $niveau=4, $type="page") {
 		if (!$header instanceof HtmlHeader) {
-			$header=new HtmlHeader("header-" . $this->identifier, $niveau, $header, $type);
+			$header=new HtmlHeader("header-".$this->identifier, $niveau, $header, $type);
 		}
-		return $this->addElementIn("header",$this->createContent($header));
+		return $this->addElementIn("header", $this->createContent($header));
 	}
 
 	public function addImage($image, $title="") {
 		if (!$image instanceof HtmlImage) {
-			$image=new HtmlImage("image-" . $this->identifier, $image, $title);
+			$image=new HtmlImage("image-".$this->identifier, $image, $title);
 		}
 		$image->setClass("ui image");
-		return $this->content["image"]= $image;
+		return $this->content["image"]=$image;
 	}
 
-	public function addReveal($visibleContent, $hiddenContent=NULL, $type=RevealType::FADE, $attributeType=NULL,$key="extra-content") {
+	public function addReveal($visibleContent, $hiddenContent=NULL, $type=RevealType::FADE, $attributeType=NULL, $key="extra-content") {
 		$reveal=$visibleContent;
 		if (!$visibleContent instanceof HtmlReveal) {
-			$reveal=new HtmlReveal("reveral-" . $this->identifier, $visibleContent, $hiddenContent, $type, $attributeType);
+			$reveal=new HtmlReveal("reveral-".$this->identifier, $visibleContent, $hiddenContent, $type, $attributeType);
 		}
-		return $this->content[$key]= $reveal;
+		return $this->content[$key]=$reveal;
 	}
 
 	public function addRevealImage($visibleContent, $hiddenContent=NULL, $type=RevealType::FADE, $attributeType=NULL) {
@@ -90,11 +90,11 @@ 
 		if (!$visibleContent instanceof HtmlReveal) {
 			return $this->addReveal(new HtmlImage("", $visibleContent), new HtmlImage("", $hiddenContent), $type, $attributeType);
 		}
-		return $this->content["image"]= $reveal;
+		return $this->content["image"]=$reveal;
 	}
 
 	public function addExtraContent($content=array()) {
-		return $this->content["extra-content"]= $this->createContent($content, "extra content");
+		return $this->content["extra-content"]=$this->createContent($content, "extra content");
 	}
 
 	/*public function addContent($content=array(), $before=false) {
@@ -112,23 +112,23 @@ 
 	 * @param boolean $before
 	 * @return HtmlButtonGroups
 	 */
-	public function addContentButtons($elements=array(), $asIcons=false,$part="extra",$before=false){
-		return $this->content["content"]->addContentButtons($elements,$asIcons,$part, $before);
+	public function addContentButtons($elements=array(), $asIcons=false, $part="extra", $before=false) {
+		return $this->content["content"]->addContentButtons($elements, $asIcons, $part, $before);
 	}
 
 
 
-	public function addItemHeaderContent($header, $metas=array(), $description=NULL,$extra=NULL) {
-		return $this->content["content"]->addHeaderContent($header, $metas, $description,$extra);
+	public function addItemHeaderContent($header, $metas=array(), $description=NULL, $extra=NULL) {
+		return $this->content["content"]->addHeaderContent($header, $metas, $description, $extra);
 	}
 
 	public function addItemContent($content=array()) {
 		$count=\sizeof($this->content);
-		return $this->addElementIn("content", new HtmlViewContent("content-" . $count . "-" . $this->identifier, $content));
+		return $this->addElementIn("content", new HtmlViewContent("content-".$count."-".$this->identifier, $content));
 	}
 
 	public function getItemContent() {
-		return $this->getPart("content",null,true);
+		return $this->getPart("content", null, true);
 	}
 
 	public function getItemExtraContent() {
@@ -150,13 +150,13 @@ 
 	 * @see HtmlSemDoubleElement::compile()
 	 */
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
-		$this->content=JArray::sortAssociative($this->content, ["header","image","icon","content","extra-content"]);
+		$this->content=JArray::sortAssociative($this->content, ["header", "image", "icon", "content", "extra-content"]);
 		return parent::compile($js, $view);
 	}
 
-	public function asLink($href="",$target=NULL) {
+	public function asLink($href="", $target=NULL) {
 		$this->addToProperty("class", "link");
-		if ($href !== "") {
+		if ($href!=="") {
 			$this->setProperty("href", $href);
 			if (isset($target))
 				$this->setProperty("target", $target);