Passed
Push — master ( ccea9a...7a8d95 )
by Jean-Christophe
01:50
created
Ajax/common/html/HtmlCollection.php 1 patch
Braces   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
24 24
 			foreach ($items as $k=>$v){
25 25
 				$this->addItem([$k,$v]);
26 26
 			}
27
-		}else{
27
+		} else{
28 28
 			foreach ($items as $item){
29 29
 				$this->addItem($item);
30 30
 			}
@@ -80,9 +80,9 @@  discard block
 block discarded – undo
80 80
 	 * @return \Ajax\common\html\HtmlDoubleElement
81 81
 	 */
82 82
 	public function getItem($index) {
83
-		if (is_int($index))
84
-			return $this->content[$index];
85
-		else {
83
+		if (is_int($index)) {
84
+					return $this->content[$index];
85
+		} else {
86 86
 			$elm=$this->getElementById($index, $this->content);
87 87
 			return $elm;
88 88
 		}
@@ -144,10 +144,11 @@  discard block
 block discarded – undo
144 144
 		$i=0;
145 145
 		foreach ($properties as $k=>$v){
146 146
 			$c=$this->content[$i++];
147
-			if(isset($c))
148
-				$c->setProperty($k,$v);
149
-			else
150
-				return $this;
147
+			if(isset($c)) {
148
+							$c->setProperty($k,$v);
149
+			} else {
150
+							return $this;
151
+			}
151 152
 		}
152 153
 		return $this;
153 154
 	}
@@ -167,8 +168,7 @@  discard block
 block discarded – undo
167 168
 			$c=$this->content[$i++];
168 169
 			if(isset($c)){
169 170
 				$c->setProperty($property,$value);
170
-			}
171
-			else{
171
+			} else{
172 172
 				return $this;
173 173
 			}
174 174
 		}
@@ -190,8 +190,7 @@  discard block
 block discarded – undo
190 190
 			$c=$this->content[$i++];
191 191
 			if(isset($c)){
192 192
 				$c->addToProperty($property,$value);
193
-			}
194
-			else{
193
+			} else{
195 194
 				return $this;
196 195
 			}
197 196
 		}
@@ -220,8 +219,9 @@  discard block
 block discarded – undo
220 219
 		foreach ( $this->content as $index=>$item ) {
221 220
 			if($item instanceof HtmlDoubleElement){
222 221
 				$href="";
223
-				if(isset($hrefs[$index]))
224
-					$href=$hrefs[$index];
222
+				if(isset($hrefs[$index])) {
223
+									$href=$hrefs[$index];
224
+				}
225 225
 				$item->asLink($href,$target);
226 226
 			}
227 227
 		}
Please login to merge, or discard this patch.
Ajax/semantic/html/modules/checkbox/AbstractCheckbox.php 1 patch
Braces   +13 added lines, -9 removed lines patch added patch discarded remove patch
@@ -15,14 +15,15 @@  discard block
 block discarded – undo
15 15
 		$field=new \Ajax\common\html\html5\HtmlInput($identifier, $inputType, $value);
16 16
 		$field->setProperty("name", $name);
17 17
 		$this->setField($field);
18
-		if (isset($label))
19
-			$this->setLabel($label,$value);
18
+		if (isset($label)) {
19
+					$this->setLabel($label,$value);
20
+		}
20 21
 	}
21 22
 
22 23
 	public function setChecked($value=true){
23 24
 		if($value===true){
24 25
 			$this->getField()->setProperty("checked", "checked");
25
-		}else{
26
+		} else{
26 27
 			$this->getField()->removeProperty("checked");
27 28
 		}
28 29
 		return $this;
@@ -38,8 +39,9 @@  discard block
 block discarded – undo
38 39
 			$labelO=new HtmlSemDoubleElement("", "label", "");
39 40
 			$labelO->setContent($label);
40 41
 			$labelO->setProperty("for", $this->getField()->getIdentifier());
41
-			if(isset($value))
42
-				$labelO->setProperty("data-value", $value);
42
+			if(isset($value)) {
43
+							$labelO->setProperty("data-value", $value);
44
+			}
43 45
 		}
44 46
 		$this->content["label"]=$labelO;
45 47
 	}
@@ -53,8 +55,9 @@  discard block
 block discarded – undo
53 55
 	 * @return mixed
54 56
 	 */
55 57
 	public function getLabel() {
56
-		if (\array_key_exists("label", $this->content))
57
-			return $this->content["label"];
58
+		if (\array_key_exists("label", $this->content)) {
59
+					return $this->content["label"];
60
+		}
58 61
 	}
59 62
 
