XOOPS / XoopsCore

xoops_lib/Xmf/Module/Helper/Cache.php

Spacing +2 added lines, -2 removed lines

@@ -44,7 +44,7 @@ 
      */
     public function init()
     {
-        $this->prefix = 'module/' . $this->module->getVar('dirname');
+        $this->prefix = 'module/'.$this->module->getVar('dirname');
         $this->cache = \Xoops::getInstance()->cache();
     }
 
@@ -57,7 +57,7 @@ 
      */
     protected function prefix($name)
     {
-        return $this->prefix . '/'. $name;
+        return $this->prefix.'/'.$name;
     }
 
     /**

xoops_lib/Xmf/Module/Admin.php

Spacing +10 added lines, -10 removed lines

@@ -276,10 +276,10 @@ 
      */
     public function addConfigError($value = '')
     {
-        $path = XOOPS_URL . '/Frameworks/moduleclasses/icons/16/';
+        $path = XOOPS_URL.'/Frameworks/moduleclasses/icons/16/';
         $line = "";
         $line .= "<span style='color : red; font-weight : bold;'>";
-        $line .= "<img src='" . $path . "0.png' >";
+        $line .= "<img src='".$path."0.png' >";
         $line .= $value;
         $line .= "</span>";
         $value = $line;
@@ -297,10 +297,10 @@ 
      */
     public function addConfigAccept($value = '')
     {
-        $path = XOOPS_URL . '/Frameworks/moduleclasses/icons/16/';
+        $path = XOOPS_URL.'/Frameworks/moduleclasses/icons/16/';
         $line = "";
         $line .= "<span style='color : green;'>";
-        $line .= "<img src='" . $path . "1.png' >";
+        $line .= "<img src='".$path."1.png' >";
         $line .= $value;
         $line .= "</span>";
         $value = $line;
@@ -318,10 +318,10 @@ 
      */
     public function addConfigWarning($value = '')
     {
-        $path = XOOPS_URL . '/Frameworks/moduleclasses/icons/16/';
+        $path = XOOPS_URL.'/Frameworks/moduleclasses/icons/16/';
         $line = "";
         $line .= "<span style='color : orange; font-weight : bold;'>";
-        $line .= "<img src='" . $path . "warning.png' >";
+        $line .= "<img src='".$path."warning.png' >";
         $line .= $value;
         $line .= "</span>";
         $value = $line;
@@ -403,7 +403,7 @@ 
         } else {
             $path = '../../Frameworks/moduleclasses/icons/32/';
 
-            return($path . $image);
+            return($path.$image);
         }
     }
 
@@ -439,12 +439,12 @@ 
         }
 
         if (static::isXng()) {
-            $path = '/media/xoops/images/icons/' . $path;
+            $path = '/media/xoops/images/icons/'.$path;
         } else {
-            $path = '/Frameworks/moduleclasses/icons/' . $path;
+            $path = '/Frameworks/moduleclasses/icons/'.$path;
         }
 
-        return(XOOPS_URL . $path . $name);
+        return(XOOPS_URL.$path.$name);
     }
 
     /**

xoops_lib/Xmf/Request.php

Spacing +6 added lines, -6 removed lines

@@ -234,7 +234,7 @@ 
     public static function getString($name, $default = '', $hash = 'default', $mask = 0)
     {
         // Cast to string, in case static::MASK_ALLOW_RAW was specified for mask
-        return (string) static::getVar($name, $default, $hash, 'string', $mask);
+        return (string)static::getVar($name, $default, $hash, 'string', $mask);
     }
 
     /**
@@ -262,7 +262,7 @@ 
      */
     public static function getText($name, $default = '', $hash = 'default')
     {
-        return (string) static::getVar($name, $default, $hash, 'string', static::MASK_ALLOW_RAW);
+        return (string)static::getVar($name, $default, $hash, 'string', static::MASK_ALLOW_RAW);
     }
 
     /**
@@ -276,7 +276,7 @@ 
      */
     public static function getUrl($name, $default = '', $hash = 'default')
     {
-        return (string) static::getVar($name, $default, $hash, 'weburl');
+        return (string)static::getVar($name, $default, $hash, 'weburl');
     }
 
     /**
@@ -290,7 +290,7 @@ 
      */
     public static function getPath($name, $default = '', $hash = 'default')
     {
-        return (string) static::getVar($name, $default, $hash, 'path');
+        return (string)static::getVar($name, $default, $hash, 'path');
     }
 
     /**
@@ -304,7 +304,7 @@ 
      */
     public static function getEmail($name, $default = '', $hash = 'default')
     {
-        $ret = (string) static::getVar($name, $default, $hash, 'email');
+        $ret = (string)static::getVar($name, $default, $hash, 'email');
         return empty($ret) ? $default : $ret;
     }
 
