XOOPS / XoopsCore

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

xoops_lib/Xoops/Core/Session/Fingerprint.php

Spacing +2 added lines, -2 removed lines

@@ -77,7 +77,7 @@ 
      */
     public function checkSessionPrint(AttributeInterface $session)
     {
-        $score = 0;   // combined levenshtein distance of changes
+        $score = 0; // combined levenshtein distance of changes
         $changes = 0; // number of changed fields
         $currentFingerprint = $this->takePrint();
         $savedFingerprint = unserialize($session->get('SESSION_FINGERPRINT'));
@@ -89,7 +89,7 @@ 
         foreach ($currentFingerprint as $key => $current) {
             $distance = levenshtein($current, $savedFingerprint[$key]);
             $score += $distance;
-            $changes += ($distance>0) ? 1 : 0;
+            $changes += ($distance > 0) ? 1 : 0;
         }
 
         $return = true;

xoops_lib/Xoops/Core/Session/Handler.php

Spacing +4 added lines, -4 removed lines

@@ -111,7 +111,7 @@ 
      **/
     public function write($session_id, $session_data)
     {
-        $expires =  (isset($_SESSION['SESSION_MANAGER_EXPIRES']))
+        $expires = (isset($_SESSION['SESSION_MANAGER_EXPIRES']))
             ? (int)($_SESSION['SESSION_MANAGER_EXPIRES'])
             : time() + (session_cache_expire() * 60);
         //$oldIsolation = $this->db->getTransactionIsolation();
@@ -129,7 +129,7 @@ 
             ->setParameter(':sessdata', $session_data, \PDO::PARAM_STR);
         $this->db->setForce(true);
         $result = $qb->execute();
-        if ($result<=0) {
+        if ($result <= 0) {
             $qb = $this->db->createXoopsQueryBuilder();
             $qb ->insertPrefix($this->sessionTable)
                 ->values(array(
@@ -146,7 +146,7 @@ 
         //$this->db->commit();
         //$this->db->setTransactionIsolation($oldIsolation);
 
-        return (boolean) ($result>0);
+        return (boolean)($result > 0);
     }
 
     /**
@@ -186,6 +186,6 @@ 
             ->setParameter(':expires', $mintime, \PDO::PARAM_INT);
         $this->db->setForce(true);
         $result = $qb->execute();
-        return (boolean) ($result>0);
+        return (boolean)($result > 0);
     }
 }

xoops_lib/Xoops/Core/Cache/CacheManager.php

Indentation +2 added lines, -2 removed lines

@@ -57,8 +57,8 @@
     {
         $this->xoops = \Xoops::getInstance();
         $defaults = $this->getDefaults();
-		$xoops_var_path = \XoopsBaseConfig::get('var-path');
-		$cache_file = $xoops_var_path . '/configs/cache.php';
+        $xoops_var_path = \XoopsBaseConfig::get('var-path');
+        $cache_file = $xoops_var_path . '/configs/cache.php';
         $poolDefs = Yaml::readWrapped($cache_file);
         if (empty($poolDefs)) {
             Yaml::saveWrapped($defaults, $cache_file);

Spacing +9 added lines, -9 removed lines

@@ -58,7 +58,7 @@ 
         $this->xoops = \Xoops::getInstance();
         $defaults = $this->getDefaults();
 		$xoops_var_path = \XoopsBaseConfig::get('var-path');
-		$cache_file = $xoops_var_path . '/configs/cache.php';
+		$cache_file = $xoops_var_path.'/configs/cache.php';
         $poolDefs = Yaml::readWrapped($cache_file);
         if (empty($poolDefs)) {
             Yaml::saveWrapped($defaults, $cache_file);
@@ -78,7 +78,7 @@ 
         $defaults = [
             'default' => [
                 'driver' => 'Sqlite',
-                'options' => ['path' => \XoopsBaseConfig::get('var-path') . '/stash/'],
+                'options' => ['path' => \XoopsBaseConfig::get('var-path').'/stash/'],
                 ],
             'temp' => [
                 'driver' => 'Ephemeral',
@@ -111,7 +111,7 @@ 
      */
     public static function createDefaultConfig()
     {
-        $configFile = \XoopsBaseConfig::get('var-path') . '/configs/cache.php';
+        $configFile = \XoopsBaseConfig::get('var-path').'/configs/cache.php';
         if (file_exists($configFile)) {
             return;
         }
@@ -120,7 +120,7 @@ 
             $defaults['default']['driver'] = 'FileSystem';
             $defaults['default']['options'] = [
                 'dirSplit' => 1,
-                'path' => \XoopsBaseConfig::get('var-path') . '/stash/'
+                'path' => \XoopsBaseConfig::get('var-path').'/stash/'
             ];
             if (false !== stripos(PHP_OS, 'WIN')) {
                 trigger_error("SQLite is strongly recommended on windows due to 260 character file path restrictions.");
@@ -140,9 +140,9 @@ 
     {
         $pool = false;
         if (array_key_exists($name, $this->pools)) {
-            $pool =  $this->pools[$name];
+            $pool = $this->pools[$name];
         } elseif (array_key_exists($name, $this->poolDefs)) {
-            $pool =  $this->startPoolAccess($name);
+            $pool = $this->startPoolAccess($name);
         }
         if ($pool === false) {
             $pool = $this->getDefaultPool($name);
@@ -178,7 +178,7 @@ 
         }
 
         $driver = $this->getDriver($driverName, $options);
-        if ($driver!==false) {
+        if ($driver !== false) {
             $pool = new Pool($driver);
             if (is_object($pool)) {
                 $pool->setLogger($this->xoops->logger());
@@ -206,7 +206,7 @@ 
         $driver = false;
         $driverClass = DriverList::getDriverClass($driverName);
 
-        if ($driverClass!==false && $driverClass::isAvailable()) {
+        if ($driverClass !== false && $driverClass::isAvailable()) {
             $options = is_array($options) ? $options : array();
             $driver = new $driverClass($options);
         }
@@ -229,7 +229,7 @@ 
             return $this->pools[$name];
         }
         $pool = $this->startPoolAccess($name);
-        if ($pool===false) {
+        if ($pool === false) {
             $this->xoops->logger()->error('Could not create default cache pool');
             $pool = new Access(new \Stash\Pool());
         }

xoops_lib/Xoops/Core/Cache/Legacy.php

Spacing +4 added lines, -4 removed lines

@@ -34,12 +34,12 @@
         $stack = debug_backtrace();
         $frameSelf = $stack[1];
         $frame = isset($stack[2]) ? $stack[2] : false;
-        $append = ' ' . get_called_class() . '::' . $frameSelf['function'] . '() called from ';
+        $append = ' '.get_called_class().'::'.$frameSelf['function'].'() called from ';
         if ($frame !== false) {
-            $append .= $frame['function'] . '() in ';
+            $append .= $frame['function'].'() in ';
         }
-        $append .= $frameSelf['file'] . ' line '. $frameSelf['line'];
-        \Xoops::getInstance()->deprecated($message . $append);
+        $append .= $frameSelf['file'].' line '.$frameSelf['line'];
+        \Xoops::getInstance()->deprecated($message.$append);
     }
 
     /**