phpMv / phpMv-UI

Ajax/semantic/widgets/dataform/FormFieldAsTrait.php

Doc Comments +14 added lines

@@ -18,7 +18,14 @@ 
 
 trait FormFieldAsTrait{
 
+	/**
+	 * @param string $prefix
+	 */
 	abstract protected function _getFieldIdentifier($prefix);
+
+	/**
+	 * @param \Closure $callback
+	 */
 	abstract public function setValueFunction($index,$callback);
 
 	private function _getLabelField($caption,$icon=NULL){
@@ -42,6 +49,9 @@ 
 		$element->fromArray($attributes);
 	}
 
+	/**
+	 * @param HtmlFormField $element
+	 */
 	protected function _addRules($element,$attributes){
 		if(isset($attributes["rules"])){
 			$rules=$attributes["rules"];
@@ -53,6 +63,10 @@ 
 		}
 	}
 
+	/**
+	 * @param \Closure $elementCallback
+	 * @param string $identifier
+	 */
 	protected function _fieldAs($elementCallback,$index,$attributes=NULL,$identifier=null){
 		$this->setValueFunction($index,function($value) use ($index,&$attributes,$elementCallback){
 			$caption=$this->_instanceViewer->getCaption($index);

Spacing +44 added lines, -44 removed lines

@@ -16,13 +16,13 @@ 
  * @property object $_modelInstance;
  */
 
-trait FormFieldAsTrait{
+trait FormFieldAsTrait {
 
 	abstract protected function _getFieldIdentifier($prefix);
-	abstract public function setValueFunction($index,$callback);
+	abstract public function setValueFunction($index, $callback);
 
-	private function _getLabelField($caption,$icon=NULL){
-		$label=new HtmlLabel($this->_getFieldIdentifier("lbl"),$caption,$icon);
+	private function _getLabelField($caption, $icon=NULL) {
+		$label=new HtmlLabel($this->_getFieldIdentifier("lbl"), $caption, $icon);
 		return $label;
 	}
 
@@ -30,22 +30,22 @@ 
 	 * @param HtmlFormField $element
 	 * @param array $attributes
 	 */
-	protected function _applyAttributes($element,&$attributes,$index){
+	protected function _applyAttributes($element, &$attributes, $index) {
 		$this->_addRules($element, $attributes);
-		if(isset($attributes["callback"])){
+		if (isset($attributes["callback"])) {
 			$callback=$attributes["callback"];
-			if(\is_callable($callback)){
-				$callback($element,$this->_modelInstance,$index);
+			if (\is_callable($callback)) {
+				$callback($element, $this->_modelInstance, $index);
 				unset($attributes["callback"]);
 			}
 		}
 		$element->fromArray($attributes);
 	}
 
-	protected function _addRules($element,&$attributes){
-		if(isset($attributes["rules"])){
+	protected function _addRules($element, &$attributes) {
+		if (isset($attributes["rules"])) {
 			$rules=$attributes["rules"];
-			if(\is_array($rules)){
+			if (\is_array($rules)) {
 				$element->addRules($rules);
 			}
 			else
@@ -54,13 +54,13 @@ 
 		}
 	}
 
-	protected function _fieldAs($elementCallback,$index,$attributes=NULL,$identifier=null){
-		$this->setValueFunction($index,function($value) use ($index,&$attributes,$elementCallback){
+	protected function _fieldAs($elementCallback, $index, $attributes=NULL, $identifier=null) {
+		$this->setValueFunction($index, function($value) use ($index, &$attributes, $elementCallback){
 			$caption=$this->_instanceViewer->getCaption($index);
 			$name=$this->_instanceViewer->getFieldName($index);
-			$element=$elementCallback($this->getIdentifier()."-".$name,$name,$value,$caption);
-			if(\is_array($attributes)){
-				$this->_applyAttributes($element, $attributes,$index);
+			$element=$elementCallback($this->getIdentifier()."-".$name, $name, $value, $caption);
+			if (\is_array($attributes)) {
+				$this->_applyAttributes($element, $attributes, $index);
 			}
 			return $element;
 		});
@@ -68,53 +68,53 @@ 
 	}
 
 
-	public function fieldAsRadio($index,$attributes=NULL){
-		return $this->_fieldAs(function($id,$name,$value,$caption){
-			return new HtmlFormRadio($id,$name,$caption,$value);
-		}, $index,$attributes);
+	public function fieldAsRadio($index, $attributes=NULL) {
+		return $this->_fieldAs(function($id, $name, $value, $caption) {
+			return new HtmlFormRadio($id, $name, $caption, $value);
+		}, $index, $attributes);
 	}
 
-	public function fieldAsRadios($index,$elements=[],$attributes=NULL){
-		return $this->_fieldAs(function($id,$name,$value,$caption,$elements){
-			return HtmlFormFields::radios($name,$elements,$caption,$value);
-		}, $index,$attributes);
+	public function fieldAsRadios($index, $elements=[], $attributes=NULL) {
+		return $this->_fieldAs(function($id, $name, $value, $caption, $elements) {
+			return HtmlFormFields::radios($name, $elements, $caption, $value);
+		}, $index, $attributes);
 	}
 
-	public function fieldAsTextarea($index,$attributes=NULL){
-		return $this->_fieldAs(function($id,$name,$value,$caption){
-			$textarea=new HtmlFormTextarea($id,$caption,$value);
+	public function fieldAsTextarea($index, $attributes=NULL) {
+		return $this->_fieldAs(function($id, $name, $value, $caption) {
+			$textarea=new HtmlFormTextarea($id, $caption, $value);
 			$textarea->setName($name);
 			return $textarea;
-		}, $index,$attributes);
+		}, $index, $attributes);
 	}
 
-	public function fieldAsInput($index,$attributes=NULL){
-		return $this->_fieldAs(function($id,$name,$value,$caption){
-			$input= new HtmlFormInput($id,$caption,"text",$value);
+	public function fieldAsInput($index, $attributes=NULL) {
+		return $this->_fieldAs(function($id, $name, $value, $caption) {
+			$input=new HtmlFormInput($id, $caption, "text", $value);
 			$input->setName($name);
 			return $input;
-		}, $index,$attributes);
+		}, $index, $attributes);
 	}
 
-	public function fieldAsCheckbox($index,$attributes=NULL){
-		return $this->_fieldAs(function($id,$name,$value,$caption){
-			return new HtmlFormCheckbox($id,$caption,$value);
-		}, $index,$attributes);
+	public function fieldAsCheckbox($index, $attributes=NULL) {
+		return $this->_fieldAs(function($id, $name, $value, $caption) {
+			return new HtmlFormCheckbox($id, $caption, $value);
+		}, $index, $attributes);
 	}
 
-	public function fieldAsDropDown($index,$elements=[],$multiple=false,$attributes=NULL){
-		return $this->_fieldAs(function($id,$name,$value,$caption) use ($elements,$multiple){
-			$dd=new HtmlFormDropdown($id,$elements,$caption,$value,$multiple);
+	public function fieldAsDropDown($index, $elements=[], $multiple=false, $attributes=NULL) {
+		return $this->_fieldAs(function($id, $name, $value, $caption) use ($elements, $multiple){
+			$dd=new HtmlFormDropdown($id, $elements, $caption, $value, $multiple);
 			$dd->setName($name);
 			return $dd;
-		}, $index,$attributes);
+		}, $index, $attributes);
 	}
 
-	public function fieldAsMessage($index,$attributes=NULL){
-		return $this->_fieldAs(function($id,$name,$value,$caption){
-			$mess= new HtmlMessage("message-".$id,$value);
+	public function fieldAsMessage($index, $attributes=NULL) {
+		return $this->_fieldAs(function($id, $name, $value, $caption) {
+			$mess=new HtmlMessage("message-".$id, $value);
 			$mess->addHeader($caption);
 			return $mess;
-		}, $index,$attributes,"message");
+		}, $index, $attributes, "message");
 	}
 }
\ No newline at end of file

Braces +2 added lines, -2 removed lines

@@ -47,9 +47,9 @@
 			$rules=$attributes["rules"];
 			if(\is_array($rules)){
 				$element->addRules($rules);
+			} else {
+							$element->addRule($rules);
 			}
-			else
-				$element->addRule($rules);
 			unset($attributes["rules"]);
 		}
 	}

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

Spacing +8 added lines, -8 removed lines

@@ -7,21 +7,21 @@
 
 class HtmlFormDropdown extends HtmlFormField {
 
-	public function __construct($identifier,$items=array(), $label=NULL,$value=NULL,$multiple=false) {
-		parent::__construct("field-".$identifier, (new HtmlDropdown("dropdown-".$identifier,$value,$items))->asSelect($identifier,$multiple), $label);
+	public function __construct($identifier, $items=array(), $label=NULL, $value=NULL, $multiple=false) {
+		parent::__construct("field-".$identifier, (new HtmlDropdown("dropdown-".$identifier, $value, $items))->asSelect($identifier, $multiple), $label);
 	}
 
-	public function setItems($items){
+	public function setItems($items) {
 		return $this->getField()->setItems($items);
 	}
-	public function addItem($item,$value=NULL,$image=NULL){
-		return $this->getField()->addItem($item,$value,$image);
+	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=NULL){
-		return new HtmlFormDropdown($identifier,$items,$label,$value,true);
+	public static function multipleDropdown($identifier, $items=array(), $label=NULL, $value=NULL) {
+		return new HtmlFormDropdown($identifier, $items, $label, $value, true);
 	}
 
-	public function getDataField(){
+	public function getDataField() {
 		return $this->getField()->getInput();
 	}
 }
\ No newline at end of file

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

Spacing +6 added lines, -6 removed lines

@@ -8,16 +8,16 @@
 class HtmlFormInput extends HtmlFormField {
 	use TextFieldsTrait;
 
-	public function __construct($identifier, $label=NULL,$type="text",$value=NULL,$placeholder=NULL) {
-		if(!isset($placeholder) && $type==="text")
+	public function __construct($identifier, $label=NULL, $type="text", $value=NULL, $placeholder=NULL) {
+		if (!isset($placeholder) && $type==="text")
 			$placeholder=$label;
-		parent::__construct("field-".$identifier, new HtmlInput($identifier,$type,$value,$placeholder), $label);
+		parent::__construct("field-".$identifier, new HtmlInput($identifier, $type, $value, $placeholder), $label);
 	}
 
-	public function getDataField(){
-		$field= $this->getField();
+	public function getDataField() {
+		$field=$this->getField();
 		//TODO check getField
-		if($field instanceof HtmlInput)
+		if ($field instanceof HtmlInput)
 			$field=$field->getDataField();
 		return $field;
 	}

Braces +6 added lines, -4 removed lines

@@ -9,16 +9,18 @@
 	use TextFieldsTrait;
 
 	public function __construct($identifier, $label=NULL,$type="text",$value=NULL,$placeholder=NULL) {
-		if(!isset($placeholder) && $type==="text")
-			$placeholder=$label;
+		if(!isset($placeholder) && $type==="text") {
+					$placeholder=$label;
+		}
 		parent::__construct("field-".$identifier, new HtmlInput($identifier,$type,$value,$placeholder), $label);
 	}
 
 	public function getDataField(){
 		$field= $this->getField();
 		//TODO check getField
-		if($field instanceof HtmlInput)
-			$field=$field->getDataField();
+		if($field instanceof HtmlInput) {
+					$field=$field->getDataField();
+		}
 		return $field;
 	}
 }
\ No newline at end of file

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

Spacing +5 added lines, -5 removed lines

@@ -6,19 +6,19 @@
 
 	abstract public function getDataField();
 	abstract public function addToProperty($name, $value, $separator=" ");
-	public function setPlaceholder($value){
+	public function setPlaceholder($value) {
 		$this->getDataField()->setPlaceholder($value);
 		return $this;
 	}
 
-	public function setValue($value){
+	public function setValue($value) {
 		$this->getDataField()->setValue($value);
 		return $this;
 	}
 
-	public function setInputType($type){
-		if($type==="hidden")
-			$this->addToProperty("style","display:none;");
+	public function setInputType($type) {
+		if ($type==="hidden")
+			$this->addToProperty("style", "display:none;");
 		$this->getDataField()->setInputType($type);
 		return $this;
 	}

Braces +3 added lines, -2 removed lines

@@ -17,8 +17,9 @@
 	}
 
 	public function setInputType($type){
-		if($type==="hidden")
-			$this->addToProperty("style","display:none;");
+		if($type==="hidden") {
+					$this->addToProperty("style","display:none;");
+		}
 		$this->getDataField()->setInputType($type);
 		return $this;
 	}

Ajax/semantic/html/collections/HtmlMessage.php

Spacing +23 added lines, -23 removed lines

@@ -31,38 +31,38 @@ 
 	 * @param string|HtmlSemDoubleElement $header
 	 * @return \Ajax\semantic\html\collections\HtmlMessage
 	 */
-	public function addHeader($header){
+	public function addHeader($header) {
 		$headerO=$header;
-		if(\is_string($header)){
-			$headerO=new HtmlSemDoubleElement("header-".$this->identifier,"div");
+		if (\is_string($header)) {
+			$headerO=new HtmlSemDoubleElement("header-".$this->identifier, "div");
 			$headerO->setClass("header");
 			$headerO->setContent($header);
 		}
-		return $this->addContent($headerO,true);
+		return $this->addContent($headerO, true);
 	}
 
-	public function addList($elements,$ordered=false){
-		$list=new HtmlList("list-".$this->identifier,$elements);
+	public function addList($elements, $ordered=false) {
+		$list=new HtmlList("list-".$this->identifier, $elements);
 		$list->setOrdered($ordered);
 		$list->setClass("ui list");
 		$this->addContent($list);
 	}
 
-	public function setIcon($icon){
+	public function setIcon($icon) {
 		$this->addToProperty("class", "icon");
-		$this->wrapContent("<div class='content'>","</div>");
+		$this->wrapContent("<div class='content'>", "</div>");
 		$this->icon=new HtmlIcon("icon-".$this->identifier, $icon);
 		return $this;
 	}
 
-	public function addLoader($loaderIcon="notched circle"){
+	public function addLoader($loaderIcon="notched circle") {
 		$this->setIcon($loaderIcon);
 		$this->icon->addToIcon("loading");
 		return $this;
 	}
 
-	public function setDismissable($dismiss=true){
-		if($dismiss===true)
+	public function setDismissable($dismiss=true) {
+		if ($dismiss===true)
 			$this->close=new HtmlIcon("close-".$this->identifier, "close");
 		else
 			$this->close=NULL;
@@ -73,34 +73,34 @@ 
 	 * {@inheritDoc}
 	 * @see \Ajax\semantic\html\base\HtmlSemDoubleElement::run()
 	 */
-	public function run(JsUtils $js){
+	public function run(JsUtils $js) {
 		parent::run($js);
-		if(isset($this->close)){
+		if (isset($this->close)) {
 			$js->execOn("click", "#".$this->identifier." .close", "$(this).closest('.message').transition('fade')");
 		}
 	}
 
-	public function setState($visible=true){
-		$visible=($visible===true)?"visible":"hidden";
-		return $this->addToPropertyCtrl("class", $visible, array("visible","hidden"));
+	public function setState($visible=true) {
+		$visible=($visible===true) ? "visible" : "hidden";
+		return $this->addToPropertyCtrl("class", $visible, array("visible", "hidden"));
 	}
 
-	public function setVariation($value="floating"){
-		return $this->addToPropertyCtrl("class", $value, array("floating","compact"));
+	public function setVariation($value="floating") {
+		return $this->addToPropertyCtrl("class", $value, array("floating", "compact"));
 	}
 
-	public function setStyle($style){
+	public function setStyle($style) {
 		return $this->addToPropertyCtrl("class", $style, Style::getConstants());
 	}
 
-	public function setError(){
+	public function setError() {
 		return $this->setStyle("error");
 	}
 
-	public function setMessage($message){
-		if(\is_array($this->content)){
+	public function setMessage($message) {
+		if (\is_array($this->content)) {
 			$this->content[\sizeof($this->content)-1]=$message;
-		}else
+		} else
 			$this->setContent($message);
 	}
 }
\ No newline at end of file

Braces +8 added lines, -6 removed lines

@@ -62,10 +62,11 @@ 
 	}
 
 	public function setDismissable($dismiss=true){
-		if($dismiss===true)
-			$this->close=new HtmlIcon("close-".$this->identifier, "close");
-		else
-			$this->close=NULL;
+		if($dismiss===true) {
+					$this->close=new HtmlIcon("close-".$this->identifier, "close");
+		} else {
+					$this->close=NULL;
+		}
 		return $this;
 	}
 
@@ -100,7 +101,8 @@ 
 	public function setMessage($message){
 		if(\is_array($this->content)){
 			$this->content[\sizeof($this->content)-1]=$message;
-		}else
-			$this->setContent($message);
+		} else {
+					$this->setContent($message);
+		}
 	}
 }
\ No newline at end of file

Ajax/semantic/widgets/base/InstanceViewer.php

Braces +35 added lines, -30 removed lines

@@ -17,8 +17,9 @@ 
 	public function __construct($instance=NULL,$captions=NULL){
 		$this->values=[];
 		$this->afterCompile=[];
-		if(isset($instance))
-			$this->setInstance($instance);
+		if(isset($instance)) {
+					$this->setInstance($instance);
+		}
 		$this->setCaptions($captions);
 	}
 
@@ -34,8 +35,9 @@ 
 
 	public function getIdentifier(){
 		$value=self::$index;
-		if(isset($this->values["identifier"]))
-			$value=$this->values["identifier"](self::$index,$this->instance);
+		if(isset($this->values["identifier"])) {
+					$value=$this->values["identifier"](self::$index,$this->instance);
+		}
 		return $value;
 	}
 
@@ -58,19 +60,19 @@ 
 			$value=$property->getValue($this->instance);
 			if(isset($this->values[$index])){
 				$value= $this->values[$index]($value,$this->instance,$index);
-			}else{
+			} else{
 				$value=$this->_getDefaultValue($property->getName(),$value, $index);
 			}
-		}else{
-			if(\is_callable($property))
-				$value=$property($this->instance);
-			elseif(\is_array($property)){
+		} else{
+			if(\is_callable($property)) {
+							$value=$property($this->instance);
+			} elseif(\is_array($property)){
 				$values=\array_map(function($v) use ($index){return $this->_getValue($v, $index);}, $property);
 				$value=\implode("", $values);
-			}else{
+			} else{
 				if(isset($this->values[$index])){
 					$value= $this->values[$index]($property,$this->instance,$index);
-				}else{
+				} else{
 					$value=$property;
 				}
 			}
@@ -93,10 +95,10 @@ 
 		if(isset($vb[$index])){
 			if(\is_array($vb[$index])){
 				$this->visibleProperties[$index][]=$field;
-			}else{
+			} else{
 				$this->visibleProperties[$index]=[$vb[$index],$field];
 			}
-		}else{
+		} else{
 			return $this->insertField($index, $field);
 		}
 		return $this;
@@ -132,26 +134,27 @@ 
 		$this->reflect=new \ReflectionClass($instance);
 		if(\sizeof($this->visibleProperties)===0){
 			$this->properties=$this->getDefaultProperties();
-		}else{
+		} else{
 			foreach ($this->visibleProperties as $property){
 				if(\is_callable($property)){
 					$this->properties[]=$property;
-				}elseif(\is_string($property)){
+				} elseif(\is_string($property)){
 					try{
 						$this->_beforeAddProperty(\sizeof($this->properties), $property);
 						$rProperty=$this->reflect->getProperty($property);
 						$this->properties[]=$rProperty;
-					}catch(\Exception $e){
+					} catch(\Exception $e){
 						$this->_beforeAddProperty(\sizeof($this->properties), $property);
 						$this->properties[]=$property;
 					}
-				}elseif(\is_int($property)){
+				} elseif(\is_int($property)){
 					$props=$this->getDefaultProperties();
-					if(isset($props[$property]))
-						$this->properties[]=$props[$property];
-					else
-						$this->properties[]=$property;
-				}else{
+					if(isset($props[$property])) {
+											$this->properties[]=$props[$property];
+					} else {
+											$this->properties[]=$property;
+					}
+				} else{
 					$this->properties[]=$property;
 				}
 			}
@@ -198,12 +201,13 @@ 
 		if(isset($this->captions[$index])){
 			return $this->captions[$index];
 		}
-		if($this->properties[$index] instanceof \ReflectionProperty)
-			return $this->properties[$index]->getName();
-		elseif(\is_callable($this->properties[$index]))
-			return "";
-		else
-			return $this->properties[$index];
+		if($this->properties[$index] instanceof \ReflectionProperty) {
+					return $this->properties[$index]->getName();
+		} elseif(\is_callable($this->properties[$index])) {
+					return "";
+		} else {
+					return $this->properties[$index];
+		}
 	}
 
 	public function getCaptions(){
@@ -224,8 +228,9 @@ 
 	}
 
 	public function setCaption($index,$caption){
-		if(isset($this->captions)===false)
-			$this->captions=[];
+		if(isset($this->captions)===false) {
+					$this->captions=[];
+		}
 		$this->captions[$index]=$caption;
 		return $this;
 	}

Spacing +69 added lines, -69 removed lines

@@ -15,145 +15,145 @@ 
 
 	public static $index=0;
 
-	public function __construct($identifier,$instance=NULL,$captions=NULL){
+	public function __construct($identifier, $instance=NULL, $captions=NULL) {
 		$this->widgetIdentifier=$identifier;
 		$this->values=[];
 		$this->afterCompile=[];
-		if(isset($instance))
+		if (isset($instance))
 			$this->setInstance($instance);
 		$this->setCaptions($captions);
 	}
 
-	public function getValues(){
+	public function getValues() {
 		$values=[];
 		$index=0;
 		$count=$this->count();
-		while($index<$count){
+		while ($index<$count) {
 			$values[]=$this->getValue($index++);
 		}
 		return $values;
 	}
 
-	public function getIdentifier(){
+	public function getIdentifier() {
 		$value=self::$index;
-		if(isset($this->values["identifier"]))
-			$value=$this->values["identifier"](self::$index,$this->instance);
+		if (isset($this->values["identifier"]))
+			$value=$this->values["identifier"](self::$index, $this->instance);
 		return $value;
 	}
 
-	public function getValue($index){
+	public function getValue($index) {
 		$property=$this->properties[$index];
 		return $this->_getValue($property, $index);
 	}
 
-	protected function _beforeAddProperty($index,&$field){
+	protected function _beforeAddProperty($index, &$field) {
 
 	}
 
-	protected function _getDefaultValue($name,$value,$index){
+	protected function _getDefaultValue($name, $value, $index) {
 		return $value;
 	}
 
-	protected function _getValue($property,$index){
-		if($property instanceof \ReflectionProperty){
+	protected function _getValue($property, $index) {
+		if ($property instanceof \ReflectionProperty) {
 			$property->setAccessible(true);
 			$value=$property->getValue($this->instance);
-			if(isset($this->values[$index])){
-				$value= $this->values[$index]($value,$this->instance,$index);
-			}else{
-				$value=$this->_getDefaultValue($property->getName(),$value, $index);
+			if (isset($this->values[$index])) {
+				$value=$this->values[$index]($value, $this->instance, $index);
+			} else {
+				$value=$this->_getDefaultValue($property->getName(), $value, $index);
 			}
-		}else{
-			if(\is_callable($property))
+		} else {
+			if (\is_callable($property))
 				$value=$property($this->instance);
-			elseif(\is_array($property)){
-				$values=\array_map(function($v) use ($index){return $this->_getValue($v, $index);}, $property);
+			elseif (\is_array($property)) {
+				$values=\array_map(function($v) use ($index){return $this->_getValue($v, $index); }, $property);
 				$value=\implode("", $values);
-			}else{
-				if(isset($this->values[$index])){
-					$value= $this->values[$index]($property,$this->instance,$index);
-				}else{
+			} else {
+				if (isset($this->values[$index])) {
+					$value=$this->values[$index]($property, $this->instance, $index);
+				} else {
 					$value=$property;
 				}
 			}
 		}
-		if(isset($this->afterCompile[$index])){
-			if(\is_callable($this->afterCompile[$index])){
-				$this->afterCompile[$index]($value,$this->instance,$index);
+		if (isset($this->afterCompile[$index])) {
+			if (\is_callable($this->afterCompile[$index])) {
+				$this->afterCompile[$index]($value, $this->instance, $index);
 			}
 		}
 		return $value;
 	}
 
-	public function insertField($index,$field){
-		array_splice( $this->visibleProperties, $index, 0, $field );
+	public function insertField($index, $field) {
+		array_splice($this->visibleProperties, $index, 0, $field);
 		return $this;
 	}
 
-	public function insertInField($index,$field){
+	public function insertInField($index, $field) {
 		$vb=$this->visibleProperties;
-		if(isset($vb[$index])){
-			if(\is_array($vb[$index])){
+		if (isset($vb[$index])) {
+			if (\is_array($vb[$index])) {
 				$this->visibleProperties[$index][]=$field;
-			}else{
-				$this->visibleProperties[$index]=[$vb[$index],$field];
+			} else {
+				$this->visibleProperties[$index]=[$vb[$index], $field];
 			}
-		}else{
+		} else {
 			return $this->insertField($index, $field);
 		}
 		return $this;
 	}
 
-	public function addField($field){
+	public function addField($field) {
 		$this->visibleProperties[]=$field;
 		return $this;
 	}
 
-	public function count(){
+	public function count() {
 		return \sizeof($this->properties);
 	}
 
-	public function visiblePropertiesCount(){
+	public function visiblePropertiesCount() {
 		return \sizeof($this->visibleProperties);
 	}
 
-	public function getProperty($index){
+	public function getProperty($index) {
 		return $this->properties[$index];
 	}
 
-	protected function showableProperty(\ReflectionProperty $rProperty){
-		return JString::startswith($rProperty->getName(),"_")===false;
+	protected function showableProperty(\ReflectionProperty $rProperty) {
+		return JString::startswith($rProperty->getName(), "_")===false;
 	}
 
 	public function setInstance($instance) {
-		if(\is_string($instance)){
+		if (\is_string($instance)) {
 			$instance=new $instance();
 		}
 		$this->instance=$instance;
 		$this->properties=[];
 		$this->reflect=new \ReflectionClass($instance);
-		if(\sizeof($this->visibleProperties)===0){
+		if (\sizeof($this->visibleProperties)===0) {
 			$this->properties=$this->getDefaultProperties();
-		}else{
-			foreach ($this->visibleProperties as $property){
-				if(\is_callable($property)){
+		} else {
+			foreach ($this->visibleProperties as $property) {
+				if (\is_callable($property)) {
 					$this->properties[]=$property;
-				}elseif(\is_string($property)){
-					try{
+				}elseif (\is_string($property)) {
+					try {
 						$this->_beforeAddProperty(\sizeof($this->properties), $property);
 						$rProperty=$this->reflect->getProperty($property);
 						$this->properties[]=$rProperty;
-					}catch(\Exception $e){
+					}catch (\Exception $e) {
 						$this->_beforeAddProperty(\sizeof($this->properties), $property);
 						$this->properties[]=$property;
 					}
-				}elseif(\is_int($property)){
+				}elseif (\is_int($property)) {
 					$props=$this->getDefaultProperties();
-					if(isset($props[$property]))
+					if (isset($props[$property]))
 						$this->properties[]=$props[$property];
 					else
 						$this->properties[]=$property;
-				}else{
+				} else {
 					$this->properties[]=$property;
 				}
 			}
@@ -161,12 +161,12 @@ 
 		return $this;
 	}
 
-	protected function getDefaultProperties(){
+	protected function getDefaultProperties() {
 		$result=[];
 		$properties=$this->reflect->getProperties();
-		foreach ($properties as $property){
+		foreach ($properties as $property) {
 			$showable=$this->showableProperty($property);
-			if($showable!==false){
+			if ($showable!==false) {
 				$result[]=$property;
 			}
 		}
@@ -178,12 +178,12 @@ 
 		return $this;
 	}
 
-	public function setValueFunction($index,$callback){
+	public function setValueFunction($index, $callback) {
 		$this->values[$index]=$callback;
 		return $this;
 	}
 
-	public function setIdentifierFunction($callback){
+	public function setIdentifierFunction($callback) {
 		$this->values["identifier"]=$callback;
 		return $this;
 	}
@@ -196,22 +196,22 @@ 
 		return $this->properties;
 	}
 
-	public function getCaption($index){
-		if(isset($this->captions[$index])){
+	public function getCaption($index) {
+		if (isset($this->captions[$index])) {
 			return $this->captions[$index];
 		}
-		if($this->properties[$index] instanceof \ReflectionProperty)
+		if ($this->properties[$index] instanceof \ReflectionProperty)
 			return $this->properties[$index]->getName();
-		elseif(\is_callable($this->properties[$index]))
+		elseif (\is_callable($this->properties[$index]))
 			return "";
 		else
 			return $this->properties[$index];
 	}
 
-	public function getCaptions(){
-		if(isset($this->captions)){
-			$result= $this->captions;
-			for($i=\sizeof($result);$i<$this->count();$i++){
+	public function getCaptions() {
+		if (isset($this->captions)) {
+			$result=$this->captions;
+			for ($i=\sizeof($result); $i<$this->count(); $i++) {
 				$result[]="";
 			}
 			return $result;
@@ -219,14 +219,14 @@ 
 		$captions=[];
 		$index=0;
 		$count=$this->count();
-		while($index<$count){
+		while ($index<$count) {
 			$captions[]=$this->getCaption($index++);
 		}
 		return $captions;
 	}
 
-	public function setCaption($index,$caption){
-		if(isset($this->captions)===false)
+	public function setCaption($index, $caption) {
+		if (isset($this->captions)===false)
 			$this->captions=[];
 		$this->captions[$index]=$caption;
 		return $this;
@@ -244,7 +244,7 @@ 
 	 * @param callable $callback function called after the field compilation
 	 * @return \Ajax\semantic\widgets\datatable\InstanceViewer
 	 */
-	public function afterCompile($index,$callback){
+	public function afterCompile($index, $callback) {
 		$this->afterCompile[$index]=$callback;
 		return $this;
 	}

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

Spacing +27 added lines, -27 removed lines

@@ -20,7 +20,7 @@ 
  */
 class HtmlForm extends HtmlSemCollection {
 
-	use FieldsTrait,FormTrait;
+	use FieldsTrait, FormTrait;
 	/**
 	 * @var array
 	 */
@@ -33,14 +33,14 @@ 
 
 	public function __construct($identifier, $elements=array()) {
 		parent::__construct($identifier, "form", "ui form");
-		$this->_states=[ State::ERROR,State::SUCCESS,State::WARNING,State::DISABLED ];
+		$this->_states=[State::ERROR, State::SUCCESS, State::WARNING, State::DISABLED];
 		$this->setProperty("name", $this->identifier);
-		$this->_fields=array ();
+		$this->_fields=array();
 		$this->_validationParams=[];
 		$this->addItems($elements);
 	}
 
-	protected function getForm(){
+	protected function getForm() {
 		return $this;
 	}
 
@@ -61,14 +61,14 @@ 
 	 * @param string $caption
 	 * @return \Ajax\semantic\html\collections\form\HtmlForm
 	 */
-	public function addDivider($caption=NULL){
-		return $this->addContent(new HtmlDivider("",$caption));
+	public function addDivider($caption=NULL) {
+		return $this->addContent(new HtmlDivider("", $caption));
 	}
 
 	public function addFields($fields=NULL, $label=NULL) {
 		if (isset($fields)) {
 			if (!$fields instanceof HtmlFormFields) {
-				if (\is_array($fields) === false) {
+				if (\is_array($fields)===false) {
 					$fields=\func_get_args();
 					$end=\end($fields);
 					if (\is_string($end)) {
@@ -78,12 +78,12 @@ 
 						$label=NULL;
 				}
 				$this->_fields=\array_merge($this->_fields, $fields);
-				$fields=new HtmlFormFields("fields-" . $this->identifier . "-" . $this->count(), $fields);
+				$fields=new HtmlFormFields("fields-".$this->identifier."-".$this->count(), $fields);
 			}
 			if (isset($label))
 				$fields=new HtmlFormField("", $fields, $label);
 		} else {
-			$fields=new HtmlFormFields("fields-" . $this->identifier . "-" . $this->count());
+			$fields=new HtmlFormFields("fields-".$this->identifier."-".$this->count());
 		}
 		$this->addItem($fields);
 		return $fields;
@@ -91,7 +91,7 @@ 
 
 	public function addItem($item) {
 		$item=parent::addItem($item);
-		if (\is_subclass_of($item, HtmlFormField::class) === true) {
+		if (\is_subclass_of($item, HtmlFormField::class)===true) {
 			$this->_fields[]=$item;
 		}
 		return $item;
@@ -143,10 +143,10 @@ 
 		return $this->addItem($message);
 	}
 
-	private function addCompoValidation($js,$compo,$field){
+	private function addCompoValidation($js, $compo, $field) {
 		$validation=$field->getValidation();
-		if(isset($validation)){
-			if(isset($compo)===false){
+		if (isset($validation)) {
+			if (isset($compo)===false) {
 				$compo=$js->semantic()->form("#".$this->identifier);
 			}
 			$validation->setIdentifier($field->getDataField()->getIdentifier());
@@ -155,34 +155,34 @@ 
 		return $compo;
 	}
 
-	public function compile(JsUtils $js=NULL,&$view=NULL){
-		if(\sizeof($this->_validationParams)>0)
+	public function compile(JsUtils $js=NULL, &$view=NULL) {
+		if (\sizeof($this->_validationParams)>0)
 			$this->setProperty("novalidate", "");
-		return parent::compile($js,$view);
+		return parent::compile($js, $view);
 	}
 
 	public function run(JsUtils $js) {
 		$compo=NULL;
-		foreach ($this->_fields as $field){
-			if($field instanceof HtmlFormField)
+		foreach ($this->_fields as $field) {
+			if ($field instanceof HtmlFormField)
 				$compo=$this->addCompoValidation($js, $compo, $field);
 		}
-		foreach ($this->content as $field){
-			if($field instanceof HtmlFormFields){
+		foreach ($this->content as $field) {
+			if ($field instanceof HtmlFormFields) {
 				$items=$field->getItems();
-				foreach ($items as $_field){
-					if($_field instanceof HtmlFormField)
+				foreach ($items as $_field) {
+					if ($_field instanceof HtmlFormField)
 						$compo=$this->addCompoValidation($js, $compo, $_field);
 				}
 			}
 		}
-		if(isset($compo)===false){
+		if (isset($compo)===false) {
 			return parent::run($js);
 		}
-		if(isset($this->_validationParams["_ajaxSubmit"])){
-			if($this->_validationParams["_ajaxSubmit"] instanceof AjaxCall){
+		if (isset($this->_validationParams["_ajaxSubmit"])) {
+			if ($this->_validationParams["_ajaxSubmit"] instanceof AjaxCall) {
 				$compilation=$this->_validationParams["_ajaxSubmit"]->compile($js);
-				$compilation=str_ireplace("\"","%quote%", $compilation);
+				$compilation=str_ireplace("\"", "%quote%", $compilation);
 				$this->onSuccess($compilation);
 				unset($this->_validationParams["_ajaxSubmit"]);
 			}
@@ -193,7 +193,7 @@ 
 		return $this->_bsComponent;
 	}
 
-	public function addValidationParam($paramName,$paramValue){
+	public function addValidationParam($paramName, $paramValue) {
 		$this->_validationParams[$paramName]=$paramValue;
 		return $this;
 	}

Braces +27 added lines, -18 removed lines

@@ -52,8 +52,9 @@ 
 	 */
 	public function addHeader($title, $niveau=1, $dividing=true) {
 		$header=new HtmlHeader("", $niveau, $title);
-		if ($dividing)
-			$header->setDividing();
+		if ($dividing) {
+					$header->setDividing();
+		}
 		return $this->addItem($header);
 	}
 
@@ -74,14 +75,16 @@ 
 					if (\is_string($end)) {
 						$label=$end;
 						\array_pop($fields);
-					} else
-						$label=NULL;
+					} else {
+											$label=NULL;
+					}
 				}
 				$this->_fields=\array_merge($this->_fields, $fields);
 				$fields=new HtmlFormFields("fields-" . $this->identifier . "-" . $this->count(), $fields);
 			}
-			if (isset($label))
-				$fields=new HtmlFormField("", $fields, $label);
+			if (isset($label)) {
+							$fields=new HtmlFormField("", $fields, $label);
+			}
 		} else {
 			$fields=new HtmlFormFields("fields-" . $this->identifier . "-" . $this->count());
 		}
@@ -134,12 +137,15 @@ 
 	 */
 	public function addMessage($identifier, $content, $header=NULL, $icon=NULL, $type=NULL) {
 		$message=new HtmlMessage($identifier, $content);
-		if (isset($header))
-			$message->addHeader($header);
-		if (isset($icon))
-			$message->setIcon($icon);
-		if (isset($type))
-			$message->setStyle($type);
+		if (isset($header)) {
+					$message->addHeader($header);
+		}
+		if (isset($icon)) {
+					$message->setIcon($icon);
+		}
+		if (isset($type)) {
+					$message->setStyle($type);
+		}
 		return $this->addItem($message);
 	}
 
@@ -156,23 +162,26 @@ 
 	}
 
 	public function compile(JsUtils $js=NULL,&$view=NULL){
-		if(\sizeof($this->_validationParams)>0)
-			$this->setProperty("novalidate", "");
+		if(\sizeof($this->_validationParams)>0) {
+					$this->setProperty("novalidate", "");
+		}
 		return parent::compile($js,$view);
 	}
 
 	public function run(JsUtils $js) {
 		$compo=NULL;
 		foreach ($this->_fields as $field){
-			if($field instanceof HtmlFormField)
-				$compo=$this->addCompoValidation($js, $compo, $field);
+			if($field instanceof HtmlFormField) {
+							$compo=$this->addCompoValidation($js, $compo, $field);
+			}
 		}
 		foreach ($this->content as $field){
 			if($field instanceof HtmlFormFields){
 				$items=$field->getItems();
 				foreach ($items as $_field){
-					if($_field instanceof HtmlFormField)
-						$compo=$this->addCompoValidation($js, $compo, $_field);
+					if($_field instanceof HtmlFormField) {
+											$compo=$this->addCompoValidation($js, $compo, $_field);
+					}
 				}
 			}
 		}