Passed
Push — master ( fa067a...81b8d1 )
by Jean-Christophe
02:55
created
Ajax/semantic/html/elements/HtmlButton.php 2 patches
Spacing   +40 added lines, -40 removed lines patch added patch discarded remove patch
@@ -30,9 +30,9 @@  discard block
 block discarded – undo
30 30
 	 * @param string $onClick
31 31
 	 *        	JS Code for click event
32 32
 	 */
33
-	public function __construct($identifier, $value = null, $cssStyle = null, $onClick = null) {
33
+	public function __construct($identifier, $value=null, $cssStyle=null, $onClick=null) {
34 34
 		parent::__construct($identifier, "button", "ui button");
35
-		$this->content = $value;
35
+		$this->content=$value;
36 36
 		if (isset($cssStyle)) {
37 37
 			$this->setStyle($cssStyle);
38 38
 		}
@@ -51,12 +51,12 @@  discard block
 block discarded – undo
51 51
 		if (is_array($this->content)) {
52 52
 			foreach ($this->content as $i => $content) {
53 53
 				if (is_string($content)) {
54
-					$this->content[$i] = $value;
54
+					$this->content[$i]=$value;
55 55
 					return $this;
56 56
 				}
57 57
 			}
58 58
 		}
59
-		$this->content = $value;
59
+		$this->content=$value;
60 60
 		return $this;
61 61
 	}
62 62
 
@@ -70,8 +70,8 @@  discard block
 block discarded – undo
70 70
 		return $this->addToProperty("class", $cssStyle);
71 71
 	}
72 72
 
