Passed
Push — master ( 0bebef...727fe8 )
by Jean-Christophe
02:17
created
Ajax/semantic/html/modules/HtmlDropdown.php 1 patch
Braces   +24 added lines, -19 removed lines patch added patch discarded remove patch
@@ -36,7 +36,7 @@  discard block
 block discarded – undo
36 36
 		if(isset($value)){
37 37
 			if($value instanceof HtmlSemDoubleElement){
38 38
 				$text=$value;
39
-			}else{
39
+			} else{
40 40
 				$text=new HtmlSemDoubleElement("text-".$this->identifier,"div");
41 41
 				$text->setClass("text");
42 42
 				$this->setValue($value);
@@ -109,7 +109,7 @@  discard block
 block discarded – undo
109 109
 		}
110 110
 		if(!$item instanceof HtmlDropdownItem){
111 111
 			$itemO=new HtmlDropdownItem("dd-item-".$this->identifier."-".\sizeof($this->items),$item,$value,$image,$description);
112
-		}elseif($itemO instanceof HtmlDropdownItem){
112
+		} elseif($itemO instanceof HtmlDropdownItem){
113 113
 			$this->addToProperty("class", "vertical");
114 114
 		}
115 115
 		return $itemO;
@@ -123,8 +123,9 @@  discard block
 block discarded – undo
123 123
 	}
124 124
 
125 125
 	public function addInput($name){
126
-		if(!isset($name))
127
-			$name="input-".$this->identifier;
126
+		if(!isset($name)) {
127
+					$name="input-".$this->identifier;
128
+		}
128 129
 		$this->setAction("activate");
129 130
 		$this->input=new HtmlInput($name,"hidden");
130 131
 		$this->input->setIdentifier("input-".$this->identifier);
@@ -183,7 +184,7 @@  discard block
 block discarded – undo
183 184
 			foreach ($items as $k=>$v){
184 185
 				$this->addItem($v)->setData($k);
185 186
 			}
186
-		}else{
187
+		} else{
187 188
 			foreach ($items as $item){
188 189
 				$this->addItem($item);
189 190
 			}
@@ -205,8 +206,7 @@  discard block
 block discarded – undo
205 206
 			$c=$this->items[$i++];
206 207
 			if(isset($c)){
207 208
 				$c->setProperty($property,$value);
208
-			}
209
-			else{
209
+			} else{
210 210
 				return $this;
211 211
 			}
212 212
 		}
@@ -237,7 +237,7 @@  discard block
 block discarded – undo
237 237
 		if($dropdown===false){
238 238
 			$this->_template=include dirname(__FILE__).'/../templates/tplDropdownMenu.php';
239 239
 			$dropdown="menu";
240
-		}else{
240
+		} else{
241 241
 			$dropdown="dropdown";
242 242
 			$this->mClass="menu";
243 243
 		}
@@ -258,22 +258,25 @@  discard block
 block discarded – undo
258 258
 
259 259
 	public function asButton($floating=false){
260 260
 		$this->removeArrow();
261
-		if($floating)
262
-			$this->addToProperty("class", "floating");
261
+		if($floating) {
262
+					$this->addToProperty("class", "floating");
263
+		}
263 264
 		$this->removePropertyValue("class", "selection");
264 265
 		return $this->addToProperty("class", "button");
265 266
 	}
266 267
 
267 268
 	public function asSelect($name=NULL,$multiple=false,$selection=true){
268 269
 		$this->_multiple=$multiple;
269
-		if(isset($name))
270
-			$this->addInput($name);
270
+		if(isset($name)) {
271
+					$this->addInput($name);
272
+		}
271 273
 		if($multiple){
272 274
 			$this->addToProperty("class", "multiple");
273 275
 		}
274 276
 		if ($selection){
275
-			if($this->propertyContains("class", "button")===false)
276
-				$this->addToPropertyCtrl("class", "selection",array("selection"));
277
+			if($this->propertyContains("class", "button")===false) {
278
+							$this->addToPropertyCtrl("class", "selection",array("selection"));
279
+			}
277 280
 		}
278 281
 		return $this;
279 282
 	}
@@ -284,8 +287,9 @@  discard block
 block discarded – undo
284 287
 	}
285 288
 