60 63
 	/**
@@ -129,8 +132,9 @@  discard block
 block discarded – undo
129 132
 	}
130 133
 
131 134
 	public function run(JsUtils $js) {
132
-		if(!isset($this->_bsComponent))
133
-			$this->_bsComponent=$js->semantic()->checkbox("#" . $this->identifier, $this->_params);
135
+		if(!isset($this->_bsComponent)) {
136
+					$this->_bsComponent=$js->semantic()->checkbox("#" . $this->identifier, $this->_params);
137
+		}
134 138
 		return parent::run($js);
135 139
 	}
136 140
 }
Please login to merge, or discard this patch.
Ajax/ui/components/DialogButton.php 1 patch
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -14,8 +14,9 @@
 block discarded – undo
14 14
 class DialogButton extends BaseComponent {
15 15
 
16 16
 	private function addFunction($jsCode) {
17
-		if (!JString::startsWith($jsCode, "function"))
18
-			$jsCode="%function(){" . $jsCode . "}%";
17
+		if (!JString::startsWith($jsCode, "function")) {
18
+					$jsCode="%function(){" . $jsCode . "}%";
19
+		}
19 20
 		return $jsCode;
20 21
 	}
21 22
 
Please login to merge, or discard this patch.
Ajax/bootstrap/html/HtmlDropdown.php 1 patch
Braces   +12 added lines, -9 removed lines patch added patch discarded remove patch
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
 	public function addItem($caption, $href="#") {
91 91
 		if($caption instanceof HtmlDropdownItem){
92 92
 			$item=$caption;
93
-		}else{
93
+		} else{
94 94
 			$iid=$this->getItemsCount()+1;
95 95
 			$item=new HtmlDropdownItem($this->identifier."-dropdown-item-".$iid);
96 96
 			$item->setCaption($caption)->setHref($href);
@@ -130,10 +130,11 @@  discard block
 block discarded – undo
130 130
 	 * @see BaseHtml::fromArray()
131 131
 	 */
132 132
 	public function fromArray($array) {
133
-		if (array_keys($array)!==range(0, count($array)-1))
134
-			return parent::fromArray($array);
135
-		else
136
-			return $this->addItems($array);
133
+		if (array_keys($array)!==range(0, count($array)-1)) {
134
+					return parent::fromArray($array);
135
+		} else {
136
+					return $this->addItems($array);
137
+		}
137 138
 	}
138 139
 
139 140
 	public function setItems($items) {
@@ -186,8 +187,9 @@  discard block
 block discarded – undo
186 187
 	 * @see BaseHtml::setTagName()
187 188
 	 */
188 189
 	public function setTagName($tagName) {
189
-		if ($tagName=="button")
190
-			$this->class="btn dropdown-toggle";
190
+		if ($tagName=="button") {
191
+					$this->class="btn dropdown-toggle";
192
+		}
191 193
 		return parent::setTagName($tagName);
192 194
 	}
193 195
 
@@ -205,8 +207,9 @@  discard block
 block discarded – undo
205 207
 	}
206 208
 
207 209
 	public function setAlignment($alignment) {
208
-		if (is_int($alignment))
209
-			$alignment="dropdown-menu-".CssRef::alignment()[$alignment];
210
+		if (is_int($alignment)) {
211
+					$alignment="dropdown-menu-".CssRef::alignment()[$alignment];
212
+		}
210 213
 		return $this->addToMemberCtrl($this->class, $alignment, CssRef::alignment());
211 214
 	}
212 215
 
Please login to merge, or discard this patch.
Ajax/bootstrap/html/HtmlPagination.php 1 patch
Braces   +12 added lines, -9 removed lines patch added patch discarded remove patch
@@ -52,10 +52,11 @@  discard block
 block discarded – undo
52 52
 		$this->from=$from;
53 53
 		$this->to=$to;
54 54
 		$this->urlMask="%page%";
55
-		if(!isset($countVisible))
56
-			$this->countVisible=$to-$from+1;
57
-		else
58
-			$this->countVisible=$countVisible;
55
+		if(!isset($countVisible)) {
56
+					$this->countVisible=$to-$from+1;
57
+		} else {
58
+					$this->countVisible=$countVisible;
59
+		}
59 60
 		$this->createContent();
60 61
 	}
61 62
 
@@ -74,7 +75,7 @@  discard block
 block discarded – undo
74 75
 			$href=new HtmlLink("a-".$this->identifier."-".$count,$url,$content);
75 76
 			$href->setProperty($this->attr, $url);
76 77
 			$elem->setContent($href);
77
-		}else{
78
+		} else{
78 79
 			$elem->setContent($content);
79 80
 		}
80 81
 		$this->content[]=$elem;
@@ -107,8 +108,9 @@  discard block
 block discarded – undo
107 108
 	public function _addEvent($event, $jsCode) {
108 109
 		foreach ($this->content as $li){
109 110
 			$content=$li->getContent();
110
-			if($content instanceof BaseHtml)
111
-				$content->_addEvent($event,$jsCode);
111
+			if($content instanceof BaseHtml) {
112
+							$content->_addEvent($event,$jsCode);
113
+			}
112 114
 		}
113 115
 	}
114 116
 	/**
@@ -146,8 +148,9 @@  discard block
 block discarded – undo
146 148
 		if (is_int($size)) {
147 149
 			return $this->addToPropertyUnique("class", CssRef::sizes("pagination")[$size], CssRef::sizes("pagination"));
148 150
 		}
149
-		if(!JString::startsWith($size, "pagination-") && $size!=="")
150
-			$size="pagination-".$size;
151
+		if(!JString::startsWith($size, "pagination-") && $size!=="") {
152
+					$size="pagination-".$size;
153
+		}
151 154
 		return $this->addToPropertyCtrl("class", $size, CssRef::sizes("pagination"));
152 155
 	}
153 156
 
Please login to merge, or discard this patch.
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.