Passed
Push — master ( 468e47...0bebef )
by Jean-Christophe
02:09
created
Ajax/common/html/traits/BaseHtmlPropertiesTrait.php 1 patch
Braces   +17 added lines, -12 removed lines patch added patch discarded remove patch
@@ -33,8 +33,9 @@  discard block
 block discarded – undo
33 33
 	}
34 34
 
35 35
 	public function getProperty($name) {
36
-		if (array_key_exists($name, $this->_self->properties))
37
-			return $this->_self->properties[$name];
36
+		if (array_key_exists($name, $this->_self->properties)) {
37
+					return $this->_self->properties[$name];
38
+		}
38 39
 	}
39 40
 
40 41
 	/**
@@ -51,10 +52,11 @@  discard block
 block discarded – undo
51 52
 		} else if ($value !== "" && $this->_self->propertyContains($name, $value) === false) {
52 53
 			if(isset($this->_self->properties[$name])){
53 54
 				$v=$this->_self->properties[$name];
54
-				if (isset($v) && $v !== "")
55
-					$v=$v . $separator . $value;
56
-					else
57
-						$v=$value;
55
+				if (isset($v) && $v !== "") {
56
+									$v=$v . $separator . $value;
57
+				} else {
58
+											$v=$value;
59
+					}
58 60
 	
59 61
 				return $this->_self->setProperty($name, $v);
60 62
 			}
@@ -78,8 +80,9 @@  discard block
 block discarded – undo
78 80
 	}
79 81
 
80 82
 	protected function addToPropertyUnique($name, $value, $typeCtrl) {
81
-		if (is_string($typeCtrl) && @class_exists($typeCtrl, true))
82
-			$typeCtrl=$typeCtrl::getConstants();
83
+		if (is_string($typeCtrl) && @class_exists($typeCtrl, true)) {
84
+					$typeCtrl=$typeCtrl::getConstants();
85
+		}
83 86
 			if (\is_array($typeCtrl)) {
84 87
 				$this->_self->removeOldValues($this->_self->properties[$name], $typeCtrl);
85 88
 			}
@@ -98,8 +101,9 @@  discard block
 block discarded – undo
98 101
 	}
99 102
 
100 103
 	public function removeProperty($name) {
101
-		if (\array_key_exists($name, $this->_self->properties))
102
-			unset($this->_self->properties[$name]);
104
+		if (\array_key_exists($name, $this->_self->properties)) {
105
+					unset($this->_self->properties[$name]);
106
+		}
103 107
 			return $this;
104 108
 	}
105 109
 
@@ -112,8 +116,9 @@  discard block
 block discarded – undo
112 116
 	}
113 117
 
114 118
 	protected function setPropertyCtrl($name, $value, $typeCtrl) {
115
-		if ($this->_self->ctrl($name, $value, $typeCtrl) === true)
116
-			return $this->_self->setProperty($name, $value);
119
+		if ($this->_self->ctrl($name, $value, $typeCtrl) === true) {
120
+					return $this->_self->setProperty($name, $value);
121
+		}
117 122
 			return $this;
118 123
 	}
119 124
 
Please login to merge, or discard this patch.
Ajax/bootstrap/html/HtmlPanel.php 1 patch
Braces   +6 added lines, -4 removed lines patch added patch discarded remove patch
@@ -89,8 +89,9 @@  discard block
 block discarded – undo
89 89
 	 * @return \Ajax\bootstrap\html\HtmlPanel default : "panel-default"
90 90
 	 */
91 91
 	public function setStyle($cssStyle) {
92
-		if (!JString::startsWith($cssStyle, "panel"))
93
-			$cssStyle="panel".$cssStyle;
92
+		if (!JString::startsWith($cssStyle, "panel")) {
93
+					$cssStyle="panel".$cssStyle;
94
+		}
94 95
 		return $this->addToPropertyCtrl("class", $cssStyle, CssRef::Styles("panel"));
95 96
 	}
96 97
 
@@ -132,7 +133,8 @@  discard block
 block discarded – undo
132 133
 	 * @return $this default : false
133 134
 	 */
134 135
 	public function show($value) {
135
-		if ($this->_collapsable)
136
-			$this->_showOnStartup=$value;
136
+		if ($this->_collapsable) {
137
+					$this->_showOnStartup=$value;
138
+		}
137 139
 	}
