Passed
Push — master ( 6cf655...807644 )
by Jean-Christophe
02:16
created
Ajax/semantic/widgets/dataform/DataForm.php 1 patch
Braces   +8 added lines, -6 removed lines patch added patch discarded remove patch
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
71 71
 				$this->_generateFields($form, [$v], $headers, $i, $wrappers,$nb++,$names[$k]??'');
72 72
 				$i++;
73 73
 			}
74
-		}else{
74
+		} else{
75 75
 			$separators[]=$count;
76 76
 			for($i=0;$i<$size;$i++){
77 77
 				$fields=\array_slice($values, $separators[$i]+1,$separators[$i+1]-$separators[$i]);
@@ -85,18 +85,20 @@  discard block
 block discarded – undo
85 85
 
86 86
 	protected function _generateFields($form, $values, $headers, $sepFirst, $wrappers, $nb, $name){
87 87
 		$wrapper=null;
88
-		if(isset($headers[$sepFirst+1]))
89
-			$form->addHeader($headers[$sepFirst+1],4,true);
88
+		if(isset($headers[$sepFirst+1])) {
89
+					$form->addHeader($headers[$sepFirst+1],4,true);
90
+		}
90 91
 		if(isset($wrappers[$sepFirst+1])){
91 92
 			$wrapper=$wrappers[$sepFirst+1];
92 93
 		}
93 94
 		$count=\count($values);
94 95
 		if($count===1){
95 96
 			$added=$form->addField($values[0]);
96
-		}elseif($count>1){
97
+		} elseif($count>1){
97 98
 			$added=$form->addFields($values);
98
-		}else
99
-			return;
99
+		} else {
100
+					return;
101
+		}
100 102
 		if(isset($wrapper)){
101 103
 			$added->wrap($wrapper[0],$wrapper[1]);
102 104
 		}
Please login to merge, or discard this patch.
Ajax/semantic/html/collections/form/HtmlFormDropdown.php 1 patch
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -66,8 +66,9 @@
 block discarded – undo
66 66
 		$actionO = $action;
67 67
 		if (\is_object($action) === false) {
68 68
 			$actionO = new HtmlButton("action-" . $this->identifier, $action);
69
-			if (isset($icon))
70
-				$actionO->addIcon($icon, true, $labeled);
69
+			if (isset($icon)) {
70
+							$actionO->addIcon($icon, true, $labeled);
71
+			}
71 72
 		}
72 73
 		$this->addContent($actionO, \strstr($direction, Direction::LEFT) !== false);
73 74
 		return $actionO;
Please login to merge, or discard this patch.
Ajax/semantic/html/base/HtmlSemNavElement.php 1 patch
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -66,8 +66,9 @@
 block discarded – undo
66 66
 
67 67
 	protected function setDivider($divider, $index) {
68 68
 		if (isset($index)) {
69
-			if (! \is_array($this->_contentSeparator))
70
-				$this->_contentSeparator = array_fill(0, $this->count() - 1, $this->_contentSeparator);
69
+			if (! \is_array($this->_contentSeparator)) {
70
+							$this->_contentSeparator = array_fill(0, $this->count() - 1, $this->_contentSeparator);
71
+			}
71 72
 			$this->_contentSeparator[$index] = $divider;
72 73
 		} else {
73 74
 			$this->_contentSeparator = $divider;
Please login to merge, or discard this patch.
Ajax/semantic/html/modules/HtmlModal.php 1 patch
Braces   +6 added lines, -4 removed lines patch added patch discarded remove patch
@@ -85,8 +85,9 @@  discard block
 block discarded – undo
85 85
 				$class = 'cancel';
86 86
 			}
87 87
 			$action = new HtmlButton('action-' . $this->identifier . '-' . JArray::count($this->content['actions']->getContent()), $action);
88
-			if ($class !== '')
89
-				$action->addToProperty('class', $class);
88
+			if ($class !== '') {
89
+							$action->addToProperty('class', $class);
90
+			}
90 91
 		}
91 92
 		return $this->addElementInPart($action, 'actions');
92 93
 	}
@@ -214,8 +215,9 @@  discard block
 block discarded – undo
214 215
 	 * @see BaseHtml::run()
215 216
 	 */
216 217
 	public function run(JsUtils $js) {
217
-		if (isset($this->_bsComponent) === false)
218
-			$this->_bsComponent = $js->semantic()->modal('#' . $this->identifier, $this->_params, $this->_paramParts);
218
+		if (isset($this->_bsComponent) === false) {
219
+					$this->_bsComponent = $js->semantic()->modal('#' . $this->identifier, $this->_params, $this->_paramParts);
220
+		}
219 221
 		$this->addEventsOnRun($js);
220 222
 		return $this->_bsComponent;
221 223
 	}
Please login to merge, or discard this patch.
Ajax/semantic/widgets/datatable/DataTableFieldAsTrait.php 1 patch
Braces   +12 added lines, -8 removed lines patch added patch discarded remove patch
@@ -68,15 +68,17 @@  discard block
 block discarded – undo
68 68
 	private function getFieldButton($caption,$visibleHover=true,$icon=null){
69 69
 		$id=$this->_instanceViewer->getIdentifier();
70 70
 		$bt= new HtmlButton($this->cleanIdentifier($icon??$caption).'-'.$id,$caption);
71
-		if($visibleHover)
72
-			$this->_visibleOver($bt);
71
+		if($visibleHover) {
72
+					$this->_visibleOver($bt);
73
+		}
73 74
 		return $bt;
74 75
 	}
75 76
 
76 77
 	private function getFieldButtons($buttons,$visibleHover=true){
77 78
 		$bts=new HtmlButtonGroups("",$buttons);
78
-		if($visibleHover)
79
-			$this->_visibleOver($bts);
79
+		if($visibleHover) {
80
+					$this->_visibleOver($bts);
81
+		}
80 82
 		return $bts;
81 83
 	}
82 84
 
@@ -93,8 +95,9 @@  discard block
 block discarded – undo
93 95
 		return $this->_fieldAs(function($id,$name,$value,$caption) use ($url,$responseElement,$cssStyle,$attributes){
94 96
 			$button=new HtmlButton($id,$value,$cssStyle);
95 97
 			$button->postOnClick($url,"$(event.target).closest('tr').find(':input').serialize()",$responseElement,$attributes["ajax"]);
96
-			if(!isset($attributes["visibleHover"]) || $attributes["visibleHover"])
97
-				$this->_visibleOver($button);
98
+			if(!isset($attributes["visibleHover"]) || $attributes["visibleHover"]) {
99
+							$this->_visibleOver($button);
100
+			}
98 101
 				return $button;
99 102
 		}, $index,$attributes);
100 103
 	}
