XOOPS / XoopsCore

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'] = '';
         }

htdocs/modules/system/class/system.php

Spacing +12 added lines, -12 removed lines

@@ -121,11 +121,11 @@ 
             return false;
         }
         $language = empty($language) ? $xoops->getConfig('language') : $language;
-        $path = 'modules/' . $domain . '/language/';
-        if (XoopsLoad::fileExists($file = $xoops->path($path . $language . '/admin/' . $name . '.php'))) {
+        $path = 'modules/'.$domain.'/language/';
+        if (XoopsLoad::fileExists($file = $xoops->path($path.$language.'/admin/'.$name.'.php'))) {
             $ret = include_once $file;
         } else {
-            $ret = include_once $xoops->path($path . 'english/admin/' . $name . '.php');
+            $ret = include_once $xoops->path($path.'english/admin/'.$name.'.php');
         }
         return $ret;
     }
@@ -138,16 +138,16 @@ 
     public function adminVersion($version, $value = '')
     {
         static $tblVersion = array();
-        if (is_array($tblVersion) && array_key_exists($version . '.' . $value, $tblVersion)) {
-            return $tblVersion[$version . '.' . $value];
+        if (is_array($tblVersion) && array_key_exists($version.'.'.$value, $tblVersion)) {
+            return $tblVersion[$version.'.'.$value];
         }
         $xoops = Xoops::getInstance();
-        $path = $xoops->path('modules/system/admin/' . $version . '/xoops_version.php');
+        $path = $xoops->path('modules/system/admin/'.$version.'/xoops_version.php');
         if (XoopsLoad::fileExists($path)) {
             $modversion = array();
             include $path;
             $retvalue = $modversion[$value];
-            $tblVersion[$version . '.' . $value] = $retvalue;
+            $tblVersion[$version.'.'.$value] = $retvalue;
             return $retvalue;
         }
         return '';
@@ -173,7 +173,7 @@ 
      */
     public function cleanCache($cache)
     {
-        $cachePath = \XoopsBaseConfig::get('var-path') . '/caches/';
+        $cachePath = \XoopsBaseConfig::get('var-path').'/caches/';
         $total_smarty_cache = 0;
         $total_smarty_compile = 0;
         $total_xoops_cache = 0;
@@ -181,7 +181,7 @@ 
             for ($i = 0; $i < count($cache); ++$i) {
                 switch ($cache[$i]) {
                     case 1:
-                        $files = glob($cachePath . 'smarty_cache/*.*');
+                        $files = glob($cachePath.'smarty_cache/*.*');
                         $total_smarty_cache = 0;
                         foreach ($files as $filename) {
                             if (basename(strtolower($filename)) !== 'index.html') {
@@ -192,7 +192,7 @@ 
                         break;
 
                     case 2:
-                        $files = glob($cachePath . 'smarty_compile/*.*');
+                        $files = glob($cachePath.'smarty_compile/*.*');
                         $total_smarty_compile = 0;
                         foreach ($files as $filename) {
                             if (basename(strtolower($filename)) !== 'index.html') {
@@ -206,7 +206,7 @@ 
                         // ask the cache to clear itself
                         $status = Xoops::getInstance()->cache()->delete('system');
                         // this section captures legacy cache use only
-                        $files = glob($cachePath . 'xoops_cache/*.*');
+                        $files = glob($cachePath.'xoops_cache/*.*');
                         $total_xoops_cache = 0;
                         foreach ($files as $filename) {
                             if (basename(strtolower($filename)) != 'index.html') {
@@ -214,7 +214,7 @@ 
                                 ++$total_xoops_cache;
                             }
                         }
-                        $total_xoops_cache = $status || ($total_xoops_cache>0);
+                        $total_xoops_cache = $status || ($total_xoops_cache > 0);
                         break;
                 }
             }

htdocs/modules/system/class/plugin/usermenu.php

Spacing +2 added lines, -2 removed lines

@@ -28,7 +28,7 @@ 
             // View Account
             $ret[] = [
                 '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-user',
             ];
 
@@ -58,7 +58,7 @@ 
 
             $name = XoopsLocale::INBOX;
             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>';
             }
 
             $ret[] = [

htdocs/modules/system/class/CountryFlagProvider.php

Spacing +21 added lines, -21 removed lines

@@ -75,13 +75,13 @@ 
         }
 
         $xoops = \Xoops::getInstance();
-        $flagDir = $this->flagSource . $sizeDir . '/';
-        $flagFile = $flagDir . $countryCode . '.png';
+        $flagDir = $this->flagSource.$sizeDir.'/';
+        $flagFile = $flagDir.$countryCode.'.png';
 
         $file = $xoops->path($flagFile);
         // switch to unknown if file is not readable
         if (!is_readable($file)) {
-            $flagFile = $flagDir . '_unknown.png';
+            $flagFile = $flagDir.'_unknown.png';
         }
         $url = $xoops->url($flagFile);
         return $url;
@@ -94,24 +94,24 @@ 
      *
      * @var string[]
      */
-    private $overrideMap = array (
-        'AC' => 'SH',       // *Ascension Island part of Saint Helena, Ascension and Tristan da Cunha
-        'BQ' => 'NL',       // Caribbean Netherlands
-        'BV' => 'NO',       // Bouvet Island, dependency of Norway
-        'CP' => 'FR',       // *Clipperton Island,  overseas possession of France
-        'DG' => 'GB',       // *Diego Garcia, British Indian Ocean Territory disputed sovereignty
-        'EA' => 'ES',       // *Ceuta & Melilla, Spanish cities on the north coast of Africa
-        'GF' => 'FR',       // French Guiana, overseas region of France
-        'GP' => 'FR',       // Guadeloupe, overseas region of France
-        'HM' => 'AU',       // Heard & McDonald Islands, Australian external territory
-        'IO' => 'GB',       // British Indian Ocean Territory
-        'PM' => 'FR',       // St. Pierre & Miquelon, territorial overseas collectivity of France
-        'RE' => 'FR',       // Réunion, overseas region of France
-        'SJ' => 'NO',       // Svalbard & Jan Mayen, integrated parts of Norway not allocated to counties
-        'SX' => 'NL',       // Sint Maarten,  constituent country of the Kingdom of the Netherlands
-        'TA' => 'SH',       // *Tristan da Cunha part of Saint Helena, Ascension and Tristan da Cunha
-        'UM' => 'US',       // U.S. Outlying Islands
-        'XK' => '_kosovo',  // (User-assigned range) temporary assigned code
+    private $overrideMap = array(
+        'AC' => 'SH', // *Ascension Island part of Saint Helena, Ascension and Tristan da Cunha
+        'BQ' => 'NL', // Caribbean Netherlands
+        'BV' => 'NO', // Bouvet Island, dependency of Norway
+        'CP' => 'FR', // *Clipperton Island,  overseas possession of France
+        'DG' => 'GB', // *Diego Garcia, British Indian Ocean Territory disputed sovereignty
+        'EA' => 'ES', // *Ceuta & Melilla, Spanish cities on the north coast of Africa
+        'GF' => 'FR', // French Guiana, overseas region of France
+        'GP' => 'FR', // Guadeloupe, overseas region of France
+        'HM' => 'AU', // Heard & McDonald Islands, Australian external territory
+        'IO' => 'GB', // British Indian Ocean Territory
+        'PM' => 'FR', // St. Pierre & Miquelon, territorial overseas collectivity of France
+        'RE' => 'FR', // Réunion, overseas region of France
+        'SJ' => 'NO', // Svalbard & Jan Mayen, integrated parts of Norway not allocated to counties
+        'SX' => 'NL', // Sint Maarten,  constituent country of the Kingdom of the Netherlands
+        'TA' => 'SH', // *Tristan da Cunha part of Saint Helena, Ascension and Tristan da Cunha
+        'UM' => 'US', // U.S. Outlying Islands
+        'XK' => '_kosovo', // (User-assigned range) temporary assigned code
     );
 
     /**

htdocs/modules/system/class/cookie.php

Spacing +8 added lines, -8 removed lines

@@ -45,7 +45,7 @@ 
                 list ($k, $v) = each($key);
 
                 // Set string representation
-                $key = $k . '[' . $v . ']';
+                $key = $k.'['.$v.']';
 
                 // Set expiration time to -1hr (will cause browser deletion)
                 setcookie($key, false, time() - 3600);
@@ -60,7 +60,7 @@ 
                 foreach ($_COOKIE[$key] as $k => $v)
                 {
                     // Set string representation
-                    $cookie = $key . '[' . $k . ']';
+                    $cookie = $key.'['.$k.']';
 
                     // Set expiration time to -1hr (will cause browser deletion)
                     setcookie($cookie, false, time() - 3600);
@@ -138,12 +138,12 @@ 
     public static function set(
         $key,
         $value,
-        $expire = 0,            /* Default expire time (session, 1 week = 604800) */
-        $path = '',             /* Default path */
-        $domain = '',           /* Default domain */
-        $secure = false,        /* Does this cookie need a secure HTTPS connection? */
+        $expire = 0, /* Default expire time (session, 1 week = 604800) */
+        $path = '', /* Default path */
+        $domain = '', /* Default domain */
+        $secure = false, /* Does this cookie need a secure HTTPS connection? */
         $httponly = true        /* Can non-HTTP services access this cookie (IE: javascript)? */
-    ){
+    ) {
         // Make sure they aren't trying to set a reserved word
         if (!in_array($key, self::$_reserved))
         {
@@ -171,7 +171,7 @@ 
                 list ($k, $v) = each($key);
 
                 // Set string representation
-                $key = $k . '[' . $v . ']';
+                $key = $k.'['.$v.']';
             }
         }
 

Braces +18 added lines, -6 removed lines

@@ -95,11 +95,15 @@ 
             list ($k, $v) = each($key);
 
             // Check for key/value pair and return
-            if (isset($_COOKIE[$k][$v])) return true;
+            if (isset($_COOKIE[$k][$v])) {
+                return true;
+            }
         }
 
         // If key exists, return true
-        else if (isset($_COOKIE[$key])) return true;
+        else if (isset($_COOKIE[$key])) {
+            return true;
+        }
 
         // Key does not exist
         return false;
@@ -118,14 +122,20 @@ 
             list ($k, $v) = each($key);
 
             // Check for key/value pair and return
-            if (isset($_COOKIE[$k][$v])) return $_COOKIE[$k][$v];
+            if (isset($_COOKIE[$k][$v])) {
+                return $_COOKIE[$k][$v];
+            }
         }
 
         // Return single key if it's set
-        else if (isset($_COOKIE[$key])) return $_COOKIE[$key];
+        else if (isset($_COOKIE[$key])) {
+            return $_COOKIE[$key];
+        }
 
         // Otherwise return null
-        else return null;
+        else {
+            return null;
+        }
     }
 
     // Return the cookie array
@@ -155,7 +165,9 @@ 
         }
 
         // Otherwise, throw an error
-        else Error::warning('Could not set key -- it is reserved.', __CLASS__);
+        else {
+            Error::warning('Could not set key -- it is reserved.', __CLASS__);
+        }
     }
 
     // Converts strings to arrays (or vice versa if toString = true)

htdocs/modules/system/class/systembreadcrumb.php

Indentation -4 removed lines

@@ -72,7 +72,6 @@ 
 
     /**
      * Add link to breadcrumb
-
      */
     function addLink($title = '', $link = '', $home = false)
     {
@@ -83,7 +82,6 @@ 
 
     /**
      * Add Help link
-
      */
     function addHelp($link = '')
     {
@@ -92,7 +90,6 @@ 
 
     /**
      * Add Tips
-
      */
     function addTips($value)
     {
@@ -101,7 +98,6 @@ 
 
     /**
      * Render System BreadCrumb
-
      */
     function render()
     {

Spacing +9 added lines, -9 removed lines

@@ -115,34 +115,34 @@
                 }
             }
             // Call template
-            if (XoopsLoad::fileExists(\XoopsBaseConfig::get('root-path') . '/modules/system/language/' . $xoops->getConfig('language') . '/help/' . $this->_directory . '.html')) {
+            if (XoopsLoad::fileExists(\XoopsBaseConfig::get('root-path').'/modules/system/language/'.$xoops->getConfig('language').'/help/'.$this->_directory.'.html')) {
                 $xoops->tpl()
-                        ->assign('help_content', \XoopsBaseConfig::get('root-path') . '/modules/system/language/' . $xoops->getConfig('language') . '/help/' . $this->_directory . '.html');
+                        ->assign('help_content', \XoopsBaseConfig::get('root-path').'/modules/system/language/'.$xoops->getConfig('language').'/help/'.$this->_directory.'.html');
             } else {
-                if (XoopsLoad::fileExists(\XoopsBaseConfig::get('root-path') . '/modules/system/language/english/help/' . $this->_directory . '.html')) {
+                if (XoopsLoad::fileExists(\XoopsBaseConfig::get('root-path').'/modules/system/language/english/help/'.$this->_directory.'.html')) {
                     $xoops->tpl()
-                            ->assign('help_content', \XoopsBaseConfig::get('root-path') . '/modules/system/language/english/help/' . $this->_directory . '.html');
+                            ->assign('help_content', \XoopsBaseConfig::get('root-path').'/modules/system/language/english/help/'.$this->_directory.'.html');
                 } else {
                     $xoops->tpl()->assign('load_error', 1);
                 }
             }
         } else {
-            $out = $menu = '<style type="text/css" media="screen">@import ' . \XoopsBaseConfig::get('url') . '/modules/system/css/menu.css;</style>';
+            $out = $menu = '<style type="text/css" media="screen">@import '.\XoopsBaseConfig::get('url').'/modules/system/css/menu.css;</style>';
             $out .= '<ul id="xo-breadcrumb">';
             foreach ($this->_bread as $menu) {
                 if ($menu['home']) {
-                    $out .= '<li><a href="' . $menu['link'] . '" title="' . $menu['title'] . '"><img src="images/home.png" alt="' . $menu['title'] . '" class="home" /></a></li>';
+                    $out .= '<li><a href="'.$menu['link'].'" title="'.$menu['title'].'"><img src="images/home.png" alt="'.$menu['title'].'" class="home" /></a></li>';
                 } else {
                     if ($menu['link'] != '') {
-                        $out .= '<li><a href="' . $menu['link'] . '" title="' . $menu['title'] . '">' . $menu['title'] . '</a></li>';
+                        $out .= '<li><a href="'.$menu['link'].'" title="'.$menu['title'].'">'.$menu['title'].'</a></li>';
                     } else {
-                        $out .= '<li>' . $menu['title'] . '</li>';
+                        $out .= '<li>'.$menu['title'].'</li>';
                     }
                 }
             }
             $out .= '</ul>';
             if ($this->_tips) {
-                $out .= '<div class="tips">' . $this->_tips . '</div>';
+                $out .= '<div class="tips">'.$this->_tips.'</div>';
             }
             echo $out;
         }

htdocs/modules/system/class/extension.php

Spacing +10 added lines, -10 removed lines

@@ -67,7 +67,7 @@ 
         $i = 0;
         foreach ($this->modulesList as $file) {
             $file = trim($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();
                 /* @var $module XoopsModule */
                 $module = $module_handler->create();
@@ -97,7 +97,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->getInfo('mid')
                             );
                         }
@@ -106,28 +106,28 @@ 
                     }
                     $module->setInfo('version', round($module->getInfo('version'), 2));
                     if (XoopsLoad::fileExists(
-                        \XoopsBaseConfig::get('root-path') . '/modules/' . $module->getInfo('dirname') . '/icons/logo_small.png'
+                        \XoopsBaseConfig::get('root-path').'/modules/'.$module->getInfo('dirname').'/icons/logo_small.png'
                     )) {
                         $module->setInfo(
                             'logo_small',
-                            \XoopsBaseConfig::get('url') . '/modules/' . $module->getInfo('dirname') . '/icons/logo_small.png'
+                            \XoopsBaseConfig::get('url').'/modules/'.$module->getInfo('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');
                     }
                     if (XoopsLoad::fileExists(
-                        \XoopsBaseConfig::get('root-path') . '/modules/' . $module->getInfo('dirname') . '/icons/logo_large.png'
+                        \XoopsBaseConfig::get('root-path').'/modules/'.$module->getInfo('dirname').'/icons/logo_large.png'
                     )) {
                         $module->setInfo(
                             'logo_large',
-                            \XoopsBaseConfig::get('url') . '/modules/' . $module->getInfo('dirname') . '/icons/logo_large.png'
+                            \XoopsBaseConfig::get('url').'/modules/'.$module->getInfo('dirname').'/icons/logo_large.png'
                         );
                     } else {
-                        $module->setInfo('logo_large', \XoopsBaseConfig::get('url') . '/media/xoops/images/icons/32/default.png');
+                        $module->setInfo('logo_large', \XoopsBaseConfig::get('url').'/media/xoops/images/icons/32/default.png');
                     }
                     $module->setInfo(
                         'link_admin',
-                        \XoopsBaseConfig::get('url') . '/modules/' . $module->getInfo('dirname') . '/' . $module->getInfo('adminindex')
+                        \XoopsBaseConfig::get('url').'/modules/'.$module->getInfo('dirname').'/'.$module->getInfo('adminindex')
                     );
                     $module->setInfo('options', $module->getAdminMenu());
                     $ret[] = $module;
@@ -153,7 +153,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)) {