Completed
Push — master ( 235574...667888 )
by Jean-Christophe
06:30 queued 03:14
created
Ajax/semantic/widgets/dataform/DataForm.php 3 patches
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -13,6 +13,9 @@
 block discarded – undo
13 13
  */
14 14
 class HtmlIconGroups extends HtmlSemCollection {
15 15
 
16
+	/**
17
+	 * @param string $identifier
18
+	 */
16 19
 	public function __construct($identifier, $icons=array(), $size="") {
17 20
 		parent::__construct($identifier, "i", "icons");
18 21
 		$this->addItems($icons);
Please login to merge, or discard this patch.
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -18,43 +18,43 @@  discard block
 block discarded – undo
18 18
 	use FormFieldAsTrait;
19 19
 
20 20
 	public function __construct($identifier, $modelInstance=NULL) {
21
-		parent::__construct($identifier, null,$modelInstance);
21
+		parent::__construct($identifier, null, $modelInstance);
22 22
 		$this->_instanceViewer=new FormInstanceViewer();
23 23
 		$this->content=["form"=>new HtmlForm($identifier)];
24 24
 		$this->_toolbarPosition=PositionInTable::BEFORETABLE;
25 25
 	}
26 26
 
27
-	public function compile(JsUtils $js=NULL,&$view=NULL){
27
+	public function compile(JsUtils $js=NULL, &$view=NULL) {
28 28
 		$this->_instanceViewer->setInstance($this->_modelInstance);
29 29
 
30 30
 		$form=$this->content["form"];
31 31
 		$this->_generateContent($form);
32 32
 
33
-		if(isset($this->_toolbar)){
33
+		if (isset($this->_toolbar)) {
34 34
 			$this->_setToolbarPosition($form);
35 35
 		}
36
-		$this->content=JArray::sortAssociative($this->content, [PositionInTable::BEFORETABLE,"form",PositionInTable::AFTERTABLE]);
37
-		return parent::compile($js,$view);
36
+		$this->content=JArray::sortAssociative($this->content, [PositionInTable::BEFORETABLE, "form", PositionInTable::AFTERTABLE]);
37
+		return parent::compile($js, $view);
38 38
 	}
39 39
 
40 40
 	/**
41 41
 	 * @param HtmlForm $form
42 42
 	 */
43
-	protected function _generateContent($form){
44
-		$values= $this->_instanceViewer->getValues();
43
+	protected function _generateContent($form) {
44
+		$values=$this->_instanceViewer->getValues();
45 45
 		$count=$this->_instanceViewer->count();
46 46
 		$separators=$this->_instanceViewer->getSeparators();
47 47
 		$separators[]=$count;
48
-		for($i=0;$i<\sizeof($separators)-1;$i++){
49
-			$fields=\array_slice($values, $separators[$i]+1,$separators[$i+1]-$separators[$i]);
50
-			if(\sizeof($fields)===1){
48
+		for ($i=0; $i<\sizeof($separators)-1; $i++) {
49
+			$fields=\array_slice($values, $separators[$i]+1, $separators[$i+1]-$separators[$i]);
50
+			if (\sizeof($fields)===1) {
51 51
 				$form->addField($fields[0]);
52
-			}else
52
+			} else
53 53
 				$form->addFields($fields);
54 54
 		}
55 55
 	}
56 56
 
57
-	public function addSeparatorAfter($fieldNum){
57
+	public function addSeparatorAfter($fieldNum) {
58 58
 		$this->_instanceViewer->addSeparatorAfter($fieldNum);
59 59
 		return $this;
60 60
 	}
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
 		$this->content[$this->_toolbarPosition]=$this->_toolbar;
85 85
 	}
86 86
 
87
-	public function setValidationParams(array $_validationParams){
87
+	public function setValidationParams(array $_validationParams) {
88 88
 		return $this->getHtmlComponent()->setValidationParams($_validationParams);
89 89
 	}
90 90
 }
91 91
\ No newline at end of file
Please login to merge, or discard this patch.
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -49,8 +49,9 @@
 block discarded – undo
49 49
 			$fields=\array_slice($values, $separators[$i]+1,$separators[$i+1]-$separators[$i]);
50 50
 			if(\sizeof($fields)===1){
51 51
 				$form->addField($fields[0]);
52
-			}else
53
-				$form->addFields($fields);
52
+			} else {
53
+							$form->addFields($fields);
54
+			}
54 55
 		}
55 56
 	}
56 57
 
Please login to merge, or discard this patch.
Ajax/semantic/widgets/dataform/FormFieldAsTrait.php 3 patches
Doc Comments   +10 added lines patch added patch discarded remove patch
@@ -16,7 +16,14 @@  discard block
 block discarded – undo
16 16
 
17 17
 trait FormFieldAsTrait{
18 18
 
19
+	/**
20
+	 * @param string $prefix
21
+	 */
19 22
 	abstract protected function _getFieldIdentifier($prefix);
23
+
24
+	/**
25
+	 * @param \Closure $callback
26
+	 */
20 27
 	abstract public function setValueFunction($index,$callback);
21 28
 
22 29
 	private function _getLabelField($caption,$icon=NULL){
@@ -47,6 +54,9 @@  discard block
 block discarded – undo
47 54
 		$element->fromArray($attributes);
48 55
 	}
49 56
 
57
+	/**
58
+	 * @param \Closure $elementCallback
59
+	 */
50 60
 	protected function _fieldAs($elementCallback,$index,$attributes=NULL){
51 61
 		$this->setValueFunction($index,function($value)use ($index,&$attributes,$elementCallback){
52 62
 			$caption=$this->_instanceViewer->getCaption($index);
Please login to merge, or discard this patch.
Spacing   +35 added lines, -35 removed lines patch added patch discarded remove patch
@@ -14,13 +14,13 @@  discard block
 block discarded – undo
14 14
  * @property object $_modelInstance;
15 15
  */
16 16
 
17
-trait FormFieldAsTrait{
17
+trait FormFieldAsTrait {
18 18
 
19 19
 	abstract protected function _getFieldIdentifier($prefix);
20
-	abstract public function setValueFunction($index,$callback);
20
+	abstract public function setValueFunction($index, $callback);
21 21
 
22
-	private function _getLabelField($caption,$icon=NULL){
23
-		$label=new HtmlLabel($this->_getFieldIdentifier("lbl"),$caption,$icon);
22
+	private function _getLabelField($caption, $icon=NULL) {
23
+		$label=new HtmlLabel($this->_getFieldIdentifier("lbl"), $caption, $icon);
24 24
 		return $label;
25 25
 	}
26 26
 
@@ -28,65 +28,65 @@  discard block
 block discarded – undo
28 28
 	 * @param HtmlFormField $element
29 29
 	 * @param array $attributes
30 30
 	 */
31
-	protected function _applyAttributes($element,&$attributes,$index){
32
-		if(isset($attributes["rules"])){
31
+	protected function _applyAttributes($element, &$attributes, $index) {
32
+		if (isset($attributes["rules"])) {
33 33
 			$rules=$attributes["rules"];
34
-			if(\is_array($rules))
34
+			if (\is_array($rules))
35 35
 				$element->addRules($rules);
36 36
 			else
37 37
 				$element->addRule($rules);
38 38
 			unset($attributes["rules"]);
39 39
 		}
40
-		if(isset($attributes["callback"])){
40
+		if (isset($attributes["callback"])) {
41 41
 			$callback=$attributes["callback"];
42
-			if(\is_callable($callback)){
43
-				$callback($element,$this->_modelInstance,$index);
42
+			if (\is_callable($callback)) {
43
+				$callback($element, $this->_modelInstance, $index);
44 44
 				unset($attributes["callback"]);
45 45
 			}
46 46
 		}
47 47
 		$element->fromArray($attributes);
48 48
 	}
49 49
 
50
-	protected function _fieldAs($elementCallback,$index,$attributes=NULL){
51
-		$this->setValueFunction($index,function($value)use ($index,&$attributes,$elementCallback){
50
+	protected function _fieldAs($elementCallback, $index, $attributes=NULL) {
51
+		$this->setValueFunction($index, function($value)use ($index, &$attributes, $elementCallback){
52 52
 			$caption=$this->_instanceViewer->getCaption($index);
53 53
 			$name=$this->_instanceViewer->getFieldName($index);
54
-			$element=$elementCallback($name,$caption,$value);
55
-			if(\is_array($attributes))
56
-				$this->_applyAttributes($element, $attributes,$index);
54
+			$element=$elementCallback($name, $caption, $value);
55
+			if (\is_array($attributes))
56
+				$this->_applyAttributes($element, $attributes, $index);
57 57
 			return $element;
58 58
 		});
59 59
 			return $this;
60 60
 	}
61 61
 
62 62
 
63
-	public function fieldAsRadio($index,$attributes=NULL){
64
-		return $this->_fieldAs(function($name,$caption,$value){
65
-			return new HtmlFormRadio($name,$name,$caption,$value);
66
-		}, $index,$attributes);
63
+	public function fieldAsRadio($index, $attributes=NULL) {
64
+		return $this->_fieldAs(function($name, $caption, $value) {
65
+			return new HtmlFormRadio($name, $name, $caption, $value);
66
+		}, $index, $attributes);
67 67
 	}
68 68
 
69
-	public function fieldAsTextarea($index,$attributes=NULL){
70
-		return $this->_fieldAs(function($name,$caption,$value){
71
-			return new HtmlFormTextarea($name,$caption,$value);
72
-		}, $index,$attributes);
69
+	public function fieldAsTextarea($index, $attributes=NULL) {
70
+		return $this->_fieldAs(function($name, $caption, $value) {
71
+			return new HtmlFormTextarea($name, $caption, $value);
72
+		}, $index, $attributes);
73 73
 	}
74 74
 
75
-	public function fieldAsInput($index,$attributes=NULL){
76
-		return $this->_fieldAs(function($name,$caption,$value){
77
-			return new HtmlFormInput($name,$caption,"text",$value);
78
-		}, $index,$attributes);
75
+	public function fieldAsInput($index, $attributes=NULL) {
76
+		return $this->_fieldAs(function($name, $caption, $value) {
77
+			return new HtmlFormInput($name, $caption, "text", $value);
78
+		}, $index, $attributes);
79 79
 	}
80 80
 
81
-	public function fieldAsCheckbox($index,$attributes=NULL){
82
-		return $this->_fieldAs(function($name,$caption,$value){
83
-			return new HtmlFormCheckbox($name,$caption,$value);
84
-		}, $index,$attributes);
81
+	public function fieldAsCheckbox($index, $attributes=NULL) {
82
+		return $this->_fieldAs(function($name, $caption, $value) {
83
+			return new HtmlFormCheckbox($name, $caption, $value);
84
+		}, $index, $attributes);
85 85
 	}
86 86
 
87
-	public function fieldAsDropDown($index,$elements=[],$multiple=false,$attributes=NULL){
88
-		return $this->_fieldAs(function($name,$caption,$value) use ($elements,$multiple){
89
-			return new HtmlFormDropdown($name,$elements,$caption,$value,$multiple);
90
-		}, $index,$attributes);
87
+	public function fieldAsDropDown($index, $elements=[], $multiple=false, $attributes=NULL) {
88
+		return $this->_fieldAs(function($name, $caption, $value) use ($elements, $multiple){
89
+			return new HtmlFormDropdown($name, $elements, $caption, $value, $multiple);
90
+		}, $index, $attributes);
91 91
 	}
92 92
 }
93 93
\ No newline at end of file
Please login to merge, or discard this patch.
Braces   +8 added lines, -6 removed lines patch added patch discarded remove patch
@@ -31,10 +31,11 @@  discard block
 block discarded – undo
31 31
 	protected function _applyAttributes($element,&$attributes,$index){
32 32
 		if(isset($attributes["rules"])){
33 33
 			$rules=$attributes["rules"];
34
-			if(\is_array($rules))
35
-				$element->addRules($rules);
36
-			else
37
-				$element->addRule($rules);
34
+			if(\is_array($rules)) {
35
+							$element->addRules($rules);
36
+			} else {
37
+							$element->addRule($rules);
38
+			}
38 39
 			unset($attributes["rules"]);
39 40
 		}
40 41
 		if(isset($attributes["callback"])){
@@ -52,8 +53,9 @@  discard block
 block discarded – undo
52 53
 			$caption=$this->_instanceViewer->getCaption($index);
53 54
 			$name=$this->_instanceViewer->getFieldName($index);
54 55
 			$element=$elementCallback($name,$caption,$value);
55
-			if(\is_array($attributes))
56
-				$this->_applyAttributes($element, $attributes,$index);
56
+			if(\is_array($attributes)) {
57
+							$this->_applyAttributes($element, $attributes,$index);
58
+			}
57 59
 			return $element;
58 60
 		});
59 61
 			return $this;
Please login to merge, or discard this patch.
Ajax/semantic/html/collections/form/HtmlFormField.php 2 patches
Spacing   +32 added lines, -32 removed lines patch added patch discarded remove patch
@@ -12,34 +12,34 @@  discard block
 block discarded – undo
12 12
 class HtmlFormField extends HtmlSemDoubleElement {
13 13
 	protected $_container;
14 14
 	protected $_validation;
15
-	public function __construct($identifier, $field,$label=NULL) {
16
-		parent::__construct($identifier, "div","field");
15
+	public function __construct($identifier, $field, $label=NULL) {
16
+		parent::__construct($identifier, "div", "field");
17 17
 		$this->content=array();
18
-		$this->_states=[State::ERROR,State::DISABLED];
19
-		if(isset($label)===true)
18
+		$this->_states=[State::ERROR, State::DISABLED];
19
+		if (isset($label)===true)
20 20
 			$this->setLabel($label);
21 21
 		$this->setField($field);
22 22
 		$this->_validation=NULL;
23 23
 	}
24 24
 
25
-	public function addPointingLabel($label,$pointing=Direction::NONE){
26
-		$labelO=new HtmlLabel("",$label);
25
+	public function addPointingLabel($label, $pointing=Direction::NONE) {
26
+		$labelO=new HtmlLabel("", $label);
27 27
 		$labelO->setPointing($pointing);
28
-		$this->addContent($labelO,$pointing==="below" || $pointing==="right");
28
+		$this->addContent($labelO, $pointing==="below" || $pointing==="right");
29 29
 		return $labelO;
30 30
 	}
31 31
 
32
-	public function setLabel($label){
32
+	public function setLabel($label) {
33 33
 		$labelO=$label;
34
-		if(\is_string($label)){
35
-			$labelO=new HtmlSemDoubleElement("","label","");
34
+		if (\is_string($label)) {
35
+			$labelO=new HtmlSemDoubleElement("", "label", "");
36 36
 			$labelO->setContent($label);
37
-			$labelO->setProperty("for", \str_replace("field-", "",$this->identifier));
37
+			$labelO->setProperty("for", \str_replace("field-", "", $this->identifier));
38 38
 		}
39 39
 		$this->content["label"]=$labelO;
40 40
 	}
41 41
 
42
-	public function setField($field){
42
+	public function setField($field) {
43 43
 		$this->content["field"]=$field;
44 44
 	}
45 45
 
@@ -47,8 +47,8 @@  discard block
 block discarded – undo
47 47
 	 * Returns the label or null
48 48
 	 * @return mixed
49 49
 	 */
50
-	public function getLabel(){
51
-		if(\array_key_exists("label", $this->content))
50
+	public function getLabel() {
51
+		if (\array_key_exists("label", $this->content))
52 52
 			return $this->content["label"];
53 53
 	}
54 54
 
@@ -56,7 +56,7 @@  discard block
 block discarded – undo
56 56
 	 * Return the field
57 57
 	 * @return mixed
58 58
 	 */
59
-	public function getField(){
59
+	public function getField() {
60 60
 		return $this->content["field"];
61 61
 	}
62 62
 
@@ -64,14 +64,14 @@  discard block
 block discarded – undo
64 64
 	 * Return the field with data
65 65
 	 * @return mixed
66 66
 	 */
67
-	public function getDataField(){
67
+	public function getDataField() {
68 68
 		return $this->content["field"];
69 69
 	}
70 70
 
71 71
 	/**
72 72
 	 * puts the label before or behind
73 73
 	 */
74
-	public function swapLabel(){
74
+	public function swapLabel() {
75 75
 		$label=$this->getLabel();
76 76
 		unset($this->content["label"]);
77 77
 		$this->content["label"]=$label;
@@ -82,31 +82,31 @@  discard block
 block discarded – undo
82 82
 	 * @param int $width
83 83
 	 * @return \Ajax\semantic\html\collections\form\HtmlFormField
84 84
 	 */
85
-	public function setWidth($width){
86
-		if(\is_int($width)){
85
+	public function setWidth($width) {
86
+		if (\is_int($width)) {
87 87
 			$width=Wide::getConstants()["W".$width];
88 88
 		}
89 89
 		$this->addToPropertyCtrl("class", $width, Wide::getConstants());
90
-		if(isset($this->_container)){
90
+		if (isset($this->_container)) {
91 91
 			$this->_container->setEqualWidth(false);
92 92
 		}
93
-		return $this->addToPropertyCtrl("class", "wide",array("wide"));
93
+		return $this->addToPropertyCtrl("class", "wide", array("wide"));
94 94
 	}
95 95
 
96 96
 	/**
97 97
 	 * Field displays an error state
98 98
 	 * @return \Ajax\semantic\html\collections\form\HtmlFormField
99 99
 	 */
100
-	public function setError(){
100
+	public function setError() {
101 101
 		return $this->addToProperty("class", "error");
102 102
 	}
103 103
 
104
-	public function setInline(){
104
+	public function setInline() {
105 105
 		return $this->addToProperty("class", "inline");
106 106
 	}
107 107
 
108
-	public function jsState($state){
109
-		return $this->jsDoJquery("addClass",$state);
108
+	public function jsState($state) {
109
+		return $this->jsDoJquery("addClass", $state);
110 110
 	}
111 111
 
112 112
 	public function setContainer($_container) {
@@ -114,23 +114,23 @@  discard block
 block discarded – undo
114 114
 		return $this;
115 115
 	}
116 116
 
117
-	public function setReadonly(){
117
+	public function setReadonly() {
118 118
 		$this->getField()->setProperty("readonly", "");
119 119
 	}
120 120
 
121
-	public function addRule($type,$prompt=NULL,$value=NULL){
121
+	public function addRule($type, $prompt=NULL, $value=NULL) {
122 122
 		$field=$this->getDataField();
123
-		if(isset($field)){
124
-			if(!isset($this->_validation)){
123
+		if (isset($field)) {
124
+			if (!isset($this->_validation)) {
125 125
 				$this->_validation=new FieldValidation($field->getIdentifier());
126 126
 			}
127
-			$this->_validation->addRule($type,$prompt,$value);
127
+			$this->_validation->addRule($type, $prompt, $value);
128 128
 		}
129 129
 		return $this;
130 130
 	}
131 131
 
132
-	public function addRules(array $rules){
133
-		foreach ($rules as $rule){
132
+	public function addRules(array $rules) {
133
+		foreach ($rules as $rule) {
134 134
 			$this->addRule($rule);
135 135
 		}
136 136
 		return $this;
Please login to merge, or discard this patch.
Braces   +6 added lines, -4 removed lines patch added patch discarded remove patch
@@ -16,8 +16,9 @@  discard block
 block discarded – undo
16 16
 		parent::__construct($identifier, "div","field");
17 17
 		$this->content=array();
18 18
 		$this->_states=[State::ERROR,State::DISABLED];
19
-		if(isset($label)===true)
20
-			$this->setLabel($label);
19
+		if(isset($label)===true) {
20
+					$this->setLabel($label);
21
+		}
21 22
 		$this->setField($field);
22 23
 		$this->_validation=NULL;
23 24
 	}
@@ -48,8 +49,9 @@  discard block
 block discarded – undo
48 49
 	 * @return mixed
49 50
 	 */
50 51
 	public function getLabel(){
51
-		if(\array_key_exists("label", $this->content))
52
-			return $this->content["label"];
52
+		if(\array_key_exists("label", $this->content)) {
53
+					return $this->content["label"];
54
+		}
53 55
 	}
54 56
 
55 57
 	/**
Please login to merge, or discard this patch.
Ajax/semantic/widgets/dataform/FormInstanceViewer.php 2 patches
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -14,36 +14,36 @@
 block discarded – undo
14 14
 		$this->separators=[-1];
15 15
 	}
16 16
 
17
-	protected function _beforeAddProperty($index,&$field){
18
-		if(JString::endswith($field, "\n")===true){
17
+	protected function _beforeAddProperty($index, &$field) {
18
+		if (JString::endswith($field, "\n")===true) {
19 19
 			$this->addSeparatorAfter($index);
20 20
 		}
21
-		if($index>1 && JString::startswith($field, "\n")===true){
21
+		if ($index>1 && JString::startswith($field, "\n")===true) {
22 22
 			$this->addSeparatorAfter($index-1);
23 23
 		}
24 24
 		$field=\str_replace("\n", "", $field);
25 25
 	}
26 26
 
27
-	protected function _getDefaultValue($name,$value,$index){
27
+	protected function _getDefaultValue($name, $value, $index) {
28 28
 		$caption=$this->getCaption($index);
29
-		$input=new HtmlFormInput($name,$caption,"text",$value);
29
+		$input=new HtmlFormInput($name, $caption, "text", $value);
30 30
 		return $input;
31 31
 	}
32 32
 
33
-	public function getFieldName($index){
33
+	public function getFieldName($index) {
34 34
 		$property=$this->getProperty($index);
35
-		if($property instanceof \ReflectionProperty){
35
+		if ($property instanceof \ReflectionProperty) {
36 36
 			$result=$property->getName();
37
-		}elseif(\is_callable($property)){
37
+		}elseif (\is_callable($property)) {
38 38
 			$result=$this->visibleProperties[$index];
39
-		}else{
39
+		} else {
40 40
 			$result=\strtolower($this->getCaption($index));
41 41
 		}
42 42
 		return $result;
43 43
 	}
44 44
 
45
-	public function addSeparatorAfter($fieldNum){
46
-		if(\array_search($fieldNum, $this->separators)===false)
45
+	public function addSeparatorAfter($fieldNum) {
46
+		if (\array_search($fieldNum, $this->separators)===false)
47 47
 			$this->separators[]=$fieldNum;
48 48
 			return $this;
49 49
 	}
Please login to merge, or discard this patch.
Braces   +5 added lines, -4 removed lines patch added patch discarded remove patch
@@ -34,17 +34,18 @@
 block discarded – undo
34 34
 		$property=$this->getProperty($index);
35 35
 		if($property instanceof \ReflectionProperty){
36 36
 			$result=$property->getName();
37
-		}elseif(\is_callable($property)){
37
+		} elseif(\is_callable($property)){
38 38
 			$result=$this->visibleProperties[$index];
39
-		}else{
39
+		} else{
40 40
 			$result=\strtolower($this->getCaption($index));
41 41
 		}
42 42
 		return $result;
43 43
 	}
44 44
 
45 45
 	public function addSeparatorAfter($fieldNum){
46
-		if(\array_search($fieldNum, $this->separators)===false)
47
-			$this->separators[]=$fieldNum;
46
+		if(\array_search($fieldNum, $this->separators)===false) {
47
+					$this->separators[]=$fieldNum;
48
+		}
48 49
 			return $this;
49 50
 	}
50 51
 
Please login to merge, or discard this patch.
Ajax/semantic/widgets/base/InstanceViewer.php 2 patches
Spacing   +63 added lines, -63 removed lines patch added patch discarded remove patch
@@ -14,139 +14,139 @@  discard block
 block discarded – undo
14 14
 
15 15
 	public static $index=0;
16 16
 
17
-	public function __construct($instance=NULL,$captions=NULL){
17
+	public function __construct($instance=NULL, $captions=NULL) {
18 18
 		$this->values=[];
19 19
 		$this->afterCompile=[];
20
-		if(isset($instance))
20
+		if (isset($instance))
21 21
 			$this->setInstance($instance);
22 22
 		$this->setCaptions($captions);
23 23
 	}
24 24
 
25
-	public function getValues(){
25
+	public function getValues() {
26 26
 		$values=[];
27 27
 		$index=0;
28 28
 		$count=$this->count();
29
-		while($index<$count){
29
+		while ($index<$count) {
30 30
 			$values[]=$this->getValue($index++);
31 31
 		}
32 32
 		return $values;
33 33
 	}
34 34
 
35
-	public function getIdentifier(){
35
+	public function getIdentifier() {
36 36
 		$value=self::$index;
37
-		if(isset($this->values["identifier"]))
38
-			$value=$this->values["identifier"](self::$index,$this->instance);
37
+		if (isset($this->values["identifier"]))
38
+			$value=$this->values["identifier"](self::$index, $this->instance);
39 39
 		return $value;
40 40
 	}
41 41
 
42
-	public function getValue($index){
42
+	public function getValue($index) {
43 43
 		$property=$this->properties[$index];
44 44
 		return $this->_getValue($property, $index);
45 45
 	}
46 46
 
47
-	protected function _beforeAddProperty($index,&$field){
47
+	protected function _beforeAddProperty($index, &$field) {
48 48
 
49 49
 	}
50 50
 
51
-	protected function _getDefaultValue($name,$value,$index){
51
+	protected function _getDefaultValue($name, $value, $index) {
52 52
 		return $value;
53 53
 	}
54 54
 
55
-	protected function _getValue($property,$index){
56
-		if($property instanceof \ReflectionProperty){
55
+	protected function _getValue($property, $index) {
56
+		if ($property instanceof \ReflectionProperty) {
57 57
 			$property->setAccessible(true);
58 58
 			$value=$property->getValue($this->instance);
59
-			if(isset($this->values[$index])){
60
-				$value= $this->values[$index]($value,$this->instance,$index);
61
-			}else{
62
-				$value=$this->_getDefaultValue($property->getName(),$value, $index);
59
+			if (isset($this->values[$index])) {
60
+				$value=$this->values[$index]($value, $this->instance, $index);
61
+			} else {
62
+				$value=$this->_getDefaultValue($property->getName(), $value, $index);
63 63
 			}
64
-		}else{
65
-			if(\is_callable($property))
64
+		} else {
65
+			if (\is_callable($property))
66 66
 				$value=$property($this->instance);
67
-			elseif(\is_array($property)){
68
-				$values=\array_map(function($v) use ($index){return $this->_getValue($v, $index);}, $property);
67
+			elseif (\is_array($property)) {
68
+				$values=\array_map(function($v) use ($index){return $this->_getValue($v, $index); }, $property);
69 69
 				$value=\implode("", $values);
70
-			}else
70
+			} else
71 71
 				$value=$property;
72 72
 		}
73
-		if(isset($this->afterCompile[$index])){
74
-			if(\is_callable($this->afterCompile[$index])){
75
-				$this->afterCompile[$index]($value,$this->instance,$index);
73
+		if (isset($this->afterCompile[$index])) {
74
+			if (\is_callable($this->afterCompile[$index])) {
75
+				$this->afterCompile[$index]($value, $this->instance, $index);
76 76
 			}
77 77
 		}
78 78
 		return $value;
79 79
 	}
80 80
 
81
-	public function insertField($index,$field){
82
-		array_splice( $this->visibleProperties, $index, 0, $field );
81
+	public function insertField($index, $field) {
82
+		array_splice($this->visibleProperties, $index, 0, $field);
83 83
 		return $this;
84 84
 	}
85 85
 
86
-	public function insertInField($index,$field){
86
+	public function insertInField($index, $field) {
87 87
 		$vb=$this->visibleProperties;
88
-		if(isset($vb[$index])){
89
-			if(\is_array($vb[$index])){
88
+		if (isset($vb[$index])) {
89
+			if (\is_array($vb[$index])) {
90 90
 				$this->visibleProperties[$index][]=$field;
91
-			}else{
92
-				$this->visibleProperties[$index]=[$vb[$index],$field];
91
+			} else {
92
+				$this->visibleProperties[$index]=[$vb[$index], $field];
93 93
 			}
94
-		}else{
94
+		} else {
95 95
 			return $this->insertField($index, $field);
96 96
 		}
97 97
 		return $this;
98 98
 	}
99 99
 
100
-	public function addField($field){
100
+	public function addField($field) {
101 101
 		$this->visibleProperties[]=$field;
102 102
 		return $this;
103 103
 	}
104 104
 
105
-	public function count(){
105
+	public function count() {
106 106
 		return \sizeof($this->properties);
107 107
 	}
108 108
 
109
-	public function visiblePropertiesCount(){
109
+	public function visiblePropertiesCount() {
110 110
 		return \sizeof($this->visibleProperties);
111 111
 	}
112 112
 
113
-	public function getProperty($index){
113
+	public function getProperty($index) {
114 114
 		return $this->properties[$index];
115 115
 	}
116 116
 
117
-	protected function showableProperty(\ReflectionProperty $rProperty){
118
-		return JString::startswith($rProperty->getName(),"_")===false;
117
+	protected function showableProperty(\ReflectionProperty $rProperty) {
118
+		return JString::startswith($rProperty->getName(), "_")===false;
119 119
 	}
120 120
 
121 121
 	public function setInstance($instance) {
122
-		if(\is_string($instance)){
122
+		if (\is_string($instance)) {
123 123
 			$instance=new $instance();
124 124
 		}
125 125
 		$this->instance=$instance;
126 126
 		$this->properties=[];
127 127
 		$this->reflect=new \ReflectionClass($instance);
128
-		if(\sizeof($this->visibleProperties)===0){
128
+		if (\sizeof($this->visibleProperties)===0) {
129 129
 			$this->properties=$this->getDefaultProperties();
130
-		}else{
131
-			foreach ($this->visibleProperties as $property){
132
-				if(\is_callable($property)){
130
+		} else {
131
+			foreach ($this->visibleProperties as $property) {
132
+				if (\is_callable($property)) {
133 133
 					$this->properties[]=$property;
134
-				}elseif(\is_string($property)){
135
-					try{
134
+				}elseif (\is_string($property)) {
135
+					try {
136 136
 						$this->_beforeAddProperty(\sizeof($this->properties), $property);
137 137
 						$rProperty=$this->reflect->getProperty($property);
138 138
 						$this->properties[]=$rProperty;
139
-					}catch(\Exception $e){
139
+					}catch (\Exception $e) {
140 140
 						$this->_beforeAddProperty(\sizeof($this->properties), $property);
141 141
 						$this->properties[]=$property;
142 142
 					}
143
-				}elseif(\is_int($property)){
143
+				}elseif (\is_int($property)) {
144 144
 					$props=$this->getDefaultProperties();
145
-					if(isset($props[$property]))
145
+					if (isset($props[$property]))
146 146
 						$this->properties[]=$props[$property];
147 147
 					else
148 148
 						$this->properties[]=$property;
149
-				}else{
149
+				} else {
150 150
 					$this->properties[]=$property;
151 151
 				}
152 152
 			}
@@ -154,12 +154,12 @@  discard block
 block discarded – undo
154 154
 		return $this;
155 155
 	}
156 156
 
157
-	protected function getDefaultProperties(){
157
+	protected function getDefaultProperties() {
158 158
 		$result=[];
159 159
 		$properties=$this->reflect->getProperties();
160
-		foreach ($properties as $property){
160
+		foreach ($properties as $property) {
161 161
 			$showable=$this->showableProperty($property);
162
-			if($showable!==false){
162
+			if ($showable!==false) {
163 163
 				$result[]=$property;
164 164
 			}
165 165
 		}
@@ -171,12 +171,12 @@  discard block
 block discarded – undo
171 171
 		return $this;
172 172
 	}
173 173
 
174
-	public function setValueFunction($index,$callback){
174
+	public function setValueFunction($index, $callback) {
175 175
 		$this->values[$index]=$callback;
176 176
 		return $this;
177 177
 	}
178 178
 
179
-	public function setIdentifierFunction($callback){
179
+	public function setIdentifierFunction($callback) {
180 180
 		$this->values["identifier"]=$callback;
181 181
 		return $this;
182 182
 	}
@@ -189,19 +189,19 @@  discard block
 block discarded – undo
189 189
 		return $this->properties;
190 190
 	}
191 191
 
192
-	public function getCaption($index){
193
-		if($this->properties[$index] instanceof \ReflectionProperty)
192
+	public function getCaption($index) {
193
+		if ($this->properties[$index] instanceof \ReflectionProperty)
194 194
 			return $this->properties[$index]->getName();
195
-			elseif(\is_callable($this->properties[$index]))
195
+			elseif (\is_callable($this->properties[$index]))
196 196
 			return "";
197 197
 			else
198 198
 				return $this->properties[$index];
199 199
 	}
200 200
 
201
-	public function getCaptions(){
202
-		if(isset($this->captions)){
203
-			$result= $this->captions;
204
-			for($i=\sizeof($result);$i<$this->count();$i++){
201
+	public function getCaptions() {
202
+		if (isset($this->captions)) {
203
+			$result=$this->captions;
204
+			for ($i=\sizeof($result); $i<$this->count(); $i++) {
205 205
 				$result[]="";
206 206
 			}
207 207
 			return $result;
@@ -209,7 +209,7 @@  discard block
 block discarded – undo
209 209
 		$captions=[];
210 210
 		$index=0;
211 211
 		$count=$this->count();
212
-		while($index<$count){
212
+		while ($index<$count) {
213 213
 			$captions[]=$this->getCaption($index++);
214 214
 		}
215 215
 		return $captions;
@@ -227,7 +227,7 @@  discard block
 block discarded – undo
227 227
 	 * @param callable $callback function called after the field compilation
228 228
 	 * @return \Ajax\semantic\widgets\datatable\InstanceViewer
229 229
 	 */
230
-	public function afterCompile($index,$callback){
230
+	public function afterCompile($index, $callback) {
231 231
 		$this->afterCompile[$index]=$callback;
232 232
 		return $this;
233 233
 	}
Please login to merge, or discard this patch.
Braces   +33 added lines, -28 removed lines patch added patch discarded remove patch
@@ -17,8 +17,9 @@  discard block
 block discarded – undo
17 17
 	public function __construct($instance=NULL,$captions=NULL){
18 18
 		$this->values=[];
19 19
 		$this->afterCompile=[];
20
-		if(isset($instance))
21
-			$this->setInstance($instance);
20
+		if(isset($instance)) {
21
+					$this->setInstance($instance);
22
+		}
22 23
 		$this->setCaptions($captions);
23 24
 	}
24 25
 
@@ -34,8 +35,9 @@  discard block
 block discarded – undo
34 35
 
35 36
 	public function getIdentifier(){
36 37
 		$value=self::$index;
37
-		if(isset($this->values["identifier"]))
38
-			$value=$this->values["identifier"](self::$index,$this->instance);
38
+		if(isset($this->values["identifier"])) {
39
+					$value=$this->values["identifier"](self::$index,$this->instance);
40
+		}
39 41
 		return $value;
40 42
 	}
41 43
 
@@ -58,17 +60,18 @@  discard block
 block discarded – undo
58 60
 			$value=$property->getValue($this->instance);
59 61
 			if(isset($this->values[$index])){
60 62
 				$value= $this->values[$index]($value,$this->instance,$index);
61
-			}else{
63
+			} else{
62 64
 				$value=$this->_getDefaultValue($property->getName(),$value, $index);
63 65
 			}
64
-		}else{
65
-			if(\is_callable($property))
66
-				$value=$property($this->instance);
67
-			elseif(\is_array($property)){
66
+		} else{
67
+			if(\is_callable($property)) {
68
+							$value=$property($this->instance);
69
+			} elseif(\is_array($property)){
68 70
 				$values=\array_map(function($v) use ($index){return $this->_getValue($v, $index);}, $property);
69 71
 				$value=\implode("", $values);
70
-			}else
71
-				$value=$property;
72
+			} else {
73
+							$value=$property;
74
+			}
72 75
 		}
73 76
 		if(isset($this->afterCompile[$index])){
74 77
 			if(\is_callable($this->afterCompile[$index])){
@@ -88,10 +91,10 @@  discard block
 block discarded – undo
88 91
 		if(isset($vb[$index])){
89 92
 			if(\is_array($vb[$index])){
90 93
 				$this->visibleProperties[$index][]=$field;
91
-			}else{
94
+			} else{
92 95
 				$this->visibleProperties[$index]=[$vb[$index],$field];
93 96
 			}
94
-		}else{
97
+		} else{
95 98
 			return $this->insertField($index, $field);
96 99
 		}
97 100
 		return $this;
@@ -127,26 +130,27 @@  discard block
 block discarded – undo
127 130
 		$this->reflect=new \ReflectionClass($instance);
128 131
 		if(\sizeof($this->visibleProperties)===0){
129 132
 			$this->properties=$this->getDefaultProperties();
130
-		}else{
133
+		} else{
131 134
 			foreach ($this->visibleProperties as $property){
132 135
 				if(\is_callable($property)){
133 136
 					$this->properties[]=$property;
134
-				}elseif(\is_string($property)){
137
+				} elseif(\is_string($property)){
135 138
 					try{
136 139
 						$this->_beforeAddProperty(\sizeof($this->properties), $property);
137 140
 						$rProperty=$this->reflect->getProperty($property);
138 141
 						$this->properties[]=$rProperty;
139
-					}catch(\Exception $e){
142
+					} catch(\Exception $e){
140 143
 						$this->_beforeAddProperty(\sizeof($this->properties), $property);
141 144
 						$this->properties[]=$property;
142 145
 					}
143
-				}elseif(\is_int($property)){
146
+				} elseif(\is_int($property)){
144 147
 					$props=$this->getDefaultProperties();
145
-					if(isset($props[$property]))
146
-						$this->properties[]=$props[$property];
147
-					else
148
-						$this->properties[]=$property;
149
-				}else{
148
+					if(isset($props[$property])) {
149
+											$this->properties[]=$props[$property];
150
+					} else {
151
+											$this->properties[]=$property;
152
+					}
153
+				} else{
150 154
 					$this->properties[]=$property;
151 155
 				}
152 156
 			}
@@ -190,12 +194,13 @@  discard block
 block discarded – undo
190 194
 	}
191 195
 
192 196
 	public function getCaption($index){
193
-		if($this->properties[$index] instanceof \ReflectionProperty)
194
-			return $this->properties[$index]->getName();
195
-			elseif(\is_callable($this->properties[$index]))
196
-			return "";
197
-			else
198
-				return $this->properties[$index];
197
+		if($this->properties[$index] instanceof \ReflectionProperty) {
198
+					return $this->properties[$index]->getName();
199
+		} elseif(\is_callable($this->properties[$index])) {
200
+						return "";
201
+			} else {
202
+							return $this->properties[$index];
203
+			}
199 204
 	}
200 205
 
201 206
 	public function getCaptions(){
Please login to merge, or discard this patch.
Ajax/semantic/traits/SemanticWidgetsTrait.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -15,8 +15,8 @@  discard block
 block discarded – undo
15 15
 	 * @param array $instances
16 16
 	 * @return DataTable
17 17
 	 */
18
-	public function dataTable($identifier,$model, $instances){
19
-		return $this->addHtmlComponent(new DataTable($identifier,$model,$instances));
18
+	public function dataTable($identifier, $model, $instances) {
19
+		return $this->addHtmlComponent(new DataTable($identifier, $model, $instances));
20 20
 	}
21 21
 
22 22
 	/**
@@ -24,8 +24,8 @@  discard block
 block discarded – undo
24 24
 	 * @param object $instance
25 25
 	 * @return DataElement
26 26
 	 */
27
-	public function dataElement($identifier, $instance){
28
-		return $this->addHtmlComponent(new DataElement($identifier,$instance));
27
+	public function dataElement($identifier, $instance) {
28
+		return $this->addHtmlComponent(new DataElement($identifier, $instance));
29 29
 	}
30 30
 
31 31
 	/**
@@ -33,7 +33,7 @@  discard block
 block discarded – undo
33 33
 	 * @param object $instance
34 34
 	 * @return DataForm
35 35
 	 */
36
-	public function dataForm($identifier, $instance){
37
-		return $this->addHtmlComponent(new DataForm($identifier,$instance));
36
+	public function dataForm($identifier, $instance) {
37
+		return $this->addHtmlComponent(new DataForm($identifier, $instance));
38 38
 	}
39 39
 }
40 40
\ No newline at end of file
Please login to merge, or discard this patch.