XOOPS / XoopsCore

xoops_lib/Xoops/Core/Kernel/Handlers/XoopsBlockHandler.php

Spacing +20 added lines, -20 removed lines

@@ -182,7 +182,7 @@ 
 
         if (is_array($groupid)) {
             if (count($groupid) > 1) {
-                $in=array();
+                $in = array();
                 foreach ($groupid as $gid) {
                     $in[] = $qb->createNamedParameter($gid, ParameterType::INTEGER);
                 }
@@ -195,9 +195,9 @@ 
         if (isset($side)) {
             // get both sides in sidebox? (some themes need this)
             if ($side == XOOPS_SIDEBLOCK_BOTH) {
-                $qb->andWhere($eb->in('b.side', array(0,1)));
+                $qb->andWhere($eb->in('b.side', array(0, 1)));
             } elseif ($side == XOOPS_CENTERBLOCK_ALL) {
-                $qb->andWhere($eb->in('b.side', array(3,4,5,7,8,9)));
+                $qb->andWhere($eb->in('b.side', array(3, 4, 5, 7, 8, 9)));
             } else {
                 $qb->andWhere($eb->eq('b.side', $qb->createNamedParameter($side, ParameterType::INTEGER)));
             }
@@ -248,9 +248,9 @@ 
         if (isset($side)) {
             // get both sides in sidebox? (some themes need this)
             if ($side == XOOPS_SIDEBLOCK_BOTH) {
-                $qb->andWhere($eb->in('side', array(0,1)));
+                $qb->andWhere($eb->in('side', array(0, 1)));
             } elseif ($side == XOOPS_CENTERBLOCK_ALL) {
-                $qb->andWhere($eb->in('side', array(3,4,5,7,8,9)));
+                $qb->andWhere($eb->in('side', array(3, 4, 5, 7, 8, 9)));
             } else {
                 $qb->andWhere($eb->eq('side', $qb->createNamedParameter($side, ParameterType::INTEGER)));
             }
@@ -347,7 +347,7 @@ 
         $qb = $this->db2->createXoopsQueryBuilder();
         $eb = $qb->expr();
 
-        $blockids=null;
+        $blockids = null;
         if (isset($groupid)) {
             $qb ->select('DISTINCT gperm_itemid')
                 ->fromPrefix('system_permission', null)
@@ -377,17 +377,17 @@ 
             $qb ->fromPrefix('system_blockmodule', 'm')
                 ->andWhere($eb->eq('m.block_id', 'b.bid'));
             if (!empty($module_id)) {
-                $in=array();
-                $in[]=0;
-                $in[]=(int)($module_id);
+                $in = array();
+                $in[] = 0;
+                $in[] = (int)($module_id);
                 if ($toponlyblock) {
-                    $in[]=(int)(-1);
+                    $in[] = (int)(-1);
                 }
             } else {
                 if ($toponlyblock) {
-                    $in=array(0, -1);
+                    $in = array(0, -1);
                 } else {
-                    $in=0;
+                    $in = 0;
                 }
             }
             if (is_array($in)) {
@@ -463,17 +463,17 @@ 
                 $qb ->fromPrefix('system_blockmodule', 'm')
                     ->andWhere($eb->eq('m.block_id', 'b.bid'));
                 if (!empty($module_id)) {
-                    $in=array();
-                    $in[]=0;
-                    $in[]=(int)($module_id);
+                    $in = array();
+                    $in[] = 0;
+                    $in[] = (int)($module_id);
                     if ($toponlyblock) {
-                        $in[]=(int)(-1);
+                        $in[] = (int)(-1);
                     }
                 } else {
                     if ($toponlyblock) {
-                        $in=array(0, -1);
+                        $in = array(0, -1);
                     } else {
-                        $in=0;
+                        $in = 0;
                     }
                 }
                 if (is_array($in)) {
@@ -546,10 +546,10 @@ 
     {
         $ret = '';
         if ($position == 0) {
-            $ret = $contentdb . $content;
+            $ret = $contentdb.$content;
         } else {
             if ($position == 1) {
-                $ret = $content . $contentdb;
+                $ret = $content.$contentdb;
             }
         }
         return $ret;

xoops_lib/Xoops/Core/Kernel/Handlers/XoopsMembershipHandler.php

Spacing +1 added lines, -1 removed lines

@@ -96,7 +96,7 @@
             ->fromPrefix('system_usergroup', 'g')
             ->where($eb->eq('g.groupid', ':gid'))
             ->setParameter(':gid', $groupid, ParameterType::INTEGER);
-        if ($limit!=0 || $start!=0) {
+        if ($limit != 0 || $start != 0) {
             $qb->setFirstResult($start)
                 ->setMaxResults($limit);
         }

htdocs/admin.php

Spacing +9 added lines, -9 removed lines

@@ -20,7 +20,7 @@ 
  * @version     $Id$
  */
 
-include __DIR__ . '/mainfile.php';
+include __DIR__.'/mainfile.php';
 
 $xoops = Xoops::getInstance();
 $xoops->isAdminSide = true;
@@ -94,23 +94,23 @@ 
     //$rssurl[] = 'http://sourceforge.net/export/rss2_projnews.php?group_id=41586&rss_fulltext=1';
     $rssurl[] = 'https://xoops.org/modules/publisher/backend.php';
     $rssurl = array_unique(array_merge($rssurl, XoopsLocale::getAdminRssUrls()));
-    $rssfile = 'admin/rss/adminnews-' . $xoops->getConfig('locale');
+    $rssfile = 'admin/rss/adminnews-'.$xoops->getConfig('locale');
     $xoops->cache()->delete($rssfile);
-    $items = $xoops->cache()->cacheRead($rssfile, 'buildRssFeedCache', 24*60*60, $rssurl);
+    $items = $xoops->cache()->cacheRead($rssfile, 'buildRssFeedCache', 24 * 60 * 60, $rssurl);
     if ($items != '') {
         $ret = '<table class="outer width100">';
         foreach (array_keys($items) as $i) {
-            $ret .= '<tr class="head"><td><a href="' . htmlspecialchars($items[$i]['link']) . '" rel="external">';
-            $ret .= htmlspecialchars($items[$i]['title']) . '</a> (' . htmlspecialchars($items[$i]['pubdate']) . ')</td></tr>';
+            $ret .= '<tr class="head"><td><a href="'.htmlspecialchars($items[$i]['link']).'" rel="external">';
+            $ret .= htmlspecialchars($items[$i]['title']).'</a> ('.htmlspecialchars($items[$i]['pubdate']).')</td></tr>';
             if ($items[$i]['description'] != "") {
-                $ret .= '<tr><td class="odd">' . $items[$i]['description'];
+                $ret .= '<tr><td class="odd">'.$items[$i]['description'];
                 if (!empty($items[$i]['guid'])) {
-                    $ret .= '&nbsp;&nbsp;<a href="' . htmlspecialchars($items[$i]['guid']) . '" rel="external" title="">' . XoopsLocale::MORE . '</a>';
+                    $ret .= '&nbsp;&nbsp;<a href="'.htmlspecialchars($items[$i]['guid']).'" rel="external" title="">'.XoopsLocale::MORE.'</a>';
                 }
                 $ret .= '</td></tr>';
             } else {
                 if ($items[$i]['guid'] != "") {
-                    $ret .= '<tr><td class="even aligntop"></td><td colspan="2" class="odd"><a href="' . htmlspecialchars($items[$i]['guid']) . '" rel="external">' . _MORE . '</a></td></tr>';
+                    $ret .= '<tr><td class="even aligntop"></td><td colspan="2" class="odd"><a href="'.htmlspecialchars($items[$i]['guid']).'" rel="external">'._MORE.'</a></td></tr>';
                 }
             }
         }
@@ -137,7 +137,7 @@ 
                 for ($i = 0; $i < $count; $i++) {
                     // $_items[$i]['title'] = XoopsLocale::convert_encoding($_items[$i]['title'], XoopsLocale::getCharset(), 'UTF-8');
                     // $_items[$i]['description'] = XoopsLocale::convert_encoding($_items[$i]['description'], XoopsLocale::getCharset(), 'UTF-8');
-                    $items[(string)(strtotime($_items[$i]['pubdate'])) . "-" . (string)(++$cnt)] = $_items[$i];
+                    $items[(string)(strtotime($_items[$i]['pubdate']))."-".(string)(++$cnt)] = $_items[$i];
                 }
             } else {
                 echo $rss2parser->getErrors();

htdocs/modules/schematool/admin/schematool.php

Spacing +3 added lines, -3 removed lines

@@ -15,7 +15,7 @@ 
  * @author          Richard Griffith <richard@geekwright.com>
  */
 
-require __DIR__ . '/admin_header.php';
+require __DIR__.'/admin_header.php';
 
 /* --------------------------------------------------------------- */
 
@@ -62,7 +62,7 @@ 
         $schema = $schemaManager->createSchema();
 
         // invoke our RemovePrefixes visitor with list of module's tables
-        $visitor = new RemovePrefixes(\XoopsBaseConfig::get('db-prefix') . '_', $table_list);
+        $visitor = new RemovePrefixes(\XoopsBaseConfig::get('db-prefix').'_', $table_list);
         $schema->visit($visitor);
 
         // Get the schema we built with the RemovePrefixes visitor.
@@ -91,7 +91,7 @@ 
             }
         }
 
-        echo '<h2>' . _MI_SCHEMATOOL_EXPORT_SCHEMA . '</h2>';
+        echo '<h2>'._MI_SCHEMATOOL_EXPORT_SCHEMA.'</h2>';
         $yamldump = Yaml::dump($schemaArray, 5);
         //echo '<div contenteditable><pre>' . $yamldump . '</pre></div>';
         $schemadump = <<<EOT1

htdocs/modules/system/class/module.php

Spacing +128 added lines, -128 removed lines

@@ -107,20 +107,20 @@ 
                     $module->setInfo('warning_update', true);
                 }
                 if (XoopsLoad::fileExists(
-                    \XoopsBaseConfig::get('root-path') . '/modules/' . $module->getVar('dirname') . '/icons/logo_small.png'
+                    \XoopsBaseConfig::get('root-path').'/modules/'.$module->getVar('dirname').'/icons/logo_small.png'
                 )) {
                     $module->setInfo(
                         'logo_small',
-                        \XoopsBaseConfig::get('url') . '/modules/' . $module->getVar('dirname') . '/icons/logo_small.png'
+                        \XoopsBaseConfig::get('url').'/modules/'.$module->getVar('dirname').'/icons/logo_small.png'
                     );
                 } else {
-                    $module->setInfo('logo_small', \XoopsBaseConfig::get('url') . '/media/xoops/images/icons/16/default.png');
+                    $module->setInfo('logo_small', \XoopsBaseConfig::get('url').'/media/xoops/images/icons/16/default.png');
                 }
                 $module->setInfo('version', round($module->getVar('version') / 100, 2));
                 $module->setInfo('update', XoopsLocale::formatTimestamp($module->getVar('last_update'), 's'));
                 $module->setInfo(
                     'link_admin',
-                    \XoopsBaseConfig::get('url') . '/modules/' . $module->getVar('dirname') . '/' . $module->getInfo('adminindex')
+                    \XoopsBaseConfig::get('url').'/modules/'.$module->getVar('dirname').'/'.$module->getInfo('adminindex')
                 );
 
                 if ($module->getVar('isactive')) {
@@ -142,7 +142,7 @@ 
                 ) {
                     $module->setInfo(
                         'link_pref',
-                        \XoopsBaseConfig::get('url') . '/modules/system/admin.php?fct=preferences&op=showmod&mod='
+                        \XoopsBaseConfig::get('url').'/modules/system/admin.php?fct=preferences&op=showmod&mod='
                         . $module->getVar('mid')
                     );
                 }
@@ -167,7 +167,7 @@ 
         $ret = array();
         $i = 0;
         foreach ($this->modulesList as $file) {
-            if (XoopsLoad::fileExists(\XoopsBaseConfig::get('root-path') . '/modules/' . $file . '/xoops_version.php')) {
+            if (XoopsLoad::fileExists(\XoopsBaseConfig::get('root-path').'/modules/'.$file.'/xoops_version.php')) {
                 clearstatcache();
                 $file = trim($file);
                 if (!in_array($file, $this->modulesDirnames)) {
@@ -215,7 +215,7 @@ 
             $this->addTemporaryAutoloader($module);
             $install_script = $module->getInfo('onInstall');
             if ($install_script && trim($install_script) != '') {
-                XoopsLoad::loadFile($xoops->path('modules/' . $mod . '/' . trim($install_script)));
+                XoopsLoad::loadFile($xoops->path('modules/'.$mod.'/'.trim($install_script)));
             }
             $func = "xoops_module_pre_install_{$mod}";
             // If pre install function is defined, execute
@@ -236,20 +236,20 @@ 
                 $schema_file = $module->getInfo('schema');
                 $sql_file = $module->getInfo('sqlfile');
                 if (!empty($schema_file)) {
-                    $schema_file_path = \XoopsBaseConfig::get('root-path') . '/modules/' . $mod . '/' . $schema_file;
+                    $schema_file_path = \XoopsBaseConfig::get('root-path').'/modules/'.$mod.'/'.$schema_file;
                     if (!XoopsLoad::fileExists($schema_file_path)) {
                         $this->error[] =
                             sprintf(SystemLocale::EF_SQL_FILE_NOT_FOUND, "<strong>{$schema_file}</strong>");
                         return false;
                     }
-                    $importer = new ImportSchema(\XoopsBaseConfig::get('db-prefix') . '_');
+                    $importer = new ImportSchema(\XoopsBaseConfig::get('db-prefix').'_');
                     $importSchema = $importer->importSchemaArray(Yaml::read($schema_file_path));
                     $synchronizer = new SingleDatabaseSynchronizer($xoops->db());
                     $synchronizer->updateSchema($importSchema, true);
                 } elseif (is_array($sql_file) && !empty($sql_file[\XoopsBaseConfig::get('db-type')])) {
                     $xoops->deprecated('Install SQL files are deprecated since 2.6.0. Convert to portable Schemas');
 
-                    $sql_file_path = \XoopsBaseConfig::get('root-path') . '/modules/' . $mod . '/' . $sql_file[\XoopsBaseConfig::get('db-type')];
+                    $sql_file_path = \XoopsBaseConfig::get('root-path').'/modules/'.$mod.'/'.$sql_file[\XoopsBaseConfig::get('db-type')];
                     if (!XoopsLoad::fileExists($sql_file_path)) {
                         $this->error[] =
                             sprintf(SystemLocale::EF_SQL_FILE_NOT_FOUND, "<strong>{$sql_file_path}</strong>");
@@ -266,10 +266,10 @@ 
                             // [4] contains unprefixed table name
                             $prefixed_query = SqlUtility::prefixQuery($piece, $xoops->db()->prefix());
                             if (!$prefixed_query) {
-                                $this->error[]['sub'] = '<span class="red">' . sprintf(
+                                $this->error[]['sub'] = '<span class="red">'.sprintf(
                                     XoopsLocale::EF_INVALID_SQL,
-                                    '<strong>' . $piece . '</strong>'
-                                ) . '</span>';
+                                    '<strong>'.$piece.'</strong>'
+                                ).'</span>';
                                 break;
                             }
                             // check if the table name is reserved
@@ -279,7 +279,7 @@ 
                                     $result = $xoops->db()->query($prefixed_query[0]);
                                 } catch (Exception $e) {
                                     $xoops->events()->triggerEvent('core.exception', $e);
-                                    $result=false;
+                                    $result = false;
                                 }
 
                                 if (!$result) {
@@ -289,13 +289,13 @@ 
                                     if (!in_array($prefixed_query[4], $created_tables)) {
                                         $this->trace[]['sub'] = sprintf(
                                             XoopsLocale::SF_TABLE_CREATED,
-                                            '<strong>' . $xoops->db()->prefix($prefixed_query[4]) . '</strong>'
+                                            '<strong>'.$xoops->db()->prefix($prefixed_query[4]).'</strong>'
                                         );
                                         $created_tables[] = $prefixed_query[4];
                                     } else {
                                         $this->trace[]['sub'] = sprintf(
                                             XoopsLocale::SF_DATA_INSERTED_TO_TABLE,
-                                            '<strong>' . $xoops->db()->prefix($prefixed_query[4]) . '</strong>'
+                                            '<strong>'.$xoops->db()->prefix($prefixed_query[4]).'</strong>'
                                         );
                                     }
                                 }
@@ -303,7 +303,7 @@ 
                                 // the table name is reserved, so halt the installation
                                 $this->error[]['sub'] = sprintf(
                                     SystemLocale::EF_TABLE_IS_RESERVED,
-                                    '<strong>' . $prefixed_query[4] . '</strong>'
+                                    '<strong>'.$prefixed_query[4].'</strong>'
                                 );
                                 break;
                             }
@@ -313,7 +313,7 @@ 
                         if (count($this->error) > 0) {
                             foreach ($created_tables as $table) {
                                 try {
-                                    $xoops->db()->query('DROP TABLE ' . $xoops->db()->prefix($table));
+                                    $xoops->db()->query('DROP TABLE '.$xoops->db()->prefix($table));
                                 } catch (Exception $e) {
                                     $xoops->events()->triggerEvent('core.exception', $e);
                                 }
@@ -328,25 +328,25 @@ 
                 if (!$module_handler->insertModule($module)) {
                     $this->error[] = sprintf(
                         XoopsLocale::EF_NOT_INSERTED_TO_DATABASE,
-                        '<strong>' . $module->getVar('name') . '</strong>'
+                        '<strong>'.$module->getVar('name').'</strong>'
                     );
                     foreach ($created_tables as $ct) {
                         try {
-                            $xoops->db()->query('DROP TABLE ' . $xoops->db()->prefix($ct));
+                            $xoops->db()->query('DROP TABLE '.$xoops->db()->prefix($ct));
                         } catch (Exception $e) {
                             $xoops->events()->triggerEvent('core.exception', $e);
                         }
                     }
-                    $this->error[] = sprintf(XoopsLocale::EF_NOT_INSTALLED, "<strong>" . $module->name() . "</strong>");
+                    $this->error[] = sprintf(XoopsLocale::EF_NOT_INSTALLED, "<strong>".$module->name()."</strong>");
                     $this->error[] = XoopsLocale::C_ERRORS;
                     unset($module);
                     unset($created_tables);
                     return false;
                 }
                 unset($created_tables);
-                $this->trace[] = XoopsLocale::S_DATA_INSERTED . sprintf(
+                $this->trace[] = XoopsLocale::S_DATA_INSERTED.sprintf(
                     SystemLocale::F_MODULE_ID,
-                    '<strong>' . $module->getVar('mid') . '</strong>'
+                    '<strong>'.$module->getVar('mid').'</strong>'
                 );
                 $xoops->db()->beginTransaction();
                 // install Templates
@@ -378,14 +378,14 @@ 
                         $mperm->setVar('gperm_name', 'module_admin');
                         $mperm->setVar('gperm_modid', 1);
                         if (!$gperm_handler->insert($mperm)) {
-                            $this->trace[]['sub'] = '<span class="red">' . sprintf(
+                            $this->trace[]['sub'] = '<span class="red">'.sprintf(
                                 SystemLocale::EF_GROUP_ID_ADMIN_ACCESS_RIGHT_NOT_ADDED,
-                                '<strong>' . $mygroup . '</strong>'
-                            ) . '</span>';
+                                '<strong>'.$mygroup.'</strong>'
+                            ).'</span>';
                         } else {
                             $this->trace[]['sub'] = sprintf(
                                 SystemLocale::SF_GROUP_ID_ADMIN_ACCESS_RIGHT_ADDED,
-                                '<strong>' . $mygroup . '</strong>'
+                                '<strong>'.$mygroup.'</strong>'
                             );
                         }
                         unset($mperm);
@@ -396,14 +396,14 @@ 
                     $mperm->setVar('gperm_name', 'module_read');
                     $mperm->setVar('gperm_modid', 1);
                     if (!$gperm_handler->insert($mperm)) {
-                        $this->trace[]['sub'] = '<span class="red">' . sprintf(
+                        $this->trace[]['sub'] = '<span class="red">'.sprintf(
                             SystemLocale::EF_GROUP_ID_USER_ACCESS_RIGHT_NOT_ADDED,
-                            '<strong>' . $mygroup . '</strong>'
-                        ) . '</span>';
+                            '<strong>'.$mygroup.'</strong>'
+                        ).'</span>';
                     } else {
                         $this->trace[]['sub'] = sprintf(
                             SystemLocale::SF_GROUP_ID_USER_ACCESS_RIGHT_ADDED,
-                            '<strong>' . $mygroup . '</strong>'
+                            '<strong>'.$mygroup.'</strong>'
                         );
                     }
                     unset($mperm);
@@ -415,12 +415,12 @@ 
                         $bperm->setVar('gperm_modid', 1);
                         if (!$gperm_handler->insert($bperm)) {
                             $this->trace[]['sub'] = '<span class="red">'
-                            . SystemLocale::E_BLOCK_ACCESS_NOT_ADDED . ' Block ID: <strong>'
-                            . $blc . '</strong> Group ID: <strong>' . $mygroup . '</strong></span>';
+                            . SystemLocale::E_BLOCK_ACCESS_NOT_ADDED.' Block ID: <strong>'
+                            . $blc.'</strong> Group ID: <strong>'.$mygroup.'</strong></span>';
                         } else {
                             $this->trace[]['sub'] = SystemLocale::S_BLOCK_ACCESS_ADDED
-                                . sprintf(SystemLocale::F_BLOCK_ID, "<strong>" . $blc . "</strong>")
-                                . sprintf(SystemLocale::F_GROUP_ID, "<strong>" . $mygroup . "</strong>");
+                                . sprintf(SystemLocale::F_BLOCK_ID, "<strong>".$blc."</strong>")
+                                . sprintf(SystemLocale::F_GROUP_ID, "<strong>".$mygroup."</strong>");
                         }
                         unset($bperm);
                     }
@@ -444,7 +444,7 @@ 
 
                 $this->trace[] = sprintf(
                     XoopsLocale::SF_INSTALLED,
-                    '<strong>' . $module->getVar('name', 's') . '</strong>'
+                    '<strong>'.$module->getVar('name', 's').'</strong>'
                 );
                 unset($blocks);
 
@@ -456,8 +456,8 @@ 
         } else {
             $this->error[] = sprintf(
                 XoopsLocale::EF_NOT_INSTALLED,
-                '<strong>' . $mod . '</strong>'
-            ) . "&nbsp;" . XoopsLocale::C_ERRORS;
+                '<strong>'.$mod.'</strong>'
+            )."&nbsp;".XoopsLocale::C_ERRORS;
             return false;
         }
         return false;
@@ -480,22 +480,22 @@ 
         if ($module->getVar('dirname') === 'system') {
             $this->error[] = sprintf(
                 XoopsLocale::EF_NOT_UNINSTALLED,
-                '<strong>' . $module->getVar('name') . '</strong>'
-            ) . "&nbsp;" . XoopsLocale::C_ERRORS;
-            $this->error[] = " - " . SystemLocale::E_SYSTEM_MODULE_CANNOT_BE_DEACTIVATED;
+                '<strong>'.$module->getVar('name').'</strong>'
+            )."&nbsp;".XoopsLocale::C_ERRORS;
+            $this->error[] = " - ".SystemLocale::E_SYSTEM_MODULE_CANNOT_BE_DEACTIVATED;
             return false;
         } elseif ($module->getVar('dirname') == $xoops->getConfig('startpage')) {
             $this->error[] = sprintf(
                 XoopsLocale::EF_NOT_UNINSTALLED,
-                '<strong>' . $module->getVar('name') . '</strong>'
-            ) . "&nbsp;" . XoopsLocale::C_ERRORS;
-            $this->error[] = " - " . SystemLocale::E_THIS_MODULE_IS_SET_AS_DEFAULT_START_PAGE;
+                '<strong>'.$module->getVar('name').'</strong>'
+            )."&nbsp;".XoopsLocale::C_ERRORS;
+            $this->error[] = " - ".SystemLocale::E_THIS_MODULE_IS_SET_AS_DEFAULT_START_PAGE;
             return false;
         } else {
             // Load module specific install script if any
             $uninstall_script = $module->getInfo('onUninstall');
             if ($uninstall_script && trim($uninstall_script) != '') {
-                XoopsLoad::loadFile($xoops->path('modules/' . $mod . '/' . trim($uninstall_script)));
+                XoopsLoad::loadFile($xoops->path('modules/'.$mod.'/'.trim($uninstall_script)));
             }
             $func = "xoops_module_pre_uninstall_{$mod}";
             // If pre uninstall function is defined, execute
@@ -505,8 +505,8 @@ 
                     $this->error[] = sprintf(XoopsLocale::EF_NOT_EXECUTED, $func);
                     $this->error[] = sprintf(
                         XoopsLocale::EF_NOT_UNINSTALLED,
-                        '<strong>' . $module->getVar('name') . '</strong>'
-                    ) . "&nbsp;" . XoopsLocale::C_ERRORS;
+                        '<strong>'.$module->getVar('name').'</strong>'
+                    )."&nbsp;".XoopsLocale::C_ERRORS;
                     $this->error = array_merge($this->error, $module->getErrors());
                     return false;
                 } else {
@@ -540,13 +540,13 @@ 
                             $toSchema->dropTable($xoops->db()->prefix($table));
                             $this->trace[]['sub'] = sprintf(
                                 XoopsLocale::SF_TABLE_DROPPED,
-                                '<strong>' . $xoops->db()->prefix($table) . '</strong>'
+                                '<strong>'.$xoops->db()->prefix($table).'</strong>'
                             );
                         } else {
-                            $this->trace[]['sub'] = '<span class="red">' . sprintf(
+                            $this->trace[]['sub'] = '<span class="red">'.sprintf(
                                 XoopsLocale::EF_TABLE_DROP_NOT_ALLOWED,
-                                '<strong>' . $xoops->db()->prefix($table) . '</strong>'
-                            ) . '</span>';
+                                '<strong>'.$xoops->db()->prefix($table).'</strong>'
+                            ).'</span>';
                         }
                     }
                     $synchronizer = new SingleDatabaseSynchronizer($xoops->db());
@@ -556,7 +556,7 @@ 
                 // delete permissions if any
                 $gperm_handler = $xoops->getHandlerGroupPermission();
                 if (false === $gperm_handler->deleteByModule($module->getVar('mid'))) {
-                    $this->trace[] = '<span class="red">' . SystemLocale::E_GROUP_PERMISSIONS_NOT_DELETED . '</span>';
+                    $this->trace[] = '<span class="red">'.SystemLocale::E_GROUP_PERMISSIONS_NOT_DELETED.'</span>';
                 } else {
                     $this->trace[] = SystemLocale::S_GROUP_PERMISSIONS_DELETED;
                 }
@@ -565,7 +565,7 @@ 
                 $this->deleteConfigs($module);
 
                 // execute module specific install script if any
-                $func = 'xoops_module_uninstall_' . $mod;
+                $func = 'xoops_module_uninstall_'.$mod;
                 if (function_exists($func)) {
                     $result = $func($module);
                     if (!$result) {
@@ -578,7 +578,7 @@ 
                 }
                 $this->trace[] = sprintf(
                     XoopsLocale::SF_UNINSTALLED,
-                    '<strong>' . $module->getVar('name') . '</strong>'
+                    '<strong>'.$module->getVar('name').'</strong>'
                 );
                 $xoops->events()->triggerEvent('system.module.uninstall', $module);
                 return $module;
@@ -609,14 +609,14 @@ 
         $this->addTemporaryAutoloader($module);
 
         if (!$module_handler->insertModule($module)) {
-            $this->error[] = sprintf(XoopsLocale::EF_NOT_UPDATED, "<strong>" . $module->getVar('name') . "</strong>");
+            $this->error[] = sprintf(XoopsLocale::EF_NOT_UPDATED, "<strong>".$module->getVar('name')."</strong>");
             return false;
         } else {
             // execute module specific preupdate script if any
             $update_script = $module->getInfo('onUpdate');
             if (false != $update_script && trim($update_script) != '') {
-                XoopsLoad::loadFile($xoops->path('modules/' . $mod . '/' . trim($update_script)));
-                $func = 'xoops_module_pre_update_' . $mod;
+                XoopsLoad::loadFile($xoops->path('modules/'.$mod.'/'.trim($update_script)));
+                $func = 'xoops_module_pre_update_'.$mod;
                 if (function_exists($func)) {
                     $result = $func($module, $prev_version);
                     if (!$result) {
@@ -632,13 +632,13 @@ 
             // update schema
             $schema_file = $module->getInfo('schema');
             if (!empty($schema_file)) {
-                $schema_file_path = \XoopsBaseConfig::get('root-path') . '/modules/' . $mod . '/' . $schema_file;
+                $schema_file_path = \XoopsBaseConfig::get('root-path').'/modules/'.$mod.'/'.$schema_file;
                 if (!XoopsLoad::fileExists($schema_file_path)) {
                     $this->error[] =
                         sprintf(SystemLocale::EF_SQL_FILE_NOT_FOUND, "<strong>{$schema_file}</strong>");
                     return false;
                 }
-                $importer = new ImportSchema(\XoopsBaseConfig::get('db-prefix') . '_');
+                $importer = new ImportSchema(\XoopsBaseConfig::get('db-prefix').'_');
                 $importSchema = $importer->importSchemaArray(Yaml::read($schema_file_path));
                 $synchronizer = new SingleDatabaseSynchronizer($xoops->db());
                 $synchronizer->updateSchema($importSchema, true);
@@ -665,8 +665,8 @@ 
             // execute module specific update script if any
             $update_script = $module->getInfo('onUpdate');
             if (false != $update_script && trim($update_script) != '') {
-                XoopsLoad::loadFile($xoops->path('modules/' . $mod . '/' . trim($update_script)));
-                $func = 'xoops_module_update_' . $mod;
+                XoopsLoad::loadFile($xoops->path('modules/'.$mod.'/'.trim($update_script)));
+                $func = 'xoops_module_update_'.$mod;
                 if (function_exists($func)) {
                     $result = $func($module, $prev_version);
                     if (!$result) {
@@ -678,7 +678,7 @@ 
                     }
                 }
             }
-            $this->trace[] = sprintf(XoopsLocale::SF_UPDATED, '<strong>' . $module->getVar('name', 's') . '</strong>');
+            $this->trace[] = sprintf(XoopsLocale::SF_UPDATED, '<strong>'.$module->getVar('name', 's').'</strong>');
             return $module;
         }
     }
@@ -699,10 +699,10 @@ 
         switch ($type) {
             case 'blocks':
             case 'admin':
-                $path = $xoops->path('modules/' . $dirname . '/templates/' . $type . '/' . $template);
+                $path = $xoops->path('modules/'.$dirname.'/templates/'.$type.'/'.$template);
                 break;
             default:
-                $path = $xoops->path('modules/' . $dirname . '/templates/' . $template);
+                $path = $xoops->path('modules/'.$dirname.'/templates/'.$template);
                 break;
         }
         if (!XoopsLoad::fileExists($path)) {
@@ -755,39 +755,39 @@ 
                     $tplfile->setVar('tpl_file', $tpl['file']);
                     $tplfile->setVar('tpl_desc', $tpl['description']);
                     if (!$tplfile_handler->insertTpl($tplfile)) {
-                        $this->trace[]['sub'] = '<span class="red">' . sprintf(
+                        $this->trace[]['sub'] = '<span class="red">'.sprintf(
                             SystemLocale::EF_TEMPLATE_NOT_ADDED_TO_DATABASE,
-                            '<strong>' . $tpl['file'] . '</strong>'
-                        ) . '</span>';
+                            '<strong>'.$tpl['file'].'</strong>'
+                        ).'</span>';
                     } else {
                         $newid = $tplfile->getVar('tpl_id');
                         $this->trace[]['sub'] = sprintf(
                             SystemLocale::SF_TEMPLATE_ADDED,
-                            '<strong>' . $tpl['file'] . '</strong>'
+                            '<strong>'.$tpl['file'].'</strong>'
                         );
                         if ($module->getVar('dirname') === 'system') {
                             if (!$xoops->templateTouch($newid)) {
-                                $this->trace[]['sub'] = '<span class="red">' . sprintf(
+                                $this->trace[]['sub'] = '<span class="red">'.sprintf(
                                     SystemLocale::EF_TEMPLATE_NOT_RECOMPILED,
-                                    '<strong>' . $tpl['file'] . '</strong>'
-                                ) . '</span>';
+                                    '<strong>'.$tpl['file'].'</strong>'
+                                ).'</span>';
                             } else {
                                 $this->trace[]['sub'] = sprintf(
                                     SystemLocale::SF_TEMPLATE_RECOMPILED,
-                                    '<strong>' . $tpl['file'] . '</strong>'
+                                    '<strong>'.$tpl['file'].'</strong>'
                                 );
                             }
                         } else {
                             if ($xoops->config['template_set'] === 'default') {
                                 if (!$xoops->templateTouch($newid)) {
-                                    $this->trace[]['sub'] = '<span class="red">' . sprintf(
+                                    $this->trace[]['sub'] = '<span class="red">'.sprintf(
                                         SystemLocale::EF_TEMPLATE_NOT_RECOMPILED,
-                                        '<strong>' . $tpl['file'] . '</strong>'
-                                    ) . '</span>';
+                                        '<strong>'.$tpl['file'].'</strong>'
+                                    ).'</span>';
                                 } else {
                                     $this->trace[]['sub'] = sprintf(
                                         SystemLocale::SF_TEMPLATE_RECOMPILED,
-                                        '<strong>' . $tpl['file'] . '</strong>'
+                                        '<strong>'.$tpl['file'].'</strong>'
                                     );
                                 }
                             }
@@ -795,10 +795,10 @@ 
                     }
                     unset($tpldata);
                 } else {
-                    $this->trace[]['sub'] = '<span class="red">' . sprintf(
+                    $this->trace[]['sub'] = '<span class="red">'.sprintf(
                         SystemLocale::EF_TEMPLATE_NOT_DELETED,
-                        '<strong>' . $tpl['file'] . '</strong>'
-                    ) . '</span>';
+                        '<strong>'.$tpl['file'].'</strong>'
+                    ).'</span>';
                 }
             }
         }
@@ -887,17 +887,17 @@ 
                     $block_obj[0]->setVar('last_modified', time());
 
                     if (!$block_handler->insert($block_obj[0])) {
-                        $this->trace[]['sub'] = '<span class="red">' . sprintf(
+                        $this->trace[]['sub'] = '<span class="red">'.sprintf(
                             XoopsLocale::EF_NOT_UPDATED,
                             $block_obj[0]->getVar('name')
-                        ) . '</span>';
+                        ).'</span>';
                     } else {
                         $this->trace[]['sub'] = sprintf(
                             SystemLocale::SF_BLOCK_UPDATED,
-                            '<strong>' . $block_obj[0]->getVar('name')
-                        ) . '</strong>' . sprintf(
+                            '<strong>'.$block_obj[0]->getVar('name')
+                        ).'</strong>'.sprintf(
                             SystemLocale::F_BLOCK_ID,
-                            '<strong>' . $block_obj[0]->getVar('bid') . '</strong>'
+                            '<strong>'.$block_obj[0]->getVar('bid').'</strong>'
                         );
 
                         if (0 == $blockmodulelink_handler->getCount(new Criteria('block_id', $block_obj[0]->getVar('bid')))) {
@@ -926,38 +926,38 @@ 
                             $tplfile_new->setVar('tpl_lastmodified', time());
                             $tplfile_new->setVar('tpl_lastimported', 0);
                             if (!$tplfile_handler->insertTpl($tplfile_new)) {
-                                $this->trace[]['sub'] = '<span class="red">' . sprintf(
+                                $this->trace[]['sub'] = '<span class="red">'.sprintf(
                                     SystemLocale::EF_TEMPLATE_NOT_UPDATED,
-                                    '<strong>' . $block['template'] . '</strong>'
-                                ) . '</span>';
+                                    '<strong>'.$block['template'].'</strong>'
+                                ).'</span>';
                             } else {
                                 $this->trace[]['sub'] = sprintf(
                                     SystemLocale::SF_TEMPLATE_UPDATED,
-                                    '<strong>' . $block['template'] . '</strong>'
+                                    '<strong>'.$block['template'].'</strong>'
                                 );
                                 if ($module->getVar('dirname') === 'system') {
                                     if (!$xoops->templateTouch($tplfile_new->getVar('tpl_id'))) {
-                                        $this->trace[]['sub'] = '<span class="red">' . sprintf(
+                                        $this->trace[]['sub'] = '<span class="red">'.sprintf(
                                             SystemLocale::EF_TEMPLATE_NOT_RECOMPILED,
-                                            '<strong>' . $block['template'] . '</strong>'
-                                        ) . '</span>';
+                                            '<strong>'.$block['template'].'</strong>'
+                                        ).'</span>';
                                     } else {
                                         $this->trace[]['sub'] = sprintf(
                                             SystemLocale::SF_TEMPLATE_RECOMPILED,
-                                            '<strong>' . $block['template'] . '</strong>'
+                                            '<strong>'.$block['template'].'</strong>'
                                         );
                                     }
                                 } else {
                                     if ($xoops->config['template_set'] === 'default') {
                                         if (!$xoops->templateTouch($tplfile_new->getVar('tpl_id'))) {
-                                            $this->trace[]['sub'] = '<span class="red">' . sprintf(
+                                            $this->trace[]['sub'] = '<span class="red">'.sprintf(
                                                 SystemLocale::EF_TEMPLATE_NOT_RECOMPILED,
-                                                '<strong>' . $block['template'] . '</strong>'
-                                            ) . '</span>';
+                                                '<strong>'.$block['template'].'</strong>'
+                                            ).'</span>';
                                         } else {
                                             $this->trace[]['sub'] = sprintf(
                                                 SystemLocale::SF_TEMPLATE_RECOMPILED,
-                                                '<strong>' . $block['template'] . '</strong>'
+                                                '<strong>'.$block['template'].'</strong>'
                                             );
                                         }
                                     }
@@ -975,20 +975,20 @@ 
                 || !in_array($block->getVar('func_file'), $funcfiles)
             ) {
                 if (!$block_handler->delete($block)) {
-                    $this->trace[]['sub'] = '<span class="red">' . sprintf(
+                    $this->trace[]['sub'] = '<span class="red">'.sprintf(
                         SystemLocale::EF_BLOCK_NOT_DELETED,
-                        "<strong>" . $block->getVar('name') . "</strong>"
-                    ) . sprintf(
+                        "<strong>".$block->getVar('name')."</strong>"
+                    ).sprintf(
                         SystemLocale::F_BLOCK_ID,
-                        "<strong>" . $block->getVar('bid') . "</strong>"
-                    ) . '</span>';
+                        "<strong>".$block->getVar('bid')."</strong>"
+                    ).'</span>';
                 } else {
                     $this->trace[]['sub'] = sprintf(
                         SystemLocale::SF_BLOCK_DELETED,
-                        '<strong>' . $block->getVar('name') . '</strong>'
-                    ) . '&nbsp;' . sprintf(
+                        '<strong>'.$block->getVar('name').'</strong>'
+                    ).'&nbsp;'.sprintf(
                         SystemLocale::F_BLOCK_ID,
-                        '<strong>' . $block->getVar('bid') . '</strong>'
+                        '<strong>'.$block->getVar('bid').'</strong>'
                     );
                     if ($block->getVar('template') != '') {
                         $tplfiles = $tplfile_handler->find(null, 'block', $block->getVar('bid'));
@@ -998,11 +998,11 @@ 
                                 if (!$tplfile_handler->deleteTpl($tplfile)) {
                                     $this->trace[]['sub'] = '<span class="red">'
                                         . SystemLocale::E_BLOCK_TEMPLATE_DEPRECATED_NOT_REMOVED
-                                        . '(ID: <strong>' . $tplfile->getVar('tpl_id') . '</strong>)</span>';
+                                        . '(ID: <strong>'.$tplfile->getVar('tpl_id').'</strong>)</span>';
                                 } else {
                                     $this->trace[]['sub'] = sprintf(
                                         SystemLocale::SF_BLOCK_TEMPLATE_DEPRECATED,
-                                        "<strong>" . $tplfile->getVar('tpl_file') . "</strong>"
+                                        "<strong>".$tplfile->getVar('tpl_file')."</strong>"
                                     );
                                 }
                             }
@@ -1031,20 +1031,20 @@ 
             /* @var $block XoopsBlock */
             foreach ($blocks as $block) {
                 if (false === $block_handler->deleteBlock($block)) {
-                    $this->trace[]['sub'] = '<span class="red">' . sprintf(
+                    $this->trace[]['sub'] = '<span class="red">'.sprintf(
                         SystemLocale::EF_BLOCK_NOT_DELETED,
-                        "<strong>" . $block->getVar('name') . "</strong>"
-                    ) . sprintf(
+                        "<strong>".$block->getVar('name')."</strong>"
+                    ).sprintf(
                         SystemLocale::F_BLOCK_ID,
-                        "<strong>" . $block->getVar('bid') . "</strong>"
-                    ) . '</span>';
+                        "<strong>".$block->getVar('bid')."</strong>"
+                    ).'</span>';
                 } else {
                     $this->trace[]['sub'] = sprintf(
                         SystemLocale::SF_BLOCK_DELETED,
-                        "<strong>" . $block->getVar('name') . "</strong>"
-                    ) . sprintf(
+                        "<strong>".$block->getVar('name')."</strong>"
+                    ).sprintf(
                         SystemLocale::F_BLOCK_ID,
-                        "<strong>" . $block->getVar('bid') . "</strong>"
+                        "<strong>".$block->getVar('bid')."</strong>"
                     );
                 }
                 if ($block->getVar('template') != '') {
@@ -1052,20 +1052,20 @@ 
                     /* @var $template XoopsTplFile */
                     foreach ($templates as $template) {
                         if (!$tplfile_handler->delete($template)) {
-                            $this->trace[]['sub'] = '<span class="red">' . sprintf(
+                            $this->trace[]['sub'] = '<span class="red">'.sprintf(
                                 SystemLocale::EF_BLOCK_TEMPLATE_NOT_DELETED,
                                 $template->getVar('tpl_file')
-                            ) . sprintf(
+                            ).sprintf(
                                 SystemLocale::F_TEMPLATE_ID,
-                                "<strong>" . $template->getVar('tpl_id') . "</strong>"
-                            ) . '</span>';
+                                "<strong>".$template->getVar('tpl_id')."</strong>"
+                            ).'</span>';
                         } else {
                             $this->trace[]['sub'] = sprintf(
                                 SystemLocale::SF_BLOCK_TEMPLATE_DELETED,
-                                "<strong>" . $template->getVar('tpl_file') . "</strong>"
-                            ) . sprintf(
+                                "<strong>".$template->getVar('tpl_file')."</strong>"
+                            ).sprintf(
                                 SystemLocale::F_TEMPLATE_ID,
-                                "<strong>" . $template->getVar('tpl_id') . "</strong>"
+                                "<strong>".$template->getVar('tpl_id')."</strong>"
                             );
                         }
                     }
@@ -1095,7 +1095,7 @@ 
                 if (!$config_handler->deleteConfig($config)) {
                     $this->trace[]['sub'] = '<span class="red">'
                         . SystemLocale::E_CONFIG_DATA_NOT_DELETED
-                        . sprintf(SystemLocale::F_CONFIG_ID, "<strong>" . $config->getVar('conf_id') . "</strong>")
+                        . sprintf(SystemLocale::F_CONFIG_ID, "<strong>".$config->getVar('conf_id')."</strong>")
                         . '</span>';
                     // save the name of config failed to delete for later use
                     $this->config_delng[] = $config->getVar('conf_name');
@@ -1104,7 +1104,7 @@ 
                     $this->config_old[$config->getVar('conf_name')]['formtype'] = $config->getVar('conf_formtype');
                     $this->config_old[$config->getVar('conf_name')]['valuetype'] = $config->getVar('conf_valuetype');
                     $this->trace[]['sub'] = SystemLocale::S_CONFIG_DATA_DELETED
-                        . sprintf(SystemLocale::F_CONFIG_ID, "<strong>" . $config->getVar('conf_id') . "</strong>");
+                        . sprintf(SystemLocale::F_CONFIG_ID, "<strong>".$config->getVar('conf_id')."</strong>");
                 }
             }
         }
@@ -1170,7 +1170,7 @@ 
                             $confop_msgs .= ' <strong>'
                                 . \Xoops\Locale::translate($key, $module->getVar('dirname'))
                                 . '</strong> ';
-                            $confop_msgs .= XoopsLocale::C_VALUE . ' <strong>' . $value . '</strong> ';
+                            $confop_msgs .= XoopsLocale::C_VALUE.' <strong>'.$value.'</strong> ';
                             unset($confop);
                         }
                     }
@@ -1178,11 +1178,11 @@ 
                     if (false != $config_handler->insertConfig($confobj)) {
                         $this->trace[]['sub'] = sprintf(
                             SystemLocale::SF_CONFIG_ADDED,
-                            "<strong>" . $config['name'] . "</strong>"
-                        ) . $confop_msgs;
+                            "<strong>".$config['name']."</strong>"
+                        ).$confop_msgs;
                     } else {
                         $this->trace[]['sub'] = '<span class="red">'
-                            . sprintf(SystemLocale::EF_CONFIG_NOT_ADDED, "<strong>" . $config['name'] . "</strong>")
+                            . sprintf(SystemLocale::EF_CONFIG_NOT_ADDED, "<strong>".$config['name']."</strong>")
                             . '</span>';
                     }
                     unset($confobj);
@@ -1210,7 +1210,7 @@ 
             return;
         }
         $xoops = Xoops::getInstance();
-        $base_dir = $xoops->path('modules/' . $module->getVar('dirname') . '/src');
+        $base_dir = $xoops->path('modules/'.$module->getVar('dirname').'/src');
         $loader = new Psr4ClassLoader;
         $loader->register();
         $loader->addNamespace($prefix, $base_dir);

xoops_lib/Xoops/Core/Database/Schema/PrefixStripper.php

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
         try {
             $name = $table->getName();
             $len = strlen($this->xPrefix);
-            if (substr_compare($name, $this->xPrefix, 0, $len)===0) {
+            if (substr_compare($name, $this->xPrefix, 0, $len) === 0) {
                 $name = substr($name, $len);
                 if (empty($this->tableList) || in_array($name, $this->tableList)) {
                     $idGeneratorType = 0; // how should we handle this?

xoops_lib/Xoops/Core/Database/Schema/ImportSchema.php

Spacing +2 added lines, -2 removed lines

@@ -82,9 +82,9 @@
      */
     public function importTables(array $tableArray)
     {
-        $tables=array();
+        $tables = array();
         foreach ($tableArray as $name => $tabledef) {
-            $tableName = $this->xPrefix . $name;
+            $tableName = $this->xPrefix.$name;
             $columns = array();
             $indexes = array();
             $fkConstraints = array();

htdocs/modules/menus/class/menu.php

Spacing +8 added lines, -8 removed lines

@@ -62,26 +62,26 @@
      */
     public function updateWeights(MenusMenu $obj)
     {
-        $sql = "UPDATE " . $this->table
+        $sql = "UPDATE ".$this->table
         . " SET weight = weight+1"
-        . " WHERE weight >= " . $obj->getVar('weight')
-        . " AND id <> " . $obj->getVar('id')
+        . " WHERE weight >= ".$obj->getVar('weight')
+        . " AND id <> ".$obj->getVar('id')
         /*. " AND pid = " . $obj->getVar('pid')*/
-        . " AND mid = " . $obj->getVar('mid')
+        . " AND mid = ".$obj->getVar('mid')
         ;
         $originalForce = $this->db2->getForce();
         $this->db2->setForce(true);
         $this->db2->query($sql);
 
-        $sql = "SELECT id FROM " . $this->table
-        . " WHERE mid = " . $obj->getVar('mid')
+        $sql = "SELECT id FROM ".$this->table
+        . " WHERE mid = ".$obj->getVar('mid')
         /*. " AND pid = " . $obj->getVar('pid')*/
         . " ORDER BY weight ASC"
         ;
         $result = $this->db2->query($sql);
-        $i = 1;  //lets start at 1 please!
+        $i = 1; //lets start at 1 please!
         while (false !== (list($id) = $result->fetch(FetchMode::NUMERIC))) {
-            $sql = "UPDATE " . $this->table
+            $sql = "UPDATE ".$this->table
             . " SET weight = {$i}"
             . " WHERE id = {$id}"
             ;

htdocs/modules/comments/admin/main.php

Spacing +17 added lines, -17 removed lines

@@ -25,7 +25,7 @@ 
  * @package         comments
  */
 
-include __DIR__ . '/header.php';
+include __DIR__.'/header.php';
 
 // Get main instance
 $xoops = Xoops::getInstance();
@@ -46,9 +46,9 @@ 
 $status_array =
     array(Comments::STATUS_PENDING => _MD_COMMENTS_PENDING, Comments::STATUS_ACTIVE => _MD_COMMENTS_ACTIVE, Comments::STATUS_HIDDEN => _MD_COMMENTS_HIDDEN);
 $status_array2 = array(
-    Comments::STATUS_PENDING => '<span style="text-decoration: none; font-weight: bold; color: #008000;">' . _MD_COMMENTS_PENDING . '</span>',
-    Comments::STATUS_ACTIVE => '<span style="text-decoration: none; font-weight: bold; color: #ff0000;">' . _MD_COMMENTS_ACTIVE . '</span>',
-    Comments::STATUS_HIDDEN => '<span style="text-decoration: none; font-weight: bold; color: #0000ff;">' . _MD_COMMENTS_HIDDEN . '</span>'
+    Comments::STATUS_PENDING => '<span style="text-decoration: none; font-weight: bold; color: #008000;">'._MD_COMMENTS_PENDING.'</span>',
+    Comments::STATUS_ACTIVE => '<span style="text-decoration: none; font-weight: bold; color: #ff0000;">'._MD_COMMENTS_ACTIVE.'</span>',
+    Comments::STATUS_HIDDEN => '<span style="text-decoration: none; font-weight: bold; color: #0000ff;">'._MD_COMMENTS_HIDDEN.'</span>'
 );
 $start = 0;
 $status_array[0] = _AM_COMMENTS_FORM_ALL_STATUS;
@@ -63,7 +63,7 @@ 
 $module_handler = $xoops->getHandlerModule();
 $available_plugins = \Xoops\Module\Plugin::getPlugins('comments');
 if (!empty($available_plugins)) {
-    $criteria = new Criteria('dirname', "('" . implode("','", array_keys($available_plugins)).  "')", 'IN');
+    $criteria = new Criteria('dirname', "('".implode("','", array_keys($available_plugins))."')", 'IN');
     $module_array = $module_handler->getNameList($criteria);
 }
 
@@ -81,7 +81,7 @@ 
                 /* @var $plugin CommentsPluginInterface */
                 $module = $xoops->getModuleById($comment->getVar('modid'));
                 $plugin = Xoops\Module\Plugin::getPlugin($module->getVar('dirname'), 'comments');
-                header('Location: ' . \XoopsBaseConfig::get('url') . '/modules/' . $module->getVar('dirname') . '/' . $plugin->pageName() . '?' . $plugin->itemName() . '=' . $comment->getVar('itemid') . '&id=' . $comment->getVar('id') . '&rootid=' . $comment->getVar('rootid') . '&mode=thread&' . str_replace('&amp;', '&', $comment->getVar('exparams')) . '#comment' . $comment->getVar('id'));
+                header('Location: '.\XoopsBaseConfig::get('url').'/modules/'.$module->getVar('dirname').'/'.$plugin->pageName().'?'.$plugin->itemName().'='.$comment->getVar('itemid').'&id='.$comment->getVar('id').'&rootid='.$comment->getVar('rootid').'&mode=thread&'.str_replace('&amp;', '&', $comment->getVar('exparams')).'#comment'.$comment->getVar('id'));
                 exit();
             }
         }
@@ -257,7 +257,7 @@ 
         }
 
         $url = $helper->url('admin/main.php');
-        $form = '<form class="form-inline" action="' . $url . '" method="post">
+        $form = '<form class="form-inline" action="'.$url.'" method="post">
                 <select class="span2" name="comments_module">';
 
         foreach ($module_array as $k => $v) {
@@ -265,7 +265,7 @@ 
             if ($k == $module) {
                 $sel = ' selected="selected"';
             }
-            $form .= '<option value="' . $k . '"' . $sel . '>' . $v . '</option>';
+            $form .= '<option value="'.$k.'"'.$sel.'>'.$v.'</option>';
         }
         $form .= '</select>&nbsp;<select class="span2" name="comments_status">';
 
@@ -274,7 +274,7 @@ 
             if (isset($status) && $k == $status) {
                 $sel = ' selected="selected"';
             }
-            $form .= '<option value="' . $k . '"' . $sel . '>' . $v . '</option>';
+            $form .= '<option value="'.$k.'"'.$sel.'>'.$v.'</option>';
         }
 
 
@@ -284,12 +284,12 @@ 
             if (isset($limit) && $k == $limit) {
                 $sel = ' selected="selected"';
             }
-            $form .= '<option value="' . $k . '"' . $sel . '>' . $k . '</option>';
+            $form .= '<option value="'.$k.'"'.$sel.'>'.$k.'</option>';
         }
-        $form .= '</select>&nbsp;<input class ="btn" type="submit" value="' . XoopsLocale::A_GO . '" name="selsubmit" /></form>';
+        $form .= '</select>&nbsp;<input class ="btn" type="submit" value="'.XoopsLocale::A_GO.'" name="selsubmit" /></form>';
 
         $xoops->tpl()->assign('form_sort', $form);
-        $xoops->tpl()->assign('php_selft', $_SERVER['PHP_SELF'] . '?op=comments_purge');
+        $xoops->tpl()->assign('php_selft', $_SERVER['PHP_SELF'].'?op=comments_purge');
 
         if ($comments_count > 0) {
             foreach (array_keys($comments_arr) as $i) {
@@ -298,18 +298,18 @@ 
                 if ($comments_arr[$i]->getVar('uid') > 0) {
                     $poster = $member_handler->getUser($comments_arr[$i]->getVar('uid'));
                     if (is_object($poster)) {
-                        $comments_poster_uname = '<a href="' . \XoopsBaseConfig::get('url') . '/userinfo.php?uid=' . $comments_arr[$i]->getVar('uid') . '">' . $poster->getVar('uname') . '</a>';
+                        $comments_poster_uname = '<a href="'.\XoopsBaseConfig::get('url').'/userinfo.php?uid='.$comments_arr[$i]->getVar('uid').'">'.$poster->getVar('uname').'</a>';
                     }
                 }
 
                 $comments_icon = ($comments_arr[$i]->getVar('icon') == '') ? '/images/icons/no_posticon.gif'
-                    : '/images/subject/' . htmlspecialchars($comments_arr[$i]->getVar('icon'), ENT_QUOTES);
-                $comments_icon = '<img src="' . \XoopsBaseConfig::get('url') . $comments_icon . '" alt="" />';
+                    : '/images/subject/'.htmlspecialchars($comments_arr[$i]->getVar('icon'), ENT_QUOTES);
+                $comments_icon = '<img src="'.\XoopsBaseConfig::get('url').$comments_icon.'" alt="" />';
 
                 $comments['comments_id'] = $id;
                 $comments['comments_poster'] = $comments_poster_uname;
                 $comments['comments_icon'] = $comments_icon;
-                $comments['comments_title'] = '<a href="main.php?op=comments_jump&amp;item_id=' . $comments_arr[$i]->getVar("id") . '">' . $comments_arr[$i]->getVar("title")  . '</a>';
+                $comments['comments_title'] = '<a href="main.php?op=comments_jump&amp;item_id='.$comments_arr[$i]->getVar("id").'">'.$comments_arr[$i]->getVar("title").'</a>';
                 $comments['comments_ip'] = $comments_arr[$i]->getVar('ip');
                 $comments['comments_date'] = XoopsLocale::formatTimestamp($comments_arr[$i]->getVar('created'));
                 $comments['comments_text'] = $myts->undoHtmlSpecialChars($comments_arr[$i]->getVar('text'));
@@ -324,7 +324,7 @@ 
             }
 
             if ($comments_count > $comments_limit) {
-                $nav = new XoopsPageNav($comments_count, $comments_limit, $comments_start, 'comments_start', 'comments_module=' . $comments_module . '&amp;comments_status=' . $comments_status);
+                $nav = new XoopsPageNav($comments_count, $comments_limit, $comments_start, 'comments_start', 'comments_module='.$comments_module.'&amp;comments_status='.$comments_status);
                 $xoops->tpl()->assign('nav', $nav->renderNav());
             }
         }