XOOPS / XoopsCore

htdocs/modules/profile/changemail.php

Spacing +5 added lines, -5 removed lines

@@ -21,13 +21,13 @@ 
  * @author          Taiwen Jiang <phppp@users.sourceforge.net>
  */
 
-include __DIR__ . '/header.php';
+include __DIR__.'/header.php';
 
 $xoops = Xoops::getInstance();
 $xoops->getConfigs();
 
 if (!$xoops->user || $xoops->getConfig('allow_chgmail') != 1) {
-    $xoops->redirect(\XoopsBaseConfig::get('url') . "/modules/" . $xoops->module->getVar('dirname', 'n') . "/", 2, XoopsLocale::E_NO_ACCESS_PERMISSION);
+    $xoops->redirect(\XoopsBaseConfig::get('url')."/modules/".$xoops->module->getVar('dirname', 'n')."/", 2, XoopsLocale::E_NO_ACCESS_PERMISSION);
 }
 
 $xoops->header('module:profile/profile_email.tpl');
@@ -67,7 +67,7 @@ 
             $xoopsMailer->setTemplate('emailchanged.tpl');
             $xoopsMailer->assign("SITENAME", $xoops->getConfig('sitename'));
             $xoopsMailer->assign("ADMINMAIL", $xoops->getConfig('adminmail'));
-            $xoopsMailer->assign("SITEURL", \XoopsBaseConfig::get('url') . "/");
+            $xoopsMailer->assign("SITEURL", \XoopsBaseConfig::get('url')."/");
             $xoopsMailer->assign("NEWEMAIL", $email);
             $xoopsMailer->setToEmails($email);
             $xoopsMailer->setFromEmail($xoops->getConfig('adminmail'));
@@ -79,9 +79,9 @@ 
             $msg = implode('<br />', $xoops->user->getErrors());
         }
     }
-    $xoops->redirect(\XoopsBaseConfig::get('url') . '/modules/' . $xoops->module->getVar('dirname', 'n') . '/userinfo.php?uid=' . $xoops->user->getVar('uid'), 2, $msg);
+    $xoops->redirect(\XoopsBaseConfig::get('url').'/modules/'.$xoops->module->getVar('dirname', 'n').'/userinfo.php?uid='.$xoops->user->getVar('uid'), 2, $msg);
 }
 
 $xoops->registry()->get('profile_breadcrumbs')->addItem(new Link(['caption' => _PROFILE_MA_CHANGEMAIL]));
 
-include __DIR__ . '/footer.php';
+include __DIR__.'/footer.php';

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

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
         // View Account
         $ret[] = [
             'name' => XoopsLocale::VIEW_ACCOUNT,
-            'link' => $helper->url('userinfo.php?uid=' . $helper->xoops()->user->getVar('uid')),
+            'link' => $helper->url('userinfo.php?uid='.$helper->xoops()->user->getVar('uid')),
             'icon' => 'glyphicon-user',
         ];
 

htdocs/modules/profile/class/plugin/menus.php

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
         if (\Xoops::getInstance()->isUser()) {
             $ret[] = array('name' => _PROFILE_MI_EDITACCOUNT, 'url' => "edituser.php");
             $ret[] = array('name' => _PROFILE_MI_PAGE_SEARCH, 'url' => "search.php");
-            $ret[] = array('name' => _PROFILE_MI_CHANGEPASS,  'url' => "changepass.php");
+            $ret[] = array('name' => _PROFILE_MI_CHANGEPASS, 'url' => "changepass.php");
         }
         return $ret;
     }

htdocs/modules/profile/class/profile.php

Spacing +4 added lines, -4 removed lines

@@ -227,9 +227,9 @@ 
             $field->setVar('field_options', array());
         }
         if ($this->insertField($field)) {
-            $msg = '&nbsp;&nbsp;Field <strong>' . $vars['name'] . '</strong> added to the database';
+            $msg = '&nbsp;&nbsp;Field <strong>'.$vars['name'].'</strong> added to the database';
         } else {
-            $msg = '&nbsp;&nbsp;<span class="red">ERROR: Could not insert field <strong>' . $vars['name'] . '</strong> into the database. ' . implode(' ', $field->getErrors()) . $this->db2->errorInfo() . '</span>';
+            $msg = '&nbsp;&nbsp;<span class="red">ERROR: Could not insert field <strong>'.$vars['name'].'</strong> into the database. '.implode(' ', $field->getErrors()).$this->db2->errorInfo().'</span>';
         }
         unset($field);
         return $msg;
