Passed
Push — master ( 70921f...f0f16f )
by Jean-Christophe
01:59
created
Ajax/semantic/html/base/traits/BaseTrait.php 1 patch
Braces   +28 added lines, -20 removed lines patch added patch discarded remove patch
@@ -48,8 +48,9 @@  discard block
 block discarded – undo
48 48
 
49 49
 	public function setVariations($variations) {
50 50
 		$this->_self->setProperty("class", $this->_self->getBaseClass());
51
-		if (\is_string($variations))
52
-			$variations=\explode(" ", $variations);
51
+		if (\is_string($variations)) {
52
+					$variations=\explode(" ", $variations);
53
+		}
53 54
 		foreach ( $variations as $variation ) {
54 55
 			$this->_self->addVariation($variation);
55 56
 		}
@@ -62,8 +63,9 @@  discard block
 block discarded – undo
62 63
 	}
63 64
 
64 65
 	public function addVariations($variations=array()) {
65
-		if (\is_string($variations))
66
-			$variations=\explode(" ", $variations);
66
+		if (\is_string($variations)) {
67
+					$variations=\explode(" ", $variations);
68
+		}
67 69
 		foreach ( $variations as $variation ) {
68 70
 			$this->_self->addVariation($variation);
69 71
 		}
@@ -71,8 +73,9 @@  discard block
 block discarded – undo
71 73
 	}
72 74
 
