XOOPS / XoopsCore25

htdocs/include/findusers.php

Indentation +16 added lines, -16 removed lines

@@ -352,14 +352,14 @@ 
 
             $mailok_radio = new XoopsFormRadio(_MA_USER_SHOWMAILOK, 'user_mailok', empty($_POST['user_mailok']) ? 'both' : $_POST['user_mailok']);
             $mailok_radio->addOptionArray(array(
-                                              'mailok' => _MA_USER_MAILOK,
-                                              'mailng' => _MA_USER_MAILNG,
-                                              'both' => _MA_USER_BOTH));
+                                                'mailok' => _MA_USER_MAILOK,
+                                                'mailng' => _MA_USER_MAILNG,
+                                                'both' => _MA_USER_BOTH));
             $avatar_radio = new XoopsFormRadio(_MA_USER_HASAVATAR, 'user_avatar', empty($_POST['user_avatar']) ? 'both' : $_POST['user_avatar']);
             $avatar_radio->addOptionArray(array(
-                                              'y' => _YES,
-                                              'n' => _NO,
-                                              'both' => _MA_USER_BOTH));
+                                                'y' => _YES,
+                                                'n' => _NO,
+                                                'both' => _MA_USER_BOTH));
 
             $level_radio = new XoopsFormRadio(_MA_USER_LEVEL, 'level', @$_POST['level']);
             $levels      = array(
@@ -391,8 +391,8 @@ 
             $form->addElement($rank_select);
         } else {
             foreach (array(
-                         'uname',
-                         'email') as $var) {
+                            'uname',
+                            'email') as $var) {
                 $title      = $items_match[$var];
                 $text       = new XoopsFormText('', $var, 30, 100, @$_POST[$var]);
                 $match      = new XoopsFormSelectMatchOption('', "{$var}_match", @$_POST["{$var}_match"]);
@@ -406,14 +406,14 @@ 
 
         $sort_select = new XoopsFormSelect(_MA_USER_SORT, 'user_sort', @$_POST['user_sort']);
         $sort_select->addOptionArray(array(
-                                         'uname' => _MA_USER_UNAME,
-                                         'last_login' => _MA_USER_LASTLOGIN,
-                                         'user_regdate' => _MA_USER_REGDATE,
-                                         'posts' => _MA_USER_POSTS));
+                                            'uname' => _MA_USER_UNAME,
+                                            'last_login' => _MA_USER_LASTLOGIN,
+                                            'user_regdate' => _MA_USER_REGDATE,
+                                            'posts' => _MA_USER_POSTS));
         $order_select = new XoopsFormSelect(_MA_USER_ORDER, 'user_order', @$_POST['user_order']);
         $order_select->addOptionArray(array(
-                                          'ASC' => _MA_USER_ASC,
-                                          'DESC' => _MA_USER_DESC));
+                                            'ASC' => _MA_USER_ASC,
+                                            'DESC' => _MA_USER_DESC));
 
         $form->addElement($sort_select);
         $form->addElement($order_select);