73
-	public function setFocusable($value = true) {
74
-		if ($value === true)
73
+	public function setFocusable($value=true) {
74
+		if ($value===true)
75 75
 			$this->setProperty("tabindex", "0");
76 76
 		else {
77 77
 			$this->removeProperty("tabindex");
@@ -79,16 +79,16 @@  discard block
 block discarded – undo
79 79
 		return $this;
80 80
 	}
81 81
 
82
-	public function setAnimated($content, $animation = "") {
82
+	public function setAnimated($content, $animation="") {
83 83
 		$this->setTagName("div");
84
-		$this->addToProperty("class", "animated " . $animation);
85
-		$visible = new HtmlSemDoubleElement("visible-" . $this->identifier, "div");
84
+		$this->addToProperty("class", "animated ".$animation);
85
+		$visible=new HtmlSemDoubleElement("visible-".$this->identifier, "div");
86 86
 		$visible->setClass("visible content");
87 87
 		$visible->setContent($this->content);
88
-		$hidden = new HtmlSemDoubleElement("hidden-" . $this->identifier, "div");
88
+		$hidden=new HtmlSemDoubleElement("hidden-".$this->identifier, "div");
89 89
 		$hidden->setClass("hidden content");
90 90
 		$hidden->setContent($content);
91
-		$this->content = array(
91
+		$this->content=array(
92 92
 			$visible,
93 93
 			$hidden
94 94
 		);
@@ -101,12 +101,12 @@  discard block
 block discarded – undo
101 101
 	 * @return HtmlButton
102 102
 	 */
103 103
 	public function asIcon($icon) {
104
-		$iconO = $icon;
104
+		$iconO=$icon;
105 105
 		if (\is_string($icon)) {
106
-			$iconO = new HtmlIcon("icon-" . $this->identifier, $icon);
106
+			$iconO=new HtmlIcon("icon-".$this->identifier, $icon);
107 107
 		}
108 108
 		$this->addToProperty("class", "icon");
109
-		$this->content = $iconO;
109
+		$this->content=$iconO;
110 110
 		return $this;
111 111
 	}
112 112
 
@@ -123,19 +123,19 @@  discard block
 block discarded – undo
123 123
 	 * @param string $icon
124 124
 	 * @return HtmlLabel
125 125
 	 */
126
-	public function addLabel($label, $before = false, $icon = NULL) {
127
-		$this->tagName = "div";
128
-		$prefix = "";
126
+	public function addLabel($label, $before=false, $icon=NULL) {
127
+		$this->tagName="div";
128
+		$prefix="";
129 129
 		if ($before)
130
-			$prefix = "left ";
131
-		$this->addToProperty("class", $prefix . "labeled");
132
-		$isIcon = (isset($this->content[0]) && $this->content[0] instanceof HtmlIcon);
133
-		$this->content = new HtmlButton("button-" . $this->identifier, $this->content);
130
+			$prefix="left ";
131
+		$this->addToProperty("class", $prefix."labeled");
132
+		$isIcon=(isset($this->content[0]) && $this->content[0] instanceof HtmlIcon);
133
+		$this->content=new HtmlButton("button-".$this->identifier, $this->content);
134 134
 		if ($isIcon) {
135 135
 			$this->content->addClass("icon");
136 136
 		}
137 137
 		$this->content->setTagName("div");
138
-		$label = new HtmlLabel("label-" . $this->identifier, $label, $icon, "a");
138
+		$label=new HtmlLabel("label-".$this->identifier, $label, $icon, "a");
139 139
 		$label->setBasic();
140 140
 		$this->addContent($label, $before);
141 141
 		return $label;
@@ -146,7 +146,7 @@  discard block
 block discarded – undo
146 146
 	 * @see \Ajax\common\html\BaseHtml::fromArray()
147 147
 	 */
148 148
 	public function fromArray($array) {
149
-		$array = parent::fromArray($array);
149
+		$array=parent::fromArray($array);
150 150
 		foreach ($array as $key => $value) {
151 151
 			$this->setProperty($key, $value);
152 152
 		}
@@ -187,9 +187,9 @@  discard block
 block discarded – undo
187 187
 	 *
188 188
 	 * @return HtmlButton
189 189
 	 */
190
-	public function setToggle($active = "") {
191
-		$this->onCreate("$('#" . $this->identifier . "').state();");
192
-		return $this->addToProperty("class", "toggle " . $active);
190
+	public function setToggle($active="") {
191
+		$this->onCreate("$('#".$this->identifier."').state();");
192
+		return $this->addToProperty("class", "toggle ".$active);
193 193
 	}
194 194
 
195 195
 	/**
@@ -225,10 +225,10 @@  discard block
 block discarded – undo
225 225
 	 * @param string $value
226 226
 	 * @return HtmlButton
227 227
 	 */
228
-	public static function social($identifier, $social, $value = NULL) {
229
-		if ($value === NULL)
230
-			$value = \ucfirst($social);
231
-		$return = new HtmlButton($identifier, $value);
228
+	public static function social($identifier, $social, $value=NULL) {
229
+		if ($value===NULL)
230
+			$value=\ucfirst($social);
231
+		$return=new HtmlButton($identifier, $value);
232 232
 		$return->addIcon($social);
233 233
 		return $return->addToPropertyCtrl("class", $social, Social::getConstants());
234 234
 	}
@@ -242,8 +242,8 @@  discard block
 block discarded – undo
242 242
 	 * @param boolean $before
243 243
 	 * @return \Ajax\semantic\html\elements\HtmlButton
244 244
 	 */
245
-	public static function labeled($identifier, $value, $icon, $before = true) {
246
-		$result = new HtmlButton($identifier, $value);
245
+	public static function labeled($identifier, $value, $icon, $before=true) {
246
+		$result=new HtmlButton($identifier, $value);
247 247
 		$result->addIcon($icon, $before, true);
248 248
 		return $result;
249 249
 	}
@@ -256,7 +256,7 @@  discard block
 block discarded – undo
256 256
 	 * @return HtmlButton
257 257
 	 */
258 258
 	public static function icon($identifier, $icon) {
259
-		$result = new HtmlButton($identifier);
259
+		$result=new HtmlButton($identifier);
260 260
 		$result->asIcon($icon);
261 261
 		return $result;
262 262
 	}
@@ -266,7 +266,7 @@  discard block
 block discarded – undo
266 266
 	 * {@inheritdoc}
267 267
 	 * @see HtmlSemDoubleElement::asLink()
268 268
 	 */
269
-	public function asLink($href = NULL, $target = NULL) {
269
+	public function asLink($href=NULL, $target=NULL) {
270 270
 		parent::asLink($href, $target);
271 271
 		return $this;
272 272
 	}
@@ -281,20 +281,20 @@  discard block
 block discarded – undo
281 281
 	 * @param string $icon
282 282
 	 * @return HtmlButtonGroups
283 283
 	 */
284
-	public static function dropdown($identifier, $value, $items = [], $asCombo = false, $icon = null) {
285
-		$result = new HtmlButtonGroups($identifier, [
284
+	public static function dropdown($identifier, $value, $items=[], $asCombo=false, $icon=null) {
285
+		$result=new HtmlButtonGroups($identifier, [
286 286
 			$value
287 287
 		]);
288
-		$dd = $result->addDropdown($items, $asCombo);
288
+		$dd=$result->addDropdown($items, $asCombo);
289 289
 		if (isset($icon) && $dd instanceof HtmlDropdown)
290 290
 			$dd->setIcon($icon);
291 291
 		return $result;
292 292
 	}
293 293
 
294
-	public function addPopupConfirmation($message, $buttons = ["Okay","Cancel"]) {
295
-		$elm = new HtmlSemDoubleElement('popup-confirm-' . $this->_identifier);
294
+	public function addPopupConfirmation($message, $buttons=["Okay", "Cancel"]) {
295
+		$elm=new HtmlSemDoubleElement('popup-confirm-'.$this->_identifier);
296 296
 		$elm->setContent([
297
-			'message' => new HtmlSemDoubleElement('popup-confirm-message-' . $this->_identifier, 'p', '', $message)
297
+			'message' => new HtmlSemDoubleElement('popup-confirm-message-'.$this->_identifier, 'p', '', $message)
298 298
 		]);
299 299
 		$this->addPopupHtml($elm, null, [
300 300
 			'on' => 'click'
Please login to merge, or discard this patch.
Braces   +18 added lines, -13 removed lines patch added patch discarded remove patch
@@ -71,9 +71,9 @@  discard block
 block discarded – undo
71 71
 	}
72 72
 
73 73
 	public function setFocusable($value = true) {
74
-		if ($value === true)
75
-			$this->setProperty("tabindex", "0");
76
-		else {
74
+		if ($value === true) {
75
+					$this->setProperty("tabindex", "0");
76
+		} else {
77 77
 			$this->removeProperty("tabindex");
78 78
 		}
79 79
 		return $this;
@@ -126,8 +126,9 @@  discard block
 block discarded – undo
126 126
 	public function addLabel($label, $before = false, $icon = NULL) {
127 127
 		$this->tagName = "div";
128 128
 		$prefix = "";
129
-		if ($before)
130
-			$prefix = "left ";
129
+		if ($before) {
130
+					$prefix = "left ";
131
+		}
131 132
 		$this->addToProperty("class", $prefix . "labeled");
132 133
 		$isIcon = (isset($this->content[0]) && $this->content[0] instanceof HtmlIcon);
133 134
 		$this->content = new HtmlButton("button-" . $this->identifier, $this->content);
@@ -165,11 +166,13 @@  discard block
 block discarded – undo
165 166
 	public function setColor($color) {
166 167
 		if (\is_array($this->content)) {
167 168
 			foreach ($this->content as $content) {
168
-				if ($content instanceof HtmlButton)
169
-					$content->setColor($color);
169
+				if ($content instanceof HtmlButton) {
170
+									$content->setColor($color);
171
+				}
170 172
 			}
171
-		} else
172
-			parent::setColor($color);
173
+		} else {
174
+					parent::setColor($color);
175
+		}
173 176
 		return $this;
174 177
 	}
175 178
 
@@ -226,8 +229,9 @@  discard block
 block discarded – undo
226 229
 	 * @return HtmlButton
227 230
 	 */
228 231
 	public static function social($identifier, $social, $value = NULL) {
229
-		if ($value === NULL)
230
-			$value = \ucfirst($social);
232
+		if ($value === NULL) {
233
+					$value = \ucfirst($social);
234
+		}
231 235
 		$return = new HtmlButton($identifier, $value);
232 236
 		$return->addIcon($social);
233 237
 		return $return->addToPropertyCtrl("class", $social, Social::getConstants());
@@ -286,8 +290,9 @@  discard block
 block discarded – undo
286 290
 			$value
287 291
 		]);
288 292
 		$dd = $result->addDropdown($items, $asCombo);
289
-		if (isset($icon) && $dd instanceof HtmlDropdown)
290
-			$dd->setIcon($icon);
293
+		if (isset($icon) && $dd instanceof HtmlDropdown) {
294
+					$dd->setIcon($icon);
295
+		}
291 296
 		return $result;
292 297
 	}
293 298
 
Please login to merge, or discard this patch.
Ajax/semantic/html/collections/menus/HtmlMenu.php 2 patches
Spacing   +31 added lines, -31 removed lines patch added patch discarded remove patch
@@ -41,12 +41,12 @@  discard block
 block discarded – undo
41 41
 	 * @return HtmlMenu
42 42
 	 */
43 43
 	public function setType($type="") {
44
-		return $this->addToPropertyCtrl("class", $type, array ("","item","text" ));
44
+		return $this->addToPropertyCtrl("class", $type, array("", "item", "text"));
45 45
 	}
46 46
 
47 47
 	public function setActiveItem($index) {
48 48
 		$item=$this->getItem($index);
49
-		if ($item !== null) {
49
+		if ($item!==null) {
50 50
 			$item->addToProperty("class", "active");
51 51
 		}
52 52
 		return $this;
@@ -54,7 +54,7 @@  discard block
 block discarded – undo
54 54
 
55 55
 	private function getItemToInsert($item) {
56 56
 		if ($item instanceof HtmlInput || $item instanceof HtmlImg || $item instanceof HtmlIcon || $item instanceof HtmlButtonGroups || $item instanceof HtmlButton || $item instanceof HtmlLabel) {
57
-			$itemO=new HtmlMenuItem("item-" . $this->identifier . "-" . \sizeof($this->content) , $item);
57
+			$itemO=new HtmlMenuItem("item-".$this->identifier."-".\sizeof($this->content), $item);
58 58
 			$itemO->addClass("no-active");
59 59
 			$item=$itemO;
60 60
 		}
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
 
64 64
 	private function afterInsert($item) {
65 65
 		if (!$item instanceof HtmlMenu && $item->propertyContains("class", "header")===false)
66
-			$item->addToPropertyCtrl("class", "item", array ("item" ));
66
+			$item->addToPropertyCtrl("class", "item", array("item"));
67 67
 		else {
68 68
 			$this->setSecondary();
69 69
 		}
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
 	public function addItem($item) {
80 80
 		$number=$item;
81 81
 		$item=parent::addItem($this->getItemToInsert($item));
82
-		if(\is_int($number))
82
+		if (\is_int($number))
83 83
 			$item->setProperty("data-page", $number);
84 84
 		return $this->afterInsert($item);
85 85
 	}
@@ -97,15 +97,15 @@  discard block
 block discarded – undo
97 97
 
98 98
 	public function generateMenuAsItem($menu, $header=null) {
99 99
 		$count=$this->count();
100
-		$item=new HtmlSemDoubleElement("item-" . $this->identifier . "-" . $count, "div");
100
+		$item=new HtmlSemDoubleElement("item-".$this->identifier."-".$count, "div");
101 101
 		if (isset($header)) {
102
-			$headerItem=new HtmlSemDoubleElement("item-header-" . $this->identifier . "-" . $count, "div", "header");
102
+			$headerItem=new HtmlSemDoubleElement("item-header-".$this->identifier."-".$count, "div", "header");
103 103
 			$headerItem->setContent($header);
104 104
 			$item->addContent($headerItem);
105 105
 			$this->_itemHeader=$headerItem;
106 106
 		}
107
-		if(\is_array($menu)){
108
-			$menu=new HtmlMenu("menu-" . $this->identifier . "-" . $count,$menu);
107
+		if (\is_array($menu)) {
108
+			$menu=new HtmlMenu("menu-".$this->identifier."-".$count, $menu);
109 109
 		}
110 110
 		$menu->setClass("menu");
111 111
 		$item->addContent($menu);
@@ -117,11 +117,11 @@  discard block
 block discarded – undo
117 117
 	 * @param String|HtmlDoubleElement $caption
118 118
 	 * @return \Ajax\common\html\HtmlDoubleElement
119 119
 	 */
120
-	public function addHeader($caption){
121
-		if(!($caption instanceof HtmlDoubleElement)){
122
-			$header=new HtmlDoubleElement('','div');
120
+	public function addHeader($caption) {
121
+		if (!($caption instanceof HtmlDoubleElement)) {
122
+			$header=new HtmlDoubleElement('', 'div');
123 123
 			$header->setContent($caption);
124
-		}else{
124
+		} else {
125 125
 			$header=$caption;
126 126
 		}
127 127
 		$header->addClass('item header');
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
 		$value=new HtmlSemDoubleElement($identifier, "a", "browse item", $value);
138 138
 		$value->addContent(new HtmlIcon("", "dropdown"));
139 139
 		$value=$this->addItem($value);
140
-		$popup=new HtmlPopup($value, "popup-" . $this->identifier . "-" . $this->count(), $content);
140
+		$popup=new HtmlPopup($value, "popup-".$this->identifier."-".$this->count(), $content);
141 141
 		$popup->setFlowing()->setPosition("bottom left")->setOn("click");
142 142
 		$this->wrap("", $popup);
143 143
 		return $popup;
@@ -146,7 +146,7 @@  discard block
 block discarded – undo
146 146
 	public function addDropdownAsItem($value, $items=NULL) {
147 147
 		$dd=$value;
148 148
 		if (\is_string($value)) {
149
-			$dd=new HtmlDropdown("dropdown-" . $this->identifier . "-" . $this->count(), $value, $items);
149
+			$dd=new HtmlDropdown("dropdown-".$this->identifier."-".$this->count(), $value, $items);
150 150
 		}
151 151
 		$this->addItem($dd);
152 152
 		return $dd;
@@ -159,14 +159,14 @@  discard block
 block discarded – undo
159 159
 	 * @see HtmlCollection::createItem()
160 160
 	 */
161 161
 	protected function createItem($value) {
162
-		$itemO=new HtmlMenuItem($this->identifier."-item-" . \sizeof($this->content),"");
162
+		$itemO=new HtmlMenuItem($this->identifier."-item-".\sizeof($this->content), "");
163 163
 		$itemO->setTagName("a");
164 164
 		$itemO->setContent($value);
165 165
 		return $itemO;
166 166
 	}
167 167
 
168 168
 	public function setSecondary($value=true) {
169
-		if($value)
169
+		if ($value)
170 170
 			$this->addToProperty("class", "secondary");
171 171
 		else
172 172
 			$this->removePropertyValue("class", "secondary");
@@ -174,28 +174,28 @@  discard block
 block discarded – undo
174 174
 	}
175 175
 
176 176
 	public function setVertical() {
177
-		return $this->addToPropertyCtrl("class", "vertical", array ("vertical" ));
177
+		return $this->addToPropertyCtrl("class", "vertical", array("vertical"));
178 178
 	}
179 179
 
180 180
 	public function setPosition($value="right") {
181
-		return $this->addToPropertyCtrl("class", $value, array ("right","left" ));
181
+		return $this->addToPropertyCtrl("class", $value, array("right", "left"));
182 182
 	}
183 183
 
184 184
 	public function setPointing($value=Direction::NONE) {
185
-		return $this->addToPropertyCtrl("class", $value . " pointing", Direction::getConstantValues("pointing"));
185
+		return $this->addToPropertyCtrl("class", $value." pointing", Direction::getConstantValues("pointing"));
186 186
 	}
187 187
 
188 188
 	public function asTab($vertical=false) {
189
-		$this->apply(function (HtmlDoubleElement &$item) {
189
+		$this->apply(function(HtmlDoubleElement &$item) {
190 190
 			$item->setTagName("a");
191 191
 		});
192
-		if ($vertical === true)
192
+		if ($vertical===true)
193 193
 			$this->setVertical();
194 194
 		return $this->addToProperty("class", "tabular");
195 195
 	}
196 196
 
197 197
 	public function asPagination() {
198
-		$this->apply(function (HtmlDoubleElement &$item) {
198
+		$this->apply(function(HtmlDoubleElement &$item) {
199 199
 			$item->setTagName("a");
200 200
 		});
201 201
 		return $this->addToProperty("class", "pagination");
@@ -232,10 +232,10 @@  discard block
 block discarded – undo
232 232
 	 */
233 233
 	public function setWidth($width) {
234 234
 		if (\is_int($width)) {
235
-			$width=Wide::getConstants()["W" . $width];
235
+			$width=Wide::getConstants()["W".$width];
236 236
 		}
237 237
 		$this->addToPropertyCtrl("class", $width, Wide::getConstants());
238
-		return $this->addToPropertyCtrl("class", "item", array ("item" ));
238
+		return $this->addToPropertyCtrl("class", "item", array("item"));
239 239
 	}
240 240
 
241 241
 	public function addImage($identifier, $src="", $alt="") {
@@ -250,14 +250,14 @@  discard block
 block discarded – undo
250 250
 		return $this->_itemHeader;
251 251
 	}
252 252
 
253
-	public function setHasContainer(){
254
-		return $this->wrapContent("<div class='ui container'>","</div>");
253
+	public function setHasContainer() {
254
+		return $this->wrapContent("<div class='ui container'>", "</div>");
255 255
 	}
256 256
 
257
-	public function run(JsUtils $js){
258
-		if($this->identifier!=="" && !isset($this->_bsComponent))
259
-			$this->onClick('if(!$(this).hasClass("dropdown")&&!$(this).hasClass("no-active")){$(this).addClass("active").siblings().removeClass("active");}',false,false);
260
-		$result= parent::run($js);
257
+	public function run(JsUtils $js) {
258
+		if ($this->identifier!=="" && !isset($this->_bsComponent))
259
+			$this->onClick('if(!$(this).hasClass("dropdown")&&!$(this).hasClass("no-active")){$(this).addClass("active").siblings().removeClass("active");}', false, false);
260
+		$result=parent::run($js);
261 261
 		return $result->setItemSelector(">.item:not(.header)");
262 262
 	}
263 263
 }
Please login to merge, or discard this patch.
Braces   +23 added lines, -18 removed lines patch added patch discarded remove patch
@@ -62,9 +62,9 @@  discard block
 block discarded – undo
62 62
 	}
63 63
 
64 64
 	private function afterInsert($item) {
65
-		if (!$item instanceof HtmlMenu && $item->propertyContains("class", "header")===false)
66
-			$item->addToPropertyCtrl("class", "item", array ("item" ));
67
-		else {
65
+		if (!$item instanceof HtmlMenu && $item->propertyContains("class", "header")===false) {
66
+					$item->addToPropertyCtrl("class", "item", array ("item" ));
67
+		} else {
68 68
 			$this->setSecondary();
69 69
 		}
70 70
 		return $item;
@@ -79,8 +79,9 @@  discard block
 block discarded – undo
79 79
 	public function addItem($item) {
80 80
 		$number=$item;
81 81
 		$item=parent::addItem($this->getItemToInsert($item));
82
-		if(\is_int($number))
83
-			$item->setProperty("data-page", $number);
82
+		if(\is_int($number)) {
83
+					$item->setProperty("data-page", $number);
84
+		}
84 85
 		return $this->afterInsert($item);
85 86
 	}
86 87
 
@@ -121,7 +122,7 @@  discard block
 block discarded – undo
121 122
 		if(!($caption instanceof HtmlDoubleElement)){
122 123
 			$header=new HtmlDoubleElement('','div');
123 124
 			$header->setContent($caption);
124
-		}else{
125
+		} else{
125 126
 			$header=$caption;
126 127
 		}
127 128
 		$header->addClass('item header');
@@ -166,10 +167,11 @@  discard block
 block discarded – undo
166 167
 	}
167 168
 
168 169
 	public function setSecondary($value=true) {
169
-		if($value)
170
-			$this->addToProperty("class", "secondary");
171
-		else
172
-			$this->removePropertyValue("class", "secondary");
170
+		if($value) {
171
+					$this->addToProperty("class", "secondary");
172
+		} else {
173
+					$this->removePropertyValue("class", "secondary");
174
+		}
173 175
 		return $this;
174 176
 	}
175 177
 
@@ -189,8 +191,9 @@  discard block
 block discarded – undo
189 191
 		$this->apply(function (HtmlDoubleElement &$item) {
190 192
 			$item->setTagName("a");
191 193
 		});
192
-		if ($vertical === true)
193
-			$this->setVertical();
194
+		if ($vertical === true) {
195
+					$this->setVertical();
196
+		}
194 197
 		return $this->addToProperty("class", "tabular");
195 198
 	}
196 199
 
@@ -219,10 +222,11 @@  discard block
 block discarded – undo
219 222
 	 */
220 223
 	public function fromDatabaseObject($object, $function) {
221 224
 		$return=$function($object);
222
-		if (\is_array($return))
223
-			$this->addItems($return);
224
-		else
225
-			$this->addItem($return);
225
+		if (\is_array($return)) {
226
+					$this->addItems($return);
227
+		} else {
228
+					$this->addItem($return);
229
+		}
226 230
 	}
227 231
 
228 232
 	/**
@@ -255,8 +259,9 @@  discard block
 block discarded – undo
255 259
 	}
256 260
 
257 261
 	public function run(JsUtils $js){
258
-		if($this->identifier!=="" && !isset($this->_bsComponent))
259
-			$this->onClick('if(!$(this).hasClass("dropdown")&&!$(this).hasClass("no-active")){$(this).addClass("active").siblings().removeClass("active");}',false,false);
262
+		if($this->identifier!=="" && !isset($this->_bsComponent)) {
263
+					$this->onClick('if(!$(this).hasClass("dropdown")&&!$(this).hasClass("no-active")){$(this).addClass("active").siblings().removeClass("active");}',false,false);
264
+		}
260 265
 		$result= parent::run($js);
261 266
 		return $result->setItemSelector(">.item:not(.header)");
262 267
 	}
Please login to merge, or discard this patch.