@@ -283,10 +283,10 @@ 
         $searchvars_profile = array_diff($searchvars, $uservars);
         $sv = array('u.uid, u.uname, u.email, u.user_viewemail');
         if (!empty($searchvars_user)) {
-            $sv[0] .= ",u." . implode(", u.", $searchvars_user);
+            $sv[0] .= ",u.".implode(", u.", $searchvars_user);
         }
         if (!empty($searchvars_profile)) {
-            $sv[] = "p." . implode(", p.", $searchvars_profile);
+            $sv[] = "p.".implode(", p.", $searchvars_profile);
         }
 
         $qb = $xoops->db()->createXoopsQueryBuilder();

htdocs/modules/profile/class/form/field.php

Spacing +1 added lines, -1 removed lines

@@ -124,7 +124,7 @@
                     $this->addElement($remove_options);
                 }
 
-                $option_text = "<table  cellspacing='1'><tr><td class='width20'>" . _PROFILE_AM_KEY . "</td><td>" . _PROFILE_AM_VALUE . "</td></tr>";
+                $option_text = "<table  cellspacing='1'><tr><td class='width20'>"._PROFILE_AM_KEY."</td><td>"._PROFILE_AM_VALUE."</td></tr>";
                 for ($i = 0; $i < 3; ++$i) {
                     $option_text .= "<tr><td><input type='text' name='addOption[{$i}][key]' id='addOption[{$i}][key]' size='15' /></td><td><input type='text' name='addOption[{$i}][value]' id='addOption[{$i}][value]' size='35' /></td></tr>";
                     $option_text .= "<tr height='3px'><td colspan='2'> </td></tr>";

htdocs/modules/profile/class/field.php

Spacing +11 added lines, -11 removed lines

@@ -143,7 +143,7 @@ 
 
                     $eltmsg = empty($caption) ? sprintf(XoopsLocale::F_ENTER, $name) : sprintf(XoopsLocale::F_ENTER, $caption);
                     $eltmsg = str_replace('"', '\"', stripslashes($eltmsg));
-                    $element->addCustomValidationCode("\nvar hasSelected = false; var selectBox = myform.{$name};" . "for (i = 0; i < selectBox.options.length; i++  ) { if ( selectBox.options[i].selected == true && selectBox.options[i].value != '' ) { hasSelected = true; break; } }" . "if ( !hasSelected ) { window.alert(\"{$eltmsg}\"); selectBox.focus(); return false; }");
+                    $element->addCustomValidationCode("\nvar hasSelected = false; var selectBox = myform.{$name};"."for (i = 0; i < selectBox.options.length; i++  ) { if ( selectBox.options[i].selected == true && selectBox.options[i].value != '' ) { hasSelected = true; break; } }"."if ( !hasSelected ) { window.alert(\"{$eltmsg}\"); selectBox.focus(); return false; }");
                 }
                 $element->addOptionArray($options);
                 break;
@@ -209,11 +209,11 @@ 
             case 'theme':
                 $element = new Xoops\Form\Select($caption, $name, $value);
                 $element->addOption("0", _PROFILE_MA_SITEDEFAULT);
