Passed
Push — master ( a57f3d...497376 )
by Jean-Christophe
02:17
created
Ajax/semantic/html/modules/HtmlTab.php 1 patch
Braces   +9 added lines, -6 removed lines patch added patch discarded remove patch
@@ -86,8 +86,9 @@  discard block
 block discarded – undo
86 86
 			if (isset($this->content[$index]) === false) {
87 87
 				$this->content[$index] = $this->createSegment($index, $content, $menu->getItem($index)
88 88
 					->getIdentifier());
89
-			} else
90
-				$this->content[$index]->setContent($content);
89
+			} else {
90
+							$this->content[$index]->setContent($content);
91
+			}
91 92
 		}
92 93
 		return $this;
93 94
 	}
@@ -281,15 +282,17 @@  discard block
 block discarded – undo
281 282
 	 * @see BaseHtml::run()
282 283
 	 */
283 284
 	public function run(JsUtils $js) {
284
-		if (isset($this->_bsComponent) === false)
285
-			$this->_bsComponent = $js->semantic()->tab("#" . $this->identifier . " .item", $this->params);
285
+		if (isset($this->_bsComponent) === false) {
286
+					$this->_bsComponent = $js->semantic()->tab("#" . $this->identifier . " .item", $this->params);
287
+		}
286 288
 		$this->addEventsOnRun($js);
287 289
 		return $this->_bsComponent;
288 290
 	}
289 291
 
290 292
 	public function compile(JsUtils $js = NULL, &$view = NULL) {
291
-		if (! $this->_activated && $this->content["menu"]->count() > 0 && \sizeof($this->content) > 1)
292
-			$this->activate(0);
293
+		if (! $this->_activated && $this->content["menu"]->count() > 0 && \sizeof($this->content) > 1) {
294
+					$this->activate(0);
295
+		}
293 296
 		return parent::compile($js, $view);
294 297
 	}
295 298
 	
Please login to merge, or discard this patch.
Ajax/common/Widget.php 1 patch
Braces   +6 added lines, -4 removed lines patch added patch discarded remove patch
@@ -288,8 +288,9 @@  discard block
 block discarded – undo
288 288
 	 */
289 289
 	public function addItemInToolbar($caption, $icon = NULL, $callback = NULL) {
290 290
 		$result = $this->addInToolbar($caption, $callback);
291
-		if (isset($icon) && method_exists($result, "addIcon"))
292
-			$result->addIcon($icon);
291
+		if (isset($icon) && method_exists($result, "addIcon")) {
292
+					$result->addIcon($icon);
293
+		}
293 294
 		return $result;
294 295
 	}
295 296
 
@@ -478,8 +479,9 @@  discard block
 block discarded – undo
478 479
 	protected function _compileForm() {
479 480
 		if (isset($this->_form)) {
480 481
 			$noValidate = "";
481
-			if (\sizeof($this->_form->getValidationParams()) > 0)
482
-				$noValidate = "novalidate";
482
+			if (\sizeof($this->_form->getValidationParams()) > 0) {
483
+							$noValidate = "novalidate";
484
+			}
483 485
 				$this->wrapContent("<form class='".$this->_form->getProperty('class')."' id='frm-" . $this->identifier . "' name='frm-" . $this->identifier . "' " . $noValidate . ">", "</form>");
484 486
 		}
485 487
 	}
Please login to merge, or discard this patch.
Ajax/semantic/components/validation/FieldValidation.php 1 patch
Braces   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -62,12 +62,12 @@
 block discarded – undo
62 62
 				$this->customRules[]=$type;
63 63
 				$this->hasCustomRules=true;
64 64
 			}
65
-		}elseif(\is_array($type)){
65
+		} elseif(\is_array($type)){
66 66
 			$value=JArray::getValue($type, "value", 2);
67 67
 			$prompt=JArray::getValue($type, "prompt", 1);
68 68
 			$type=JArray::getValue($type, "type", 0);
69 69
 			$rule=new Rule($type,$prompt,$value);
70
-		}else {
70
+		} else {
71 71
 			$rule = new Rule($type, $prompt, $value);
72 72
 		}
73 73
 		$this->rules[]=$rule;
Please login to merge, or discard this patch.
Ajax/common/html/HtmlDoubleElement.php 1 patch
Braces   +16 added lines, -12 removed lines patch added patch discarded remove patch
@@ -35,17 +35,19 @@  discard block
 block discarded – undo
35 35
 
36 36
 	public function addContent($content, $before = false) {
37 37
 		if (! \is_array($this->content)) {
38
-			if (isset($this->content))
39
-				$this->content = array(
38
+			if (isset($this->content)) {
39
+							$this->content = array(
40 40
 					$this->content
41 41
 				);
42
-			else
43
-				$this->content = array();
42
+			} else {
43
+							$this->content = array();
44
+			}
45
+		}
46
+		if ($before) {
47
+					\array_unshift($this->content, $content);
48
+		} else {
49
+					$this->content[] = $content;
44 50
 		}
45
-		if ($before)
46
-			\array_unshift($this->content, $content);
47
-		else
48
-			$this->content[] = $content;
49 51
 		return $this;
50 52
 	}
51 53
 
@@ -108,10 +110,12 @@  discard block
 block discarded – undo
108 110
 	 * @return HtmlDoubleElement
109 111
 	 */
110 112
 	public function asLink($href = NULL, $target = NULL) {
111
-		if (isset($href))
112
-			$this->setProperty("href", $href);
113
-		if (isset($target))
114
-			$this->setProperty("target", $target);
113
+		if (isset($href)) {
114
+					$this->setProperty("href", $href);
115
+		}
116
+		if (isset($target)) {
117
+					$this->setProperty("target", $target);
118
+		}
115 119
 		return $this->setTagName("a");
116 120
 	}
117 121
 
Please login to merge, or discard this patch.
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.