@@ -319,7 +319,7 @@ 
      */
     public static function getIP($name, $default = '', $hash = 'default')
     {
-        $ret = (string) static::getVar($name, $default, $hash, 'ip');
+        $ret = (string)static::getVar($name, $default, $hash, 'ip');
         return empty($ret) ? $default : $ret;
     }
 

xoops_lib/Xmf/Debug.php

Spacing +3 added lines, -3 removed lines

@@ -126,7 +126,7 @@ 
      */
     public static function useEventDumper($value = true)
     {
-        self::$eventDumper = (bool) $value;
+        self::$eventDumper = (bool)$value;
     }
 
     /**
@@ -174,7 +174,7 @@ 
         if ($events->hasListeners($eventName)) {
             $events->triggerEvent($eventName, $name);
         } else {
-            echo $name . ' - ' . (int)(microtime(true) - self::$times[$name]) . " \n";
+            echo $name.' - '.(int)(microtime(true) - self::$times[$name])." \n";
         }
     }
 
@@ -255,7 +255,7 @@ 
             ini_set('xdebug.collect_params', $collect_params);
             ini_set('xdebug.collect_return', $collect_return);
             if ($tracefile == '') {
-                $tracefile = \XoopsBaseConfig::get('var-path') . '/logs/php_trace';
+                $tracefile = \XoopsBaseConfig::get('var-path').'/logs/php_trace';
             }
             xdebug_start_trace($tracefile);
         }

xoops_lib/Xmf/Yaml.php

Spacing +1 added lines, -1 removed lines

@@ -134,7 +134,7 @@
     {
         try {
             $yamlString = VendorYaml::dump($var, $inline, $indent);
-            $ret = empty($yamlString) ? false : "<?php\n/*\n---\n" . $yamlString . "\n...\n*/\n";
+            $ret = empty($yamlString) ? false : "<?php\n/*\n---\n".$yamlString."\n...\n*/\n";
         } catch (\Exception $e) {
             static::logError($e);
             $ret = false;

xoops_lib/Xoops.php

Spacing +71 added lines, -71 removed lines

@@ -141,14 +141,14 @@ 
 
         $this->paths['www'] = array($root, $url);
         $this->paths['var'] = array($var, null);
-        $this->paths['lib'] = array($lib, $url . '/browse.php');
-        $this->paths['XOOPS'] = array($lib, $url . '/browse.php');
+        $this->paths['lib'] = array($lib, $url.'/browse.php');
+        $this->paths['XOOPS'] = array($lib, $url.'/browse.php');
         $this->paths['assets'] = array(\XoopsBaseConfig::get('asset-path'), \XoopsBaseConfig::get('asset-url'));
-        $this->paths['images'] = array($root . '/images', $url . '/images');
-        $this->paths['language'] = array($root . '/language', $url . '/language');
-        $this->paths['locale'] = array($root . '/locale', $url . '/locale');
+        $this->paths['images'] = array($root.'/images', $url.'/images');
+        $this->paths['language'] = array($root.'/language', $url.'/language');
+        $this->paths['locale'] = array($root.'/locale', $url.'/locale');
         $this->paths['media'] = array(\XoopsBaseConfig::get('media-path'), \XoopsBaseConfig::get('media-url'));
-        $this->paths['modules'] = array($root . '/modules', $url . '/modules');
+        $this->paths['modules'] = array($root.'/modules', $url.'/modules');
         $this->paths['themes'] = array(\XoopsBaseConfig::get('themes-path'), \XoopsBaseConfig::get('themes-url'));
         $this->paths['uploads'] = array(\XoopsBaseConfig::get('uploads-path'), \XoopsBaseConfig::get('uploads-url'));
 
@@ -371,7 +371,7 @@ 
     public function path($url, $virtual = false)
     {
         $url = $this->normalizePath($url);
-        $rootPath = $this->normalizePath(\XoopsBaseConfig::get('root-path') . '/');
+        $rootPath = $this->normalizePath(\XoopsBaseConfig::get('root-path').'/');
         if (0 === strpos($url, $rootPath)) {
             $url = substr($url, strlen($rootPath));
         }
@@ -383,11 +383,11 @@ 
             list($root, $path) = array('www', $url);
         }
         if (!$virtual) { // Returns a physical path
-            $path = $this->paths[$root][0] . '/' . $path;
+            $path = $this->paths[$root][0].'/'.$path;
             //$path = str_replace('/', DIRECTORY_SEPARATOR, $path);
             return $path;
         }
-        return !isset($this->paths[$root][1]) ? '' : ($this->paths[$root][1] . '/' . $path);
+        return !isset($this->paths[$root][1]) ? '' : ($this->paths[$root][1].'/'.$path);
     }
 
     /**
@@ -435,9 +435,9 @@ 
         }
         if (!empty($params)) {
             foreach ($params as $k => $v) {
-                $params[$k] = $k . '=' . rawurlencode($v);
+                $params[$k] = $k.'='.rawurlencode($v);
             }
-            $url .= '?' . implode('&', $params);
+            $url .= '?'.implode('&', $params);
         }
         return $url;
     }
@@ -518,7 +518,7 @@ 
                 $_SERVER['REQUEST_URI'] = $_SERVER['SCRIPT_NAME'];
             }
             if (isset($_SERVER['QUERY_STRING'])) {
-                $_SERVER['REQUEST_URI'] .= '?' . $_SERVER['QUERY_STRING'];
+                $_SERVER['REQUEST_URI'] .= '?'.$_SERVER['QUERY_STRING'];
             }
         }
     }
@@ -558,18 +558,18 @@ 
         if ($matched) {
             $names = array('tpl_name', 'type', 'module', 'file');
             $ret = array();
-            for ($i=0; $i<4; ++$i) {
+            for ($i = 0; $i < 4; ++$i) {
                 $ret[$names[$i]] = $parts[$i];
             }
         } else {
             // this should be eliminated
-            $this->events()->triggerEvent('debug.log', "Sloppy template: " . $tpl_name);
+            $this->events()->triggerEvent('debug.log', "Sloppy template: ".$tpl_name);
             $ret = array();
             $ret['type'] = $this->isAdminSide ? 'admin' : 'module';
             $info = explode(':', $tpl_name);
             if (count($info) == 2) {
                 $ret['type'] = $info[0];
-                $tpl_name = str_replace($ret['type'] . ':', '', $tpl_name);
+                $tpl_name = str_replace($ret['type'].':', '', $tpl_name);
             }
 
             if ($ret['type'] === 'db') {
@@ -588,7 +588,7 @@ 
                     $ret['module'] = $this->module->getVar('dirname', 'n');
                 }
             }
-            $ret['tpl_name'] = $ret['type'] . ':' . $ret['module'] . '/' . $ret['file'];
+            $ret['tpl_name'] = $ret['type'].':'.$ret['module'].'/'.$ret['file'];
         }
 
         return $ret;
@@ -633,8 +633,8 @@ 
                 $smarty = $repeat = null;
                 $this->theme()->headContent(
                     null,
-                    "<base href='" . \XoopsBaseConfig::get('url') . '/modules/'
-                    . $this->getConfig('startpage') . "/' />",
+                    "<base href='".\XoopsBaseConfig::get('url').'/modules/'
+                    . $this->getConfig('startpage')."/' />",
                     $smarty,
                     $repeat
                 );
@@ -681,7 +681,7 @@ 
         $this->events()->triggerEvent('core.footer.start');
 
         if (!headers_sent()) {
-            header('Content-Type:text/html; charset=' . XoopsLocale::getCharset());
+            header('Content-Type:text/html; charset='.XoopsLocale::getCharset());
             header('Expires: Mon, 26 Jul 1997 05:00:00 GMT');
             header('Cache-Control: private, no-cache');
             header('Pragma: no-cache');
@@ -1009,10 +1009,10 @@ 
             }
         }
         if (XoopsLoad::fileExists(
-            $hnd_file = \XoopsBaseConfig::get('root-path') . "/modules/{$module_dir}/class/form/{$name}.php"
+            $hnd_file = \XoopsBaseConfig::get('root-path')."/modules/{$module_dir}/class/form/{$name}.php"
         )) {
             include_once $hnd_file;
-            $class = ucfirst(strtolower($module_dir)) . ucfirst($name) . 'Form';
+            $class = ucfirst(strtolower($module_dir)).ucfirst($name).'Form';
             if (class_exists($class)) {
                 $instance = new $class($obj);
                 if ($instance instanceof \Xoops\Form\Form) {
@@ -1058,7 +1058,7 @@ 
         if ((empty($domain) || 'global' === $domain)) {
             $path = '';
         } else {
-            $path = (is_array($domain)) ? array_shift($domain) . '/' : "modules/{$domain}/";
+            $path = (is_array($domain)) ? array_shift($domain).'/' : "modules/{$domain}/";
         }
         $path .= 'language';
 
@@ -1200,9 +1200,9 @@ 
         $xoopsConfigMetaFooter = $this->getConfigs();
 
         if (!headers_sent()) {
-            header('Content-Type:text/html; charset=' . XoopsLocale::getCharset());
+            header('Content-Type:text/html; charset='.XoopsLocale::getCharset());
             header('Expires: Mon, 26 Jul 1997 05:00:00 GMT');
-            header("Last-Modified: " . gmdate("D, d M Y H:i:s") . " GMT");
+            header("Last-Modified: ".gmdate("D, d M Y H:i:s")." GMT");
             header(
                 'Cache-Control: no-store, no-cache, max-age=1, s-maxage=1, must-revalidate, post-check=0, pre-check=0'
             );
@@ -1211,28 +1211,28 @@ 
 
         echo "<!DOCTYPE html>\n";
         $xoops_url = \XoopsBaseConfig::get('url');
-        echo '<html lang="' . XoopsLocale::getLangCode() . '">
+        echo '<html lang="'.XoopsLocale::getLangCode().'">
               <head>
               <meta charset="utf-8">
               <meta http-equiv="X-UA-Compatible" content="IE=edge">
               <meta name="viewport" content="width=device-width, initial-scale=1">
-              <meta name="robots" content="' . htmlspecialchars($xoopsConfigMetaFooter['meta_robots']) . '" />
-              <meta name="keywords" content="' . htmlspecialchars($xoopsConfigMetaFooter['meta_keywords']) . '" />
-              <meta name="description" content="' . htmlspecialchars($xoopsConfigMetaFooter['meta_description']) . '" />
-              <meta name="rating" content="' . htmlspecialchars($xoopsConfigMetaFooter['meta_rating']) . '" />
-              <meta name="author" content="' . htmlspecialchars($xoopsConfigMetaFooter['meta_author']) . '" />
+              <meta name="robots" content="' . htmlspecialchars($xoopsConfigMetaFooter['meta_robots']).'" />
+              <meta name="keywords" content="' . htmlspecialchars($xoopsConfigMetaFooter['meta_keywords']).'" />
+              <meta name="description" content="' . htmlspecialchars($xoopsConfigMetaFooter['meta_description']).'" />
+              <meta name="rating" content="' . htmlspecialchars($xoopsConfigMetaFooter['meta_rating']).'" />
+              <meta name="author" content="' . htmlspecialchars($xoopsConfigMetaFooter['meta_author']).'" />
               <meta name="generator" content="XOOPS" />
-              <title>' . htmlspecialchars($this->getConfig('sitename')) . '</title>'
+              <title>' . htmlspecialchars($this->getConfig('sitename')).'</title>'
             . $this->theme->renderBaseAssets();
 
         $locale = $this->getConfig('locale');
-        if (XoopsLoad::fileExists($this->path('locale/' . $locale . '/style.css'))) {
-            echo '<link rel="stylesheet" type="text/css" media="all" href="' . $xoops_url
-                . '/locale/' . $locale . '/style.css" />';
+        if (XoopsLoad::fileExists($this->path('locale/'.$locale.'/style.css'))) {
+            echo '<link rel="stylesheet" type="text/css" media="all" href="'.$xoops_url
+                . '/locale/'.$locale.'/style.css" />';
         }
         $themecss = $this->getCss($this->getConfig('theme_set'));
         if ($themecss) {
-            echo '<link rel="stylesheet" type="text/css" media="all" href="' . $themecss . '" />';
+            echo '<link rel="stylesheet" type="text/css" media="all" href="'.$themecss.'" />';
         }
         if ($closehead) {
             echo '</head><body>';
@@ -1332,13 +1332,13 @@ 
         foreach ($hiddens as $name => $value) {
             if (is_array($value)) {
                 foreach ($value as $caption => $newvalue) {
-                    $str_hiddens .= '<input type="radio" name="' . $name . '" value="'
-                        . htmlspecialchars($newvalue) . '" > ' . $caption . NWLINE;
+                    $str_hiddens .= '<input type="radio" name="'.$name.'" value="'
+                        . htmlspecialchars($newvalue).'" > '.$caption.NWLINE;
                 }
-                $str_hiddens .= '<br />' . NWLINE;
+                $str_hiddens .= '<br />'.NWLINE;
             } else {
-                $str_hiddens .= '<input type="hidden" name="' . $name . '" value="'
-                    . htmlspecialchars($value) . '" />' . NWLINE;
+                $str_hiddens .= '<input type="hidden" name="'.$name.'" value="'
+                    . htmlspecialchars($value).'" />'.NWLINE;
             }
         }
         if ($addtoken != false) {
@@ -1450,7 +1450,7 @@ 
         $url = trim($url);
         if ($url != '') {
             if (!preg_match('/^(https?|ftps?|ed2k)\:\/\//i', $url)) {
-                $url = 'http://' . $url;
+                $url = 'http://'.$url;
             }
         }
         return $url;
@@ -1519,7 +1519,7 @@ 
         )));
         $this->setTpl($this->theme()->template);
         $this->tpl()->assign(array(
-            'xoops_theme'      => $theme, 'xoops_imageurl' => \XoopsBaseConfig::get('themes-url') . '/' . $theme . '/',
+            'xoops_theme'      => $theme, 'xoops_imageurl' => \XoopsBaseConfig::get('themes-url').'/'.$theme.'/',
             'xoops_themecss'   => $this->getCss($theme),
             'xoops_requesturi' => htmlspecialchars($_SERVER['REQUEST_URI'], ENT_QUOTES),
             'xoops_sitename'   => htmlspecialchars($this->getConfig('sitename'), ENT_QUOTES),
@@ -1531,8 +1531,8 @@ 
 
         $this->tpl()->assign('time', (int)($time));
         if (!empty($_SERVER['REQUEST_URI']) && $addredirect && strstr($url, 'user.php')) {
-            $joiner = (false===strpos($url, '?')) ? '?' : '&amp;';
-            $url .= $joiner . 'xoops_redirect=' . urlencode($_SERVER['REQUEST_URI']);
+            $joiner = (false === strpos($url, '?')) ? '?' : '&amp;';
+            $url .= $joiner.'xoops_redirect='.urlencode($_SERVER['REQUEST_URI']);
         }
         $url = preg_replace("/&amp;/i", '&', htmlspecialchars($url, ENT_QUOTES));
         $this->tpl()->assign('url', $url);
@@ -1596,25 +1596,25 @@ 
         }
         $xoops_theme_path = \XoopsBaseConfig::get('themes-path');
         $xoops_theme_url = \XoopsBaseConfig::get('themes-url');
-        if (is_dir($xoops_theme_path . '/' . $theme)) {
-            if (XoopsLoad::fileExists($xoops_theme_path . '/' . $theme . '/' . $str_css)) {
-                return $xoops_theme_url . '/' . $theme . '/' . $str_css;
-            } elseif (XoopsLoad::fileExists($xoops_theme_path . '/' . $theme . '/style.css')) {
-                return $xoops_theme_url . '/' . $theme . '/style.css';
+        if (is_dir($xoops_theme_path.'/'.$theme)) {
+            if (XoopsLoad::fileExists($xoops_theme_path.'/'.$theme.'/'.$str_css)) {
+                return $xoops_theme_url.'/'.$theme.'/'.$str_css;
+            } elseif (XoopsLoad::fileExists($xoops_theme_path.'/'.$theme.'/style.css')) {
+                return $xoops_theme_url.'/'.$theme.'/style.css';
             }
         }
-        if (is_dir($xoops_theme_path . '/' . $theme . '/css')) {
-            if (XoopsLoad::fileExists($xoops_theme_path . '/' . $theme . '/css/' . $str_css)) {
-                return $xoops_theme_url . '/' . $theme . '/css/' . $str_css;
-            } elseif (XoopsLoad::fileExists($xoops_theme_path . '/' . $theme . '/css/style.css')) {
-                return $xoops_theme_url . '/' . $theme . '/css/style.css';
+        if (is_dir($xoops_theme_path.'/'.$theme.'/css')) {
+            if (XoopsLoad::fileExists($xoops_theme_path.'/'.$theme.'/css/'.$str_css)) {
+                return $xoops_theme_url.'/'.$theme.'/css/'.$str_css;
+            } elseif (XoopsLoad::fileExists($xoops_theme_path.'/'.$theme.'/css/style.css')) {
+                return $xoops_theme_url.'/'.$theme.'/css/style.css';
             }
         }
-        if (is_dir($xoops_theme_path . '/' . $theme . '/assets/css')) {
-            if (XoopsLoad::fileExists($xoops_theme_path . '/' . $theme . '/assets/css/' . $str_css)) {
-                return $xoops_theme_url . '/' . $theme . '/assets/css/' . $str_css;
-            } elseif (XoopsLoad::fileExists($xoops_theme_path . '/' . $theme . '/assets/css/style.css')) {
-                return $xoops_theme_url . '/' . $theme . '/assets/css/style.css';
+        if (is_dir($xoops_theme_path.'/'.$theme.'/assets/css')) {
+            if (XoopsLoad::fileExists($xoops_theme_path.'/'.$theme.'/assets/css/'.$str_css)) {
+                return $xoops_theme_url.'/'.$theme.'/assets/css/'.$str_css;
+            } elseif (XoopsLoad::fileExists($xoops_theme_path.'/'.$theme.'/assets/css/style.css')) {
+                return $xoops_theme_url.'/'.$theme.'/assets/css/style.css';
             }
         }
         return '';
@@ -1728,7 +1728,7 @@ 
             if (empty($dirname)) {
                 $dirname = $this->isModule() ? $this->module->getVar('dirname') : 'system';
             }
-            $this->moduleConfigs[$dirname][$key] =& $value;
+            $this->moduleConfigs[$dirname][$key] = & $value;
         }
     }
 
@@ -1812,18 +1812,18 @@ 
             if (is_object($module)) {
                 $configs = $this->getHandlerConfig()->getConfigsByModule($module->getVar('mid'));
                 $this->cache()->write($key, $configs);
-                $this->moduleConfigs[$dirname] =& $configs;
+                $this->moduleConfigs[$dirname] = & $configs;
             }
         } else {
-            $this->moduleConfigs[$dirname] =& $configs;
+            $this->moduleConfigs[$dirname] = & $configs;
         }
 
         if ($this->isModule()) {
             //for legacy
-            $this->moduleConfig =& $this->moduleConfigs[$this->module->getVar('dirname')];
+            $this->moduleConfig = & $this->moduleConfigs[$this->module->getVar('dirname')];
         }
         if ($dirname === 'system') {
-            $this->config =& $this->moduleConfigs['system'];
+            $this->config = & $this->moduleConfigs['system'];
         }
         return $this->moduleConfigs[$dirname];
     }
@@ -1850,10 +1850,10 @@ 
         }
         if ($appendWithKey) {
             foreach ($values as $key2 => $value) {
-                $this->moduleConfigs[$dirname][$key][$key2] =& $value;
+                $this->moduleConfigs[$dirname][$key][$key2] = & $value;
             }
         } else {
-            $this->moduleConfigs[$dirname][$key][] =& $values;
+            $this->moduleConfigs[$dirname][$key][] = & $values;
         }
     }
 
@@ -1883,7 +1883,7 @@ 
      */
     public function getBaseDomain($url, $includeSubdomain = false)
     {
-        $url=mb_strtolower($url, 'UTF-8');
+        $url = mb_strtolower($url, 'UTF-8');
 
         $host = parse_url($url, PHP_URL_HOST);
         if (empty($host)) {
@@ -1894,13 +1894,13 @@ 
         }
 
         // check for exceptions, localhost and ip address (v4 & v6)
-        if ($host==='localhost') {
+        if ($host === 'localhost') {
             return $host;
         }
         // Check for IPV6 URL (see http://www.ietf.org/rfc/rfc2732.txt)
         // strip brackets before validating
-        if (substr($host, 0, 1)==='[' && substr($host, -1)===']') {
-            $host = substr($host, 1, (strlen($host)-2));
+        if (substr($host, 0, 1) === '[' && substr($host, -1) === ']') {
+            $host = substr($host, 1, (strlen($host) - 2));
         }
         if (filter_var($host, FILTER_VALIDATE_IP)) {
             return $host;
@@ -1930,7 +1930,7 @@ 
             $module = $tplfile->getVar('tpl_module', 'n');
             $type = $tplfile->getVar('tpl_type', 'n');
             $tpl = new XoopsTpl();
-            return $tpl->touch($type . ':' . $module . '/' . $file);
+            return $tpl->touch($type.':'.$module.'/'.$file);
         }
         return false;
     }

xoops_lib/Xoops/Locale.php

Spacing +9 added lines, -9 removed lines

@@ -221,9 +221,9 @@ 
             $fullPath = $xoops->path("{$path}/locale/{$locale}/locale.php");
             $fullPath2 = $xoops->path("{$path}/locale/{$locale}/{$locale}.php");
             if (\XoopsLoad::fileExists($fullPath)) {
-                \XoopsLoad::addMap(array($domain . 'locale' => $fullPath));
+                \XoopsLoad::addMap(array($domain.'locale' => $fullPath));
                 if (\XoopsLoad::fileExists($fullPath2)) {
-                    \XoopsLoad::addMap(array(strtolower($domain . "locale{$locale}") => $fullPath2));
+                    \XoopsLoad::addMap(array(strtolower($domain."locale{$locale}") => $fullPath2));
                 }
                 return true;
             }
@@ -246,9 +246,9 @@ 
             $fullPath = $xoops->path($theme->resourcePath("locale/{$locale}/locale.php"));
             $fullPath2 = $xoops->path($theme->resourcePath("locale/{$locale}/{$locale}.php"));
             if (\XoopsLoad::fileExists($fullPath)) {
-                \XoopsLoad::addMap(array(strtolower($theme->folderName . 'ThemeLocale') => $fullPath));
+                \XoopsLoad::addMap(array(strtolower($theme->folderName.'ThemeLocale') => $fullPath));
                 if (\XoopsLoad::fileExists($fullPath2)) {
-                    \XoopsLoad::addMap(array(strtolower($theme->folderName . "ThemeLocale{$locale}") => $fullPath2));
+                    \XoopsLoad::addMap(array(strtolower($theme->folderName."ThemeLocale{$locale}") => $fullPath2));
                 }
                 return true;
             }
@@ -346,7 +346,7 @@ 
 
         $p = [];
         foreach ($params as $name => $value) {
-            $p['{' . $name . '}'] = $value;
+            $p['{'.$name.'}'] = $value;
         }
 
         return strtr($message, $p);
@@ -372,7 +372,7 @@ 
      */
     protected static function getClassFromDirname($dirname)
     {
-        return ucfirst($dirname) . 'Locale';
+        return ucfirst($dirname).'Locale';
     }
 
     /**
@@ -385,7 +385,7 @@ 
         if (!$dirname) {
             $dirname = \Xoops::getInstance()->theme()->folderName;
         }
-        return ucfirst($dirname) . 'ThemeLocale';
+        return ucfirst($dirname).'ThemeLocale';
     }
 
     /**
@@ -461,8 +461,8 @@ 
             $keys = Data::explodeLocale($locale);
             $key = strtolower($keys['language']);
             $key .= (empty($keys['script']) || false === $withScript) ?
-                '' : $separator . ucfirst(strtolower($keys['script']));
-            $key .= empty($keys['territory']) ? '' : $separator . strtoupper($keys['territory']);
+                '' : $separator.ucfirst(strtolower($keys['script']));
+            $key .= empty($keys['territory']) ? '' : $separator.strtoupper($keys['territory']);
         } catch (InvalidLocale $e) {
             $key = '';
         }

xoops_lib/Xoops/Core/Session/Manager.php

Spacing +2 added lines, -2 removed lines

@@ -83,7 +83,7 @@ 
         $name = $this->xoops->getConfig('session_name');
         $name = (empty($name)) ? 'xoops_session' : $name;
         $expire = (int)($this->xoops->getConfig('session_expire'));
-        $expire = ($expire>0) ? $expire : 300;
+        $expire = ($expire > 0) ? $expire : 300;
 
         $path = \XoopsBaseConfig::get('cookie-path');
         $domain = \XoopsBaseConfig::get('cookie-domain');
@@ -198,7 +198,7 @@ 
         }
 
         // if we don't have the expires key, use a future value for test
-        if ($this->get('SESSION_MANAGER_EXPIRES', time()+10) < time()) {
+        if ($this->get('SESSION_MANAGER_EXPIRES', time() + 10) < time()) {
             return false;
         }
 

xoops_lib/Xoops/Core/Session/RememberMe.php

Spacing +2 added lines, -2 removed lines

@@ -96,7 +96,7 @@ 
         $this->now = time();
         $cookieData = $this->readUserCookie();
         if (false === $cookieData) {
-            return false;   // no or invalid cookie
+            return false; // no or invalid cookie
         }
         list($userId, $series, $token) = $cookieData;
         $this->readUserTokens($userId);
@@ -380,7 +380,7 @@ 
         if ($cookieData !== $notFound) {
             $temp = explode('-', $cookieData);
             if (count($temp) == 3) {
-                $temp[0] = (integer) $temp[0];
+                $temp[0] = (integer)$temp[0];
                 return $temp;
             }
             $this->clearUserCookie(); // clean up garbage cookie