phpMv / phpMv-UI

Ajax/semantic/components/validation/AjaxRule.php

Spacing +4 added lines, -4 removed lines

@@ -16,9 +16,9 @@ 
 
 	private $ajaxCall;
 
-	public function __construct($type, $url, $params, $jsCallback = null, $method = 'post', $parameters = [], $prompt = NULL, $value = NULL) {
+	public function __construct($type, $url, $params, $jsCallback=null, $method='post', $parameters=[], $prompt=NULL, $value=NULL) {
 		parent::__construct($type, $prompt, $value);
-		$parameters = \array_merge([
+		$parameters=\array_merge([
 			'async' => false,
 			'url' => $url,
 			'params' => $params,
@@ -29,10 +29,10 @@ 
 			'preventDefault' => false,
 			'responseElement' => null
 		], $parameters);
-		$this->ajaxCall = new AjaxCall($method, $parameters);
+		$this->ajaxCall=new AjaxCall($method, $parameters);
 	}
 
 	public function compile(JsUtils $js) {
-		$js->exec(Rule::custom($this->getType(), "function(value,ruleValue){var result=true;" . $this->ajaxCall->compile($js) . "return result;}"), true);
+		$js->exec(Rule::custom($this->getType(), "function(value,ruleValue){var result=true;".$this->ajaxCall->compile($js)."return result;}"), true);
 	}
 }
\ No newline at end of file

Ajax/semantic/components/validation/CustomRule.php

Spacing +2 added lines, -2 removed lines

@@ -15,9 +15,9 @@
 
 	protected $jsFunction;
 
-	public function __construct($type, $jsFunction, $prompt = NULL, $value = NULL) {
+	public function __construct($type, $jsFunction, $prompt=NULL, $value=NULL) {
 		parent::__construct($type, $prompt, $value);
-		$this->jsFunction = $jsFunction;
+		$this->jsFunction=$jsFunction;
 	}
 
 	public function compile(JsUtils $js) {

Ajax/semantic/html/elements/HtmlRail.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
  */
 class HtmlRail extends HtmlSemDoubleElement {
 
-	public function __construct($identifier, $content = NULL) {
+	public function __construct($identifier, $content=NULL) {
 		parent::__construct($identifier, 'div', 'ui rail', $content);
 	}
 

Ajax/semantic/traits/SemanticHtmlElementsTrait.php

Spacing +16 added lines, -16 removed lines

@@ -45,7 +45,7 @@ 
 	 * @param string $onClick
 	 * @return HtmlButton
 	 */
-	public function htmlButton($identifier, $value = null, $cssStyle = null, $onClick = null) {
+	public function htmlButton($identifier, $value=null, $cssStyle=null, $onClick=null) {
 		return $this->addHtmlComponent(new HtmlButton($identifier, $value, $cssStyle, $onClick));
 	}
 
@@ -59,7 +59,7 @@ 
 	 * @param boolean $asIcons
 	 * @return HtmlButtonGroups
 	 */
-	public function htmlButtonGroups($identifier, $elements = array(), $asIcons = false) {
+	public function htmlButtonGroups($identifier, $elements=array(), $asIcons=false) {
 		return $this->addHtmlComponent(new HtmlButtonGroups($identifier, $elements, $asIcons));
 	}
 
@@ -72,7 +72,7 @@ 
 	 * @param string $content
 	 * @return HtmlContainer
 	 */
-	public function htmlContainer($identifier, $content = "") {
+	public function htmlContainer($identifier, $content="") {
 		return $this->addHtmlComponent(new HtmlContainer($identifier, $content));
 	}
 
@@ -85,7 +85,7 @@ 
 	 * @param string $content
 	 * @return HtmlDivider
 	 */
-	public function htmlDivider($identifier, $content = "", $tagName = "div") {
+	public function htmlDivider($identifier, $content="", $tagName="div") {
 		return $this->addHtmlComponent(new HtmlDivider($identifier, $content, $tagName));
 	}
 
@@ -100,7 +100,7 @@ 
 	 * @param string $type
 	 * @return HtmlHeader
 	 */
-	public function htmlHeader($identifier, $niveau = 1, $content = NULL, $type = "page") {
+	public function htmlHeader($identifier, $niveau=1, $content=NULL, $type="page") {
 		return $this->addHtmlComponent(new HtmlHeader($identifier, $niveau, $content, $type));
 	}
 
@@ -128,7 +128,7 @@ 
 	 * @param string $size
 	 * @return HtmlImage
 	 */
-	public function htmlImage($identifier, $src = "", $alt = "", $size = NULL) {
+	public function htmlImage($identifier, $src="", $alt="", $size=NULL) {
 		return $this->addHtmlComponent(new HtmlImage($identifier, $src, $alt, $size));
 	}
 
@@ -142,7 +142,7 @@ 
 	 * @param string $size
 	 * @return HtmlIconGroups
 	 */
-	public function htmlIconGroups($identifier, $icons = array(), $size = "") {
+	public function htmlIconGroups($identifier, $icons=array(), $size="") {
 		return $this->addHtmlComponent(new HtmlIconGroups($identifier, $icons, $size));
 	}
 
@@ -157,7 +157,7 @@ 
 	 * @param string $placeholder
 	 * @return HtmlInput
 	 */
-	public function htmlInput($identifier, $type = "text", $value = "", $placeholder = "") {
+	public function htmlInput($identifier, $type="text", $value="", $placeholder="") {
 		return $this->addHtmlComponent(new HtmlInput($identifier, $type, $value, $placeholder));
 	}
 
@@ -171,7 +171,7 @@ 
 	 * @param string $tagName
 	 * @return HtmlLabel
 	 */
-	public function htmlLabel($identifier, $content = "", $icon = NULL, $tagName = "div") {
+	public function htmlLabel($identifier, $content="", $icon=NULL, $tagName="div") {
 		return $this->addHtmlComponent(new HtmlLabel($identifier, $content, $icon, $tagName));
 	}
 
@@ -182,7 +182,7 @@ 
 	 * @param array $attributes
 	 * @return HtmlLabelGroups
 	 */
-	public function htmlLabelGroups($identifier, $labels = array(), $attributes = array()) {
+	public function htmlLabelGroups($identifier, $labels=array(), $attributes=array()) {
 		return $this->addHtmlComponent(new HtmlLabelGroups($identifier, $labels, $attributes));
 	}
 
@@ -193,7 +193,7 @@ 
 	 * @param array $items
 	 * @return HtmlList
 	 */
-	public function htmlList($identifier, $items = array()) {
+	public function htmlList($identifier, $items=array()) {
 		return $this->addHtmlComponent(new HtmlList($identifier, $items));
 	}
 
@@ -204,7 +204,7 @@ 
 	 * @param string $content
 	 * @return HtmlSegment
 	 */
-	public function htmlSegment($identifier, $content = "") {
+	public function htmlSegment($identifier, $content="") {
 		return $this->addHtmlComponent(new HtmlSegment($identifier, $content));
 	}
 
@@ -216,7 +216,7 @@ 
 	 *        	the segments
 	 * @return HtmlSegmentGroups
 	 */
-	public function htmlSegmentGroups($identifier, $items = array()) {
+	public function htmlSegmentGroups($identifier, $items=array()) {
 		return $this->addHtmlComponent(new HtmlSegmentGroups($identifier, $items));
 	}
 
@@ -230,7 +230,7 @@ 
 	 * @param Direction|string $attributeType
 	 * @return HtmlReveal
 	 */
-	public function htmlReveal($identifier, $visibleContent, $hiddenContent, $type = RevealType::FADE, $attributeType = NULL) {
+	public function htmlReveal($identifier, $visibleContent, $hiddenContent, $type=RevealType::FADE, $attributeType=NULL) {
 		return $this->addHtmlComponent(new HtmlReveal($identifier, $visibleContent, $hiddenContent, $type, $attributeType));
 	}
 
@@ -241,7 +241,7 @@ 
 	 * @param array $steps
 	 * @return HtmlStep
 	 */
-	public function htmlStep($identifier, $steps = array()) {
+	public function htmlStep($identifier, $steps=array()) {
 		return $this->addHtmlComponent(new HtmlStep($identifier, $steps));
 	}
 
@@ -275,7 +275,7 @@ 
 	 * @param mixed $content
 	 * @return HtmlRail
 	 */
-	public function htmlRail($identifier, $content = null) {
+	public function htmlRail($identifier, $content=null) {
 		return $this->addHtmlComponent(new HtmlRail($identifier, $content));
 	}
 }

Ajax/common/html/HtmlDoubleElement.php

Spacing +36 added lines, -36 removed lines

@@ -13,19 +13,19 @@ 
 	 */
 	protected $content;
 
-	protected $wrapContentBefore = "";
+	protected $wrapContentBefore="";
 
-	protected $wrapContentAfter = "";
+	protected $wrapContentAfter="";
 
 	protected $_editableContent;
 
-	public function __construct($identifier, $tagName = "p") {
+	public function __construct($identifier, $tagName="p") {
 		parent::__construct($identifier, $tagName);
-		$this->_template = '<%tagName% id="%identifier%" %properties%>%wrapContentBefore%%content%%wrapContentAfter%</%tagName%>';
+		$this->_template='<%tagName% id="%identifier%" %properties%>%wrapContentBefore%%content%%wrapContentAfter%</%tagName%>';
 	}
 
 	public function setContent($content) {
-		$this->content = $content;
+		$this->content=$content;
 		return $this;
 	}
 
@@ -33,19 +33,19 @@ 
 		return $this->content;
 	}
 
-	public function addContent($content, $before = false) {
-		if (! \is_array($this->content)) {
+	public function addContent($content, $before=false) {
+		if (!\is_array($this->content)) {
 			if (isset($this->content))
-				$this->content = array(
+				$this->content=array(
 					$this->content
 				);
 			else
-				$this->content = array();
+				$this->content=array();
 		}
 		if ($before)
 			\array_unshift($this->content, $content);
 		else
-			$this->content[] = $content;
+			$this->content[]=$content;
 		return $this;
 	}
 
@@ -68,9 +68,9 @@ 
 
 	public function setValue($value) {}
 
-	public function wrapContent($before, $after = "") {
-		$this->wrapContentBefore .= $before;
-		$this->wrapContentAfter = $after . $this->wrapContentAfter;
+	public function wrapContent($before, $after="") {
+		$this->wrapContentBefore.=$before;
+		$this->wrapContentAfter=$after.$this->wrapContentAfter;
 		return $this;
 	}
 
@@ -89,14 +89,14 @@ 
 	}
 
 	protected function _getContentInstances($class, $content) {
-		$instances = [];
+		$instances=[];
 		if ($content instanceof $class) {
-			$instances[] = $content;
+			$instances[]=$content;
 		} elseif ($content instanceof HtmlDoubleElement) {
-			$instances = \array_merge($instances, $content->getContentInstances($class));
+			$instances=\array_merge($instances, $content->getContentInstances($class));
 		} elseif (\is_array($content)) {
 			foreach ($content as $element) {
-				$instances = \array_merge($instances, $this->_getContentInstances($class, $element));
+				$instances=\array_merge($instances, $this->_getContentInstances($class, $element));
 			}
 		}
 		return $instances;
@@ -107,7 +107,7 @@ 
 	 *
 	 * @return HtmlDoubleElement
 	 */
-	public function asLink($href = NULL, $target = NULL) {
+	public function asLink($href=NULL, $target=NULL) {
 		if (isset($href))
 			$this->setProperty("href", $href);
 		if (isset($target))
@@ -122,28 +122,28 @@ 
 		return \strip_tags($this->content);
 	}
 
-	public function asEditable(HtmlFormField $field, $asForm = false, $setValueProperty = "val()") {
-		$idF = $field->getIdentifier();
-		$idE = $idF;
+	public function asEditable(HtmlFormField $field, $asForm=false, $setValueProperty="val()") {
+		$idF=$field->getIdentifier();
+		$idE=$idF;
 		if ($asForm) {
-			$frm = new HtmlForm("frm-" . $field->getIdentifier());
+			$frm=new HtmlForm("frm-".$field->getIdentifier());
 			$frm->setProperty("onsubmit", "return false;");
-			$fields = $frm->addFields();
-			$idE = $frm->getIdentifier();
+			$fields=$frm->addFields();
+			$idE=$frm->getIdentifier();
 			$fields->addItem($field);
-			$fields->addButtonIcon("bt-okay", "check", "green mini", "\$('#" . $idE . "').trigger('validate',{value: $('#'+idF+' input').val()});");
-			$fields->addButtonIcon("bt-cancel", "close", "mini", "\$('#" . $idE . "').trigger('endEdit');");
-			$this->_editableContent = $frm;
-			$keypress = "";
-			$focusOut = "";
+			$fields->addButtonIcon("bt-okay", "check", "green mini", "\$('#".$idE."').trigger('validate',{value: $('#'+idF+' input').val()});");
+			$fields->addButtonIcon("bt-cancel", "close", "mini", "\$('#".$idE."').trigger('endEdit');");
+			$this->_editableContent=$frm;
+			$keypress="";
+			$focusOut="";
 		} else {
-			$focusOut = "if(e.relatedTarget==null)elm.trigger('endEdit');";
-			$this->_editableContent = $field;
-			$keypress = "$('#" . $idF . "').keyup(function(e){if(e.which == 13) {\$('#" . $idE . "').trigger('validate',{value: $('#'+idF+' input').val()});}if(e.keyCode===27) {\$('#" . $idE . "').trigger('endEdit');}});";
+			$focusOut="if(e.relatedTarget==null)elm.trigger('endEdit');";
+			$this->_editableContent=$field;
+			$keypress="$('#".$idF."').keyup(function(e){if(e.which == 13) {\$('#".$idE."').trigger('validate',{value: $('#'+idF+' input').val()});}if(e.keyCode===27) {\$('#".$idE."').trigger('endEdit');}});";
 		}
 		$this->_editableContent->setProperty("style", "display:none;");
-		$this->onCreate("let idF='" . $idF . "';let idE='" . $idE . "';let elm=$('#'+idE);let self=$('#" . $this->getIdentifier() . "');" . $keypress . "elm.on('validate',function(){self.html($('#'+idE+' input')." . $setValueProperty . ");elm.trigger('endEdit');});elm.on('endEdit',function(){self.show();$(this).hide();});elm.focusout(function(e){" . $focusOut . "});");
-		$this->onClick("let self=$(this);self.hide();" . $field->setJsContent("self.html()") . ";$('#" . $idF . " input').trigger('change');elm.show();$('#'+idE+' input').focus();");
+		$this->onCreate("let idF='".$idF."';let idE='".$idE."';let elm=$('#'+idE);let self=$('#".$this->getIdentifier()."');".$keypress."elm.on('validate',function(){self.html($('#'+idE+' input').".$setValueProperty.");elm.trigger('endEdit');});elm.on('endEdit',function(){self.show();$(this).hide();});elm.focusout(function(e){".$focusOut."});");
+		$this->onClick("let self=$(this);self.hide();".$field->setJsContent("self.html()").";$('#".$idF." input').trigger('change');elm.show();$('#'+idE+' input').focus();");
 	}
 
 	/**
@@ -151,8 +151,8 @@ 
 	 * {@inheritdoc}
 	 * @see \Ajax\common\html\BaseHtml::compile_once()
 	 */
-	protected function compile_once(\Ajax\JsUtils $js = NULL, &$view = NULL) {
-		if (! $this->_compiled && isset($this->_editableContent)) {
+	protected function compile_once(\Ajax\JsUtils $js=NULL, &$view=NULL) {
+		if (!$this->_compiled && isset($this->_editableContent)) {
 			$this->wrap("", $this->_editableContent);
 		}
 		parent::compile_once($js, $view);

Braces +16 added lines, -12 removed lines

@@ -35,17 +35,19 @@ 
 
 	public function addContent($content, $before = false) {
 		if (! \is_array($this->content)) {
-			if (isset($this->content))
-				$this->content = array(
+			if (isset($this->content)) {
+							$this->content = array(
 					$this->content
 				);
-			else
-				$this->content = array();
+			} else {
+							$this->content = array();
+			}
+		}
+		if ($before) {
+					\array_unshift($this->content, $content);
+		} else {
+					$this->content[] = $content;
 		}
-		if ($before)
-			\array_unshift($this->content, $content);
-		else
-			$this->content[] = $content;
 		return $this;
 	}
 
@@ -108,10 +110,12 @@ 
 	 * @return HtmlDoubleElement
 	 */
 	public function asLink($href = NULL, $target = NULL) {
-		if (isset($href))
-			$this->setProperty("href", $href);
-		if (isset($target))
-			$this->setProperty("target", $target);
+		if (isset($href)) {
+					$this->setProperty("href", $href);
+		}
+		if (isset($target)) {
+					$this->setProperty("target", $target);
+		}
 		return $this->setTagName("a");
 	}
 

Ajax/semantic/widgets/dataform/DataForm.php

Spacing +40 added lines, -40 removed lines

@@ -24,39 +24,39 @@ 
 	private $_inverted;
 	
 	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]);
-			if($this->_inverted){
+			$this->content=JArray::sortAssociative($this->content, [PositionInTable::BEFORETABLE, 'form', PositionInTable::AFTERTABLE]);
+			if ($this->_inverted) {
 				$this->content['form']->setInverted(true);
 			}
 			$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();
@@ -65,42 +65,42 @@ 
 		\sort($separators);
 		$size=\count($separators);
 		$nb=0;
-		if($size===1){
+		if ($size===1) {
 			$i=-1;
-			foreach ($values as $k=>$v){
-				$this->_generateFields($form, [$v], $headers, $i, $wrappers,$nb++,$names[$k]??'');
+			foreach ($values as $k=>$v) {
+				$this->_generateFields($form, [$v], $headers, $i, $wrappers, $nb++, $names[$k] ?? '');
 				$i++;
 			}
-		}else{
+		} else {
 			$separators[]=$count;
-			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,$nb++,$names[$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, $nb++, $names[$i] ?? '');
 			}
 		}
-		if($this->_hasRules && !$this->getForm()->hasValidationParams()){
+		if ($this->_hasRules && !$this->getForm()->hasValidationParams()) {
 				$this->setValidationParams(['inline'=>true]);
 		}
 	}
 
-	protected function _generateFields($form, $values, $headers, $sepFirst, $wrappers, $nb, $name){
+	protected function _generateFields($form, $values, $headers, $sepFirst, $wrappers, $nb, $name) {
 		$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];
 		}
 		$count=\count($values);
-		if($count===1){
+		if ($count===1) {
 			$added=$form->addField($values[0]);
-		}elseif($count>1){
+		}elseif ($count>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]);
 		}
-		$this->execHook('onGenerateFields',$added,$nb,$name);
+		$this->execHook('onGenerateFields', $added, $nb, $name);
 	}
 	
 	/**
@@ -108,18 +108,18 @@ 
 	 * the generated field is the first parameter
 	 * @param callable $callback the fonction to call when a field is generated
 	 */
-	public function onGenerateField($callback){
-		$this->addHook('onGenerateFields',$callback);
+	public function onGenerateField($callback) {
+		$this->addHook('onGenerateFields', $callback);
 	}
 
 	/**
 	 * @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;
@@ -134,12 +134,12 @@ 
 		return $this;
 	}
 
-	public function fieldAsReset($index,$cssStyle=NULL,$attributes=NULL){
-		return $this->_fieldAs(function($id,$name,$value) use ($cssStyle){
-			$button=new HtmlButton($id,$value,$cssStyle);
+	public function fieldAsReset($index, $cssStyle=NULL, $attributes=NULL) {
+		return $this->_fieldAs(function($id, $name, $value) use ($cssStyle){
+			$button=new HtmlButton($id, $value, $cssStyle);
 			$button->setProperty('type', 'reset');
 			return $button;
-		}, $index,$attributes);
+		}, $index, $attributes);
 	}
 
 	/**
@@ -158,19 +158,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) {
 		parent::run($js);
 	}
 	

Braces +8 added lines, -6 removed lines

@@ -71,7 +71,7 @@ 
 				$this->_generateFields($form, [$v], $headers, $i, $wrappers,$nb++,$names[$k]??'');
 				$i++;
 			}
-		}else{
+		} else{
 			$separators[]=$count;
 			for($i=0;$i<$size;$i++){
 				$fields=\array_slice($values, $separators[$i]+1,$separators[$i+1]-$separators[$i]);
@@ -85,18 +85,20 @@ 
 
 	protected function _generateFields($form, $values, $headers, $sepFirst, $wrappers, $nb, $name){
 		$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];
 		}
 		$count=\count($values);
 		if($count===1){
 			$added=$form->addField($values[0]);
-		}elseif($count>1){
+		} elseif($count>1){
 			$added=$form->addFields($values);
-		}else
-			return;
+		} else {
+					return;
+		}
 		if(isset($wrapper)){
 			$added->wrap($wrapper[0],$wrapper[1]);
 		}

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

Spacing +15 added lines, -15 removed lines

@@ -7,20 +7,20 @@ 
 
 class HtmlFormDropdown extends HtmlFormField {
 
-	public function __construct($identifier, $items = array(), $label = NULL, $value = "", $multiple = false, $associative = true) {
-		parent::__construct("field-" . $identifier, (new HtmlDropdown("dropdown-" . $identifier, $value, $items, $associative))->asSelect($identifier, $multiple), $label);
-		$this->_identifier = $identifier;
+	public function __construct($identifier, $items=array(), $label=NULL, $value="", $multiple=false, $associative=true) {
+		parent::__construct("field-".$identifier, (new HtmlDropdown("dropdown-".$identifier, $value, $items, $associative))->asSelect($identifier, $multiple), $label);
+		$this->_identifier=$identifier;
 	}
 
 	public function setItems($items) {
 		return $this->getField()->setItems($items);
 	}
 
-	public function addItem($item, $value = NULL, $image = NULL) {
+	public function addItem($item, $value=NULL, $image=NULL) {
 		return $this->getField()->addItem($item, $value, $image);
 	}
 
-	public static function multipleDropdown($identifier, $items = array(), $label = NULL, $value = "", $associative = true) {
+	public static function multipleDropdown($identifier, $items=array(), $label=NULL, $value="", $associative=true) {
 		return new HtmlFormDropdown($identifier, $items, $label, $value, true, $associative);
 	}
 
@@ -32,7 +32,7 @@ 
 		return $this->getField()->getInput();
 	}
 
-	public function asSelect($name = NULL, $multiple = false, $selection = true) {
+	public function asSelect($name=NULL, $multiple=false, $selection=true) {
 		$this->getField()->asSelect($name, $multiple, $selection);
 		return $this;
 	}
@@ -42,11 +42,11 @@ 
 	 * @param boolean $floating
 	 * @return HtmlDropdown
 	 */
-	public function asButton($floating = false) {
-		$field = $this->content["field"];
-		$label = $this->content["label"];
+	public function asButton($floating=false) {
+		$field=$this->content["field"];
+		$label=$this->content["label"];
 		$field->addContent($label);
-		$this->content = [
+		$this->content=[
 			"field" => $field
 		];
 		$this->content["field"]->asButton($floating);
@@ -61,15 +61,15 @@ 
 	 * @param boolean $labeled
 	 * @return mixed|HtmlButton
 	 */
-	public function addAction($action, $direction = Direction::RIGHT, $icon = NULL, $labeled = false) {
+	public function addAction($action, $direction=Direction::RIGHT, $icon=NULL, $labeled=false) {
 		$this->getField()->setStyle('display:inline-block; width: auto');
-		$actionO = $action;
-		if (\is_object($action) === false) {
-			$actionO = new HtmlButton("action-" . $this->identifier, $action);
+		$actionO=$action;
+		if (\is_object($action)===false) {
+			$actionO=new HtmlButton("action-".$this->identifier, $action);
 			if (isset($icon))
 				$actionO->addIcon($icon, true, $labeled);
 		}
-		$this->addContent($actionO, \strstr($direction, Direction::LEFT) !== false);
+		$this->addContent($actionO, \strstr($direction, Direction::LEFT)!==false);
 		return $actionO;
 	}
 }

Braces +3 added lines, -2 removed lines

@@ -66,8 +66,9 @@
 		$actionO = $action;
 		if (\is_object($action) === false) {
 			$actionO = new HtmlButton("action-" . $this->identifier, $action);
-			if (isset($icon))
-				$actionO->addIcon($icon, true, $labeled);
+			if (isset($icon)) {
+							$actionO->addIcon($icon, true, $labeled);
+			}
 		}
 		$this->addContent($actionO, \strstr($direction, Direction::LEFT) !== false);
 		return $actionO;

Ajax/semantic/html/base/HtmlSemNavElement.php

Spacing +11 added lines, -11 removed lines

@@ -29,12 +29,12 @@ 
 	 *
 	 * @var string|array
 	 */
-	protected $_contentSeparator = "";
+	protected $_contentSeparator="";
 
 	public function __construct($identifier, $tagName, $baseClass) {
 		parent::__construct($identifier, $tagName, $baseClass);
-		$this->root = "";
-		$this->attr = "data-ajax";
+		$this->root="";
+		$this->attr="data-ajax";
 	}
 
 	/**
@@ -54,23 +54,23 @@ 
 		return JArray::implode($this->_contentSeparator, $this->content);
 	}
 
-	public function setContentDivider($divider, $index = NULL) {
-		$divider = "<div class='divider'> {$divider} </div>";
+	public function setContentDivider($divider, $index=NULL) {
+		$divider="<div class='divider'> {$divider} </div>";
 		return $this->setDivider($divider, $index);
 	}
 
-	public function setIconContentDivider($iconContentDivider, $index = NULL) {
-		$contentDivider = "<i class='" . $iconContentDivider . " icon divider'></i>";
+	public function setIconContentDivider($iconContentDivider, $index=NULL) {
+		$contentDivider="<i class='".$iconContentDivider." icon divider'></i>";
 		return $this->setDivider($contentDivider, $index);
 	}
 
 	protected function setDivider($divider, $index) {
 		if (isset($index)) {
-			if (! \is_array($this->_contentSeparator))
-				$this->_contentSeparator = array_fill(0, $this->count() - 1, $this->_contentSeparator);
-			$this->_contentSeparator[$index] = $divider;
+			if (!\is_array($this->_contentSeparator))
+				$this->_contentSeparator=array_fill(0, $this->count()-1, $this->_contentSeparator);
+			$this->_contentSeparator[$index]=$divider;
 		} else {
-			$this->_contentSeparator = $divider;
+			$this->_contentSeparator=$divider;
 		}
 		return $this;
 	}

Braces +3 added lines, -2 removed lines

@@ -66,8 +66,9 @@
 
 	protected function setDivider($divider, $index) {
 		if (isset($index)) {
-			if (! \is_array($this->_contentSeparator))
-				$this->_contentSeparator = array_fill(0, $this->count() - 1, $this->_contentSeparator);
+			if (! \is_array($this->_contentSeparator)) {
+							$this->_contentSeparator = array_fill(0, $this->count() - 1, $this->_contentSeparator);
+			}
 			$this->_contentSeparator[$index] = $divider;
 		} else {
 			$this->_contentSeparator = $divider;

Ajax/semantic/html/modules/HtmlModal.php

Spacing +43 added lines, -43 removed lines

@@ -16,11 +16,11 @@ 
  */
 class HtmlModal extends HtmlSemDoubleElement {
 
-	protected $_params = [];
+	protected $_params=[];
 
-	protected $_paramParts = [];
+	protected $_paramParts=[];
 
-	public function __construct($identifier, $header = '', $content = '', $actions = null) {
+	public function __construct($identifier, $header='', $content='', $actions=null) {
 		parent::__construct($identifier, 'div', 'ui modal');
 		if (isset($header)) {
 			$this->setHeader($header);
@@ -34,12 +34,12 @@ 
 	}
 
 	public function setHeader($value) {
-		$this->content['header'] = new HtmlSemDoubleElement('header-' . $this->identifier, 'a', 'header', $value);
+		$this->content['header']=new HtmlSemDoubleElement('header-'.$this->identifier, 'a', 'header', $value);
 		return $this;
 	}
 
 	public function setContent($value) {
-		$this->content['content'] = new HtmlSemDoubleElement('content-' . $this->identifier, 'div', 'content', $value);
+		$this->content['content']=new HtmlSemDoubleElement('content-'.$this->identifier, 'div', 'content', $value);
 		return $this;
 	}
 
@@ -50,14 +50,14 @@ 
 	 * @return HtmlButton[]
 	 */
 	public function setActions($actions): array {
-		$this->content['actions'] = new HtmlSemDoubleElement('content-' . $this->identifier, 'div', 'actions');
-		$r = [];
+		$this->content['actions']=new HtmlSemDoubleElement('content-'.$this->identifier, 'div', 'actions');
+		$r=[];
 		if (\is_array($actions)) {
 			foreach ($actions as $action) {
-				$r[] = $this->addAction($action);
+				$r[]=$this->addAction($action);
 			}
 		} else {
-			$r[] = $this->addAction($actions);
+			$r[]=$this->addAction($actions);
 		}
 		return $r;
 	}
@@ -68,24 +68,24 @@ 
 	 * @return HtmlButton
 	 */
 	public function addAction($action) {
-		if (! $action instanceof BaseHtml) {
-			$class = '';
+		if (!$action instanceof BaseHtml) {
+			$class='';
 			if (\array_search($action, [
 				'Okay',
 				'Yes',
 				'Validate'
-			]) !== false) {
-				$class = 'approve';
+			])!==false) {
+				$class='approve';
 			}
 			if (\array_search($action, [
 				'Close',
 				'Cancel',
 				'No'
-			]) !== false) {
-				$class = 'cancel';
+			])!==false) {
+				$class='cancel';
 			}
-			$action = new HtmlButton('action-' . $this->identifier . '-' . JArray::count($this->content['actions']->getContent()), $action);
-			if ($class !== '')
+			$action=new HtmlButton('action-'.$this->identifier.'-'.JArray::count($this->content['actions']->getContent()), $action);
+			if ($class!=='')
 				$action->addToProperty('class', $class);
 		}
 		return $this->addElementInPart($action, 'actions');
@@ -100,33 +100,33 @@ 
 		return $this->content['actions']->getContent()[$index];
 	}
 
-	public function addContent($content, $before = false) {
+	public function addContent($content, $before=false) {
 		$this->content['content']->addContent($content, $before);
 		return $this;
 	}
 
-	public function addImageContent($image, $description = NULL) {
-		$content = $this->content['content'];
+	public function addImageContent($image, $description=NULL) {
+		$content=$this->content['content'];
 		if (isset($description)) {
-			$description = new HtmlSemDoubleElement('description-' . $this->identifier, 'div', 'description', $description);
+			$description=new HtmlSemDoubleElement('description-'.$this->identifier, 'div', 'description', $description);
 			$content->addContent($description, true);
 		}
-		if ($image !== '') {
-			$img = new HtmlImage('image-' . $this->identifier, $image, '', 'medium');
+		if ($image!=='') {
+			$img=new HtmlImage('image-'.$this->identifier, $image, '', 'medium');
 			$content->addContent($img, true);
 			$content->addToProperty('class', 'image');
 		}
 		return $this;
 	}
 
-	public function addIconContent($icon, $description = NULL) {
-		$content = $this->content['content'];
+	public function addIconContent($icon, $description=NULL) {
+		$content=$this->content['content'];
 		if (isset($description)) {
-			$description = new HtmlSemDoubleElement('description-' . $this->identifier, 'div', 'description', $description);
+			$description=new HtmlSemDoubleElement('description-'.$this->identifier, 'div', 'description', $description);
 			$content->addContent($description, true);
 		}
-		if ($icon !== '') {
-			$img = new HtmlIcon('image-' . $this->identifier, $icon);
+		if ($icon!=='') {
+			$img=new HtmlIcon('image-'.$this->identifier, $icon);
 			$content->addContent($img, true);
 			$content->addToProperty('class', 'image');
 		}
@@ -139,15 +139,15 @@ 
 	}
 
 	public function showDimmer($value) {
-		$value = $value ? 'show' : 'hide';
-		$this->_paramParts[] = [
-			"'" . $value . " dimmer'"
+		$value=$value ? 'show' : 'hide';
+		$this->_paramParts[]=[
+			"'".$value." dimmer'"
 		];
 		return $this;
 	}
 
-	public function setInverted($recursive = true) {
-		$this->_params['inverted'] = true;
+	public function setInverted($recursive=true) {
+		$this->_params['inverted']=true;
 		return $this;
 	}
 
@@ -156,10 +156,10 @@ 
 	}
 
 	public function setTransition($value) {
-		$this->_paramParts[] = [
+		$this->_paramParts[]=[
 			"'setting'",
 			"'transition'",
-			"'" . $value . "'"
+			"'".$value."'"
 		];
 	}
 
@@ -172,7 +172,7 @@ 
 	 * @param array $params
 	 *        	The parameters to pass to the view
 	 */
-	public function renderView(JsUtils $js, $initialController, $viewName, $params = array()) {
+	public function renderView(JsUtils $js, $initialController, $viewName, $params=array()) {
 		return $this->setContent($js->renderContent($initialController, $viewName, $params));
 	}
 
@@ -187,7 +187,7 @@ 
 	 *        	the action name
 	 * @param array $params
 	 */
-	public function forward(JsUtils $js, $initialControllerInstance, $controllerName, $actionName, $params = NULL) {
+	public function forward(JsUtils $js, $initialControllerInstance, $controllerName, $actionName, $params=NULL) {
 		return $this->setContent($js->forward($initialControllerInstance, $controllerName, $actionName, $params));
 	}
 
@@ -197,8 +197,8 @@ 
 	 *
 	 * @see \Ajax\semantic\html\base\HtmlSemDoubleElement::compile()
 	 */
-	public function compile(JsUtils $js = NULL, &$view = NULL) {
-		$this->content = JArray::sortAssociative($this->content, [
+	public function compile(JsUtils $js=NULL, &$view=NULL) {
+		$this->content=JArray::sortAssociative($this->content, [
 			'header',
 			'content',
 			'actions'
@@ -214,14 +214,14 @@ 
 	 * @see BaseHtml::run()
 	 */
 	public function run(JsUtils $js) {
-		if (isset($this->_bsComponent) === false)
-			$this->_bsComponent = $js->semantic()->modal('#' . $this->identifier, $this->_params, $this->_paramParts);
+		if (isset($this->_bsComponent)===false)
+			$this->_bsComponent=$js->semantic()->modal('#'.$this->identifier, $this->_params, $this->_paramParts);
 		$this->addEventsOnRun($js);
 		return $this->_bsComponent;
 	}
 
 	public function jsDo($behavior) {
-		return "$('#" . $this->identifier . "').modal('" . $behavior . "');";
+		return "$('#".$this->identifier."').modal('".$behavior."');";
 	}
 
 	public function jsHide() {
@@ -229,6 +229,6 @@ 
 	}
 
 	public function onHidden($js) {
-		$this->_params['onHidden'] = $js;
+		$this->_params['onHidden']=$js;
 	}
 }

Braces +6 added lines, -4 removed lines

@@ -85,8 +85,9 @@ 
 				$class = 'cancel';
 			}
 			$action = new HtmlButton('action-' . $this->identifier . '-' . JArray::count($this->content['actions']->getContent()), $action);
-			if ($class !== '')
-				$action->addToProperty('class', $class);
+			if ($class !== '') {
+							$action->addToProperty('class', $class);
+			}
 		}
 		return $this->addElementInPart($action, 'actions');
 	}
@@ -214,8 +215,9 @@ 
 	 * @see BaseHtml::run()
 	 */
 	public function run(JsUtils $js) {
-		if (isset($this->_bsComponent) === false)
-			$this->_bsComponent = $js->semantic()->modal('#' . $this->identifier, $this->_params, $this->_paramParts);
+		if (isset($this->_bsComponent) === false) {
+					$this->_bsComponent = $js->semantic()->modal('#' . $this->identifier, $this->_params, $this->_paramParts);
+		}
 		$this->addEventsOnRun($js);
 		return $this->_bsComponent;
 	}