Completed
Pull Request — master (#586)
by Richard
14:46
created
xoops_lib/Xoops/Form/TabTray.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -49,17 +49,17 @@  discard block
 block discarded – undo
49 49
     {
50 50
         $xoops = \Xoops::getInstance();
51 51
 
52
-        $ret = '<div id="tabs_' . $this->getName() . '">' . "\n";
53
-        $ret .= '<ul class="nav nav-tabs">' . "\n";
52
+        $ret = '<div id="tabs_'.$this->getName().'">'."\n";
53
+        $ret .= '<ul class="nav nav-tabs">'."\n";
54 54
         $active = ' active';
55 55
         foreach ($this->getElements() as $ele) {
56 56
             if ($ele instanceof Tab) {
57
-                $ret .= '<li class="nav' . $active . '"><a href="#tab_' . $ele->getName()
58
-                    . '" data-toggle="tab">' . $ele->getCaption() . '</a></li>' . "\n";
57
+                $ret .= '<li class="nav'.$active.'"><a href="#tab_'.$ele->getName()
58
+                    . '" data-toggle="tab">'.$ele->getCaption().'</a></li>'."\n";
59 59
                 $active = '';
60 60
             }
61 61
         }
62
-        $ret .= '</ul><br>' . "\n";
62
+        $ret .= '</ul><br>'."\n";
63 63
 
64 64
         $hidden = '';
65 65
         $extras = array();
@@ -72,9 +72,9 @@  discard block
 block discarded – undo
72 72
             if (!$ele->isHidden()) {
73 73
                 if (!$ele instanceof Raw) {
74 74
                     if ($ele instanceof Tab) {
75
-                        $ret .= '<div class="tab-pane fade' . $active .'" id="tab_'. $ele->getName() . '">';
75
+                        $ret .= '<div class="tab-pane fade'.$active.'" id="tab_'.$ele->getName().'">';
76 76
                         $ret .= $ele->render();
77
-                        $ret .= '</div>' . "\n";
77
+                        $ret .= '</div>'."\n";
78 78
                         $active = '';
79 79
                     } else {
80 80
                         $extras[] = $ele;
@@ -95,9 +95,9 @@  discard block
 block discarded – undo
95 95
             $ret .= "\n";
96 96
         }
97 97
 
98
-        $ret .= $hidden . "\n";
99
-        $ret .= '</div>' . "\n";
100
-        $ret .= '</div>' . "\n";
98
+        $ret .= $hidden."\n";
99
+        $ret .= '</div>'."\n";
100
+        $ret .= '</div>'."\n";
101 101
         return $ret;
102 102
     }
103 103
 }
Please login to merge, or discard this patch.
xoops_lib/Xoops/Form/Hidden.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -52,6 +52,6 @@
 block discarded – undo
52 52
     public function render()
53 53
     {
54 54
         $attributes = $this->renderAttributeString();
55
-        return '<input ' . $attributes . $this->getExtra() .' >' . "\n";
55
+        return '<input '.$attributes.$this->getExtra().' >'."\n";
56 56
     }
57 57
 }
Please login to merge, or discard this patch.
xoops_lib/Xoops/Form/DhtmlTextArea.php 1 patch
Spacing   +43 added lines, -44 removed lines patch added patch discarded remove patch
@@ -142,7 +142,7 @@  discard block
 block discarded – undo
142 142
                 }