@@ -165,8 +168,9 @@  discard block
 block discarded – undo
165 168
 	private function getDefaultButton($icon,$class=null,$visibleHover=true){
166 169
 		$bt=$this->getFieldButton("",$visibleHover,$icon);
167 170
 		$bt->asIcon($icon);
168
-		if(isset($class))
169
-			$bt->addClass($class);
171
+		if(isset($class)) {
172
+					$bt->addClass($class);
173
+		}
170 174
 		return $bt;
171 175
 	}
172 176
 
Please login to merge, or discard this patch.
Ajax/semantic/widgets/base/InstanceViewer.php 1 patch
Braces   +29 added lines, -24 removed lines patch added patch discarded remove patch
@@ -35,8 +35,9 @@  discard block
 block discarded – undo
35 35
 		$this->widgetIdentifier = $identifier;
36 36
 		$this->values = [];
37 37
 		$this->afterCompile = [];
38
-		if (isset($instance))
39
-			$this->setInstance($instance);
38
+		if (isset($instance)) {
39
+					$this->setInstance($instance);
40
+		}
40 41
 		$this->setCaptions($captions);
41 42
 		$this->captionCallback = NULL;
42 43
 		$this->defaultValueFunction = function ($name, $value) {
@@ -86,14 +87,16 @@  discard block
 block discarded – undo
86 87
 	}