-                $handle = opendir(\XoopsBaseConfig::get('themes-path') . '/');
+                $handle = opendir(\XoopsBaseConfig::get('themes-path').'/');
                 $dirlist = array();
                 while (false !== ($file = readdir($handle))) {
-                    if (is_dir(\XoopsBaseConfig::get('themes-path') . '/' . $file) && !preg_match("/^[.]{1,2}$/", $file) && strtolower($file) !== 'cvs') {
-                        if (XoopsLoad::fileExists(\XoopsBaseConfig::get('themes-path') . "/" . $file . "/theme.tpl") && in_array($file, $xoops->getConfig('theme_set_allowed'))) {
+                    if (is_dir(\XoopsBaseConfig::get('themes-path').'/'.$file) && !preg_match("/^[.]{1,2}$/", $file) && strtolower($file) !== 'cvs') {
+                        if (XoopsLoad::fileExists(\XoopsBaseConfig::get('themes-path')."/".$file."/theme.tpl") && in_array($file, $xoops->getConfig('theme_set_allowed'))) {
                             $dirlist[$file] = $file;
                         }
                     }
@@ -251,7 +251,7 @@ 
             default:
             case "textbox":
                 if ($this->getVar('field_name') === 'url' && $value != '') {
-                    return '<a href="' . $xoops->formatURL($value) . '" rel="external">' . $value . '</a>';
+                    return '<a href="'.$xoops->formatURL($value).'" rel="external">'.$value.'</a>';
                 } else {
                     return $value;
                 }
@@ -341,9 +341,9 @@ 
                 $userrank = $user->rank();
                 $user_rankimage = "";
                 if (isset($userrank['image']) && $userrank['image'] != "") {
-                    $user_rankimage = '<img src="' . $userrank['image'] . '" alt="' . $userrank['title'] . '" /><br />';
+                    $user_rankimage = '<img src="'.$userrank['image'].'" alt="'.$userrank['title'].'" /><br />';
                 }
-                return $user_rankimage . $userrank['title'];
+                return $user_rankimage.$userrank['title'];
                 break;
 
             case "yesno":
@@ -518,9 +518,9 @@ 
                 $changetype = "ADD";
             } else {
                 //update column information
-                $changetype = "CHANGE `" . $obj->getVar('field_name', 'n') . "`";
+                $changetype = "CHANGE `".$obj->getVar('field_name', 'n')."`";
             }
-            $maxlengthstring = $obj->getVar('field_maxlength') > 0 ? "(" . $obj->getVar('field_maxlength') . ")" : "";
+            $maxlengthstring = $obj->getVar('field_maxlength') > 0 ? "(".$obj->getVar('field_maxlength').")" : "";
 
             //set type
             switch ($obj->getVar('field_valuetype')) {
@@ -562,7 +562,7 @@ 
                     break;
             }
 
-            $sql = "ALTER TABLE `" . $profile_handler->table . "` " . $changetype . " `" . $obj->cleanVars['field_name'] . "` " . $type . $maxlengthstring . ' NULL';
+            $sql = "ALTER TABLE `".$profile_handler->table."` ".$changetype." `".$obj->cleanVars['field_name']."` ".$type.$maxlengthstring.' NULL';
             if ($force) {
                 $this->db2->setForce(true);
             }
@@ -592,7 +592,7 @@ 
         $xoops = Xoops::getInstance();
         $profile_handler = \Xoops::getModuleHelper('profile')->getHandler('profile');
         // remove column from table
-        $sql = "ALTER TABLE " . $profile_handler->table . " DROP `" . $obj->getVar('field_name', 'n') . "`";
+        $sql = "ALTER TABLE ".$profile_handler->table." DROP `".$obj->getVar('field_name', 'n')."`";
         if ($this->db2->query($sql)) {
             //change this to update the cached field information storage
             if (!parent::delete($obj, $force)) {

htdocs/modules/profile/index.php

Spacing +11 added lines, -11 removed lines

@@ -22,7 +22,7 @@ 
  * @author          Taiwen Jiang <phppp@users.sourceforge.net>
  */
 
-include __DIR__ . '/header.php';
+include __DIR__.'/header.php';
 $xoops = Xoops::getInstance();
 
 $op = 'main';
@@ -50,7 +50,7 @@ 
         $xoops->tpl()->assign('lang_youremail', XoopsLocale::C_YOUR_EMAIL);
         $xoops->tpl()->assign('lang_sendpassword', XoopsLocale::SEND_PASSWORD);
         $xoops->tpl()->assign('mailpasswd_token', $xoops->security()->createToken());
-        include __DIR__ . '/footer.php';
+        include __DIR__.'/footer.php';
     }
     if (!empty($_GET['xoops_redirect'])) {
         $redirect = trim($_GET['xoops_redirect']);
@@ -62,11 +62,11 @@ 
             }
         }
         if (!$isExternal) {
-            header('Location: ' . $redirect);
+            header('Location: '.$redirect);
             exit();
         }
     }
-    header('Location: ./userinfo.php?uid=' . $xoops->user->getVar('uid'));
+    header('Location: ./userinfo.php?uid='.$xoops->user->getVar('uid'));
     exit();
 }
 