143 143
             } else {
144 144
                 list ($class, $path) = $this->htmlEditor;
145
-                include_once \XoopsBaseConfig::get('root-path') . $path;
145
+                include_once \XoopsBaseConfig::get('root-path').$path;
146 146
                 if (class_exists($class)) {
147 147
                     $this->htmlEditor = new $class($options);
148 148
                 }
@@ -171,17 +171,17 @@  discard block
 block discarded – undo
171 171
 
172 172
         $xoops = \Xoops::getInstance();
173 173
 
174
-        $extra = ($this->getExtra() != '' ? " " . $this->getExtra() : '');
174
+        $extra = ($this->getExtra() != '' ? " ".$this->getExtra() : '');
175 175
         $ret = "";
176 176
         // actions
177
-        $ret .= $this->codeIcon() . "<br />\n";
177
+        $ret .= $this->codeIcon()."<br />\n";
178 178
         // fonts
179 179
         $ret .= $this->fontArray();
180 180
         // length checker
181 181
         $ret .= '<button type="button" class="btn btn-xs btn-default" onclick="XoopsCheckLength(\''
182
-            . $this->getName() . '\', \'' . @$this->configs['maxlength'] . '\', \''
183
-            . \XoopsLocale::F_CURRENT_TEXT_LENGTH . '\', \'' . \XoopsLocale::MAXIMUM_LENGTH . '\');"'
184
-            . ' title="' . \XoopsLocale::CHECK_TEXT_LENGTH . '">'
182
+            . $this->getName().'\', \''.@$this->configs['maxlength'].'\', \''
183
+            . \XoopsLocale::F_CURRENT_TEXT_LENGTH.'\', \''.\XoopsLocale::MAXIMUM_LENGTH.'\');"'
184
+            . ' title="'.\XoopsLocale::CHECK_TEXT_LENGTH.'">'
185 185
             . '<span class="glyphicon glyphicon-check"></span></button>';
186 186
         $ret .= "\n";
187 187
         // the textarea box
@@ -190,7 +190,7 @@  discard block
 block discarded – undo
190 190
         $this->themeDecorateElement();
191 191
         $attributes = $this->renderAttributeString();
192 192
 
193
-        $ret .= '<textarea ' . $attributes . $extra . '>' . $this->getValue() . "</textarea>\n";
193
+        $ret .= '<textarea '.$attributes.$extra.'>'.$this->getValue()."</textarea>\n";
194 194
 
195 195
         if (empty($this->skipPreview)) {
196 196
             if (!$xoops->theme()) {
@@ -198,15 +198,15 @@  discard block
 block discarded – undo
198 198
             } else {
199 199
                 $xoops->theme()->addScript('media/xoops/image.js', array('type' => 'text/javascript'));
200 200
             }
201
-            $button = "<input id='" . $this->getName() . "_preview_button' " . "type='button' " . "class='btn btn-sm btn-default' value='" . \XoopsLocale::A_PREVIEW . "' " . "onclick=\"form_instantPreview('" . XOOPS_URL . "', '" . $this->getName() . "','" . XOOPS_URL . "/images', " . (int)($this->doHtml) . ", '" . $xoops->security()->createToken() . "')\"" . " />";
202
-            $ret .= "<br />" . "<div id='" . $this->getName() . "_hidden' style='display: block;'> " . "<fieldset>" . "<legend>" . $button . "</legend>" . "<div id='" . $this->getName() . "_hidden_data'>" . \XoopsLocale::CLICK_PREVIEW_TO_SEE_CONTENT . "</div>" . "</fieldset>" . "</div>";
201
+            $button = "<input id='".$this->getName()."_preview_button' "."type='button' "."class='btn btn-sm btn-default' value='".\XoopsLocale::A_PREVIEW."' "."onclick=\"form_instantPreview('".XOOPS_URL."', '".$this->getName()."','".XOOPS_URL."/images', ".(int)($this->doHtml).", '".$xoops->security()->createToken()."')\""." />";
202
+            $ret .= "<br />"."<div id='".$this->getName()."_hidden' style='display: block;'> "."<fieldset>"."<legend>".$button."</legend>"."<div id='".$this->getName()."_hidden_data'>".\XoopsLocale::CLICK_PREVIEW_TO_SEE_CONTENT."</div>"."</fieldset>"."</div>";
203 203
         }
204 204
         // Load javascript
205 205
         if (empty($js_loaded)) {
206 206
             $javascript = (($this->js)
207
-                ? '<script type="text/javascript">' . $this->js . '</script>'
208
-                : '') . '<script type="text/javascript" src="' . \XoopsBaseConfig::get('url') . '/include/formdhtmltextarea.js"></script>';
209
-            $ret = $javascript . $ret;
207
+                ? '<script type="text/javascript">'.$this->js.'</script>'
208
+                : '').'<script type="text/javascript" src="'.\XoopsBaseConfig::get('url').'/include/formdhtmltextarea.js"></script>';
209
+            $ret = $javascript.$ret;
210 210
             $js_loaded = true;
211 211
         }