286 289
 	public function setSelect($name=NULL,$multiple=false){
287
-		if(!isset($name))
288
-			$name="select-".$this->identifier;
290
+		if(!isset($name)) {
291
+					$name="select-".$this->identifier;
292
+		}
289 293
 		$this->input=null;
290 294
 		if($multiple){
291 295
 			$this->setProperty("multiple", true);
@@ -321,12 +325,13 @@  discard block
 block discarded – undo
321 325
 		$value=$this->value;
322 326
 		if(isset($this->input) && isset($value)){
323 327
 			$this->input->setProperty("value", $value);
324
-		}else{
328
+		} else{
325 329
 			$this->setProperty("value", $value);
326 330
 		}
327 331
 			$textElement=$this->getElementById("text-".$this->identifier, $this->content);
328
-			if(isset($textElement) && ($textElement instanceof HtmlDoubleElement) && !$this->_multiple)
329
-				$textElement->setContent($value);
332
+			if(isset($textElement) && ($textElement instanceof HtmlDoubleElement) && !$this->_multiple) {
333
+							$textElement->setContent($value);
334
+			}
330 335
 		return $this;
331 336
 	}
332 337
 
Please login to merge, or discard this patch.
Ajax/semantic/html/elements/HtmlLabel.php 1 patch
Braces   +13 added lines, -10 removed lines patch added patch discarded remove patch
@@ -23,8 +23,9 @@  discard block
 block discarded – undo
23 23
 	public function __construct($identifier, $caption="", $icon=NULL, $tagName="div") {
24 24
 		parent::__construct($identifier, $tagName, "ui label");
25 25
 		$this->content=$caption;
26
-		if (isset($icon))
27
-			$this->addIcon($icon);
26
+		if (isset($icon)) {
27
+					$this->addIcon($icon);
28
+		}
28 29
 	}
29 30
 
30 31
 	/**
@@ -33,10 +34,11 @@  discard block
 block discarded – undo
33 34
 	 * @return HtmlLabel
34 35
 	 */
35 36
 	public function setPointing($value=Direction::NONE) {
36
-		if($value==="left" || $value==="right")
37
-			return $this->addToPropertyCtrl("class", $value." pointing", Direction::getConstantValues("pointing"));
38
-		else
39
-			return $this->addToPropertyCtrl("class", "pointing ".$value, Direction::getConstantValues("pointing",true));
37
+		if($value==="left" || $value==="right") {
38
+					return $this->addToPropertyCtrl("class", $value." pointing", Direction::getConstantValues("pointing"));
39
+		} else {
40
+					return $this->addToPropertyCtrl("class", "pointing ".$value, Direction::getConstantValues("pointing",true));
41
+		}
40 42
 	}
41 43
 
42 44
 	/**
@@ -134,10 +136,11 @@  discard block
 block discarded – undo
134 136
 	}
135 137
 
136 138
 	public function setAttached($side=Side::TOP,$direction=Direction::NONE){
137
-		if($direction!==Direction::NONE)
138
-			return $this->addToPropertyCtrl("class", $side." ".$direction." attached",Side::getConstantValues($direction." attached"));
139
-		else
140
-			return $this->addToPropertyCtrl("class", $side." attached",Side::getConstantValues("attached"));
139
+		if($direction!==Direction::NONE) {
140
+					return $this->addToPropertyCtrl("class", $side." ".$direction." attached",Side::getConstantValues($direction." attached"));
141
+		} else {
142
+					return $this->addToPropertyCtrl("class", $side." attached",Side::getConstantValues("attached"));
143
+		}
141 144
 	}
142 145
 
143 146
 	/**
Please login to merge, or discard this patch.
Ajax/common/html/HtmlDoubleElement.php 1 patch
Braces   +18 added lines, -14 removed lines patch added patch discarded remove patch
@@ -29,15 +29,17 @@  discard block
 block discarded – undo
29 29
 
30 30
 	public function addContent($content,$before=false) {
31 31
 		if (!\is_array($this->content)) {
32
-			if(isset($this->content))
33
-				$this->content=array ($this->content);
34
-			else
35
-				$this->content=array();
32
+			if(isset($this->content)) {
33
+							$this->content=array ($this->content);
34
+			} else {
35
+							$this->content=array();
36
+			}
37
+		}
38
+		if($before) {
39
+					array_unshift($this->content,$content);
40
+		} else {
41
+					$this->content []=$content;
36 42
 		}
37
-		if($before)
38
-			array_unshift($this->content,$content);
39
-		else
40
-			$this->content []=$content;
41 43
 		return $this;
42 44
 	}
43 45
 
@@ -75,9 +77,9 @@  discard block
 block discarded – undo
75 77
 		$instances=[];
76 78
 		if($content instanceof $class){
77 79
 			$instances[]=$content;
78
-		}elseif($content instanceof HtmlDoubleElement){
80
+		} elseif($content instanceof HtmlDoubleElement){
79 81
 			$instances=\array_merge($instances,$content->getContentInstances($class));
80
-		}elseif (\is_array($content)){
82
+		} elseif (\is_array($content)){
81 83
 			foreach ($content as $element){
82 84
 				$instances=\array_merge($instances,$this->_getContentInstances($class, $element));
83 85
 			}
@@ -90,10 +92,12 @@  discard block
 block discarded – undo
90 92
 	 * @return HtmlDoubleElement
91 93
 	 */
92 94
 	public function asLink($href=NULL,$target=NULL) {
93
-		if (isset($href))
94
-			$this->setProperty("href", $href);
95
-		if(isset($target))
96
-			$this->setProperty("target", $target);
95
+		if (isset($href)) {
96
+					$this->setProperty("href", $href);
97
+		}
98
+		if(isset($target)) {
99
+					$this->setProperty("target", $target);
100
+		}
97 101
 		return $this->setTagName("a");
98 102
 	}
99 103
 }
