mambax7 / XoopsCore25

htdocs/class/xoopseditor/xoopseditor.php

Spacing +14 added lines, -14 removed lines

@@ -52,8 +52,8 @@ 
         // TODO: switch to property_exists() as of PHP 5.1.0
         $vars = get_class_vars(__CLASS__);
         foreach ($configs as $key => $val) {
-            if (method_exists($this, 'set' . ucfirst($key))) {
-                $this->{'set' . ucfirst($key)}($val);
+            if (method_exists($this, 'set'.ucfirst($key))) {
+                $this->{'set'.ucfirst($key)}($val);
             } elseif (array_key_exists("_{$key}", $vars)) {
                 $this->{"_{$key}"} = $val;
             } elseif (array_key_exists($key, $vars)) {
@@ -98,7 +98,7 @@ 
      */
     public function __construct()
     {
-        $this->root_path = XOOPS_ROOT_PATH . '/class/xoopseditor';
+        $this->root_path = XOOPS_ROOT_PATH.'/class/xoopseditor';
     }
 
     /**
@@ -157,8 +157,8 @@ 
             $result = array_flip($editor_handler->getList());
         */
         if (!isset($this->root_path)) {
-            $this->root_path = XOOPS_ROOT_PATH . '/class/xoopseditor';
-            $GLOBALS['xoopsLogger']->addDeprecated(__CLASS__ . '::' . __FUNCTION__ . '() should not be called statically.');
+            $this->root_path = XOOPS_ROOT_PATH.'/class/xoopseditor';
+            $GLOBALS['xoopsLogger']->addDeprecated(__CLASS__.'::'.__FUNCTION__.'() should not be called statically.');
         }
 
         xoops_load('XoopsCache');
@@ -167,14 +167,14 @@ 
             $list  = array();
             $order = array();
             xoops_load('XoopsLists');
-            $_list = XoopsLists::getDirListAsArray($this->root_path . '/');
+            $_list = XoopsLists::getDirListAsArray($this->root_path.'/');
             foreach ($_list as $item) {
-                if (file_exists($file = $this->root_path . '/' . $item . '/language/' . $GLOBALS['xoopsConfig']['language'] . '.php')) {
+                if (file_exists($file = $this->root_path.'/'.$item.'/language/'.$GLOBALS['xoopsConfig']['language'].'.php')) {
                     include_once $file;
-                } elseif (file_exists($file = $this->root_path . '/' . $item . '/language/english.php')) {
+                } elseif (file_exists($file = $this->root_path.'/'.$item.'/language/english.php')) {
                     include_once $file;
                 }
-                if (file_exists($file = $this->root_path . '/' . $item . '/editor_registry.php')) {
+                if (file_exists($file = $this->root_path.'/'.$item.'/editor_registry.php')) {
                     include $file;
                     if (empty($config['order'])) {
                         continue;
@@ -210,7 +210,7 @@ 
      */
     public function render($editor)
     {
-        trigger_error(__CLASS__ . '::' . __FUNCTION__ . '() deprecated', E_USER_WARNING);
+        trigger_error(__CLASS__.'::'.__FUNCTION__.'() deprecated', E_USER_WARNING);
 
         return $editor->render();
     }
@@ -246,13 +246,13 @@ 
         if (empty($name) || !array_key_exists($name, $this->getList())) {
             return $editor;
         }
-        $editor_path = $this->root_path . '/' . $name;
-        if (file_exists($file = $editor_path . '/language/' . $GLOBALS['xoopsConfig']['language'] . '.php')) {
+        $editor_path = $this->root_path.'/'.$name;
+        if (file_exists($file = $editor_path.'/language/'.$GLOBALS['xoopsConfig']['language'].'.php')) {
             include_once $file;
-        } elseif (file_exists($file = $editor_path . '/language/english.php')) {
+        } elseif (file_exists($file = $editor_path.'/language/english.php')) {
             include_once $file;
         }
-        if (file_exists($file = $editor_path . '/editor_registry.php')) {
+        if (file_exists($file = $editor_path.'/editor_registry.php')) {
             include $file;
         } else {
             return $editor;

htdocs/class/xoopsmailer.php

Spacing +17 added lines, -17 removed lines

@@ -168,18 +168,18 @@ 
     public function getTemplatePath()
     {
         if (!$path = $this->templatedir) {
-            $path = XOOPS_ROOT_PATH . '/language/';
+            $path = XOOPS_ROOT_PATH.'/language/';
         } elseif (false === strpos($path, '/')) {
-            $path = XOOPS_ROOT_PATH . '/modules/' . $path . '/language/';
+            $path = XOOPS_ROOT_PATH.'/modules/'.$path.'/language/';
         } elseif (substr($path, -1, 1) !== '/') {
             $path .= '/';
         }
-        if (file_exists($path . $GLOBALS['xoopsConfig']['language'] . '/mail_template/' . $this->template)) {
-            return $path . $GLOBALS['xoopsConfig']['language'] . '/mail_template/' . $this->template;
-        } elseif (file_exists($path . 'english/mail_template/' . $this->template)) {
-            return $path . 'english/mail_template/' . $this->template;
-        } elseif (file_exists($path . $this->template)) {
-            return $path . $this->template;
+        if (file_exists($path.$GLOBALS['xoopsConfig']['language'].'/mail_template/'.$this->template)) {
+            return $path.$GLOBALS['xoopsConfig']['language'].'/mail_template/'.$this->template;
+        } elseif (file_exists($path.'english/mail_template/'.$this->template)) {
+            return $path.'english/mail_template/'.$this->template;
+        } elseif (file_exists($path.$this->template)) {
+            return $path.$this->template;
         } else {
             return false;
         }
@@ -292,7 +292,7 @@ 
         // for sending mail only
         if ($this->isMail || !empty($this->toEmails)) {
             if (!empty($this->priority)) {
-                $this->headers[] = 'X-Priority: ' . $this->priority;
+                $this->headers[] = 'X-Priority: '.$this->priority;
             }
             // $this->headers[] = "X-Mailer: PHP/".phpversion();
             // $this->headers[] = "Return-Path: ".$this->fromEmail;
@@ -307,14 +307,14 @@ 
 
         $this->assign('X_ADMINMAIL', $xoopsConfig['adminmail']);
         $this->assign('X_SITENAME', $xoopsConfig['sitename']);
-        $this->assign('X_SITEURL', XOOPS_URL . '/');
+        $this->assign('X_SITEURL', XOOPS_URL.'/');
         // TODO: also X_ADMINNAME??
         // TODO: X_SIGNATURE, X_DISCLAIMER ?? - these are probably best
         // done as includes if mail templates ever get this sophisticated
         // replace tags with actual values
         foreach ($this->assignedTags as $k => $v) {
-            $this->body    = str_replace('{' . $k . '}', $v, $this->body);
-            $this->subject = str_replace('{' . $k . '}', $v, $this->subject);
+            $this->body    = str_replace('{'.$k.'}', $v, $this->body);
+            $this->subject = str_replace('{'.$k.'}', $v, $this->subject);
         }
         $this->body = str_replace("\r\n", "\n", $this->body);
         $this->body = str_replace("\r", "\n", $this->body);
@@ -341,7 +341,7 @@ 
             $text    = str_replace('{X_UID}', $user->getVar('uid'), $this->body);
             $text    = str_replace('{X_UEMAIL}', $user->getVar('email'), $text);
             $text    = str_replace('{X_UNAME}', $user->getVar('uname'), $text);
-            $text    = str_replace('{X_UACTLINK}', XOOPS_URL . '/register.php?op=actv&id=' . $user->getVar('uid') . '&actkey=' . $user->getVar('actkey'), $text);
+            $text    = str_replace('{X_UACTLINK}', XOOPS_URL.'/register.php?op=actv&id='.$user->getVar('uid').'&actkey='.$user->getVar('actkey'), $text);
             // send mail
             if ($this->isMail) {
                 if (!$this->sendMail($user->getVar('email'), $subject, $text, $headers)) {
@@ -450,9 +450,9 @@ 
             return $this->errors;
         } else {
             if (!empty($this->errors)) {
-                $ret = '<h4>' . _ERRORS . '</h4>';
+                $ret = '<h4>'._ERRORS.'</h4>';
                 foreach ($this->errors as $error) {
-                    $ret .= $error . '<br>';
+                    $ret .= $error.'<br>';
                 }
             } else {
                 $ret = '';
@@ -476,7 +476,7 @@ 
             $ret = '';
             if (!empty($this->success)) {
                 foreach ($this->success as $suc) {
-                    $ret .= $suc . '<br>';
+                    $ret .= $suc.'<br>';
                 }
             }
 
@@ -513,7 +513,7 @@ 
      */
     public function addHeaders($value)
     {
-        $this->headers[] = trim($value) . $this->LE;
+        $this->headers[] = trim($value).$this->LE;
     }
 
     // public

htdocs/class/xoopslocal.php

Spacing +5 added lines, -5 removed lines

@@ -36,12 +36,12 @@ 
     public static function substr($str, $start, $length, $trimmarker = '...')
     {
         if (!XOOPS_USE_MULTIBYTES) {
-            return (strlen($str) - $start <= $length) ? substr($str, $start, $length) : substr($str, $start, $length - strlen($trimmarker)) . $trimmarker;
+            return (strlen($str) - $start <= $length) ? substr($str, $start, $length) : substr($str, $start, $length - strlen($trimmarker)).$trimmarker;
         }
         if (function_exists('mb_internal_encoding') && @mb_internal_encoding(_CHARSET)) {
             $str2 = mb_strcut($str, $start, $length - strlen($trimmarker));
 
-            return $str2 . (mb_strlen($str) != mb_strlen($str2) ? $trimmarker : '');
+            return $str2.(mb_strlen($str) != mb_strlen($str2) ? $trimmarker : '');
         }
 
         return $str;
@@ -87,7 +87,7 @@ 
         if (XOOPS_USE_MULTIBYTES && function_exists('mb_convert_encoding')) {
             $converted_text = @mb_convert_encoding($text, $to, $from);
         } elseif (function_exists('iconv')) {
-            $converted_text = @iconv($from, $to . '//TRANSLIT', $text);
+            $converted_text = @iconv($from, $to.'//TRANSLIT', $text);
         } elseif ('utf-8' === $to) {
             $converted_text = utf8_encode($text);
         }
@@ -138,9 +138,9 @@ 
             if (isset($GLOBALS['xoopsConfig']['server_TZ'])) {
                 $server_TZ = abs((int)($GLOBALS['xoopsConfig']['server_TZ'] * 3600.0));
                 $prefix    = ($GLOBALS['xoopsConfig']['server_TZ'] < 0) ? ' -' : ' +';
-                $TIME_ZONE = $prefix . date('Hi', $server_TZ);
+                $TIME_ZONE = $prefix.date('Hi', $server_TZ);
             }
-            $date = gmdate('D, d M Y H:i:s', (int)$time) . $TIME_ZONE;
+            $date = gmdate('D, d M Y H:i:s', (int)$time).$TIME_ZONE;
 
             return $date;
         }

htdocs/class/template.php

Indentation +6 added lines, -6 removed lines

@@ -60,12 +60,12 @@
         parent::__construct();
         $this->setCompileId();
         $this->assign(array(
-                          'xoops_url'        => XOOPS_URL,
-                          'xoops_rootpath'   => XOOPS_ROOT_PATH,
-                          'xoops_langcode'   => _LANGCODE,
-                          'xoops_charset'    => _CHARSET,
-                          'xoops_version'    => XOOPS_VERSION,
-                          'xoops_upload_url' => XOOPS_UPLOAD_URL));
+                            'xoops_url'        => XOOPS_URL,
+                            'xoops_rootpath'   => XOOPS_ROOT_PATH,
+                            'xoops_langcode'   => _LANGCODE,
+                            'xoops_charset'    => _CHARSET,
+                            'xoops_version'    => XOOPS_VERSION,
+                            'xoops_upload_url' => XOOPS_UPLOAD_URL));
     }
 
     /**

Spacing +13 added lines, -13 removed lines

@@ -22,8 +22,8 @@ 
 /**
  * Base class: Smarty template engine
  */
-define('SMARTY_DIR', XOOPS_ROOT_PATH . '/class/smarty/');
-require_once SMARTY_DIR . 'Smarty.class.php';
+define('SMARTY_DIR', XOOPS_ROOT_PATH.'/class/smarty/');
+require_once SMARTY_DIR.'Smarty.class.php';
 
 /**
  * Template engine
@@ -45,12 +45,12 @@ 
         $this->left_delimiter  = '<{';
         $this->right_delimiter = '}>';
         $this->template_dir    = XOOPS_THEME_PATH;
-        $this->cache_dir       = XOOPS_VAR_PATH . '/caches/smarty_cache';
-        $this->compile_dir     = XOOPS_VAR_PATH . '/caches/smarty_compile';
+        $this->cache_dir       = XOOPS_VAR_PATH.'/caches/smarty_cache';
+        $this->compile_dir     = XOOPS_VAR_PATH.'/caches/smarty_compile';
         $this->compile_check   = ($xoopsConfig['theme_fromfile'] == 1);
         $this->plugins_dir     = array(
-            XOOPS_ROOT_PATH . '/class/smarty/xoops_plugins',
-            XOOPS_ROOT_PATH . '/class/smarty/plugins');
+            XOOPS_ROOT_PATH.'/class/smarty/xoops_plugins',
+            XOOPS_ROOT_PATH.'/class/smarty/plugins');
         if ($xoopsConfig['debug_mode']) {
             $this->debugging_ctrl = 'URL';
             if ($xoopsConfig['debug_mode'] == 3) {
@@ -80,7 +80,7 @@ 
     public function fetchFromData($tplSource, $display = false, $vars = null)
     {
         if (!function_exists('smarty_function_eval')) {
-            require_once SMARTY_DIR . '/plugins/function.eval.php';
+            require_once SMARTY_DIR.'/plugins/function.eval.php';
         }
         if (isset($vars)) {
             $oldVars = $this->_tpl_vars;
@@ -123,7 +123,7 @@ 
     public function _get_auto_id($cache_id = null, $compile_id = null)
     {
         if (isset($cache_id)) {
-            return isset($compile_id) ? $compile_id . '-' . $cache_id : $cache_id;
+            return isset($compile_id) ? $compile_id.'-'.$cache_id : $cache_id;
         } elseif (isset($compile_id)) {
             return $compile_id;
         } else {
@@ -146,7 +146,7 @@ 
         $template_set      = empty($template_set) ? $xoopsConfig['template_set'] : $template_set;
         $theme_set         = empty($theme_set) ? $xoopsConfig['theme_set'] : $theme_set;
         $module_dirname    = empty($module_dirname) ? (empty($GLOBALS['xoopsModule']) ? 'system' : $GLOBALS['xoopsModule']->getVar('dirname', 'n')) : $module_dirname;
-        $this->compile_id  = substr(md5(XOOPS_URL), 0, 8) . '-' . $module_dirname . '-' . $theme_set . '-' . $template_set;
+        $this->compile_id  = substr(md5(XOOPS_URL), 0, 8).'-'.$module_dirname.'-'.$theme_set.'-'.$template_set;
         $this->_compile_id = $this->compile_id;
     }
 
@@ -162,13 +162,13 @@ 
     {
         $compile_id = $this->compile_id;
         $this->setCompileId($module_dirname, $template_set, $theme_set);
-        $_params           = array(
+        $_params = array(
             'auto_base'   => $this->cache_dir,
             'auto_source' => null,
             'auto_id'     => $this->compile_id,
             'exp_time'    => null);
         $this->_compile_id = $this->compile_id = $compile_id;
-        require_once SMARTY_CORE_DIR . 'core.rm_auto.php';
+        require_once SMARTY_CORE_DIR.'core.rm_auto.php';
 
         return smarty_core_rm_auto($_params, $this);
     }
@@ -288,7 +288,7 @@ 
         $file = $tplfile->getVar('tpl_file', 'n');
         $tpl  = new XoopsTpl();
 
-        return $tpl->touch('db:' . $file);
+        return $tpl->touch('db:'.$file);
     }
 
     return false;
@@ -309,7 +309,7 @@ 
         $xoopsTpl->caching = 2;
         for ($i = 0; $i < $count; ++$i) {
             if ($block_arr[$i]->getVar('template') != '') {
-                $xoopsTpl->clear_cache('db:' . $block_arr[$i]->getVar('template'), 'blk_' . $block_arr[$i]->getVar('bid'));
+                $xoopsTpl->clear_cache('db:'.$block_arr[$i]->getVar('template'), 'blk_'.$block_arr[$i]->getVar('bid'));
             }
         }
     }

htdocs/class/zipdownloader.php

Spacing +1 added lines, -1 removed lines

@@ -112,7 +112,7 @@
      */
     public function download($name, $gzip = true)
     {
-        $this->_header($name . $this->ext);
+        $this->_header($name.$this->ext);
         echo $this->archiver->file();
     }
 }

htdocs/class/xoopsform/formselect.php

Spacing +1 added lines, -1 removed lines

@@ -219,7 +219,7 @@
             $eltmsg     = empty($eltcaption) ? sprintf(_FORM_ENTER, $eltname) : sprintf(_FORM_ENTER, $eltcaption);
             $eltmsg     = str_replace('"', '\"', stripslashes($eltmsg));
 
-            return "\nvar hasSelected = false; var selectBox = myform.{$eltname};" . "for (i = 0; i < selectBox.options.length; i++) { if (selectBox.options[i].selected == true && selectBox.options[i].value != '') { hasSelected = true; break; } }" . "if (!hasSelected) { window.alert(\"{$eltmsg}\"); selectBox.focus(); return false; }";
+            return "\nvar hasSelected = false; var selectBox = myform.{$eltname};"."for (i = 0; i < selectBox.options.length; i++) { if (selectBox.options[i].selected == true && selectBox.options[i].value != '') { hasSelected = true; break; } }"."if (!hasSelected) { window.alert(\"{$eltmsg}\"); selectBox.focus(); return false; }";
         }
 
         return '';

htdocs/class/xoopsform/renderer/XoopsFormRendererBootstrap3.php

Indentation +5 added lines, -5 removed lines

@@ -50,11 +50,11 @@
                 . '" onclick="this.form.elements.op.value=\'delete\'"> ';
         }
         $ret .= '<input type="button" class="btn btn-danger" value="' . _CANCEL
-             . '" onClick="history.go(-1);return true;" />&nbsp;'
-             . '<input type="reset" class="btn btn-warning"  name="reset"  id="reset" value="' . _RESET . '" />&nbsp;'
-             . '<input type="' . $element->getType() . '" class="btn btn-success"  name="' . $element->getName()
-             . '"  id="' . $element->getName() . '" value="' . $element->getValue() . '"' . $element->getExtra()
-             . '  />';
+                . '" onClick="history.go(-1);return true;" />&nbsp;'
+                . '<input type="reset" class="btn btn-warning"  name="reset"  id="reset" value="' . _RESET . '" />&nbsp;'
+                . '<input type="' . $element->getType() . '" class="btn btn-success"  name="' . $element->getName()
+                . '"  id="' . $element->getName() . '" value="' . $element->getValue() . '"' . $element->getExtra()
+                . '  />';
 
         return $ret;
     }

Spacing +153 added lines, -153 removed lines

@@ -30,9 +30,9 @@ 
      */
     public function renderFormButton(XoopsFormButton $element)
     {
-        return "<input type='" . $element->getType() . "' class='btn btn-default' name='"
-            . $element->getName() . "'  id='" . $element->getName() . "' value='" . $element->getValue()
-            . "' title='" . $element->getValue() . "'" . $element->getExtra() . ' />';
+        return "<input type='".$element->getType()."' class='btn btn-default' name='"
+            . $element->getName()."'  id='".$element->getName()."' value='".$element->getValue()
+            . "' title='".$element->getValue()."'".$element->getExtra().' />';
     }
 
     /**
@@ -46,14 +46,14 @@ 
     {
         $ret = '';
         if ($element->_showDelete) {
-            $ret .= '<input type="submit" class="btn btn-danger" name="delete" id="delete" value="' . _DELETE
+            $ret .= '<input type="submit" class="btn btn-danger" name="delete" id="delete" value="'._DELETE
                 . '" onclick="this.form.elements.op.value=\'delete\'">&nbsp;';
         }
-        $ret .= '<input type="button" class="btn btn-danger" value="' . _CANCEL
+        $ret .= '<input type="button" class="btn btn-danger" value="'._CANCEL
              . '" onClick="history.go(-1);return true;" />&nbsp;'
-             . '<input type="reset" class="btn btn-warning"  name="reset"  id="reset" value="' . _RESET . '" />&nbsp;'
-             . '<input type="' . $element->getType() . '" class="btn btn-success"  name="' . $element->getName()
-             . '"  id="' . $element->getName() . '" value="' . $element->getValue() . '"' . $element->getExtra()
+             . '<input type="reset" class="btn btn-warning"  name="reset"  id="reset" value="'._RESET.'" />&nbsp;'
+             . '<input type="'.$element->getType().'" class="btn btn-success"  name="'.$element->getName()
+             . '"  id="'.$element->getName().'" value="'.$element->getValue().'"'.$element->getExtra()
              . '  />';
 
         return $ret;
@@ -76,7 +76,7 @@ 
             $element->setName($elementName);
         }
 
-        switch ((int) ($element->columns)) {
+        switch ((int)($element->columns)) {
             case 0:
                 return $this->renderCheckedInline($element, 'checkbox', $elementId, $elementName);
             case 1:
@@ -97,7 +97,7 @@ 
      */
     protected function renderCheckedInline($element, $type, $elementId, $elementName)
     {
-        $class = $type . '-inline';
+        $class = $type.'-inline';
         $ret = '';
 
         $idSuffix = 0;
@@ -105,15 +105,15 @@ 
         $elementOptions = $element->getOptions();
         foreach ($elementOptions as $value => $name) {
             ++$idSuffix;
-            $ret .= '<label class="' . $class . '">';
-            $ret .= "<input type='" . $type . "' name='{$elementName}' id='{$elementId}{$idSuffix}' title='"
-                . htmlspecialchars(strip_tags($name), ENT_QUOTES) . "' value='"
-                . htmlspecialchars($value, ENT_QUOTES) . "'";
+            $ret .= '<label class="'.$class.'">';
+            $ret .= "<input type='".$type."' name='{$elementName}' id='{$elementId}{$idSuffix}' title='"
+                . htmlspecialchars(strip_tags($name), ENT_QUOTES)."' value='"
+                . htmlspecialchars($value, ENT_QUOTES)."'";
 
-            if (is_array($elementValue) ? in_array($value, $elementValue): $value == $elementValue) {
+            if (is_array($elementValue) ? in_array($value, $elementValue) : $value == $elementValue) {
                 $ret .= ' checked';
             }
-            $ret .= $element->getExtra() . ' />' . $name . $element->getDelimeter();
+            $ret .= $element->getExtra().' />'.$name.$element->getDelimeter();
             $ret .= '</label>';
         }
 
@@ -139,16 +139,16 @@ 
         $elementOptions = $element->getOptions();
         foreach ($elementOptions as $value => $name) {
             ++$idSuffix;
-            $ret .= '<div class="' . $class . '">';
+            $ret .= '<div class="'.$class.'">';
             $ret .= '<label>';
-            $ret .= "<input type='" . $type . "' name='{$elementName}' id='{$elementId}{$idSuffix}' title='"
-                . htmlspecialchars(strip_tags($name), ENT_QUOTES) . "' value='"
-                . htmlspecialchars($value, ENT_QUOTES) . "'";
+            $ret .= "<input type='".$type."' name='{$elementName}' id='{$elementId}{$idSuffix}' title='"
+                . htmlspecialchars(strip_tags($name), ENT_QUOTES)."' value='"
+                . htmlspecialchars($value, ENT_QUOTES)."'";
 
-            if (is_array($elementValue) ? in_array($value, $elementValue): $value == $elementValue) {
+            if (is_array($elementValue) ? in_array($value, $elementValue) : $value == $elementValue) {
                 $ret .= ' checked';
             }
-            $ret .= $element->getExtra() . ' />' . $name . $element->getDelimeter();
+            $ret .= $element->getExtra().' />'.$name.$element->getDelimeter();
             $ret .= '</label>';
             $ret .= '</div>';
         }
@@ -175,16 +175,16 @@ 
         $elementOptions = $element->getOptions();
         foreach ($elementOptions as $value => $name) {
             ++$idSuffix;
-            $ret .= '<div class="' . $class . ' col-md-2">';
+            $ret .= '<div class="'.$class.' col-md-2">';
             $ret .= '<label>';
-            $ret .= "<input type='" . $type . "' name='{$elementName}' id='{$elementId}{$idSuffix}' title='"
-                . htmlspecialchars(strip_tags($name), ENT_QUOTES) . "' value='"
-                . htmlspecialchars($value, ENT_QUOTES) . "'";
+            $ret .= "<input type='".$type."' name='{$elementName}' id='{$elementId}{$idSuffix}' title='"
+                . htmlspecialchars(strip_tags($name), ENT_QUOTES)."' value='"
+                . htmlspecialchars($value, ENT_QUOTES)."'";
 
-            if (is_array($elementValue) ? in_array($value, $elementValue): $value == $elementValue) {
+            if (is_array($elementValue) ? in_array($value, $elementValue) : $value == $elementValue) {
                 $ret .= ' checked';
             }
-            $ret .= $element->getExtra() . ' />' . $name . $element->getDelimeter();
+            $ret .= $element->getExtra().' />'.$name.$element->getDelimeter();
             $ret .= '</label>';
             $ret .= '</div>';
         }
@@ -204,12 +204,12 @@ 
             $GLOBALS['xoTheme']->addScript('include/spectrum.js');
             $GLOBALS['xoTheme']->addStylesheet('include/spectrum.css');
         } else {
-            echo '<script type="text/javascript" src="' . XOOPS_URL . '/include/spectrum.js"></script>';
-            echo '<link rel="stylesheet" type="text/css" href="' . XOOPS_URL . '/include/spectrum.css">';
+            echo '<script type="text/javascript" src="'.XOOPS_URL.'/include/spectrum.js"></script>';
+            echo '<link rel="stylesheet" type="text/css" href="'.XOOPS_URL.'/include/spectrum.css">';
         }
-        return '<input class="form-control" style="width: 25%;" type="color" name="' . $element->getName()
-            . "' title='" . $element->getTitle() . "' id='" . $element->getName()
-            . '" size="7" maxlength="7" value="' . $element->getValue() . '"' . $element->getExtra() . ' />';
+        return '<input class="form-control" style="width: 25%;" type="color" name="'.$element->getName()
+            . "' title='".$element->getTitle()."' id='".$element->getName()
+            . '" size="7" maxlength="7" value="'.$element->getValue().'"'.$element->getExtra().' />';
     }
 
     /**
@@ -226,43 +226,43 @@ 
         xoops_loadLanguage('formdhtmltextarea');
         $ret = '';
         // actions
-        $ret .= $this->renderFormDhtmlTAXoopsCode($element) . "<br>\n";
+        $ret .= $this->renderFormDhtmlTAXoopsCode($element)."<br>\n";
         // fonts
         $ret .= $this->renderFormDhtmlTATypography($element);
         // length checker
 
         $ret .= "<br>\n";
         // the textarea box
-        $ret .= "<textarea class='form-control' id='" . $element->getName() . "' name='" . $element->getName()
-            . "' title='" . $element->getTitle() . "' onselect=\"xoopsSavePosition('" . $element->getName()
-            . "');\" onclick=\"xoopsSavePosition('" . $element->getName()
-            . "');\" onkeyup=\"xoopsSavePosition('" . $element->getName() . "');\" cols='"
-            . $element->getCols() . "' rows='" . $element->getRows() . "'" . $element->getExtra()
-            . '>' . $element->getValue() . "</textarea>\n";
+        $ret .= "<textarea class='form-control' id='".$element->getName()."' name='".$element->getName()
+            . "' title='".$element->getTitle()."' onselect=\"xoopsSavePosition('".$element->getName()
+            . "');\" onclick=\"xoopsSavePosition('".$element->getName()
+            . "');\" onkeyup=\"xoopsSavePosition('".$element->getName()."');\" cols='"
+            . $element->getCols()."' rows='".$element->getRows()."'".$element->getExtra()
+            . '>'.$element->getValue()."</textarea>\n";
 
         if (empty($element->skipPreview)) {
             if (empty($GLOBALS['xoTheme'])) {
-                $element->js .= implode('', file(XOOPS_ROOT_PATH . '/class/textsanitizer/image/image.js'));
+                $element->js .= implode('', file(XOOPS_ROOT_PATH.'/class/textsanitizer/image/image.js'));
             } else {
                 $GLOBALS['xoTheme']->addScript(
                     '/class/textsanitizer/image/image.js',
                     array('type' => 'text/javascript')
                 );
             }
-            $button = "<button type='button' class='btn btn-primary' onclick=\"form_instantPreview('" . XOOPS_URL
-                . "', '" . $element->getName() . "','" . XOOPS_URL . "/images', " . (int)$element->doHtml . ", '"
-                . $GLOBALS['xoopsSecurity']->createToken() . "')\" title='" . _PREVIEW . "'>" . _PREVIEW . "</button>";
-
-            $ret .= '<br>' . "<div id='" . $element->getName() . "_hidden' style='display: block;'> "
-                . '   <fieldset>' . '       <legend>' . $button . '</legend>'
-                . "       <div id='" . $element->getName() . "_hidden_data'>" . _XOOPS_FORM_PREVIEW_CONTENT
-                . '</div>' . '   </fieldset>' . '</div>';
+            $button = "<button type='button' class='btn btn-primary' onclick=\"form_instantPreview('".XOOPS_URL
+                . "', '".$element->getName()."','".XOOPS_URL."/images', ".(int)$element->doHtml.", '"
+                . $GLOBALS['xoopsSecurity']->createToken()."')\" title='"._PREVIEW."'>"._PREVIEW."</button>";
+
+            $ret .= '<br>'."<div id='".$element->getName()."_hidden' style='display: block;'> "
+                . '   <fieldset>'.'       <legend>'.$button.'</legend>'
+                . "       <div id='".$element->getName()."_hidden_data'>"._XOOPS_FORM_PREVIEW_CONTENT
+                . '</div>'.'   </fieldset>'.'</div>';
         }
         // Load javascript
         if (empty($js_loaded)) {
-            $javascript = ($element->js ? '<script type="text/javascript">' . $element->js . '</script>' : '')
-                . '<script type="text/javascript" src="' . XOOPS_URL . '/include/formdhtmltextarea.js"></script>';
-            $ret        = $javascript . $ret;
+            $javascript = ($element->js ? '<script type="text/javascript">'.$element->js.'</script>' : '')
+                . '<script type="text/javascript" src="'.XOOPS_URL.'/include/formdhtmltextarea.js"></script>';
+            $ret        = $javascript.$ret;
             $js_loaded  = true;
         }
 
@@ -281,11 +281,11 @@ 
         $textarea_id = $element->getName();
         $code = '';
         $code .= "<div class='row'><div class='col-md-12'>";
-        $code .= "<button type='button' class='btn btn-default btn-sm' onclick='xoopsCodeUrl(\"{$textarea_id}\", \"" . htmlspecialchars(_ENTERURL, ENT_QUOTES) . "\", \"" . htmlspecialchars(_ENTERWEBTITLE, ENT_QUOTES) . "\");' onmouseover='style.cursor=\"hand\"' title='" . _XOOPS_FORM_ALT_URL . "'><span class='fa fa-fw fa-link' aria-hidden='true'></span></button>";
-        $code .= "<button type='button' class='btn btn-default btn-sm' onclick='xoopsCodeEmail(\"{$textarea_id}\", \"" . htmlspecialchars(_ENTEREMAIL, ENT_QUOTES) . "\", \"" . htmlspecialchars(_ENTERWEBTITLE, ENT_QUOTES) . "\");' onmouseover='style.cursor=\"hand\"' title='" . _XOOPS_FORM_ALT_EMAIL . "'><span class='fa fa-fw fa-envelope-o' aria-hidden='true'></span></button>";
-        $code .= "<button type='button' class='btn btn-default btn-sm' onclick='xoopsCodeImg(\"{$textarea_id}\", \"" . htmlspecialchars(_ENTERIMGURL, ENT_QUOTES) . "\", \"" . htmlspecialchars(_ENTERIMGPOS, ENT_QUOTES) . "\", \"" . htmlspecialchars(_IMGPOSRORL, ENT_QUOTES) . "\", \"" . htmlspecialchars(_ERRORIMGPOS, ENT_QUOTES) . "\", \"" . htmlspecialchars(_XOOPS_FORM_ALT_ENTERWIDTH, ENT_QUOTES) . "\");' onmouseover='style.cursor=\"hand\"' title='" . _XOOPS_FORM_ALT_IMG . "'><span class='fa fa-fw fa-file-image-o' aria-hidden='true'></span></button>";
-        $code .= "<button type='button' class='btn btn-default btn-sm' onclick='openWithSelfMain(\"" . XOOPS_URL . "/imagemanager.php?target={$textarea_id}\",\"imgmanager\",400,430);' onmouseover='style.cursor=\"hand\"' title='" . _XOOPS_FORM_ALT_IMAGE . "'><span class='fa fa-file-image-o' aria-hidden='true'></span><small> Manager</small></button>";
-        $code .= "<button type='button' class='btn btn-default btn-sm' onclick='openWithSelfMain(\"" . XOOPS_URL . "/misc.php?action=showpopups&amp;type=smilies&amp;target={$textarea_id}\",\"smilies\",300,475);' onmouseover='style.cursor=\"hand\"' title='" . _XOOPS_FORM_ALT_SMILEY . "'><span class='fa fa-fw fa-smile-o' aria-hidden='true'></span></button>";
+        $code .= "<button type='button' class='btn btn-default btn-sm' onclick='xoopsCodeUrl(\"{$textarea_id}\", \"".htmlspecialchars(_ENTERURL, ENT_QUOTES)."\", \"".htmlspecialchars(_ENTERWEBTITLE, ENT_QUOTES)."\");' onmouseover='style.cursor=\"hand\"' title='"._XOOPS_FORM_ALT_URL."'><span class='fa fa-fw fa-link' aria-hidden='true'></span></button>";
+        $code .= "<button type='button' class='btn btn-default btn-sm' onclick='xoopsCodeEmail(\"{$textarea_id}\", \"".htmlspecialchars(_ENTEREMAIL, ENT_QUOTES)."\", \"".htmlspecialchars(_ENTERWEBTITLE, ENT_QUOTES)."\");' onmouseover='style.cursor=\"hand\"' title='"._XOOPS_FORM_ALT_EMAIL."'><span class='fa fa-fw fa-envelope-o' aria-hidden='true'></span></button>";
+        $code .= "<button type='button' class='btn btn-default btn-sm' onclick='xoopsCodeImg(\"{$textarea_id}\", \"".htmlspecialchars(_ENTERIMGURL, ENT_QUOTES)."\", \"".htmlspecialchars(_ENTERIMGPOS, ENT_QUOTES)."\", \"".htmlspecialchars(_IMGPOSRORL, ENT_QUOTES)."\", \"".htmlspecialchars(_ERRORIMGPOS, ENT_QUOTES)."\", \"".htmlspecialchars(_XOOPS_FORM_ALT_ENTERWIDTH, ENT_QUOTES)."\");' onmouseover='style.cursor=\"hand\"' title='"._XOOPS_FORM_ALT_IMG."'><span class='fa fa-fw fa-file-image-o' aria-hidden='true'></span></button>";
+        $code .= "<button type='button' class='btn btn-default btn-sm' onclick='openWithSelfMain(\"".XOOPS_URL."/imagemanager.php?target={$textarea_id}\",\"imgmanager\",400,430);' onmouseover='style.cursor=\"hand\"' title='"._XOOPS_FORM_ALT_IMAGE."'><span class='fa fa-file-image-o' aria-hidden='true'></span><small> Manager</small></button>";
+        $code .= "<button type='button' class='btn btn-default btn-sm' onclick='openWithSelfMain(\"".XOOPS_URL."/misc.php?action=showpopups&amp;type=smilies&amp;target={$textarea_id}\",\"smilies\",300,475);' onmouseover='style.cursor=\"hand\"' title='"._XOOPS_FORM_ALT_SMILEY."'><span class='fa fa-fw fa-smile-o' aria-hidden='true'></span></button>";
 
         $myts = MyTextSanitizer::getInstance();
 
@@ -301,8 +301,8 @@ 
                 $element->js .= $js;
             }
         }
-        $code .= "<button type='button' class='btn btn-default btn-sm' onclick='xoopsCodeCode(\"{$textarea_id}\", \"" . htmlspecialchars(_ENTERCODE, ENT_QUOTES) . "\");' onmouseover='style.cursor=\"hand\"' title='" . _XOOPS_FORM_ALT_CODE . "'><span class='fa fa-fw fa-code' aria-hidden='true'></span></button>";
-        $code .= "<button type='button' class='btn btn-default btn-sm' onclick='xoopsCodeQuote(\"{$textarea_id}\", \"" . htmlspecialchars(_ENTERQUOTE, ENT_QUOTES) . "\");' onmouseover='style.cursor=\"hand\"' title='" . _XOOPS_FORM_ALT_QUOTE . "'><span class='fa fa-fw fa-quote-right' aria-hidden='true'></span></button>";
+        $code .= "<button type='button' class='btn btn-default btn-sm' onclick='xoopsCodeCode(\"{$textarea_id}\", \"".htmlspecialchars(_ENTERCODE, ENT_QUOTES)."\");' onmouseover='style.cursor=\"hand\"' title='"._XOOPS_FORM_ALT_CODE."'><span class='fa fa-fw fa-code' aria-hidden='true'></span></button>";
+        $code .= "<button type='button' class='btn btn-default btn-sm' onclick='xoopsCodeQuote(\"{$textarea_id}\", \"".htmlspecialchars(_ENTERQUOTE, ENT_QUOTES)."\");' onmouseover='style.cursor=\"hand\"' title='"._XOOPS_FORM_ALT_QUOTE."'><span class='fa fa-fw fa-quote-right' aria-hidden='true'></span></button>";
         $code .= "</div></div>";
 
         $xoopsPreload = XoopsPreload::getInstance();
@@ -348,63 +348,63 @@ 
 
         $fontStr = '<div class="row"><div class="col-md-12"><div class="btn-group" role="toolbar">';
         $fontStr .= '<div class="btn-group">'
-            . '<button type="button" class="btn btn-default btn-sm dropdown-toggle" title="'. _SIZE .'"'
+            . '<button type="button" class="btn btn-default btn-sm dropdown-toggle" title="'._SIZE.'"'
             . ' data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">'
             . '<span class = "glyphicon glyphicon-text-height"></span><span class="caret"></span></button>'
             . '<ul class="dropdown-menu">';
             //. _SIZE . '&nbsp;&nbsp;<span class="caret"></span></button><ul class="dropdown-menu">';
         foreach ($GLOBALS['formtextdhtml_sizes'] as $value => $name) {
-            $fontStr .= '<li><a href="javascript:xoopsSetElementAttribute(\'size\', \'' . $value . '\', \''
-                . $textarea_id . '\', \'' . $hiddentext . '\');">' . $name . '</a></li>';
+            $fontStr .= '<li><a href="javascript:xoopsSetElementAttribute(\'size\', \''.$value.'\', \''
+                . $textarea_id.'\', \''.$hiddentext.'\');">'.$name.'</a></li>';
         }
         $fontStr .= '</ul></div>';
 
         $fontStr .= '<div class="btn-group">'
-            . '<button type="button" class="btn btn-default btn-sm dropdown-toggle" title="'. _FONT .'"'
+            . '<button type="button" class="btn btn-default btn-sm dropdown-toggle" title="'._FONT.'"'
             . ' data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">'
             . '<span class = "glyphicon glyphicon-font"></span><span class="caret"></span></button>'
             . '<ul class="dropdown-menu">';
             //. _FONT . '&nbsp;&nbsp;<span class="caret"></span></button><ul class="dropdown-menu">';
         foreach ($fontarray as $font) {
-            $fontStr .= '<li><a href="javascript:xoopsSetElementAttribute(\'font\', \'' . $font . '\', \''
-                . $textarea_id . '\', \'' . $hiddentext . '\');">' . $font . '</a></li>';
+            $fontStr .= '<li><a href="javascript:xoopsSetElementAttribute(\'font\', \''.$font.'\', \''
+                . $textarea_id.'\', \''.$hiddentext.'\');">'.$font.'</a></li>';
         }
         $fontStr .= '</ul></div>';
 
         $fontStr .= '<div class="btn-group">'
-            . '<button type="button" class="btn btn-default btn-sm dropdown-toggle" title="'. _COLOR .'"'
+            . '<button type="button" class="btn btn-default btn-sm dropdown-toggle" title="'._COLOR.'"'
             . ' data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">'
             . '<span class = "glyphicon glyphicon-text-color"></span><span class="caret"></span></button>'
             . '<ul class="dropdown-menu">';
             //. _COLOR . '&nbsp;&nbsp;<span class="caret"></span></button><ul class="dropdown-menu">';
         foreach ($colorArray as $color => $hex) {
-            $fontStr .= '<li><a href="javascript:xoopsSetElementAttribute(\'color\', \'' . $hex . '\', \''
-                . $textarea_id . '\', \'' . $hiddentext . '\');">'
-                . '<span style="color:#' . $hex . ';">' . $color .'</span></a></li>';
+            $fontStr .= '<li><a href="javascript:xoopsSetElementAttribute(\'color\', \''.$hex.'\', \''
+                . $textarea_id.'\', \''.$hiddentext.'\');">'
+                . '<span style="color:#'.$hex.';">'.$color.'</span></a></li>';
         }
         $fontStr .= '</ul></div>';
         $fontStr .= '</div>';
 
         //$styleStr = "<div class='row'><div class='col-md-12'>";
         $styleStr  = "<div class='btn-group' role='group'>";
-        $styleStr .= "<button type='button' class='btn btn-default btn-sm' onclick='xoopsMakeBold(\"{$hiddentext}\", \"{$textarea_id}\");' title='" . _XOOPS_FORM_ALT_BOLD . "' aria-label='Left Align'><span class='fa fa-bold' aria-hidden='true'></span></button>";
-        $styleStr .= "<button type='button' class='btn btn-default btn-sm' onclick='xoopsMakeItalic(\"{$hiddentext}\", \"{$textarea_id}\");' title='" . _XOOPS_FORM_ALT_ITALIC . "' aria-label='Left Align'><span class='fa fa-italic' aria-hidden='true'></span></button>";
-        $styleStr .= "<button type='button' class='btn btn-default btn-sm' onclick='xoopsMakeUnderline(\"{$hiddentext}\", \"{$textarea_id}\");' title='" . _XOOPS_FORM_ALT_UNDERLINE . "' aria-label='Left Align'>" . '<span class="fa fa-underline"></span></button>';
-        $styleStr .= "<button type='button' class='btn btn-default btn-sm' onclick='xoopsMakeLineThrough(\"{$hiddentext}\", \"{$textarea_id}\");' title='" . _XOOPS_FORM_ALT_LINETHROUGH . "' aria-label='Left Align'>" . '<span class="fa fa-strikethrough"></span></button>';
+        $styleStr .= "<button type='button' class='btn btn-default btn-sm' onclick='xoopsMakeBold(\"{$hiddentext}\", \"{$textarea_id}\");' title='"._XOOPS_FORM_ALT_BOLD."' aria-label='Left Align'><span class='fa fa-bold' aria-hidden='true'></span></button>";
+        $styleStr .= "<button type='button' class='btn btn-default btn-sm' onclick='xoopsMakeItalic(\"{$hiddentext}\", \"{$textarea_id}\");' title='"._XOOPS_FORM_ALT_ITALIC."' aria-label='Left Align'><span class='fa fa-italic' aria-hidden='true'></span></button>";
+        $styleStr .= "<button type='button' class='btn btn-default btn-sm' onclick='xoopsMakeUnderline(\"{$hiddentext}\", \"{$textarea_id}\");' title='"._XOOPS_FORM_ALT_UNDERLINE."' aria-label='Left Align'>".'<span class="fa fa-underline"></span></button>';
+        $styleStr .= "<button type='button' class='btn btn-default btn-sm' onclick='xoopsMakeLineThrough(\"{$hiddentext}\", \"{$textarea_id}\");' title='"._XOOPS_FORM_ALT_LINETHROUGH."' aria-label='Left Align'>".'<span class="fa fa-strikethrough"></span></button>';
         $styleStr .= "</div>";
 
         $alignStr = "<div class='btn-group' role='group'>";
-        $alignStr .= "<button type='button' class='btn btn-default btn-sm' onclick='xoopsMakeLeft(\"{$hiddentext}\", \"{$textarea_id}\");' title='" . _XOOPS_FORM_ALT_LEFT . "' aria-label='Left Align'><span class='fa fa-align-left' aria-hidden='true'></span></button>";
-        $alignStr .= "<button type='button' class='btn btn-default btn-sm' onclick='xoopsMakeCenter(\"{$hiddentext}\", \"{$textarea_id}\");' title='" . _XOOPS_FORM_ALT_CENTER . "' aria-label='Left Align'><span class='fa fa-align-center' aria-hidden='true'></span></button>";
-        $alignStr .= "<button type='button' class='btn btn-default btn-sm' onclick='xoopsMakeRight(\"{$hiddentext}\", \"{$textarea_id}\");' title='" . _XOOPS_FORM_ALT_RIGHT . "' aria-label='Left Align'><span class='fa fa-align-right' aria-hidden='true'></span></button>";
+        $alignStr .= "<button type='button' class='btn btn-default btn-sm' onclick='xoopsMakeLeft(\"{$hiddentext}\", \"{$textarea_id}\");' title='"._XOOPS_FORM_ALT_LEFT."' aria-label='Left Align'><span class='fa fa-align-left' aria-hidden='true'></span></button>";
+        $alignStr .= "<button type='button' class='btn btn-default btn-sm' onclick='xoopsMakeCenter(\"{$hiddentext}\", \"{$textarea_id}\");' title='"._XOOPS_FORM_ALT_CENTER."' aria-label='Left Align'><span class='fa fa-align-center' aria-hidden='true'></span></button>";
+        $alignStr .= "<button type='button' class='btn btn-default btn-sm' onclick='xoopsMakeRight(\"{$hiddentext}\", \"{$textarea_id}\");' title='"._XOOPS_FORM_ALT_RIGHT."' aria-label='Left Align'><span class='fa fa-align-right' aria-hidden='true'></span></button>";
         $alignStr .= "</div>";
 
         $fontStr .= "&nbsp;{$styleStr}&nbsp;{$alignStr}&nbsp;\n";
 
         $fontStr .= "<button type='button' class='btn btn-default btn-sm' onclick=\"XoopsCheckLength('"
-            . $element->getName() . "', '" . @$element->configs['maxlength'] . "', '"
-            . _XOOPS_FORM_ALT_LENGTH . "', '" . _XOOPS_FORM_ALT_LENGTH_MAX . "');\" title='"
-            . _XOOPS_FORM_ALT_CHECKLENGTH . "'><span class='fa fa-check-square-o' aria-hidden='true'></span></button>";
+            . $element->getName()."', '".@$element->configs['maxlength']."', '"
+            . _XOOPS_FORM_ALT_LENGTH."', '"._XOOPS_FORM_ALT_LENGTH_MAX."');\" title='"
+            . _XOOPS_FORM_ALT_CHECKLENGTH."'><span class='fa fa-check-square-o' aria-hidden='true'></span></button>";
         $fontStr .= "</div></div>";
 
         return $fontStr;
@@ -426,9 +426,9 @@ 
                 $ret .= $element->getDelimeter();
             }
             if ($ele->getCaption() != '') {
-                $ret .= $ele->getCaption() . '&nbsp;';
+                $ret .= $ele->getCaption().'&nbsp;';
             }
-            $ret .= $ele->render() . NWLINE;
+            $ret .= $ele->render().NWLINE;
             if (!$ele->isHidden()) {
                 ++$count;
             }
@@ -455,12 +455,12 @@ 
      */
     public function renderFormFile(XoopsFormFile $element)
     {
-        return '<input type="hidden" name="MAX_FILE_SIZE" value="' . $element->getMaxFileSize() . '" />'
-            . '<input class="form-control-static" type="file" name="' . $element->getName()
-            . '" id="' . $element->getName()
-            . '" title="' . $element->getTitle() . '" ' . $element->getExtra() . ' />'
+        return '<input type="hidden" name="MAX_FILE_SIZE" value="'.$element->getMaxFileSize().'" />'
+            . '<input class="form-control-static" type="file" name="'.$element->getName()
+            . '" id="'.$element->getName()
+            . '" title="'.$element->getTitle().'" '.$element->getExtra().' />'
             . '<input type="hidden" name="xoops_upload_file[]" id="xoops_upload_file[]" value="'
-            . $element->getName() . '" />';
+            . $element->getName().'" />';
     }
 
     /**
@@ -472,7 +472,7 @@ 
      */
     public function renderFormLabel(XoopsFormLabel $element)
     {
-        return '<div class="form-control-static">' . $element->getValue() . '</div>';
+        return '<div class="form-control-static">'.$element->getValue().'</div>';
     }
 
     /**
@@ -485,9 +485,9 @@ 
     public function renderFormPassword(XoopsFormPassword $element)
     {
         return '<input class="form-control" type="password" name="'
-            . $element->getName() . '" id="' . $element->getName() . '" size="' . $element->getSize()
-            . '" maxlength="' . $element->getMaxlength() . '" value="' . $element->getValue() . '"'
-            . $element->getExtra() . ' ' . ($element->autoComplete ? '' : 'autocomplete="off" ') . '/>';
+            . $element->getName().'" id="'.$element->getName().'" size="'.$element->getSize()
+            . '" maxlength="'.$element->getMaxlength().'" value="'.$element->getValue().'"'
+            . $element->getExtra().' '.($element->autoComplete ? '' : 'autocomplete="off" ').'/>';
     }
 
     /**
@@ -503,7 +503,7 @@ 
         $elementName = $element->getName();
         $elementId = $elementName;
 
-        switch ((int) ($element->columns)) {
+        switch ((int)($element->columns)) {
             case 0:
                 return $this->renderCheckedInline($element, 'radio', $elementId, $elementName);
             case 1:
@@ -527,19 +527,19 @@ 
         $ele_value   = $element->getValue();
         $ele_options = $element->getOptions();
         $ret = '<select class="form-control" size="'
-            . $element->getSize() . '"' . $element->getExtra();
+            . $element->getSize().'"'.$element->getExtra();
         if ($element->isMultiple() != false) {
-            $ret .= ' name="' . $ele_name . '[]" id="' . $ele_name . '" title="' . $ele_title
+            $ret .= ' name="'.$ele_name.'[]" id="'.$ele_name.'" title="'.$ele_title
                 . '" multiple="multiple">';
         } else {
-            $ret .= ' name="' . $ele_name . '" id="' . $ele_name . '" title="' . $ele_title . '">';
+            $ret .= ' name="'.$ele_name.'" id="'.$ele_name.'" title="'.$ele_title.'">';
         }
         foreach ($ele_options as $value => $name) {
-            $ret .= '<option value="' . htmlspecialchars($value, ENT_QUOTES) . '"';
+            $ret .= '<option value="'.htmlspecialchars($value, ENT_QUOTES).'"';
             if (count($ele_value) > 0 && in_array($value, $ele_value)) {
                 $ret .= ' selected';
             }
-            $ret .= '>' . $name . '</option>';
+            $ret .= '>'.$name.'</option>';
         }
         $ret .= '</select>';
 
@@ -555,9 +555,9 @@ 
     public function renderFormText(XoopsFormText $element)
     {
         return "<input class='form-control' type='text' name='"
-            . $element->getName() . "' title='" . $element->getTitle() . "' id='" . $element->getName()
-            . "' size='" . $element->getSize() . "' maxlength='" . $element->getMaxlength()
-            . "' value='" . $element->getValue() . "'" . $element->getExtra() . ' />';
+            . $element->getName()."' title='".$element->getTitle()."' id='".$element->getName()
+            . "' size='".$element->getSize()."' maxlength='".$element->getMaxlength()
+            . "' value='".$element->getValue()."'".$element->getExtra().' />';
     }
 
     /**
@@ -570,9 +570,9 @@ 
     public function renderFormTextArea(XoopsFormTextArea $element)
     {
         return "<textarea class='form-control' name='"
-            . $element->getName() . "' id='" . $element->getName() . "'  title='" . $element->getTitle()
-            . "' rows='" . $element->getRows() . "' cols='" . $element->getCols() . "'"
-            . $element->getExtra() . '>' . $element->getValue() . '</textarea>';
+            . $element->getName()."' id='".$element->getName()."'  title='".$element->getTitle()
+            . "' rows='".$element->getRows()."' cols='".$element->getCols()."'"
+            . $element->getExtra().'>'.$element->getValue().'</textarea>';
     }
 
     /**
@@ -585,10 +585,10 @@ 
     public function renderFormTextDateSelect(XoopsFormTextDateSelect $element)
     {
         static $included = false;
-        if (file_exists(XOOPS_ROOT_PATH . '/language/' . $GLOBALS['xoopsConfig']['language'] . '/calendar.php')) {
-            include_once XOOPS_ROOT_PATH . '/language/' . $GLOBALS['xoopsConfig']['language'] . '/calendar.php';
+        if (file_exists(XOOPS_ROOT_PATH.'/language/'.$GLOBALS['xoopsConfig']['language'].'/calendar.php')) {
+            include_once XOOPS_ROOT_PATH.'/language/'.$GLOBALS['xoopsConfig']['language'].'/calendar.php';
         } else {
-            include_once XOOPS_ROOT_PATH . '/language/english/calendar.php';
+            include_once XOOPS_ROOT_PATH.'/language/english/calendar.php';
         }
 
         $ele_name  = $element->getName();
@@ -635,7 +635,7 @@ 
                     if (calendar != null) {
                     calendar.hide();
                     } else {
-                    var cal = new Calendar(true, "' . $jstime . '", selected, closeHandler);
+                    var cal = new Calendar(true, "' . $jstime.'", selected, closeHandler);
                     calendar = cal;
                     cal.setRange(1900, 2100);
                     calendar.create();
@@ -649,57 +649,57 @@ 
                     }
 
                     Calendar._DN = new Array
-                    ("' . _CAL_SUNDAY . '",
-                    "' . _CAL_MONDAY . '",
-                    "' . _CAL_TUESDAY . '",
-                    "' . _CAL_WEDNESDAY . '",
-                    "' . _CAL_THURSDAY . '",
-                    "' . _CAL_FRIDAY . '",
-                    "' . _CAL_SATURDAY . '",
-                    "' . _CAL_SUNDAY . '");
+                    ("' . _CAL_SUNDAY.'",
+                    "' . _CAL_MONDAY.'",
+                    "' . _CAL_TUESDAY.'",
+                    "' . _CAL_WEDNESDAY.'",
+                    "' . _CAL_THURSDAY.'",
+                    "' . _CAL_FRIDAY.'",
+                    "' . _CAL_SATURDAY.'",
+                    "' . _CAL_SUNDAY.'");
                     Calendar._MN = new Array
-                    ("' . _CAL_JANUARY . '",
-                    "' . _CAL_FEBRUARY . '",
-                    "' . _CAL_MARCH . '",
-                    "' . _CAL_APRIL . '",
-                    "' . _CAL_MAY . '",
-                    "' . _CAL_JUNE . '",
-                    "' . _CAL_JULY . '",
-                    "' . _CAL_AUGUST . '",
-                    "' . _CAL_SEPTEMBER . '",
-                    "' . _CAL_OCTOBER . '",
-                    "' . _CAL_NOVEMBER . '",
-                    "' . _CAL_DECEMBER . '");
+                    ("' . _CAL_JANUARY.'",
+                    "' . _CAL_FEBRUARY.'",
+                    "' . _CAL_MARCH.'",
+                    "' . _CAL_APRIL.'",
+                    "' . _CAL_MAY.'",
+                    "' . _CAL_JUNE.'",
+                    "' . _CAL_JULY.'",
+                    "' . _CAL_AUGUST.'",
+                    "' . _CAL_SEPTEMBER.'",
+                    "' . _CAL_OCTOBER.'",
+                    "' . _CAL_NOVEMBER.'",
+                    "' . _CAL_DECEMBER.'");
 
                     Calendar._TT = {};
-                    Calendar._TT["TOGGLE"] = "' . _CAL_TGL1STD . '";
-                    Calendar._TT["PREV_YEAR"] = "' . _CAL_PREVYR . '";
-                    Calendar._TT["PREV_MONTH"] = "' . _CAL_PREVMNTH . '";
-                    Calendar._TT["GO_TODAY"] = "' . _CAL_GOTODAY . '";
-                    Calendar._TT["NEXT_MONTH"] = "' . _CAL_NXTMNTH . '";
-                    Calendar._TT["NEXT_YEAR"] = "' . _CAL_NEXTYR . '";
-                    Calendar._TT["SEL_DATE"] = "' . _CAL_SELDATE . '";
-                    Calendar._TT["DRAG_TO_MOVE"] = "' . _CAL_DRAGMOVE . '";
-                    Calendar._TT["PART_TODAY"] = "(' . _CAL_TODAY . ')";
-                    Calendar._TT["MON_FIRST"] = "' . _CAL_DISPM1ST . '";
-                    Calendar._TT["SUN_FIRST"] = "' . _CAL_DISPS1ST . '";
-                    Calendar._TT["CLOSE"] = "' . _CLOSE . '";
-                    Calendar._TT["TODAY"] = "' . _CAL_TODAY . '";
+                    Calendar._TT["TOGGLE"] = "' . _CAL_TGL1STD.'";
+                    Calendar._TT["PREV_YEAR"] = "' . _CAL_PREVYR.'";
+                    Calendar._TT["PREV_MONTH"] = "' . _CAL_PREVMNTH.'";
+                    Calendar._TT["GO_TODAY"] = "' . _CAL_GOTODAY.'";
+                    Calendar._TT["NEXT_MONTH"] = "' . _CAL_NXTMNTH.'";
+                    Calendar._TT["NEXT_YEAR"] = "' . _CAL_NEXTYR.'";
+                    Calendar._TT["SEL_DATE"] = "' . _CAL_SELDATE.'";
+                    Calendar._TT["DRAG_TO_MOVE"] = "' . _CAL_DRAGMOVE.'";
+                    Calendar._TT["PART_TODAY"] = "(' . _CAL_TODAY.')";
+                    Calendar._TT["MON_FIRST"] = "' . _CAL_DISPM1ST.'";
+                    Calendar._TT["SUN_FIRST"] = "' . _CAL_DISPS1ST.'";
+                    Calendar._TT["CLOSE"] = "' . _CLOSE.'";
+                    Calendar._TT["TODAY"] = "' . _CAL_TODAY.'";
 
                     // date formats
-                    Calendar._TT["DEF_DATE_FORMAT"] = "' . _SHORTDATESTRING . '";
-                    Calendar._TT["TT_DATE_FORMAT"] = "' . _SHORTDATESTRING . '";
+                    Calendar._TT["DEF_DATE_FORMAT"] = "' . _SHORTDATESTRING.'";
+                    Calendar._TT["TT_DATE_FORMAT"] = "' . _SHORTDATESTRING.'";
 
                     Calendar._TT["WK"] = "";
                 ');
             }
         }
         return '<div class="input-group">'
-            . '<input class="form-control" type="text" name="' . $ele_name . '" id="' . $ele_name
-            . '" size="' . $element->getSize() . '" maxlength="' . $element->getMaxlength()
-            . '" value="' . $display_value . '"' . $element->getExtra() . ' />'
+            . '<input class="form-control" type="text" name="'.$ele_name.'" id="'.$ele_name
+            . '" size="'.$element->getSize().'" maxlength="'.$element->getMaxlength()
+            . '" value="'.$display_value.'"'.$element->getExtra().' />'
             . '<span class="input-group-btn"><button class="btn btn-default" type="button"'
-            . ' onclick="return showCalendar(\'' . $ele_name . '\');">'
+            . ' onclick="return showCalendar(\''.$ele_name.'\');">'
             . '<span class="fa fa-calendar" aria-hidden="true"></span></button>'
             . '</span>'
             . '</div>';
@@ -717,11 +717,11 @@ 
         $ele_name = $form->getName();
 
         $ret = '<div>';
-        $ret .= '<form class="form-horizontal" name="' . $ele_name . '" id="' . $ele_name . '" action="'
-            . $form->getAction() . '" method="' . $form->getMethod()
-            . '" onsubmit="return xoopsFormValidate_' . $ele_name . '();"' . $form->getExtra() . '>'
-            . '<h3>' . $form->getTitle() . '</h3>';
-        $hidden   = '';
+        $ret .= '<form class="form-horizontal" name="'.$ele_name.'" id="'.$ele_name.'" action="'
+            . $form->getAction().'" method="'.$form->getMethod()
+            . '" onsubmit="return xoopsFormValidate_'.$ele_name.'();"'.$form->getExtra().'>'
+            . '<h3>'.$form->getTitle().'</h3>';
+        $hidden = '';
 
         foreach ($form->getElements() as $element) {
             if (!is_object($element)) { // see $form->addBreak()
@@ -735,7 +735,7 @@ 
 
             $ret .= '<div class="form-group">';
             if (($caption = $element->getCaption()) != '') {
-                $ret .= '<label for="' . $element->getName() . '" class="col-md-2 control-label">'
+                $ret .= '<label for="'.$element->getName().'" class="col-md-2 control-label">'
                     . $element->getCaption()
                     . ($element->isRequired() ? '<span class="caption-required">*</span>' : '')
                     . '</label>';
@@ -745,7 +745,7 @@ 
             $ret .= '<div class="col-md-10">';
             $ret .= $element->render();
             if (($desc = $element->getDescription()) != '') {
-                $ret .= '<p class="help-block">' . $desc . '</p>';
+                $ret .= '<p class="help-block">'.$desc.'</p>';
             }
             $ret .= '</div>';
             $ret .= '</div>';
@@ -769,6 +769,6 @@ 
     public function addThemeFormBreak(XoopsThemeForm $form, $extra, $class)
     {
         $class = ($class != '') ? preg_replace('/[^A-Za-z0-9\s\s_-]/i', '', $class) : '';
-        $form->addElement('<div class="col-sm-12 ' . $class .'">'. $extra . '</div>');
+        $form->addElement('<div class="col-sm-12 '.$class.'">'.$extra.'</div>');
     }
 }

htdocs/class/xoopsform/renderer/XoopsFormRendererLegacy.php

Spacing +126 added lines, -126 removed lines

@@ -30,9 +30,9 @@ 
      */
     public function renderFormButton(XoopsFormButton $element)
     {
-        return "<input type='" . $element->getType() . "' class='formButton' name='" . $element->getName()
-            . "'  id='" . $element->getName() . "' value='" . $element->getValue() . "' title='"
-            . $element->getValue() . "'" . $element->getExtra() . ' />';
+        return "<input type='".$element->getType()."' class='formButton' name='".$element->getName()
+            . "'  id='".$element->getName()."' value='".$element->getValue()."' title='"
+            . $element->getValue()."'".$element->getExtra().' />';
     }
 
     /**
@@ -46,13 +46,13 @@ 
     {
         $ret = '';
         if ($element->_showDelete) {
-            $ret .= '<input type="submit" class="formbutton" name="delete" id="delete" value="' . _DELETE
+            $ret .= '<input type="submit" class="formbutton" name="delete" id="delete" value="'._DELETE
                 . '" onclick="this.form.elements.op.value=\'delete\'">&nbsp;';
         }
-        $ret .= '<input type="button" value="' . _CANCEL . '" onClick="history.go(-1);return true;" />&nbsp;'
-            . '<input type="reset" class="formbutton"  name="reset"  id="reset" value="' . _RESET . '" />&nbsp;'
-            . '<input type="' . $element->getType() . '" class="formbutton"  name="' . $element->getName()
-            . '" id="' . $element->getName() . '" value="' . $element->getValue() . '"' . $element->getExtra()
+        $ret .= '<input type="button" value="'._CANCEL.'" onClick="history.go(-1);return true;" />&nbsp;'
+            . '<input type="reset" class="formbutton"  name="reset"  id="reset" value="'._RESET.'" />&nbsp;'
+            . '<input type="'.$element->getType().'" class="formbutton"  name="'.$element->getName()
+            . '" id="'.$element->getName().'" value="'.$element->getValue().'"'.$element->getExtra()
             . ' />';
 
         return $ret;
@@ -97,14 +97,14 @@ 
                 $ret .= '<td>';
             }
             // $name may be a link, should we use $name in the title tag?
-            $ret .= '<input type="checkbox" name="' . $ele_name . '" id="' . $ele_id .$id_ele . '" '
-                . ' title="' . $ele_title . '" value="' . htmlspecialchars($value, ENT_QUOTES) . '"';
+            $ret .= '<input type="checkbox" name="'.$ele_name.'" id="'.$ele_id.$id_ele.'" '
+                . ' title="'.$ele_title.'" value="'.htmlspecialchars($value, ENT_QUOTES).'"';
 
             if (count($ele_value) > 0 && in_array($value, $ele_value)) {
                 $ret .= ' checked';
             }
-            $ret .= $ele_extra . ' />' . '<label name="xolb_' . $ele_name . '" for="'
-            . $ele_id . $id_ele . '" >' . $name . '</label>' . $ele_delimiter;
+            $ret .= $ele_extra.' />'.'<label name="xolb_'.$ele_name.'" for="'
+            . $ele_id.$id_ele.'" >'.$name.'</label>'.$ele_delimiter;
 
 
             if (!empty($element->columns)) {
@@ -116,7 +116,7 @@ 
         }
         if (!empty($element->columns)) {
             if ($span = $i % $element->columns) {
-                $ret .= '<td colspan="' . ($element->columns - $span) . '"></td></tr>';
+                $ret .= '<td colspan="'.($element->columns - $span).'"></td></tr>';
             }
             $ret .= '</table>';
         }
@@ -138,12 +138,12 @@ 
             $GLOBALS['xoTheme']->addScript('include/spectrum.js');
             $GLOBALS['xoTheme']->addStylesheet('include/spectrum.css');
         } else {
-            echo '<script type="text/javascript" src="' . XOOPS_URL . '/include/spectrum.js"></script>';
-            echo '<link rel="stylesheet" type="text/css" href="' . XOOPS_URL . '/include/spectrum.css">';
+            echo '<script type="text/javascript" src="'.XOOPS_URL.'/include/spectrum.js"></script>';
+            echo '<link rel="stylesheet" type="text/css" href="'.XOOPS_URL.'/include/spectrum.css">';
         }
-        return "<input type='color' name='" . $element->getName() . "' title='" . $element->getTitle()
-            . "' id='" . $element->getName() . "' size='" . $element->getSize() . "' maxlength='"
-            . $element->getMaxlength() . "' value='" . $element->getValue() . "'" . $element->getExtra()
+        return "<input type='color' name='".$element->getName()."' title='".$element->getTitle()
+            . "' id='".$element->getName()."' size='".$element->getSize()."' maxlength='"
+            . $element->getMaxlength()."' value='".$element->getValue()."'".$element->getExtra()
             . ' />';
     }
 
@@ -161,29 +161,29 @@ 
         xoops_loadLanguage('formdhtmltextarea');
         $ret = '';
         // actions
-        $ret .= $this->renderFormDhtmlTAXoopsCode($element) . "<br>\n";
+        $ret .= $this->renderFormDhtmlTAXoopsCode($element)."<br>\n";
         // fonts
         $ret .= $this->renderFormDhtmlTATypography($element);
         // length checker
-        $ret .= "<button type='button' class='btn btn-default' onclick=\"XoopsCheckLength('" . $element->getName() . "', '" . @$element->configs['maxlength'] . "', '" . _XOOPS_FORM_ALT_LENGTH . "', '" . _XOOPS_FORM_ALT_LENGTH_MAX . "');\" title='" . _XOOPS_FORM_ALT_CHECKLENGTH . "'><span class='fa fa-check-square-o' aria-hidden='true'></span></button>&nbsp;";
+        $ret .= "<button type='button' class='btn btn-default' onclick=\"XoopsCheckLength('".$element->getName()."', '".@$element->configs['maxlength']."', '"._XOOPS_FORM_ALT_LENGTH."', '"._XOOPS_FORM_ALT_LENGTH_MAX."');\" title='"._XOOPS_FORM_ALT_CHECKLENGTH."'><span class='fa fa-check-square-o' aria-hidden='true'></span></button>&nbsp;";
         $ret .= "<br>\n";
         // the textarea box
-        $ret .= "<textarea id='" . $element->getName() . "' name='" . $element->getName() . "' title='" . $element->getTitle() . "' onselect=\"xoopsSavePosition('" . $element->getName() . "');\" onclick=\"xoopsSavePosition('" . $element->getName() . "');\" onkeyup=\"xoopsSavePosition('" . $element->getName() . "');\" cols='" . $element->getCols() . "' rows='" . $element->getRows() . "'" . $element->getExtra() . '>' . $element->getValue() . "</textarea><br>\n";
+        $ret .= "<textarea id='".$element->getName()."' name='".$element->getName()."' title='".$element->getTitle()."' onselect=\"xoopsSavePosition('".$element->getName()."');\" onclick=\"xoopsSavePosition('".$element->getName()."');\" onkeyup=\"xoopsSavePosition('".$element->getName()."');\" cols='".$element->getCols()."' rows='".$element->getRows()."'".$element->getExtra().'>'.$element->getValue()."</textarea><br>\n";
 
         if (empty($element->skipPreview)) {
             if (empty($GLOBALS['xoTheme'])) {
-                $element->js .= implode('', file(XOOPS_ROOT_PATH . '/class/textsanitizer/image/image.js'));
+                $element->js .= implode('', file(XOOPS_ROOT_PATH.'/class/textsanitizer/image/image.js'));
             } else {
                 $GLOBALS['xoTheme']->addScript('/class/textsanitizer/image/image.js', array('type' => 'text/javascript'));
             }
-            $button = "<button type='button' class='btn btn-primary' onclick=\"form_instantPreview('" . XOOPS_URL . "', '" . $element->getName() . "','" . XOOPS_URL . "/images', " . (int)$element->doHtml . ", '" . $GLOBALS['xoopsSecurity']->createToken() . "')\" title='" . _PREVIEW . "'>" . _PREVIEW . "</button>";
+            $button = "<button type='button' class='btn btn-primary' onclick=\"form_instantPreview('".XOOPS_URL."', '".$element->getName()."','".XOOPS_URL."/images', ".(int)$element->doHtml.", '".$GLOBALS['xoopsSecurity']->createToken()."')\" title='"._PREVIEW."'>"._PREVIEW."</button>";
 
-            $ret .= '<br>' . "<div id='" . $element->getName() . "_hidden' style='display: block;'> " . '   <fieldset>' . '       <legend>' . $button . '</legend>' . "       <div id='" . $element->getName() . "_hidden_data'>" . _XOOPS_FORM_PREVIEW_CONTENT . '</div>' . '   </fieldset>' . '</div>';
+            $ret .= '<br>'."<div id='".$element->getName()."_hidden' style='display: block;'> ".'   <fieldset>'.'       <legend>'.$button.'</legend>'."       <div id='".$element->getName()."_hidden_data'>"._XOOPS_FORM_PREVIEW_CONTENT.'</div>'.'   </fieldset>'.'</div>';
         }
         // Load javascript
         if (empty($js_loaded)) {
-            $javascript = ($element->js ? '<script type="text/javascript">' . $element->js . '</script>' : '') . '<script type="text/javascript" src="' . XOOPS_URL . '/include/formdhtmltextarea.js"></script>';
-            $ret        = $javascript . $ret;
+            $javascript = ($element->js ? '<script type="text/javascript">'.$element->js.'</script>' : '').'<script type="text/javascript" src="'.XOOPS_URL.'/include/formdhtmltextarea.js"></script>';
+            $ret        = $javascript.$ret;
             $js_loaded  = true;
         }
 
@@ -202,13 +202,13 @@ 
         $textarea_id = $element->getName();
         $code = '';
         $code .= '<a name="moresmiley"></a>';
-        $code .= "<button type='button' class='btn btn-default' onclick='xoopsCodeUrl(\"{$textarea_id}\", \"" . htmlspecialchars(_ENTERURL, ENT_QUOTES) . "\", \"" . htmlspecialchars(_ENTERWEBTITLE, ENT_QUOTES) . "\");' onmouseover='style.cursor=\"hand\"' title='" . _XOOPS_FORM_ALT_URL . "'><span class='fa fa-fw fa-link' aria-hidden='true'></span></button>";
-        $code .= "<button type='button' class='btn btn-default' onclick='xoopsCodeEmail(\"{$textarea_id}\", \"" . htmlspecialchars(_ENTEREMAIL, ENT_QUOTES) . "\", \"" . htmlspecialchars(_ENTERWEBTITLE, ENT_QUOTES) . "\");' onmouseover='style.cursor=\"hand\"' title='" . _XOOPS_FORM_ALT_EMAIL . "'><span class='fa fa-fw fa-envelope-o' aria-hidden='true'></span></button>";
-        $code .= "<button type='button' class='btn btn-default' onclick='xoopsCodeImg(\"{$textarea_id}\", \"" . htmlspecialchars(_ENTERIMGURL, ENT_QUOTES) . "\", \"" . htmlspecialchars(_ENTERIMGPOS, ENT_QUOTES) . "\", \"" . htmlspecialchars(_IMGPOSRORL, ENT_QUOTES) . "\", \"" . htmlspecialchars(_ERRORIMGPOS, ENT_QUOTES) . "\", \"" . htmlspecialchars(_XOOPS_FORM_ALT_ENTERWIDTH, ENT_QUOTES) . "\");' onmouseover='style.cursor=\"hand\"' title='" . _XOOPS_FORM_ALT_IMG . "'><span class='fa fa-fw fa-file-image-o' aria-hidden='true'></span></button>";
-        $code .= "<button type='button' class='btn btn-default' onclick='openWithSelfMain(\"" . XOOPS_URL . "/imagemanager.php?target={$textarea_id}\",\"imgmanager\",400,430);' onmouseover='style.cursor=\"hand\"' title='" . _XOOPS_FORM_ALT_IMAGE . "'><span class='fa fa-file-image-o' aria-hidden='true'></span><span style='font-size:75%;'> Manager</span></button>";
-        $code .= "<button type='button' class='btn btn-default' onclick='openWithSelfMain(\"" . XOOPS_URL . "/misc.php?action=showpopups&amp;type=smilies&amp;target={$textarea_id}\",\"smilies\",300,475);' onmouseover='style.cursor=\"hand\"' title='" . _XOOPS_FORM_ALT_SMILEY . "'><span class='fa fa-fw fa-smile-o' aria-hidden='true'></span></button>";
+        $code .= "<button type='button' class='btn btn-default' onclick='xoopsCodeUrl(\"{$textarea_id}\", \"".htmlspecialchars(_ENTERURL, ENT_QUOTES)."\", \"".htmlspecialchars(_ENTERWEBTITLE, ENT_QUOTES)."\");' onmouseover='style.cursor=\"hand\"' title='"._XOOPS_FORM_ALT_URL."'><span class='fa fa-fw fa-link' aria-hidden='true'></span></button>";
+        $code .= "<button type='button' class='btn btn-default' onclick='xoopsCodeEmail(\"{$textarea_id}\", \"".htmlspecialchars(_ENTEREMAIL, ENT_QUOTES)."\", \"".htmlspecialchars(_ENTERWEBTITLE, ENT_QUOTES)."\");' onmouseover='style.cursor=\"hand\"' title='"._XOOPS_FORM_ALT_EMAIL."'><span class='fa fa-fw fa-envelope-o' aria-hidden='true'></span></button>";
+        $code .= "<button type='button' class='btn btn-default' onclick='xoopsCodeImg(\"{$textarea_id}\", \"".htmlspecialchars(_ENTERIMGURL, ENT_QUOTES)."\", \"".htmlspecialchars(_ENTERIMGPOS, ENT_QUOTES)."\", \"".htmlspecialchars(_IMGPOSRORL, ENT_QUOTES)."\", \"".htmlspecialchars(_ERRORIMGPOS, ENT_QUOTES)."\", \"".htmlspecialchars(_XOOPS_FORM_ALT_ENTERWIDTH, ENT_QUOTES)."\");' onmouseover='style.cursor=\"hand\"' title='"._XOOPS_FORM_ALT_IMG."'><span class='fa fa-fw fa-file-image-o' aria-hidden='true'></span></button>";
+        $code .= "<button type='button' class='btn btn-default' onclick='openWithSelfMain(\"".XOOPS_URL."/imagemanager.php?target={$textarea_id}\",\"imgmanager\",400,430);' onmouseover='style.cursor=\"hand\"' title='"._XOOPS_FORM_ALT_IMAGE."'><span class='fa fa-file-image-o' aria-hidden='true'></span><span style='font-size:75%;'> Manager</span></button>";
+        $code .= "<button type='button' class='btn btn-default' onclick='openWithSelfMain(\"".XOOPS_URL."/misc.php?action=showpopups&amp;type=smilies&amp;target={$textarea_id}\",\"smilies\",300,475);' onmouseover='style.cursor=\"hand\"' title='"._XOOPS_FORM_ALT_SMILEY."'><span class='fa fa-fw fa-smile-o' aria-hidden='true'></span></button>";
 
-        $myts        = MyTextSanitizer::getInstance();
+        $myts = MyTextSanitizer::getInstance();
 
         $extensions = array_filter($myts->config['extensions']);
         foreach (array_keys($extensions) as $key) {
@@ -222,8 +222,8 @@ 
                 $element->js .= $js;
             }
         }
-        $code .= "<button type='button' class='btn btn-default' onclick='xoopsCodeCode(\"{$textarea_id}\", \"" . htmlspecialchars(_ENTERCODE, ENT_QUOTES) . "\");' onmouseover='style.cursor=\"hand\"' title='" . _XOOPS_FORM_ALT_CODE . "'><span class='fa fa-fw fa-code' aria-hidden='true'></span></button>";
-        $code .= "<button type='button' class='btn btn-default' onclick='xoopsCodeQuote(\"{$textarea_id}\", \"" . htmlspecialchars(_ENTERQUOTE, ENT_QUOTES) . "\");' onmouseover='style.cursor=\"hand\"' title='" . _XOOPS_FORM_ALT_QUOTE . "'><span class='fa fa-fw fa-quote-right' aria-hidden='true'></span></button>";
+        $code .= "<button type='button' class='btn btn-default' onclick='xoopsCodeCode(\"{$textarea_id}\", \"".htmlspecialchars(_ENTERCODE, ENT_QUOTES)."\");' onmouseover='style.cursor=\"hand\"' title='"._XOOPS_FORM_ALT_CODE."'><span class='fa fa-fw fa-code' aria-hidden='true'></span></button>";
+        $code .= "<button type='button' class='btn btn-default' onclick='xoopsCodeQuote(\"{$textarea_id}\", \"".htmlspecialchars(_ENTERQUOTE, ENT_QUOTES)."\");' onmouseover='style.cursor=\"hand\"' title='"._XOOPS_FORM_ALT_QUOTE."'><span class='fa fa-fw fa-quote-right' aria-hidden='true'></span></button>";
 
         $xoopsPreload = XoopsPreload::getInstance();
         $xoopsPreload->triggerEvent('core.class.xoopsform.formdhtmltextarea.codeicon', array(&$code));
@@ -242,13 +242,13 @@ 
     {
         $textarea_id = $element->getName();
         $hiddentext  = $element->_hiddenText;
-        $fontStr = "<script type=\"text/javascript\">" . "var _editor_dialog = ''" . "+ '<select class=\"input-sm form-control\" id=\'{$textarea_id}Size\' onchange=\'xoopsSetElementAttribute(\"size\", this.options[this.selectedIndex].value, \"{$textarea_id}\", \"{$hiddentext}\");\'>'" . "+ '<option value=\'SIZE\'>" . _SIZE . "</option>'";
+        $fontStr = "<script type=\"text/javascript\">"."var _editor_dialog = ''"."+ '<select class=\"input-sm form-control\" id=\'{$textarea_id}Size\' onchange=\'xoopsSetElementAttribute(\"size\", this.options[this.selectedIndex].value, \"{$textarea_id}\", \"{$hiddentext}\");\'>'"."+ '<option value=\'SIZE\'>"._SIZE."</option>'";
 
         foreach ($GLOBALS['formtextdhtml_sizes'] as $_val => $_name) {
             $fontStr .= " + '<option value=\'{$_val}\'>{$_name}</option>'";
         }
         $fontStr .= " + '</select> '";
-        $fontStr .= "+ '<select class=\"input-sm form-control\" id=\'{$textarea_id}Font\' onchange=\'xoopsSetElementAttribute(\"font\", this.options[this.selectedIndex].value, \"{$textarea_id}\", \"{$hiddentext}\");\'>'" . "+ '<option value=\'FONT\'>" . _FONT . "</option>'";
+        $fontStr .= "+ '<select class=\"input-sm form-control\" id=\'{$textarea_id}Font\' onchange=\'xoopsSetElementAttribute(\"font\", this.options[this.selectedIndex].value, \"{$textarea_id}\", \"{$hiddentext}\");\'>'"."+ '<option value=\'FONT\'>"._FONT."</option>'";
         $fontarray = !empty($GLOBALS['formtextdhtml_fonts']) ? $GLOBALS['formtextdhtml_fonts'] : array(
             'Arial',
             'Courier',
@@ -261,7 +261,7 @@ 
             $fontStr .= " + '<option value=\'{$font}\'>{$font}</option>'";
         }
         $fontStr .= " + '</select> '";
-        $fontStr .= "+ '<select class=\"input-sm form-control\" id=\'{$textarea_id}Color\' onchange=\'xoopsSetElementAttribute(\"color\", this.options[this.selectedIndex].value, \"{$textarea_id}\", \"{$hiddentext}\");\'>'" . "+ '<option value=\'COLOR\'>" . _COLOR . "</option>';" . "var _color_array = new Array('00', '33', '66', '99', 'CC', 'FF');
+        $fontStr .= "+ '<select class=\"input-sm form-control\" id=\'{$textarea_id}Color\' onchange=\'xoopsSetElementAttribute(\"color\", this.options[this.selectedIndex].value, \"{$textarea_id}\", \"{$hiddentext}\");\'>'"."+ '<option value=\'COLOR\'>"._COLOR."</option>';"."var _color_array = new Array('00', '33', '66', '99', 'CC', 'FF');
                 for (var i = 0; i < _color_array.length; i ++) {
                     for (var j = 0; j < _color_array.length; j ++) {
                         for (var k = 0; k < _color_array.length; k ++) {
@@ -274,14 +274,14 @@ 
         $fontStr .= ";";
         $fontStr .= 'document.write(_editor_dialog); </script>';
 
-        $styleStr  = "<button type='button' class='btn btn-default' onclick='xoopsMakeBold(\"{$hiddentext}\", \"{$textarea_id}\");' title='" . _XOOPS_FORM_ALT_BOLD . "' aria-label='Left Align'><span class='fa fa-bold' aria-hidden='true'></span></button>";
-        $styleStr .= "<button type='button' class='btn btn-default' onclick='xoopsMakeItalic(\"{$hiddentext}\", \"{$textarea_id}\");' title='" . _XOOPS_FORM_ALT_ITALIC . "' aria-label='Left Align'><span class='fa fa-italic' aria-hidden='true'></span></button>";
-        $styleStr .= "<button type='button' class='btn btn-default' onclick='xoopsMakeUnderline(\"{$hiddentext}\", \"{$textarea_id}\");' title='" . _XOOPS_FORM_ALT_UNDERLINE . "' aria-label='Left Align'>" . '<span class="fa fa-underline"></span></button>';
-        $styleStr .= "<button type='button' class='btn btn-default' onclick='xoopsMakeLineThrough(\"{$hiddentext}\", \"{$textarea_id}\");' title='" . _XOOPS_FORM_ALT_LINETHROUGH . "' aria-label='Left Align'>" . '<span class="fa fa-strikethrough"></span></button>';
+        $styleStr  = "<button type='button' class='btn btn-default' onclick='xoopsMakeBold(\"{$hiddentext}\", \"{$textarea_id}\");' title='"._XOOPS_FORM_ALT_BOLD."' aria-label='Left Align'><span class='fa fa-bold' aria-hidden='true'></span></button>";
+        $styleStr .= "<button type='button' class='btn btn-default' onclick='xoopsMakeItalic(\"{$hiddentext}\", \"{$textarea_id}\");' title='"._XOOPS_FORM_ALT_ITALIC."' aria-label='Left Align'><span class='fa fa-italic' aria-hidden='true'></span></button>";
+        $styleStr .= "<button type='button' class='btn btn-default' onclick='xoopsMakeUnderline(\"{$hiddentext}\", \"{$textarea_id}\");' title='"._XOOPS_FORM_ALT_UNDERLINE."' aria-label='Left Align'>".'<span class="fa fa-underline"></span></button>';
+        $styleStr .= "<button type='button' class='btn btn-default' onclick='xoopsMakeLineThrough(\"{$hiddentext}\", \"{$textarea_id}\");' title='"._XOOPS_FORM_ALT_LINETHROUGH."' aria-label='Left Align'>".'<span class="fa fa-strikethrough"></span></button>';
 
-        $alignStr  = "<button type='button' class='btn btn-default' onclick='xoopsMakeLeft(\"{$hiddentext}\", \"{$textarea_id}\");' title='" . _XOOPS_FORM_ALT_LEFT . "' aria-label='Left Align'><span class='fa fa-align-left' aria-hidden='true'></span></button>";
-        $alignStr .= "<button type='button' class='btn btn-default' onclick='xoopsMakeCenter(\"{$hiddentext}\", \"{$textarea_id}\");' title='" . _XOOPS_FORM_ALT_CENTER . "' aria-label='Left Align'><span class='fa fa-align-center' aria-hidden='true'></span></button>";
-        $alignStr .= "<button type='button' class='btn btn-default' onclick='xoopsMakeRight(\"{$hiddentext}\", \"{$textarea_id}\");' title='" . _XOOPS_FORM_ALT_RIGHT . "' aria-label='Left Align'><span class='fa fa-align-right' aria-hidden='true'></span></button>";
+        $alignStr  = "<button type='button' class='btn btn-default' onclick='xoopsMakeLeft(\"{$hiddentext}\", \"{$textarea_id}\");' title='"._XOOPS_FORM_ALT_LEFT."' aria-label='Left Align'><span class='fa fa-align-left' aria-hidden='true'></span></button>";
+        $alignStr .= "<button type='button' class='btn btn-default' onclick='xoopsMakeCenter(\"{$hiddentext}\", \"{$textarea_id}\");' title='"._XOOPS_FORM_ALT_CENTER."' aria-label='Left Align'><span class='fa fa-align-center' aria-hidden='true'></span></button>";
+        $alignStr .= "<button type='button' class='btn btn-default' onclick='xoopsMakeRight(\"{$hiddentext}\", \"{$textarea_id}\");' title='"._XOOPS_FORM_ALT_RIGHT."' aria-label='Left Align'><span class='fa fa-align-right' aria-hidden='true'></span></button>";
 
         $fontStr .= "<br>\n{$styleStr}&nbsp;{$alignStr}&nbsp;\n";
         return $fontStr;
@@ -303,9 +303,9 @@ 
                 $ret .= $element->getDelimeter();
             }
             if ($ele->getCaption() != '') {
-                $ret .= $ele->getCaption() . '&nbsp;';
+                $ret .= $ele->getCaption().'&nbsp;';
             }
-            $ret .= $ele->render() . NWLINE;
+            $ret .= $ele->render().NWLINE;
             if (!$ele->isHidden()) {
                 ++$count;
             }
@@ -322,10 +322,10 @@ 
      */
     public function renderFormFile(XoopsFormFile $element)
     {
-        return '<input type="hidden" name="MAX_FILE_SIZE" value="' . $element->getMaxFileSize() . '" />'
-            . '<input type="file" name="' . $element->getName() . '" id="' . $element->getName() . '" title="'
-            . $element->getTitle() . '" ' . $element->getExtra() . ' />'
-            . '<input type="hidden" name="xoops_upload_file[]" id="xoops_upload_file[]" value="' . $element->getName()
+        return '<input type="hidden" name="MAX_FILE_SIZE" value="'.$element->getMaxFileSize().'" />'
+            . '<input type="file" name="'.$element->getName().'" id="'.$element->getName().'" title="'
+            . $element->getTitle().'" '.$element->getExtra().' />'
+            . '<input type="hidden" name="xoops_upload_file[]" id="xoops_upload_file[]" value="'.$element->getName()
             . '" />';
 
     }
@@ -351,9 +351,9 @@ 
      */
     public function renderFormPassword(XoopsFormPassword $element)
     {
-        return '<input type="password" name="' . $element->getName() . '" id="' . $element->getName() . '" size="'
-            . $element->getSize() . '" maxlength="' . $element->getMaxlength() . '" value="' . $element->getValue()
-            . '"' . $element->getExtra() . ' ' . ($element->autoComplete ? '' : 'autocomplete="off" ') . '/>';
+        return '<input type="password" name="'.$element->getName().'" id="'.$element->getName().'" size="'
+            . $element->getSize().'" maxlength="'.$element->getMaxlength().'" value="'.$element->getValue()
+            . '"'.$element->getExtra().' '.($element->autoComplete ? '' : 'autocomplete="off" ').'/>';
     }
 
     /**
@@ -386,14 +386,14 @@ 
                 $ret .= '<td>';
             }
 
-            $ret .= '<input type="radio" name="' . $ele_name . '" id="' . $ele_name . $id_ele
-                . '" title = "' . htmlspecialchars($ele_title, ENT_QUOTES) . '" value="'
-                . htmlspecialchars($value, ENT_QUOTES) . '"';
+            $ret .= '<input type="radio" name="'.$ele_name.'" id="'.$ele_name.$id_ele
+                . '" title = "'.htmlspecialchars($ele_title, ENT_QUOTES).'" value="'
+                . htmlspecialchars($value, ENT_QUOTES).'"';
             if (isset($ele_value) && $value == $ele_value) {
                 $ret .= ' checked';
             }
-            $ret .= $ele_extra . ' />' . "<label name='xolb_{$ele_name}' for='" . $ele_name . $id_ele
-                . "'>" . $name . '</label>' . $ele_delimiter;
+            $ret .= $ele_extra.' />'."<label name='xolb_{$ele_name}' for='".$ele_name.$id_ele
+                . "'>".$name.'</label>'.$ele_delimiter;
             if (!empty($element->columns)) {
                 $ret .= '</td>';
                 if (++$i % $element->columns == 0) {
@@ -403,7 +403,7 @@ 
         }
         if (!empty($element->columns)) {
             if ($span = $i % $element->columns) {
-                $ret .= '<td colspan="' . ($element->columns - $span) . '"></td></tr>';
+                $ret .= '<td colspan="'.($element->columns - $span).'"></td></tr>';
             }
             $ret .= '</table>';
         }
@@ -424,19 +424,19 @@ 
         $ele_title   = $element->getTitle();
         $ele_value   = $element->getValue();
         $ele_options = $element->getOptions();
-        $ret = '<select size="' . $element->getSize() . '"' . $element->getExtra();
+        $ret = '<select size="'.$element->getSize().'"'.$element->getExtra();
         if ($element->isMultiple() != false) {
-            $ret .= ' name="' . $ele_name . '[]" id="' . $ele_name . '" title="' . $ele_title
+            $ret .= ' name="'.$ele_name.'[]" id="'.$ele_name.'" title="'.$ele_title
                 . '" multiple="multiple">';
         } else {
-            $ret .= ' name="' . $ele_name . '" id="' . $ele_name . '" title="' . $ele_title . '">';
+            $ret .= ' name="'.$ele_name.'" id="'.$ele_name.'" title="'.$ele_title.'">';
         }
         foreach ($ele_options as $value => $name) {
-            $ret .= '<option value="' . htmlspecialchars($value, ENT_QUOTES) . '"';
+            $ret .= '<option value="'.htmlspecialchars($value, ENT_QUOTES).'"';
             if (count($ele_value) > 0 && in_array($value, $ele_value)) {
                 $ret .= ' selected';
             }
-            $ret .= '>' . $name . '</option>';
+            $ret .= '>'.$name.'</option>';
         }
         $ret .= '</select>';
 
@@ -452,9 +452,9 @@ 
      */
     public function renderFormText(XoopsFormText $element)
     {
-        return "<input type='text' name='" . $element->getName() . "' title='" . $element->getTitle()
-            . "' id='" . $element->getName() . "' size='" . $element->getSize() . "' maxlength='"
-            . $element->getMaxlength() . "' value='" . $element->getValue() . "'" . $element->getExtra()
+        return "<input type='text' name='".$element->getName()."' title='".$element->getTitle()
+            . "' id='".$element->getName()."' size='".$element->getSize()."' maxlength='"
+            . $element->getMaxlength()."' value='".$element->getValue()."'".$element->getExtra()
             . ' />';
     }
 
@@ -467,9 +467,9 @@ 
      */
     public function renderFormTextArea(XoopsFormTextArea $element)
     {
-        return "<textarea name='" . $element->getName() . "' id='" . $element->getName() . "'  title='"
-            . $element->getTitle() . "' rows='" . $element->getRows() . "' cols='" . $element->getCols()
-            . "'" . $element->getExtra() . '>' . $element->getValue() . '</textarea>';
+        return "<textarea name='".$element->getName()."' id='".$element->getName()."'  title='"
+            . $element->getTitle()."' rows='".$element->getRows()."' cols='".$element->getCols()
+            . "'".$element->getExtra().'>'.$element->getValue().'</textarea>';
     }
 
     /**
@@ -482,10 +482,10 @@ 
     public function renderFormTextDateSelect(XoopsFormTextDateSelect $element)
     {
         static $included = false;
-        if (file_exists(XOOPS_ROOT_PATH . '/language/' . $GLOBALS['xoopsConfig']['language'] . '/calendar.php')) {
-            include_once XOOPS_ROOT_PATH . '/language/' . $GLOBALS['xoopsConfig']['language'] . '/calendar.php';
+        if (file_exists(XOOPS_ROOT_PATH.'/language/'.$GLOBALS['xoopsConfig']['language'].'/calendar.php')) {
+            include_once XOOPS_ROOT_PATH.'/language/'.$GLOBALS['xoopsConfig']['language'].'/calendar.php';
         } else {
-            include_once XOOPS_ROOT_PATH . '/language/english/calendar.php';
+            include_once XOOPS_ROOT_PATH.'/language/english/calendar.php';
         }
 
         $ele_name  = $element->getName();
@@ -532,7 +532,7 @@ 
                     if (calendar != null) {
                     calendar.hide();
                     } else {
-                    var cal = new Calendar(true, "' . $jstime . '", selected, closeHandler);
+                    var cal = new Calendar(true, "' . $jstime.'", selected, closeHandler);
                     calendar = cal;
                     cal.setRange(1900, 2100);
                     calendar.create();
@@ -546,55 +546,55 @@ 
                     }
 
                     Calendar._DN = new Array
-                    ("' . _CAL_SUNDAY . '",
-                    "' . _CAL_MONDAY . '",
-                    "' . _CAL_TUESDAY . '",
-                    "' . _CAL_WEDNESDAY . '",
-                    "' . _CAL_THURSDAY . '",
-                    "' . _CAL_FRIDAY . '",
-                    "' . _CAL_SATURDAY . '",
-                    "' . _CAL_SUNDAY . '");
+                    ("' . _CAL_SUNDAY.'",
+                    "' . _CAL_MONDAY.'",
+                    "' . _CAL_TUESDAY.'",
+                    "' . _CAL_WEDNESDAY.'",
+                    "' . _CAL_THURSDAY.'",
+                    "' . _CAL_FRIDAY.'",
+                    "' . _CAL_SATURDAY.'",
+                    "' . _CAL_SUNDAY.'");
                     Calendar._MN = new Array
-                    ("' . _CAL_JANUARY . '",
-                    "' . _CAL_FEBRUARY . '",
-                    "' . _CAL_MARCH . '",
-                    "' . _CAL_APRIL . '",
-                    "' . _CAL_MAY . '",
-                    "' . _CAL_JUNE . '",
-                    "' . _CAL_JULY . '",
-                    "' . _CAL_AUGUST . '",
-                    "' . _CAL_SEPTEMBER . '",
-                    "' . _CAL_OCTOBER . '",
-                    "' . _CAL_NOVEMBER . '",
-                    "' . _CAL_DECEMBER . '");
+                    ("' . _CAL_JANUARY.'",
+                    "' . _CAL_FEBRUARY.'",
+                    "' . _CAL_MARCH.'",
+                    "' . _CAL_APRIL.'",
+                    "' . _CAL_MAY.'",
+                    "' . _CAL_JUNE.'",
+                    "' . _CAL_JULY.'",
+                    "' . _CAL_AUGUST.'",
+                    "' . _CAL_SEPTEMBER.'",
+                    "' . _CAL_OCTOBER.'",
+                    "' . _CAL_NOVEMBER.'",
+                    "' . _CAL_DECEMBER.'");
 
                     Calendar._TT = {};
-                    Calendar._TT["TOGGLE"] = "' . _CAL_TGL1STD . '";
-                    Calendar._TT["PREV_YEAR"] = "' . _CAL_PREVYR . '";
-                    Calendar._TT["PREV_MONTH"] = "' . _CAL_PREVMNTH . '";
-                    Calendar._TT["GO_TODAY"] = "' . _CAL_GOTODAY . '";
-                    Calendar._TT["NEXT_MONTH"] = "' . _CAL_NXTMNTH . '";
-                    Calendar._TT["NEXT_YEAR"] = "' . _CAL_NEXTYR . '";
-                    Calendar._TT["SEL_DATE"] = "' . _CAL_SELDATE . '";
-                    Calendar._TT["DRAG_TO_MOVE"] = "' . _CAL_DRAGMOVE . '";
-                    Calendar._TT["PART_TODAY"] = "(' . _CAL_TODAY . ')";
-                    Calendar._TT["MON_FIRST"] = "' . _CAL_DISPM1ST . '";
-                    Calendar._TT["SUN_FIRST"] = "' . _CAL_DISPS1ST . '";
-                    Calendar._TT["CLOSE"] = "' . _CLOSE . '";
-                    Calendar._TT["TODAY"] = "' . _CAL_TODAY . '";
+                    Calendar._TT["TOGGLE"] = "' . _CAL_TGL1STD.'";
+                    Calendar._TT["PREV_YEAR"] = "' . _CAL_PREVYR.'";
+                    Calendar._TT["PREV_MONTH"] = "' . _CAL_PREVMNTH.'";
+                    Calendar._TT["GO_TODAY"] = "' . _CAL_GOTODAY.'";
+                    Calendar._TT["NEXT_MONTH"] = "' . _CAL_NXTMNTH.'";
+                    Calendar._TT["NEXT_YEAR"] = "' . _CAL_NEXTYR.'";
+                    Calendar._TT["SEL_DATE"] = "' . _CAL_SELDATE.'";
+                    Calendar._TT["DRAG_TO_MOVE"] = "' . _CAL_DRAGMOVE.'";
+                    Calendar._TT["PART_TODAY"] = "(' . _CAL_TODAY.')";
+                    Calendar._TT["MON_FIRST"] = "' . _CAL_DISPM1ST.'";
+                    Calendar._TT["SUN_FIRST"] = "' . _CAL_DISPS1ST.'";
+                    Calendar._TT["CLOSE"] = "' . _CLOSE.'";
+                    Calendar._TT["TODAY"] = "' . _CAL_TODAY.'";
 
                     // date formats
-                    Calendar._TT["DEF_DATE_FORMAT"] = "' . _SHORTDATESTRING . '";
-                    Calendar._TT["TT_DATE_FORMAT"] = "' . _SHORTDATESTRING . '";
+                    Calendar._TT["DEF_DATE_FORMAT"] = "' . _SHORTDATESTRING.'";
+                    Calendar._TT["TT_DATE_FORMAT"] = "' . _SHORTDATESTRING.'";
 
                     Calendar._TT["WK"] = "";
                 ');
             }
         }
-        return '<input type="text" name="' . $ele_name . '" id="' . $ele_name . '" size="'
-            . $element->getSize() . '" maxlength="' . $element->getMaxlength() . '" value="'
-            . $display_value . '"' . $element->getExtra()
-            . ' /><input type="reset" value=" ... " onclick="return showCalendar(\'' . $ele_name
+        return '<input type="text" name="'.$ele_name.'" id="'.$ele_name.'" size="'
+            . $element->getSize().'" maxlength="'.$element->getMaxlength().'" value="'
+            . $display_value.'"'.$element->getExtra()
+            . ' /><input type="reset" value=" ... " onclick="return showCalendar(\''.$ele_name
             . '\');">';
         /*
         return "<input type='text' name='" . $ele_name . "' id='" . $ele_name . "' size='"
@@ -615,9 +615,9 @@ 
     public function renderThemeForm(XoopsThemeForm $form)
     {
         $ele_name = $form->getName();
-        $ret      = '<form name="' . $ele_name . '" id="' . $ele_name . '" action="' . $form->getAction() . '" method="' . $form->getMethod() . '" onsubmit="return xoopsFormValidate_' . $ele_name . '();"' . $form->getExtra() . '>
+        $ret      = '<form name="'.$ele_name.'" id="'.$ele_name.'" action="'.$form->getAction().'" method="'.$form->getMethod().'" onsubmit="return xoopsFormValidate_'.$ele_name.'();"'.$form->getExtra().'>
             <table width="100%" class="outer" cellspacing="1">
-            <tr><th colspan="2">' . $form->getTitle() . '</th></tr>
+            <tr><th colspan="2">' . $form->getTitle().'</th></tr>
         ';
         $hidden   = '';
         $class    = 'even';
@@ -628,30 +628,30 @@ 
                 if (!$ele->getNocolspan()) {
                     $ret .= '<tr valign="top" align="left"><td class="head">';
                     if (($caption = $ele->getCaption()) != '') {
-                        $ret .= '<div class="xoops-form-element-caption' . ($ele->isRequired() ? '-required' : '') . '">';
-                        $ret .= '<span class="caption-text">' . $caption . '</span>';
+                        $ret .= '<div class="xoops-form-element-caption'.($ele->isRequired() ? '-required' : '').'">';
+                        $ret .= '<span class="caption-text">'.$caption.'</span>';
                         $ret .= '<span class="caption-marker">*</span>';
                         $ret .= '</div>';
                     }
                     if (($desc = $ele->getDescription()) != '') {
-                        $ret .= '<div class="xoops-form-element-help">' . $desc . '</div>';
+                        $ret .= '<div class="xoops-form-element-help">'.$desc.'</div>';
                     }
-                    $ret .= '</td><td class="' . $class . '">' . $ele->render() . '</td></tr>' . NWLINE;
+                    $ret .= '</td><td class="'.$class.'">'.$ele->render().'</td></tr>'.NWLINE;
                 } else {
                     $ret .= '<tr valign="top" align="left"><td class="head" colspan="2">';
                     if (($caption = $ele->getCaption()) != '') {
-                        $ret .= '<div class="xoops-form-element-caption' . ($ele->isRequired() ? '-required' : '') . '">';
-                        $ret .= '<span class="caption-text">' . $caption . '</span>';
+                        $ret .= '<div class="xoops-form-element-caption'.($ele->isRequired() ? '-required' : '').'">';
+                        $ret .= '<span class="caption-text">'.$caption.'</span>';
                         $ret .= '<span class="caption-marker">*</span>';
                         $ret .= '</div>';
                     }
-                    $ret .= '</td></tr><tr valign="top" align="left"><td class="' . $class . '" colspan="2">' . $ele->render() . '</td></tr>';
+                    $ret .= '</td></tr><tr valign="top" align="left"><td class="'.$class.'" colspan="2">'.$ele->render().'</td></tr>';
                 }
             } else {
                 $hidden .= $ele->render();
             }
         }
-        $ret .= '</table>' . NWLINE . ' ' . $hidden . '</form>' . NWLINE;
+        $ret .= '</table>'.NWLINE.' '.$hidden.'</form>'.NWLINE;
         $ret .= $form->renderValidationJS(true);
 
         return $ret;
@@ -668,13 +668,13 @@ 
      */
     public function addThemeFormBreak(XoopsThemeForm $form, $extra, $class)
     {
-        $class = ($class != '') ? " class='" . preg_replace('/[^A-Za-z0-9\s\s_-]/i', '', $class) . "'" : '';
+        $class = ($class != '') ? " class='".preg_replace('/[^A-Za-z0-9\s\s_-]/i', '', $class)."'" : '';
         // Fix for $extra tag not showing
         if ($extra) {
-            $extra = '<tr><td colspan="2" ' . $class . '>' . $extra . '</td></tr>';
+            $extra = '<tr><td colspan="2" '.$class.'>'.$extra.'</td></tr>';
             $form->addElement($extra);
         } else {
-            $extra = '<tr><td colspan="2" ' . $class . '>&nbsp;</td></tr>';
+            $extra = '<tr><td colspan="2" '.$class.'>&nbsp;</td></tr>';
             $form->addElement($extra);
         }
     }

htdocs/class/xoopsform/formcheckbox.php

Spacing +1 added lines, -1 removed lines

@@ -203,7 +203,7 @@
             $eltmsg     = empty($eltcaption) ? sprintf(_FORM_ENTER, $eltname) : sprintf(_FORM_ENTER, $eltcaption);
             $eltmsg     = str_replace('"', '\"', stripslashes($eltmsg));
 
-            return NWLINE . "var hasChecked = false; var checkBox = myform.elements['{$eltname}']; if (checkBox.length) {for (var i = 0; i < checkBox.length; i++) {if (checkBox[i].checked == true) {hasChecked = true; break;}}} else {if (checkBox.checked == true) {hasChecked = true;}}if (!hasChecked) {window.alert(\"{$eltmsg}\");if (checkBox.length) {checkBox[0].focus();} else {checkBox.focus();}return false;}";
+            return NWLINE."var hasChecked = false; var checkBox = myform.elements['{$eltname}']; if (checkBox.length) {for (var i = 0; i < checkBox.length; i++) {if (checkBox[i].checked == true) {hasChecked = true; break;}}} else {if (checkBox.checked == true) {hasChecked = true;}}if (!hasChecked) {window.alert(\"{$eltmsg}\");if (checkBox.length) {checkBox[0].focus();} else {checkBox.focus();}return false;}";
         }
 
         return '';