212 212
         return $ret;
@@ -224,22 +224,22 @@  discard block
 block discarded – undo
224 224
         $myts = \Xoops\Core\Text\Sanitizer::getInstance();
225 225
 
226 226
         $code = '';
227
-        $code .= '<img src="' . $xoops->url('images/form/url.gif') . '" alt="' . \XoopsLocale::URL
228
-            . '" title="' . \XoopsLocale::URL . '" onclick="xoopsCodeUrl(\'' . $textarea_id . '\', \''
229
-            . $myts->escapeForJavascript(\XoopsLocale::ENTER_LINK_URL) . '\', \''
227
+        $code .= '<img src="'.$xoops->url('images/form/url.gif').'" alt="'.\XoopsLocale::URL
228
+            . '" title="'.\XoopsLocale::URL.'" onclick="xoopsCodeUrl(\''.$textarea_id.'\', \''
229
+            . $myts->escapeForJavascript(\XoopsLocale::ENTER_LINK_URL).'\', \''
230 230
             . $myts->escapeForJavascript(\XoopsLocale::ENTER_WEBSITE_TITLE)
231 231
             . '\')" onmouseover="style.cursor=\'hand\'" />&nbsp;';
232
-        $code .= '<img src="' . $xoops->url('images/form/email.gif') . '" alt="' . \XoopsLocale::EMAIL
233
-            . '" title="' . \XoopsLocale::EMAIL . '" onclick="xoopsCodeEmail(\'' . $textarea_id . '\', \''
232
+        $code .= '<img src="'.$xoops->url('images/form/email.gif').'" alt="'.\XoopsLocale::EMAIL
233
+            . '" title="'.\XoopsLocale::EMAIL.'" onclick="xoopsCodeEmail(\''.$textarea_id.'\', \''
234 234
             . $myts->escapeForJavascript(\XoopsLocale::ENTER_EMAIL)
235 235
             . '\');"  onmouseover="style.cursor=\'hand\'" />&nbsp;';
236
-        $code .= '<img src="' . $xoops->url('images/form/imgsrc.gif') . '" alt="' . \XoopsLocale::IMAGES
237
-            . '" title="' . \XoopsLocale::IMAGES . '" onclick="xoopsCodeImg(\'' . $textarea_id . '\', \''
238
-            . $myts->escapeForJavascript(\XoopsLocale::ENTER_IMAGE_URL) . '\', \''
239
-            . $myts->escapeForJavascript(\XoopsLocale::ENTER_IMAGE_POSITION) . '\', \''
240
-            . $myts->escapeForJavascript(\XoopsLocale::IMAGE_POSITION_DESCRIPTION) . '\', \''
241
-            . $myts->escapeForJavascript(\XoopsLocale::E_ENTER_IMAGE_POSITION) . '\', \''
242
-            . $myts->escapeForJavascript(\XoopsLocale::WIDTH) . '\');" onmouseover="style.cursor=\'hand\'" />&nbsp;';
236
+        $code .= '<img src="'.$xoops->url('images/form/imgsrc.gif').'" alt="'.\XoopsLocale::IMAGES
237
+            . '" title="'.\XoopsLocale::IMAGES.'" onclick="xoopsCodeImg(\''.$textarea_id.'\', \''
238
+            . $myts->escapeForJavascript(\XoopsLocale::ENTER_IMAGE_URL).'\', \''
239
+            . $myts->escapeForJavascript(\XoopsLocale::ENTER_IMAGE_POSITION).'\', \''
240
+            . $myts->escapeForJavascript(\XoopsLocale::IMAGE_POSITION_DESCRIPTION).'\', \''
241
+            . $myts->escapeForJavascript(\XoopsLocale::E_ENTER_IMAGE_POSITION).'\', \''
242
+            . $myts->escapeForJavascript(\XoopsLocale::WIDTH).'\');" onmouseover="style.cursor=\'hand\'" />&nbsp;';
243 243
 
