XOOPS / XoopsCore

htdocs/modules/system/help.php

Spacing +48 added lines, -48 removed lines

@@ -23,7 +23,7 @@ 
  */
 
 // Include header
-include __DIR__ . '/header.php';
+include __DIR__.'/header.php';
 
 $xoops = Xoops::getInstance();
 $system = System::getInstance();
@@ -42,24 +42,24 @@ 
 if ($mid > 0) {
     $module = $xoops->getModuleById($mid);
 
-    $system_breadcrumb->addLink($module->getVar('name'), 'help.php?mid=' . $module->getVar('mid', 's'));
+    $system_breadcrumb->addLink($module->getVar('name'), 'help.php?mid='.$module->getVar('mid', 's'));
     $system_breadcrumb->addLink(system_adminVersion($page, 'name'));
     $system_breadcrumb->render();
 
     if ($module->getVar('dirname', 'e') === 'system') {
 
-        $admin_dir = \XoopsBaseConfig::get('root-path') . '/modules/system/admin';
+        $admin_dir = \XoopsBaseConfig::get('root-path').'/modules/system/admin';
         $dirlist = XoopsLists::getDirListAsArray($admin_dir);
 
         foreach ($dirlist as $directory) {
-            if (XoopsLoad::fileExists($file = $admin_dir . '/' . $directory . '/xoops_version.php')) {
+            if (XoopsLoad::fileExists($file = $admin_dir.'/'.$directory.'/xoops_version.php')) {
 
                 require $file;
                 unset($file);
 
                 if ($modversion['help']) {
                     $help['name'] = system_adminVersion($directory, 'name');
-                    $help['link'] = 'help.php?mid=' . $mid . '&' . system_adminVersion($directory, 'help');
+                    $help['link'] = 'help.php?mid='.$mid.'&'.system_adminVersion($directory, 'help');
 
                     $xoops->tpl()->appendByRef('help', $help);
                     unset($help);
@@ -72,12 +72,12 @@ 
         $list_help = array();
         $listed_mods[0] = $module->toArray();
         $helplist = $module->getInfo('helpsection');
-        $j=0;
+        $j = 0;
         if (is_array($helplist)) {
             foreach ($helplist as $helpitem) {
                 if (($helpitem['name'] != '') && ($helpitem['link'] != '')) {
                     $list_help[$j]['name'] = $helpitem['name'];
-                    $list_help[$j]['link'] = 'help.php?mid=' . $mid . '&' . $helpitem['link'];
+                    $list_help[$j]['link'] = 'help.php?mid='.$mid.'&'.$helpitem['link'];
                     ++$j;
                 }
             }
@@ -85,9 +85,9 @@ 
             $xoopsTpl->assign('list_mods', $listed_mods);
         }
         unset ($helplist);
-        if (( $module->getInfo('help') != '' ) && ($j == 0)) {
+        if (($module->getInfo('help') != '') && ($j == 0)) {
             $help['name'] = $module->getInfo('name');
-            $help['link'] = 'help.php?mid=' . $mid . '&' . $module->getInfo('help');
+            $help['link'] = 'help.php?mid='.$mid.'&'.$module->getInfo('help');
             $xoopsTpl->appendByRef('help', $help);
         }
         unset($help);
@@ -100,22 +100,22 @@ 
 
     if ($page !== '') {
         // Call template
-        if ($helpfile = XoopsLoad::fileExists(\XoopsBaseConfig::get('root-path') . '/modules/' . $module->getVar('dirname', 'e') . '/locale/' . XoopsLocale::getLocale() . '/help/' . $page . '.tpl')) {
-            $helpcontent = $xoops->tpl()->fetch(\XoopsBaseConfig::get('root-path') . '/modules/' . $module->getVar('dirname', 'e') . '/locale/' . XoopsLocale::getLocale() . '/help/' . $page . '.tpl');
-        } elseif ($helpfile = XoopsLoad::fileExists(\XoopsBaseConfig::get('root-path') . '/modules/' . $module->getVar('dirname', 'e') . '/locale/' . XoopsLocale::getLocale() . '/help/' . $page . '.html')) {
-            $helpcontent = $xoops->tpl()->fetch(\XoopsBaseConfig::get('root-path') . '/modules/' . $module->getVar('dirname', 'e') . '/locale/' . XoopsLocale::getLocale() . '/help/' . $page . '.html');
-        } elseif (XoopsLoad::fileExists(\XoopsBaseConfig::get('root-path') . '/modules/' . $module->getVar('dirname', 'e') . '/language/' . $xoopsConfig['language'] . '/help/' . $page . '.tpl')) {
-            $helpcontent = $xoops->tpl()->fetch(\XoopsBaseConfig::get('root-path') . '/modules/' . $module->getVar('dirname', 'e') . '/language/' . $xoopsConfig['language'] . '/help/' . $page . '.tpl');
-        } elseif (XoopsLoad::fileExists(\XoopsBaseConfig::get('root-path') . '/modules/' . $module->getVar('dirname', 'e') . '/language/' . $xoopsConfig['language'] . '/help/' . $page . '.html')) {
-            $helpcontent = $xoops->tpl()->fetch(\XoopsBaseConfig::get('root-path') . '/modules/' . $module->getVar('dirname', 'e') . '/language/' . $xoopsConfig['language'] . '/help/' . $page . '.html');
-        } elseif ($helpfile = XoopsLoad::fileExists(\XoopsBaseConfig::get('root-path') . '/modules/' . $module->getVar('dirname', 'e') . '/locale/en_US/help/' . $page . '.tpl')) {
-            $helpcontent = $xoops->tpl()->fetch(\XoopsBaseConfig::get('root-path') . '/modules/' . $module->getVar('dirname', 'e') . '/locale/en_US/help/' . $page . '.tpl');
-        } elseif ($helpfile = XoopsLoad::fileExists(\XoopsBaseConfig::get('root-path') . '/modules/' . $module->getVar('dirname', 'e') . '/locale/en_US/help/' . $page . '.html')) {
-            $helpcontent = $xoops->tpl()->fetch(\XoopsBaseConfig::get('root-path') . '/modules/' . $module->getVar('dirname', 'e') . '/locale/en_US/help/' . $page . '.html');
-        } elseif (XoopsLoad::fileExists(\XoopsBaseConfig::get('root-path') . '/modules/' . $module->getVar('dirname', 'e') . '/language/english/help/' . $page . '.tpl')) {
-            $helpcontent = $xoops->tpl()->fetch(\XoopsBaseConfig::get('root-path') . '/modules/' . $module->getVar('dirname', 'e') . '/language/english/help/' . $page . '.tpl');
-        } elseif (XoopsLoad::fileExists(\XoopsBaseConfig::get('root-path') . '/modules/' . $module->getVar('dirname', 'e') . '/language/english/help/' . $page . '.html')) {
-            $helpcontent = $xoops->tpl()->fetch(\XoopsBaseConfig::get('root-path') . '/modules/' . $module->getVar('dirname', 'e') . '/language/english/help/' . $page . '.html');
+        if ($helpfile = XoopsLoad::fileExists(\XoopsBaseConfig::get('root-path').'/modules/'.$module->getVar('dirname', 'e').'/locale/'.XoopsLocale::getLocale().'/help/'.$page.'.tpl')) {
+            $helpcontent = $xoops->tpl()->fetch(\XoopsBaseConfig::get('root-path').'/modules/'.$module->getVar('dirname', 'e').'/locale/'.XoopsLocale::getLocale().'/help/'.$page.'.tpl');
+        } elseif ($helpfile = XoopsLoad::fileExists(\XoopsBaseConfig::get('root-path').'/modules/'.$module->getVar('dirname', 'e').'/locale/'.XoopsLocale::getLocale().'/help/'.$page.'.html')) {
+            $helpcontent = $xoops->tpl()->fetch(\XoopsBaseConfig::get('root-path').'/modules/'.$module->getVar('dirname', 'e').'/locale/'.XoopsLocale::getLocale().'/help/'.$page.'.html');
+        } elseif (XoopsLoad::fileExists(\XoopsBaseConfig::get('root-path').'/modules/'.$module->getVar('dirname', 'e').'/language/'.$xoopsConfig['language'].'/help/'.$page.'.tpl')) {
+            $helpcontent = $xoops->tpl()->fetch(\XoopsBaseConfig::get('root-path').'/modules/'.$module->getVar('dirname', 'e').'/language/'.$xoopsConfig['language'].'/help/'.$page.'.tpl');
+        } elseif (XoopsLoad::fileExists(\XoopsBaseConfig::get('root-path').'/modules/'.$module->getVar('dirname', 'e').'/language/'.$xoopsConfig['language'].'/help/'.$page.'.html')) {
+            $helpcontent = $xoops->tpl()->fetch(\XoopsBaseConfig::get('root-path').'/modules/'.$module->getVar('dirname', 'e').'/language/'.$xoopsConfig['language'].'/help/'.$page.'.html');
+        } elseif ($helpfile = XoopsLoad::fileExists(\XoopsBaseConfig::get('root-path').'/modules/'.$module->getVar('dirname', 'e').'/locale/en_US/help/'.$page.'.tpl')) {
+            $helpcontent = $xoops->tpl()->fetch(\XoopsBaseConfig::get('root-path').'/modules/'.$module->getVar('dirname', 'e').'/locale/en_US/help/'.$page.'.tpl');
+        } elseif ($helpfile = XoopsLoad::fileExists(\XoopsBaseConfig::get('root-path').'/modules/'.$module->getVar('dirname', 'e').'/locale/en_US/help/'.$page.'.html')) {
+            $helpcontent = $xoops->tpl()->fetch(\XoopsBaseConfig::get('root-path').'/modules/'.$module->getVar('dirname', 'e').'/locale/en_US/help/'.$page.'.html');
+        } elseif (XoopsLoad::fileExists(\XoopsBaseConfig::get('root-path').'/modules/'.$module->getVar('dirname', 'e').'/language/english/help/'.$page.'.tpl')) {
+            $helpcontent = $xoops->tpl()->fetch(\XoopsBaseConfig::get('root-path').'/modules/'.$module->getVar('dirname', 'e').'/language/english/help/'.$page.'.tpl');
+        } elseif (XoopsLoad::fileExists(\XoopsBaseConfig::get('root-path').'/modules/'.$module->getVar('dirname', 'e').'/language/english/help/'.$page.'.html')) {
+            $helpcontent = $xoops->tpl()->fetch(\XoopsBaseConfig::get('root-path').'/modules/'.$module->getVar('dirname', 'e').'/language/english/help/'.$page.'.html');
         } else {
             $xoops->tpl()->assign('load_error', 1);
         }
@@ -126,23 +126,23 @@ 
         $xoops->tpl()->assign('helpcontent', $helpcontent);
     } else {
         if ($helpfile = XoopsLoad::fileExists(
-            \XoopsBaseConfig::get('root-path') . '/modules/' . $module->getVar('dirname', 'e')
-            . '/locale/' . XoopsLocale::getLocale() . '/help/module_index.html'
+            \XoopsBaseConfig::get('root-path').'/modules/'.$module->getVar('dirname', 'e')
+            . '/locale/'.XoopsLocale::getLocale().'/help/module_index.html'
         )) {
             $helpcontent = $xoops->tpl()->fetch(
-                \XoopsBaseConfig::get('root-path') . '/modules/' . $module->getVar('dirname', 'e')
-                . '/locale/' . XoopsLocale::getLocale() . '/help/module_index.html'
+                \XoopsBaseConfig::get('root-path').'/modules/'.$module->getVar('dirname', 'e')
+                . '/locale/'.XoopsLocale::getLocale().'/help/module_index.html'
             );
         } elseif (XoopsLoad::fileExists(
-            \XoopsBaseConfig::get('root-path') . '/modules/' . $module->getVar('dirname', 'e')
-            . '/language/' . $xoops->getConfig('language') . '/help/module_index.html'
+            \XoopsBaseConfig::get('root-path').'/modules/'.$module->getVar('dirname', 'e')
+            . '/language/'.$xoops->getConfig('language').'/help/module_index.html'
         )) {
             $helpcontent = $xoops->tpl()->fetch(
-                \XoopsBaseConfig::get('root-path') . '/modules/' . $module->getVar('dirname', 'e')
-                . '/language/' . $xoops->getConfig('language') . '/help/module_index.html'
+                \XoopsBaseConfig::get('root-path').'/modules/'.$module->getVar('dirname', 'e')
+                . '/language/'.$xoops->getConfig('language').'/help/module_index.html'
             );
         } else {
-            $helpcontent = '<p>' . $module->getInfo('description') . '</p>';
+            $helpcontent = '<p>'.$module->getInfo('description').'</p>';
         }
         $xoops->tpl()->assign('helpcontent', $helpcontent);
     }
@@ -169,19 +169,19 @@ 
         $listed_mods[$i]['description'] = $module->getInfo('description');
 
         if ($module->getVar('dirname', 'e') === 'system') {
-            $admin_dir = \XoopsBaseConfig::get('root-path') . '/modules/system/admin';
+            $admin_dir = \XoopsBaseConfig::get('root-path').'/modules/system/admin';
             $dirlist = XoopsLists::getDirListAsArray($admin_dir);
 
             foreach ($dirlist as $directory) {
-                if (XoopsLoad::fileExists($file = $admin_dir . '/' . $directory . '/xoops_version.php')) {
+                if (XoopsLoad::fileExists($file = $admin_dir.'/'.$directory.'/xoops_version.php')) {
 
                     require $file;
                     unset($file);
 
                     if ($modversion['help']) {
                         $list_help[$j]['name'] = system_adminVersion($directory, 'name');
-                        $list_help[$j]['link'] = 'help.php?mid=' . $module->getVar('mid', 'e')
-                            . '&amp;' . system_adminVersion($directory, 'help');
+                        $list_help[$j]['link'] = 'help.php?mid='.$module->getVar('mid', 'e')
+                            . '&amp;'.system_adminVersion($directory, 'help');
                     }
                     unset($modversion);
                     ++$j;
@@ -190,15 +190,15 @@ 
             unset($dirlist);
         } else {
             $helplist = $module->getInfo('helpsection');
-            $k=0;
+            $k = 0;
 
             // Only build the list if one has been defined.
             if (is_array($helplist)) {
                 foreach ($helplist as $helpitem) {
                     if (($helpitem['name'] != '') && ($helpitem['link'] != '')) {
                         $list_help[$j]['name'] = $helpitem['name'];
-                        $list_help[$j]['link'] = 'help.php?mid=' . $module->getVar('mid', 'e')
-                            . '&amp;' . $helpitem['link'];
+                        $list_help[$j]['link'] = 'help.php?mid='.$module->getVar('mid', 'e')
+                            . '&amp;'.$helpitem['link'];
                         ++$j;
                         ++$k;
                     }
@@ -207,10 +207,10 @@ 
             unset($helplist);
 
             // If there is no help section ($k=0), and a lone help parameter has been defined.
-            if (( $module->getInfo('help') != '' ) && ($k == 0)) {
+            if (($module->getInfo('help') != '') && ($k == 0)) {
                 $list_help[$j]['name'] = $module->getInfo('name');
-                $list_help[$j]['link'] = 'help.php?mid=' . $module->getVar('mid', 'e')
-                    . '&amp;' . $module->getInfo('help');
+                $list_help[$j]['link'] = 'help.php?mid='.$module->getVar('mid', 'e')
+                    . '&amp;'.$module->getInfo('help');
             }
         }
         $listed_mods[$i]['help_page'] = $list_help;
@@ -225,15 +225,15 @@ 
     $xoops->tpl()->assign('list_mods', $listed_mods);
 
     if (XoopsLoad::fileExists(
-        \XoopsBaseConfig::get('root-path') . '/modules/system/language/'
-        . $xoops->getConfig('language') . '/help/help_center.html'
+        \XoopsBaseConfig::get('root-path').'/modules/system/language/'
+        . $xoops->getConfig('language').'/help/help_center.html'
     )) {
         $helpcontent = $xoops->tpl()->fetch(
-            \XoopsBaseConfig::get('root-path') . '/modules/system/language/' . $xoops->getConfig('language')
+            \XoopsBaseConfig::get('root-path').'/modules/system/language/'.$xoops->getConfig('language')
             . '/help/help_center.html'
         );
     } else {
-        $helpcontent = '<div id="non-modhelp">' . SystemLocale::WELCOME_TO_XOOPS_HELP_CENTER . '</div>';
+        $helpcontent = '<div id="non-modhelp">'.SystemLocale::WELCOME_TO_XOOPS_HELP_CENTER.'</div>';
     }
 
     $xoops->tpl()->assign('helpcontent', $helpcontent);

htdocs/modules/system/menu.php

Spacing +3 added lines, -3 removed lines

@@ -40,15 +40,15 @@
 $dirlist = XoopsLists::getDirListAsArray($admin_dir);
 $index = 0;
 foreach ($dirlist as $file) {
-    if (XoopsLoad::fileExists($fileinc = $admin_dir . '/' . $file . '/xoops_version.php')) {
+    if (XoopsLoad::fileExists($fileinc = $admin_dir.'/'.$file.'/xoops_version.php')) {
         include $fileinc;
         unset($fileinc);
         if ($modversion['hasAdmin']) {
-            if ($xoops->getModuleConfig('active_' . $file, 'system')) {
+            if ($xoops->getModuleConfig('active_'.$file, 'system')) {
                 $category = isset($modversion['category']) ? (int)($modversion['category']) : 0;
                 if (false != $all_ok || in_array($modversion['category'], $ok_syscats)) {
                     $adminmenu[$index]['title'] = trim($modversion['name']);
-                    $adminmenu[$index]['link'] = 'admin.php?fct=' . $file;
+                    $adminmenu[$index]['link'] = 'admin.php?fct='.$file;
                     $adminmenu[$index]['image'] = $modversion['image'];
                 }
             }

htdocs/modules/system/blocks/login.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
         $block['lang_lostpass'] = XoopsLocale::Q_LOST_YOUR_PASSWORD;
         $block['lang_registernow'] = XoopsLocale::REGISTER_NOW;
         if ($xoops->getConfig('use_ssl') == 1 && $xoops->getConfig('sslloginlink') != '') {
-            $block['sslloginlink'] = "<a href=\"javascript:openWithSelfMain('" . $xoops->getConfig('sslloginlink') . "', 'ssllogin', 300, 200);\">" . SystemLocale::SECURE_LOGIN . "</a>";
+            $block['sslloginlink'] = "<a href=\"javascript:openWithSelfMain('".$xoops->getConfig('sslloginlink')."', 'ssllogin', 300, 200);\">".SystemLocale::SECURE_LOGIN."</a>";
         } elseif ($xoops->getConfig('usercookie')) {
             $block['lang_rememberme'] = XoopsLocale::REMEMBER_ME;
         }

htdocs/modules/system/blocks/main.php

Indentation +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
                 $block['modules'][$i]['highlight'] = true;
                 $block['nothome'] = true;
             }
-             /* @var $plugin MenusPluginInterface */
+                /* @var $plugin MenusPluginInterface */
             if ($xoops->isModule() && $module->getVar('dirname') == $xoops->module->getVar('dirname') && $plugin = \Xoops\Module\Plugin::getPlugin($module->getVar('dirname'), 'menus')) {
                 $sublinks = $plugin->subMenus();
                 foreach ($sublinks as $sublink) {

Spacing +3 added lines, -3 removed lines

@@ -41,8 +41,8 @@ 
         if (in_array($i, $read_allowed)) {
             $block['modules'][$i]['name'] = $module->getVar('name');
             $block['modules'][$i]['dirname'] = $module->getVar('dirname');
-            if (XoopsLoad::fileExists($xoops->path('modules/' . $module->getVar('dirname') . '/icons/logo_small.png'))) {
-                $block['modules'][$i]['image'] = $xoops->url('modules/' . $module->getVar('dirname') . '/icons/logo_small.png');
+            if (XoopsLoad::fileExists($xoops->path('modules/'.$module->getVar('dirname').'/icons/logo_small.png'))) {
+                $block['modules'][$i]['image'] = $xoops->url('modules/'.$module->getVar('dirname').'/icons/logo_small.png');
             }
             if ($xoops->isModule() && ($i == $xoops->module->getVar('mid'))) {
                 $block['modules'][$i]['highlight'] = true;
@@ -58,7 +58,7 @@ 
                 foreach ($sublinks as $sublink) {
                     $block['modules'][$i]['sublinks'][] = array(
                         'name' => $sublink['name'],
-                        'url'  => \XoopsBaseConfig::get('url') . '/modules/' . $module->getVar('dirname') . '/' . $sublink['url']
+                        'url'  => \XoopsBaseConfig::get('url').'/modules/'.$module->getVar('dirname').'/'.$sublink['url']
                     );
                 }
             }

htdocs/modules/system/blocks/newmembers.php

Spacing +2 added lines, -2 removed lines

@@ -40,7 +40,7 @@ 
     $count = count($newmembers);
     for ($i = 0; $i < $count; ++$i) {
         if ($options[1] == 1) {
-            $block['users'][$i]['avatar'] = $newmembers[$i]->getVar('user_avatar') !== 'blank.gif' ? \XoopsBaseConfig::get('uploads-url') . '/' . $newmembers[$i]->getVar('user_avatar') : '';
+            $block['users'][$i]['avatar'] = $newmembers[$i]->getVar('user_avatar') !== 'blank.gif' ? \XoopsBaseConfig::get('uploads-url').'/'.$newmembers[$i]->getVar('user_avatar') : '';
         } else {
             $block['users'][$i]['avatar'] = '';
         }
@@ -54,7 +54,7 @@ 
 function b_system_newmembers_edit($options)
 {
     $block_form = new Xoops\Form\BlockForm();
-    $block_form->addElement( new Xoops\Form\Text(SystemLocale::NUMBER_OF_USERS_TO_DISPLAY, 'options[0]', 1, 3, $options[0]), true);
+    $block_form->addElement(new Xoops\Form\Text(SystemLocale::NUMBER_OF_USERS_TO_DISPLAY, 'options[0]', 1, 3, $options[0]), true);
     $block_form->addElement(new Xoops\Form\RadioYesNo(SystemLocale::DISPLAY_USERS_AVATARS, 'options[1]', $options[1]));
     return $block_form->render();
 }

htdocs/modules/system/blocks/themes.php

Spacing +6 added lines, -6 removed lines

@@ -25,20 +25,20 @@ 
     $theme_options = '';
     $theme_set_allowed = $xoops->getConfig('theme_set_allowed');
     foreach ($theme_set_allowed as $theme) {
-        $theme_options .= '<option value="' . $theme . '"';
+        $theme_options .= '<option value="'.$theme.'"';
         if ($theme == $xoops->getConfig('theme_set')) {
             $theme_options .= ' selected="selected"';
         }
-        $theme_options .= '>' . $theme . '</option>';
+        $theme_options .= '>'.$theme.'</option>';
     }
     $block = array();
     if ($options[0] == 1) {
-        $block['theme_select'] = "<img vspace=\"2\" id=\"xoops_theme_img\" src=\"" . \XoopsBaseConfig::get('themes-url') . "/" . $xoops->getConfig('theme_set') . "/screenshot.png\" alt=\"screenshot\" width=\"" . (int)($options[1]) . "\" /><br /><select class=\"span2\" id=\"xoops_theme_select\" name=\"xoops_theme_select\" onchange=\"showImgSelected('xoops_theme_img', 'xoops_theme_select', 'themes', '/screenshot.png', '" . \XoopsBaseConfig::get('url') . "');\">" . $theme_options . "</select><br /><input type=\"submit\" value=\"" . XoopsLocale::A_GO . "\" />";
+        $block['theme_select'] = "<img vspace=\"2\" id=\"xoops_theme_img\" src=\"".\XoopsBaseConfig::get('themes-url')."/".$xoops->getConfig('theme_set')."/screenshot.png\" alt=\"screenshot\" width=\"".(int)($options[1])."\" /><br /><select class=\"span2\" id=\"xoops_theme_select\" name=\"xoops_theme_select\" onchange=\"showImgSelected('xoops_theme_img', 'xoops_theme_select', 'themes', '/screenshot.png', '".\XoopsBaseConfig::get('url')."');\">".$theme_options."</select><br /><input type=\"submit\" value=\"".XoopsLocale::A_GO."\" />";
     } else {
-        $block['theme_select'] = '<select class="span2" name="xoops_theme_select" onchange="submit();" size="3">' . $theme_options . '</select>';
+        $block['theme_select'] = '<select class="span2" name="xoops_theme_select" onchange="submit();" size="3">'.$theme_options.'</select>';
     }
 
-    $block['theme_select'] .= '<br />(' . sprintf(SystemLocale::F_THEMES, '<strong>' . count($theme_set_allowed) . '</strong>') . ')<br />';
+    $block['theme_select'] .= '<br />('.sprintf(SystemLocale::F_THEMES, '<strong>'.count($theme_set_allowed).'</strong>').')<br />';
     return $block;
 }
 
@@ -46,6 +46,6 @@ 
 {
     $block_form = new Xoops\Form\BlockForm();
     $block_form->addElement(new Xoops\Form\RadioYesNo(SystemLocale::DISPLAY_SCREENSHOT_IMAGE, 'options[0]', $options[0]));
-    $block_form->addElement( new Xoops\Form\Text(SystemLocale::SCREENSHOT_IMAGE_WIDTH, 'options[1]', 1, 3, $options[1]), true);
+    $block_form->addElement(new Xoops\Form\Text(SystemLocale::SCREENSHOT_IMAGE_WIDTH, 'options[1]', 1, 3, $options[1]), true);
     return $block_form->render();
 }

htdocs/modules/system/blocks/user.php

Spacing +4 added lines, -4 removed lines

@@ -40,7 +40,7 @@ 
 
         if (is_array($menu) && !empty($menu)) {
             $block['modules'][$i]['name'] = $menu['name'];
-            $block['modules'][$i]['link'] = $xoops->url('modules/' . $dirname . '/' . $menu['link']);
+            $block['modules'][$i]['link'] = $xoops->url('modules/'.$dirname.'/'.$menu['link']);
             $block['modules'][$i]['image'] = $menu['image'];
             $block['modules'][$i]['icon'] = 'glyphicon glyphicon-none';
             $block['modules'][$i]['dirname'] = $dirname;
@@ -54,7 +54,7 @@ 
                         $block['modules'][$i]['sublinks'][] = array(
                             'name' => $sublink['name'],
                             'title' => $sublink['name'],
-                            'url'  => $xoops->url('modules/' . $dirname . '/' . $sublink['url'])
+                            'url'  => $xoops->url('modules/'.$dirname.'/'.$sublink['url'])
                         );
                     }
                 }
@@ -67,7 +67,7 @@ 
     // View Account
     array_unshift($block['modules'], array(
         'name' => XoopsLocale::VIEW_ACCOUNT,
-        'link' => $xoops->url('userinfo.php?uid=' . $xoops->user->getVar('uid')),
+        'link' => $xoops->url('userinfo.php?uid='.$xoops->user->getVar('uid')),
         'icon' => 'glyphicon glyphicon-user',
         'title' => XoopsLocale::VIEW_ACCOUNT,
     ));
@@ -100,7 +100,7 @@ 
     $name = XoopsLocale::INBOX;
     $class = '';
     if ($pm_count = $pm_handler->getCount($criteria)) {
-        $name = XoopsLocale::INBOX . ' <span class="badge">' . $pm_count . '</span>';
+        $name = XoopsLocale::INBOX.' <span class="badge">'.$pm_count.'</span>';
         //$class = 'text-info';
     }
 

htdocs/modules/system/blocks/online.php

Spacing +2 added lines, -2 removed lines

@@ -50,7 +50,7 @@ 
         $members = '';
         for ($i = 0; $i < $total; ++$i) {
             if ($onlines[$i]['online_uid'] > 0) {
-                $members .= ' <a href="' . \XoopsBaseConfig::get('url') . '/userinfo.php?uid=' . $onlines[$i]['online_uid'] . '" title="' . $onlines[$i]['online_uname'] . '">' . $onlines[$i]['online_uname'] . '</a>,';
+                $members .= ' <a href="'.\XoopsBaseConfig::get('url').'/userinfo.php?uid='.$onlines[$i]['online_uid'].'" title="'.$onlines[$i]['online_uname'].'">'.$onlines[$i]['online_uname'].'</a>,';
             } else {
                 ++$guests;
             }
@@ -58,7 +58,7 @@ 
         $block['online_total'] = sprintf(XoopsLocale::F_USERS_ONLINE, $total);
         if ($xoops->isModule()) {
             $mytotal = $online_handler->getCount(new Criteria('online_module', $xoops->module->getVar('mid')));
-            $block['online_total'] .= ' (' . sprintf(XoopsLocale::F_USERS_BROWSING, $mytotal, $xoops->module->getVar('name')) . ')';
+            $block['online_total'] .= ' ('.sprintf(XoopsLocale::F_USERS_BROWSING, $mytotal, $xoops->module->getVar('name')).')';
         }
         $block['lang_members'] = XoopsLocale::MEMBERS;
         $block['lang_guests'] = XoopsLocale::GUESTS;

htdocs/modules/system/blocks/topposters.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
     for ($i = 0; $i < $count; ++$i) {
         $block['users'][$i]['rank'] = $i + 1;
         if ($options[1] == 1) {
-            $block['users'][$i]['avatar'] = $topposters[$i]->getVar('user_avatar') !== 'blank.gif' ? \XoopsBaseConfig::get('uploads-url') . '/' . $topposters[$i]->getVar('user_avatar') : '';
+            $block['users'][$i]['avatar'] = $topposters[$i]->getVar('user_avatar') !== 'blank.gif' ? \XoopsBaseConfig::get('uploads-url').'/'.$topposters[$i]->getVar('user_avatar') : '';
         } else {
             $block['users'][$i]['avatar'] = '';
         }