138 140
 }
Please login to merge, or discard this patch.
Ajax/common/Widget.php 1 patch
Braces   +7 added lines, -5 removed lines patch added patch discarded remove patch
@@ -250,8 +250,9 @@  discard block
 block discarded – undo
250 250
 	 */
251 251
 	public function addItemInToolbar($caption,$icon=NULL,$callback=NULL){
252 252
 		$result=$this->addInToolbar($caption,$callback);
253
-		if(isset($icon) && method_exists($result, "addIcon"))
254
-			$result->addIcon($icon);
253
+		if(isset($icon) && method_exists($result, "addIcon")) {
254
+					$result->addIcon($icon);
255
+		}
255 256
 		return $result;
256 257
 	}
257 258
 
@@ -265,7 +266,7 @@  discard block
 block discarded – undo
265 266
 			foreach ($items as $icon=>$item){
266 267
 				$this->addItemInToolbar($item,$icon,$callback);
267 268
 			}
268
-		}else{
269
+		} else{
269 270
 			foreach ($items as $item){
270 271
 				$this->addItemInToolbar($item,null,$callback);
271 272
 			}
@@ -416,8 +417,9 @@  discard block
 block discarded – undo
416 417
 	protected function _compileForm(){
417 418
 		if(isset($this->_form)){
418 419
 			$noValidate="";
419
-			if(\sizeof($this->_form->getValidationParams())>0)
420
-				$noValidate="novalidate";
420
+			if(\sizeof($this->_form->getValidationParams())>0) {
421
+							$noValidate="novalidate";
422
+			}
421 423
 			$this->wrapContent("<form class='ui form' id='frm-".$this->identifier."' name='frm-".$this->identifier."' ".$noValidate.">","</form>");
422 424
 		}
423 425
 	}
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
@@ -63,15 +63,17 @@  discard block
 block discarded – undo
63 63
 	 */
64 64
 	private function getFieldButton($caption,$visibleHover=true){
65 65
 		$bt= new HtmlButton($this->cleanIdentifier($caption),$caption);
66
-		if($visibleHover)
67
-			$this->_visibleOver($bt);
66
+		if($visibleHover) {
67
+					$this->_visibleOver($bt);
68
+		}
68 69
 		return $bt;
69 70
 	}
70 71
 
71 72
 	private function getFieldButtons($buttons,$visibleHover=true){
72 73
 		$bts=new HtmlButtonGroups("",$buttons);
73
-		if($visibleHover)
74
-			$this->_visibleOver($bts);
74
+		if($visibleHover) {
75
+					$this->_visibleOver($bts);
76
+		}
75 77
 		return $bts;
76 78
 	}
77 79
 
@@ -88,8 +90,9 @@  discard block
 block discarded – undo
88 90
 		return $this->_fieldAs(function($id,$name,$value,$caption) use ($url,$responseElement,$cssStyle,$attributes){
89 91
 			$button=new HtmlButton($id,$value,$cssStyle);
90 92
 			$button->postOnClick($url,"$(event.target).closest('tr').find(':input').serialize()",$responseElement,$attributes["ajax"]);
91
-			if(!isset($attributes["visibleHover"]) || $attributes["visibleHover"])
92
-				$this->_visibleOver($button);
93
+			if(!isset($attributes["visibleHover"]) || $attributes["visibleHover"]) {
94
+							$this->_visibleOver($button);
95
+			}
93 96
 				return $button;
94 97
 		}, $index,$attributes);
95 98
 	}
@@ -160,8 +163,9 @@  discard block
 block discarded – undo
160 163
 	private function getDefaultButton($icon,$class=null,$visibleHover=true){
161 164
 		$bt=$this->getFieldButton("",$visibleHover);
162 165
 		$bt->asIcon($icon);
163
-		if(isset($class))
164
-			$bt->addClass($class);
166
+		if(isset($class)) {
167
+					$bt->addClass($class);
168
+		}
165 169
 		return $bt;
166 170
 	}
167 171
 
Please login to merge, or discard this patch.
Ajax/semantic/html/content/HtmlMenuItem.php 1 patch
Braces   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -17,7 +17,7 @@  discard block
 block discarded – undo