244 244
         $extensions = array_filter($myts->listExtensions());
245 245
         foreach ($extensions as $extension) {
@@ -251,13 +251,13 @@  discard block
 block discarded – undo
251 251
                 $this->js .= $js;
252 252
             }
253 253
         }
254
-        $code .= '<img src="' . $xoops->url('images/form/code.gif') .'" alt="' . \XoopsLocale::SOURCE_CODE . '" title="'
255
-            . \XoopsLocale::SOURCE_CODE . '" onclick="xoopsCodeCode(\'' . $textarea_id . '\', \''
256
-            . $myts->escapeForJavascript(\XoopsLocale::ENTER_CODE) . '\');" onmouseover="style.cursor=\'hand\'" />&nbsp;';
254
+        $code .= '<img src="'.$xoops->url('images/form/code.gif').'" alt="'.\XoopsLocale::SOURCE_CODE.'" title="'
255
+            . \XoopsLocale::SOURCE_CODE.'" onclick="xoopsCodeCode(\''.$textarea_id.'\', \''
256
+            . $myts->escapeForJavascript(\XoopsLocale::ENTER_CODE).'\');" onmouseover="style.cursor=\'hand\'" />&nbsp;';
257 257
 
258
-        $code .= '<img src="' . $xoops->url('images/form/quote.gif') .'" alt="' . \XoopsLocale::QUOTE . '" title="'
259
-            . \XoopsLocale::QUOTE . '" onclick="xoopsCodeQuote(\'' . $textarea_id . '\', \''
260
-            . $myts->escapeForJavascript(\XoopsLocale::ENTER_QUOTE) . '\');" onmouseover="style.cursor=\'hand\'" />&nbsp;';
258
+        $code .= '<img src="'.$xoops->url('images/form/quote.gif').'" alt="'.\XoopsLocale::QUOTE.'" title="'
259
+            . \XoopsLocale::QUOTE.'" onclick="xoopsCodeQuote(\''.$textarea_id.'\', \''
260
+            . $myts->escapeForJavascript(\XoopsLocale::ENTER_QUOTE).'\');" onmouseover="style.cursor=\'hand\'" />&nbsp;';
261 261
 
262 262
         $response = \Xoops::getInstance()->service('emoji')->renderEmojiSelector($this->getName());
