Passed
Push — master ( 4fc601...e10dfd )
by Jean-Christophe
02:05
created
Ajax/service/JString.php 2 patches
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -64,8 +64,9 @@
 block discarded – undo
64 64
 	}
65 65
 	
66 66
 	public static function doubleBackSlashes($value){
67
-		if(is_string($value))
68
-			return str_replace("\\", "\\\\", $value);
67
+		if(is_string($value)) {
68
+					return str_replace("\\", "\\\\", $value);
69
+		}
69 70
 			return $value;
70 71
 	}
71 72
 
Please login to merge, or discard this patch.
Spacing   +20 added lines, -20 removed lines patch added patch discarded remove patch
@@ -2,69 +2,69 @@
 block discarded – undo
2 2
 namespace Ajax\service;
3 3
 class JString {
4 4
 
5
-	public static function contains($hay,$needle){
6
-		return strpos($hay, "$needle") !== false;
5
+	public static function contains($hay, $needle) {
6
+		return strpos($hay, "$needle")!==false;
7 7
 	}
8 8
 	public static function startswith($hay, $needle) {
9
-		return substr($hay, 0, strlen($needle)) === $needle;
9
+		return substr($hay, 0, strlen($needle))===$needle;
10 10
 	}
11 11
 
12 12
 	public static function endswith($hay, $needle) {
13
-		return substr($hay, -strlen($needle)) === $needle;
13
+		return substr($hay, -strlen($needle))===$needle;
14 14
 	}
15 15
 
16
-	public static function isNull($s){
16
+	public static function isNull($s) {
17 17
 		return (!isset($s) || NULL===$s || ""===$s);
18 18
 	}
19
-	public static function isNotNull($s){
19
+	public static function isNotNull($s) {
20 20
 		return (isset($s) && NULL!==$s && ""!==$s);
21 21
 	}
22 22
 
23
-	public static function isBoolean($value){
23
+	public static function isBoolean($value) {
24 24
 		return \is_bool($value) || $value==1 || $value==0;
25 25
 	}
26 26
 
27
-	public static function isBooleanTrue($value){
27
+	public static function isBooleanTrue($value) {
28 28
 		return $value==1 || $value;
29 29
 	}
30 30
 
31
-	public static function isBooleanFalse($value){
31
+	public static function isBooleanFalse($value) {
32 32
 		return $value==0 || !$value;
33 33
 	}
34 34
 
35
-	public static function camelCaseToSeparated($input,$separator=" "){
35
+	public static function camelCaseToSeparated($input, $separator=" ") {
36 36
 		return strtolower(preg_replace('/(?<!^)[A-Z]/', $separator.'$0', $input));
37 37
 	}
38 38
 
39
-	public static function replaceAtFirst($subject,$from, $to){
40
-		$from = '/\A'.preg_quote($from, '/').'/';
39
+	public static function replaceAtFirst($subject, $from, $to) {
40
+		$from='/\A'.preg_quote($from, '/').'/';
41 41
 		return \preg_replace($from, $to, $subject, 1);
42 42
 	}
43 43
 
44
-	public static function replaceAtLast($subject,$from, $to){
45
-		$from = '/'.preg_quote($from, '/').'\z/';
44
+	public static function replaceAtLast($subject, $from, $to) {
45
+		$from='/'.preg_quote($from, '/').'\z/';
46 46
 		return \preg_replace($from, $to, $subject, 1);
47 47
 	}
48 48
 
49
-	public static function replaceAtFirstAndLast($subject,$fromFirst,$toFirst,$fromLast,$toLast){
49
+	public static function replaceAtFirstAndLast($subject, $fromFirst, $toFirst, $fromLast, $toLast) {
50 50
 		$s=self::replaceAtFirst($subject, $fromFirst, $toFirst);
51 51
 		return self::replaceAtLast($s, $fromLast, $toLast);
52 52
 	}
53 53
 
54
-	public static function getValueBetween(&$str,$before="{{",$after="}}"){
54
+	public static function getValueBetween(&$str, $before="{{", $after="}}") {
55 55
 		$matches=[];
56 56
 		$result=null;
57 57
 		$_before=\preg_quote($before);
58 58
 		$_after=\preg_quote($after);
59
-		if(\preg_match('/'.$_before.'(.*?)'.$_after.'/s', $str, $matches)===1){
59
+		if (\preg_match('/'.$_before.'(.*?)'.$_after.'/s', $str, $matches)===1) {
60 60
 			$result=$matches[1];
61
-			$str=\str_replace($before.$result.$after,"", $str);
61
+			$str=\str_replace($before.$result.$after, "", $str);
62 62
 		}
63 63
 		return $result;
64 64
 	}
65 65
 	
66
-	public static function doubleBackSlashes($value){
67
-		if(is_string($value))
66
+	public static function doubleBackSlashes($value) {
67
+		if (is_string($value))
68 68
 			return str_replace("\\", "\\\\", $value);
69 69
 			return $value;
70 70
 	}
Please login to merge, or discard this patch.
Ajax/service/Javascript.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -6,19 +6,19 @@  discard block
 block discarded – undo
6 6
 	public static $preventDefault="\nif(event && event.preventDefault) event.preventDefault();\n";
7 7
 	public static $stopPropagation="\nif(event && event.stopPropagation) event.stopPropagation();\n";
8 8
 
9
-	public static function draggable($attr="id"){
9
+	public static function draggable($attr="id") {
10 10
 		return 'var dt=event.dataTransfer || event.originalEvent.dataTransfer;dt.setData("text/plain",JSON.stringify({id:$(event.target).attr("id"),data:$(event.target).attr("'.$attr.'")}));';
11 11
 	}
12 12
 	
13
-	public static function dropZone($jqueryDone,$jsCallback=""){
13
+	public static function dropZone($jqueryDone, $jsCallback="") {
14 14
 		return 'var dt=event.dataTransfer || event.originalEvent.dataTransfer;var _data=JSON.parse(dt.getData("text/plain"));$(event.target).'.$jqueryDone.'($("#"+_data.id));var data=_data.data;'.$jsCallback;
15 15
 	}
16 16
 	
17
-	public static function containsCode($expression){
18
-		return strrpos($expression, 'this')!==false||strrpos($expression, 'event')!==false||strrpos($expression, 'self')!==false;
17
+	public static function containsCode($expression) {
18
+		return strrpos($expression, 'this')!==false || strrpos($expression, 'event')!==false || strrpos($expression, 'self')!==false;
19 19
 	}
20 20
 	
21
-	public static function isFunction($jsCode){
21
+	public static function isFunction($jsCode) {
22 22
 		return JString::startswith($jsCode, "function");
23 23
 	}
24 24
 
@@ -50,14 +50,14 @@  discard block
 block discarded – undo
50 50
 			$value=implode(",", $value);
51 51
 		}
52 52
 		if (self::containsCode($value)===false) {
53
-			$value=\str_replace(["\\","\""], ["\\\\","\\\""], $value);
53
+			$value=\str_replace(["\\", "\""], ["\\\\", "\\\""], $value);
54 54
 			$value='"'.$value.'"';
55 55
 		}
56
-		return trim($value,"%");
56
+		return trim($value, "%");
57 57
 	}
58 58
 
59
-	public static function prep_jquery_selector($value){
60
-		if(JString::startswith($value, '$(')===false){
59
+	public static function prep_jquery_selector($value) {
60
+		if (JString::startswith($value, '$(')===false) {
61 61
 			return '$('.self::prep_value($value).')';
62 62
 		}
63 63
 		return $value;
Please login to merge, or discard this patch.
Ajax/semantic/widgets/datatable/JsonDataTable.php 2 patches
Spacing   +32 added lines, -32 removed lines patch added patch discarded remove patch
@@ -22,25 +22,25 @@  discard block
 block discarded – undo
22 22
 		$this->_rowClass="_json _element";
23 23
 	}
24 24
 
25
-	protected function _generateContent($table){
25
+	protected function _generateContent($table) {
26 26
 		$this->_addRowModel($table);
27 27
 		parent::_generateContent($table);
28 28
 	}
29 29
 
30
-	protected function _addRowModel($table){
30
+	protected function _addRowModel($table) {
31 31
 		$fields=$this->_instanceViewer->getSimpleProperties();
32
-		$row=$this->_createRow($table, $this->_modelClass,$fields);
33
-		$row->setProperty("style","display:none;");
32
+		$row=$this->_createRow($table, $this->_modelClass, $fields);
33
+		$row->setProperty("style", "display:none;");
34 34
 		$table->getBody()->_addRow($row);
35 35
 	}
36 36
 
37
-	protected function _createRow($table,$rowClass,$fields){
37
+	protected function _createRow($table, $rowClass, $fields) {
38 38
 		$object=JReflection::jsonObject($this->_model);
39
-		if(isset($this->_rowModelCallback)){
39
+		if (isset($this->_rowModelCallback)) {
40 40
 			$callback=$this->_rowModelCallback;
41 41
 			$callback($object);
42 42
 		}
43
-		$row=$this->_generateRow($object, $fields,$table,"_jsonArrayChecked");
43
+		$row=$this->_generateRow($object, $fields, $table, "_jsonArrayChecked");
44 44
 		$row->setClass($rowClass." _element");
45 45
 		return $row;
46 46
 	}
@@ -49,24 +49,24 @@  discard block
 block discarded – undo
49 49
 	 * {@inheritDoc}
50 50
 	 * @see DataTable::_associatePaginationBehavior()
51 51
 	 */
52
-	protected function _associatePaginationBehavior(JsUtils $js=NULL,$offset=null){
52
+	protected function _associatePaginationBehavior(JsUtils $js=NULL, $offset=null) {
53 53
 		$callback=null;
54 54
 		$menu=$this->_pagination->getMenu();
55 55
 		
56
-		if(isset($js)){
56
+		if (isset($js)) {
57 57
 			$id=$this->identifier;
58 58
 			$callback=$js->getScript($offset).$this->getHtmlComponent()->getInnerScript();
59
-			$callback.=$js->trigger("#".$id." [name='selection[]']","change",false)."$('#".$id." tbody .ui.checkbox').checkbox();".$js->execOn("change", "#".$id." [name='selection[]']", $this->_getCheckedChange($js));
59
+			$callback.=$js->trigger("#".$id." [name='selection[]']", "change", false)."$('#".$id." tbody .ui.checkbox').checkbox();".$js->execOn("change", "#".$id." [name='selection[]']", $this->_getCheckedChange($js));
60 60
 			$callback.=$this->_generatePaginationScript($id);
61
-			if(isset($this->_urls["refresh"])){
62
-				if(isset($menu))
63
-				$js->jsonArrayOn("click", "#".$menu->getIdentifier()." a","#".$this->_identifier." tr.".$this->_modelClass, $this->_urls["refresh"],"post",["params"=>"{'p':$(this).attr('data-page'),'_model':'".JString::doubleBackSlashes($this->_model)."'}","jsCallback"=>$callback]);
61
+			if (isset($this->_urls["refresh"])) {
62
+				if (isset($menu))
63
+				$js->jsonArrayOn("click", "#".$menu->getIdentifier()." a", "#".$this->_identifier." tr.".$this->_modelClass, $this->_urls["refresh"], "post", ["params"=>"{'p':$(this).attr('data-page'),'_model':'".JString::doubleBackSlashes($this->_model)."'}", "jsCallback"=>$callback]);
64 64
 			}
65 65
 		}
66 66
 
67 67
 	}
68 68
 	
69
-	protected function _generatePaginationScript($id){
69
+	protected function _generatePaginationScript($id) {
70 70
 		return ";var page=parseInt($(self).attr('data-page')) || 1;var pages_visibles=$('#pagination-{$id} .item').length-2;
71 71
 			var lastPage=$('#pagination-{$id} ._lastPage');
72 72
 			var middle= Math.ceil((pages_visibles-1)/ 2);
@@ -86,16 +86,16 @@  discard block
 block discarded – undo
86 86
 			lastPage.attr('data-page',Math.min(lastPage.attr('data-max'),page+1));
87 87
 			$('#{$id}').trigger('pageChange');$('#{$id}').trigger('activeRowChange');$('#pagination-{$id}').show();";
88 88
 	}
89
-	protected function _compileSearchFieldBehavior(JsUtils $js=NULL){
89
+	protected function _compileSearchFieldBehavior(JsUtils $js=NULL) {
90 90
 		
91 91
 	}
92
-	protected function _associateSearchFieldBehavior(JsUtils $js=NULL,$offset=null){
93
-		if(isset($this->_searchField) && isset($js) && isset($this->_urls["refresh"])){
92
+	protected function _associateSearchFieldBehavior(JsUtils $js=NULL, $offset=null) {
93
+		if (isset($this->_searchField) && isset($js) && isset($this->_urls["refresh"])) {
94 94
 			$id=$this->identifier;
95 95
 			$callback=$js->getScript($offset).$this->getHtmlComponent()->getInnerScript();
96
-			$callback.=$js->trigger("#".$id." [name='selection[]']","change",false)."$('#".$id." tbody .ui.checkbox').checkbox();".$js->execOn("change", "#".$id." [name='selection[]']", $this->_getCheckedChange($js));
96
+			$callback.=$js->trigger("#".$id." [name='selection[]']", "change", false)."$('#".$id." tbody .ui.checkbox').checkbox();".$js->execOn("change", "#".$id." [name='selection[]']", $this->_getCheckedChange($js));
97 97
 			$callback.="$('#pagination-{$id}').hide();$('#".$this->identifier."').trigger('searchTerminate',[$(self).val()]);";
98
-			$js->jsonArrayOn("change", "#".$this->_searchField->getDataField()->getIdentifier(),"#".$this->_identifier." tr.".$this->_modelClass, $this->_urls["refresh"],"post",["params"=>"{'s':$(self).val(),'_model':'".JString::doubleBackSlashes($this->_model)."'}","jsCallback"=>$callback]);
98
+			$js->jsonArrayOn("change", "#".$this->_searchField->getDataField()->getIdentifier(), "#".$this->_identifier." tr.".$this->_modelClass, $this->_urls["refresh"], "post", ["params"=>"{'s':$(self).val(),'_model':'".JString::doubleBackSlashes($this->_model)."'}", "jsCallback"=>$callback]);
99 99
 		}
100 100
 	}
101 101
 	
@@ -109,29 +109,29 @@  discard block
 block discarded – undo
109 109
 	 * @param string $jsCallback
110 110
 	 * @return AjaxCall
111 111
 	 */
112
-	public function jsJsonArray($url, $method="get", $params="{}", $jsCallback=NULL,$parameters=[]){
113
-		$parameters=\array_merge($parameters,["modelSelector"=>"#".$this->_identifier." tr.".$this->_modelClass,"url"=>$url,"method"=>$method,"params"=>$params,"jsCallback"=>$jsCallback]);
112
+	public function jsJsonArray($url, $method="get", $params="{}", $jsCallback=NULL, $parameters=[]) {
113
+		$parameters=\array_merge($parameters, ["modelSelector"=>"#".$this->_identifier." tr.".$this->_modelClass, "url"=>$url, "method"=>$method, "params"=>$params, "jsCallback"=>$jsCallback]);
114 114
 		return new AjaxCall("jsonArray", $parameters);
115 115
 	}
116 116
 
117
-	public function jsClear(){
117
+	public function jsClear() {
118 118
 		return "$('#{$this->identifier} tbody').find('._json').remove();";
119 119
 	}
120 120
 
121
-	public function clearOn(BaseHtml $element,$event, $stopPropagation=false, $preventDefault=false){
122
-		return $element->addEvent($event, $this->jsClear(),$stopPropagation,$preventDefault);
121
+	public function clearOn(BaseHtml $element, $event, $stopPropagation=false, $preventDefault=false) {
122
+		return $element->addEvent($event, $this->jsClear(), $stopPropagation, $preventDefault);
123 123
 	}
124 124
 
125
-	public function clearOnClick(BaseHtml $element,$stopPropagation=false, $preventDefault=false){
126
-		return $this->clearOn($element, "click",$stopPropagation,$preventDefault);
125
+	public function clearOnClick(BaseHtml $element, $stopPropagation=false, $preventDefault=false) {
126
+		return $this->clearOn($element, "click", $stopPropagation, $preventDefault);
127 127
 	}
128 128
 
129
-	public function jsonArrayOn(BaseHtml $element,$event,$url, $method="get", $params="{}", $jsCallback=NULL,$parameters=[]){
130
-		return $element->_addEvent($event, $this->jsJsonArray($url,$method,$params,$jsCallback,$parameters));
129
+	public function jsonArrayOn(BaseHtml $element, $event, $url, $method="get", $params="{}", $jsCallback=NULL, $parameters=[]) {
130
+		return $element->_addEvent($event, $this->jsJsonArray($url, $method, $params, $jsCallback, $parameters));
131 131
 	}
132 132
 
133
-	public function jsonArrayOnClick(BaseHtml $element,$url, $method="get", $params="{}", $jsCallback=NULL,$parameters=[]){
134
-		return $this->jsonArrayOn($element, "click", $url,$method,$params,$jsCallback,$parameters);
133
+	public function jsonArrayOnClick(BaseHtml $element, $url, $method="get", $params="{}", $jsCallback=NULL, $parameters=[]) {
134
+		return $this->jsonArrayOn($element, "click", $url, $method, $params, $jsCallback, $parameters);
135 135
 	}
136 136
 
137 137
 	/**
@@ -142,8 +142,8 @@  discard block
 block discarded – undo
142 142
 	 * @param number $pages_visibles The number of visible pages in the Pagination component
143 143
 	 * @return DataTable
144 144
 	 */
145
-	public function paginate($page,$total_rowcount,$items_per_page=10,$pages_visibles=null){
146
-		return parent::paginate($page, $total_rowcount,$items_per_page,$pages_visibles);
145
+	public function paginate($page, $total_rowcount, $items_per_page=10, $pages_visibles=null) {
146
+		return parent::paginate($page, $total_rowcount, $items_per_page, $pages_visibles);
147 147
 	}
148 148
 
149 149
 	public function setRowModelCallback($_rowModelCallback) {
Please login to merge, or discard this patch.
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -59,8 +59,9 @@
 block discarded – undo
59 59
 			$callback.=$js->trigger("#".$id." [name='selection[]']","change",false)."$('#".$id." tbody .ui.checkbox').checkbox();".$js->execOn("change", "#".$id." [name='selection[]']", $this->_getCheckedChange($js));
60 60
 			$callback.=$this->_generatePaginationScript($id);
61 61
 			if(isset($this->_urls["refresh"])){
62
-				if(isset($menu))
63
-				$js->jsonArrayOn("click", "#".$menu->getIdentifier()." a","#".$this->_identifier." tr.".$this->_modelClass, $this->_urls["refresh"],"post",["params"=>"{'p':$(this).attr('data-page'),'_model':'".JString::doubleBackSlashes($this->_model)."'}","jsCallback"=>$callback]);
62
+				if(isset($menu)) {
63
+								$js->jsonArrayOn("click", "#".$menu->getIdentifier()." a","#".$this->_identifier." tr.".$this->_modelClass, $this->_urls["refresh"],"post",["params"=>"{'p':$(this).attr('data-page'),'_model':'".JString::doubleBackSlashes($this->_model)."'}","jsCallback"=>$callback]);
64
+				}
64 65
 			}
65 66
 		}
66 67
 
Please login to merge, or discard this patch.
Ajax/semantic/widgets/datatable/DataTable.php 2 patches
Braces   +39 added lines, -27 removed lines patch added patch discarded remove patch
@@ -61,16 +61,19 @@  discard block
 block discarded – undo
61 61
 			$js->execOn("mouseover", "#".$this->identifier." tr", "$(event.target).closest('tr').find('.visibleover').css('visibility', 'visible');",["preventDefault"=>false,"stopPropagation"=>true]);
62 62
 			$js->execOn("mouseout", "#".$this->identifier." tr", "$(event.target).closest('tr').find('.visibleover').css('visibility', 'hidden');",["preventDefault"=>false,"stopPropagation"=>true]);
63 63
 		}
64
-		if(\is_array($this->_deleteBehavior))
65
-			$this->_generateBehavior("delete",$this->_deleteBehavior, $js);
66
-		if(\is_array($this->_editBehavior))
67
-			$this->_generateBehavior("edit",$this->_editBehavior,$js);
64
+		if(\is_array($this->_deleteBehavior)) {
65
+					$this->_generateBehavior("delete",$this->_deleteBehavior, $js);
66
+		}
67
+		if(\is_array($this->_editBehavior)) {
68
+					$this->_generateBehavior("edit",$this->_editBehavior,$js);
69
+		}
68 70
 		if(\is_array($this->_displayBehavior)){
69 71
 			$this->_generateBehavior("display",$this->_displayBehavior,$js);
70 72
 		}
71 73
 		parent::run($js);
72
-		if(isset($this->_pagination))
73
-			$this->_associatePaginationBehavior($js,$offset);
74
+		if(isset($this->_pagination)) {
75
+					$this->_associatePaginationBehavior($js,$offset);
76
+		}
74 77
 		$this->_associateSearchFieldBehavior($js,$offset);
75 78
 			
76 79
 	}
@@ -105,8 +108,9 @@  discard block
 block discarded – undo
105 108
 			$table->setRowCount(0, \sizeof($captions));
106 109
 			$this->_generateHeader($table,$captions);
107 110
 
108
-			if(isset($this->_compileParts))
109
-				$table->setCompileParts($this->_compileParts);
111
+			if(isset($this->_compileParts)) {
112
+							$table->setCompileParts($this->_compileParts);
113
+			}
110 114
 
111 115
 			$this->_generateContent($table);
112 116
 
@@ -136,8 +140,9 @@  discard block
 block discarded – undo
136 140
 	}
137 141
 
138 142
 	protected function _applyStyleAttributes($table){
139
-		if(isset($this->_hiddenColumns))
140
-			$this->_hideColumns();
143
+		if(isset($this->_hiddenColumns)) {
144
+					$this->_hideColumns();
145
+		}
141 146
 			if(isset($this->_colWidths)){
142 147
 				foreach ($this->_colWidths as $colIndex=>$width){
143 148
 					$table->setColWidth($colIndex,$width);
@@ -173,7 +178,7 @@  discard block
 block discarded – undo
173 178
 				$table->fromDatabaseObjects($objects, function($instance) use($table,$fields){
174 179
 					return $this->_generateRow($instance, $fields,$table);
175 180
 				});
176
-			}else{
181
+			} else{
177 182
 				$activeValues=array_combine($groupByFields, array_fill(0, sizeof($groupByFields), null));
178 183
 				$uuids=[];
179 184
 				$table->fromDatabaseObjects($objects, function($instance) use($table,$fields,&$activeValues,$groupByFields,&$uuids){
@@ -235,8 +240,9 @@  discard block
 block discarded – undo
235 240
 			$field=$ck->getField();
236 241
 			$field->setProperty("value",$dataAjax);
237 242
 			$field->setProperty("name", "selection[]");
238
-			if(isset($checkedClass))
239
-				$field->setClass($checkedClass);
243
+			if(isset($checkedClass)) {
244
+							$field->setClass($checkedClass);
245
+			}
240 246
 			\array_unshift($values, $ck);
241 247
 		}
242 248
 		$result=$table->newRow();
@@ -253,8 +259,9 @@  discard block
 block discarded – undo
253 259
 
254 260
 	protected function _generatePagination($table){
255 261
 		if(isset($this->_toolbar)){
256
-			if($this->_toolbarPosition==PositionInTable::FOOTER)
257
-				$this->_toolbar->setFloated("left");
262
+			if($this->_toolbarPosition==PositionInTable::FOOTER) {
263
+							$this->_toolbar->setFloated("left");
264
+			}
258 265
 		}
259 266
 		$footer=$table->getFooter();
260 267
 		$footer->mergeCol();
@@ -286,8 +293,9 @@  discard block
 block discarded – undo
286 293
 
287 294
 	protected function _getFieldName($index){
288 295
 		$fieldName=parent::_getFieldName($index);
289
-		if(\is_object($fieldName))
290
-			$fieldName="field-".$index;
296
+		if(\is_object($fieldName)) {
297
+					$fieldName="field-".$index;
298
+		}
291 299
 		return $fieldName."[]";
292 300
 	}
293 301
 
@@ -327,7 +335,7 @@  discard block
 block discarded – undo
327 335
 		$hasPart=$table->hasPart($part);
328 336
 		if($hasPart){
329 337
 			$row=$table->getPart($part)->addRow(\sizeof($captions));
330
-		}else{
338
+		} else{
331 339
 			$row=$table->getPart($part)->getRow(0);
332 340
 		}
333 341
 		$row->mergeCol();
@@ -359,7 +367,7 @@  discard block
 block discarded – undo
359 367
 			$this->_urls["edit"]=JArray::getValue($urls, "edit",1);
360 368
 			$this->_urls["delete"]=JArray::getValue($urls, "delete",2);
361 369
 			$this->_urls["display"]=JArray::getValue($urls, "display",3);
362
-		}else{
370
+		} else{
363 371
 			$this->_urls=["refresh"=>$urls,"edit"=>$urls,"delete"=>$urls,"display"=>$urls];
364 372
 		}
365 373
 		return $this;
@@ -442,8 +450,9 @@  discard block
 block discarded – undo
442 450
 
443 451
 	protected function getTargetSelector($op) {
444 452
 		$result=$this->_targetSelector;
445
-		if(!isset($result[$op]))
446
-			$result="#".$this->identifier;
453
+		if(!isset($result[$op])) {
454
+					$result="#".$this->identifier;
455
+		}
447 456
 		return $result[$op];
448 457
 	}
449 458
 
@@ -461,8 +470,9 @@  discard block
 block discarded – undo
461 470
 	}
462 471
 
463 472
 	public function getRefreshSelector() {
464
-		if(isset($this->_refreshSelector))
465
-			return $this->_refreshSelector;
473
+		if(isset($this->_refreshSelector)) {
474
+					return $this->_refreshSelector;
475
+		}
466 476
 		return "#".$this->identifier." tbody";
467 477
 	}
468 478
 
@@ -481,8 +491,9 @@  discard block
 block discarded – undo
481 491
 	 */
482 492
 	public function show($modelInstance){
483 493
 		if(\is_array($modelInstance)){
484
-			if(isset($modelInstance[0]) && \is_array(array_values($modelInstance)[0]))
485
-				$modelInstance=\json_decode(\json_encode($modelInstance), FALSE);
494
+			if(isset($modelInstance[0]) && \is_array(array_values($modelInstance)[0])) {
495
+							$modelInstance=\json_decode(\json_encode($modelInstance), FALSE);
496
+			}
486 497
 		}
487 498
 		$this->_modelInstance=$modelInstance;
488 499
 	}
@@ -522,8 +533,9 @@  discard block
 block discarded – undo
522 533
 	}
523 534
 
524 535
 	public function hideColumn($colIndex){
525
-		if(!\is_array($this->_hiddenColumns))
526
-			$this->_hiddenColumns=[];
536
+		if(!\is_array($this->_hiddenColumns)) {
537
+					$this->_hiddenColumns=[];
538
+		}
527 539
 		$this->_hiddenColumns[]=$colIndex;
528 540
 		return $this;
529 541
 	}
Please login to merge, or discard this patch.
Spacing   +145 added lines, -145 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@  discard block
 block discarded – undo
25 25
  *
26 26
  */
27 27
 class DataTable extends Widget {
28
-	use TableTrait,DataTableFieldAsTrait,HasCheckboxesTrait,BaseTrait;
28
+	use TableTrait, DataTableFieldAsTrait, HasCheckboxesTrait, BaseTrait;
29 29
 	protected $_searchField;
30 30
 	protected $_urls;
31 31
 	protected $_pagination;
@@ -44,41 +44,41 @@  discard block
 block discarded – undo
44 44
 	protected $_colWidths;
45 45
 
46 46
 
47
-	public function __construct($identifier,$model,$modelInstance=NULL) {
48
-		parent::__construct($identifier, $model,$modelInstance);
49
-		$this->_init(new InstanceViewer($identifier), "table", new HtmlTable($identifier, 0,0), false);
47
+	public function __construct($identifier, $model, $modelInstance=NULL) {
48
+		parent::__construct($identifier, $model, $modelInstance);
49
+		$this->_init(new InstanceViewer($identifier), "table", new HtmlTable($identifier, 0, 0), false);
50 50
 		$this->_urls=[];
51
-		$this->_emptyMessage=new HtmlMessage("","nothing to display");
51
+		$this->_emptyMessage=new HtmlMessage("", "nothing to display");
52 52
 		$this->_emptyMessage->setIcon("info circle");
53 53
 	}
54 54
 
55
-	public function run(JsUtils $js){
55
+	public function run(JsUtils $js) {
56 56
 		$offset=$js->scriptCount();
57
-		if($this->_hasCheckboxes && isset($js)){
57
+		if ($this->_hasCheckboxes && isset($js)) {
58 58
 			$this->_runCheckboxes($js);
59 59
 		}
60
-		if($this->_visibleHover){
61
-			$js->execOn("mouseover", "#".$this->identifier." tr", "$(event.target).closest('tr').find('.visibleover').css('visibility', 'visible');",["preventDefault"=>false,"stopPropagation"=>true]);
62
-			$js->execOn("mouseout", "#".$this->identifier." tr", "$(event.target).closest('tr').find('.visibleover').css('visibility', 'hidden');",["preventDefault"=>false,"stopPropagation"=>true]);
60
+		if ($this->_visibleHover) {
61
+			$js->execOn("mouseover", "#".$this->identifier." tr", "$(event.target).closest('tr').find('.visibleover').css('visibility', 'visible');", ["preventDefault"=>false, "stopPropagation"=>true]);
62
+			$js->execOn("mouseout", "#".$this->identifier." tr", "$(event.target).closest('tr').find('.visibleover').css('visibility', 'hidden');", ["preventDefault"=>false, "stopPropagation"=>true]);
63 63
 		}
64
-		if(\is_array($this->_deleteBehavior))
65
-			$this->_generateBehavior("delete",$this->_deleteBehavior, $js);
66
-		if(\is_array($this->_editBehavior))
67
-			$this->_generateBehavior("edit",$this->_editBehavior,$js);
68
-		if(\is_array($this->_displayBehavior)){
69
-			$this->_generateBehavior("display",$this->_displayBehavior,$js);
64
+		if (\is_array($this->_deleteBehavior))
65
+			$this->_generateBehavior("delete", $this->_deleteBehavior, $js);
66
+		if (\is_array($this->_editBehavior))
67
+			$this->_generateBehavior("edit", $this->_editBehavior, $js);
68
+		if (\is_array($this->_displayBehavior)) {
69
+			$this->_generateBehavior("display", $this->_displayBehavior, $js);
70 70
 		}
71 71
 		parent::run($js);
72
-		if(isset($this->_pagination))
73
-			$this->_associatePaginationBehavior($js,$offset);
74
-		$this->_associateSearchFieldBehavior($js,$offset);
72
+		if (isset($this->_pagination))
73
+			$this->_associatePaginationBehavior($js, $offset);
74
+		$this->_associateSearchFieldBehavior($js, $offset);
75 75
 			
76 76
 	}
77 77
 
78
-	protected function _generateBehavior($op,$params,JsUtils $js){
79
-		if(isset($this->_urls[$op])){
80
-			$params=\array_merge($params,["attr"=>"data-ajax"]);
81
-			$js->ajaxOnClick("#".$this->identifier." ._".$op, $this->_urls[$op],$this->getTargetSelector($op),$params);
78
+	protected function _generateBehavior($op, $params, JsUtils $js) {
79
+		if (isset($this->_urls[$op])) {
80
+			$params=\array_merge($params, ["attr"=>"data-ajax"]);
81
+			$js->ajaxOnClick("#".$this->identifier." ._".$op, $this->_urls[$op], $this->getTargetSelector($op), $params);
82 82
 		}
83 83
 	}
84 84
 
@@ -90,34 +90,34 @@  discard block
 block discarded – undo
90 90
 		return $this->content["table"];
91 91
 	}
92 92
 	
93
-	public function refreshTR(){
93
+	public function refreshTR() {
94 94
 		$this->getTable()->refreshTR();
95 95
 		return $this;
96 96
 	}
97 97
 	
98
-	public function refreshTD($fieldName,$jquery,$view){
98
+	public function refreshTD($fieldName, $jquery, $view) {
99 99
 		$index=$this->_getIndex($fieldName);
100
-		$this->compile($jquery,$view);
101
-		return $this->refreshTR()->getTable()->getCell(0,$index);
100
+		$this->compile($jquery, $view);
101
+		return $this->refreshTR()->getTable()->getCell(0, $index);
102 102
 	}
103 103
 	
104 104
 
105 105
 
106
-	public function compile(JsUtils $js=NULL,&$view=NULL){
107
-		if(!$this->_generated){
108
-			if(isset($this->_buttonsColumn)){
106
+	public function compile(JsUtils $js=NULL, &$view=NULL) {
107
+		if (!$this->_generated) {
108
+			if (isset($this->_buttonsColumn)) {
109 109
 				$this->_instanceViewer->sortColumnContent($this->_buttonsColumn, $this->_buttons);
110 110
 			}
111 111
 			$this->_instanceViewer->setInstance($this->_model);
112 112
 			$captions=$this->_instanceViewer->getCaptions();
113 113
 			$table=$this->content["table"];
114
-			if($this->_hasCheckboxes){
114
+			if ($this->_hasCheckboxes) {
115 115
 				$this->_generateMainCheckbox($captions);
116 116
 			}
117 117
 			$table->setRowCount(0, \sizeof($captions));
118
-			$this->_generateHeader($table,$captions);
118
+			$this->_generateHeader($table, $captions);
119 119
 
120
-			if(isset($this->_compileParts))
120
+			if (isset($this->_compileParts))
121 121
 				$table->setCompileParts($this->_compileParts);
122 122
 
123 123
 			$this->_generateContent($table);
@@ -125,158 +125,158 @@  discard block
 block discarded – undo
125 125
 			$this->compileExtraElements($table, $captions);
126 126
 			$this->_compileSearchFieldBehavior($js);
127 127
 
128
-			$this->content=JArray::sortAssociative($this->content, [PositionInTable::BEFORETABLE,"table",PositionInTable::AFTERTABLE]);
128
+			$this->content=JArray::sortAssociative($this->content, [PositionInTable::BEFORETABLE, "table", PositionInTable::AFTERTABLE]);
129 129
 			$this->_compileForm();
130 130
 			$this->_applyStyleAttributes($table);
131 131
 			$this->_generated=true;
132 132
 		}
133
-		return parent::compile($js,$view);
133
+		return parent::compile($js, $view);
134 134
 	}
135 135
 
136
-	protected function compileExtraElements($table,$captions){
136
+	protected function compileExtraElements($table, $captions) {
137 137
 
138
-		if($this->_hasCheckboxes && $table->hasPart("thead")){
138
+		if ($this->_hasCheckboxes && $table->hasPart("thead")) {
139 139
 			$table->getHeader()->getCell(0, 0)->addClass("no-sort");
140 140
 		}
141 141
 
142
-		if(isset($this->_toolbar)){
142
+		if (isset($this->_toolbar)) {
143 143
 			$this->_setToolbarPosition($table, $captions);
144 144
 		}
145
-		if(isset($this->_pagination) && $this->_pagination->getVisible()){
145
+		if (isset($this->_pagination) && $this->_pagination->getVisible()) {
146 146
 			$this->_generatePagination($table);
147 147
 		}
148 148
 	}
149 149
 
150
-	protected function _applyStyleAttributes($table){
151
-		if(isset($this->_hiddenColumns))
150
+	protected function _applyStyleAttributes($table) {
151
+		if (isset($this->_hiddenColumns))
152 152
 			$this->_hideColumns();
153
-			if(isset($this->_colWidths)){
154
-				foreach ($this->_colWidths as $colIndex=>$width){
155
-					$table->setColWidth($colIndex,$width);
153
+			if (isset($this->_colWidths)) {
154
+				foreach ($this->_colWidths as $colIndex=>$width) {
155
+					$table->setColWidth($colIndex, $width);
156 156
 				}
157 157
 			}
158 158
 	}
159 159
 
160
-	protected function _hideColumns(){
161
-		foreach ($this->_hiddenColumns as $colIndex){
160
+	protected function _hideColumns() {
161
+		foreach ($this->_hiddenColumns as $colIndex) {
162 162
 			$this->_self->hideColumn($colIndex);
163 163
 		}
164 164
 		return $this;
165 165
 	}
166 166
 
167
-	protected function _generateHeader(HtmlTable $table,$captions){
167
+	protected function _generateHeader(HtmlTable $table, $captions) {
168 168
 		$table->setHeaderValues($captions);
169
-		if(isset($this->_sortable)){
169
+		if (isset($this->_sortable)) {
170 170
 			$table->setSortable($this->_sortable);
171 171
 		}
172 172
 	}
173 173
 
174 174
 
175 175
 
176
-	protected function _generateContent($table){
176
+	protected function _generateContent($table) {
177 177
 		$objects=$this->_modelInstance;
178
-		if(isset($this->_pagination)){
178
+		if (isset($this->_pagination)) {
179 179
 			$objects=$this->_pagination->getObjects($this->_modelInstance);
180 180
 		}
181 181
 			InstanceViewer::setIndex(0);
182 182
 			$fields=$this->_instanceViewer->getSimpleProperties();
183 183
 			$groupByFields=$this->_instanceViewer->getGroupByFields();
184
-			if(!is_array($groupByFields)){
185
-				$table->fromDatabaseObjects($objects, function($instance) use($table,$fields){
186
-					return $this->_generateRow($instance, $fields,$table);
184
+			if (!is_array($groupByFields)) {
185
+				$table->fromDatabaseObjects($objects, function($instance) use($table, $fields){
186
+					return $this->_generateRow($instance, $fields, $table);
187 187
 				});
188
-			}else{
188
+			} else {
189 189
 				$activeValues=array_combine($groupByFields, array_fill(0, sizeof($groupByFields), null));
190 190
 				$uuids=[];
191
-				$table->fromDatabaseObjects($objects, function($instance) use($table,$fields,&$activeValues,$groupByFields,&$uuids){
191
+				$table->fromDatabaseObjects($objects, function($instance) use($table, $fields, &$activeValues, $groupByFields, &$uuids){
192 192
 					$this->_instanceViewer->setInstance($instance);
193
-					foreach ($groupByFields as $index=>$gbField){
194
-						$this->_generateGroupByRow($index, $gbField, $table,$fields,$activeValues, $uuids);
193
+					foreach ($groupByFields as $index=>$gbField) {
194
+						$this->_generateGroupByRow($index, $gbField, $table, $fields, $activeValues, $uuids);
195 195
 					}
196
-					return $this->_generateRow($instance, $fields,$table,null,$uuids);
196
+					return $this->_generateRow($instance, $fields, $table, null, $uuids);
197 197
 				});
198 198
 			}
199
-		if($table->getRowCount()==0){
199
+		if ($table->getRowCount()==0) {
200 200
 			$result=$table->addRow();
201 201
 			$result->mergeRow();
202 202
 			$result->setValues([$this->_emptyMessage]);
203 203
 		}
204 204
 	}
205 205
 	
206
-	protected function _generateGroupByRow($index,$gbField,$table,$fields,&$activeValues,&$uuids){
206
+	protected function _generateGroupByRow($index, $gbField, $table, $fields, &$activeValues, &$uuids) {
207 207
 		$newValue=$this->_instanceViewer->getValue($gbField);
208
-		if($this->getElementContent($activeValues[$gbField])!==$this->getElementContent($newValue)){
209
-			if($index==0){
208
+		if ($this->getElementContent($activeValues[$gbField])!==$this->getElementContent($newValue)) {
209
+			if ($index==0) {
210 210
 				$uuids=[];
211 211
 			}
212 212
 			$uuid=uniqid("grp");
213 213
 			$uuids[$gbField]=$uuid;
214 214
 			$id=$this->_instanceViewer->getIdentifier();
215
-			$result=$table->addMergeRow(sizeof($fields)+1,$newValue);
215
+			$result=$table->addMergeRow(sizeof($fields)+1, $newValue);
216 216
 			$result->setIdentifier($this->identifier."-tr-gb-".$id);
217
-			$result->setProperty("data-ajax",$id);
218
-			$result->setProperty("data-group",$uuid);
219
-			$result->addToProperty("class",$this->_rowClass);
217
+			$result->setProperty("data-ajax", $id);
218
+			$result->setProperty("data-group", $uuid);
219
+			$result->addToProperty("class", $this->_rowClass);
220 220
 			$activeValues[$gbField]=$newValue;
221 221
 		}
222 222
 	}
223 223
 	
224
-	private function getElementContent($elm){
225
-		if($elm instanceof HtmlDoubleElement){
224
+	private function getElementContent($elm) {
225
+		if ($elm instanceof HtmlDoubleElement) {
226 226
 			return $elm->getTextContent();
227 227
 		}
228 228
 		return $elm;
229 229
 	}
230 230
 	
231
-	public function getFieldValue($index){
231
+	public function getFieldValue($index) {
232 232
 		$index=$this->_getIndex($index);
233
-		if(is_numeric($index)){
234
-			$values= $this->_instanceViewer->getValues();
235
-			if(isset($values[$index])){
233
+		if (is_numeric($index)) {
234
+			$values=$this->_instanceViewer->getValues();
235
+			if (isset($values[$index])) {
236 236
 				return $values[$index];
237 237
 			}
238 238
 		}
239 239
 		return null;
240 240
 	}
241 241
 
242
-	protected function _generateRow($instance,$fields,&$table,$checkedClass=null,$uuids=null){
242
+	protected function _generateRow($instance, $fields, &$table, $checkedClass=null, $uuids=null) {
243 243
 		$this->_instanceViewer->setInstance($instance);
244 244
 		InstanceViewer::$index++;
245
-		$values= $this->_instanceViewer->getValues();
245
+		$values=$this->_instanceViewer->getValues();
246 246
 		$id=$this->_instanceViewer->getIdentifier();
247 247
 		$dataAjax=$id;
248 248
 		$id=$this->cleanIdentifier($id);
249
-		if($this->_hasCheckboxes){
250
-			$ck=new HtmlCheckbox("ck-".$this->identifier."-".$id,"");
249
+		if ($this->_hasCheckboxes) {
250
+			$ck=new HtmlCheckbox("ck-".$this->identifier."-".$id, "");
251 251
 			$checked=false;
252
-			if(isset($this->_checkedCallback)){
252
+			if (isset($this->_checkedCallback)) {
253 253
 				$func=$this->_checkedCallback;
254 254
 				$checked=$func($instance);
255 255
 			}
256 256
 			$ck->setChecked($checked);
257 257
 			$ck->setOnChange("event.stopPropagation();");
258 258
 			$field=$ck->getField();
259
-			$field->setProperty("value",$dataAjax);
259
+			$field->setProperty("value", $dataAjax);
260 260
 			$field->setProperty("name", "selection[]");
261
-			if(isset($checkedClass))
261
+			if (isset($checkedClass))
262 262
 				$field->setClass($checkedClass);
263 263
 			\array_unshift($values, $ck);
264 264
 		}
265 265
 		$result=$table->newRow();
266 266
 		$result->setIdentifier($this->identifier."-tr-".$id);
267
-		$result->setProperty("data-ajax",$dataAjax);
267
+		$result->setProperty("data-ajax", $dataAjax);
268 268
 		$result->setValues($values);
269
-		$result->addToProperty("class",$this->_rowClass);
269
+		$result->addToProperty("class", $this->_rowClass);
270 270
 		$result->setPropertyValues("data-field", $fields);
271
-		if(isset($uuids)){
272
-			$result->setProperty("data-child",implode(" ", $uuids));
271
+		if (isset($uuids)) {
272
+			$result->setProperty("data-child", implode(" ", $uuids));
273 273
 		}
274 274
 		return $result;
275 275
 	}
276 276
 
277
-	protected function _generatePagination($table){
278
-		if(isset($this->_toolbar)){
279
-			if($this->_toolbarPosition==PositionInTable::FOOTER)
277
+	protected function _generatePagination($table) {
278
+		if (isset($this->_toolbar)) {
279
+			if ($this->_toolbarPosition==PositionInTable::FOOTER)
280 280
 				$this->_toolbar->setFloated("left");
281 281
 		}
282 282
 		$footer=$table->getFooter();
@@ -284,51 +284,51 @@  discard block
 block discarded – undo
284 284
 		$footer->addValues($this->_pagination->generateMenu($this->identifier));
285 285
 	}
286 286
 
287
-	protected function _associatePaginationBehavior(JsUtils $js=NULL,$offset=null){
288
-		if(isset($this->_urls["refresh"])){
287
+	protected function _associatePaginationBehavior(JsUtils $js=NULL, $offset=null) {
288
+		if (isset($this->_urls["refresh"])) {
289 289
 			$menu=$this->_pagination->getMenu();
290
-			if(isset($menu) && isset($js)){
291
-				$js->postOnClick("#".$menu->getIdentifier()." .item",$this->_urls["refresh"],"{'p':$(this).attr('data-page'),'_model':'".JString::doubleBackSlashes($this->_model)."'}",$this->getRefreshSelector(),["preventDefault"=>false,"jqueryDone"=>"replaceWith","hasLoader"=>false,"jsCallback"=>'$("#'.$this->identifier.'").trigger("pageChange");$("#'.$this->identifier.'").trigger("activeRowChange");']);
290
+			if (isset($menu) && isset($js)) {
291
+				$js->postOnClick("#".$menu->getIdentifier()." .item", $this->_urls["refresh"], "{'p':$(this).attr('data-page'),'_model':'".JString::doubleBackSlashes($this->_model)."'}", $this->getRefreshSelector(), ["preventDefault"=>false, "jqueryDone"=>"replaceWith", "hasLoader"=>false, "jsCallback"=>'$("#'.$this->identifier.'").trigger("pageChange");$("#'.$this->identifier.'").trigger("activeRowChange");']);
292 292
 				$page=URequest::post("p");
293
-				if(isset($page)){
293
+				if (isset($page)) {
294 294
 					$js->execAtLast('$("#'.$this->getIdentifier().' .pagination").children("a.item").removeClass("active");$("#'.$this->getIdentifier().' .pagination").children("a.item[data-page='.$page.']:not(.no-active)").addClass("active");');
295 295
 				}				
296 296
 			}
297 297
 		}
298 298
 	}
299 299
 	
300
-	protected function _compileSearchFieldBehavior(JsUtils $js=NULL){
301
-		if(isset($this->_searchField) && isset($js) && isset($this->_urls["refresh"])){
302
-			$this->_searchField->postOn("change", $this->_urls["refresh"],"{'s':$(self).val(),'_model':'".JString::doubleBackSlashes($this->_model)."'}","#".$this->identifier." tbody",["preventDefault"=>false,"jqueryDone"=>"replaceWith","hasLoader"=>"internal","jsCallback"=>'$("#'.$this->identifier.'").trigger("searchTerminate",[$(self).val()]);']);
300
+	protected function _compileSearchFieldBehavior(JsUtils $js=NULL) {
301
+		if (isset($this->_searchField) && isset($js) && isset($this->_urls["refresh"])) {
302
+			$this->_searchField->postOn("change", $this->_urls["refresh"], "{'s':$(self).val(),'_model':'".JString::doubleBackSlashes($this->_model)."'}", "#".$this->identifier." tbody", ["preventDefault"=>false, "jqueryDone"=>"replaceWith", "hasLoader"=>"internal", "jsCallback"=>'$("#'.$this->identifier.'").trigger("searchTerminate",[$(self).val()]);']);
303 303
 		}
304 304
 	}
305
-	protected function _associateSearchFieldBehavior(JsUtils $js=NULL,$offset=null){
305
+	protected function _associateSearchFieldBehavior(JsUtils $js=NULL, $offset=null) {
306 306
 		
307 307
 	}
308 308
 
309
-	protected function _getFieldName($index){
309
+	protected function _getFieldName($index) {
310 310
 		$fieldName=parent::_getFieldName($index);
311
-		if(\is_object($fieldName))
311
+		if (\is_object($fieldName))
312 312
 			$fieldName="field-".$index;
313 313
 		return $fieldName."[]";
314 314
 	}
315 315
 
316
-	protected function _getFieldCaption($index){
316
+	protected function _getFieldCaption($index) {
317 317
 		return null;
318 318
 	}
319 319
 
320
-	protected function _setToolbarPosition($table,$captions=NULL){
321
-		switch ($this->_toolbarPosition){
320
+	protected function _setToolbarPosition($table, $captions=NULL) {
321
+		switch ($this->_toolbarPosition) {
322 322
 			case PositionInTable::BEFORETABLE:
323 323
 			case PositionInTable::AFTERTABLE:
324
-				if(isset($this->_compileParts)===false){
324
+				if (isset($this->_compileParts)===false) {
325 325
 					$this->content[$this->_toolbarPosition]=$this->_toolbar;
326 326
 				}
327 327
 				break;
328 328
 			case PositionInTable::HEADER:
329 329
 			case PositionInTable::FOOTER:
330 330
 			case PositionInTable::BODY:
331
-				$this->addToolbarRow($this->_toolbarPosition,$table, $captions);
331
+				$this->addToolbarRow($this->_toolbarPosition, $table, $captions);
332 332
 				break;
333 333
 		}
334 334
 	}
@@ -340,16 +340,16 @@  discard block
 block discarded – undo
340 340
 	 * @param callable $callback function called after the field compilation
341 341
 	 * @return DataTable
342 342
 	 */
343
-	public function afterCompile($index,$callback){
344
-		$this->_instanceViewer->afterCompile($index,$callback);
343
+	public function afterCompile($index, $callback) {
344
+		$this->_instanceViewer->afterCompile($index, $callback);
345 345
 		return $this;
346 346
 	}
347 347
 
348
-	private function addToolbarRow($part,$table,$captions){
348
+	private function addToolbarRow($part, $table, $captions) {
349 349
 		$hasPart=$table->hasPart($part);
350
-		if($hasPart){
350
+		if ($hasPart) {
351 351
 			$row=$table->getPart($part)->addRow(\sizeof($captions));
352
-		}else{
352
+		} else {
353 353
 			$row=$table->getPart($part)->getRow(0);
354 354
 		}
355 355
 		$row->mergeCol();
@@ -361,7 +361,7 @@  discard block
 block discarded – undo
361 361
 	 * @see Widget::getHtmlComponent()
362 362
 	 * @return HtmlTable
363 363
 	 */
364
-	public function getHtmlComponent(){
364
+	public function getHtmlComponent() {
365 365
 		return $this->content["table"];
366 366
 	}
367 367
 
@@ -376,13 +376,13 @@  discard block
 block discarded – undo
376 376
 	 * @return DataTable
377 377
 	 */
378 378
 	public function setUrls($urls) {
379
-		if(\is_array($urls)){
380
-			$this->_urls["refresh"]=JArray::getValue($urls, "refresh",0);
381
-			$this->_urls["edit"]=JArray::getValue($urls, "edit",1);
382
-			$this->_urls["delete"]=JArray::getValue($urls, "delete",2);
383
-			$this->_urls["display"]=JArray::getValue($urls, "display",3);
384
-		}else{
385
-			$this->_urls=["refresh"=>$urls,"edit"=>$urls,"delete"=>$urls,"display"=>$urls];
379
+		if (\is_array($urls)) {
380
+			$this->_urls["refresh"]=JArray::getValue($urls, "refresh", 0);
381
+			$this->_urls["edit"]=JArray::getValue($urls, "edit", 1);
382
+			$this->_urls["delete"]=JArray::getValue($urls, "delete", 2);
383
+			$this->_urls["display"]=JArray::getValue($urls, "display", 3);
384
+		} else {
385
+			$this->_urls=["refresh"=>$urls, "edit"=>$urls, "delete"=>$urls, "display"=>$urls];
386 386
 		}
387 387
 		return $this;
388 388
 	}
@@ -395,8 +395,8 @@  discard block
 block discarded – undo
395 395
 	 * @param number $pages_visibles The number of visible pages in the Pagination component
396 396
 	 * @return DataTable
397 397
 	 */
398
-	public function paginate($page,$total_rowcount,$items_per_page=10,$pages_visibles=null){
399
-		$this->_pagination=new Pagination($items_per_page,$pages_visibles,$page,$total_rowcount);
398
+	public function paginate($page, $total_rowcount, $items_per_page=10, $pages_visibles=null) {
399
+		$this->_pagination=new Pagination($items_per_page, $pages_visibles, $page, $total_rowcount);
400 400
 		return $this;
401 401
 	}
402 402
 
@@ -407,8 +407,8 @@  discard block
 block discarded – undo
407 407
 	 * @param number $pages_visibles The number of visible pages in the Pagination component
408 408
 	 * @return DataTable
409 409
 	 */
410
-	public function autoPaginate($page=1,$items_per_page=10,$pages_visibles=4){
411
-		$this->_pagination=new Pagination($items_per_page,$pages_visibles,$page);
410
+	public function autoPaginate($page=1, $items_per_page=10, $pages_visibles=4) {
411
+		$this->_pagination=new Pagination($items_per_page, $pages_visibles, $page);
412 412
 		return $this;
413 413
 	}
414 414
 
@@ -418,7 +418,7 @@  discard block
 block discarded – undo
418 418
 	 * @param array $compileParts
419 419
 	 * @return DataTable
420 420
 	 */
421
-	public function refresh($compileParts=["tbody"]){
421
+	public function refresh($compileParts=["tbody"]) {
422 422
 		$this->_compileParts=$compileParts;
423 423
 		return $this;
424 424
 	}
@@ -429,14 +429,14 @@  discard block
 block discarded – undo
429 429
 	 * @param string $position
430 430
 	 * @return \Ajax\common\html\HtmlDoubleElement
431 431
 	 */
432
-	public function addSearchInToolbar($position=Direction::RIGHT){
432
+	public function addSearchInToolbar($position=Direction::RIGHT) {
433 433
 		return $this->addInToolbar($this->getSearchField())->setPosition($position);
434 434
 	}
435 435
 
436
-	public function getSearchField(){
437
-		if(isset($this->_searchField)===false){
438
-			$this->_searchField=new HtmlInput("search-".$this->identifier,"search","","Search...");
439
-			$this->_searchField->addIcon("search",Direction::RIGHT);
436
+	public function getSearchField() {
437
+		if (isset($this->_searchField)===false) {
438
+			$this->_searchField=new HtmlInput("search-".$this->identifier, "search", "", "Search...");
439
+			$this->_searchField->addIcon("search", Direction::RIGHT);
440 440
 		}
441 441
 		return $this->_searchField;
442 442
 	}
@@ -456,7 +456,7 @@  discard block
 block discarded – undo
456 456
 	 * Returns a form corresponding to the Datatable
457 457
 	 * @return \Ajax\semantic\html\collections\form\HtmlForm
458 458
 	 */
459
-	public function asForm(){
459
+	public function asForm() {
460 460
 		return $this->getForm();
461 461
 	}
462 462
 
@@ -464,7 +464,7 @@  discard block
 block discarded – undo
464 464
 
465 465
 	protected function getTargetSelector($op) {
466 466
 		$result=$this->_targetSelector;
467
-		if(!isset($result[$op]))
467
+		if (!isset($result[$op]))
468 468
 			$result="#".$this->identifier;
469 469
 		return $result[$op];
470 470
 	}
@@ -475,15 +475,15 @@  discard block
 block discarded – undo
475 475
 	 * @return DataTable
476 476
 	 */
477 477
 	public function setTargetSelector($_targetSelector) {
478
-		if(!\is_array($_targetSelector)){
479
-			$_targetSelector=["edit"=>$_targetSelector,"delete"=>$_targetSelector];
478
+		if (!\is_array($_targetSelector)) {
479
+			$_targetSelector=["edit"=>$_targetSelector, "delete"=>$_targetSelector];
480 480
 		}
481 481
 		$this->_targetSelector=$_targetSelector;
482 482
 		return $this;
483 483
 	}
484 484
 
485 485
 	public function getRefreshSelector() {
486
-		if(isset($this->_refreshSelector))
486
+		if (isset($this->_refreshSelector))
487 487
 			return $this->_refreshSelector;
488 488
 		return "#".$this->identifier." tbody";
489 489
 	}
@@ -501,9 +501,9 @@  discard block
 block discarded – undo
501 501
 	 * {@inheritDoc}
502 502
 	 * @see \Ajax\common\Widget::show()
503 503
 	 */
504
-	public function show($modelInstance){
505
-		if(\is_array($modelInstance)){
506
-			if(isset($modelInstance[0]) && \is_array(array_values($modelInstance)[0]))
504
+	public function show($modelInstance) {
505
+		if (\is_array($modelInstance)) {
506
+			if (isset($modelInstance[0]) && \is_array(array_values($modelInstance)[0]))
507 507
 				$modelInstance=\json_decode(\json_encode($modelInstance), FALSE);
508 508
 		}
509 509
 		$this->_modelInstance=$modelInstance;
@@ -538,37 +538,37 @@  discard block
 block discarded – undo
538 538
 		return $this;
539 539
 	}
540 540
 
541
-	public function setActiveRowSelector($class="active",$event="click",$multiple=false){
542
-		$this->_self->setActiveRowSelector($class,$event,$multiple);
541
+	public function setActiveRowSelector($class="active", $event="click", $multiple=false) {
542
+		$this->_self->setActiveRowSelector($class, $event, $multiple);
543 543
 		return $this;
544 544
 	}
545 545
 
546
-	public function hideColumn($colIndex){
547
-		if(!\is_array($this->_hiddenColumns))
546
+	public function hideColumn($colIndex) {
547
+		if (!\is_array($this->_hiddenColumns))
548 548
 			$this->_hiddenColumns=[];
549 549
 		$this->_hiddenColumns[]=$colIndex;
550 550
 		return $this;
551 551
 	}
552 552
 
553
-	public function setColWidth($colIndex,$width){
553
+	public function setColWidth($colIndex, $width) {
554 554
 		$this->_colWidths[$colIndex]=$width;
555 555
 		return $this;
556 556
 	}
557 557
 	public function setColWidths($_colWidths) {
558
-		$this->_colWidths = $_colWidths;
558
+		$this->_colWidths=$_colWidths;
559 559
 		return $this;
560 560
 	}
561 561
 
562
-	public function setColAlignment($colIndex,$alignment){
563
-		$this->content["table"]->setColAlignment($colIndex,$alignment);
562
+	public function setColAlignment($colIndex, $alignment) {
563
+		$this->content["table"]->setColAlignment($colIndex, $alignment);
564 564
 		return $this;
565 565
 	}
566 566
 	
567
-	public function trigger($event,$params="[]"){
568
-		return $this->getHtmlComponent()->trigger($event,$params);
567
+	public function trigger($event, $params="[]") {
568
+		return $this->getHtmlComponent()->trigger($event, $params);
569 569
 	}
570 570
 	
571
-	public function onActiveRowChange($jsCode){
571
+	public function onActiveRowChange($jsCode) {
572 572
 		$this->getHtmlComponent()->onActiveRowChange($jsCode);
573 573
 		return $this;
574 574
 	}
@@ -596,7 +596,7 @@  discard block
 block discarded – undo
596 596
 	 * @param mixed $_displayBehavior
597 597
 	 */
598 598
 	public function setDisplayBehavior($_displayBehavior) {
599
-		$this->_displayBehavior = $_displayBehavior;
599
+		$this->_displayBehavior=$_displayBehavior;
600 600
 	}
601 601
 	/**
602 602
 	 * @return mixed
Please login to merge, or discard this patch.
Ajax/semantic/html/content/table/HtmlTableContent.php 2 patches
Braces   +28 added lines, -19 removed lines patch added patch discarded remove patch
@@ -26,8 +26,9 @@  discard block
 block discarded – undo
26 26
 	 */
27 27
 	public function __construct($identifier, $tagName="tbody", $rowCount=NULL, $colCount=NULL) {
28 28
 		parent::__construct($identifier, $tagName, "");
29
-		if (isset($rowCount) && isset($colCount))
30
-			$this->setRowCount($rowCount, $colCount);
29
+		if (isset($rowCount) && isset($colCount)) {
30
+					$this->setRowCount($rowCount, $colCount);
31
+		}
31 32
 	}
32 33
 
33 34
 	/**
@@ -112,7 +113,7 @@  discard block
 block discarded – undo
112 113
 		$row=$this->getItem($row);
113 114
 		if (isset($row) && $row instanceof HtmlCollection) {
114 115
 			$col=$row->getItem($col);
115
-		}else{
116
+		} else{
116 117
 			$col=$row;
117 118
 		}
118 119
 		return $col;
@@ -170,8 +171,9 @@  discard block
 block discarded – undo
170 171
 			$values=\array_fill(0, $count, $values);
171 172
 			$isArray=false;
172 173
 		}
173
-		if (JArray::dimension($values) == 1 && $isArray)
174
-			$values=[ $values ];
174
+		if (JArray::dimension($values) == 1 && $isArray) {
175
+					$values=[ $values ];
176
+		}
175 177
 
176 178
 		$count=\min(\sizeof($values), $count);
177 179
 
@@ -198,8 +200,9 @@  discard block
 block discarded – undo
198 200
 		$count=$this->count();
199 201
 		for($i=0; $i < $count; $i++) {
200 202
 			$cell=$this->getCell($i, $colIndex);
201
-			if($cell instanceof BaseTrait)
202
-				$cell->addVariations($variations);
203
+			if($cell instanceof BaseTrait) {
204
+							$cell->addVariations($variations);
205
+			}
203 206
 		}
204 207
 		return $this;
205 208
 	}
@@ -208,8 +211,9 @@  discard block
 block discarded – undo
208 211
 		$count=$this->count();
209 212
 		for($i=0; $i < $count; $i++) {
210 213
 			$cell=$this->getCell($i, $colIndex);
211
-			if(isset($cell))
212
-				$cell->addToProperty($name,$value);
214
+			if(isset($cell)) {
215
+							$cell->addToProperty($name,$value);
216
+			}
213 217
 		}
214 218
 		return $this;
215 219
 	}
@@ -227,8 +231,9 @@  discard block
 block discarded – undo
227 231
 		$count=$this->count();
228 232
 		for($i=0; $i < $count; $i++) {
229 233
 			$index=$this->content[$i]->getColPosition($colIndex);
230
-			if ($index !== NULL)
231
-				$this->getCell($i, $index)->$function();
234
+			if ($index !== NULL) {
235
+							$this->getCell($i, $index)->$function();
236
+			}
232 237
 		}
233 238
 		return $this;
234 239
 	}
@@ -239,8 +244,9 @@  discard block
 block discarded – undo
239 244
 			$maxRow=$this->content[$i]->count();
240 245
 			$index=$maxRow-$colIndex-1;
241 246
 			if (($cell=$this->getCell($i, $index))!== NULL){
242
-				if($cell->getColspan()==1)
243
-					$cell->$function();
247
+				if($cell->getColspan()==1) {
248
+									$cell->$function();
249
+				}
244 250
 			}
245 251
 		}
246 252
 		return $this;
@@ -284,8 +290,9 @@  discard block
 block discarded – undo
284 290
 	 */
285 291
 	public function getColCount() {
286 292
 		$result=0;
287
-		if ($this->count() > 0)
288
-			$result=$this->getItem(0)->count();
293
+		if ($this->count() > 0) {
294
+					$result=$this->getItem(0)->count();
295
+		}
289 296
 		return $result;
290 297
 	}
291 298
 
@@ -309,8 +316,9 @@  discard block
 block discarded – undo
309 316
 
310 317
 	public function toDelete($rowIndex, $colIndex){
311 318
 		$row=$this->getItem($rowIndex);
312
-		if (isset($row) === true)
313
-			$row->toDelete($colIndex);
319
+		if (isset($row) === true) {
320
+					$row->toDelete($colIndex);
321
+		}
314 322
 		return $this;
315 323
 	}
316 324
 
@@ -408,8 +416,9 @@  discard block
 block discarded – undo
408 416
 		foreach ( $rows as $row ) {
409 417
 			$cell=$row->getItem($colIndex);
410 418
 			$value=$cell->getContent();
411
-			if($functionExists)
412
-				$value=\call_user_func($function,$value);
419
+			if($functionExists) {
420
+							$value=\call_user_func($function,$value);
421
+			}
413 422
 			if($value!==$identiqual){
414 423
 				if($counter>0 && isset($cellToMerge)){
415 424
 					$cellToMerge->setRowspan($counter);
Please login to merge, or discard this patch.
Spacing   +51 added lines, -51 removed lines patch added patch discarded remove patch
@@ -14,7 +14,7 @@  discard block
 block discarded – undo
14 14
  *
15 15
  */
16 16
 class HtmlTableContent extends HtmlSemCollection {
17
-	protected $_tdTagNames=[ "thead" => "th","tbody" => "td","tfoot" => "th" ];
17
+	protected $_tdTagNames=["thead" => "th", "tbody" => "td", "tfoot" => "th"];
18 18
 	protected $_merged=false;
19 19
 
20 20
 	/**
@@ -38,7 +38,7 @@  discard block
 block discarded – undo
38 38
 	 */
39 39
 	public function setRowCount($rowCount, $colCount) {
40 40
 		$count=$this->count();
41
-		for($i=$count; $i < $rowCount; $i++) {
41
+		for ($i=$count; $i<$rowCount; $i++) {
42 42
 			$this->addItem($colCount);
43 43
 		}
44 44
 		return $this;
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
 		return $this->_tdTagNames[$this->tagName];
49 49
 	}
50 50
 	
51
-	public function refreshTR(){
51
+	public function refreshTR() {
52 52
 		$this->_template="%wrapContentBefore%%content%%wrapContentAfter%";
53 53
 	}
54 54
 
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
 		$tr=new HtmlTR("");
65 65
 		$tr->setContainer($this, $count);
66 66
 		$tr->setTdTagName($this->_tdTagNames[$this->tagName]);
67
-		if (isset($value) === true) {
67
+		if (isset($value)===true) {
68 68
 			$tr->setColCount($value);
69 69
 		}
70 70
 		return $tr;
@@ -90,10 +90,10 @@  discard block
 block discarded – undo
90 90
 		return $this->addItem($row);
91 91
 	}
92 92
 	
93
-	public function addMergeRow($colCount,$value=null){
93
+	public function addMergeRow($colCount, $value=null) {
94 94
 		$row=$this->addRow($colCount);
95 95
 		$row->mergeCol();
96
-		if(isset($value)){
96
+		if (isset($value)) {
97 97
 			$row->setValues([$value]);
98 98
 		}
99 99
 		return $row;
@@ -102,7 +102,7 @@  discard block
 block discarded – undo
102 102
 	/**
103 103
 	 * @return HtmlTR
104 104
 	 */
105
-	public function getItem($index){
105
+	public function getItem($index) {
106 106
 		return parent::getItem($index);
107 107
 	}
108 108
 
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
116 116
 		$row=$this->getItem($row);
117 117
 		if (isset($row) && $row instanceof HtmlCollection) {
118 118
 			$col=$row->getItem($col);
119
-		}else{
119
+		} else {
120 120
 			$col=$row;
121 121
 		}
122 122
 		return $col;
@@ -140,7 +140,7 @@  discard block
 block discarded – undo
140 140
 	 */
141 141
 	public function setCellValue($row, $col, $value="") {
142 142
 		$cell=$this->getCell($row, $col);
143
-		if (isset($cell) === true) {
143
+		if (isset($cell)===true) {
144 144
 			$cell->setValue($value);
145 145
 		}
146 146
 		return $this;
@@ -151,7 +151,7 @@  discard block
 block discarded – undo
151 151
 	 * @param mixed $values
152 152
 	 */
153 153
 	public function setValues($values=array()) {
154
-		return $this->_addOrSetValues($values, function(HtmlTR $row,$_values){$row->setValues($_values);});
154
+		return $this->_addOrSetValues($values, function(HtmlTR $row, $_values) {$row->setValues($_values); });
155 155
 	}
156 156
 
157 157
 	/**
@@ -159,7 +159,7 @@  discard block
 block discarded – undo
159 159
 	 * @param mixed $values
160 160
 	 */
161 161
 	public function addValues($values=array()) {
162
-		return $this->_addOrSetValues($values, function(HtmlTR $row,$_values){$row->addValues($_values);});
162
+		return $this->_addOrSetValues($values, function(HtmlTR $row, $_values) {$row->addValues($_values); });
163 163
 	}
164 164
 
165 165
 	/**
@@ -167,21 +167,21 @@  discard block
 block discarded – undo
167 167
 	 * @param mixed $values
168 168
 	 * @param callable $callback
169 169
 	 */
170
-	protected function _addOrSetValues($values,$callback) {
170
+	protected function _addOrSetValues($values, $callback) {
171 171
 		$count=$this->count();
172 172
 		$isArray=true;
173 173
 		if (!\is_array($values)) {
174 174
 			$values=\array_fill(0, $count, $values);
175 175
 			$isArray=false;
176 176
 		}
177
-		if (JArray::dimension($values) == 1 && $isArray)
178
-			$values=[ $values ];
177
+		if (JArray::dimension($values)==1 && $isArray)
178
+			$values=[$values];
179 179
 
180 180
 		$count=\min(\sizeof($values), $count);
181 181
 
182
-		for($i=0; $i < $count; $i++) {
182
+		for ($i=0; $i<$count; $i++) {
183 183
 			$row=$this->content[$i];
184
-			$callback($row,$values[$i]);
184
+			$callback($row, $values[$i]);
185 185
 		}
186 186
 		return $this;
187 187
 	}
@@ -192,7 +192,7 @@  discard block
 block discarded – undo
192 192
 			$values=\array_fill(0, $count, $values);
193 193
 		}
194 194
 		$count=\min(\sizeof($values), $count);
195
-		for($i=0; $i < $count; $i++) {
195
+		for ($i=0; $i<$count; $i++) {
196 196
 			$this->getCell($i, $colIndex)->setValue($values[$i]);
197 197
 		}
198 198
 		return $this;
@@ -200,20 +200,20 @@  discard block
 block discarded – undo
200 200
 
201 201
 	public function addColVariations($colIndex, $variations=array()) {
202 202
 		$count=$this->count();
203
-		for($i=0; $i < $count; $i++) {
203
+		for ($i=0; $i<$count; $i++) {
204 204
 			$cell=$this->getCell($i, $colIndex);
205
-			if($cell instanceof BaseTrait)
205
+			if ($cell instanceof BaseTrait)
206 206
 				$cell->addVariations($variations);
207 207
 		}
208 208
 		return $this;
209 209
 	}
210 210
 	
211
-	public function addPropertyCol($colIndex, $name,$value) {
211
+	public function addPropertyCol($colIndex, $name, $value) {
212 212
 		$count=$this->count();
213
-		for($i=0; $i < $count; $i++) {
213
+		for ($i=0; $i<$count; $i++) {
214 214
 			$cell=$this->getCell($i, $colIndex);
215
-			if(isset($cell))
216
-				$cell->addToProperty($name,$value);
215
+			if (isset($cell))
216
+				$cell->addToProperty($name, $value);
217 217
 		}
218 218
 		return $this;
219 219
 	}
@@ -229,9 +229,9 @@  discard block
 block discarded – undo
229 229
 
230 230
 	private function colAlign($colIndex, $function) {
231 231
 		$count=$this->count();
232
-		for($i=0; $i < $count; $i++) {
232
+		for ($i=0; $i<$count; $i++) {
233 233
 			$index=$this->content[$i]->getColPosition($colIndex);
234
-			if ($index !== NULL)
234
+			if ($index!==NULL)
235 235
 				$this->getCell($i, $index)->$function();
236 236
 		}
237 237
 		return $this;
@@ -239,11 +239,11 @@  discard block
 block discarded – undo
239 239
 	
240 240
 	private function colAlignFromRight($colIndex, $function) {
241 241
 		$count=$this->count();
242
-		for($i=0; $i < $count; $i++) {
242
+		for ($i=0; $i<$count; $i++) {
243 243
 			$maxRow=$this->content[$i]->count();
244 244
 			$index=$maxRow-$colIndex-1;
245
-			if (($cell=$this->getCell($i, $index))!== NULL){
246
-				if($cell->getColspan()==1)
245
+			if (($cell=$this->getCell($i, $index))!==NULL) {
246
+				if ($cell->getColspan()==1)
247 247
 					$cell->$function();
248 248
 			}
249 249
 		}
@@ -288,7 +288,7 @@  discard block
 block discarded – undo
288 288
 	 */
289 289
 	public function getColCount() {
290 290
 		$result=0;
291
-		if ($this->count() > 0)
291
+		if ($this->count()>0)
292 292
 			$result=$this->getItem(0)->count();
293 293
 		return $result;
294 294
 	}
@@ -302,7 +302,7 @@  discard block
 block discarded – undo
302 302
 	public function delete($rowIndex, $colIndex=NULL) {
303 303
 		if (isset($colIndex)) {
304 304
 			$row=$this->getItem($rowIndex);
305
-			if (isset($row) === true) {
305
+			if (isset($row)===true) {
306 306
 				$row->delete($colIndex);
307 307
 			}
308 308
 		} else {
@@ -311,9 +311,9 @@  discard block
 block discarded – undo
311 311
 		return $this;
312 312
 	}
313 313
 
314
-	public function toDelete($rowIndex, $colIndex){
314
+	public function toDelete($rowIndex, $colIndex) {
315 315
 		$row=$this->getItem($rowIndex);
316
-		if (isset($row) === true)
316
+		if (isset($row)===true)
317 317
 			$row->toDelete($colIndex);
318 318
 		return $this;
319 319
 	}
@@ -342,17 +342,17 @@  discard block
 block discarded – undo
342 342
 	 */
343 343
 	public function conditionalCellFormat($callback, $format) {
344 344
 		$rows=$this->content;
345
-		foreach ( $rows as $row ) {
345
+		foreach ($rows as $row) {
346 346
 			$row->conditionalCellFormat($callback, $format);
347 347
 		}
348 348
 		return $this;
349 349
 	}
350 350
 
351
-	public function conditionalColFormat($colIndex,$callback,$format){
351
+	public function conditionalColFormat($colIndex, $callback, $format) {
352 352
 		$rows=$this->content;
353
-		foreach ( $rows as $row ) {
353
+		foreach ($rows as $row) {
354 354
 			$cell=$row->getItem($colIndex);
355
-			$cell->conditionnalCellFormat($callback,$format);
355
+			$cell->conditionnalCellFormat($callback, $format);
356 356
 		}
357 357
 		return $this;
358 358
 	}
@@ -364,17 +364,17 @@  discard block
 block discarded – undo
364 364
 	 */
365 365
 	public function conditionalRowFormat($callback, $format) {
366 366
 		$rows=$this->content;
367
-		foreach ( $rows as $row ) {
367
+		foreach ($rows as $row) {
368 368
 			$row->conditionalRowFormat($callback, $format);
369 369
 		}
370 370
 		return $this;
371 371
 	}
372 372
 
373
-	public function hideColumn($colIndex){
373
+	public function hideColumn($colIndex) {
374 374
 		$rows=$this->content;
375
-		foreach ( $rows as $row ) {
375
+		foreach ($rows as $row) {
376 376
 			$cell=$row->getItem($colIndex);
377
-			$cell->addToProperty("style","display:none;");
377
+			$cell->addToProperty("style", "display:none;");
378 378
 		}
379 379
 		return $this;
380 380
 	}
@@ -385,7 +385,7 @@  discard block
 block discarded – undo
385 385
 	 */
386 386
 	public function applyCells($callback) {
387 387
 		$rows=$this->content;
388
-		foreach ( $rows as $row ) {
388
+		foreach ($rows as $row) {
389 389
 			$row->applyCells($callback);
390 390
 		}
391 391
 		return $this;
@@ -397,25 +397,25 @@  discard block
 block discarded – undo
397 397
 	 */
398 398
 	public function applyRows($callback) {
399 399
 		$rows=$this->content;
400
-		foreach ( $rows as $row ) {
400
+		foreach ($rows as $row) {
401 401
 			$row->apply($callback);
402 402
 		}
403 403
 		return $this;
404 404
 	}
405 405
 
406
-	public function mergeIdentiqualValues($colIndex,$function="strip_tags"){
406
+	public function mergeIdentiqualValues($colIndex, $function="strip_tags") {
407 407
 		$rows=$this->content;
408 408
 		$identiqual=null;
409 409
 		$counter=0;
410 410
 		$cellToMerge=null;
411 411
 		$functionExists=\function_exists($function);
412
-		foreach ( $rows as $row ) {
412
+		foreach ($rows as $row) {
413 413
 			$cell=$row->getItem($colIndex);
414 414
 			$value=$cell->getContent();
415
-			if($functionExists)
416
-				$value=\call_user_func($function,$value);
417
-			if($value!==$identiqual){
418
-				if($counter>0 && isset($cellToMerge)){
415
+			if ($functionExists)
416
+				$value=\call_user_func($function, $value);
417
+			if ($value!==$identiqual) {
418
+				if ($counter>0 && isset($cellToMerge)) {
419 419
 					$cellToMerge->setRowspan($counter);
420 420
 				}
421 421
 				$counter=0;
@@ -424,17 +424,17 @@  discard block
 block discarded – undo
424 424
 			}
425 425
 			$counter++;
426 426
 		}
427
-		if($counter>0 && isset($cellToMerge)){
427
+		if ($counter>0 && isset($cellToMerge)) {
428 428
 			$cellToMerge->setRowspan($counter);
429 429
 		}
430 430
 		return $this;
431 431
 	}
432 432
 
433
-	public function _isMerged(){
433
+	public function _isMerged() {
434 434
 		return $this->_merged;
435 435
 	}
436 436
 
437
-	public function _setMerged($value){
437
+	public function _setMerged($value) {
438 438
 		$this->_merged=$value;
439 439
 		return $this;
440 440
 	}
Please login to merge, or discard this patch.
Ajax/semantic/widgets/base/InstanceViewer.php 2 patches
Spacing   +115 added lines, -115 removed lines patch added patch discarded remove patch
@@ -21,50 +21,50 @@  discard block
 block discarded – undo
21 21
 
22 22
 	public static $index=0;
23 23
 
24
-	public function __construct($identifier,$instance=NULL,$captions=NULL){
24
+	public function __construct($identifier, $instance=NULL, $captions=NULL) {
25 25
 		$this->widgetIdentifier=$identifier;
26 26
 		$this->values=[];
27 27
 		$this->afterCompile=[];
28
-		if(isset($instance))
28
+		if (isset($instance))
29 29
 			$this->setInstance($instance);
30 30
 		$this->setCaptions($captions);
31 31
 		$this->captionCallback=NULL;
32
-		$this->defaultValueFunction=function($name,$value){return $value;};
32
+		$this->defaultValueFunction=function($name, $value) {return $value; };
33 33
 	}
34 34
 
35
-	public function moveFieldTo($from,$to){
36
-		if(JArray::moveElementTo($this->visibleProperties, $from, $to)){
35
+	public function moveFieldTo($from, $to) {
36
+		if (JArray::moveElementTo($this->visibleProperties, $from, $to)) {
37 37
 			return JArray::moveElementTo($this->values, $from, $to);
38 38
 		}
39 39
 		return false;
40 40
 	}
41 41
 
42
-	public function swapFields($index1,$index2){
43
-		if(JArray::swapElements($this->visibleProperties, $index1, $index2)){
42
+	public function swapFields($index1, $index2) {
43
+		if (JArray::swapElements($this->visibleProperties, $index1, $index2)) {
44 44
 			return JArray::swapElements($this->values, $index1, $index2);
45 45
 		}
46 46
 		return false;
47 47
 	}
48 48
 
49
-	public function removeField($index){
50
-		\array_splice($this->visibleProperties,$index,1);
51
-		\array_splice($this->values,$index,1);
52
-		\array_splice($this->captions,$index,1);
49
+	public function removeField($index) {
50
+		\array_splice($this->visibleProperties, $index, 1);
51
+		\array_splice($this->values, $index, 1);
52
+		\array_splice($this->captions, $index, 1);
53 53
 		return $this;
54 54
 	}
55 55
 
56
-	public function getValues(){
56
+	public function getValues() {
57 57
 		$values=[];
58 58
 		$index=0;
59 59
 		$count=$this->count();
60 60
 		$hasGroupby=is_array($this->groupByFields);
61
-		if(!$hasGroupby){
62
-			while($index<$count){
61
+		if (!$hasGroupby) {
62
+			while ($index<$count) {
63 63
 					$values[]=$this->getValue($index++);
64 64
 			}
65
-		}else{
66
-			while($index<$count){
67
-				if(array_search($index, $this->groupByFields)===false){
65
+		} else {
66
+			while ($index<$count) {
67
+				if (array_search($index, $this->groupByFields)===false) {
68 68
 					$values[]=$this->getValue($index);
69 69
 				}
70 70
 				$index++;
@@ -73,203 +73,203 @@  discard block
 block discarded – undo
73 73
 		return $values;
74 74
 	}
75 75
 
76
-	public function getIdentifier($index=NULL){
77
-		if(!isset($index))
76
+	public function getIdentifier($index=NULL) {
77
+		if (!isset($index))
78 78
 			$index=self::$index;
79 79
 		$value=$index;
80
-		if(isset($this->values["identifier"])){
81
-			if(\is_string($this->values["identifier"]))
80
+		if (isset($this->values["identifier"])) {
81
+			if (\is_string($this->values["identifier"]))
82 82
 				$value=JReflection::callMethod($this->instance, $this->values["identifier"], []);
83 83
 			else
84
-				$value=$this->values["identifier"]($index,$this->instance);
84
+				$value=$this->values["identifier"]($index, $this->instance);
85 85
 		}
86 86
 		return $value;
87 87
 	}
88 88
 
89
-	public function getValue($index){
89
+	public function getValue($index) {
90 90
 		$property=$this->properties[$index];
91 91
 		return $this->_getValue($property, $index);
92 92
 	}
93 93
 
94
-	protected function _beforeAddProperty($index,&$field){
94
+	protected function _beforeAddProperty($index, &$field) {
95 95
 
96 96
 	}
97 97
 
98
-	protected function _getDefaultValue($name,$value,$index){
98
+	protected function _getDefaultValue($name, $value, $index) {
99 99
 		$func=$this->defaultValueFunction;
100
-		return $func($name,$value,$index,$this->instance);
100
+		return $func($name, $value, $index, $this->instance);
101 101
 	}
102 102
 
103
-	protected function _getPropertyValue(\ReflectionProperty $property){
103
+	protected function _getPropertyValue(\ReflectionProperty $property) {
104 104
 		$property->setAccessible(true);
105 105
 		return $property->getValue($this->instance);
106 106
 	}
107 107
 
108
-	protected function _getValue($property,$index){
108
+	protected function _getValue($property, $index) {
109 109
 		$value=null;
110 110
 		$propertyName=$property;
111
-		if($property instanceof \ReflectionProperty){
111
+		if ($property instanceof \ReflectionProperty) {
112 112
 			$value=$this->_getPropertyValue($property);
113 113
 			$propertyName=$property->getName();
114
-		}elseif(\is_callable($property) && array_search($property, ["system"])===false)
114
+		}elseif (\is_callable($property) && array_search($property, ["system"])===false)
115 115
 			$value=$property($this->instance);
116
-		elseif(\is_array($property)){
117
-			$values=\array_map(function($v) use ($index){return $this->_getValue($v, $index);}, $property);
116
+		elseif (\is_array($property)) {
117
+			$values=\array_map(function($v) use ($index){return $this->_getValue($v, $index); }, $property);
118 118
 			$value=\implode("", $values);
119
-		}elseif(\is_string($property)){
119
+		}elseif (\is_string($property)) {
120 120
 			$value=$property;
121
-			if(isset($this->instance->{$property})){
121
+			if (isset($this->instance->{$property})) {
122 122
 				$value=$this->instance->{$property};
123
-			}elseif(\method_exists($this->instance, $getter=JReflection::getterName($property))){
123
+			}elseif (\method_exists($this->instance, $getter=JReflection::getterName($property))) {
124 124
 				$value=JReflection::callMethod($this->instance, $getter, []);
125 125
 			}
126 126
 		}
127 127
 		return $this->_postGetValue($index, $propertyName, $value);
128 128
 	}
129 129
 
130
-	protected function _postGetValue($index,$propertyName,$value){
131
-		if(isset($this->values[$index])){
132
-			$value= $this->values[$index]($value,$this->instance,$index,self::$index);
133
-		}else{
134
-			$value=$this->_getDefaultValue($propertyName,$value, $index);
130
+	protected function _postGetValue($index, $propertyName, $value) {
131
+		if (isset($this->values[$index])) {
132
+			$value=$this->values[$index]($value, $this->instance, $index, self::$index);
133
+		} else {
134
+			$value=$this->_getDefaultValue($propertyName, $value, $index);
135 135
 		}
136
-		if(isset($this->afterCompile[$index])){
137
-			if(\is_callable($this->afterCompile[$index])){
138
-				$this->afterCompile[$index]($value,$this->instance,self::$index);
136
+		if (isset($this->afterCompile[$index])) {
137
+			if (\is_callable($this->afterCompile[$index])) {
138
+				$this->afterCompile[$index]($value, $this->instance, self::$index);
139 139
 			}
140 140
 		}
141 141
 		return $value;
142 142
 	}
143 143
 
144
-	public function insertField($index,$field,$key=null){
145
-		if(isset($key)){
146
-			array_splice( $this->visibleProperties, $index, 0, [$key=>$field] );
147
-		}else{
148
-			array_splice( $this->visibleProperties, $index, 0, $field );
144
+	public function insertField($index, $field, $key=null) {
145
+		if (isset($key)) {
146
+			array_splice($this->visibleProperties, $index, 0, [$key=>$field]);
147
+		} else {
148
+			array_splice($this->visibleProperties, $index, 0, $field);
149 149
 		}
150 150
 		return $this;
151 151
 	}
152 152
 	
153
-	public function sortColumnContent($index,$array){
154
-		if(isset($this->visibleProperties[$index])){
155
-			if(is_array($this->visibleProperties[$index])){
156
-				$this->visibleProperties[$index]=JArray::sortAssociative($this->visibleProperties[$index],$array);
153
+	public function sortColumnContent($index, $array) {
154
+		if (isset($this->visibleProperties[$index])) {
155
+			if (is_array($this->visibleProperties[$index])) {
156
+				$this->visibleProperties[$index]=JArray::sortAssociative($this->visibleProperties[$index], $array);
157 157
 			}
158 158
 		}
159 159
 		return $this;
160 160
 	}
161 161
 
162
-	public function insertInField($index,$field,$key=null){
162
+	public function insertInField($index, $field, $key=null) {
163 163
 		$vb=$this->visibleProperties;
164
-		if(isset($vb[$index])){
165
-			if(isset($key)){
166
-				if(\is_array($vb[$index])){
164
+		if (isset($vb[$index])) {
165
+			if (isset($key)) {
166
+				if (\is_array($vb[$index])) {
167 167
 					$this->visibleProperties[$index][$key]=$field;
168
-				}else{
169
-					$this->visibleProperties[$index]=[$vb[$index],$key=>$field];
168
+				} else {
169
+					$this->visibleProperties[$index]=[$vb[$index], $key=>$field];
170 170
 				}
171
-			}else{
172
-				if(\is_array($vb[$index])){
171
+			} else {
172
+				if (\is_array($vb[$index])) {
173 173
 					$this->visibleProperties[$index][]=$field;
174
-				}else{
175
-					$this->visibleProperties[$index]=[$vb[$index],$field];
174
+				} else {
175
+					$this->visibleProperties[$index]=[$vb[$index], $field];
176 176
 				}
177 177
 			}
178
-		}else{
178
+		} else {
179 179
 			return $this->insertField($index, $field);
180 180
 		}
181 181
 		return $this;
182 182
 	}
183 183
 
184
-	public function addField($field,$key=null){
185
-		if(isset($key)){
184
+	public function addField($field, $key=null) {
185
+		if (isset($key)) {
186 186
 			$this->visibleProperties[]=[$key=>$field];
187
-		}else{
187
+		} else {
188 188
 			$this->visibleProperties[]=$field;
189 189
 		}
190 190
 		return $this;
191 191
 	}
192 192
 
193
-	public function addFields($fields){
194
-		$this->visibleProperties=\array_merge($this->visibleProperties,$fields);
193
+	public function addFields($fields) {
194
+		$this->visibleProperties=\array_merge($this->visibleProperties, $fields);
195 195
 		return $this;
196 196
 	}
197 197
 
198
-	public function count(){
198
+	public function count() {
199 199
 		return \sizeof($this->properties);
200 200
 	}
201 201
 
202
-	public function visiblePropertiesCount(){
202
+	public function visiblePropertiesCount() {
203 203
 		return \sizeof($this->visibleProperties);
204 204
 	}
205 205
 
206
-	public function getProperty($index){
206
+	public function getProperty($index) {
207 207
 		return $this->properties[$index];
208 208
 	}
209 209
 
210
-	public function getFieldName($index){
210
+	public function getFieldName($index) {
211 211
 		$property=$this->getProperty($index);
212
-		if($property instanceof \ReflectionProperty){
212
+		if ($property instanceof \ReflectionProperty) {
213 213
 			$result=$property->getName();
214
-		}elseif(\is_callable($property)){
214
+		}elseif (\is_callable($property)) {
215 215
 			$result=$this->visibleProperties[$index];
216
-		}else{
216
+		} else {
217 217
 			$result=$property;
218 218
 		}
219 219
 		return $result;
220 220
 	}
221 221
 
222 222
 
223
-	protected function showableProperty(\ReflectionProperty $rProperty){
224
-		return JString::startswith($rProperty->getName(),"_")===false;
223
+	protected function showableProperty(\ReflectionProperty $rProperty) {
224
+		return JString::startswith($rProperty->getName(), "_")===false;
225 225
 	}
226 226
 
227 227
 	public function setInstance($instance) {
228
-		if(\is_string($instance)){
228
+		if (\is_string($instance)) {
229 229
 			$instance=new $instance();
230 230
 		}
231 231
 		$this->instance=$instance;
232 232
 		$this->properties=[];
233 233
 		$this->reflect=new \ReflectionClass($instance);
234
-		if(JArray::count($this->visibleProperties)===0){
234
+		if (JArray::count($this->visibleProperties)===0) {
235 235
 			$this->properties=$this->getDefaultProperties();
236
-		}else{
237
-			foreach ($this->visibleProperties as $property){
236
+		} else {
237
+			foreach ($this->visibleProperties as $property) {
238 238
 				$this->setInstanceProperty($property);
239 239
 			}
240 240
 		}
241 241
 		return $this;
242 242
 	}
243 243
 
244
-	private function setInstanceProperty($property){
245
-		if(\is_callable($property)){
244
+	private function setInstanceProperty($property) {
245
+		if (\is_callable($property)) {
246 246
 			$this->properties[]=$property;
247
-		}elseif(\is_string($property)){
248
-			try{
247
+		}elseif (\is_string($property)) {
248
+			try {
249 249
 				$this->_beforeAddProperty(\sizeof($this->properties), $property);
250 250
 				$rProperty=$this->reflect->getProperty($property);
251 251
 				$this->properties[]=$rProperty;
252
-			}catch(\Exception $e){
252
+			}catch (\Exception $e) {
253 253
 				$this->_beforeAddProperty(\sizeof($this->properties), $property);
254 254
 				$this->properties[]=$property;
255 255
 			}
256
-		}elseif(\is_int($property)){
256
+		}elseif (\is_int($property)) {
257 257
 			$props=$this->getDefaultProperties();
258
-			if(isset($props[$property]))
258
+			if (isset($props[$property]))
259 259
 				$this->properties[]=$props[$property];
260 260
 				else
261 261
 					$this->properties[]=$property;
262
-		}else{
262
+		} else {
263 263
 			$this->properties[]=$property;
264 264
 		}
265 265
 	}
266 266
 
267
-	protected function getDefaultProperties(){
267
+	protected function getDefaultProperties() {
268 268
 		$result=[];
269 269
 		$properties=$this->reflect->getProperties();
270
-		foreach ($properties as $property){
270
+		foreach ($properties as $property) {
271 271
 			$showable=$this->showableProperty($property);
272
-			if($showable!==false){
272
+			if ($showable!==false) {
273 273
 				$result[]=$property;
274 274
 			}
275 275
 		}
@@ -281,12 +281,12 @@  discard block
 block discarded – undo
281 281
 		return $this;
282 282
 	}
283 283
 
284
-	public function setValueFunction($index,$callback){
284
+	public function setValueFunction($index, $callback) {
285 285
 		$this->values[$index]=$callback;
286 286
 		return $this;
287 287
 	}
288 288
 
289
-	public function setIdentifierFunction($callback){
289
+	public function setIdentifierFunction($callback) {
290 290
 		$this->values["identifier"]=$callback;
291 291
 		return $this;
292 292
 	}
@@ -299,49 +299,49 @@  discard block
 block discarded – undo
299 299
 		return $this->properties;
300 300
 	}
301 301
 
302
-	public function getCaption($index){
303
-		if(isset($this->captions[$index])){
302
+	public function getCaption($index) {
303
+		if (isset($this->captions[$index])) {
304 304
 			return $this->captions[$index];
305 305
 		}
306
-		if($this->properties[$index] instanceof \ReflectionProperty)
306
+		if ($this->properties[$index] instanceof \ReflectionProperty)
307 307
 			return $this->properties[$index]->getName();
308
-		elseif(\is_callable($this->properties[$index]))
308
+		elseif (\is_callable($this->properties[$index]))
309 309
 			return "";
310 310
 		else
311 311
 			return $this->properties[$index];
312 312
 	}
313 313
 
314
-	public function getCaptions(){
314
+	public function getCaptions() {
315 315
 		$hasGroupby=is_array($this->groupByFields);
316 316
 		$count=$this->count();
317 317
 		$moreAdded=false;
318
-		if(isset($this->captions)){
319
-			$captions= \array_values($this->captions);
320
-			$gbSize=$hasGroupby?sizeof($this->groupByFields):0;
318
+		if (isset($this->captions)) {
319
+			$captions=\array_values($this->captions);
320
+			$gbSize=$hasGroupby ?sizeof($this->groupByFields) : 0;
321 321
 			$captionsSize=\sizeof($captions);
322
-			for($i=$captionsSize;$i<$count-$gbSize;$i++){
322
+			for ($i=$captionsSize; $i<$count-$gbSize; $i++) {
323 323
 				$captions[]="";
324 324
 				$moreAdded=true;
325 325
 			}
326
-		}else{
326
+		} else {
327 327
 			$captions=[];
328 328
 			$index=0;
329
-			while($index<$count){
329
+			while ($index<$count) {
330 330
 					$captions[]=$this->getCaption($index++);
331 331
 			}
332 332
 		}
333
-		if($hasGroupby && sizeof($captions)>=$count && !$moreAdded){
333
+		if ($hasGroupby && sizeof($captions)>=$count && !$moreAdded) {
334 334
 			$captions=UArray::removeByKeys($captions, $this->groupByFields);
335 335
 		}
336
-		if(isset($this->captionCallback) && \is_callable($this->captionCallback)){
336
+		if (isset($this->captionCallback) && \is_callable($this->captionCallback)) {
337 337
 			$callback=$this->captionCallback;
338
-			$callback($captions,$this->instance);
338
+			$callback($captions, $this->instance);
339 339
 		}
340 340
 		return $captions;
341 341
 	}
342 342
 
343
-	public function setCaption($index,$caption){
344
-		if(isset($this->captions)===false)
343
+	public function setCaption($index, $caption) {
344
+		if (isset($this->captions)===false)
345 345
 			$this->captions=[];
346 346
 		$this->captions[$index]=$caption;
347 347
 		return $this;
@@ -359,7 +359,7 @@  discard block
 block discarded – undo
359 359
 	 * @param callable $callback function called after the field compilation
360 360
 	 * @return InstanceViewer
361 361
 	 */
362
-	public function afterCompile($index,$callback){
362
+	public function afterCompile($index, $callback) {
363 363
 		$this->afterCompile[$index]=$callback;
364 364
 		return $this;
365 365
 	}
@@ -390,7 +390,7 @@  discard block
 block discarded – undo
390 390
 	}
391 391
 	
392 392
 	public function getSimpleProperties() {
393
-		return array_filter($this->visibleProperties,function($item){
393
+		return array_filter($this->visibleProperties, function($item) {
394 394
 			return !(is_array($item) || is_object($item));
395 395
 		});
396 396
 	}
@@ -412,11 +412,11 @@  discard block
 block discarded – undo
412 412
 	 * @param mixed $groupByFields
413 413
 	 */
414 414
 	public function setGroupByFields($groupByFields) {
415
-		$this->groupByFields = $groupByFields;
415
+		$this->groupByFields=$groupByFields;
416 416
 	}
417 417
 	
418
-	public function getGroupByFieldsCount(){
419
-		if(is_array($this->groupByFields)){
418
+	public function getGroupByFieldsCount() {
419
+		if (is_array($this->groupByFields)) {
420 420
 			return sizeof($this->groupByFields);
421 421
 		}
422 422
 		return 0;
Please login to merge, or discard this patch.
Braces   +47 added lines, -41 removed lines patch added patch discarded remove patch
@@ -25,8 +25,9 @@  discard block
 block discarded – undo
25 25
 		$this->widgetIdentifier=$identifier;
26 26
 		$this->values=[];
27 27
 		$this->afterCompile=[];
28
-		if(isset($instance))
29
-			$this->setInstance($instance);
28
+		if(isset($instance)) {
29
+					$this->setInstance($instance);
30
+		}
30 31
 		$this->setCaptions($captions);
31 32
 		$this->captionCallback=NULL;
32 33
 		$this->defaultValueFunction=function($name,$value){return $value;};
@@ -62,7 +63,7 @@  discard block
 block discarded – undo
62 63
 			while($index<$count){
63 64
 					$values[]=$this->getValue($index++);
64 65
 			}
65
-		}else{
66
+		} else{
66 67
 			while($index<$count){
67 68
 				if(array_search($index, $this->groupByFields)===false){
68 69
 					$values[]=$this->getValue($index);
@@ -74,14 +75,16 @@  discard block
 block discarded – undo
74 75
 	}
75 76
 
76 77
 	public function getIdentifier($index=NULL){
77
-		if(!isset($index))
78
-			$index=self::$index;
78
+		if(!isset($index)) {
79
+					$index=self::$index;
80
+		}
79 81
 		$value=$index;
80 82
 		if(isset($this->values["identifier"])){
81
-			if(\is_string($this->values["identifier"]))
82
-				$value=JReflection::callMethod($this->instance, $this->values["identifier"], []);
83
-			else
84
-				$value=$this->values["identifier"]($index,$this->instance);
83
+			if(\is_string($this->values["identifier"])) {
84
+							$value=JReflection::callMethod($this->instance, $this->values["identifier"], []);
85
+			} else {
86
+							$value=$this->values["identifier"]($index,$this->instance);
87
+			}
85 88
 		}
86 89
 		return $value;
87 90
 	}
@@ -111,16 +114,16 @@  discard block
 block discarded – undo
111 114
 		if($property instanceof \ReflectionProperty){
112 115
 			$value=$this->_getPropertyValue($property);
113 116
 			$propertyName=$property->getName();
114
-		}elseif(\is_callable($property) && array_search($property, ["system"])===false)
115
-			$value=$property($this->instance);
116
-		elseif(\is_array($property)){
117
+		} elseif(\is_callable($property) && array_search($property, ["system"])===false) {
118
+					$value=$property($this->instance);
119
+		} elseif(\is_array($property)){
117 120
 			$values=\array_map(function($v) use ($index){return $this->_getValue($v, $index);}, $property);
118 121
 			$value=\implode("", $values);
119
-		}elseif(\is_string($property)){
122
+		} elseif(\is_string($property)){
120 123
 			$value=$property;
121 124
 			if(isset($this->instance->{$property})){
122 125
 				$value=$this->instance->{$property};
123
-			}elseif(\method_exists($this->instance, $getter=JReflection::getterName($property))){
126
+			} elseif(\method_exists($this->instance, $getter=JReflection::getterName($property))){
124 127
 				$value=JReflection::callMethod($this->instance, $getter, []);
125 128
 			}
126 129
 		}
@@ -130,7 +133,7 @@  discard block
 block discarded – undo
130 133
 	protected function _postGetValue($index,$propertyName,$value){
131 134
 		if(isset($this->values[$index])){
132 135
 			$value= $this->values[$index]($value,$this->instance,$index,self::$index);
133
-		}else{
136
+		} else{
134 137
 			$value=$this->_getDefaultValue($propertyName,$value, $index);
135 138
 		}
136 139
 		if(isset($this->afterCompile[$index])){
@@ -144,7 +147,7 @@  discard block
 block discarded – undo
144 147
 	public function insertField($index,$field,$key=null){
145 148
 		if(isset($key)){
146 149
 			array_splice( $this->visibleProperties, $index, 0, [$key=>$field] );
147
-		}else{
150
+		} else{
148 151
 			array_splice( $this->visibleProperties, $index, 0, $field );
149 152
 		}
150 153
 		return $this;
@@ -165,17 +168,17 @@  discard block
 block discarded – undo
165 168
 			if(isset($key)){
166 169
 				if(\is_array($vb[$index])){
167 170
 					$this->visibleProperties[$index][$key]=$field;
168
-				}else{
171
+				} else{
169 172
 					$this->visibleProperties[$index]=[$vb[$index],$key=>$field];
170 173
 				}
171
-			}else{
174
+			} else{
172 175
 				if(\is_array($vb[$index])){
173 176
 					$this->visibleProperties[$index][]=$field;
174
-				}else{
177
+				} else{
175 178
 					$this->visibleProperties[$index]=[$vb[$index],$field];
176 179
 				}
177 180
 			}
178
-		}else{
181
+		} else{
179 182
 			return $this->insertField($index, $field);
180 183
 		}
181 184
 		return $this;
@@ -184,7 +187,7 @@  discard block
 block discarded – undo
184 187
 	public function addField($field,$key=null){
185 188
 		if(isset($key)){
186 189
 			$this->visibleProperties[]=[$key=>$field];
187
-		}else{
190
+		} else{
188 191
 			$this->visibleProperties[]=$field;
189 192
 		}
190 193
 		return $this;
@@ -211,9 +214,9 @@  discard block
 block discarded – undo
211 214
 		$property=$this->getProperty($index);
212 215
 		if($property instanceof \ReflectionProperty){
213 216
 			$result=$property->getName();
214
-		}elseif(\is_callable($property)){
217
+		} elseif(\is_callable($property)){
215 218
 			$result=$this->visibleProperties[$index];
216
-		}else{
219
+		} else{
217 220
 			$result=$property;
218 221
 		}
219 222
 		return $result;
@@ -233,7 +236,7 @@  discard block
 block discarded – undo
233 236
 		$this->reflect=new \ReflectionClass($instance);
234 237
 		if(JArray::count($this->visibleProperties)===0){
235 238
 			$this->properties=$this->getDefaultProperties();
236
-		}else{
239
+		} else{
237 240
 			foreach ($this->visibleProperties as $property){
238 241
 				$this->setInstanceProperty($property);
239 242
 			}
@@ -244,22 +247,23 @@  discard block
 block discarded – undo
244 247
 	private function setInstanceProperty($property){
245 248
 		if(\is_callable($property)){
246 249
 			$this->properties[]=$property;
247
-		}elseif(\is_string($property)){
250
+		} elseif(\is_string($property)){
248 251
 			try{
249 252
 				$this->_beforeAddProperty(\sizeof($this->properties), $property);
250 253
 				$rProperty=$this->reflect->getProperty($property);
251 254
 				$this->properties[]=$rProperty;
252
-			}catch(\Exception $e){
255
+			} catch(\Exception $e){
253 256
 				$this->_beforeAddProperty(\sizeof($this->properties), $property);
254 257
 				$this->properties[]=$property;
255 258
 			}
256
-		}elseif(\is_int($property)){
259
+		} elseif(\is_int($property)){
257 260
 			$props=$this->getDefaultProperties();
258
-			if(isset($props[$property]))
259
-				$this->properties[]=$props[$property];
260
-				else
261
-					$this->properties[]=$property;
262
-		}else{
261
+			if(isset($props[$property])) {
262
+							$this->properties[]=$props[$property];
263
+			} else {
264
+									$this->properties[]=$property;
265
+				}
266
+		} else{
263 267
 			$this->properties[]=$property;
264 268
 		}
265 269
 	}
@@ -303,12 +307,13 @@  discard block
 block discarded – undo
303 307
 		if(isset($this->captions[$index])){
304 308
 			return $this->captions[$index];
305 309
 		}
306
-		if($this->properties[$index] instanceof \ReflectionProperty)
307
-			return $this->properties[$index]->getName();
308
-		elseif(\is_callable($this->properties[$index]))
309
-			return "";
310
-		else
311
-			return $this->properties[$index];
310
+		if($this->properties[$index] instanceof \ReflectionProperty) {
311
+					return $this->properties[$index]->getName();
312
+		} elseif(\is_callable($this->properties[$index])) {
313
+					return "";
314
+		} else {
315
+					return $this->properties[$index];
316
+		}
312 317
 	}
313 318
 
314 319
 	public function getCaptions(){
@@ -323,7 +328,7 @@  discard block
 block discarded – undo
323 328
 				$captions[]="";
324 329
 				$moreAdded=true;
325 330
 			}
326
-		}else{
331
+		} else{
327 332
 			$captions=[];
328 333
 			$index=0;
329 334
 			while($index<$count){
@@ -341,8 +346,9 @@  discard block
 block discarded – undo
341 346
 	}
342 347
 
343 348
 	public function setCaption($index,$caption){
344
-		if(isset($this->captions)===false)
345
-			$this->captions=[];
349
+		if(isset($this->captions)===false) {
350
+					$this->captions=[];
351
+		}
346 352
 		$this->captions[$index]=$caption;
347 353
 		return $this;
348 354
 	}
Please login to merge, or discard this patch.
Ajax/semantic/html/collections/menus/HtmlMenu.php 2 patches
Braces   +22 added lines, -17 removed lines patch added patch discarded remove patch
@@ -62,9 +62,9 @@  discard block
 block discarded – undo
62 62
 	}
63 63
 
64 64
 	private function afterInsert($item) {
65
-		if (!$item instanceof HtmlMenu && $item->propertyContains("class", "header")===false)
66
-			$item->addToPropertyCtrl("class", "item", array ("item" ));
67
-		else {
65
+		if (!$item instanceof HtmlMenu && $item->propertyContains("class", "header")===false) {
66
+					$item->addToPropertyCtrl("class", "item", array ("item" ));
67
+		} else {
68 68
 			$this->setSecondary();
69 69
 		}
70 70
 		return $item;
@@ -79,8 +79,9 @@  discard block
 block discarded – undo
79 79
 	public function addItem($item) {
80 80
 		$number=$item;
81 81
 		$item=parent::addItem($this->getItemToInsert($item));
82
-		if(\is_int($number))
83
-			$item->setProperty("data-page", $number);
82
+		if(\is_int($number)) {
83
+					$item->setProperty("data-page", $number);
84
+		}
84 85
 		return $this->afterInsert($item);
85 86
 	}
86 87
 
@@ -149,10 +150,11 @@  discard block
 block discarded – undo
149 150
 	}
150 151
 
151 152
 	public function setSecondary($value=true) {
152
-		if($value)
153
-			$this->addToProperty("class", "secondary");
154
-		else
155
-			$this->removePropertyValue("class", "secondary");
153
+		if($value) {
154
+					$this->addToProperty("class", "secondary");
155
+		} else {
156
+					$this->removePropertyValue("class", "secondary");
157
+		}
156 158
 		return $this;
157 159
 	}
158 160
 
@@ -172,8 +174,9 @@  discard block
 block discarded – undo
172 174
 		$this->apply(function (HtmlDoubleElement &$item) {
173 175
 			$item->setTagName("a");
174 176
 		});
175
-		if ($vertical === true)
176
-			$this->setVertical();
177
+		if ($vertical === true) {
178
+					$this->setVertical();
179
+		}
177 180
 		return $this->addToProperty("class", "tabular");
178 181
 	}
179 182
 
@@ -202,10 +205,11 @@  discard block
 block discarded – undo
202 205
 	 */
203 206
 	public function fromDatabaseObject($object, $function) {
204 207
 		$return=$function($object);
205
-		if (\is_array($return))
206
-			$this->addItems($return);
207
-		else
208
-			$this->addItem($return);
208
+		if (\is_array($return)) {
209
+					$this->addItems($return);
210
+		} else {
211
+					$this->addItem($return);
212
+		}
209 213
 	}
210 214
 
211 215
 	/**
@@ -238,8 +242,9 @@  discard block
 block discarded – undo
238 242
 	}
239 243
 
240 244
 	public function run(JsUtils $js){
241
-		if($this->identifier!=="" && !isset($this->_bsComponent))
242
-			$this->onClick('if(!$(this).hasClass("dropdown")&&!$(this).hasClass("no-active")){$(this).addClass("active").siblings().removeClass("active");}',false,false);
245
+		if($this->identifier!=="" && !isset($this->_bsComponent)) {
246
+					$this->onClick('if(!$(this).hasClass("dropdown")&&!$(this).hasClass("no-active")){$(this).addClass("active").siblings().removeClass("active");}',false,false);
247
+		}
243 248
 		$result= parent::run($js);
244 249
 		return $result->setItemSelector(">.item");
245 250
 	}
Please login to merge, or discard this patch.
Spacing   +27 added lines, -27 removed lines patch added patch discarded remove patch
@@ -41,12 +41,12 @@  discard block
 block discarded – undo
41 41
 	 * @return HtmlMenu
42 42
 	 */
43 43
 	public function setType($type="") {
44
-		return $this->addToPropertyCtrl("class", $type, array ("","item","text" ));
44
+		return $this->addToPropertyCtrl("class", $type, array("", "item", "text"));
45 45
 	}
46 46
 
47 47
 	public function setActiveItem($index) {
48 48
 		$item=$this->getItem($index);
49
-		if ($item !== null) {
49
+		if ($item!==null) {
50 50
 			$item->addToProperty("class", "active");
51 51
 		}
52 52
 		return $this;
@@ -54,7 +54,7 @@  discard block
 block discarded – undo
54 54
 
55 55
 	private function getItemToInsert($item) {
56 56
 		if ($item instanceof HtmlInput || $item instanceof HtmlImg || $item instanceof HtmlIcon || $item instanceof HtmlButtonGroups || $item instanceof HtmlButton || $item instanceof HtmlLabel) {
57
-			$itemO=new HtmlMenuItem("item-" . $this->identifier . "-" . \sizeof($this->content) , $item);
57
+			$itemO=new HtmlMenuItem("item-".$this->identifier."-".\sizeof($this->content), $item);
58 58
 			$itemO->addClass("no-active");
59 59
 			$item=$itemO;
60 60
 		}
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
 
64 64
 	private function afterInsert($item) {
65 65
 		if (!$item instanceof HtmlMenu && $item->propertyContains("class", "header")===false)
66
-			$item->addToPropertyCtrl("class", "item", array ("item" ));
66
+			$item->addToPropertyCtrl("class", "item", array("item"));
67 67
 		else {
68 68
 			$this->setSecondary();
69 69
 		}
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
 	public function addItem($item) {
80 80
 		$number=$item;
81 81
 		$item=parent::addItem($this->getItemToInsert($item));
82
-		if(\is_int($number))
82
+		if (\is_int($number))
83 83
 			$item->setProperty("data-page", $number);
84 84
 		return $this->afterInsert($item);
85 85
 	}
@@ -97,15 +97,15 @@  discard block
 block discarded – undo
97 97
 
98 98
 	public function generateMenuAsItem($menu, $header=null) {
99 99
 		$count=$this->count();
100
-		$item=new HtmlSemDoubleElement("item-" . $this->identifier . "-" . $count, "div");
100
+		$item=new HtmlSemDoubleElement("item-".$this->identifier."-".$count, "div");
101 101
 		if (isset($header)) {
102
-			$headerItem=new HtmlSemDoubleElement("item-header-" . $this->identifier . "-" . $count, "div", "header");
102
+			$headerItem=new HtmlSemDoubleElement("item-header-".$this->identifier."-".$count, "div", "header");
103 103
 			$headerItem->setContent($header);
104 104
 			$item->addContent($headerItem);
105 105
 			$this->_itemHeader=$headerItem;
106 106
 		}
107
-		if(\is_array($menu)){
108
-			$menu=new HtmlMenu("menu-" . $this->identifier . "-" . $count,$menu);
107
+		if (\is_array($menu)) {
108
+			$menu=new HtmlMenu("menu-".$this->identifier."-".$count, $menu);
109 109
 		}
110 110
 		$menu->setClass("menu");
111 111
 		$item->addContent($menu);
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
 		$value=new HtmlSemDoubleElement($identifier, "a", "browse item", $value);
121 121
 		$value->addContent(new HtmlIcon("", "dropdown"));
122 122
 		$value=$this->addItem($value);
123
-		$popup=new HtmlPopup($value, "popup-" . $this->identifier . "-" . $this->count(), $content);
123
+		$popup=new HtmlPopup($value, "popup-".$this->identifier."-".$this->count(), $content);
124 124
 		$popup->setFlowing()->setPosition("bottom left")->setOn("click");
125 125
 		$this->wrap("", $popup);
126 126
 		return $popup;
@@ -129,7 +129,7 @@  discard block
 block discarded – undo
129 129
 	public function addDropdownAsItem($value, $items=NULL) {
130 130
 		$dd=$value;
131 131
 		if (\is_string($value)) {
132
-			$dd=new HtmlDropdown("dropdown-" . $this->identifier . "-" . $this->count(), $value, $items);
132
+			$dd=new HtmlDropdown("dropdown-".$this->identifier."-".$this->count(), $value, $items);
133 133
 		}
134 134
 		$this->addItem($dd);
135 135
 		return $dd;
@@ -142,14 +142,14 @@  discard block
 block discarded – undo
142 142
 	 * @see HtmlCollection::createItem()
143 143
 	 */
144 144
 	protected function createItem($value) {
145
-		$itemO=new HtmlMenuItem($this->identifier."-item-" . \sizeof($this->content),"");
145
+		$itemO=new HtmlMenuItem($this->identifier."-item-".\sizeof($this->content), "");
146 146
 		$itemO->setTagName("a");
147 147
 		$itemO->setContent($value);
148 148
 		return $itemO;
149 149
 	}
150 150
 
151 151
 	public function setSecondary($value=true) {
152
-		if($value)
152
+		if ($value)
153 153
 			$this->addToProperty("class", "secondary");
154 154
 		else
155 155
 			$this->removePropertyValue("class", "secondary");
@@ -157,28 +157,28 @@  discard block
 block discarded – undo
157 157
 	}
158 158
 
159 159
 	public function setVertical() {
160
-		return $this->addToPropertyCtrl("class", "vertical", array ("vertical" ));
160
+		return $this->addToPropertyCtrl("class", "vertical", array("vertical"));
161 161
 	}
162 162
 
163 163
 	public function setPosition($value="right") {
164
-		return $this->addToPropertyCtrl("class", $value, array ("right","left" ));
164
+		return $this->addToPropertyCtrl("class", $value, array("right", "left"));
165 165
 	}
166 166
 
167 167
 	public function setPointing($value=Direction::NONE) {
168
-		return $this->addToPropertyCtrl("class", $value . " pointing", Direction::getConstantValues("pointing"));
168
+		return $this->addToPropertyCtrl("class", $value." pointing", Direction::getConstantValues("pointing"));
169 169
 	}
170 170
 
171 171
 	public function asTab($vertical=false) {
172
-		$this->apply(function (HtmlDoubleElement &$item) {
172
+		$this->apply(function(HtmlDoubleElement &$item) {
173 173
 			$item->setTagName("a");
174 174
 		});
175
-		if ($vertical === true)
175
+		if ($vertical===true)
176 176
 			$this->setVertical();
177 177
 		return $this->addToProperty("class", "tabular");
178 178
 	}
179 179
 
180 180
 	public function asPagination() {
181
-		$this->apply(function (HtmlDoubleElement &$item) {
181
+		$this->apply(function(HtmlDoubleElement &$item) {
182 182
 			$item->setTagName("a");
183 183
 		});
184 184
 		return $this->addToProperty("class", "pagination");
@@ -215,10 +215,10 @@  discard block
 block discarded – undo
215 215
 	 */
216 216
 	public function setWidth($width) {
217 217
 		if (\is_int($width)) {
218
-			$width=Wide::getConstants()["W" . $width];
218
+			$width=Wide::getConstants()["W".$width];
219 219
 		}
220 220
 		$this->addToPropertyCtrl("class", $width, Wide::getConstants());
221
-		return $this->addToPropertyCtrl("class", "item", array ("item" ));
221
+		return $this->addToPropertyCtrl("class", "item", array("item"));
222 222
 	}
223 223
 
224 224
 	public function addImage($identifier, $src="", $alt="") {
@@ -233,14 +233,14 @@  discard block
 block discarded – undo
233 233
 		return $this->_itemHeader;
234 234
 	}
235 235
 
236
-	public function setHasContainer(){
237
-		return $this->wrapContent("<div class='ui container'>","</div>");
236
+	public function setHasContainer() {
237
+		return $this->wrapContent("<div class='ui container'>", "</div>");
238 238
 	}
239 239
 
240
-	public function run(JsUtils $js){
241
-		if($this->identifier!=="" && !isset($this->_bsComponent))
242
-			$this->onClick('if(!$(this).hasClass("dropdown")&&!$(this).hasClass("no-active")){$(this).addClass("active").siblings().removeClass("active");}',false,false);
243
-		$result= parent::run($js);
240
+	public function run(JsUtils $js) {
241
+		if ($this->identifier!=="" && !isset($this->_bsComponent))
242
+			$this->onClick('if(!$(this).hasClass("dropdown")&&!$(this).hasClass("no-active")){$(this).addClass("active").siblings().removeClass("active");}', false, false);
243
+		$result=parent::run($js);
244 244
 		return $result->setItemSelector(">.item");
245 245
 	}
246 246
 }
Please login to merge, or discard this patch.
Ajax/semantic/html/modules/HtmlDropdown.php 2 patches
Braces   +24 added lines, -19 removed lines patch added patch discarded remove patch
@@ -35,7 +35,7 @@  discard block
 block discarded – undo
35 35
 		if(isset($value)){
36 36
 			if($value instanceof HtmlSemDoubleElement){
37 37
 				$text=$value;
38
-			}else{
38
+			} else{
39 39
 				$text=new HtmlSemDoubleElement("text-".$this->identifier,"div");
40 40
 				$text->setClass("text");
41 41
 				$this->setValue($value);
@@ -112,7 +112,7 @@  discard block
 block discarded – undo
112 112
 		}
113 113
 		if(!$item instanceof HtmlDropdownItem){
114 114
 			$itemO=new HtmlDropdownItem("dd-item-".$this->identifier."-".\sizeof($this->items),$item,$value,$image,$description);
115
-		}elseif($itemO instanceof HtmlDropdownItem){
115
+		} elseif($itemO instanceof HtmlDropdownItem){
116 116
 			$this->addToProperty("class", "vertical");
117 117
 		}
118 118
 		return $itemO;
@@ -126,8 +126,9 @@  discard block
 block discarded – undo
126 126
 	}
127 127
 
128 128
 	public function addInput($name){
129
-		if(!isset($name))
130
-			$name="input-".$this->identifier;
129
+		if(!isset($name)) {
130
+					$name="input-".$this->identifier;
131
+		}
131 132
 		$this->setAction("activate");
132 133
 		$this->input=new HtmlInput($name,"hidden");
133 134
 		$this->input->setIdentifier("input-".$this->identifier);
@@ -186,7 +187,7 @@  discard block
 block discarded – undo
186 187
 			foreach ($items as $k=>$v){
187 188
 				$this->addItem($v)->setData($k);
188 189
 			}
189
-		}else{
190
+		} else{
190 191
 			foreach ($items as $item){
191 192
 				$this->addItem($item);
192 193
 			}
@@ -208,8 +209,7 @@  discard block
 block discarded – undo
208 209
 			$c=$this->items[$i++];
209 210
 			if(isset($c)){
210 211
 				$c->setProperty($property,$value);
211
-			}
212
-			else{
212
+			} else{
213 213
 				return $this;
214 214
 			}
215 215
 		}
@@ -240,7 +240,7 @@  discard block
 block discarded – undo
240 240
 		if($dropdown===false){
241 241
 			$this->_template=include dirname(__FILE__).'/../templates/tplDropdownMenu.php';
242 242
 			$dropdown="menu";
243
-		}else{
243
+		} else{
244 244
 			$dropdown="dropdown";
245 245
 			$this->mClass="menu";
246 246
 		}
@@ -261,22 +261,25 @@  discard block
 block discarded – undo
261 261
 
262 262
 	public function asButton($floating=false){
263 263
 		$this->removeArrow();
264
-		if($floating)
265
-			$this->addToProperty("class", "floating");
264
+		if($floating) {
265
+					$this->addToProperty("class", "floating");
266
+		}
266 267
 		$this->removePropertyValue("class", "selection");
267 268
 		return $this->addToProperty("class", "button");
268 269
 	}
269 270
 
270 271
 	public function asSelect($name=NULL,$multiple=false,$selection=true){
271 272
 		$this->_multiple=$multiple;
272
-		if(isset($name))
273
-			$this->addInput($name);
273
+		if(isset($name)) {
274
+					$this->addInput($name);
275
+		}
274 276
 		if($multiple){
275 277
 			$this->addToProperty("class", "multiple");
276 278
 		}
277 279
 		if ($selection){
278
-			if($this->propertyContains("class", "button")===false)
279
-				$this->addToPropertyCtrl("class", "selection",array("selection"));
280
+			if($this->propertyContains("class", "button")===false) {
281
+							$this->addToPropertyCtrl("class", "selection",array("selection"));
282
+			}
280 283
 		}
281 284
 		return $this;
282 285
 	}
@@ -288,8 +291,9 @@  discard block
 block discarded – undo
288 291
 
289 292
 	public function setSelect($name=NULL,$multiple=false){
290 293
 		$this->_template='<%tagName% id="%identifier%" %properties%>%items%</%tagName%>';
291
-		if(!isset($name))
292
-			$name="select-".$this->identifier;
294
+		if(!isset($name)) {
295
+					$name="select-".$this->identifier;
296
+		}
293 297
 		$this->input=null;
294 298
 		if($multiple){
295 299
 			$this->setProperty("multiple", true);
@@ -330,12 +334,13 @@  discard block
 block discarded – undo
330 334
 		$value=$this->value;
331 335
 		if(isset($this->input) && isset($value)){
332 336
 			$this->input->setProperty("value", $value);
333
-		}else{
337
+		} else{
334 338
 			$this->setProperty("value", $value);
335 339
 		}
336 340
 			$textElement=$this->getElementById("text-".$this->identifier, $this->content);
337
-			if(isset($textElement) && ($textElement instanceof HtmlDoubleElement) && !$this->_multiple)
338
-				$textElement->setContent($value);
341
+			if(isset($textElement) && ($textElement instanceof HtmlDoubleElement) && !$this->_multiple) {
342
+							$textElement->setContent($value);
343
+			}
339 344
 		return $this;
340 345
 	}
341 346
 
Please login to merge, or discard this patch.
Spacing   +103 added lines, -103 removed lines patch added patch discarded remove patch
@@ -14,29 +14,29 @@  discard block
 block discarded – undo
14 14
 use Ajax\common\html\HtmlDoubleElement;
15 15
 
16 16
 class HtmlDropdown extends HtmlSemDoubleElement {
17
-	use FieldTrait,LabeledIconTrait {
17
+	use FieldTrait, LabeledIconTrait {
18 18
 		addIcon as addIconP;
19 19
 	}
20 20
 	protected $mClass="menu";
21 21
 	protected $mTagName="div";
22
-	protected $items=array ();
23
-	protected $_params=array("action"=>"nothing","on"=>"hover","showOnFocus"=>false);
22
+	protected $items=array();
23
+	protected $_params=array("action"=>"nothing", "on"=>"hover", "showOnFocus"=>false);
24 24
 	protected $input;
25 25
 	protected $value;
26 26
 	protected $_associative;
27 27
 	protected $_multiple;
28 28
 
29
-	public function __construct($identifier, $value="", $items=array(),$associative=true) {
29
+	public function __construct($identifier, $value="", $items=array(), $associative=true) {
30 30
 		parent::__construct($identifier, "div");
31 31
 		$this->_template=include dirname(__FILE__).'/../templates/tplDropdown.php';
32 32
 		$this->setProperty("class", "ui dropdown");
33 33
 		$this->_multiple=false;
34 34
 		$content=[];
35
-		if(isset($value)){
36
-			if($value instanceof HtmlSemDoubleElement){
35
+		if (isset($value)) {
36
+			if ($value instanceof HtmlSemDoubleElement) {
37 37
 				$text=$value;
38
-			}else{
39
-				$text=new HtmlSemDoubleElement("text-".$this->identifier,"div");
38
+			} else {
39
+				$text=new HtmlSemDoubleElement("text-".$this->identifier, "div");
40 40
 				$text->setClass("text");
41 41
 				$this->setValue($value);
42 42
 			}
@@ -49,33 +49,33 @@  discard block
 block discarded – undo
49 49
 		$this->addItems($items);
50 50
 	}
51 51
 
52
-	public function getField(){
52
+	public function getField() {
53 53
 		return $this->input;
54 54
 	}
55 55
 
56
-	public function getDataField(){
56
+	public function getDataField() {
57 57
 		return $this->input;
58 58
 	}
59 59
 
60
-	public function addItem($item,$value=NULL,$image=NULL,$description=NULL){
61
-		$itemO=$this->beforeAddItem($item,$value,$image,$description);
60
+	public function addItem($item, $value=NULL, $image=NULL, $description=NULL) {
61
+		$itemO=$this->beforeAddItem($item, $value, $image, $description);
62 62
 		$this->items[]=$itemO;
63 63
 		return $itemO;
64 64
 	}
65 65
 
66
-	public function addIcon($icon,$before=true,$labeled=false){
66
+	public function addIcon($icon, $before=true, $labeled=false) {
67 67
 		$this->removeArrow();
68
-		$this->addIconP($icon,$before,$labeled);
68
+		$this->addIconP($icon, $before, $labeled);
69 69
 		$elm=$this->getElementById("text-".$this->identifier, $this->content);
70
-		if(isset($elm)){
70
+		if (isset($elm)) {
71 71
 			$elm->setWrapAfter("");
72 72
 		}
73 73
 		return $this;
74 74
 	}
75 75
 
76
-	public function addIcons($icons){
76
+	public function addIcons($icons) {
77 77
 		$count=$this->count();
78
-		for ($i=0;$i<\sizeof($icons) && $i<$count;$i++){
78
+		for ($i=0; $i<\sizeof($icons) && $i<$count; $i++) {
79 79
 			$this->getItem($i)->addIcon($icons[$i]);
80 80
 		}
81 81
 	}
@@ -86,33 +86,33 @@  discard block
 block discarded – undo
86 86
 	 * @param int $position
87 87
 	 * @return HtmlDropdownItem
88 88
 	 */
89
-	public function insertItem($item,$position=0){
89
+	public function insertItem($item, $position=0) {
90 90
 		$itemO=$this->beforeAddItem($item);
91
-		 $start = array_slice($this->items, 0, $position);
92
-		 $end = array_slice($this->items, $position);
93
-		 $start[] = $item;
91
+		 $start=array_slice($this->items, 0, $position);
92
+		 $end=array_slice($this->items, $position);
93
+		 $start[]=$item;
94 94
 		 $this->items=array_merge($start, $end);
95 95
 		 return $itemO;
96 96
 	}
97 97
 
98
-	protected function removeArrow(){
99
-		if(\sizeof($this->content)>1){
98
+	protected function removeArrow() {
99
+		if (\sizeof($this->content)>1) {
100 100
 			unset($this->content["arrow"]);
101 101
 			$this->content=\array_values($this->content);
102 102
 		}
103 103
 	}
104 104
 
105
-	protected function beforeAddItem($item,$value=NULL,$image=NULL,$description=NULL){
105
+	protected function beforeAddItem($item, $value=NULL, $image=NULL, $description=NULL) {
106 106
 		$itemO=$item;
107
-		if(\is_array($item)){
107
+		if (\is_array($item)) {
108 108
 			$description=JArray::getValue($item, "description", 3);
109 109
 			$value=JArray::getValue($item, "value", 1);
110 110
 			$image=JArray::getValue($item, "image", 2);
111 111
 			$item=JArray::getValue($item, "item", 0);
112 112
 		}
113
-		if(!$item instanceof HtmlDropdownItem){
114
-			$itemO=new HtmlDropdownItem("dd-item-".$this->identifier."-".\sizeof($this->items),$item,$value,$image,$description);
115
-		}elseif($itemO instanceof HtmlDropdownItem){
113
+		if (!$item instanceof HtmlDropdownItem) {
114
+			$itemO=new HtmlDropdownItem("dd-item-".$this->identifier."-".\sizeof($this->items), $item, $value, $image, $description);
115
+		}elseif ($itemO instanceof HtmlDropdownItem) {
116 116
 			$this->addToProperty("class", "vertical");
117 117
 		}
118 118
 		return $itemO;
@@ -125,11 +125,11 @@  discard block
 block discarded – undo
125 125
 		$this->addItem($function($object));
126 126
 	}
127 127
 
128
-	public function addInput($name){
129
-		if(!isset($name))
128
+	public function addInput($name) {
129
+		if (!isset($name))
130 130
 			$name="input-".$this->identifier;
131 131
 		$this->setAction("activate");
132
-		$this->input=new HtmlInput($name,"hidden");
132
+		$this->input=new HtmlInput($name, "hidden");
133 133
 		$this->input->setIdentifier("input-".$this->identifier);
134 134
 		return $this->input;
135 135
 	}
@@ -140,15 +140,15 @@  discard block
 block discarded – undo
140 140
 	 * @param string $icon
141 141
 	 * @return HtmlDropdownItem
142 142
 	 */
143
-	public function addSearchInputItem($placeHolder=NULL,$icon=NULL){
144
-		return $this->addItem(HtmlDropdownItem::searchInput($placeHolder,$icon));
143
+	public function addSearchInputItem($placeHolder=NULL, $icon=NULL) {
144
+		return $this->addItem(HtmlDropdownItem::searchInput($placeHolder, $icon));
145 145
 	}
146 146
 
147 147
 	/**
148 148
 	 * Adds a divider item
149 149
 	 * @return HtmlDropdownItem
150 150
 	 */
151
-	public function addDividerItem(){
151
+	public function addDividerItem() {
152 152
 		return $this->addItem(HtmlDropdownItem::divider());
153 153
 	}
154 154
 
@@ -158,8 +158,8 @@  discard block
 block discarded – undo
158 158
 	 * @param string $icon
159 159
 	 * @return HtmlDropdownItem
160 160
 	 */
161
-	public function addHeaderItem($caption=NULL,$icon=NULL){
162
-		return $this->addItem(HtmlDropdownItem::header($caption,$icon));
161
+	public function addHeaderItem($caption=NULL, $icon=NULL) {
162
+		return $this->addItem(HtmlDropdownItem::header($caption, $icon));
163 163
 	}
164 164
 
165 165
 	/**
@@ -168,7 +168,7 @@  discard block
 block discarded – undo
168 168
 	 * @param string $color
169 169
 	 * @return HtmlDropdownItem
170 170
 	 */
171
-	public function addCircularLabelItem($caption,$color){
171
+	public function addCircularLabelItem($caption, $color) {
172 172
 		return $this->addItem(HtmlDropdownItem::circular($caption, $color));
173 173
 	}
174 174
 
@@ -177,17 +177,17 @@  discard block
 block discarded – undo
177 177
 	 * @param string $image
178 178
 	 * @return \Ajax\semantic\html\content\HtmlDropdownItem
179 179
 	 */
180
-	public function addMiniAvatarImageItem($caption,$image){
180
+	public function addMiniAvatarImageItem($caption, $image) {
181 181
 		return $this->addItem(HtmlDropdownItem::avatar($caption, $image));
182 182
 	}
183 183
 
184
-	public function addItems($items){
185
-		if(\is_array($items) && $this->_associative){
186
-			foreach ($items as $k=>$v){
184
+	public function addItems($items) {
185
+		if (\is_array($items) && $this->_associative) {
186
+			foreach ($items as $k=>$v) {
187 187
 				$this->addItem($v)->setData($k);
188 188
 			}
189
-		}else{
190
-			foreach ($items as $item){
189
+		} else {
190
+			foreach ($items as $item) {
191 191
 				$this->addItem($item);
192 192
 			}
193 193
 		}
@@ -199,99 +199,99 @@  discard block
 block discarded – undo
199 199
 	 * @param array|mixed $values
200 200
 	 * @return $this
201 201
 	 */
202
-	public function setPropertyValues($property,$values){
202
+	public function setPropertyValues($property, $values) {
203 203
 		$i=0;
204
-		if(\is_array($values)===false){
205
-			$values=\array_fill(0, $this->count(),$values);
204
+		if (\is_array($values)===false) {
205
+			$values=\array_fill(0, $this->count(), $values);
206 206
 		}
207
-		foreach ($values as $value){
207
+		foreach ($values as $value) {
208 208
 			$c=$this->items[$i++];
209
-			if(isset($c)){
210
-				$c->setProperty($property,$value);
209
+			if (isset($c)) {
210
+				$c->setProperty($property, $value);
211 211
 			}
212
-			else{
212
+			else {
213 213
 				return $this;
214 214
 			}
215 215
 		}
216 216
 		return $this;
217 217
 	}
218 218
 
219
-	public function each($callBack){
220
-		foreach ($this->items as $index=>$value){
221
-			$callBack($index,$value);
219
+	public function each($callBack) {
220
+		foreach ($this->items as $index=>$value) {
221
+			$callBack($index, $value);
222 222
 		}
223 223
 		return $this;
224 224
 	}
225 225
 
226
-	public function getItem($index){
226
+	public function getItem($index) {
227 227
 		return $this->items[$index];
228 228
 	}
229 229
 
230 230
 	/**
231 231
 	 * @return int
232 232
 	 */
233
-	public function count(){
233
+	public function count() {
234 234
 		return \sizeof($this->items);
235 235
 	}
236 236
 	/**
237 237
 	 * @param boolean $dropdown
238 238
 	 */
239
-	public function asDropdown($dropdown){
240
-		if($dropdown===false){
239
+	public function asDropdown($dropdown) {
240
+		if ($dropdown===false) {
241 241
 			$this->_template=include dirname(__FILE__).'/../templates/tplDropdownMenu.php';
242 242
 			$dropdown="menu";
243
-		}else{
243
+		} else {
244 244
 			$dropdown="dropdown";
245 245
 			$this->mClass="menu";
246 246
 		}
247
-		return $this->addToPropertyCtrl("class", $dropdown,array("menu","dropdown"));
247
+		return $this->addToPropertyCtrl("class", $dropdown, array("menu", "dropdown"));
248 248
 	}
249 249
 
250
-	public function setVertical(){
251
-		return $this->addToPropertyCtrl("class", "vertical",array("vertical"));
250
+	public function setVertical() {
251
+		return $this->addToPropertyCtrl("class", "vertical", array("vertical"));
252 252
 	}
253 253
 
254
-	public function setInline(){
255
-		return $this->addToPropertyCtrl("class", "inline",["inline"]);
254
+	public function setInline() {
255
+		return $this->addToPropertyCtrl("class", "inline", ["inline"]);
256 256
 	}
257 257
 
258
-	public function setSimple(){
259
-		return $this->addToPropertyCtrl("class", "simple",array("simple"));
258
+	public function setSimple() {
259
+		return $this->addToPropertyCtrl("class", "simple", array("simple"));
260 260
 	}
261 261
 
262
-	public function asButton($floating=false){
262
+	public function asButton($floating=false) {
263 263
 		$this->removeArrow();
264
-		if($floating)
264
+		if ($floating)
265 265
 			$this->addToProperty("class", "floating");
266 266
 		$this->removePropertyValue("class", "selection");
267 267
 		return $this->addToProperty("class", "button");
268 268
 	}
269 269
 
270
-	public function asSelect($name=NULL,$multiple=false,$selection=true){
270
+	public function asSelect($name=NULL, $multiple=false, $selection=true) {
271 271
 		$this->_multiple=$multiple;
272
-		if(isset($name))
272
+		if (isset($name))
273 273
 			$this->addInput($name);
274
-		if($multiple){
274
+		if ($multiple) {
275 275
 			$this->addToProperty("class", "multiple");
276 276
 		}
277
-		if ($selection){
278
-			if($this->propertyContains("class", "button")===false)
279
-				$this->addToPropertyCtrl("class", "selection",array("selection"));
277
+		if ($selection) {
278
+			if ($this->propertyContains("class", "button")===false)
279
+				$this->addToPropertyCtrl("class", "selection", array("selection"));
280 280
 		}
281 281
 		return $this;
282 282
 	}
283 283
 
284
-	public function asSearch($name=NULL,$multiple=false,$selection=true){
285
-		$this->asSelect($name,$multiple,$selection);
284
+	public function asSearch($name=NULL, $multiple=false, $selection=true) {
285
+		$this->asSelect($name, $multiple, $selection);
286 286
 		return $this->addToProperty("class", "search");
287 287
 	}
288 288
 
289
-	public function setSelect($name=NULL,$multiple=false){
289
+	public function setSelect($name=NULL, $multiple=false) {
290 290
 		$this->_template='<%tagName% id="%identifier%" %properties%>%items%</%tagName%>';
291
-		if(!isset($name))
291
+		if (!isset($name))
292 292
 			$name="select-".$this->identifier;
293 293
 		$this->input=null;
294
-		if($multiple){
294
+		if ($multiple) {
295 295
 			$this->setProperty("multiple", true);
296 296
 			$this->addToProperty("class", "multiple");
297 297
 		}
@@ -299,42 +299,42 @@  discard block
 block discarded – undo
299 299
 		$this->tagName="select";
300 300
 		$this->setProperty("name", $name);
301 301
 		$this->content=null;
302
-		foreach ($this->items as $item){
302
+		foreach ($this->items as $item) {
303 303
 			$item->asOption();
304 304
 		}
305 305
 		return $this;
306 306
 	}
307 307
 
308
-	public function asSubmenu($pointing=NULL){
308
+	public function asSubmenu($pointing=NULL) {
309 309
 		$this->setClass("ui dropdown link item");
310
-		if(isset($pointing)){
310
+		if (isset($pointing)) {
311 311
 			$this->setPointing($pointing);
312 312
 		}
313 313
 		return $this;
314 314
 	}
315 315
 
316
-	public function setPointing($value=Direction::NONE){
317
-		return $this->addToPropertyCtrl("class", $value." pointing",Direction::getConstantValues("pointing"));
316
+	public function setPointing($value=Direction::NONE) {
317
+		return $this->addToPropertyCtrl("class", $value." pointing", Direction::getConstantValues("pointing"));
318 318
 	}
319 319
 
320
-	public function setValue($value){
320
+	public function setValue($value) {
321 321
 		$this->value=$value;
322 322
 		return $this;
323 323
 	}
324 324
 	
325
-	public function setDefaultText($text){
326
-		$this->content["text"]=new HtmlSemDoubleElement("","div","default text",$text);
325
+	public function setDefaultText($text) {
326
+		$this->content["text"]=new HtmlSemDoubleElement("", "div", "default text", $text);
327 327
 	}
328 328
 	
329
-	private function applyValue(){
329
+	private function applyValue() {
330 330
 		$value=$this->value;
331
-		if(isset($this->input) && isset($value)){
331
+		if (isset($this->input) && isset($value)) {
332 332
 			$this->input->setProperty("value", $value);
333
-		}else{
333
+		} else {
334 334
 			$this->setProperty("value", $value);
335 335
 		}
336 336
 			$textElement=$this->getElementById("text-".$this->identifier, $this->content);
337
-			if(isset($textElement) && ($textElement instanceof HtmlDoubleElement) && !$this->_multiple)
337
+			if (isset($textElement) && ($textElement instanceof HtmlDoubleElement) && !$this->_multiple)
338 338
 				$textElement->setContent($value);
339 339
 		return $this;
340 340
 	}
@@ -344,9 +344,9 @@  discard block
 block discarded – undo
344 344
 	 * @see BaseHtml::run()
345 345
 	 */
346 346
 	public function run(JsUtils $js) {
347
-		if($this->propertyContains("class", "simple")===false){
348
-			if(isset($this->_bsComponent)===false){
349
-				$this->_bsComponent=$js->semantic()->dropdown("#".$this->identifier,$this->_params);
347
+		if ($this->propertyContains("class", "simple")===false) {
348
+			if (isset($this->_bsComponent)===false) {
349
+				$this->_bsComponent=$js->semantic()->dropdown("#".$this->identifier, $this->_params);
350 350
 				$this->_bsComponent->setItemSelector(".item");
351 351
 			}
352 352
 			$this->addEventsOnRun($js);
@@ -354,50 +354,50 @@  discard block
 block discarded – undo
354 354
 		}
355 355
 	}
356 356
 
357
-	public function setCompact(){
357
+	public function setCompact() {
358 358
 		return $this->addToPropertyCtrl("class", "compact", array("compact"));
359 359
 	}
360 360
 
361
-	public function setAction($action){
361
+	public function setAction($action) {
362 362
 		$this->_params["action"]=$action;
363 363
 	}
364 364
 
365
-	public function setOn($on){
365
+	public function setOn($on) {
366 366
 		$this->_params["on"]=$on;
367 367
 	}
368 368
 
369
-	public function setShowOnFocus($value){
369
+	public function setShowOnFocus($value) {
370 370
 		$this->_params["showOnFocus"]=$value;
371 371
 	}
372 372
 	
373
-	public function setAllowAdditions($value){
373
+	public function setAllowAdditions($value) {
374 374
 		$this->_params["allowAdditions"]=$value;
375 375
 	}
376 376
 
377
-	public function setFullTextSearch($value){
377
+	public function setFullTextSearch($value) {
378 378
 		$this->_params["fullTextSearch"]=$value;
379 379
 	}
380 380
 
381 381
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
382 382
 		$this->applyValue();
383
-		return parent::compile($js,$view);
383
+		return parent::compile($js, $view);
384 384
 	}
385 385
 
386 386
 	public function getInput() {
387 387
 		return $this->input;
388 388
 	}
389 389
 
390
-	public function setIcon($icon="dropdown"){
390
+	public function setIcon($icon="dropdown") {
391 391
 		$this->content["arrow"]=new HtmlIcon($this->identifier."-icon", $icon);
392 392
 		return $this;
393 393
 	}
394 394
 
395
-	public function jsAddItem($caption){
395
+	public function jsAddItem($caption) {
396 396
 		$js="var first=$('#{$this->identifier} .item').first();if(first!=undefined){var new =first.clone();first.parent().append(new);first.html('{$caption}};')";
397 397
 		return $js;
398 398
 	}
399 399
 	
400
-	public function setClearable($value){
400
+	public function setClearable($value) {
401 401
 		$this->_params["clearable"]=$value;
402 402
 	}
403 403
 }
Please login to merge, or discard this patch.
Ajax/semantic/html/collections/form/HtmlFormFields.php 2 patches
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -35,7 +35,7 @@  discard block
 block discarded – undo
35 35
 		}
36 36
 		if (isset($label))
37 37
 			$this->setLabel($label);
38
-		foreach ( $fields as $field ) {
38
+		foreach ($fields as $field) {
39 39
 			$this->addItem($field);
40 40
 		}
41 41
 		return $this;
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
57 57
 
58 58
 	public function addItem($item) {
59 59
 		$item=parent::addItem($item);
60
-		if($item instanceof HtmlFormField)
60
+		if ($item instanceof HtmlFormField)
61 61
 			$item->setContainer($this);
62 62
 		return $item;
63 63
 	}
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
65 65
 	/**
66 66
 	 * @return HtmlFormField
67 67
 	 */
68
-	public function getItem($index){
68
+	public function getItem($index) {
69 69
 		return parent::getItem($index);
70 70
 	}
71 71
 
@@ -104,10 +104,10 @@  discard block
 block discarded – undo
104 104
 	}
105 105
 
106 106
 
107
-	public static function radios($identifier,$name, $items=array(), $label=NULL, $value=null, $type=NULL) {
108
-		$fields=array ();
107
+	public static function radios($identifier, $name, $items=array(), $label=NULL, $value=null, $type=NULL) {
108
+		$fields=array();
109 109
 		$i=0;
110
-		foreach ( $items as $val => $caption ) {
110
+		foreach ($items as $val => $caption) {
111 111
 			$itemO=new HtmlFormRadio($name."-".$i++, $name, $caption, $val, $type);
112 112
 			if ($val===$value) {
113 113
 				$itemO->getDataField()->setProperty("checked", "");
@@ -120,10 +120,10 @@  discard block
 block discarded – undo
120 120
 		return $radios;
121 121
 	}
122 122
 
123
-	public static function checkeds($identifier,$name, $items=array(), $label=NULL, $values=array(), $type=NULL) {
124
-		$fields=array ();
123
+	public static function checkeds($identifier, $name, $items=array(), $label=NULL, $values=array(), $type=NULL) {
124
+		$fields=array();
125 125
 		$i=0;
126
-		foreach ( $items as $val => $caption ) {
126
+		foreach ($items as $val => $caption) {
127 127
 			$itemO=new HtmlFormCheckbox($name."-".$i++, $caption, $val, $type);
128 128
 			$itemO->setName($name);
129 129
 			if (\array_search($val, $values)!==false) {
@@ -142,8 +142,8 @@  discard block
 block discarded – undo
142 142
 		return $this;
143 143
 	}
144 144
 	
145
-	public function run(JsUtils $js){
146
-		$result= parent::run($js);
145
+	public function run(JsUtils $js) {
146
+		$result=parent::run($js);
147 147
 		//return $result->setItemSelector("[data-value]");
148 148
 	}
149 149
 }
Please login to merge, or discard this patch.
Braces   +18 added lines, -12 removed lines patch added patch discarded remove patch
@@ -29,12 +29,14 @@  discard block
 block discarded – undo
29 29
 				if (\is_string($end)) {
30 30
 					$label=$end;
31 31
 					\array_pop($fields);
32
-				} else
33
-					$label=NULL;
32
+				} else {
33
+									$label=NULL;
34
+				}
34 35
 			}
35 36
 		}
36
-		if (isset($label))
37
-			$this->setLabel($label);
37
+		if (isset($label)) {
38
+					$this->setLabel($label);
39
+		}
38 40
 		foreach ( $fields as $field ) {
39 41
 			$this->addItem($field);
40 42
 		}
@@ -57,8 +59,9 @@  discard block
 block discarded – undo
57 59
 
58 60
 	public function addItem($item) {
59 61
 		$item=parent::addItem($item);
60
-		if($item instanceof HtmlFormField)
61
-			$item->setContainer($this);
62
+		if($item instanceof HtmlFormField) {
63
+					$item->setContainer($this);
64
+		}
62 65
 		return $item;
63 66
 	}
64 67
 	
@@ -73,8 +76,9 @@  discard block
 block discarded – undo
73 76
 		if ($this->_equalWidth) {
74 77
 			$count=$this->count();
75 78
 			$this->addToProperty("class", Wide::getConstants()["W".$count]." fields");
76
-		} else
77
-			$this->addToProperty("class", "fields");
79
+		} else {
80
+					$this->addToProperty("class", "fields");
81
+		}
78 82
 		return parent::compile($js, $view);
79 83
 	}
80 84
 
@@ -115,8 +119,9 @@  discard block
 block discarded – undo
115 119
 			$fields[]=$itemO;
116 120
 		}
117 121
 		$radios=new HtmlFormFields($identifier, $fields);
118
-		if (isset($label))
119
-			$radios->setLabel($label)->setProperty("for", $name);
122
+		if (isset($label)) {
123
+					$radios->setLabel($label)->setProperty("for", $name);
124
+		}
120 125
 		return $radios;
121 126
 	}
122 127
 
@@ -132,8 +137,9 @@  discard block
 block discarded – undo
132 137
 			$fields[]=$itemO;
133 138
 		}
134 139
 		$radios=new HtmlFormFields($identifier, $fields);
135
-		if (isset($label))
136
-			$radios->setLabel($label)->setProperty("for", $name);
140
+		if (isset($label)) {
141
+					$radios->setLabel($label)->setProperty("for", $name);
142
+		}
137 143
 		return $radios;
138 144
 	}
139 145
 
Please login to merge, or discard this patch.