73 75
 	public function addStates($states=array()) {
74
-		if (\is_string($states))
75
-			$states=\explode(" ", $states);
76
+		if (\is_string($states)) {
77
+					$states=\explode(" ", $states);
78
+		}
76 79
 		foreach ( $states as $state ) {
77 80
 			$this->_self->addState($state);
78 81
 		}
@@ -81,8 +84,9 @@  discard block
 block discarded – undo
81 84
 
82 85
 	public function setStates($states) {
83 86
 		$this->_self->setProperty("class", $this->_self->getBaseClass());
84
-		if (\is_string($states))
85
-			$states=\explode(" ", $states);
87
+		if (\is_string($states)) {
88
+					$states=\explode(" ", $states);
89
+		}
86 90
 		foreach ( $states as $state ) {
87 91
 			$this->_self->addState($state);
88 92
 		}
@@ -114,8 +118,9 @@  discard block
 block discarded – undo
114 118
 	 * @return HtmlSemDoubleElement
115 119
 	 */
116 120
 	public function setDisabled($disable=true) {
117
-		if($disable)
118
-			$this->_self->addToProperty("class", "disabled");
121
+		if($disable) {
122
+					$this->_self->addToProperty("class", "disabled");
123
+		}
119 124
 		return $this;
120 125
 	}
121 126
 
@@ -149,14 +154,16 @@  discard block
 block discarded – undo
149 154
 	 * @return HtmlSemDoubleElement
150 155
 	 */
151 156
 	public function setActive($value=true){
152
-		if($value)
153
-			$this->_self->addToProperty("class", "active");
157
+		if($value) {
158
+					$this->_self->addToProperty("class", "active");
159
+		}
154 160
 		return $this;
155 161
 	}
156 162
 
157 163
 	public function setAttached($value=true){
158
-		if($value)
159
-			$this->_self->addToPropertyCtrl("class", "attached", array ("attached" ));
164
+		if($value) {
165
+					$this->_self->addToPropertyCtrl("class", "attached", array ("attached" ));
166
+		}
160 167
 		return $this;
161 168
 	}
162 169
 
@@ -166,9 +173,9 @@  discard block
 block discarded – undo
166 173
 	public function setInverted($recursive=true) {
167 174
 		if($recursive===true){
168 175
 			$content=$this->_self->getContent();
169
-			if($content instanceof HtmlSemDoubleElement)
170
-				$content->setInverted($recursive);
171
-			elseif(\is_array($content) || $content instanceof \Traversable){
176
+			if($content instanceof HtmlSemDoubleElement) {
177
+							$content->setInverted($recursive);
178
+			} elseif(\is_array($content) || $content instanceof \Traversable){
172 179
 				foreach ($content as $elm){
173 180
 					if($elm instanceof  HtmlSemDoubleElement){
174 181
 						$elm->setInverted($recursive);
@@ -204,9 +211,10 @@  discard block
 block discarded – undo
204 211
 			if(isset($array[$key])){
205 212
 				$p=JString::replaceAtFirstAndLast($array[$key], $before, "", $after, "");
206 213
 				$array[$key]=$before.$p.$value.$after;
207
-			}else
208
-				$array[$key]=$before.$value.$after;
209
-		}else{
214
+			} else {
215
+							$array[$key]=$before.$value.$after;
216
+			}
217
+		} else{
210 218
 			$array[$key]=$value;
211 219
 		}
212 220
 		return $this;
Please login to merge, or discard this patch.
Ajax/semantic/html/content/table/HtmlTableContent.php 1 patch
Braces   +18 added lines, -12 removed lines patch added patch discarded remove patch
@@ -22,8 +22,9 @@  discard block
 block discarded – undo
22 22
 	 */
23 23
 	public function __construct($identifier, $tagName="tbody", $rowCount=NULL, $colCount=NULL) {
24 24
 		parent::__construct($identifier, $tagName, "");
25
-		if (isset($rowCount) && isset($colCount))
26
-			$this->setRowCount($rowCount, $colCount);
25
+		if (isset($rowCount) && isset($colCount)) {
26
+					$this->setRowCount($rowCount, $colCount);
27
+		}
27 28
 	}
28 29
 
29 30
 	/**
@@ -148,8 +149,9 @@  discard block
 block discarded – undo
148 149
 			$values=\array_fill(0, $count, $values);
149 150
 			$isArray=false;
150 151
 		}
151
-		if (JArray::dimension($values) == 1 && $isArray)
152
-			$values=[ $values ];
152
+		if (JArray::dimension($values) == 1 && $isArray) {
153
+					$values=[ $values ];
154
+		}
153 155
 
154 156
 		$count=\min(\sizeof($values), $count);
155 157
 
@@ -193,8 +195,9 @@  discard block
 block discarded – undo
193 195
 		$count=$this->count();
194 196
 		for($i=0; $i < $count; $i++) {
195 197
 			$index=$this->content[$i]->getColPosition($colIndex);
196
-			if ($index !== NULL)
197
-				$this->getCell($i, $index)->$function();
198
+			if ($index !== NULL) {
199
+							$this->getCell($i, $index)->$function();
200
+			}
198 201
 		}
199 202
 		return $this;
200 203
 	}
@@ -225,8 +228,9 @@  discard block
 block discarded – undo
225 228
 	 */
226 229
 	public function getColCount() {
227 230
 		$result=0;
228
-		if ($this->count() > 0)
229
-			$result=$this->getItem(0)->count();
231
+		if ($this->count() > 0) {
232
+					$result=$this->getItem(0)->count();
233
+		}
230 234
 		return $result;
231 235
 	}
232 236
 
@@ -250,8 +254,9 @@  discard block
 block discarded – undo
250 254
 
251 255
 	public function toDelete($rowIndex, $colIndex){
252 256
 		$row=$this->getItem($rowIndex);
253
-		if (isset($row) === true)
254
-			$row->toDelete($colIndex);
257
+		if (isset($row) === true) {
258
+					$row->toDelete($colIndex);
259
+		}
255 260
 		return $this;
256 261
 	}
257 262
 
@@ -349,8 +354,9 @@  discard block
 block discarded – undo
349 354
 		foreach ( $rows as $row ) {
350 355
 			$cell=$row->getItem($colIndex);
351 356
 			$value=$cell->getContent();
352
-			if($functionExists)
353
-				$value=\call_user_func($function,$value);
357
+			if($functionExists) {
358
+							$value=\call_user_func($function,$value);
359
+			}
354 360
 			if($value!==$identiqual){
355 361
 				if($counter>0 && isset($cellToMerge)){
356 362
 					$cellToMerge->setRowspan($counter);
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
@@ -55,9 +55,9 @@  discard block
 block discarded – undo
55 55
 	}
56 56
 
57 57
 	public function setFocusable($value=true) {
58
-		if ($value === true)
59
-			$this->setProperty("tabindex", "0");
60
-		else {
58
+		if ($value === true) {
59
+					$this->setProperty("tabindex", "0");
60
+		} else {
61 61
 			$this->removeProperty("tabindex");
62 62
 		}
63 63
 		return $this;
@@ -105,8 +105,9 @@  discard block
 block discarded – undo
105 105
 	 */
106 106
 	public function addLabel($label, $before=false, $icon=NULL) {
107 107
 		$this->tagName="div";$prefix="";
108
-		if($before)
109
-			$prefix="left ";
108
+		if($before) {
109
+					$prefix="left ";
110
+		}
110 111
 		$this->addToProperty("class", $prefix."labeled");
111 112
 		$isIcon=(isset($this->content[0]) && $this->content[0] instanceof HtmlIcon);
112 113
 		$this->content=new HtmlButton("button-" . $this->identifier, $this->content);
@@ -143,12 +144,13 @@  discard block
 block discarded – undo
143 144
 	public function setColor($color){
144 145
 		if(\is_array($this->content)){
145 146
 			foreach ($this->content as $content){
146
-				if($content instanceof HtmlButton)
147
-					$content->setColor($color);
147
+				if($content instanceof HtmlButton) {
148
+									$content->setColor($color);
149
+				}
148 150
 			}
151
+		} else {
152
+					parent::setColor($color);
149 153
 		}
150
-		else
151
-			parent::setColor($color);
152 154
 		return $this;
153 155
 	}
154 156
 
@@ -201,8 +203,9 @@  discard block
 block discarded – undo
201 203
 	 * @return HtmlButton
202 204
 	 */
203 205
 	public static function social($identifier, $social, $value=NULL) {
204
-		if ($value === NULL)
205
-			$value=\ucfirst($social);
206
+		if ($value === NULL) {
207
+					$value=\ucfirst($social);
208
+		}
206 209
 		$return=new HtmlButton($identifier, $value);
207 210
 		$return->addIcon($social);
208 211
 		return $return->addToPropertyCtrl("class", $social, Social::getConstants());
@@ -255,8 +258,9 @@  discard block
 block discarded – undo
255 258
 	public static function dropdown($identifier,$value,$items=[],$asCombo=false,$icon=null){
256 259
 		$result=new HtmlButtonGroups($identifier,[$value]);
257 260
 		$result->addDropdown($items,$asCombo);
258
-		if(isset($icon))
259
-			$result->setIcon($icon);
261
+		if(isset($icon)) {
262
+					$result->setIcon($icon);
263
+		}
260 264
 		return $result;
261 265
 	}
262 266
 }
Please login to merge, or discard this patch.
Ajax/semantic/html/content/HtmlAbsractItem.php 1 patch
Braces   +10 added lines, -7 removed lines patch added patch discarded remove patch
@@ -24,8 +24,9 @@  discard block
 block discarded – undo
24 24
 	}
25 25
 
26 26
 	public function removeIcon(){
27
-		if(isset($this->content["icon"]))
28
-			unset($this->content["icon"]);
27
+		if(isset($this->content["icon"])) {
28
+					unset($this->content["icon"]);
29
+		}
29 30
 		return $this;
30 31
 	}
31 32
 
@@ -63,7 +64,7 @@  discard block
 block discarded – undo
63 64
 			$this->setTagName("div");
64 65
 			$this->removeProperty("href");
65 66
 			$this->addToPropertyCtrl("class", "active", array("active"));
66
-		}else{
67
+		} else{
67 68
 			$this->removePropertyValue("class", "active");
68 69
 		}
69 70
 		return $this;
@@ -71,8 +72,9 @@  discard block
 block discarded – undo
71 72
 
72 73
 	public function asLink($href=NULL,$part=NULL){
73 74
 		$this->setTagName("a");
74
-		if(isset($href))
75
-			$this->setProperty("href", $href);
75
+		if(isset($href)) {
76
+					$this->setProperty("href", $href);
77
+		}
76 78
 		return $this;
77 79
 	}
78 80
 
@@ -83,8 +85,9 @@  discard block
 block discarded – undo
83 85
 	 * @see \Ajax\semantic\html\base\HtmlSemDoubleElement::compile()
84 86
 	 */
85 87
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
86
-		if(\is_array($this->content) && JArray::isAssociative($this->content))
87
-			$this->content=JArray::sortAssociative($this->content, [ "icon","image","content" ]);
88
+		if(\is_array($this->content) && JArray::isAssociative($this->content)) {
89
+					$this->content=JArray::sortAssociative($this->content, [ "icon","image","content" ]);
90
+		}
88 91
 		return parent::compile($js, $view);
89 92
 	}
90 93
 }
Please login to merge, or discard this patch.
Ajax/semantic/html/elements/HtmlStep.php 1 patch
Braces   +14 added lines, -11 removed lines patch added patch discarded remove patch
@@ -55,26 +55,29 @@
 block discarded – undo
55 55
 			for($i=$this->_startStep;$i<$count;$i++){
56 56
 				$step=$this->content[$i];
57 57
 				$step->removeStatus();
58
-				if($i<$activestep)
59
-					$step->setCompleted();
60
-				elseif ($i===$activestep)
61
-					$step->setActive();
62
-				else
63
-					$step->setDisabled();
58
+				if($i<$activestep) {
59
+									$step->setCompleted();
60
+				} elseif ($i===$activestep) {
61
+									$step->setActive();
62
+				} else {
63
+									$step->setDisabled();
64
+				}
64 65
 			}
65
-		}else{
66
+		} else{
66 67
 			foreach ($this->content as $step){
67 68
 				$step->removeStatus();
68 69
 			}
69
-			if($activestep<$count)
70
-				$this->content[$activestep]->setActive();
70
+			if($activestep<$count) {
71
+							$this->content[$activestep]->setActive();
72
+			}
71 73
 		}
72 74
 		return $this;
73 75
 	}
74 76
 
75 77
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
76
-		if(isset($this->_activeStep) && \is_numeric($this->_activeStep))
77
-			$this->defineActiveStep();
78
+		if(isset($this->_activeStep) && \is_numeric($this->_activeStep)) {
79
+					$this->defineActiveStep();
80
+		}
78 81
 		return parent::compile($js,$view);
79 82
 	}
80 83
 
Please login to merge, or discard this patch.
Ajax/semantic/html/content/HtmlListItem.php 1 patch
Braces   +6 added lines, -5 removed lines patch added patch discarded remove patch
@@ -17,7 +17,7 @@  discard block
 block discarded – undo
17 17
 				$icon=@$content[0];
18 18
 				$title=@$content[1];
19 19
 				$desc=@$content[2];
20
-			}else{
20
+			} else{
21 21
 				$icon=@$content["icon"];
22 22
 				$image=@$content["image"];
23 23
 				$title=@$content["title"];
@@ -33,20 +33,21 @@  discard block
 block discarded – undo
33 33
 			}
34 34
 			if(isset($title)){
35 35
 				$this->setTitle($title,$desc);
36
-			}elseif (isset($header)){
36
+			} elseif (isset($header)){
37 37
 				$this->setTitle($header,$desc,"header");
38 38
 			}
39 39
 			if(isset($items)){
40 40
 				$this->addList($items);
41 41
 			}
42
-		}else{
42
+		} else{
43 43
 			$this->setContent($content);
44 44
 		}
45 45
 	}
46 46
 	public function addList($items=array(),$ordered=false) {
47 47
 		$list=new HtmlList("", $items);
48
-		if($ordered)
49
-			$list->setOrdered();
48
+		if($ordered) {
49
+					$list->setOrdered();
50
+		}
50 51
 		$list->setClass("list");
51 52
 		$this->content["list"]=$list;
52 53
 		return $list;
Please login to merge, or discard this patch.
Ajax/semantic/html/elements/HtmlList.php 1 patch
Braces   +10 added lines, -7 removed lines patch added patch discarded remove patch
@@ -50,8 +50,9 @@  discard block
 block discarded – undo
50 50
 
51 51
 	protected function getItemToAdd($item){
52 52
 		$itemO=parent::getItemToAdd($item);
53
-		if($itemO instanceof AbstractCheckbox)
54
-			$itemO->addClass("item");
53
+		if($itemO instanceof AbstractCheckbox) {
54
+					$itemO->addClass("item");
55
+		}
55 56
 		return $itemO;
56 57
 	}
57 58
 
@@ -106,14 +107,15 @@  discard block
 block discarded – undo
106 107
 		if (isset($masterItem)) {
107 108
 			if(\is_array($masterItem)){
108 109
 				$masterO=new HtmlFormCheckbox("master-" . $identifier, @$masterItem[0],@$masterItem[1]);
109
-				if(isset($name))
110
-					$masterO->setName($name);
110
+				if(isset($name)) {
111
+									$masterO->setName($name);
112
+				}
111 113
 				if(isset($masterItem[1])){
112 114
 					if(\array_search($masterItem[1], $values)!==false){
113 115
 						$masterO->getDataField()->setProperty("checked", "");
114 116
 					}
115 117
 				}
116
-			}else{
118
+			} else{
117 119
 				$masterO=new HtmlFormCheckbox("master-" . $identifier, $masterItem);
118 120
 			}
119 121
 			if($notAllChecked){
@@ -130,8 +132,9 @@  discard block
 block discarded – undo
130 132
 			if (\array_search($val, $values) !== false) {
131 133
 				$itemO->getDataField()->setProperty("checked", "");
132 134
 			}
133
-			if(isset($name))
134
-				$itemO->setName($name);
135
+			if(isset($name)) {
136
+							$itemO->setName($name);
137
+			}
135 138
 			$itemO->setClass("item");
136 139
 			$fields[]=$itemO;
137 140
 		}
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
@@ -62,15 +62,17 @@  discard block
 block discarded – undo
62 62
 	 */
63 63
 	private function getFieldButton($caption,$visibleHover=true){
64 64
 		$bt= new HtmlButton($this->cleanIdentifier($caption),$caption);
65
-		if($visibleHover)
66
-			$this->_visibleOver($bt);
65
+		if($visibleHover) {
66
+					$this->_visibleOver($bt);
67
+		}
67 68
 		return $bt;
68 69
 	}
69 70
 
70 71
 	private function getFieldButtons($buttons,$visibleHover=true){
71 72
 		$bts=new HtmlButtonGroups("",$buttons);
72
-		if($visibleHover)
73
-			$this->_visibleOver($bts);
73
+		if($visibleHover) {
74
+					$this->_visibleOver($bts);
75
+		}
74 76
 		return $bts;
75 77
 	}
76 78
 
@@ -87,8 +89,9 @@  discard block
 block discarded – undo
87 89
 		return $this->_fieldAs(function($id,$name,$value,$caption) use ($url,$responseElement,$cssStyle,$index,$attributes){
88 90
 			$button=new HtmlButton($id,$value,$cssStyle);
89 91
 			$button->postOnClick($url,"$(event.target).closest('tr').find(':input').serialize()",$responseElement,$attributes["ajax"]);
90
-			if(!isset($attributes["visibleHover"]) || $attributes["visibleHover"])
91
-				$this->_visibleOver($button);
92
+			if(!isset($attributes["visibleHover"]) || $attributes["visibleHover"]) {
93
+							$this->_visibleOver($button);
94
+			}
92 95
 				return $button;
93 96
 		}, $index,$attributes);
94 97
 	}
@@ -159,8 +162,9 @@  discard block
 block discarded – undo
159 162
 	private function getDefaultButton($icon,$class=null,$visibleHover=true){
160 163
 		$bt=$this->getFieldButton("",$visibleHover);
161 164
 		$bt->asIcon($icon);
162
-		if(isset($class))
163
-			$bt->addClass($class);
165
+		if(isset($class)) {
166
+					$bt->addClass($class);
167
+		}
164 168
 		return $bt;
165 169
 	}
166 170
 
Please login to merge, or discard this patch.
Ajax/semantic/widgets/datatable/DataTable.php 1 patch
Braces   +35 added lines, -24 removed lines patch added patch discarded remove patch
@@ -58,10 +58,12 @@  discard block
 block discarded – undo
58 58
 			$js->execOn("mouseover", "#".$this->identifier." tr", "$(event.target).closest('tr').find('.visibleover').css('visibility', 'visible');",["preventDefault"=>false,"stopPropagation"=>true]);
59 59
 			$js->execOn("mouseout", "#".$this->identifier." tr", "$(event.target).closest('tr').find('.visibleover').css('visibility', 'hidden');",["preventDefault"=>false,"stopPropagation"=>true]);
60 60
 		}
61
-		if(\is_array($this->_deleteBehavior))
62
-			$this->_generateBehavior("delete",$this->_deleteBehavior, $js);
63
-		if(\is_array($this->_editBehavior))
64
-			$this->_generateBehavior("edit",$this->_editBehavior,$js);
61
+		if(\is_array($this->_deleteBehavior)) {
62
+					$this->_generateBehavior("delete",$this->_deleteBehavior, $js);
63
+		}
64
+		if(\is_array($this->_editBehavior)) {
65
+					$this->_generateBehavior("edit",$this->_editBehavior,$js);
66
+		}
65 67
 		return parent::run($js);
66 68
 	}