Please login to merge, or discard this patch.
Ajax/semantic/html/collections/form/traits/FieldsTrait.php 1 patch
Braces   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -28,11 +28,12 @@  discard block
 block discarded – undo
28 28
 		if(\is_array($value)){
29 29
 			$itemO=new HtmlFormInput(JArray::getDefaultValue($value, "id",""),JArray::getDefaultValue($value, "label",null),JArray::getDefaultValue($value, "type", "text"),JArray::getDefaultValue($value, "value",""),JArray::getDefaultValue($value, "placeholder",JArray::getDefaultValue($value, "label",null)));
30 30
 			return $itemO;
31
-		}elseif(\is_object($value)){
31
+		} elseif(\is_object($value)){
32 32
 			$itemO=new HtmlFormField("field-".$this->identifier, $value);
33 33
 			return $itemO;
34
-		}else
35
-			return new HtmlFormInput($value);
34
+		} else {
35
+					return new HtmlFormInput($value);
36
+		}
36 37
 	}
37 38
 
38 39
 	protected function createCondition($value){
@@ -66,8 +67,7 @@  discard block
 block discarded – undo
66 67
 			if(isset($c)){
67 68
 				$df=$c->getDataField();
68 69
 				$df->setProperty($property,$value);
69
-			}
70
-			else{
70
+			} else{
71 71
 				return $this;
72 72
 			}
73 73
 		}
Please login to merge, or discard this patch.
Ajax/semantic/html/content/HtmlStepItem.php 1 patch
Braces   +6 added lines, -5 removed lines patch added patch discarded remove patch
@@ -25,16 +25,17 @@
 block discarded – undo
25 25
 			if(isset($title)){
26 26
 				$this->setTitle($title,$desc);
27 27
 			}
28
-		}else{
28
+		} else{
29 29
 			$this->setContent($content);
30 30
 		}
31 31
 	}
32 32
 
33 33
 	public function setActive($value=true){
34
-		if($value)
35
-			$this->setStatus(StepStatus::ACTIVE);
36
-		else
37
-			$this->setStatus(StepStatus::NONE);
34
+		if($value) {
35
+					$this->setStatus(StepStatus::ACTIVE);
36
+		} else {
37
+					$this->setStatus(StepStatus::NONE);
38
+		}
38 39
 		return $this;
39 40
 	}
40 41
 
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
 
@@ -77,7 +78,7 @@  discard block
 block discarded – undo
77 78
 			$this->setTagName("div");
78 79
 			$this->removeProperty("href");
79 80
 			$this->addToPropertyCtrl("class", "active", array("active"));
80
-		}else{
81
+		} else{
81 82
 			$this->removePropertyValue("class", "active");
82 83
 		}
83 84
 		return $this;
@@ -85,8 +86,9 @@  discard block
 block discarded – undo
85 86
 
86 87
 	public function asLink($href=NULL,$part=NULL){
87 88
 		$this->setTagName("a");
88
-		if(isset($href))
89
-			$this->setProperty("href", $href);
89
+		if(isset($href)) {
90
+					$this->setProperty("href", $href);
91
+		}
90 92
 		return $this;
91 93
 	}
92 94
 
@@ -97,8 +99,9 @@  discard block
 block discarded – undo
97 99
 	 * @see \Ajax\semantic\html\base\HtmlSemDoubleElement::compile()
98 100
 	 */
99 101
 	public function compile(JsUtils $js=NULL, &$view=NULL) {
100
-		if(\is_array($this->content) && JArray::isAssociative($this->content))
101
-			$this->content=JArray::sortAssociative($this->content, [ "right-content","icon","image","content" ]);
102
+		if(\is_array($this->content) && JArray::isAssociative($this->content)) {
103
+					$this->content=JArray::sortAssociative($this->content, [ "right-content","icon","image","content" ]);
104
+		}
102 105
 		return parent::compile($js, $view);
103 106
 	}
104 107
 }
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.