263 263
         if ($response->isSuccess()) {
@@ -279,24 +279,23 @@  discard block
 block discarded – undo
279 279
         $hiddentext = $this->hiddenText;
280 280
 
281 281
         $fontStr = "<script type=\"text/javascript\" language=\"JavaScript\">";
282
-        $fontStr .= "var _editor_dialog = ''" . "+ '<select class=\"span2\" id=\'{$textarea_id}Size\' onchange=\'xoopsSetElementAttribute(\"size\", this.options[this.selectedIndex].value, \"{$textarea_id}\", \"{$hiddentext}\");\'>'";
283
-        $fontStr .= "+ '<option value=\'SIZE\'>" . \XoopsLocale::SIZE . "</option>'";
282
+        $fontStr .= "var _editor_dialog = ''"."+ '<select class=\"span2\" id=\'{$textarea_id}Size\' onchange=\'xoopsSetElementAttribute(\"size\", this.options[this.selectedIndex].value, \"{$textarea_id}\", \"{$hiddentext}\");\'>'";
283
+        $fontStr .= "+ '<option value=\'SIZE\'>".\XoopsLocale::SIZE."</option>'";
284 284
         $localeFontSizes = \XoopsLocale::getFontSizes();
285 285
         foreach ($localeFontSizes as $_val => $_name) {
286 286
             $fontStr .= " + '<option value=\'{$_val}\'>{$_name}</option>'";
287 287
         }
288 288
         $fontStr .= " + '</select> '";
289 289
         $fontStr .= "+ '<select class=\"span2\" id=\'{$textarea_id}Font\' onchange=\'xoopsSetElementAttribute(\"font\", this.options[this.selectedIndex].value, \"{$textarea_id}\", \"{$hiddentext}\");\'>'";
290
-        $fontStr .= "+ '<option value=\'FONT\'>" . \XoopsLocale::FONT . "</option>'";
290
+        $fontStr .= "+ '<option value=\'FONT\'>".\XoopsLocale::FONT."</option>'";
291 291
         $localeFonts = \XoopsLocale::getFonts();
292
-        $fontarray = !empty($localeFonts) ? $localeFonts :
293
-            array("Arial", "Courier", "Georgia", "Helvetica", "Impact", "Verdana", "Haettenschweiler");
292
+        $fontarray = !empty($localeFonts) ? $localeFonts : array("Arial", "Courier", "Georgia", "Helvetica", "Impact", "Verdana", "Haettenschweiler");
294 293
         foreach ($fontarray as $font) {
295 294
             $fontStr .= " + '<option value=\'{$font}\'>{$font}</option>'";
296 295
         }
297 296
         $fontStr .= " + '</select> '";
298 297
         $fontStr .= "+ '<select class=\"span2\" id=\'{$textarea_id}Color\' onchange=\'xoopsSetElementAttribute(\"color\", this.options[this.selectedIndex].value, \"{$textarea_id}\", \"{$hiddentext}\");\'>'";
299
-        $fontStr .= "+ '<option value=\'COLOR\'>" . \XoopsLocale::COLOR . "</option>';";
298
+        $fontStr .= "+ '<option value=\'COLOR\'>".\XoopsLocale::COLOR."</option>';";
300 299
         $fontStr .= "var _color_array = new Array('00', '33', '66', '99', 'CC', 'FF');
301 300
             for(var i = 0; i < _color_array.length; i ++) {
302 301
                 for(var j = 0; j < _color_array.length; j ++) {
@@ -310,15 +309,15 @@  discard block
 block discarded – undo
310 309
 
311 310
         $fontStr .= "document.write(_editor_dialog); </script>";
312 311
 
313
-        $styleStr = "<img src='" . \XoopsBaseConfig::get('url') . "/images/bold.gif' alt='" . \XoopsLocale::BOLD . "' title='" . \XoopsLocale::BOLD . "' onmouseover='style.cursor=\"hand\"' onclick='xoopsMakeBold(\"{$hiddentext}\", \"{$textarea_id}\");' />&nbsp;";
314
-        $styleStr .= "<img src='" . \XoopsBaseConfig::get('url') . "/images/italic.gif' alt='" . \XoopsLocale::ITALIC . "' title='" . \XoopsLocale::ITALIC . "' onmouseover='style.cursor=\"hand\"' onclick='xoopsMakeItalic(\"{$hiddentext}\", \"{$textarea_id}\");' />&nbsp;";
315
-        $styleStr .= "<img src='" . \XoopsBaseConfig::get('url') . "/images/underline.gif' alt='" . \XoopsLocale::UNDERLINE . "' title='" . \XoopsLocale::UNDERLINE . "' onmouseover='style.cursor=\"hand\"' onclick='xoopsMakeUnderline(\"{$hiddentext}\", \"{$textarea_id}\");'/>&nbsp;";
316
-        $styleStr .= "<img src='" . \XoopsBaseConfig::get('url') . "/images/linethrough.gif' alt='" . \XoopsLocale::LINE_THROUGH . "' title='" . \XoopsLocale::LINE_THROUGH . "' onmouseover='style.cursor=\"hand\"' onclick='xoopsMakeLineThrough(\"{$hiddentext}\", \"{$textarea_id}\");' />&nbsp;";
312
+        $styleStr = "<img src='".\XoopsBaseConfig::get('url')."/images/bold.gif' alt='".\XoopsLocale::BOLD."' title='".\XoopsLocale::BOLD."' onmouseover='style.cursor=\"hand\"' onclick='xoopsMakeBold(\"{$hiddentext}\", \"{$textarea_id}\");' />&nbsp;";
313
+        $styleStr .= "<img src='".\XoopsBaseConfig::get('url')."/images/italic.gif' alt='".\XoopsLocale::ITALIC."' title='".\XoopsLocale::ITALIC."' onmouseover='style.cursor=\"hand\"' onclick='xoopsMakeItalic(\"{$hiddentext}\", \"{$textarea_id}\");' />&nbsp;";
314
+        $styleStr .= "<img src='".\XoopsBaseConfig::get('url')."/images/underline.gif' alt='".\XoopsLocale::UNDERLINE."' title='".\XoopsLocale::UNDERLINE."' onmouseover='style.cursor=\"hand\"' onclick='xoopsMakeUnderline(\"{$hiddentext}\", \"{$textarea_id}\");'/>&nbsp;";
315
+        $styleStr .= "<img src='".\XoopsBaseConfig::get('url')."/images/linethrough.gif' alt='".\XoopsLocale::LINE_THROUGH."' title='".\XoopsLocale::LINE_THROUGH."' onmouseover='style.cursor=\"hand\"' onclick='xoopsMakeLineThrough(\"{$hiddentext}\", \"{$textarea_id}\");' />&nbsp;";
317 316
 
318
-        $alignStr = "<img src='" . \XoopsBaseConfig::get('url') . "/images/alignleft.gif' alt='" . \XoopsLocale::LEFT . "' title='" . \XoopsLocale::LEFT . "' onmouseover='style.cursor=\"hand\"' onclick='xoopsMakeLeft(\"{$hiddentext}\", \"{$textarea_id}\");' />&nbsp;";
319
-        $alignStr .= "<img src='" . \XoopsBaseConfig::get('url') . "/images/aligncenter.gif' alt='" . \XoopsLocale::CENTER . "' title='" . \XoopsLocale::CENTER . "' onmouseover='style.cursor=\"hand\"' onclick='xoopsMakeCenter(\"{$hiddentext}\", \"{$textarea_id}\");' />&nbsp;";
320
-        $alignStr .= "<img src='" . \XoopsBaseConfig::get('url') . "/images/alignright.gif' alt='" . \XoopsLocale::RIGHT . "' title='" . \XoopsLocale::RIGHT . "' onmouseover='style.cursor=\"hand\"' onclick='xoopsMakeRight(\"{$hiddentext}\", \"{$textarea_id}\");' />&nbsp;";
321
-        $fontStr = $fontStr . "<br />\n{$styleStr}&nbsp;{$alignStr}&nbsp;\n";
317
+        $alignStr = "<img src='".\XoopsBaseConfig::get('url')."/images/alignleft.gif' alt='".\XoopsLocale::LEFT."' title='".\XoopsLocale::LEFT."' onmouseover='style.cursor=\"hand\"' onclick='xoopsMakeLeft(\"{$hiddentext}\", \"{$textarea_id}\");' />&nbsp;";
318
+        $alignStr .= "<img src='".\XoopsBaseConfig::get('url')."/images/aligncenter.gif' alt='".\XoopsLocale::CENTER."' title='".\XoopsLocale::CENTER."' onmouseover='style.cursor=\"hand\"' onclick='xoopsMakeCenter(\"{$hiddentext}\", \"{$textarea_id}\");' />&nbsp;";
319
+        $alignStr .= "<img src='".\XoopsBaseConfig::get('url')."/images/alignright.gif' alt='".\XoopsLocale::RIGHT."' title='".\XoopsLocale::RIGHT."' onmouseover='style.cursor=\"hand\"' onclick='xoopsMakeRight(\"{$hiddentext}\", \"{$textarea_id}\");' />&nbsp;";
320
+        $fontStr = $fontStr."<br />\n{$styleStr}&nbsp;{$alignStr}&nbsp;\n";
322 321
         return $fontStr;
323 322
     }
324 323
 
Please login to merge, or discard this patch.
xoops_lib/Xoops/Form/SimpleForm.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -33,8 +33,8 @@  discard block
 block discarded – undo
33 33
      */
34 34
     public function insertBreak($extra = '', $class = '')
35 35
     {
36
-        $class = empty($class) ? '' : ' class="' . $class . '"';
37
-        $value = '<br' . $class . ' />' . $extra;
36
+        $class = empty($class) ? '' : ' class="'.$class.'"';
37
+        $value = '<br'.$class.' />'.$extra;
38 38
         $ele = new Raw($value);
39 39
         $this->addElement($ele);
40 40
     }
@@ -46,19 +46,19 @@  discard block
 block discarded – undo
46 46
      */
47 47
     public function render()
48 48
     {
49
-        $ret = $this->getTitle() . "\n<form name=\"" . $this->getName() . "\" id=\""
50
-            . $this->getName() . "\" action=\"" . $this->getAction() . "\" method=\""
51
-            . $this->getMethod() . "\"" . $this->getExtra() . ">\n";
49
+        $ret = $this->getTitle()."\n<form name=\"".$this->getName()."\" id=\""
50
+            . $this->getName()."\" action=\"".$this->getAction()."\" method=\""
51
+            . $this->getMethod()."\"".$this->getExtra().">\n";
52 52
         foreach ($this->getElements() as $ele) {
53 53
             /* @var $ele Element */
54 54
             if (!$ele->isHidden()) {
55 55
                 if (!$ele instanceof Raw) {
56
-                    $ret .= "<strong>" . $ele->getCaption() . "</strong><br />" . $ele->render() . "<br />\n";
56
+                    $ret .= "<strong>".$ele->getCaption()."</strong><br />".$ele->render()."<br />\n";
57 57
                 } else {
58 58
                     $ret .= $ele->render();
59 59
                 }
60 60
             } else {
61
-                $ret .= $ele->render() . "\n";
61
+                $ret .= $ele->render()."\n";
62 62
             }
63 63
         }
64 64
         $ret .= "</form>\n";
Please login to merge, or discard this patch.
xoops_lib/Xoops/Form/ElementTray.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -160,10 +160,10 @@
 block discarded – undo
160 160
             }
161 161
             if ($ele->getCaption() != '') {
162 162
                 $ret .= '<div class="form-group">';
163
-                $ret .= '<label class="control-label">' . $ele->getCaption() . "</label>&nbsp;";
163
+                $ret .= '<label class="control-label">'.$ele->getCaption()."</label>&nbsp;";
164 164
                 $ret .= '</div>';
165 165
             }
166
-            $ret .= $ele->render() . "\n";
166
+            $ret .= $ele->render()."\n";
167 167
             if (!$ele->isHidden()) {
168 168
                 ++$count;
169 169
             }
Please login to merge, or discard this patch.
xoops_lib/Xoops/Form/OptionElement.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -78,7 +78,7 @@
 block discarded – undo
78 78
         $myts = Sanitizer::getInstance();
79 79
         $value = array();
80 80
         foreach ($options as $val => $name) {
81
-            $value[(bool) $encode ? $myts->htmlSpecialChars($val) : $val] = ($encode > 1)
81
+            $value[(bool)$encode ? $myts->htmlSpecialChars($val) : $val] = ($encode > 1)
82 82
                 ? $myts->htmlSpecialChars($name) : $name;
83 83
         }
84 84
         return $value;
Please login to merge, or discard this patch.
xoops_lib/Xoops/Form/TableForm.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -44,27 +44,27 @@
 block discarded – undo
44 44
      */
45 45
     public function render()
46 46
     {
47
-        $ret = $this->getTitle() . "\n" . '<form name="' . $this->getName() . '" id="'
48
-            . $this->getName() . '" action="' . $this->getAction() . '" method="' . $this->getMethod()
49
-            . '"' . $this->getExtra() . '>' . "\n" . '<table border="0" width="100%">' . "\n";
47
+        $ret = $this->getTitle()."\n".'<form name="'.$this->getName().'" id="'
48
+            . $this->getName().'" action="'.$this->getAction().'" method="'.$this->getMethod()
49
+            . '"'.$this->getExtra().'>'."\n".'<table border="0" width="100%">'."\n";
50 50
         $hidden = "";
51 51
         foreach ($this->getElements() as $ele) {
52 52
             /* @var $ele Element */
53 53
             if (!$ele->isHidden()) {
54 54
                 if (!$ele instanceof Raw) {
55
-                    $ret .= '<tr valign="top" align="left"><td>' . $ele->getCaption();
55
+                    $ret .= '<tr valign="top" align="left"><td>'.$ele->getCaption();
56 56
                     if ($ele_desc = $ele->getDescription()) {
57
-                        $ret .= '<br /><br /><span style="font-weight: normal;">' . $ele_desc . '</span>';
57
+                        $ret .= '<br /><br /><span style="font-weight: normal;">'.$ele_desc.'</span>';
58 58
                     }
59
-                    $ret .= '</td><td>' . $ele->render() . '</td></tr>';
59
+                    $ret .= '</td><td>'.$ele->render().'</td></tr>';
60 60
                 } else {
61 61
                     $ret .= $ele->render();
62 62
                 }
63 63
             } else {
64
-                $hidden .= $ele->render() . "\n";
64
+                $hidden .= $ele->render()."\n";
65 65
             }
66 66
         }
67
-        $ret .= '</table>' . "\n" . ' ' . $hidden . '</form>' . "\n";
67
+        $ret .= '</table>'."\n".' '.$hidden.'</form>'."\n";
68 68
         return $ret;
69 69
     }
70 70
 }
Please login to merge, or discard this patch.
xoops_lib/Xoops/Form/SelectEditor.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -83,7 +83,7 @@
 block discarded – undo
83 83
         $editor_handler->allowed_editors = $this->allowed_editors;
84 84
         $option_select = new Select("", $this->name, $this->value);
85 85
         $onchangeCode = '"if(this.options[this.selectedIndex].value.length > 0 ){window.document.forms.'
86
-            . $this->form->getName() . '.submit();}"';
86
+            . $this->form->getName().'.submit();}"';
87 87
         $option_select->set('onchange', $onchangeCode);
88 88
         $option_select->addOptionArray($editor_handler->getList($this->nohtml));
89 89
         $this->addElement($option_select);
Please login to merge, or discard this patch.
xoops_lib/Xoops/Form/Captcha.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -74,7 +74,7 @@
 block discarded – undo
74 74
         $configs = $this->captchaHandler->loadConfig();
75 75
 
76 76
         $this->captchaHandler->setConfigs($configs);
77
-        if (! $this->captchaHandler->isActive()) {
77
+        if (!$this->captchaHandler->isActive()) {
78 78
             $this->setHidden();
79 79
         }
80 80
     }
Please login to merge, or discard this patch.