67 69
 
@@ -94,8 +96,9 @@  discard block
 block discarded – undo
94 96
 			$table->setRowCount(0, \sizeof($captions));
95 97
 			$this->_generateHeader($table,$captions);
96 98
 
97
-			if(isset($this->_compileParts))
98
-				$table->setCompileParts($this->_compileParts);
99
+			if(isset($this->_compileParts)) {
100
+							$table->setCompileParts($this->_compileParts);
101
+			}
99 102
 
100 103
 			$this->_generateContent($table);
101 104
 
@@ -126,8 +129,9 @@  discard block
 block discarded – undo
126 129
 	}
127 130
 
128 131
 	protected function _applyStyleAttributes($table){
129
-		if(isset($this->_hiddenColumns))
130
-			$this->_hideColumns();
132
+		if(isset($this->_hiddenColumns)) {
133
+					$this->_hideColumns();
134
+		}
131 135
 			if(isset($this->_colWidths)){
132 136
 				foreach ($this->_colWidths as $colIndex=>$width){
133 137
 					$table->setColWidth($colIndex,$width);
@@ -180,8 +184,9 @@  discard block
 block discarded – undo
180 184
 			$field=$ck->getField();
181 185
 			$field->setProperty("value",$dataAjax);
182 186
 			$field->setProperty("name", "selection[]");
183
-			if(isset($checkedClass))
184
-				$field->setClass($checkedClass);
187
+			if(isset($checkedClass)) {
188
+							$field->setClass($checkedClass);
189
+			}
185 190
 			\array_unshift($values, $ck);
186 191
 		}
187 192
 		$result=$table->newRow();
@@ -194,8 +199,9 @@  discard block
 block discarded – undo
194 199
 
195 200
 	protected function _generatePagination($table,$js=NULL){
196 201
 		if(isset($this->_toolbar)){
197
-			if($this->_toolbarPosition==PositionInTable::FOOTER)
198
-				$this->_toolbar->setFloated("left");
202
+			if($this->_toolbarPosition==PositionInTable::FOOTER) {
203
+							$this->_toolbar->setFloated("left");
204
+			}
199 205
 		}
200 206
 		$footer=$table->getFooter();
201 207
 		$footer->mergeCol();
@@ -214,8 +220,9 @@  discard block
 block discarded – undo
214 220
 
215 221
 	protected function _getFieldName($index){
216 222
 		$fieldName=parent::_getFieldName($index);
217
-		if(\is_object($fieldName))
218
-			$fieldName="field-".$index;
223
+		if(\is_object($fieldName)) {
224
+					$fieldName="field-".$index;
225
+		}
219 226
 		return $fieldName."[]";
220 227
 	}
221 228
 
@@ -255,7 +262,7 @@  discard block
 block discarded – undo
255 262
 		$hasPart=$table->hasPart($part);
256 263
 		if($hasPart){
257 264
 			$row=$table->getPart($part)->addRow(\sizeof($captions));
258
-		}else{
265
+		} else{
259 266
 			$row=$table->getPart($part)->getRow(0);
260 267
 		}
261 268
 		$row->mergeCol();
@@ -286,7 +293,7 @@  discard block
 block discarded – undo
286 293
 			$this->_urls["refresh"]=JArray::getValue($urls, "refresh",0);
287 294
 			$this->_urls["edit"]=JArray::getValue($urls, "edit",1);
288 295
 			$this->_urls["delete"]=JArray::getValue($urls, "delete",2);
289
-		}else{
296
+		} else{
290 297
 			$this->_urls=["refresh"=>$urls,"edit"=>$urls,"delete"=>$urls];
291 298
 		}
292 299
 		return $this;
@@ -369,8 +376,9 @@  discard block
 block discarded – undo
369 376
 
370 377
 	protected function getTargetSelector($op) {
371 378
 		$result=$this->_targetSelector;
372
-		if(!isset($result[$op]))
373
-			$result="#".$this->identifier;
379
+		if(!isset($result[$op])) {
380
+					$result="#".$this->identifier;
381
+		}
374 382
 		return $result[$op];
375 383
 	}
376 384
 
@@ -388,8 +396,9 @@  discard block
 block discarded – undo
388 396
 	}
389 397
 
390 398
 	public function getRefreshSelector() {
391
-		if(isset($this->_refreshSelector))
392
-			return $this->_refreshSelector;
399
+		if(isset($this->_refreshSelector)) {
400
+					return $this->_refreshSelector;
401
+		}
393 402
 		return "#".$this->identifier." tbody";
394 403
 	}
395 404
 
@@ -408,8 +417,9 @@  discard block
 block discarded – undo
408 417
 	 */
409 418
 	public function show($modelInstance){
410 419
 		if(\is_array($modelInstance)){
411
-			if(isset($modelInstance[0]) && \is_array(array_values($modelInstance)[0]))
412
-				$modelInstance=\json_decode(\json_encode($modelInstance), FALSE);
420
+			if(isset($modelInstance[0]) && \is_array(array_values($modelInstance)[0])) {
421
+							$modelInstance=\json_decode(\json_encode($modelInstance), FALSE);
422
+			}
413 423
 		}
414 424
 		$this->_modelInstance=$modelInstance;
415 425
 	}
@@ -449,8 +459,9 @@  discard block
 block discarded – undo
449 459
 	}
450 460
 
451 461
 	public function hideColumn($colIndex){
452
-		if(!\is_array($this->_hiddenColumns))
453
-			$this->_hiddenColumns=[];
462
+		if(!\is_array($this->_hiddenColumns)) {
463
+					$this->_hiddenColumns=[];
464
+		}
454 465
 		$this->_hiddenColumns[]=$colIndex;
455 466
 		return $this;
456 467
 	}
Please login to merge, or discard this patch.