17 17
 			if(JArray::isAssociative($content)===false){
18 18
 				$icon=@$content[0];
19 19
 				$title=@$content[1];
20
-			}else{
20
+			} else{
21 21
 				$icon=@$content["icon"];
22 22
 				$title=@$content["title"];
23 23
 			}
@@ -27,7 +27,7 @@  discard block
 block discarded – undo
27 27
 			if(isset($title)){
28 28
 				$this->setTitle($title);
29 29
 			}
30
-		}else{
30
+		} else{
31 31
 			$this->setContent($content);
32 32
 		}
33 33
 	}
Please login to merge, or discard this patch.
Ajax/bootstrap/html/content/HtmlListgroupItem.php 1 patch
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -40,8 +40,9 @@
 block discarded – undo
40 40
 	 * @return HtmlBsDoubleElement default : "list-group-item-default"
41 41
 	 */
42 42
 	public function setStyle($cssStyle) {
43
-		if (!JString::startsWith($cssStyle, "list-group-item"))
44
-			$cssStyle="list-group-item".$cssStyle;
43
+		if (!JString::startsWith($cssStyle, "list-group-item")) {
44
+					$cssStyle="list-group-item".$cssStyle;
45
+		}
45 46
 		$this->element->addToPropertyCtrl("class", $cssStyle, CssRef::Styles("list-group-item"));
46 47
 		return $this->element;
47 48
 	}
Please login to merge, or discard this patch.
Ajax/semantic/html/content/table/HtmlTableContent.php 1 patch
Braces   +22 added lines, -15 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
 	/**
@@ -103,7 +104,7 @@  discard block
 block discarded – undo
103 104
 		$row=$this->getItem($row);
104 105
 		if (isset($row) && $row instanceof HtmlCollection) {
105 106
 			$col=$row->getItem($col);
106
-		}else{
107
+		} else{
107 108
 			$col=$row;
108 109
 		}
109 110
 		return $col;
@@ -161,8 +162,9 @@  discard block
 block discarded – undo
161 162
 			$values=\array_fill(0, $count, $values);
162 163
 			$isArray=false;
163 164
 		}
164
-		if (JArray::dimension($values) == 1 && $isArray)
165
-			$values=[ $values ];
165
+		if (JArray::dimension($values) == 1 && $isArray) {
166
+					$values=[ $values ];
167
+		}
166 168
 
167 169
 		$count=\min(\sizeof($values), $count);
168 170
 
@@ -189,8 +191,9 @@  discard block
 block discarded – undo
189 191
 		$count=$this->count();
190 192
 		for($i=0; $i < $count; $i++) {
191 193
 			$cell=$this->getCell($i, $colIndex);
192
-			if($cell instanceof BaseTrait)
193
-				$cell->addVariations($variations);
194
+			if($cell instanceof BaseTrait) {
195
+							$cell->addVariations($variations);
196
+			}
194 197
 		}
195 198
 		return $this;
196 199
 	}
@@ -208,8 +211,9 @@  discard block
 block discarded – undo
208 211
 		$count=$this->count();
209 212
 		for($i=0; $i < $count; $i++) {
210 213
 			$index=$this->content[$i]->getColPosition($colIndex);
211
-			if ($index !== NULL)
212
-				$this->getCell($i, $index)->$function();
214
+			if ($index !== NULL) {
215
+							$this->getCell($i, $index)->$function();
216
+			}
213 217
 		}
214 218
 		return $this;
215 219
 	}
@@ -240,8 +244,9 @@  discard block
 block discarded – undo
240 244
 	 */
241 245
 	public function getColCount() {
242 246
 		$result=0;
243
-		if ($this->count() > 0)
244
-			$result=$this->getItem(0)->count();
247
+		if ($this->count() > 0) {
248
+					$result=$this->getItem(0)->count();
249
+		}
245 250
 		return $result;
246 251
 	}
247 252
 
@@ -265,8 +270,9 @@  discard block
 block discarded – undo
265 270
 