@@ -479,8 +479,8 @@ 
             $criteria->add(new Criteria('user_occ', '%' . $myts->addSlashes(trim($_POST['user_occ'])) . '%', 'LIKE'));
         }
         foreach (array(
-                     'last_login',
-                     'user_regdate') as $var) {
+                        'last_login',
+                        'user_regdate') as $var) {
             if (!empty($_POST["{$var}_more"]) && is_numeric($_POST["{$var}_more"])) {
                 $time = time() - (60 * 60 * 24 * (int)trim($_POST["{$var}_more"]));
                 if ($time > 0) {

Spacing +75 added lines, -75 removed lines

@@ -17,7 +17,7 @@ 
  */
 /* @var  $xoopsUser XoopsUser */
 
-include_once dirname(__DIR__) . '/mainfile.php';
+include_once dirname(__DIR__).'/mainfile.php';
 
 xoops_header(false);
 
@@ -36,8 +36,8 @@ 
 
 $token         = isset($_REQUEST['token']) ? $_REQUEST['token'] : '';
 $name_form     = 'memberslist';
-$name_userid   = 'uid' . (!empty($_REQUEST['multiple']) ? '[]' : '');
-$name_username = 'uname' . (!empty($_REQUEST['multiple']) ? '[]' : '');
+$name_userid   = 'uid'.(!empty($_REQUEST['multiple']) ? '[]' : '');
+$name_username = 'uname'.(!empty($_REQUEST['multiple']) ? '[]' : '');
 
 xoops_loadLanguage('findusers');
 
@@ -104,7 +104,7 @@ 
     public function get($id = 0)
     {
         $object = $this->create(false);
-        $sql    = 'SELECT * FROM ' . $this->db->prefix('ranks') . ' WHERE rank_id = ' . $this->db->quoteString($id);
+        $sql    = 'SELECT * FROM '.$this->db->prefix('ranks').' WHERE rank_id = '.$this->db->quoteString($id);
         if (!$result = $this->db->query($sql)) {
             $ret = null;
 
@@ -132,11 +132,11 @@ 
             $criteria = new CriteriaCompo();
         }
 
-        $sql = 'SELECT rank_id, rank_title FROM ' . $this->db->prefix('ranks');
+        $sql = 'SELECT rank_id, rank_title FROM '.$this->db->prefix('ranks');
         if (isset($criteria) && is_subclass_of($criteria, 'CriteriaElement')) {
-            $sql .= ' ' . $criteria->renderWhere();
+            $sql .= ' '.$criteria->renderWhere();
             if ($criteria->getSort() != '') {
-                $sql .= ' ORDER BY ' . $criteria->getSort() . ' ' . $criteria->getOrder();
+                $sql .= ' ORDER BY '.$criteria->getSort().' '.$criteria->getOrder();
             }
             $limit = $criteria->getLimit();
             $start = $criteria->getStart();
@@ -227,14 +227,14 @@ 
         }
         $groups = array_filter($groups);
         if (empty($groups)) {
-            $sql = '    SELECT COUNT(DISTINCT u.uid) FROM ' . $this->db->prefix('users') . ' AS u' . '    WHERE 1=1';
+            $sql = '    SELECT COUNT(DISTINCT u.uid) FROM '.$this->db->prefix('users').' AS u'.'    WHERE 1=1';
         } else {
-            $sql = '    SELECT COUNT(DISTINCT u.uid) FROM ' . $this->db->prefix('users') . ' AS u' . '    LEFT JOIN ' . $this->db->prefix('groups_users_link') . ' AS g ON g.uid = u.uid' . '    WHERE g.groupid IN (' . implode(', ', array_map('intval', $groups)) . ')';
+            $sql = '    SELECT COUNT(DISTINCT u.uid) FROM '.$this->db->prefix('users').' AS u'.'    LEFT JOIN '.$this->db->prefix('groups_users_link').' AS g ON g.uid = u.uid'.'    WHERE g.groupid IN ('.implode(', ', array_map('intval', $groups)).')';
         }
         if (isset($criteria) && is_subclass_of($criteria, 'CriteriaElement')) {
             // Use the direct renderer, assuming no `uid` in criteria
             if ($render = $criteria->render()) {
-                $sql .= ' AND ' . $render;
+                $sql .= ' AND '.$render;
             }
         }
         $result = $this->db->query($sql);
@@ -260,16 +260,16 @@ 
         $limit  = null;
         $start  = null;
         if (empty($groups)) {
-            $sql = '    SELECT u.* FROM ' . $this->db->prefix('users') . ' AS u' . '    WHERE 1=1';
+            $sql = '    SELECT u.* FROM '.$this->db->prefix('users').' AS u'.'    WHERE 1=1';
         } else {
-            $sql = '    SELECT u.* FROM ' . $this->db->prefix('users') . ' AS u' . '    LEFT JOIN ' . $this->db->prefix('groups_users_link') . ' AS g ON g.uid = u.uid' . '    WHERE g.groupid IN (' . implode(', ', array_map('intval', $groups)) . ')';
+            $sql = '    SELECT u.* FROM '.$this->db->prefix('users').' AS u'.'    LEFT JOIN '.$this->db->prefix('groups_users_link').' AS g ON g.uid = u.uid'.'    WHERE g.groupid IN ('.implode(', ', array_map('intval', $groups)).')';
         }
         if (isset($criteria) && is_subclass_of($criteria, 'CriteriaElement')) {
             if ($render = $criteria->render()) {
-                $sql .= ' AND ' . $render;
+                $sql .= ' AND '.$render;
             }
             if ($sort = $criteria->getSort()) {
-                $sql .= ' ORDER BY ' . $sort . ' ' . $criteria->getOrder();
+                $sql .= ' ORDER BY '.$sort.' '.$criteria->getOrder();
                 $orderSet = true;
             }
             $limit = $criteria->getLimit();
@@ -428,16 +428,16 @@ 
     $acttotal   = $user_handler->getCount(new Criteria('level', 0, '>'));
     $inacttotal = $user_handler->getCount(new Criteria('level', 0, '<='));
     echo '</html><body>';
-    echo "<h2 style='text-align:left;'>" . _MA_USER_FINDUS . ' - ' . $modes[$mode] . '</h2>';
+    echo "<h2 style='text-align:left;'>"._MA_USER_FINDUS.' - '.$modes[$mode].'</h2>';
     $modes_switch = array();
     foreach ($modes as $_mode => $title) {
         if ($mode == $_mode) {
             continue;
         }
-        $modes_switch[] = "<a href='findusers.php?target=" . htmlspecialchars(@$_REQUEST['target'], ENT_QUOTES) . '&amp;multiple=' . htmlspecialchars(@$_REQUEST['multiple'], ENT_QUOTES) . '&amp;token=' . htmlspecialchars($token, ENT_QUOTES) . "&amp;mode={$_mode}'>{$title}</a>";
+        $modes_switch[] = "<a href='findusers.php?target=".htmlspecialchars(@$_REQUEST['target'], ENT_QUOTES).'&amp;multiple='.htmlspecialchars(@$_REQUEST['multiple'], ENT_QUOTES).'&amp;token='.htmlspecialchars($token, ENT_QUOTES)."&amp;mode={$_mode}'>{$title}</a>";
     }
-    echo '<h4>' . implode(' | ', $modes_switch) . '</h4>';
-    echo '(' . sprintf(_MA_USER_ACTUS, "<span style='color:#ff0000;'>$acttotal</span>") . ' ' . sprintf(_MA_USER_INACTUS, "<span style='color:#ff0000;'>$inacttotal</span>") . ')';
+    echo '<h4>'.implode(' | ', $modes_switch).'</h4>';
+    echo '('.sprintf(_MA_USER_ACTUS, "<span style='color:#ff0000;'>$acttotal</span>").' '.sprintf(_MA_USER_INACTUS, "<span style='color:#ff0000;'>$inacttotal</span>").')';
     $form->display();
 } else {
     $myts  = MyTextSanitizer::getInstance();
@@ -451,32 +451,32 @@ 
                 $value = str_replace('_', "\\\_", $myts->addSlashes(trim($_POST[$var])));
                 switch ($match) {
                     case XOOPS_MATCH_START:
-                        $criteria->add(new Criteria($var, $value . '%', 'LIKE'));
+                        $criteria->add(new Criteria($var, $value.'%', 'LIKE'));
                         break;
                     case XOOPS_MATCH_END:
-                        $criteria->add(new Criteria($var, '%' . $value, 'LIKE'));
+                        $criteria->add(new Criteria($var, '%'.$value, 'LIKE'));
                         break;
                     case XOOPS_MATCH_EQUAL:
                         $criteria->add(new Criteria($var, $value));
                         break;
                     case XOOPS_MATCH_CONTAIN:
-                        $criteria->add(new Criteria($var, '%' . $value . '%', 'LIKE'));
+                        $criteria->add(new Criteria($var, '%'.$value.'%', 'LIKE'));
                         break;
                 }
             }
         }
         if (!empty($_POST['url'])) {
             $url = formatURL(trim($_POST['url']));
-            $criteria->add(new Criteria('url', $url . '%', 'LIKE'));
+            $criteria->add(new Criteria('url', $url.'%', 'LIKE'));
         }
         if (!empty($_POST['user_from'])) {
-            $criteria->add(new Criteria('user_from', '%' . $myts->addSlashes(trim($_POST['user_from'])) . '%', 'LIKE'));
+            $criteria->add(new Criteria('user_from', '%'.$myts->addSlashes(trim($_POST['user_from'])).'%', 'LIKE'));
         }
         if (!empty($_POST['user_intrest'])) {
-            $criteria->add(new Criteria('user_intrest', '%' . $myts->addSlashes(trim($_POST['user_intrest'])) . '%', 'LIKE'));
+            $criteria->add(new Criteria('user_intrest', '%'.$myts->addSlashes(trim($_POST['user_intrest'])).'%', 'LIKE'));
         }
         if (!empty($_POST['user_occ'])) {
-            $criteria->add(new Criteria('user_occ', '%' . $myts->addSlashes(trim($_POST['user_occ'])) . '%', 'LIKE'));
+            $criteria->add(new Criteria('user_occ', '%'.$myts->addSlashes(trim($_POST['user_occ'])).'%', 'LIKE'));
         }
         foreach (array(
                      'last_login',
@@ -519,7 +519,7 @@ 
                 1 => 1,
                 2 => 0,
                 3 => -1);
-            $level       = isset($level_value[(int)$_POST['level']]) ? $level_value[(int)$_POST['level']] : 1;
+            $level = isset($level_value[(int)$_POST['level']]) ? $level_value[(int)$_POST['level']] : 1;
             $criteria->add(new Criteria('level', $level));
         }
         if (!empty($_POST['rank'])) {
@@ -555,21 +555,21 @@ 
     } else {
         $query = trim($_POST['query']);
         // Query with alias
-        if (preg_match("/select[\s]+.*[\s]+from[\s]+(" . $xoopsDB->prefix('users') . "[\s]+as[\s]+(\S+).*)/i", $query, $matches)) {
+        if (preg_match("/select[\s]+.*[\s]+from[\s]+(".$xoopsDB->prefix('users')."[\s]+as[\s]+(\S+).*)/i", $query, $matches)) {
             $alias    = $matches[2];
             $subquery = $matches[1];
 
             // Query without alias
-        } elseif (preg_match("/select[\s]+.*[\s]+from[\s]+(" . $xoopsDB->prefix('users') . "\b.*)/i", $query, $matches)) {
+        } elseif (preg_match("/select[\s]+.*[\s]+from[\s]+(".$xoopsDB->prefix('users')."\b.*)/i", $query, $matches)) {
             $alias    = '';
             $subquery = $matches[1];
 
             // Invalid query
         } else {
-            $query    = 'SELECT * FROM ' . $xoopsDB->prefix('users');
+            $query    = 'SELECT * FROM '.$xoopsDB->prefix('users');
             $subquery = $xoopsDB->prefix('users');
         }
-        $sql_count = 'SELECT COUNT(DISTINCT ' . (empty($alias) ? '' : $alias . '.') . 'uid) FROM ' . $subquery;
+        $sql_count = 'SELECT COUNT(DISTINCT '.(empty($alias) ? '' : $alias.'.').'uid) FROM '.$subquery;
         $result    = $xoopsDB->query($sql_count);
         list($total) = $xoopsDB->FetchRow($result);
         $result     = $xoopsDB->query($query, $limit, $start);
@@ -584,15 +584,15 @@ 
 
     echo $js_adduser = '
         <script type="text/javascript">
-        var multiple=' . (int)$_REQUEST['multiple'] . ';
+        var multiple=' . (int)$_REQUEST['multiple'].';
         function addusers()
         {
             var sel_str = "";
             var num = 0;
-            var mForm = document.forms["' . $name_form . '"];
+            var mForm = document.forms["' . $name_form.'"];
             for (var i=0;i!=mForm.elements.length;i++) {
                 var id=mForm.elements[i];
-                if ( ( (multiple > 0 && id.type == "checkbox") || (multiple == 0 && id.type == "radio") ) && (id.checked == true) && ( id.name == "' . $name_userid . '" ) ) {
+                if ( ( (multiple > 0 && id.type == "checkbox") || (multiple == 0 && id.type == "radio") ) && (id.checked == true) && ( id.name == "' . $name_userid.'" ) ) {
                     var name = mForm.elements[++i];
                     var len = id.value.length + name.value.length;
                     sel_str += len + ":" + id.value + ":" + name.value;
@@ -600,12 +600,12 @@ 
                 }
             }
             if (num == 0) {
-                alert("' . _MA_USER_NOUSERSELECTED . '");
+                alert("' . _MA_USER_NOUSERSELECTED.'");
                 return false;
             }
             sel_str = num + ":" + sel_str;
             window.opener.addusers(sel_str);
-            alert("' . _MA_USER_USERADDED . '");
+            alert("' . _MA_USER_USERADDED.'");
             if (multiple == 0) {
                 window.close();
                 window.opener.focus();
@@ -616,16 +616,16 @@ 
     ';
 
     echo '</html><body>';
-    echo "<a href='findusers.php?target=" . htmlspecialchars(@$_POST['target'], ENT_QUOTES) . '&amp;multiple=' . (int)(@$_POST['multiple']) . '&amp;token=' . htmlspecialchars($token, ENT_QUOTES) . "'>" . _MA_USER_FINDUS . "</a>&nbsp;<span style='font-weight:bold;'>&raquo;</span>&nbsp;" . _MA_USER_RESULTS . '<br><br>';
+    echo "<a href='findusers.php?target=".htmlspecialchars(@$_POST['target'], ENT_QUOTES).'&amp;multiple='.(int)(@$_POST['multiple']).'&amp;token='.htmlspecialchars($token, ENT_QUOTES)."'>"._MA_USER_FINDUS."</a>&nbsp;<span style='font-weight:bold;'>&raquo;</span>&nbsp;"._MA_USER_RESULTS.'<br><br>';
     if (empty($start) && empty($foundusers)) {
-        echo '<h4>' . _MA_USER_NOFOUND, '</h4>';
+        echo '<h4>'._MA_USER_NOFOUND, '</h4>';
         $hiddenform = "<form name='findnext' action='findusers.php' method='post'>";
         foreach ($_POST as $k => $v) {
             if ($k === 'XOOPS_TOKEN_REQUEST') {
                 // regenerate token value
-                $hiddenform .= $GLOBALS['xoopsSecurity']->getTokenHTML() . "\n";
+                $hiddenform .= $GLOBALS['xoopsSecurity']->getTokenHTML()."\n";
             } else {
-                $hiddenform .= "<input type='hidden' name='" . htmlspecialchars($k, ENT_QUOTES) . "' value='" . htmlspecialchars($myts->stripSlashesGPC($v), ENT_QUOTES) . "' />\n";
+                $hiddenform .= "<input type='hidden' name='".htmlspecialchars($k, ENT_QUOTES)."' value='".htmlspecialchars($myts->stripSlashesGPC($v), ENT_QUOTES)."' />\n";
             }
         }
         if (!isset($_POST['limit'])) {
@@ -634,15 +634,15 @@ 
         if (!isset($_POST['start'])) {
             $hiddenform .= "<input type='hidden' name='start' value='{$start}' />\n";
         }
-        $hiddenform .= "<input type='hidden' name='token' value='" . htmlspecialchars($token, ENT_QUOTES) . "' />\n";
+        $hiddenform .= "<input type='hidden' name='token' value='".htmlspecialchars($token, ENT_QUOTES)."' />\n";
         $hiddenform .= '</form>';
 
-        echo '<div>' . $hiddenform;
-        echo "<a href='#' onclick='document.findnext.start.value=0;document.findnext.user_submit.value=0;document.findnext.submit();'>" . _MA_USER_SEARCHAGAIN . "</a>\n";
+        echo '<div>'.$hiddenform;
+        echo "<a href='#' onclick='document.findnext.start.value=0;document.findnext.user_submit.value=0;document.findnext.submit();'>"._MA_USER_SEARCHAGAIN."</a>\n";
         echo '</div>';
     } elseif ($start < $total) {
         if (!empty($total)) {
-            echo sprintf(_MA_USER_USERSFOUND, $total) . '<br>';
+            echo sprintf(_MA_USER_USERSFOUND, $total).'<br>';
         }
         if (!empty($foundusers)) {
             echo "<form action='findusers.php' method='post' name='{$name_form}' id='{$name_form}'>
@@ -653,11 +653,11 @@ 
                 echo "<input type='checkbox' name='memberslist_checkall' id='memberslist_checkall' onclick='xoopsCheckAll(\"{$name_form}\", \"memberslist_checkall\");' />";
             }
             echo "</th>
-            <th align='center'>" . _MA_USER_UNAME . "</th>
-            <th align='center'>" . _MA_USER_REALNAME . "</th>
-            <th align='center'>" . _MA_USER_REGDATE . "</th>
-            <th align='center'>" . _MA_USER_LASTLOGIN . "</th>
-            <th align='center'>" . _MA_USER_POSTS . '</th>
+            <th align='center'>" . _MA_USER_UNAME."</th>
+            <th align='center'>" . _MA_USER_REALNAME."</th>
+            <th align='center'>" . _MA_USER_REGDATE."</th>
+            <th align='center'>" . _MA_USER_LASTLOGIN."</th>
+            <th align='center'>" . _MA_USER_POSTS.'</th>
             </tr>';
             $ucount = 0;
             foreach (array_keys($foundusers) as $j) {
@@ -670,33 +670,33 @@ 
                 echo "<tr class='$class'>
                     <td align='center'>";
                 if (!empty($_POST['multiple'])) {
-                    echo "<input type='checkbox' name='{$name_userid}' id='{$name_userid}' value='" . $foundusers[$j]->getVar('uid') . "' />";
-                    echo "<input type='hidden' name='{$name_username}' id='{$name_username}' value='" . $foundusers[$j]->getVar('uname') . "' />";
+                    echo "<input type='checkbox' name='{$name_userid}' id='{$name_userid}' value='".$foundusers[$j]->getVar('uid')."' />";
+                    echo "<input type='hidden' name='{$name_username}' id='{$name_username}' value='".$foundusers[$j]->getVar('uname')."' />";
                 } else {
-                    echo "<input type='radio' name='{$name_userid}' id='{$name_userid}' value='" . $foundusers[$j]->getVar('uid') . "' />";
-                    echo "<input type='hidden' name='{$name_username}' id='{$name_username}' value='" . $foundusers[$j]->getVar('uname') . "' />";
+                    echo "<input type='radio' name='{$name_userid}' id='{$name_userid}' value='".$foundusers[$j]->getVar('uid')."' />";
+                    echo "<input type='hidden' name='{$name_username}' id='{$name_username}' value='".$foundusers[$j]->getVar('uname')."' />";
                 }
                 echo "</td>
-                    <td><a href='" . XOOPS_URL . '/userinfo.php?uid=' . $foundusers[$j]->getVar('uid') . "' target='_blank'>" . $foundusers[$j]->getVar('uname') . '</a></td>
-                    <td>' . $fuser_name . "</td>
-                    <td align='center'>" . ($foundusers[$j]->getVar('user_regdate') ? date('Y-m-d', $foundusers[$j]->getVar('user_regdate')) : '') . "</td>
-                    <td align='center'>" . ($foundusers[$j]->getVar('last_login') ? date('Y-m-d H:i', $foundusers[$j]->getVar('last_login')) : '') . "</td>
-                    <td align='center'>" . $foundusers[$j]->getVar('posts') . '</td>';
+                    <td><a href='" . XOOPS_URL.'/userinfo.php?uid='.$foundusers[$j]->getVar('uid')."' target='_blank'>".$foundusers[$j]->getVar('uname').'</a></td>
+                    <td>' . $fuser_name."</td>
+                    <td align='center'>" . ($foundusers[$j]->getVar('user_regdate') ? date('Y-m-d', $foundusers[$j]->getVar('user_regdate')) : '')."</td>
+                    <td align='center'>" . ($foundusers[$j]->getVar('last_login') ? date('Y-m-d H:i', $foundusers[$j]->getVar('last_login')) : '')."</td>
+                    <td align='center'>" . $foundusers[$j]->getVar('posts').'</td>';
                 echo "</tr>\n";
             }
             echo "<tr class='foot'><td colspan='6'>";
 
             // placeholder for external applications
             if (empty($_POST['target'])) {
-                echo "<select name='fct'><option value='users'>" . _DELETE . "</option><option value='mailusers'>" . _MA_USER_SENDMAIL . '</option>';
+                echo "<select name='fct'><option value='users'>"._DELETE."</option><option value='mailusers'>"._MA_USER_SENDMAIL.'</option>';
                 echo '</select>&nbsp;';
-                echo $GLOBALS['xoopsSecurity']->getTokenHTML() . "<input type='submit' value='" . _SUBMIT . "' />";
+                echo $GLOBALS['xoopsSecurity']->getTokenHTML()."<input type='submit' value='"._SUBMIT."' />";
 
                 // Add selected users
             } else {
-                echo "<input type='button' value='" . _MA_USER_ADD_SELECTED . "' onclick='addusers();' />";
+                echo "<input type='button' value='"._MA_USER_ADD_SELECTED."' onclick='addusers();' />";
             }
-            echo "<input type='hidden' name='token' value='" . htmlspecialchars($token, ENT_QUOTES) . "' />\n";
+            echo "<input type='hidden' name='token' value='".htmlspecialchars($token, ENT_QUOTES)."' />\n";
             echo "</td></tr></table></form>\n";
         }
 
@@ -704,31 +704,31 @@ 
         foreach ($_POST as $k => $v) {
             if ($k === 'XOOPS_TOKEN_REQUEST') {
                 // regenerate token value
-                $hiddenform .= $GLOBALS['xoopsSecurity']->getTokenHTML() . "\n";
+                $hiddenform .= $GLOBALS['xoopsSecurity']->getTokenHTML()."\n";
             } else {
-                $hiddenform .= "<input type='hidden' name='" . htmlspecialchars($k, ENT_QUOTES) . "' value='" . htmlspecialchars($myts->stripSlashesGPC($v), ENT_QUOTES) . "' />\n";
+                $hiddenform .= "<input type='hidden' name='".htmlspecialchars($k, ENT_QUOTES)."' value='".htmlspecialchars($myts->stripSlashesGPC($v), ENT_QUOTES)."' />\n";
             }
         }
         if (!isset($_POST['limit'])) {
-            $hiddenform .= "<input type='hidden' name='limit' value='" . $limit . "' />\n";
+            $hiddenform .= "<input type='hidden' name='limit' value='".$limit."' />\n";
         }
         if (!isset($_POST['start'])) {
-            $hiddenform .= "<input type='hidden' name='start' value='" . $start . "' />\n";
+            $hiddenform .= "<input type='hidden' name='start' value='".$start."' />\n";
         }
-        $hiddenform .= "<input type='hidden' name='token' value='" . htmlspecialchars($token, ENT_QUOTES) . "' />\n";
+        $hiddenform .= "<input type='hidden' name='token' value='".htmlspecialchars($token, ENT_QUOTES)."' />\n";
         if (!isset($total) || ($totalpages = ceil($total / $limit)) > 1) {
             $prev = $start - $limit;
             if ($start - $limit >= 0) {
-                $hiddenform .= "<a href='#0' onclick='document.findnext.start.value=" . $prev . ";document.findnext.submit();'>" . _MA_USER_PREVIOUS . "</a>&nbsp;\n";
+                $hiddenform .= "<a href='#0' onclick='document.findnext.start.value=".$prev.";document.findnext.submit();'>"._MA_USER_PREVIOUS."</a>&nbsp;\n";
             }
             $counter     = 1;
             $currentpage = ($start + $limit) / $limit;
             if (!isset($total)) {
                 while ($counter <= $currentpage) {
                     if ($counter == $currentpage) {
-                        $hiddenform .= '<strong>' . $counter . '</strong> ';
+                        $hiddenform .= '<strong>'.$counter.'</strong> ';
                     } elseif (($counter > $currentpage - 4 && $counter < $currentpage + 4) || $counter == 1) {
-                        $hiddenform .= "<a href='#" . $counter . "' onclick='document.findnext.start.value=" . ($counter - 1) * $limit . ";document.findnext.submit();'>" . $counter . '</a> ';
+                        $hiddenform .= "<a href='#".$counter."' onclick='document.findnext.start.value=".($counter - 1) * $limit.";document.findnext.submit();'>".$counter.'</a> ';
                         if ($counter == 1 && $currentpage > 5) {
                             $hiddenform .= '... ';
                         }
@@ -738,12 +738,12 @@ 
             } else {
                 while ($counter <= $totalpages) {
                     if ($counter == $currentpage) {
-                        $hiddenform .= '<strong>' . $counter . '</strong> ';
+                        $hiddenform .= '<strong>'.$counter.'</strong> ';
                     } elseif (($counter > $currentpage - 4 && $counter < $currentpage + 4) || $counter == 1 || $counter == $totalpages) {
                         if ($counter == $totalpages && $currentpage < $totalpages - 4) {
                             $hiddenform .= '... ';
                         }
-                        $hiddenform .= "<a href='#" . $counter . "' onclick='document.findnext.start.value=" . ($counter - 1) * $limit . ";document.findnext.submit();'>" . $counter . '</a> ';
+                        $hiddenform .= "<a href='#".$counter."' onclick='document.findnext.start.value=".($counter - 1) * $limit.";document.findnext.submit();'>".$counter.'</a> ';
                         if ($counter == 1 && $currentpage > 5) {
                             $hiddenform .= '... ';
                         }
@@ -754,16 +754,16 @@ 
 
             $next = $start + $limit;
             if ((isset($total) && $total > $next) || (!isset($total) && count($foundusers) >= $limit)) {
-                $hiddenform .= "&nbsp;<a href='#" . $total . "' onclick='document.findnext.start.value=" . $next . ";document.findnext.submit();'>" . _MA_USER_NEXT . "</a>\n";
+                $hiddenform .= "&nbsp;<a href='#".$total."' onclick='document.findnext.start.value=".$next.";document.findnext.submit();'>"._MA_USER_NEXT."</a>\n";
             }
         }
         $hiddenform .= '</form>';
 
-        echo '<div>' . $hiddenform;
+        echo '<div>'.$hiddenform;
         if (isset($total)) {
-            echo '<br>' . sprintf(_MA_USER_USERSFOUND, $total) . '&nbsp;';
+            echo '<br>'.sprintf(_MA_USER_USERSFOUND, $total).'&nbsp;';
         }
-        echo "<a href='#' onclick='document.findnext.start.value=0;document.findnext.user_submit.value=0;document.findnext.submit();'>" . _MA_USER_SEARCHAGAIN . "</a>\n";
+        echo "<a href='#' onclick='document.findnext.start.value=0;document.findnext.user_submit.value=0;document.findnext.submit();'>"._MA_USER_SEARCHAGAIN."</a>\n";
         echo '</div>';
     }
 }

htdocs/include/common.php

Indentation +1 added lines, -1 removed lines

@@ -246,7 +246,7 @@
     } else {
         if (((int)$xoopsUser->getVar('last_login') + 60 * 5) < time()) {
             $sql = 'UPDATE ' . $xoopsDB->prefix('users') . " SET last_login = '" . time()
-                   . "' WHERE uid = " . $_SESSION['xoopsUserId'];
+                    . "' WHERE uid = " . $_SESSION['xoopsUserId'];
             @$xoopsDB->queryF($sql);
         }
         $sess_handler->update_cookie();

Spacing +14 added lines, -14 removed lines

@@ -26,14 +26,14 @@ 
 /**
  * Include files with definitions
  */
-include_once XOOPS_ROOT_PATH . '/include/defines.php';
-include_once XOOPS_ROOT_PATH . '/include/version.php';
-include_once XOOPS_ROOT_PATH . '/include/license.php';
+include_once XOOPS_ROOT_PATH.'/include/defines.php';
+include_once XOOPS_ROOT_PATH.'/include/version.php';
+include_once XOOPS_ROOT_PATH.'/include/license.php';
 
 /**
  * Include XoopsLoad
  */
-require_once XOOPS_ROOT_PATH . '/class/xoopsload.php';
+require_once XOOPS_ROOT_PATH.'/class/xoopsload.php';
 
 /**
  * YOU SHOULD BE CAREFUL WITH THE PRELOAD METHODS IN 2.4*, THEY WILL BE DEPRECATED AND IMPLEMENTED IN A DIFFERENT WAY
@@ -55,7 +55,7 @@ 
 XoopsLoad::load('xoopskernel');
 $xoops = new xos_kernel_Xoops2();
 $xoops->pathTranslation();
-$xoopsRequestUri =& $_SERVER['REQUEST_URI'];// Deprecated (use the corrected $_SERVER variable now)
+$xoopsRequestUri = & $_SERVER['REQUEST_URI']; // Deprecated (use the corrected $_SERVER variable now)
 
 /**
  * Create Instance of xoopsSecurity Object and check Supergolbals
@@ -118,7 +118,7 @@ 
     $xoopsConfig = array_merge($xoopsConfig, (array)$fileConfigs);
     unset($fileConfigs, $file);
 } else {
-    trigger_error('File Path Error: ' . 'var/configs/xoopsconfig.php' . ' does not exist.');
+    trigger_error('File Path Error: '.'var/configs/xoopsconfig.php'.' does not exist.');
 }
 
 /**
@@ -126,7 +126,7 @@ 
  */
 $xFrameOptions = isset($xoopsConfig['xFrameOptions']) ? $xoopsConfig['xFrameOptions'] : 'sameorigin';
 if (!headers_sent() && !empty($xFrameOptions)) {
-    header('X-Frame-Options: ' . $xFrameOptions);
+    header('X-Frame-Options: '.$xFrameOptions);
 }
 
 //check if user set a local timezone (from XavierS)
@@ -245,8 +245,8 @@ 
         setcookie($GLOBALS['xoopsConfig']['usercookie'], null, time() - 3600);
     } else {
         if (((int)$xoopsUser->getVar('last_login') + 60 * 5) < time()) {
-            $sql = 'UPDATE ' . $xoopsDB->prefix('users') . " SET last_login = '" . time()
-                   . "' WHERE uid = " . $_SESSION['xoopsUserId'];
+            $sql = 'UPDATE '.$xoopsDB->prefix('users')." SET last_login = '".time()
+                   . "' WHERE uid = ".$_SESSION['xoopsUserId'];
             @$xoopsDB->queryF($sql);
         }
         $sess_handler->update_cookie();
@@ -264,7 +264,7 @@ 
             $claims = array(
                 'uid' => $_SESSION['xoopsUserId'],
             );
-            $rememberTime = 60*60*24*30;
+            $rememberTime = 60 * 60 * 24 * 30;
             $token = \Xmf\Jwt\TokenFactory::build('rememberme', $claims, $rememberTime);
             setcookie(
                 $GLOBALS['xoopsConfig']['usercookie'],
@@ -325,7 +325,7 @@ 
 
     if (!$xoopsModule || !$xoopsModule->getVar('isactive')) {
         include_once $xoops->path('header.php');
-        echo '<h4>' . _MODULENOEXIST . '</h4>';
+        echo '<h4>'._MODULENOEXIST.'</h4>';
         include_once $xoops->path('footer.php');
         exit();
     }
@@ -338,14 +338,14 @@ 
         $xoopsUserIsAdmin = $xoopsUser->isAdmin($xoopsModule->getVar('mid'));
     } else {
         if (!$moduleperm_handler->checkRight('module_read', $xoopsModule->getVar('mid'), XOOPS_GROUP_ANONYMOUS)) {
-            redirect_header(XOOPS_URL . '/user.php?from=' . $xoopsModule->getVar('dirname', 'n'), 1, _NOPERM);
+            redirect_header(XOOPS_URL.'/user.php?from='.$xoopsModule->getVar('dirname', 'n'), 1, _NOPERM);
         }
     }
 
     if ($xoopsModule->getVar('dirname', 'n') !== 'system') {
-        if (file_exists($file = $xoops->path('modules/' . $xoopsModule->getVar('dirname', 'n') . '/language/' . $xoopsConfig['language'] . '/main.php'))) {
+        if (file_exists($file = $xoops->path('modules/'.$xoopsModule->getVar('dirname', 'n').'/language/'.$xoopsConfig['language'].'/main.php'))) {
             include_once $file;
-        } elseif (file_exists($file = $xoops->path('modules/' . $xoopsModule->getVar('dirname', 'n') . '/language/english/main.php'))) {
+        } elseif (file_exists($file = $xoops->path('modules/'.$xoopsModule->getVar('dirname', 'n').'/language/english/main.php'))) {
             include_once $file;
         }
         unset($file);

htdocs/include/searchform.php

Indentation +3 added lines, -3 removed lines

@@ -25,9 +25,9 @@
 $search_form->addElement(new XoopsFormText(_SR_KEYWORDS, 'query', 30, 255, htmlspecialchars(stripslashes(implode(' ', $queries)), ENT_QUOTES)), true);
 $type_select = new XoopsFormSelect(_SR_TYPE, 'andor', $andor);
 $type_select->addOptionArray(array(
-                                 'AND'   => _SR_ALL,
-                                 'OR'    => _SR_ANY,
-                                 'exact' => _SR_EXACT));
+                                    'AND'   => _SR_ALL,
+                                    'OR'    => _SR_ANY,
+                                    'exact' => _SR_EXACT));
 $search_form->addElement($type_select);
 if (!empty($mids)) {
     $mods_checkbox = new XoopsFormCheckBox(_SR_SEARCHIN, 'mids[]', $mids);

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
     $criteria->add(new Criteria('hassearch', 1));
     $criteria->add(new Criteria('isactive', 1));
     if (!empty($available_modules)) {
-        $criteria->add(new Criteria('mid', '(' . implode(',', $available_modules) . ')', 'IN'));
+        $criteria->add(new Criteria('mid', '('.implode(',', $available_modules).')', 'IN'));
     }
     /* @var $module_handler XoopsModuleHandler */
     $module_handler = xoops_getHandler('module');

htdocs/include/license.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
  */
 defined('XOOPS_ROOT_PATH') || exit('Restricted access');
 
-$licenseFile = XOOPS_VAR_PATH . '/data/license.php';
+$licenseFile = XOOPS_VAR_PATH.'/data/license.php';
 if (file_exists($licenseFile)) {
     include_once $licenseFile;
 } else {

htdocs/include/comment_delete.php

Spacing +19 added lines, -19 removed lines

@@ -49,7 +49,7 @@ 
     $module          = $module_handler->get($comment->getVar('com_modid'));
     $comment_config  = $module->getInfo('comments');
     $com_modid       = $module->getVar('mid');
-    $redirect_page   = XOOPS_URL . '/modules/system/admin.php?fct=comments&com_modid=' . $com_modid . '&com_itemid';
+    $redirect_page   = XOOPS_URL.'/modules/system/admin.php?fct=comments&com_modid='.$com_modid.'&com_itemid';
     $moddir          = $module->getVar('dirname');
     unset($comment);
 } else {
@@ -58,18 +58,18 @@ 
     }
     $comment_config        = $xoopsModule->getInfo('comments');
     $com_modid             = $xoopsModule->getVar('mid');
-    $redirect_page         = $comment_config['pageName'] . '?';
+    $redirect_page         = $comment_config['pageName'].'?';
     $comment_confirm_extra = array();
     if (isset($comment_config['extraParams']) && is_array($comment_config['extraParams'])) {
         foreach ($comment_config['extraParams'] as $extra_param) {
             if (isset(${$extra_param})) {
-                $redirect_page .= $extra_param . '=' . ${$extra_param} . '&';
+                $redirect_page .= $extra_param.'='.${$extra_param}.'&';
 
                 // for the confirmation page
                 $comment_confirm_extra[$extra_param] = ${$extra_param};
             } elseif (isset($_GET[$extra_param])) {
                 //                $redirect_page .= $extra_param . '=' . $_GET[$extra_param] . '&';
-                $redirect_page .= $extra_param . '=' . filter_input(INPUT_GET, $extra_param, FILTER_SANITIZE_STRING) . '&';
+                $redirect_page .= $extra_param.'='.filter_input(INPUT_GET, $extra_param, FILTER_SANITIZE_STRING).'&';
 
                 // for the confirmation page
                 //                $comment_confirm_extra[$extra_param] = $_GET[$extra_param];
@@ -100,7 +100,7 @@ 
     if ($ref != '') {
         redirect_header($ref, 2, _NOPERM);
     } else {
-        redirect_header($redirect_page . '?' . $comment_config['itemName'] . '=' . (int)$com_itemid, 2, _NOPERM);
+        redirect_header($redirect_page.'?'.$comment_config['itemName'].'='.(int)$com_itemid, 2, _NOPERM);
     }
     exit();
 }
@@ -113,7 +113,7 @@ 
         $comment         = $comment_handler->get($com_id);
         if (!$comment_handler->delete($comment)) {
             include_once $GLOBALS['xoops']->path('header.php');
-            xoops_error(_CM_COMDELETENG . ' (ID: ' . $comment->getVar('com_id') . ')');
+            xoops_error(_CM_COMDELETENG.' (ID: '.$comment->getVar('com_id').')');
             include_once $GLOBALS['xoops']->path('footer');
             exit();
         }
@@ -125,7 +125,7 @@ 
             if (!function_exists($comment_config['callback']['update'])) {
                 if (isset($comment_config['callbackFile'])) {
                     $callbackfile = trim($comment_config['callbackFile']);
-                    if ($callbackfile != '' && file_exists($file = $GLOBALS['xoops']->path('modules/' . $moddir . '/' . $callbackfile))) {
+                    if ($callbackfile != '' && file_exists($file = $GLOBALS['xoops']->path('modules/'.$moddir.'/'.$callbackfile))) {
                         include_once $file;
                     }
                     if (!function_exists($comment_config['callback']['update'])) {
@@ -155,10 +155,10 @@ 
         }
 
         // get all comments posted later within the same thread
-        $thread_comments =& $comment_handler->getThread($comment->getVar('com_rootid'), $com_id);
+        $thread_comments = & $comment_handler->getThread($comment->getVar('com_rootid'), $com_id);
         include_once $GLOBALS['xoops']->path('class/tree.php');
         $xot            = new XoopsObjectTree($thread_comments, 'com_id', 'com_pid', 'com_rootid');
-        $child_comments =& $xot->getFirstChild($com_id);
+        $child_comments = & $xot->getFirstChild($com_id);
         // now set new parent ID for direct child comments
         $new_pid = $comment->getVar('com_pid');
         $errs    = array();
@@ -169,7 +169,7 @@ 
                 $new_rootid = $child_comments[$i]->getVar('com_id');
                 $child_comments[$i]->setVar('com_rootid', $child_comments[$i]->getVar('com_id'));
                 if (!$comment_handler->insert($child_comments[$i])) {
-                    $errs[] = 'Could not change comment parent ID from <strong>' . $com_id . '</strong> to <strong>' . $new_pid . '</strong>. (ID: ' . $new_rootid . ')';
+                    $errs[] = 'Could not change comment parent ID from <strong>'.$com_id.'</strong> to <strong>'.$new_pid.'</strong>. (ID: '.$new_rootid.')';
                 } else {
                     // need to change root id for all its child comments as well
                     $c_child_comments = &$xot->getAllChild($new_rootid);
@@ -177,13 +177,13 @@ 
                     foreach (array_keys($c_child_comments) as $j) {
                         $c_child_comments[$j]->setVar('com_rootid', $new_rootid);
                         if (!$comment_handler->insert($c_child_comments[$j])) {
-                            $errs[] = 'Could not change comment root ID from <strong>' . $com_id . '</strong> to <strong>' . $new_rootid . '</strong>.';
+                            $errs[] = 'Could not change comment root ID from <strong>'.$com_id.'</strong> to <strong>'.$new_rootid.'</strong>.';
                         }
                     }
                 }
             } else {
                 if (!$comment_handler->insert($child_comments[$i])) {
-                    $errs[] = 'Could not change comment parent ID from <strong>' . $com_id . '</strong> to <strong>' . $new_pid . '</strong>.';
+                    $errs[] = 'Could not change comment parent ID from <strong>'.$com_id.'</strong> to <strong>'.$new_pid.'</strong>.';
                 }
             }
         }
@@ -193,7 +193,7 @@ 
             include_once $GLOBALS['xoops']->path('footer.php');
             exit();
         }
-        redirect_header($redirect_page . '=' . $com_itemid . '&amp;com_order=' . $com_order . '&amp;com_mode=' . $com_mode, 1, _CM_COMDELETED);
+        redirect_header($redirect_page.'='.$com_itemid.'&amp;com_order='.$com_order.'&amp;com_mode='.$com_mode, 1, _CM_COMDELETED);
         break;
 
     case 'delete_all':
@@ -203,12 +203,12 @@ 
         $com_rootid      = $comment->getVar('com_rootid');
 
         // get all comments posted later within the same thread
-        $thread_comments =& $comment_handler->getThread($com_rootid, $com_id);
+        $thread_comments = & $comment_handler->getThread($com_rootid, $com_id);
 
         // construct a comment tree
         include_once $GLOBALS['xoops']->path('class/tree.php');
         $xot            = new XoopsObjectTree($thread_comments, 'com_id', 'com_pid', 'com_rootid');
-        $child_comments =& $xot->getAllChild($com_id);
+        $child_comments = & $xot->getAllChild($com_id);
         // add itself here
         $child_comments[$com_id] = &$comment;
         $msgs                    = array();
@@ -217,9 +217,9 @@ 
         $member_handler          = xoops_getHandler('member');
         foreach (array_keys($child_comments) as $i) {
             if (!$comment_handler->delete($child_comments[$i])) {
-                $msgs[] = _CM_COMDELETENG . ' (ID: ' . $child_comments[$i]->getVar('com_id') . ')';
+                $msgs[] = _CM_COMDELETENG.' (ID: '.$child_comments[$i]->getVar('com_id').')';
             } else {
-                $msgs[] = _CM_COMDELETED . ' (ID: ' . $child_comments[$i]->getVar('com_id') . ')';
+                $msgs[] = _CM_COMDELETED.' (ID: '.$child_comments[$i]->getVar('com_id').')';
                 // store poster ID and deleted post number into array for later use
                 $poster_id = $child_comments[$i]->getVar('com_uid');
                 if ($poster_id > 0) {
@@ -242,7 +242,7 @@ 
             if (!function_exists($comment_config['callback']['update'])) {
                 if (isset($comment_config['callbackFile'])) {
                     $callbackfile = trim($comment_config['callbackFile']);
-                    if ($callbackfile != '' && file_exists($file = $GLOBALS['xoops']->path('modules/' . $moddir . '/' . $callbackfile))) {
+                    if ($callbackfile != '' && file_exists($file = $GLOBALS['xoops']->path('modules/'.$moddir.'/'.$callbackfile))) {
                         include_once $file;
                     }
                     if (!function_exists($comment_config['callback']['update'])) {
@@ -262,7 +262,7 @@ 
         }
         include_once $GLOBALS['xoops']->path('header.php');
         xoops_result($msgs);
-        echo '<br><a href="' . $redirect_page . '=' . $com_itemid . '&amp;com_order=' . $com_order . '&amp;com_mode=' . $com_mode . '">' . _BACK . '</a>';
+        echo '<br><a href="'.$redirect_page.'='.$com_itemid.'&amp;com_order='.$com_order.'&amp;com_mode='.$com_mode.'">'._BACK.'</a>';
         include_once $GLOBALS['xoops']->path('footer.php');
         break;
 

htdocs/include/notification_select.php

Indentation +11 added lines, -11 removed lines

@@ -58,17 +58,17 @@
         $xoops_notification['target_page']     = 'notification_update.php';
         $xoops_notification['redirect_script'] = xoops_getenv('PHP_SELF');
         $xoopsTpl->assign(array(
-                              'lang_activenotifications'  => _NOT_ACTIVENOTIFICATIONS,
-                              'lang_notificationoptions'  => _NOT_NOTIFICATIONOPTIONS,
-                              'lang_updateoptions'        => _NOT_UPDATEOPTIONS,
-                              'lang_updatenow'            => _NOT_UPDATENOW,
-                              'lang_category'             => _NOT_CATEGORY,
-                              'lang_event'                => _NOT_EVENT,
-                              'lang_events'               => _NOT_EVENTS,
-                              'lang_checkall'             => _NOT_CHECKALL,
-                              'lang_notificationmethodis' => _NOT_NOTIFICATIONMETHODIS,
-                              'lang_change'               => _NOT_CHANGE,
-                              'editprofile_url'           => XOOPS_URL . '/edituser.php?uid=' . $xoopsUser->getVar('uid')));
+                                'lang_activenotifications'  => _NOT_ACTIVENOTIFICATIONS,
+                                'lang_notificationoptions'  => _NOT_NOTIFICATIONOPTIONS,
+                                'lang_updateoptions'        => _NOT_UPDATEOPTIONS,
+                                'lang_updatenow'            => _NOT_UPDATENOW,
+                                'lang_category'             => _NOT_CATEGORY,
+                                'lang_event'                => _NOT_EVENT,
+                                'lang_events'               => _NOT_EVENTS,
+                                'lang_checkall'             => _NOT_CHECKALL,
+                                'lang_notificationmethodis' => _NOT_NOTIFICATIONMETHODIS,
+                                'lang_change'               => _NOT_CHANGE,
+                                'editprofile_url'           => XOOPS_URL . '/edituser.php?uid=' . $xoopsUser->getVar('uid')));
         switch ($xoopsUser->getVar('notify_method')) {
             case XOOPS_NOTIFICATION_METHOD_DISABLE:
                 $xoopsTpl->assign('user_method', _NOT_DISABLE);

Spacing +2 added lines, -2 removed lines

@@ -25,7 +25,7 @@ 
 $xoops_notification['show'] = isset($xoopsModule) && is_object($xoopsUser) && notificationEnabled('inline') ? 1 : 0;
 if ($xoops_notification['show']) {
     xoops_loadLanguage('notification');
-    $categories  =& notificationSubscribableCategoryInfo();
+    $categories  = & notificationSubscribableCategoryInfo();
     $event_count = 0;
     if (!empty($categories)) {
         /* @var  $notification_handler XoopsNotificationHandler */
@@ -68,7 +68,7 @@ 
                               'lang_checkall'             => _NOT_CHECKALL,
                               'lang_notificationmethodis' => _NOT_NOTIFICATIONMETHODIS,
                               'lang_change'               => _NOT_CHANGE,
-                              'editprofile_url'           => XOOPS_URL . '/edituser.php?uid=' . $xoopsUser->getVar('uid')));
+                              'editprofile_url'           => XOOPS_URL.'/edituser.php?uid='.$xoopsUser->getVar('uid')));
         switch ($xoopsUser->getVar('notify_method')) {
             case XOOPS_NOTIFICATION_METHOD_DISABLE:
                 $xoopsTpl->assign('user_method', _NOT_DISABLE);

htdocs/include/notification_update.php

Spacing +4 added lines, -4 removed lines

@@ -81,7 +81,7 @@ 
 $redirect_args = array();
 foreach ($update_list as $update_item) {
     list($category, $item_id, $event) = preg_split('/,/', $update_item['params']);
-    $category_info =& notificationCategoryInfo($category);
+    $category_info = & notificationCategoryInfo($category);
     if (!empty($category_info['item_name'])) {
         $redirect_args[$category_info['item_name']] = $item_id;
     }
@@ -93,12 +93,12 @@ 
 $first_arg = 1;
 foreach (array_keys($redirect_args) as $arg) {
     if ($first_arg) {
-        $argstring .= '?' . $arg . '=' . $redirect_args[$arg];
+        $argstring .= '?'.$arg.'='.$redirect_args[$arg];
         $first_arg = 0;
     } else {
-        $argstring .= '&' . $arg . '=' . $redirect_args[$arg];
+        $argstring .= '&'.$arg.'='.$redirect_args[$arg];
     }
 }
 
-redirect_header($_POST['not_redirect'] . $argstring, 3, _NOT_UPDATEOK);
+redirect_header($_POST['not_redirect'].$argstring, 3, _NOT_UPDATEOK);
 exit();

htdocs/include/comment_reply.php

Spacing +6 added lines, -6 removed lines

@@ -25,7 +25,7 @@ 
 include_once $GLOBALS['xoops']->path('include/comment_constants.php');
 
 if (('system' !== $xoopsModule->getVar('dirname') && XOOPS_COMMENT_APPROVENONE == $xoopsModuleConfig['com_rule']) || (!is_object($xoopsUser) && !$xoopsModuleConfig['com_anonpost']) || !is_object($xoopsModule)) {
-    redirect_header(XOOPS_URL . '/user.php', 1, _NOPERM);
+    redirect_header(XOOPS_URL.'/user.php', 1, _NOPERM);
 }
 
 xoops_loadLanguage('comment');
@@ -58,10 +58,10 @@ 
 }
 // End edit by voltan
 
-$r_text    = _CM_POSTER . ': <strong>' . $r_name . '</strong>&nbsp;&nbsp;' . _CM_POSTED . ': <strong>' . formatTimestamp($comment->getVar('com_created')) . '</strong><br><br>' . $comment->getVar('com_text');
+$r_text    = _CM_POSTER.': <strong>'.$r_name.'</strong>&nbsp;&nbsp;'._CM_POSTED.': <strong>'.formatTimestamp($comment->getVar('com_created')).'</strong><br><br>'.$comment->getVar('com_text');
 $com_title = $comment->getVar('com_title', 'E');
-if (!preg_match('/^' . _RE . '/i', $com_title)) {
-    $com_title = _RE . ' ' . xoops_substr($com_title, 0, 56);
+if (!preg_match('/^'._RE.'/i', $com_title)) {
+    $com_title = _RE.' '.xoops_substr($com_title, 0, 56);
 }
 $com_pid    = $com_id;
 $com_text   = '';
@@ -82,8 +82,8 @@ 
 
 include_once $GLOBALS['xoops']->path('header.php');
 echo '<table cellpadding="4" cellspacing="1" width="98%" class="outer">
-      <tr><td class="head">' . $comment->getVar('com_title') . '</td></tr>
-      <tr><td><br>' . $r_text . '<br></td></tr>
+      <tr><td class="head">' . $comment->getVar('com_title').'</td></tr>
+      <tr><td><br>' . $r_text.'<br></td></tr>
       </table>';
 include_once $GLOBALS['xoops']->path('include/comment_form.php');
 include_once $GLOBALS['xoops']->path('footer.php');

htdocs/include/cp_header.php

Spacing +4 added lines, -4 removed lines

@@ -20,7 +20,7 @@ 
  * the file that will include this file must be located under xoops_url/modules/module_directory_name/admin_directory_name/
  */
 $xoopsOption['pagetype'] = 'admin';
-include_once dirname(__DIR__) . '/mainfile.php';
+include_once dirname(__DIR__).'/mainfile.php';
 include_once $GLOBALS['xoops']->path('include/cp_functions.php');
 
 /* @var $moduleperm_handler XoopsGroupPermHandler  */
@@ -35,7 +35,7 @@ 
         redirect_header(XOOPS_URL, 1, _NOPERM);
     }
 } else {
-    redirect_header(XOOPS_URL . '/user.php', 1, _NOPERM);
+    redirect_header(XOOPS_URL.'/user.php', 1, _NOPERM);
 }
 
 // set config values for this module
@@ -46,9 +46,9 @@ 
 }
 
 // include the default language file for the admin interface
-if (file_exists($file = $GLOBALS['xoops']->path('modules/' . $xoopsModule->getVar('dirname') . '/language/' . $xoopsConfig['language'] . '/admin.php'))) {
+if (file_exists($file = $GLOBALS['xoops']->path('modules/'.$xoopsModule->getVar('dirname').'/language/'.$xoopsConfig['language'].'/admin.php'))) {
     include_once $file;
-} elseif (file_exists($file = $GLOBALS['xoops']->path('modules/' . $xoopsModule->getVar('dirname') . '/language/english/admin.php'))) {
+} elseif (file_exists($file = $GLOBALS['xoops']->path('modules/'.$xoopsModule->getVar('dirname').'/language/english/admin.php'))) {
     include_once $file;
 }
 // I will disable this because module developer should nod be forced to have a admin.php