87 88
 
88 89
 	public function getIdentifier($index = NULL) {
89
-		if (! isset($index))
90
-			$index = self::$index;
90
+		if (! isset($index)) {
91
+					$index = self::$index;
92
+		}
91 93
 		$value = $index;
92 94
 		if (isset($this->values["identifier"])) {
93
-			if (\is_string($this->values["identifier"]))
94
-				$value = JReflection::callMethod($this->instance, $this->values["identifier"], []);
95
-			else
96
-				$value = $this->values["identifier"]($index, $this->instance);
95
+			if (\is_string($this->values["identifier"])) {
96
+							$value = JReflection::callMethod($this->instance, $this->values["identifier"], []);
97
+			} else {
98
+							$value = $this->values["identifier"]($index, $this->instance);
99
+			}
97 100
 		}
98 101
 		return $value;
99 102
 	}
@@ -124,9 +127,8 @@  discard block
 block discarded – undo
124 127
 		} elseif (\is_callable($property) && \array_search($property, ['system','date']) === false){
125 128
 			try{
126 129
 				$value = $property($this->instance);
127
-			}catch(\Error $e){}
128
-		}
129
-		elseif (\is_array($property)) {
130
+			} catch(\Error $e){}
131
+		} elseif (\is_array($property)) {
130 132
 			$values = \array_map(function ($v) use ($index) {
131 133
 				return $this->_getValue($v, $index);
132 134
 			}, $property);
@@ -264,7 +266,7 @@  discard block
 block discarded – undo
264 266
 					$this->setInstanceProperty($property);
265 267
 				}
266 268
 			}
267
-		}catch (\Throwable $th){
269
+		} catch (\Throwable $th){
268 270
 			
269 271
 		}
270 272
 		return $this;
@@ -284,10 +286,11 @@  discard block
 block discarded – undo
284 286
 			}
285 287
 		} elseif (\is_int($property)) {
286 288
 			$props = $this->getDefaultProperties();
287
-			if (isset($props[$property]))
288
-				$this->properties[] = $props[$property];
289
-			else
290
-				$this->properties[] = $property;
289
+			if (isset($props[$property])) {
290
+							$this->properties[] = $props[$property];
291
+			} else {
292
+							$this->properties[] = $property;
293
+			}
291 294
 		} else {
292 295
 			$this->properties[] = $property;
293 296
 		}
@@ -332,12 +335,13 @@  discard block
 block discarded – undo
332 335
 		if (isset($this->captions[$index])) {
333 336
 			return $this->captions[$index];
334 337
 		}
335
-		if ($this->properties[$index] instanceof \ReflectionProperty)
336
-			return $this->properties[$index]->getName();
337
-		elseif (\is_callable($this->properties[$index]))
338
-			return "";
339
-		else
340
-			return $this->properties[$index];
338
+		if ($this->properties[$index] instanceof \ReflectionProperty) {
339
+					return $this->properties[$index]->getName();
340
+		} elseif (\is_callable($this->properties[$index])) {
341
+					return "";
342
+		} else {
343
+					return $this->properties[$index];
344
+		}
341 345
 	}
342 346
 
343 347
 	public function getCaptions() {
@@ -370,8 +374,9 @@  discard block
 block discarded – undo
370 374
 	}
371 375
 
372 376
 	public function setCaption($index, $caption) {
373
-		if (isset($this->captions) === false)
374
-			$this->captions = [];
377
+		if (isset($this->captions) === false) {
378
+					$this->captions = [];
379
+		}
375 380
 		$this->captions[$index] = $caption;
376 381
 		return $this;
377 382
 	}
Please login to merge, or discard this patch.
Ajax/common/BaseEnum.php 1 patch
Braces   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -27,9 +27,9 @@
 block discarded – undo
27 27
 	}
28 28
 
