XOOPS / XoopsCore

xoops_lib/Xoops/Module/Helper.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
             if ($xoops->isActiveModule($dirname)) {
                 //Load Module helper if available
                 if (\XoopsLoad::loadFile($xoops->path("modules/{$dirname}/class/helper.php"))) {
-                    $className = '\\' . ucfirst($dirname);
+                    $className = '\\'.ucfirst($dirname);
                     if (class_exists($className)) {
                         $class = new $className();
                         if ($class instanceof \Xoops\Module\Helper\HelperAbstract) {

xoops_lib/Xoops/Module/Plugin.php

Spacing +2 added lines, -2 removed lines

@@ -70,8 +70,8 @@
             }
             foreach ($dirnames as $dirname) {
                 if (\XoopsLoad::loadFile($xoops->path("modules/{$dirname}/class/plugin/{$pluginName}.php"))) {
-                    $className = '\\' . ucfirst($dirname) . ucfirst($pluginName) . 'Plugin';
-                    $interface = '\\' . ucfirst($pluginName) . 'PluginInterface';
+                    $className = '\\'.ucfirst($dirname).ucfirst($pluginName).'Plugin';
+                    $interface = '\\'.ucfirst($pluginName).'PluginInterface';
                     $class = new $className($dirname);
                     if ($class instanceof $interface) {
                         static::$plugins[$pluginName][$dirname] = $class;

xoops_lib/Xoops/Module/Admin.php

Spacing +38 added lines, -38 removed lines

@@ -283,21 +283,21 @@ 
      */
     public function addConfigModuleVersion($moddir, $minversion)
     {
-        $return=false;
-        $helper= \Xoops::getInstance()->getModuleHelper($moddir);
+        $return = false;
+        $helper = \Xoops::getInstance()->getModuleHelper($moddir);
         if (is_object($helper) && is_object($helper->getModule())) {
-            $mod_modversion=$helper->getModule()->getVar('version');
-            $mod_version_f = $mod_modversion/100;
-            $min_version_f = $minversion/100;
+            $mod_modversion = $helper->getModule()->getVar('version');
+            $mod_version_f = $mod_modversion / 100;
+            $min_version_f = $minversion / 100;
             $value = sprintf(
                 \XoopsLocale::EF_MODULE_VERSION,
                 strtoupper($moddir),
                 $min_version_f,
                 $mod_version_f
             );
-            if ($mod_modversion>=$minversion) {
+            if ($mod_modversion >= $minversion) {
                 $this->addConfigAccept($value);
-                $return=true;
+                $return = true;
             } else {
                 $this->addConfigError($value);
             }
@@ -305,7 +305,7 @@ 
             $value = sprintf(
                 \XoopsLocale::EF_MODULE_NOTFOUND,
                 strtoupper($moddir),
-                $minversion/100
+                $minversion / 100
             );
             $this->addConfigError($value);
         }
@@ -396,7 +396,7 @@ 
      */
     private function getTplPath($type = '')
     {
-        return 'admin:' . $this->tplModule . '/' . $this->tplFile[$type];
+        return 'admin:'.$this->tplModule.'/'.$this->tplFile[$type];
     }
 
     /**
@@ -484,19 +484,19 @@ 
         $xoops = \Xoops::getInstance();
         $this->module->loadAdminMenu();
         foreach (array_keys($this->module->adminmenu) as $i) {
-            if (\XoopsLoad::fileExists($xoops->path("media/xoops/images/icons/32/" . $this->module->adminmenu[$i]['icon']))) {
-                $this->module->adminmenu[$i]['icon'] = $xoops->url("media/xoops/images/icons/32/" . $this->module->adminmenu[$i]['icon']);
-            } elseif (\XoopsLoad::fileExists($xoops->path("modules/" . $xoops->module->dirname() . "/assets/icons/32/" . $this->module->adminmenu[$i]['icon']))) {
-                $this->module->adminmenu[$i]['icon'] = $xoops->url("modules/" . $xoops->module->dirname() . "/assets/icons/32/" . $this->module->adminmenu[$i]['icon']);
+            if (\XoopsLoad::fileExists($xoops->path("media/xoops/images/icons/32/".$this->module->adminmenu[$i]['icon']))) {
+                $this->module->adminmenu[$i]['icon'] = $xoops->url("media/xoops/images/icons/32/".$this->module->adminmenu[$i]['icon']);
+            } elseif (\XoopsLoad::fileExists($xoops->path("modules/".$xoops->module->dirname()."/assets/icons/32/".$this->module->adminmenu[$i]['icon']))) {
+                $this->module->adminmenu[$i]['icon'] = $xoops->url("modules/".$xoops->module->dirname()."/assets/icons/32/".$this->module->adminmenu[$i]['icon']);
             } else {
-                $this->module->adminmenu[$i]['icon'] = $xoops->url("modules/" . $xoops->module->dirname() . "/icons/32/" . $this->module->adminmenu[$i]['icon']);
+                $this->module->adminmenu[$i]['icon'] = $xoops->url("modules/".$xoops->module->dirname()."/icons/32/".$this->module->adminmenu[$i]['icon']);
             }
             $xoops->tpl()->append('xo_admin_index_menu', $this->module->adminmenu[$i]);
         }
         if ($this->module->getInfo('help')) {
             $help = array();
-            $help['link'] = '../system/help.php?mid=' . $this->module->getVar('mid', 's')
-                . "&" . $this->module->getInfo('help');
+            $help['link'] = '../system/help.php?mid='.$this->module->getVar('mid', 's')
+                . "&".$this->module->getInfo('help');
             $help['icon'] = $xoops->url("/media/xoops/images/icons/32/help.png");
             $help['title'] = \XoopsLocale::HELP;
             $xoops->tpl()->append('xo_admin_index_menu', $help);
@@ -557,7 +557,7 @@ 
                     if (0 >= version_compare($dbCurrentVersion, $dbRequiredVersion)) {
                         $this->addConfigBoxLine(
                             sprintf(
-                                strtoupper(\XoopsBaseConfig::get('db-type')) . ' '
+                                strtoupper(\XoopsBaseConfig::get('db-type')).' '
                                 . \XoopsLocale::F_MINIMUM_DATABASE_VERSION_REQUIRED,
                                 $dbRequiredVersion,
                                 $dbCurrentVersion
@@ -567,7 +567,7 @@ 
                     } else {
                         $this->addConfigBoxLine(
                             sprintf(
-                                strtoupper(\XoopsBaseConfig::get('db-type')) . ' ' . \XoopsLocale::F_MINIMUM_DATABASE_VERSION_REQUIRED,
+                                strtoupper(\XoopsBaseConfig::get('db-type')).' '.\XoopsLocale::F_MINIMUM_DATABASE_VERSION_REQUIRED,
                                 $dbRequiredVersion,
                                 $dbCurrentVersion
                             ),
@@ -635,13 +635,13 @@ 
 
         $this->module->loadAdminMenu();
         foreach (array_keys($this->module->adminmenu) as $i) {
-            if ($this->module->adminmenu[$i]['link'] == "admin/" . $menu) {
-                if (\XoopsLoad::fileExists($xoops->path("media/xoops/images/icons/32/" . $this->module->adminmenu[$i]['icon']))) {
-                    $this->module->adminmenu[$i]['icon'] = $xoops->url("media/xoops/images/icons/32/". $this->module->adminmenu[$i]['icon']);
-                } elseif (\XoopsLoad::fileExists($xoops->path("modules/" . $xoops->module->dirname() . "/assets/icons/32/" . $this->module->adminmenu[$i]['icon']))) {
-                    $this->module->adminmenu[$i]['icon'] = $xoops->url("modules/" . $xoops->module->dirname() . "/assets/icons/32/" . $this->module->adminmenu[$i]['icon']);
+            if ($this->module->adminmenu[$i]['link'] == "admin/".$menu) {
+                if (\XoopsLoad::fileExists($xoops->path("media/xoops/images/icons/32/".$this->module->adminmenu[$i]['icon']))) {
+                    $this->module->adminmenu[$i]['icon'] = $xoops->url("media/xoops/images/icons/32/".$this->module->adminmenu[$i]['icon']);
+                } elseif (\XoopsLoad::fileExists($xoops->path("modules/".$xoops->module->dirname()."/assets/icons/32/".$this->module->adminmenu[$i]['icon']))) {
+                    $this->module->adminmenu[$i]['icon'] = $xoops->url("modules/".$xoops->module->dirname()."/assets/icons/32/".$this->module->adminmenu[$i]['icon']);
                 } else {
-                    $this->module->adminmenu[$i]['icon'] = $xoops->url("modules/" . $xoops->module->dirname() . "/icons/32/". $this->module->adminmenu[$i]['icon']);
+                    $this->module->adminmenu[$i]['icon'] = $xoops->url("modules/".$xoops->module->dirname()."/icons/32/".$this->module->adminmenu[$i]['icon']);
                 }
                 $xoops->tpl()->assign('xo_sys_navigation', $this->module->adminmenu[$i]);
                 $ret[] = $xoops->tpl()->fetch($this->getTplPath('nav'));
@@ -707,7 +707,7 @@ 
         foreach (array_keys($author) as $i) {
             $author_list .= $author[$i];
             if (isset($nickname[$i]) && $nickname[$i] != '') {
-                $author_list .= " (" . $nickname[$i] . "), ";
+                $author_list .= " (".$nickname[$i]."), ";
             } else {
                 $author_list .= ", ";
             }
@@ -715,19 +715,19 @@ 
         $changelog = '';
         $language = $xoops->getConfig('locale');
         if (!is_file(
-            \XoopsBaseConfig::get('root-path') . "/modules/" . $this->module->getVar("dirname")
-            . "/locale/" . $language . "/changelog.txt"
+            \XoopsBaseConfig::get('root-path')."/modules/".$this->module->getVar("dirname")
+            . "/locale/".$language."/changelog.txt"
         )) {
             $language = 'en_US';
         }
-        $file = \XoopsBaseConfig::get('root-path') . "/modules/" . $this->module->getVar("dirname")
-            . "/locale/" . $language . "/changelog.txt";
+        $file = \XoopsBaseConfig::get('root-path')."/modules/".$this->module->getVar("dirname")
+            . "/locale/".$language."/changelog.txt";
         if (is_readable($file)) {
-            $changelog = utf8_encode(implode("<br />", file($file))) . "\n";
+            $changelog = utf8_encode(implode("<br />", file($file)))."\n";
         } else {
-            $file = \XoopsBaseConfig::get('root-path') . "/modules/" . $this->module->getVar("dirname") . "/docs/changelog.txt";
+            $file = \XoopsBaseConfig::get('root-path')."/modules/".$this->module->getVar("dirname")."/docs/changelog.txt";
             if (is_readable($file)) {
-                $changelog = utf8_encode(implode("<br />", file($file))) . "\n";
+                $changelog = utf8_encode(implode("<br />", file($file)))."\n";
             }
         }
         $author_list = substr($author_list, 0, -2);
@@ -742,22 +742,22 @@ 
 
         $this->addInfoBox(\XoopsLocale::MODULE_INFORMATION, 'info', 'id="xo-about"');
         $this->addInfoBoxLine(
-            \XoopsLocale::C_DESCRIPTION . ' ' . $this->module->getInfo("description"),
+            \XoopsLocale::C_DESCRIPTION.' '.$this->module->getInfo("description"),
             'info'
         );
         $this->addInfoBoxLine(
-            \XoopsLocale::C_UPDATE_DATE . ' <span class="bold">'
+            \XoopsLocale::C_UPDATE_DATE.' <span class="bold">'
             . \XoopsLocale::formatTimestamp($this->module->getVar("last_update"), "m")
             . '</span>',
             'info'
         );
         $this->addInfoBoxLine(
-            \XoopsLocale::C_WEBSITE . ' <a class="xo-tooltip" href="http://'
+            \XoopsLocale::C_WEBSITE.' <a class="xo-tooltip" href="http://'
             . $this->module->getInfo("module_website_url")
             . '" rel="external" title="'
-            . $this->module->getInfo("module_website_name") . ' - '
-            . $this->module->getInfo("module_website_url") . '">'
-            . $this->module->getInfo("module_website_name") . '</a>',
+            . $this->module->getInfo("module_website_name").' - '
+            . $this->module->getInfo("module_website_url").'">'
+            . $this->module->getInfo("module_website_name").'</a>',
             'info'
         );
 

xoops_lib/Xoops/Locale/AbstractLocale.php

Spacing +1 added lines, -1 removed lines

@@ -171,7 +171,7 @@
     public static function substr($str, $start, $length, $ellipsis = '…')
     {
         $str2 = mb_strcut($str, $start, $length - strlen($ellipsis));
-        return $str2 . (mb_strlen($str)-$start != mb_strlen($str2) ? $ellipsis : '');
+        return $str2.(mb_strlen($str) - $start != mb_strlen($str2) ? $ellipsis : '');
     }
 
     /**

xoops_lib/Xoops/Html/Img.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
      */
     public function render()
     {
-        $tag = '<img ' . $this->renderAttributeString() . ' />';
+        $tag = '<img '.$this->renderAttributeString().' />';
 
         return $tag;
     }

xoops_lib/Xoops/Html/Menu/Render/BreadCrumb.php

Spacing +2 added lines, -2 removed lines

@@ -60,13 +60,13 @@
                 $anchorEnd = '';
                 $liClass = ' class="active"';
                 if ($item->has('link')) {
-                    $anchorStart = '<a href="' . $this->xoops->url($item->get('link')) . '">';
+                    $anchorStart = '<a href="'.$this->xoops->url($item->get('link')).'">';
                     $anchorEnd = '</a>';
                     $liClass = '';
                 }
                 $caption = $item->get('caption', '');
                 $icon = $item->has('icon') ?
-                    '<span class="' . $item->get('icon') . '" aria-hidden="true"></span> ' : '';
+                    '<span class="'.$item->get('icon').'" aria-hidden="true"></span> ' : '';
                 $renderedItems .= "<li{$liClass}>{$anchorStart}{$icon}{$caption}{$anchorEnd}</li>";
                 break;
             case Item::TYPE_LIST:

xoops_lib/Xoops/Html/Menu/Render/DropDownButton.php

Spacing +5 added lines, -5 removed lines

@@ -38,10 +38,10 @@ 
         $dropdown = $menu->get('dropdown', 'dropdown');
         $renderedMenu = "<div class=\"{$dropdown}\">\n";
         $class = $menu->get('class', 'btn btn-default dropdown-toggle');
-        $id = ($menu->has('id')) ? ' id="' . $menu->get('id') . '"' : '';
-        $labeledId = ($menu->has('id')) ? ' aria-labelledby="' . $menu->get('id') . '"' : '';
+        $id = ($menu->has('id')) ? ' id="'.$menu->get('id').'"' : '';
+        $labeledId = ($menu->has('id')) ? ' aria-labelledby="'.$menu->get('id').'"' : '';
         $caption = $menu->get('caption', '');
-        $icon = $menu->has('icon') ? '<span class="' . $menu->get('icon') . '" aria-hidden="true"></span> ' : '';
+        $icon = $menu->has('icon') ? '<span class="'.$menu->get('icon').'" aria-hidden="true"></span> ' : '';
         $renderedMenu .= <<<EOT
 <button class="{$class}" type="button"{$id} data-toggle="dropdown" aria-haspopup="true" aria-expanded="true">
 {$icon}{$caption} <span class="caret"></span>
@@ -74,13 +74,13 @@ 
                 $anchorEnd = '';
                 $liClass = ' class="active"';
                 if ($item->has('link')) {
-                    $anchorStart = '<a href="' . $this->xoops->url($item->get('link')) . '">';
+                    $anchorStart = '<a href="'.$this->xoops->url($item->get('link')).'">';
                     $anchorEnd = '</a>';
                     $liClass = '';
                 }
                 $caption = $item->get('caption', '');
                 $icon = $item->has('icon') ?
-                    '<span class="' . $item->get('icon') . '" aria-hidden="true"></span> ' : '';
+                    '<span class="'.$item->get('icon').'" aria-hidden="true"></span> ' : '';
                 $renderedItems .= "<li{$liClass}>{$anchorStart}{$icon}{$caption}{$anchorEnd}</li>";
                 break;
             case Item::TYPE_LIST:

xoops_lib/Xoops/Auth/Provisioning.php

Spacing +2 added lines, -2 removed lines

@@ -197,7 +197,7 @@ 
 
             return $newuser;
         } else {
-            $xoops->redirect(\XoopsBaseConfig::get('url') . '/user.php', 5, $newuser->getHtmlErrors());
+            $xoops->redirect(\XoopsBaseConfig::get('url').'/user.php', 5, $newuser->getHtmlErrors());
         }
 
         return $ret;
@@ -225,7 +225,7 @@ 
         if ($member_handler->insertUser($xoopsUser)) {
             return $xoopsUser;
         } else {
-            $xoops->redirect(\XoopsBaseConfig::get('url') . '/user.php', 5, $xoopsUser->getHtmlErrors());
+            $xoops->redirect(\XoopsBaseConfig::get('url').'/user.php', 5, $xoopsUser->getHtmlErrors());
         }
 
         return $ret;

xoops_lib/Xoops/Auth/AuthAbstract.php

Spacing +2 added lines, -2 removed lines

@@ -94,10 +94,10 @@
         if ($xoops->getConfig('debug_mode') == 1 || $xoops->getConfig('debug_mode') == 2) {
             if (!empty($this->errors)) {
                 foreach ($this->errors as $errstr) {
-                    $ret .= $errstr . '<br/>';
+                    $ret .= $errstr.'<br/>';
                 }
             } else {
-                $ret .= \XoopsLocale::NONE . '<br />';
+                $ret .= \XoopsLocale::NONE.'<br />';
             }
             $ret .= sprintf(\XoopsLocale::F_USING_AUTHENTICATION_METHOD, $this->auth_method);
         } else {