266 271
 	public function toDelete($rowIndex, $colIndex){
267 272
 		$row=$this->getItem($rowIndex);
268
-		if (isset($row) === true)
269
-			$row->toDelete($colIndex);
273
+		if (isset($row) === true) {
274
+					$row->toDelete($colIndex);
275
+		}
270 276
 		return $this;
271 277
 	}
272 278
 
@@ -364,8 +370,9 @@  discard block
 block discarded – undo
364 370
 		foreach ( $rows as $row ) {
365 371
 			$cell=$row->getItem($colIndex);
366 372
 			$value=$cell->getContent();
367
-			if($functionExists)
368
-				$value=\call_user_func($function,$value);
373
+			if($functionExists) {
374
+							$value=\call_user_func($function,$value);
375
+			}
369 376
 			if($value!==$identiqual){
370 377
 				if($counter>0 && isset($cellToMerge)){
371 378
 					$cellToMerge->setRowspan($counter);
Please login to merge, or discard this patch.
Ajax/semantic/html/collections/HtmlGrid.php 1 patch
Braces   +27 added lines, -18 removed lines patch added patch discarded remove patch
@@ -29,8 +29,9 @@  discard block
 block discarded – undo
29 29
 			$this->_colSizing=false;
30 30
 			$this->setWide($numCols);
31 31
 		}
32
-		if($createCols)
33
-			$this->setRowsCount($numRows, $numCols);
32
+		if($createCols) {
33
+					$this->setRowsCount($numRows, $numCols);
34
+		}
34 35
 	}
35 36
 
36 37
 	public function asSegment() {
@@ -82,8 +83,9 @@  discard block
 block discarded – undo
82 83
 	public function addCol($width=NULL) {
83 84
 		$colCount=$this->colCount() + 1;
84 85
 		$this->setColsCount($colCount, true, $width);
85
-		if ($this->hasOnlyCols($this->count()))
86
-			return $this->content[$colCount - 1];
86
+		if ($this->hasOnlyCols($this->count())) {
87
+					return $this->content[$colCount - 1];
88
+		}
87 89
 		return $this;
88 90
 	}
89 91
 
@@ -178,8 +180,9 @@  discard block
 block discarded – undo
178 180
 	 */
179 181
 	public function rowCount() {
180 182
 		$count=$this->count();
181
-		if ($this->hasOnlyCols($count))
182
-			return 0;
183
+		if ($this->hasOnlyCols($count)) {
184
+					return 0;
185
+		}
183 186
 		return $count;
184 187
 	}
185 188
 
@@ -189,10 +192,12 @@  discard block
 block discarded – undo
189 192
 	 */
190 193
 	public function colCount() {
191 194
 		$count=$this->count();
192
-		if ($this->hasOnlyCols($count))
193
-			return $count;
194
-		if ($count > 0)
195
-			return $this->getItem(0)->count();
195
+		if ($this->hasOnlyCols($count)) {
196
+					return $count;
197
+		}
198
+		if ($count > 0) {
199
+					return $this->getItem(0)->count();
200
+		}
196 201
 		return 0;
197 202
 	}
198 203
 
@@ -203,8 +208,9 @@  discard block
 block discarded – undo
203 208
 	 * @return HtmlGridCol|HtmlGridRow
204 209
 	 */
205 210
 	public function getCell($row, $col) {
206
-		if ($row < 2 && $this->hasOnlyCols($this->count()))
207
-			return $this->getItem($col);
211
+		if ($row < 2 && $this->hasOnlyCols($this->count())) {
212
+					return $this->getItem($col);
213
+		}
208 214
 		$rowO=$this->getItem($row);
209 215
 		if (isset($rowO)) {
210 216
 			$colO=$rowO->getItem($col);
@@ -253,8 +259,9 @@  discard block
 block discarded – undo
253 259
 	 * @return \Ajax\semantic\html\collections\HtmlGrid
254 260
 	 */
255 261
 	public function setPadded($value=NULL) {
256
-		if (isset($value))
257
-			$this->addToPropertyCtrl("class", $value, array ("vertically","horizontally" ));
262
+		if (isset($value)) {
263
+					$this->addToPropertyCtrl("class", $value, array ("vertically","horizontally" ));
264
+		}
258 265
 		return $this->addToProperty("class", "padded");
259 266
 	}
260 267
 
@@ -279,8 +286,9 @@  discard block
 block discarded – undo
279 286
 	 * @see \Ajax\common\html\HtmlCollection::createItem()
280 287
 	 */
281 288
 	protected function createItem($value) {
282
-		if ($this->_createCols === false)
283
-			$value=null;
289
+		if ($this->_createCols === false) {
290
+					$value=null;
291
+		}
284 292
 		$item=new HtmlGridRow($this->identifier . "-row-" . ($this->count() + 1), $value, $this->_colSizing, $this->_implicitRows);
285 293
 		return $item;
286 294
 	}
@@ -328,8 +336,9 @@  discard block
 block discarded – undo
328 336
 	 */
329 337
 	public function addDivider($afterColIndex, $vertical=true, $content=NULL) {
330 338
 		$col=$this->getCell(0, $afterColIndex);
331
-		if($col instanceof HtmlGridCol)
332
-			$col->addDivider($vertical, $content);
339
+		if($col instanceof HtmlGridCol) {
340
+					$col->addDivider($vertical, $content);
341
+		}
333 342
 		return $this;
334 343
 	}
335 344
 }
Please login to merge, or discard this patch.
Ajax/semantic/html/elements/HtmlButton.php 1 patch
Braces   +17 added lines, -13 removed lines patch added patch discarded remove patch
@@ -56,9 +56,9 @@  discard block
 block discarded – undo
56 56
 	}
57 57
 
58 58
 	public function setFocusable($value=true) {
59
-		if ($value === true)
60
-			$this->setProperty("tabindex", "0");
61
-		else {
59
+		if ($value === true) {
60
+					$this->setProperty("tabindex", "0");
61
+		} else {
62 62
 			$this->removeProperty("tabindex");
63 63
 		}
64 64
 		return $this;
@@ -106,8 +106,9 @@  discard block
 block discarded – undo
106 106
 	 */
107 107
 	public function addLabel($label, $before=false, $icon=NULL) {
108 108
 		$this->tagName="div";$prefix="";
109
-		if($before)
110
-			$prefix="left ";
109
+		if($before) {
110
+					$prefix="left ";
111
+		}
111 112
 		$this->addToProperty("class", $prefix."labeled");
112 113
 		$isIcon=(isset($this->content[0]) && $this->content[0] instanceof HtmlIcon);
113 114
 		$this->content=new HtmlButton("button-" . $this->identifier, $this->content);
@@ -144,12 +145,13 @@  discard block
 block discarded – undo
144 145
 	public function setColor($color){
145 146
 		if(\is_array($this->content)){
146 147
 			foreach ($this->content as $content){
147
-				if($content instanceof HtmlButton)
148
-					$content->setColor($color);
148
+				if($content instanceof HtmlButton) {
149
+									$content->setColor($color);
150
+				}
149 151
 			}
152
+		} else {
153
+					parent::setColor($color);
150 154
 		}
151
-		else
152
-			parent::setColor($color);
153 155
 		return $this;
154 156
 	}
155 157
 
@@ -202,8 +204,9 @@  discard block
 block discarded – undo
202 204
 	 * @return HtmlButton
203 205
 	 */
204 206
 	public static function social($identifier, $social, $value=NULL) {
205
-		if ($value === NULL)
206
-			$value=\ucfirst($social);
207
+		if ($value === NULL) {
208
+					$value=\ucfirst($social);
209
+		}
207 210
 		$return=new HtmlButton($identifier, $value);
208 211
 		$return->addIcon($social);
209 212
 		return $return->addToPropertyCtrl("class", $social, Social::getConstants());
@@ -256,8 +259,9 @@  discard block
 block discarded – undo
256 259
 	public static function dropdown($identifier,$value,$items=[],$asCombo=false,$icon=null){
257 260
 		$result=new HtmlButtonGroups($identifier,[$value]);
258 261
 		$dd=$result->addDropdown($items,$asCombo);
259
-		if(isset($icon) && $dd instanceof HtmlDropdown)
260
-			$dd->setIcon($icon);
262
+		if(isset($icon) && $dd instanceof HtmlDropdown) {
263
+					$dd->setIcon($icon);
264
+		}
261 265
 		return $result;
262 266
 	}
263 267
 }
Please login to merge, or discard this patch.