29 29
 	public static function getConstantValues($postFix = "", $prefixBefore = false) {
30
-		if ($postFix == "")
31
-			return \array_values(self::getConstants());
32
-		else {
30
+		if ($postFix == "") {
31
+					return \array_values(self::getConstants());
32
+		} else {
33 33
 			if ($prefixBefore === false) {
34 34
 				return \array_map(function ($elem) use ($postFix) {
35 35
 					return $elem . " " . $postFix;
Please login to merge, or discard this patch.
Ajax/semantic/widgets/datatable/DataTable.php 1 patch
Braces   +33 added lines, -22 removed lines patch added patch discarded remove patch
@@ -89,10 +89,12 @@  discard block
 block discarded – undo
89 89
 					"stopPropagation" => true
90 90
 				]);
91 91
 			}
92
-			if (\is_array($this->_deleteBehavior))
93
-				$this->_generateBehavior("delete", $this->_deleteBehavior, $js);
94
-			if (\is_array($this->_editBehavior))
95
-				$this->_generateBehavior("edit", $this->_editBehavior, $js);
92
+			if (\is_array($this->_deleteBehavior)) {
93
+							$this->_generateBehavior("delete", $this->_deleteBehavior, $js);
94
+			}
95
+			if (\is_array($this->_editBehavior)) {
96
+							$this->_generateBehavior("edit", $this->_editBehavior, $js);
97
+			}
96 98
 			if (\is_array($this->_displayBehavior)) {
97 99
 				$this->_generateBehavior("display", $this->_displayBehavior, $js);
98 100
 			}
@@ -150,8 +152,9 @@  discard block
 block discarded – undo
150 152
 			$table->setRowCount(0, \count($captions));
151 153
 			$this->_generateHeader($table, $captions);
152 154
 
153
-			if (isset($this->_compileParts))
154
-				$table->setCompileParts($this->_compileParts);
155
+			if (isset($this->_compileParts)) {
156
+							$table->setCompileParts($this->_compileParts);
157
+			}
155 158
 
156 159
 			$this->_generateContent($table);
157 160
 
@@ -189,8 +192,9 @@  discard block
 block discarded – undo
189 192
 	}
190 193
 
191 194
 	protected function _applyStyleAttributes($table) {
192
-		if (isset($this->_hiddenColumns))
193
-			$this->_hideColumns();
195
+		if (isset($this->_hiddenColumns)) {
196
+					$this->_hideColumns();
197
+		}
194 198
 		if (isset($this->_colWidths)) {
195 199
 			foreach ($this->_colWidths as $colIndex => $width) {
196 200
 				$table->setColWidth($colIndex, $width);
@@ -303,8 +307,9 @@  discard block
 block discarded – undo
303 307
 			$field = $ck->getField();
304 308
 			$field->setProperty("value", $dataAjax);
305 309
 			$field->setProperty("name", "selection[]");
306
-			if (isset($checkedClass))
307
-				$field->setClass($checkedClass);
310
+			if (isset($checkedClass)) {
311
+							$field->setClass($checkedClass);
312
+			}
308 313
 			\array_unshift($values, $ck);
309 314
 		}
310 315
 		$result = $table->newRow();
@@ -321,8 +326,9 @@  discard block
 block discarded – undo
321 326
 
322 327
 	protected function _generatePagination($table) {
323 328
 		if (isset($this->_toolbar)) {
324
-			if ($this->_toolbarPosition == PositionInTable::FOOTER)
325
-				$this->_toolbar->setFloated("left");
329
+			if ($this->_toolbarPosition == PositionInTable::FOOTER) {
330
+							$this->_toolbar->setFloated("left");
331
+			}
326 332
 		}
327 333
 		$footer = $table->getFooter();
328 334
 		$footer->mergeCol();
@@ -364,8 +370,9 @@  discard block
 block discarded – undo
364 370
 
365 371
 	protected function _getFieldName($index) {
366 372
 		$fieldName = parent::_getFieldName($index);
367
-		if (\is_object($fieldName))
368
-			$fieldName = "field-" . $index;
373
+		if (\is_object($fieldName)) {
374
+					$fieldName = "field-" . $index;
375
+		}
369 376
 		if ($this->_namePrefix != null) {
370 377
 			$fieldName = $this->_namePrefix . '.' . $fieldName;
371 378
 		}
@@ -557,8 +564,9 @@  discard block
 block discarded – undo
557 564
 
558 565
 	protected function getTargetSelector($op) {
559 566
 		$result = $this->_targetSelector;
560
-		if (! isset($result[$op]))
561
-			$result = "#" . $this->identifier;
567
+		if (! isset($result[$op])) {
568
+					$result = "#" . $this->identifier;
569
+		}
562 570
 		return $result[$op];
563 571
 	}
564 572
 
@@ -581,8 +589,9 @@  discard block
 block discarded – undo
581 589
 	}
582 590
 
583 591
 	public function getRefreshSelector() {
584
-		if (isset($this->_refreshSelector))
585
-			return $this->_refreshSelector;
592
+		if (isset($this->_refreshSelector)) {
593
+					return $this->_refreshSelector;
594
+		}
586 595
 		return "#" . $this->identifier . " tbody";
587 596
 	}
588 597
 
@@ -603,8 +612,9 @@  discard block
 block discarded – undo
603 612
 	 */
604 613
 	public function show($modelInstance) {
605 614
 		if (\is_array($modelInstance)) {
606
-			if (isset($modelInstance[0]) && \is_array(array_values($modelInstance)[0]))
607
-				$modelInstance = \json_decode(\json_encode($modelInstance), FALSE);
615
+			if (isset($modelInstance[0]) && \is_array(array_values($modelInstance)[0])) {
616
+							$modelInstance = \json_decode(\json_encode($modelInstance), FALSE);
617
+			}
608 618
 		}
609 619
 		$this->_modelInstance = $modelInstance;
610 620
 	}
@@ -654,8 +664,9 @@  discard block
 block discarded – undo
654 664
 	}
655 665
 
656 666
 	public function hideColumn($colIndex) {
657
-		if (! \is_array($this->_hiddenColumns))
658
-			$this->_hiddenColumns = [];
667
+		if (! \is_array($this->_hiddenColumns)) {
668
+					$this->_hiddenColumns = [];
669
+		}
659 670
 		$this->_hiddenColumns[] = $colIndex;
660 671
 		return $this;
661 672
 	}
Please login to merge, or discard this patch.
Ajax/semantic/html/collections/table/HtmlTable.php 1 patch
Braces   +9 added lines, -6 removed lines patch added patch discarded remove patch
@@ -444,8 +444,9 @@  discard block
 block discarded – undo
444 444
 			$result = $this->getBody()->_addRow($result);
445 445
 		}
446 446
 		if (isset($this->_afterCompileEvents["onNewRow"])) {
447
-			if (\is_callable($this->_afterCompileEvents["onNewRow"]))
448
-				$this->_afterCompileEvents["onNewRow"]($result, $object);
447
+			if (\is_callable($this->_afterCompileEvents["onNewRow"])) {
448
+							$this->_afterCompileEvents["onNewRow"]($result, $object);
449
+			}
449 450
 		}
450 451
 		return $result;
451 452
 	}
@@ -483,8 +484,9 @@  discard block
 block discarded – undo
483 484
 			}
484 485
 		}
485 486
 		$result = parent::run($js);
486
-		if (isset($this->_footer))
487
-			$this->_footer->run($js);
487
+		if (isset($this->_footer)) {
488
+					$this->_footer->run($js);
489
+		}
488 490
 		$this->_runned = true;
489 491
 		return $result;
490 492
 	}
@@ -538,8 +540,9 @@  discard block
 block discarded – undo
538 540
 
539 541
 	public function setColWidth($colIndex, $width) {
540 542
 		$part = $this->_getFirstPart();
541
-		if ($part !== null && $part->count() > 0)
542
-			$part->getCell(0, $colIndex)->setWidth($width);
543
+		if ($part !== null && $part->count() > 0) {
544
+					$part->getCell(0, $colIndex)->setWidth($width);
545
+		}
543 546
 		return $this;
544 547
 	}
545 548
 
Please login to merge, or discard this patch.