@@ -89,8 +89,9 @@ |
||
89 | 89 | } |
90 | 90 | |
91 | 91 | public function setPagesVisibles($pages_visibles) { |
92 | - if(!isset($pages_visibles)) |
|
93 | - $pages_visibles=(int)ceil($this->row_count / $this->items_per_page)+1; |
|
92 | + if(!isset($pages_visibles)) { |
|
93 | + $pages_visibles=(int)ceil($this->row_count / $this->items_per_page)+1; |
|
94 | + } |
|
94 | 95 | $this->pages_visibles=$pages_visibles; |
95 | 96 | return $this; |
96 | 97 | } |
@@ -21,7 +21,7 @@ |
||
21 | 21 | if($class->getProperty($property)!==null){ |
22 | 22 | \call_user_func_array([$object,$name],[$value]); |
23 | 23 | } |
24 | - }catch(\Exception $e){ |
|
24 | + } catch(\Exception $e){ |
|
25 | 25 | //Nothing to do |
26 | 26 | } |
27 | 27 | } |
@@ -13,8 +13,9 @@ |
||
13 | 13 | public function __construct($identifier, $href="#", $content="Link",$target=NULL) { |
14 | 14 | parent::__construct($identifier, "a", ""); |
15 | 15 | $this->setHref($href); |
16 | - if(isset($target)) |
|
17 | - $this->setTarget($target); |
|
16 | + if(isset($target)) { |
|
17 | + $this->setTarget($target); |
|
18 | + } |
|
18 | 19 | $this->content=$content; |
19 | 20 | } |
20 | 21 |
@@ -89,8 +89,9 @@ discard block |
||
89 | 89 | public function asLink($href=NULL,$target=NULL) { |
90 | 90 | if (isset($href)) { |
91 | 91 | $_target=""; |
92 | - if(isset($target)) |
|
93 | - $_target="target='{$target}'"; |
|
92 | + if(isset($target)) { |
|
93 | + $_target="target='{$target}'"; |
|
94 | + } |
|
94 | 95 | $this->wrap("<a href='" . $href . "' {$_target}>", "</a>"); |
95 | 96 | } |
96 | 97 | return $this->addToProperty("class", "link"); |
@@ -107,8 +108,9 @@ discard block |
||
107 | 108 | */ |
108 | 109 | public function setBordered($inverted=false) { |
109 | 110 | $invertedStr=""; |
110 | - if ($inverted !== false) |
|
111 | - $invertedStr=" inverted"; |
|
111 | + if ($inverted !== false) { |
|
112 | + $invertedStr=" inverted"; |
|
113 | + } |
|
112 | 114 | return $this->addToProperty("class", "bordered" . $invertedStr); |
113 | 115 | } |
114 | 116 | |
@@ -121,12 +123,14 @@ discard block |
||
121 | 123 | } |
122 | 124 | |
123 | 125 | public function addLabel($label, $before=false, $icon=NULL) { |
124 | - if($before) |
|
125 | - $this->wrap($label); |
|
126 | - else |
|
127 | - $this->wrap("", $label); |
|
128 | - if(isset($icon)) |
|
129 | - $this->addToIcon($icon); |
|
126 | + if($before) { |
|
127 | + $this->wrap($label); |
|
128 | + } else { |
|
129 | + $this->wrap("", $label); |
|
130 | + } |
|
131 | + if(isset($icon)) { |
|
132 | + $this->addToIcon($icon); |
|
133 | + } |
|
130 | 134 | return $this; |
131 | 135 | } |
132 | 136 |
@@ -22,15 +22,17 @@ |
||
22 | 22 | |
23 | 23 | public function setBasic($very=false) { |
24 | 24 | $table=$this->_self; |
25 | - if ($very) |
|
26 | - $table->addToPropertyCtrl("class", "very", array ("very" )); |
|
25 | + if ($very) { |
|
26 | + $table->addToPropertyCtrl("class", "very", array ("very" )); |
|
27 | + } |
|
27 | 28 | return $table->addToPropertyCtrl("class", "basic", array ("basic" )); |
28 | 29 | } |
29 | 30 | |
30 | 31 | public function setCompact($very=false) { |
31 | 32 | $table=$this->_self; |
32 | - if ($very) |
|
33 | - $table->addToPropertyCtrl("class", "very", array ("very" )); |
|
33 | + if ($very) { |
|
34 | + $table->addToPropertyCtrl("class", "very", array ("very" )); |
|
35 | + } |
|
34 | 36 | return $table->addToPropertyCtrl("class", "compact", array ("compact" )); |
35 | 37 | } |
36 | 38 |
@@ -30,8 +30,9 @@ |
||
30 | 30 | if (isset($extra)) { |
31 | 31 | $result->addExtraContent($extra); |
32 | 32 | } |
33 | - } else |
|
34 | - $result->addItemContent($value); |
|
33 | + } else { |
|
34 | + $result->addItemContent($value); |
|
35 | + } |
|
35 | 36 | return $result; |
36 | 37 | } |
37 | 38 |
@@ -18,8 +18,9 @@ |
||
18 | 18 | |
19 | 19 | public function getPart($partKey, $index=NULL,$force=false) { |
20 | 20 | if (\array_key_exists($partKey, $this->content)) { |
21 | - if (isset($index)) |
|
22 | - return $this->content[$partKey][$index]; |
|
21 | + if (isset($index)) { |
|
22 | + return $this->content[$partKey][$index]; |
|
23 | + } |
|
23 | 24 | return $this->content[$partKey]; |
24 | 25 | } |
25 | 26 | if($force){ |
@@ -19,8 +19,9 @@ discard block |
||
19 | 19 | public function __construct($identifier,$baseClass,$content=NULL) { |
20 | 20 | parent::__construct($identifier, "div", $baseClass); |
21 | 21 | $this->content=["content"=>new HtmlViewContent("content-".$this->identifier)]; |
22 | - if(isset($content)) |
|
23 | - $this->setContent($content); |
|
22 | + if(isset($content)) { |
|
23 | + $this->setContent($content); |
|
24 | + } |
|
24 | 25 | } |
25 | 26 | |
26 | 27 | public function setContent($value){ |
@@ -31,10 +32,12 @@ discard block |
||
31 | 32 | if (isset($image)) { |
32 | 33 | $this->addImage($image); |
33 | 34 | } |
34 | - if(isset($content)) |
|
35 | - $this->content["content"]->setContent($content); |
|
36 | - if(isset($extra)) |
|
37 | - $this->addExtraContent($extra); |
|
35 | + if(isset($content)) { |
|
36 | + $this->content["content"]->setContent($content); |
|
37 | + } |
|
38 | + if(isset($extra)) { |
|
39 | + $this->addExtraContent($extra); |
|
40 | + } |
|
38 | 41 | } |
39 | 42 | } |
40 | 43 | |
@@ -49,10 +52,11 @@ discard block |
||
49 | 52 | if (\array_key_exists($key, $this->content) === false) { |
50 | 53 | $this->content[$key]=[]; |
51 | 54 | } |
52 | - if($this->content[$key] instanceof HtmlViewContent) |
|
53 | - $this->content[$key]->addElement($element); |
|
54 | - else |
|
55 | - $this->content[$key][]=$element; |
|
55 | + if($this->content[$key] instanceof HtmlViewContent) { |
|
56 | + $this->content[$key]->addElement($element); |
|
57 | + } else { |
|
58 | + $this->content[$key][]=$element; |
|
59 | + } |
|
56 | 60 | return $element; |
57 | 61 | } |
58 | 62 | |
@@ -153,8 +157,9 @@ discard block |
||
153 | 157 | $this->addToProperty("class", "link"); |
154 | 158 | if ($href !== "") { |
155 | 159 | $this->setProperty("href", $href); |
156 | - if (isset($target)) |
|
157 | - $this->setProperty("target", $target); |
|
160 | + if (isset($target)) { |
|
161 | + $this->setProperty("target", $target); |
|
162 | + } |
|
158 | 163 | } |
159 | 164 | return $this; |
160 | 165 | } |
@@ -64,8 +64,7 @@ discard block |
||
64 | 64 | $rules=$attributes["rules"]; |
65 | 65 | if(\is_array($rules)){ |
66 | 66 | $element->addRules($rules); |
67 | - } |
|
68 | - else{ |
|
67 | + } else{ |
|
69 | 68 | $element->addRule($rules); |
70 | 69 | } |
71 | 70 | unset($attributes["rules"]); |
@@ -117,8 +116,9 @@ discard block |
||
117 | 116 | public function fieldAsLabel($index,$icon=NULL,$attributes=NULL){ |
118 | 117 | return $this->_fieldAs(function($id,$name,$value) use($icon){ |
119 | 118 | $lbl=new HtmlLabel($id,$value); |
120 | - if(isset($icon)) |
|
121 | - $lbl->addIcon($icon); |
|
119 | + if(isset($icon)) { |
|
120 | + $lbl->addIcon($icon); |
|
121 | + } |
|
122 | 122 | return $lbl; |
123 | 123 | }, $index,$attributes,"label"); |
124 | 124 | } |
@@ -126,8 +126,9 @@ discard block |
||
126 | 126 | public function fieldAsHeader($index,$niveau=1,$icon=NULL,$attributes=NULL){ |
127 | 127 | return $this->_fieldAs(function($id,$name,$value) use($niveau,$icon){ |
128 | 128 | $header=new HtmlHeader($id,$niveau,$value); |
129 | - if(isset($icon)) |
|
130 | - $header->asIcon($icon, $value); |
|
129 | + if(isset($icon)) { |
|
130 | + $header->asIcon($icon, $value); |
|
131 | + } |
|
131 | 132 | return $header; |
132 | 133 | }, $index,$attributes,"header"); |
133 | 134 | } |
@@ -135,7 +136,9 @@ discard block |
||
135 | 136 | |
136 | 137 | public function fieldAsImage($index,$size=Size::MINI,$circular=false){ |
137 | 138 | $this->setValueFunction($index,function($img) use($size,$circular){ |
138 | - $image=new HtmlImage($this->_getFieldIdentifier("image"),$img);$image->setSize($size);if($circular)$image->setCircular(); |
|
139 | + $image=new HtmlImage($this->_getFieldIdentifier("image"),$img);$image->setSize($size);if($circular) { |
|
140 | + $image->setCircular(); |
|
141 | + } |
|
139 | 142 | return $image; |
140 | 143 | }); |
141 | 144 | return $this; |
@@ -194,8 +197,9 @@ discard block |
||
194 | 197 | |
195 | 198 | public function fieldAsCheckbox($index,$attributes=NULL){ |
196 | 199 | return $this->_fieldAs(function($id,$name,$value,$caption) use($attributes){ |
197 | - if($caption===null || $caption==="") |
|
198 | - $caption=""; |
|
200 | + if($caption===null || $caption==="") { |
|
201 | + $caption=""; |
|
202 | + } |
|
199 | 203 | $input=new HtmlFormCheckbox($id,$caption,$this->_instanceViewer->getIdentifier()); |
200 | 204 | $input->setChecked(JString::isBooleanTrue($value)); |
201 | 205 | return $this->_prepareFormFields($input, $name, $attributes); |
@@ -232,14 +236,14 @@ discard block |
||
232 | 236 | $i=0; |
233 | 237 | if(JArray::isAssociative($types)){ |
234 | 238 | foreach ($types as $type=>$attributes){ |
235 | - if(\is_int($type)) |
|
236 | - $this->fieldAs($i++,$attributes,[]); |
|
237 | - else{ |
|
239 | + if(\is_int($type)) { |
|
240 | + $this->fieldAs($i++,$attributes,[]); |
|
241 | + } else{ |
|
238 | 242 | $type=preg_replace('/\d/', '', $type ); |
239 | 243 | $this->fieldAs($i++,$type,$attributes); |
240 | 244 | } |
241 | 245 | } |
242 | - }else{ |
|
246 | + } else{ |
|
243 | 247 | foreach ($types as $type){ |
244 | 248 | $this->fieldAs($i++,$type); |
245 | 249 | } |
@@ -251,7 +255,7 @@ discard block |
||
251 | 255 | if(\method_exists($this, $method)){ |
252 | 256 | if(!\is_array($attributes)){ |
253 | 257 | $attributes=[$index]; |
254 | - }else{ |
|
258 | + } else{ |
|
255 | 259 | \array_unshift($attributes, $index); |
256 | 260 | } |
257 | 261 | \call_user_func_array([$this,$method], $attributes); |