@@ -83,8 +83,8 @@ 
     if ($xoops->isUser()) {
         $xoops->getHandlerOnline()->destroy($xoops->user->getVar('uid'));
     }
-    $message = XoopsLocale::S_YOU_ARE_NOW_LOGGED_OUT . '<br />' . XoopsLocale::S_THANK_YOU_FOR_VISITING_OUR_SITE;
-    $xoops->redirect(\XoopsBaseConfig::get('url') . '/', 1, $message);
+    $message = XoopsLocale::S_YOU_ARE_NOW_LOGGED_OUT.'<br />'.XoopsLocale::S_THANK_YOU_FOR_VISITING_OUR_SITE;
+    $xoops->redirect(\XoopsBaseConfig::get('url').'/', 1, $message);
 }
 
 if ($op === 'actv') {
@@ -95,12 +95,12 @@ 
 
 if ($op === 'delete') {
     if (!$xoops->isUser() || $xoops->getConfig('self_delete') != 1) {
-        $xoops->redirect(\XoopsBaseConfig::get('url') . '/', 5, XoopsLocale::E_NO_ACTION_PERMISSION);
+        $xoops->redirect(\XoopsBaseConfig::get('url').'/', 5, XoopsLocale::E_NO_ACTION_PERMISSION);
     } else {
         $groups = $xoops->user->getGroups();
         if (in_array(FixedGroups::ADMIN, $groups)) {
             // users in the webmasters group may not be deleted
-            $xoops->redirect(\XoopsBaseConfig::get('url') . '/', 5, XoopsLocale::E_USER_IN_WEBMASTER_GROUP_CANNOT_BE_REMOVED);
+            $xoops->redirect(\XoopsBaseConfig::get('url').'/', 5, XoopsLocale::E_USER_IN_WEBMASTER_GROUP_CANNOT_BE_REMOVED);
         }
         $ok = !isset($_POST['ok']) ? 0 : (int)($_POST['ok']);
         if ($ok != 1) {
@@ -108,9 +108,9 @@ 
             echo $xoops->confirm(
                 array('op' => 'delete', 'ok' => 1),
                 'user.php',
-                XoopsLocale::Q_ARE_YOU_SURE_TO_DELETE_ACCOUNT . '<br/>' . XoopsLocale::THIS_WILL_REMOVE_ALL_YOUR_INFO
+                XoopsLocale::Q_ARE_YOU_SURE_TO_DELETE_ACCOUNT.'<br/>'.XoopsLocale::THIS_WILL_REMOVE_ALL_YOUR_INFO
             );
-            include __DIR__ . '/footer.php';
+            include __DIR__.'/footer.php';
         } else {
             $del_uid = $xoops->user->getVar("uid");
             if (false != $xoops->getHandlerMember()->deleteUser($xoops->user)) {
@@ -122,7 +122,7 @@ 
                 }
                 $xoops->redirect('index.php', 5, XoopsLocale::S_YOUR_ACCOUNT_DELETED);
             }
-            $xoops->redirect(XOOPS_URL . '/', 5, XoopsLocale::E_NO_ACTION_PERMISSION);
+            $xoops->redirect(XOOPS_URL.'/', 5, XoopsLocale::E_NO_ACTION_PERMISSION);
         }
     }
 }

htdocs/modules/profile/header.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
  * @author          Taiwen Jiang <phppp@users.sourceforge.net>
  */
 
-include dirname(dirname(__DIR__)) . '/mainfile.php';
+include dirname(dirname(__DIR__)).'/mainfile.php';
 $xoops = Xoops::getInstance();
 
 $xoops->registry()->set('profile_breadcrumbs', new ItemList());

htdocs/modules/pdf/admin/index.php

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
  * @since           2.6.0
  * @author          Mage Grégory (AKA Mage)
  */
-include __DIR__ . '/header.php';
+include __DIR__.'/header.php';
 
 $xoops = Xoops::getInstance();