mambax7 / chess

player_stats.php

Spacing +13 added lines, -13 removed lines

@@ -37,22 +37,22 @@ 
  */
 
 $GLOBALS['xoopsOption']['template_main'] = 'chess_player_stats.tpl';
-require __DIR__ . '/header.php';
+require __DIR__.'/header.php';
 
-require_once XOOPS_ROOT_PATH . '/class/xoopsformloader.php';
-require_once XOOPS_ROOT_PATH . '/class/pagenav.php';
-require_once XOOPS_ROOT_PATH . '/modules/chess/include/constants.inc.php';
-require_once XOOPS_ROOT_PATH . '/modules/chess/include/functions.php';
-require_once XOOPS_ROOT_PATH . '/modules/chess/include/ratings.php';
+require_once XOOPS_ROOT_PATH.'/class/xoopsformloader.php';
+require_once XOOPS_ROOT_PATH.'/class/pagenav.php';
+require_once XOOPS_ROOT_PATH.'/modules/chess/include/constants.inc.php';
+require_once XOOPS_ROOT_PATH.'/modules/chess/include/functions.php';
+require_once XOOPS_ROOT_PATH.'/modules/chess/include/ratings.php';
 
 $xoopsConfig['module_cache'][$xoopsModule->getVar('mid')] = 0; // disable caching
 
 // user input
-$player_uid   = (int)($_POST['player_uid'] ?? @$_GET['player_uid']);
+$player_uid   = (int) ($_POST['player_uid'] ?? @$_GET['player_uid']);
 $player_uname = trim(@$_POST['player_uname']); // unsanitized
-$cstart       = (int)@$_GET['cstart']; // for page nav: offset of first row of results (challenges) to display (default to 0)
-$gstart       = (int)@$_GET['gstart']; // for page nav: offset of first row of results (games) to display (default to 0)
-$show_option  = (int)($_POST['show_option'] ?? @$_GET['show_option']);
+$cstart       = (int) @$_GET['cstart']; // for page nav: offset of first row of results (challenges) to display (default to 0)
+$gstart       = (int) @$_GET['gstart']; // for page nav: offset of first row of results (games) to display (default to 0)
+$show_option  = (int) ($_POST['show_option'] ?? @$_GET['show_option']);
 
 #var_dump($_REQUEST);#*#DEBUG#
 
@@ -70,13 +70,13 @@ 
 
 // Check that both user ID and username are now defined.
 if (0 == $player_uid || empty($player_uname)) {
-    redirect_header(XOOPS_URL . '/modules/chess/index.php', _CHESS_REDIRECT_DELAY_FAILURE, _MD_CHESS_PLAYER_NOT_FOUND);
+    redirect_header(XOOPS_URL.'/modules/chess/index.php', _CHESS_REDIRECT_DELAY_FAILURE, _MD_CHESS_PLAYER_NOT_FOUND);
 }
 
 // Display stats
 chess_player_stats($player_uid, $player_uname, $show_option, $cstart, $gstart);
 
-require_once XOOPS_ROOT_PATH . '/footer.php';
+require_once XOOPS_ROOT_PATH.'/footer.php';
 /**#@-*/
 
 /**
@@ -293,7 +293,7 @@ 
 
     $memberHandler = xoops_getHandler('member');
 
-    $criteria = new \Criteria('uid', '(' . implode(',', array_keys($userids)) . ')', 'IN');
+    $criteria = new \Criteria('uid', '('.implode(',', array_keys($userids)).')', 'IN');
 
     $usernames = $memberHandler->getUserList($criteria);
 

header.php

Spacing +3 added lines, -3 removed lines

@@ -35,7 +35,7 @@
 /**#@+
  */
 
-include __DIR__ . '/preloads/autoloader.php';
+include __DIR__.'/preloads/autoloader.php';
 
-require dirname(__DIR__, 2) . '/mainfile.php';
-require XOOPS_ROOT_PATH . '/header.php';
+require dirname(__DIR__, 2).'/mainfile.php';
+require XOOPS_ROOT_PATH.'/header.php';

game.php

Spacing +40 added lines, -40 removed lines

@@ -42,17 +42,17 @@ 
  */
 
 $GLOBALS['xoopsOption']['template_main'] = 'chess_game_main.tpl';
-require __DIR__ . '/header.php';
+require __DIR__.'/header.php';
 
-require_once XOOPS_ROOT_PATH . '/class/xoopsformloader.php';
+require_once XOOPS_ROOT_PATH.'/class/xoopsformloader.php';
 $xoopsConfig['module_cache'][$xoopsModule->getVar('mid')] = 0; // disable caching
-require_once XOOPS_ROOT_PATH . '/modules/chess/include/constants.inc.php';
-require_once XOOPS_ROOT_PATH . '/modules/chess/include/functions.php';
+require_once XOOPS_ROOT_PATH.'/modules/chess/include/constants.inc.php';
+require_once XOOPS_ROOT_PATH.'/modules/chess/include/functions.php';
 
 chess_game();
 
-require_once XOOPS_ROOT_PATH . '/include/comment_view.php';
-require_once XOOPS_ROOT_PATH . '/footer.php';
+require_once XOOPS_ROOT_PATH.'/include/comment_view.php';
+require_once XOOPS_ROOT_PATH.'/footer.php';
 /**#@-*/
 
 /**
@@ -68,7 +68,7 @@ 
 {
     // user input
 
-    $game_id = (int)@$_GET['game_id'];
+    $game_id = (int) @$_GET['game_id'];
 
     $submit_move = isset($_POST['submit_move']);
 
@@ -96,9 +96,9 @@ 
 
     if (($submit_move || $submit_refresh || $submit_arbitrate || $show_arbiter_ctrl) && is_object($GLOBALS['xoopsSecurity']) && !$GLOBALS['xoopsSecurity']->check()) {
         redirect_header(
-            XOOPS_URL . '/modules/chess/',
+            XOOPS_URL.'/modules/chess/',
             _CHESS_REDIRECT_DELAY_FAILURE,
-            _MD_CHESS_TOKEN_ERROR . '<br>' . implode('<br>', $GLOBALS['xoopsSecurity']->getErrors())
+            _MD_CHESS_TOKEN_ERROR.'<br>'.implode('<br>', $GLOBALS['xoopsSecurity']->getErrors())
         );
     }
 
@@ -107,15 +107,15 @@ 
     $gamedata = chess_get_game($game_id);
 
     if (false === $gamedata) {
-        redirect_header(XOOPS_URL . '/modules/chess/', _CHESS_REDIRECT_DELAY_FAILURE, _MD_CHESS_GAME_NOT_FOUND);
+        redirect_header(XOOPS_URL.'/modules/chess/', _CHESS_REDIRECT_DELAY_FAILURE, _MD_CHESS_GAME_NOT_FOUND);
     }
 
     $gamedata_updated = false;
 
     $move_performed        = false; // status result from move-handler
-    $move_result_text      = '';    // text result from move-handler
-    $draw_claim_error_text = '';    // text describing invalid draw-claim
-    $notify                = '';    // text description of action for notification
+    $move_result_text      = ''; // text result from move-handler
+    $draw_claim_error_text = ''; // text describing invalid draw-claim
+    $notify                = ''; // text description of action for notification
     $delete_game           = false;
 
     global $xoopsUser;
@@ -128,7 +128,7 @@ 
         if ($uid == $gamedata['white_uid']) {
             $user_color = $gamedata['fen_active_color']; // current user is either player, so consider him active player
         } else {
-            $user_color = '';                            // current user is not a player
+            $user_color = ''; // current user is not a player
         }
         // not self-play
     } else {
@@ -137,7 +137,7 @@ 
         } elseif ($uid == $gamedata['black_uid']) {
             $user_color = 'b'; // current user is black
         } else {
-            $user_color = '';  // current user is not a player
+            $user_color = ''; // current user is not a player
         }
     }
 
@@ -225,7 +225,7 @@ 
 
                         $gamedata['pgn_result'] = '1/2-1/2';
 
-                        $comment = '{' . $draw_claim_text . '}';
+                        $comment = '{'.$draw_claim_text.'}';
 
                         $gamedata['pgn_movetext'] = str_replace('*', "{$gamedata['pgn_result']} $comment", $gamedata['pgn_movetext']);
 
@@ -243,13 +243,13 @@ 
 
                     $gamedata['pgn_result'] = 'w' == $user_color ? '0-1' : '1-0';
 
-                    $comment = '{' . $user_color_name . ' ' . _MD_CHESS_RESIGNED . '}';
+                    $comment = '{'.$user_color_name.' '._MD_CHESS_RESIGNED.'}';
 
                     $gamedata['pgn_movetext'] = str_replace('*', "{$gamedata['pgn_result']} $comment", $gamedata['pgn_movetext']);
 
                     $gamedata_updated = true;
 
-                    $notify = "$user_color_name " . _MD_CHESS_RESIGNED;
+                    $notify = "$user_color_name "._MD_CHESS_RESIGNED;
                 }
                 break;
             case _CHESS_MOVETYPE_OFFER_DRAW:
@@ -258,7 +258,7 @@ 
 
                     $gamedata_updated = true;
 
-                    $notify = "$user_color_name " . _MD_CHESS_OFFERED_DRAW;
+                    $notify = "$user_color_name "._MD_CHESS_OFFERED_DRAW;
                 }
                 break;
             case _CHESS_MOVETYPE_ACCEPT_DRAW:
@@ -267,13 +267,13 @@ 
 
                     $gamedata['pgn_result'] = '1/2-1/2';
 
-                    $comment = '{' . _MD_CHESS_DRAW_BY_AGREEMENT . '}';
+                    $comment = '{'._MD_CHESS_DRAW_BY_AGREEMENT.'}';
 
                     $gamedata['pgn_movetext'] = str_replace('*', "{$gamedata['pgn_result']} $comment", $gamedata['pgn_movetext']);
 
                     $gamedata_updated = true;
 
-                    $notify = "$user_color_name " . _MD_CHESS_ACCEPTED_DRAW;
+                    $notify = "$user_color_name "._MD_CHESS_ACCEPTED_DRAW;
                 }
 
             // no break
@@ -283,7 +283,7 @@ 
 
                     $gamedata_updated = true;
 
-                    $notify = "$user_color_name " . _MD_CHESS_REJECTED_DRAW;
+                    $notify = "$user_color_name "._MD_CHESS_REJECTED_DRAW;
                 }
                 break;
             case _CHESS_MOVETYPE_RESTART:
@@ -324,7 +324,7 @@ 
             case _CHESS_MOVETYPE_DELETE:
                 if ($user_is_player && ($selfplay || chess_can_delete())) {
                     $delete_game = true; // must defer actual deletion until after notifications are sent
-                    $notify      = eval('return \'' . _MD_CHESS_DELETED_GAME . '\';'); // eval references $username
+                    $notify      = eval('return \''._MD_CHESS_DELETED_GAME.'\';'); // eval references $username
                 }
                 break;
             case _CHESS_MOVETYPE_WANT_ARBITRATION:
@@ -339,7 +339,7 @@ 
 
                     $gamedata_updated = true;
 
-                    $notify = "$user_color_name " . _MD_CHESS_RQSTED_ARBT . ' ' . _MD_CHESS_BEEN_SUSPENDED;
+                    $notify = "$user_color_name "._MD_CHESS_RQSTED_ARBT.' '._MD_CHESS_BEEN_SUSPENDED;
                 }
                 break;
         }
@@ -373,7 +373,7 @@ 
 
                     $gamedata_updated = true;
 
-                    $notify = eval('return \'' . _MD_CHESS_RESUMED_PLAY . '\';'); // eval references $username
+                    $notify = eval('return \''._MD_CHESS_RESUMED_PLAY.'\';'); // eval references $username
                 }
                 break;
             case _CHESS_ARBITER_DRAW:
@@ -386,7 +386,7 @@ 
 
                     $arbiter_explain = str_replace('}', ')', $arbiter_explain);
 
-                    $comment = '{' . sprintf(_MD_CHESS_DRAW_DECLARED, $arbiter_explain) . '}';
+                    $comment = '{'.sprintf(_MD_CHESS_DRAW_DECLARED, $arbiter_explain).'}';
 
                     $gamedata['pgn_movetext'] = str_replace('*', "{$gamedata['pgn_result']} $comment", $gamedata['pgn_movetext']);
 
@@ -394,13 +394,13 @@ 
 
                     $gamedata_updated = true;
 
-                    $notify = eval('return \'' . _MD_CHESS_DECLARED_DRAW . '\';'); // eval references $username
+                    $notify = eval('return \''._MD_CHESS_DECLARED_DRAW.'\';'); // eval references $username
                 }
                 break;
             case _CHESS_ARBITER_DELETE:
                 if ($gamedata['suspended']) {
                     $delete_game = true; // must defer actual deletion until after notifications are sent
-                    $notify      = eval('return \'' . _MD_CHESS_DELETED_GAME . '\';'); // eval references $username
+                    $notify      = eval('return \''._MD_CHESS_DELETED_GAME.'\';'); // eval references $username
                 }
                 break;
             case _CHESS_ARBITER_SUSPEND:
@@ -415,7 +415,7 @@ 
 
                     $gamedata_updated = true;
 
-                    $notify = eval('return \'' . _MD_CHESS_SUSPENDED_PLAY . '\';'); // eval references $username
+                    $notify = eval('return \''._MD_CHESS_SUSPENDED_PLAY.'\';'); // eval references $username
                 }
                 break;
             default:
@@ -447,7 +447,7 @@ 
         if (_CHESS_MOVETYPE_WANT_ARBITRATION == $movetype) {
             $event = 'notify_request_arbitration';
 
-            $username = $xoopsUser ? $xoopsUser->getVar('uname') : '(' . _MD_CHESS_UNKNOWN . ')';
+            $username = $xoopsUser ? $xoopsUser->getVar('uname') : '('._MD_CHESS_UNKNOWN.')';
 
             $extra_tags = ['CHESS_REQUESTOR' => $username, 'CHESS_GAME_ID' => $game_id, 'CHESS_EXPLAIN' => $move_explain];
 
@@ -460,7 +460,7 @@ 
     if ($delete_game) {
         chess_delete_game($game_id);
 
-        redirect_header(XOOPS_URL . '/modules/chess/', _CHESS_REDIRECT_DELAY_SUCCESS, _MD_CHESS_GAME_DELETED);
+        redirect_header(XOOPS_URL.'/modules/chess/', _CHESS_REDIRECT_DELAY_SUCCESS, _MD_CHESS_GAME_DELETED);
     }
 }
 
@@ -532,11 +532,11 @@ 
     );
 
     if ($xoopsDB->errno()) {
-        trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
+        trigger_error($xoopsDB->errno().':'.$xoopsDB->error(), E_USER_ERROR);
     }
 
     if ('*' != $gamedata['pgn_result'] && '1' == $gamedata['is_rated']) {
-        require_once XOOPS_ROOT_PATH . '/modules/chess/include/ratings.php';
+        require_once XOOPS_ROOT_PATH.'/modules/chess/include/ratings.php';
 
         chess_ratings_adj($game_id);
     }
@@ -560,7 +560,7 @@ 
     $xoopsDB->query("DELETE FROM $table WHERE game_id='$game_id'");
 
     if ($xoopsDB->errno()) {
-        trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
+        trigger_error($xoopsDB->errno().':'.$xoopsDB->error(), E_USER_ERROR);
     }
 }
 
@@ -733,7 +733,7 @@ 
 
     // The current position is included, since that's the position against which the other positions will be compared.
 
-    $repeats[] = $gamedata['fen_fullmove_number'] . $gamedata['fen_active_color'];
+    $repeats[] = $gamedata['fen_fullmove_number'].$gamedata['fen_active_color'];
 
     // Compare initial board position with last board position, unless the move number is the same, meaning that there haven't been any moves.
 
@@ -742,7 +742,7 @@ 
     $board_state = "{$tmp_gamedata['fen_piece_placement']} {$tmp_gamedata['fen_active_color']} {$tmp_gamedata['fen_castling_availability']} {$tmp_gamedata['fen_en_passant_target_square']}";
 
     if ($tmp_gamedata['fen_fullmove_number'] != $last_move_number && $board_state == $last_board_state) {
-        $repeats[] = $tmp_gamedata['fen_fullmove_number'] . $tmp_gamedata['fen_active_color'];
+        $repeats[] = $tmp_gamedata['fen_fullmove_number'].$tmp_gamedata['fen_active_color'];
 
         if (CHESS_LOG_3FOLD) {
             $log[] = sprintf('%08x %03d%1s %s', crc32($board_state), $tmp_gamedata['fen_fullmove_number'], $tmp_gamedata['fen_active_color'], $board_state);
@@ -806,7 +806,7 @@ 
             }
 
             if ($tmp_gamedata['fen_fullmove_number'] != $last_move_number && $board_state == $last_board_state) {
-                $repeats[] = $tmp_gamedata['fen_fullmove_number'] . $tmp_gamedata['fen_active_color'];
+                $repeats[] = $tmp_gamedata['fen_fullmove_number'].$tmp_gamedata['fen_active_color'];
 
                 #*#DEBUG# - start
 
@@ -853,7 +853,7 @@ 
     }
 
     if (CHESS_LOG_3FOLD) {
-        $logfile = XOOPS_ROOT_PATH . '/cache/' . date('Ymd_His') . '_3fold.log';
+        $logfile = XOOPS_ROOT_PATH.'/cache/'.date('Ymd_His').'_3fold.log';
 
         sort($log);
 
@@ -916,7 +916,7 @@ 
     $xoopsTpl->assign(
         'xoops_module_header',
         '
-        <link rel="stylesheet" type="text/css" media="screen" href="' . XOOPS_URL . '/modules/chess/assets/css/style.css">
+        <link rel="stylesheet" type="text/css" media="screen" href="' . XOOPS_URL.'/modules/chess/assets/css/style.css">
     '
     );
 
@@ -959,7 +959,7 @@ 
         $expanded_row = preg_replace_callback(
             '/(\d)/',
 
-            static function ($matches) {
+            static function($matches) {
                 return str_repeat('x', $matches[1]);
             },
             $rank

help.php

Spacing +7 added lines, -7 removed lines

@@ -37,18 +37,18 @@ 
  */
 
 $GLOBALS['xoopsOption']['template_main'] = 'chess_help.tpl';
-require __DIR__ . '/header.php';
+require __DIR__.'/header.php';
 
 $xoopsConfig['module_cache'][$xoopsModule->getVar('mid')] = 0; // disable caching
-require_once XOOPS_ROOT_PATH . '/modules/' . $xoopsModule->getVar('dirname') . '/include/functions.php';
-if (file_exists(XOOPS_ROOT_PATH . '/modules/' . $xoopsModule->getVar('dirname') . '/language/' . $xoopsConfig['language'] . '/help.php')) {
-    require_once XOOPS_ROOT_PATH . '/modules/' . $xoopsModule->getVar('dirname') . '/language/' . $xoopsConfig['language'] . '/help.php';
+require_once XOOPS_ROOT_PATH.'/modules/'.$xoopsModule->getVar('dirname').'/include/functions.php';
+if (file_exists(XOOPS_ROOT_PATH.'/modules/'.$xoopsModule->getVar('dirname').'/language/'.$xoopsConfig['language'].'/help.php')) {
+    require_once XOOPS_ROOT_PATH.'/modules/'.$xoopsModule->getVar('dirname').'/language/'.$xoopsConfig['language'].'/help.php';
 } else {
-    require_once XOOPS_ROOT_PATH . '/modules/' . $xoopsModule->getVar('dirname') . '/language/english/help.php';
+    require_once XOOPS_ROOT_PATH.'/modules/'.$xoopsModule->getVar('dirname').'/language/english/help.php';
 }
 
 $rating_system          = chess_moduleConfig('rating_system');
-$rating_system_des_name = '_HE_CHESS_RATINGS_' . mb_strtoupper($rating_system);
+$rating_system_des_name = '_HE_CHESS_RATINGS_'.mb_strtoupper($rating_system);
 if (defined($rating_system_des_name)) {
     $rating_system_des = constant($rating_system_des_name);
 } else {
@@ -64,6 +64,6 @@ 
 $xoopsTpl->assign('chess_allow_delete', chess_can_delete());
 $xoopsTpl->assign('chess_is_admin', is_object($xoopsUser) && $xoopsUser->isAdmin($xoopsModule->getVar('mid')));
 
-require_once XOOPS_ROOT_PATH . '/footer.php';
+require_once XOOPS_ROOT_PATH.'/footer.php';
 
 /**#@-*/

create.php

Spacing +58 added lines, -58 removed lines

@@ -47,24 +47,24 @@ 
 /**#@+
  */
 
-require __DIR__ . '/header.php';
+require __DIR__.'/header.php';
 $xoopsConfig['module_cache'][$xoopsModule->getVar('mid')] = 0; // disable caching
 
-require_once XOOPS_ROOT_PATH . '/class/xoopsformloader.php';
-require_once XOOPS_ROOT_PATH . '/modules/chess/include/constants.inc.php';
-require_once XOOPS_ROOT_PATH . '/modules/chess/include/functions.php';
+require_once XOOPS_ROOT_PATH.'/class/xoopsformloader.php';
+require_once XOOPS_ROOT_PATH.'/modules/chess/include/constants.inc.php';
+require_once XOOPS_ROOT_PATH.'/modules/chess/include/functions.php';
 
 #var_dump($_REQUEST);#*#DEBUG#
 
 if (!chess_can_play()) {
-    redirect_header(XOOPS_URL . '/index.php', _CHESS_REDIRECT_DELAY_FAILURE, _NOPERM);
+    redirect_header(XOOPS_URL.'/index.php', _CHESS_REDIRECT_DELAY_FAILURE, _NOPERM);
 }
 
 // user input
 //$gametype          = chess_sanitize(@$_POST['gametype']);
 //$opponent          = chess_sanitize(trim(@$_POST['opponent']), _CHESS_USERNAME_ALLOWED_CHARACTERS);
 $gametype     = Request::getCmd('gametype', '', 'POST');
-$opponent     = preg_replace('/[^' . _CHESS_USERNAME_ALLOWED_CHARACTERS . ']/i', '', Request::getString('opponent', '', 'POST'));
+$opponent     = preg_replace('/[^'._CHESS_USERNAME_ALLOWED_CHARACTERS.']/i', '', Request::getString('opponent', '', 'POST'));
 $opponent     = trim($opponent);
 $opponent_uid      = !empty($opponent) ? chess_opponent_uid($opponent) : 0;
 //$fen               = chess_moduleConfig('allow_setup') ? chess_sanitize(trim(@$_POST['fen']), 'A-Za-z0-9 /-') : '';
@@ -89,9 +89,9 @@ 
 // If form-submit, check security token.
 if (($submit_challenge1 || $submit_challenge2 || $submit_challenge3 || $submit_accept || $submit_delete || $show_arbiter_ctrl) && is_object($GLOBALS['xoopsSecurity']) && !$GLOBALS['xoopsSecurity']->check()) {
     redirect_header(
-        XOOPS_URL . '/modules/chess/',
+        XOOPS_URL.'/modules/chess/',
         _CHESS_REDIRECT_DELAY_FAILURE,
-        _MD_CHESS_TOKEN_ERROR . '<br>' . implode('<br>', $GLOBALS['xoopsSecurity']->getErrors())
+        _MD_CHESS_TOKEN_ERROR.'<br>'.implode('<br>', $GLOBALS['xoopsSecurity']->getErrors())
     );
 }
 
@@ -106,7 +106,7 @@ 
 $is_arbiter = is_object($xoopsUser) && $xoopsUser->isAdmin($xoopsModule->getVar('mid'));
 
 if ($cancel_challenge1) {
-    redirect_header(XOOPS_URL . '/modules/chess/', _CHESS_REDIRECT_DELAY_SUCCESS, _TAKINGBACK);
+    redirect_header(XOOPS_URL.'/modules/chess/', _CHESS_REDIRECT_DELAY_SUCCESS, _TAKINGBACK);
 } elseif ($cancel_challenge2) {
     chess_show_create_form1($gametype);
 } elseif ($cancel_challenge3) {
@@ -116,12 +116,12 @@ 
         chess_show_create_form1($gametype, $fen);
     }
 } elseif ($cancel_accept) {
-    redirect_header(XOOPS_URL . '/modules/chess/', _CHESS_REDIRECT_DELAY_SUCCESS, _TAKINGBACK);
+    redirect_header(XOOPS_URL.'/modules/chess/', _CHESS_REDIRECT_DELAY_SUCCESS, _TAKINGBACK);
 } elseif ($submit_challenge1) {
     $fen_error = chess_fen_error($fen);
 
     if (!empty($fen_error)) {
-        chess_show_create_form1($gametype, $fen, _MD_CHESS_FEN_INVALID . ': ' . $fen_error);
+        chess_show_create_form1($gametype, $fen, _MD_CHESS_FEN_INVALID.': '.$fen_error);
     } elseif (_CHESS_GAMETYPE_OPEN == $gametype || _CHESS_GAMETYPE_USER == $gametype) {
         chess_show_create_form2($gametype, $fen);
     } else {
@@ -145,7 +145,7 @@ 
     if (_CHESS_GAMETYPE_OPEN == $gametype) {
         chess_create_challenge($gametype, $fen, $coloroption, $rated, $notify_accept, $notify_move);
 
-        redirect_header(XOOPS_URL . '/modules/chess/', _CHESS_REDIRECT_DELAY_SUCCESS, _MD_CHESS_GAME_CREATED);
+        redirect_header(XOOPS_URL.'/modules/chess/', _CHESS_REDIRECT_DELAY_SUCCESS, _MD_CHESS_GAME_CREATED);
     } elseif (_CHESS_GAMETYPE_USER == $gametype) {
         if (empty($opponent)) {
             chess_show_create_form2($gametype, $fen, $coloroption, $opponent_uid, $rated, _MD_CHESS_OPPONENT_MISSING);
@@ -156,12 +156,12 @@ 
         } else {
             chess_create_challenge($gametype, $fen, $coloroption, $rated, $notify_accept, $notify_move, $opponent_uid);
 
-            redirect_header(XOOPS_URL . '/modules/chess/', _CHESS_REDIRECT_DELAY_SUCCESS, _MD_CHESS_GAME_CREATED);
+            redirect_header(XOOPS_URL.'/modules/chess/', _CHESS_REDIRECT_DELAY_SUCCESS, _MD_CHESS_GAME_CREATED);
         }
     } elseif (_CHESS_GAMETYPE_SELF == $gametype) {
         $game_id = chess_create_game($uid, $uid, $fen, $rated);
 
-        redirect_header(XOOPS_URL . "/modules/chess/game.php?game_id=$game_id", _CHESS_REDIRECT_DELAY_SUCCESS, _MD_CHESS_GAME_CREATED);
+        redirect_header(XOOPS_URL."/modules/chess/game.php?game_id=$game_id", _CHESS_REDIRECT_DELAY_SUCCESS, _MD_CHESS_GAME_CREATED);
     } else {
         chess_show_create_form1($gametype, $fen, _MD_CHESS_GAMETYPE_INVALID);
     }
@@ -172,9 +172,9 @@ 
         if ($is_arbiter || chess_is_challenger($challenge_id)) {
             chess_delete_challenge($challenge_id);
 
-            redirect_header(XOOPS_URL . '/modules/chess/', _CHESS_REDIRECT_DELAY_SUCCESS, _MD_CHESS_GAME_DELETED);
+            redirect_header(XOOPS_URL.'/modules/chess/', _CHESS_REDIRECT_DELAY_SUCCESS, _MD_CHESS_GAME_DELETED);
         } else {
-            redirect_header(XOOPS_URL . '/modules/chess/', _CHESS_REDIRECT_DELAY_FAILURE, _NOPERM);
+            redirect_header(XOOPS_URL.'/modules/chess/', _CHESS_REDIRECT_DELAY_FAILURE, _NOPERM);
         }
     } else {
         chess_show_delete_form($challenge_id, $show_arbiter_ctrl && $is_arbiter, _MD_CHESS_NO_CONFIRM_DELETE);
@@ -189,7 +189,7 @@ 
     chess_show_create_form1($gametype);
 }
 
-require_once XOOPS_ROOT_PATH . '/footer.php';
+require_once XOOPS_ROOT_PATH.'/footer.php';
 /**#@-*/
 
 /**
@@ -204,10 +204,10 @@ 
     $form = new XoopsThemeForm(_MD_CHESS_CREATE_FORM, 'create_form1', 'create.php', 'post', true);
 
     if ($error_msg) {
-        $form->addElement(new XoopsFormLabel(_MD_CHESS_ERROR . ': ', '<div class="errorMsg">' . $error_msg . '</div>'));
+        $form->addElement(new XoopsFormLabel(_MD_CHESS_ERROR.': ', '<div class="errorMsg">'.$error_msg.'</div>'));
     }
 
-    $menu_gametype = new XoopsFormSelect(_MD_CHESS_LABEL_GAMETYPE . ':', 'gametype', $gametype, 1, false);
+    $menu_gametype = new XoopsFormSelect(_MD_CHESS_LABEL_GAMETYPE.':', 'gametype', $gametype, 1, false);
 
     $menu_gametype->addOption(_CHESS_GAMETYPE_OPEN, _MD_CHESS_MENU_GAMETYPE_OPEN);
 
@@ -222,7 +222,7 @@ 
 
         $form->addElement(new XoopsFormLabel('', _MD_CHESS_LABEL_FEN_EXPLAIN));
 
-        $form->addElement(new XoopsFormText(_MD_CHESS_LABEL_FEN_SETUP . ':', 'fen', 80, _CHESS_TEXTBOX_FEN_MAXLEN, $fen));
+        $form->addElement(new XoopsFormText(_MD_CHESS_LABEL_FEN_SETUP.':', 'fen', 80, _CHESS_TEXTBOX_FEN_MAXLEN, $fen));
     }
 
     $form->addElement(new XoopsFormLabel('&nbsp;', '&nbsp;'));
@@ -257,7 +257,7 @@ 
     $form->addElement(new XoopsFormHidden('fen', $fen));
 
     if ($error_msg) {
-        $form->addElement(new XoopsFormLabel(_MD_CHESS_ERROR . ':', '<div class="errorMsg">' . $error_msg . '</div>'));
+        $form->addElement(new XoopsFormLabel(_MD_CHESS_ERROR.':', '<div class="errorMsg">'.$error_msg.'</div>'));
     }
 
     $memberHandler = xoops_getHandler('member');
@@ -269,12 +269,12 @@ 
     if (_CHESS_GAMETYPE_USER == $gametype) {
         $form->addElement(
             new XoopsFormText(
-                _MD_CHESS_LABEL_OPPONENT . ':', 'opponent', _CHESS_TEXTBOX_OPPONENT_SIZE, _CHESS_TEXTBOX_OPPONENT_MAXLEN, $opponent_username
+                _MD_CHESS_LABEL_OPPONENT.':', 'opponent', _CHESS_TEXTBOX_OPPONENT_SIZE, _CHESS_TEXTBOX_OPPONENT_MAXLEN, $opponent_username
             )
         );
     }
 
-    $radio_color = new XoopsFormRadio(_MD_CHESS_LABEL_COLOR . ':', 'coloroption', $coloroption);
+    $radio_color = new XoopsFormRadio(_MD_CHESS_LABEL_COLOR.':', 'coloroption', $coloroption);
 
     $radio_color->addOption(_CHESS_COLOROPTION_OPPONENT, _MD_CHESS_RADIO_COLOR_OPPONENT);
 
@@ -289,7 +289,7 @@ 
     if ('none' !== chess_moduleConfig('rating_system')) {
         if (_CHESS_GAMETYPE_OPEN == $gametype || _CHESS_GAMETYPE_USER == $gametype) {
             if (chess_moduleConfig('allow_unrated_games')) {
-                $radio_rated = new XoopsFormRadio(_MD_CHESS_RATED_GAME . ':', 'rated', $rated);
+                $radio_rated = new XoopsFormRadio(_MD_CHESS_RATED_GAME.':', 'rated', $rated);
 
                 $radio_rated->addOption(1, _YES);
 
@@ -344,7 +344,7 @@ 
 
     $form->addElement(new XoopsFormHidden('rated', $rated));
 
-    $form->addElement(new XoopsFormLabel('', '<div class="confirmMsg">' . _MD_CHESS_GAME_CONFIRM . '</div>'));
+    $form->addElement(new XoopsFormLabel('', '<div class="confirmMsg">'._MD_CHESS_GAME_CONFIRM.'</div>'));
 
     switch ($gametype) {
         case _CHESS_GAMETYPE_OPEN:
@@ -361,10 +361,10 @@ 
             break;
     }
 
-    $form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_GAMETYPE . ':', $label_gametype));
+    $form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_GAMETYPE.':', $label_gametype));
 
     if (!empty($fen)) {
-        $form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_FEN_SETUP . ':', $fen));
+        $form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_FEN_SETUP.':', $fen));
     }
 
     if (_CHESS_GAMETYPE_USER == $gametype) {
@@ -374,7 +374,7 @@ 
 
         $opponent_username = is_object($opponent_user) ? $opponent_user->getVar('uname') : '';
 
-        $form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_OPPONENT . ':', $opponent_username));
+        $form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_OPPONENT.':', $opponent_username));
     }
 
     if (_CHESS_GAMETYPE_OPEN == $gametype || _CHESS_GAMETYPE_USER == $gametype) {
@@ -396,10 +396,10 @@ 
                 break;
         }
 
-        $form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_COLOR . ':', $label_coloroption));
+        $form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_COLOR.':', $label_coloroption));
 
         if ('none' != chess_moduleConfig('rating_system')) {
-            $form->addElement(new XoopsFormLabel(_MD_CHESS_RATED_GAME . ':', $rated ? _YES : _NO));
+            $form->addElement(new XoopsFormLabel(_MD_CHESS_RATED_GAME.':', $rated ? _YES : _NO));
         }
 
         // Determine whether current user is subscribed to receive a notification when his challenge is accepted.
@@ -468,7 +468,7 @@ 
     );
 
     if ($xoopsDB->getRowsNum($result) < 1) {
-        redirect_header(XOOPS_URL . '/modules/chess/', _CHESS_REDIRECT_DELAY_FAILURE, _MD_CHESS_GAME_NOT_FOUND);
+        redirect_header(XOOPS_URL.'/modules/chess/', _CHESS_REDIRECT_DELAY_FAILURE, _MD_CHESS_GAME_NOT_FOUND);
     }
 
     $row = $xoopsDB->fetchArray($result);
@@ -479,7 +479,7 @@ 
 
     $form->addElement(new XoopsFormHidden('challenge_id', $challenge_id));
 
-    $form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_DATE_CREATED . ':', formatTimestamp($row['create_date'], 'm')));
+    $form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_DATE_CREATED.':', formatTimestamp($row['create_date'], 'm')));
 
     $memberHandler = xoops_getHandler('member');
 
@@ -490,7 +490,7 @@ 
         case _CHESS_GAMETYPE_USER:
             $player2_user     = $memberHandler->getUser($row['player2_uid']);
             $player2_username = is_object($player2_user) ? $player2_user->getVar('uname') : '?';
-            $label_gametype   = _MD_CHESS_LABEL_GAMETYPE_USER . ': ' . $player2_username;
+            $label_gametype   = _MD_CHESS_LABEL_GAMETYPE_USER.': '.$player2_username;
             break;
         case _CHESS_GAMETYPE_SELF:
             $label_gametype = _MD_CHESS_LABEL_GAMETYPE_SELF;
@@ -500,7 +500,7 @@ 
             break;
     }
 
-    $form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_GAMETYPE . ':', $label_gametype));
+    $form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_GAMETYPE.':', $label_gametype));
 
     $player1_user = $memberHandler->getUser($row['player1_uid']);
 
@@ -508,20 +508,20 @@ 
 
     $form->addElement(
         new XoopsFormLabel(
-            _MD_CHESS_LABEL_CHALLENGER . ':', "<a href='" . XOOPS_URL . "/modules/chess/player_stats.php?player_uid={$row['player1_uid']}'>$player1_username</a>"
+            _MD_CHESS_LABEL_CHALLENGER.':', "<a href='".XOOPS_URL."/modules/chess/player_stats.php?player_uid={$row['player1_uid']}'>$player1_username</a>"
         )
     );
 
     $player2_username = $xoopsUser ? $xoopsUser->getVar('uname') : '?';
 
-    $form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_OPPONENT . ':', $player2_username));
+    $form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_OPPONENT.':', $player2_username));
 
     if (!empty($row['fen'])) {
-        $form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_FEN_SETUP . ':', $row['fen']));
+        $form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_FEN_SETUP.':', $row['fen']));
     }
 
     if (_CHESS_COLOROPTION_OPPONENT == $row['color_option']) {
-        $radio_color = new XoopsFormRadio(_MD_CHESS_LABEL_COLOR . ':', 'coloroption', _CHESS_COLOROPTION_RANDOM);
+        $radio_color = new XoopsFormRadio(_MD_CHESS_LABEL_COLOR.':', 'coloroption', _CHESS_COLOROPTION_RANDOM);
 
         $radio_color->addOption(_CHESS_COLOROPTION_RANDOM, _MD_CHESS_RADIO_COLOR_RANDOM);
 
@@ -546,11 +546,11 @@ 
                 break;
         }
 
-        $form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_COLOR . ':', $label_coloroption));
+        $form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_COLOR.':', $label_coloroption));
     }
 
     if ('none' != chess_moduleConfig('rating_system')) {
-        $form->addElement(new XoopsFormLabel(_MD_CHESS_RATED_GAME . ':', $row['is_rated'] ? _YES : _NO));
+        $form->addElement(new XoopsFormLabel(_MD_CHESS_RATED_GAME.':', $row['is_rated'] ? _YES : _NO));
     }
 
     // Display notification-subscribe checkbox, initially checked.
@@ -598,7 +598,7 @@ 
     );
 
     if ($xoopsDB->getRowsNum($result) < 1) {
-        redirect_header(XOOPS_URL . '/modules/chess/', _CHESS_REDIRECT_DELAY_FAILURE, _MD_CHESS_GAME_NOT_FOUND);
+        redirect_header(XOOPS_URL.'/modules/chess/', _CHESS_REDIRECT_DELAY_FAILURE, _MD_CHESS_GAME_NOT_FOUND);
     }
 
     $row = $xoopsDB->fetchArray($result);
@@ -614,10 +614,10 @@ 
     }
 
     if ($error_msg) {
-        $form->addElement(new XoopsFormLabel(_MD_CHESS_ERROR . ': ', '<div class="errorMsg">' . $error_msg . '</div>'));
+        $form->addElement(new XoopsFormLabel(_MD_CHESS_ERROR.': ', '<div class="errorMsg">'.$error_msg.'</div>'));
     }
 
-    $form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_DATE_CREATED . ':', formatTimestamp($row['create_date'], 'm')));
+    $form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_DATE_CREATED.':', formatTimestamp($row['create_date'], 'm')));
 
     $memberHandler = xoops_getHandler('member');
 
@@ -628,7 +628,7 @@ 
         case _CHESS_GAMETYPE_USER:
             $player2_user     = $memberHandler->getUser($row['player2_uid']);
             $player2_username = is_object($player2_user) ? $player2_user->getVar('uname') : '?';
-            $label_gametype   = _MD_CHESS_LABEL_GAMETYPE_USER . ': ' . $player2_username;
+            $label_gametype   = _MD_CHESS_LABEL_GAMETYPE_USER.': '.$player2_username;
             break;
         case _CHESS_GAMETYPE_SELF:
             $label_gametype = _MD_CHESS_LABEL_GAMETYPE_SELF;
@@ -638,16 +638,16 @@ 
             break;
     }
 
-    $form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_GAMETYPE . ':', $label_gametype));
+    $form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_GAMETYPE.':', $label_gametype));
 
     $player1_user = $memberHandler->getUser($row['player1_uid']);
 
     $player1_username = is_object($player1_user) ? $player1_user->getVar('uname') : '?';
 
-    $form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_CHALLENGER . ':', $player1_username));
+    $form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_CHALLENGER.':', $player1_username));
 
     if (!empty($row['fen'])) {
-        $form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_FEN_SETUP . ':', $row['fen']));
+        $form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_FEN_SETUP.':', $row['fen']));
     }
 
     switch ($row['color_option']) {
@@ -668,10 +668,10 @@ 
             break;
     }
 
-    $form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_COLOR . ':', $label_coloroption));
+    $form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_COLOR.':', $label_coloroption));
 
     if ('none' != chess_moduleConfig('rating_system')) {
-        $form->addElement(new XoopsFormLabel(_MD_CHESS_RATED_GAME . ':', $row['is_rated'] ? _YES : _NO));
+        $form->addElement(new XoopsFormLabel(_MD_CHESS_RATED_GAME.':', $row['is_rated'] ? _YES : _NO));
     }
 
     // Display confirm-delete checkbox, initially unchecked.
@@ -718,7 +718,7 @@ 
     );
 
     if ($xoopsDB->getRowsNum($result) < 1) {
-        redirect_header(XOOPS_URL . '/modules/chess/', _CHESS_REDIRECT_DELAY_FAILURE, _MD_CHESS_GAME_NOT_FOUND);
+        redirect_header(XOOPS_URL.'/modules/chess/', _CHESS_REDIRECT_DELAY_FAILURE, _MD_CHESS_GAME_NOT_FOUND);
     }
 
     $row = $xoopsDB->fetchArray($result);
@@ -728,9 +728,9 @@ 
     $uid = $xoopsUser ? $xoopsUser->getVar('uid') : 0;
 
     if (_CHESS_GAMETYPE_USER == $row['game_type'] && $uid != $row['player2_uid']) {
-        redirect_header(XOOPS_URL . '/modules/chess/', _CHESS_REDIRECT_DELAY_FAILURE, _MD_CHESS_WRONG_PLAYER2);
+        redirect_header(XOOPS_URL.'/modules/chess/', _CHESS_REDIRECT_DELAY_FAILURE, _MD_CHESS_WRONG_PLAYER2);
     } elseif ($uid == $row['player1_uid']) {
-        redirect_header(XOOPS_URL . '/modules/chess/', _CHESS_REDIRECT_DELAY_FAILURE, _MD_CHESS_SAME_PLAYER2);
+        redirect_header(XOOPS_URL.'/modules/chess/', _CHESS_REDIRECT_DELAY_FAILURE, _MD_CHESS_SAME_PLAYER2);
     }
 
     switch ($row['color_option']) {
@@ -783,7 +783,7 @@ 
     $xoopsDB->query("DELETE FROM $challenges_table WHERE challenge_id = '$challenge_id'");
 
     if ($xoopsDB->errno()) {
-        trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
+        trigger_error($xoopsDB->errno().':'.$xoopsDB->error(), E_USER_ERROR);
     }
 
     // Notify player 1 that his challenge has been accepted (if he has subscribed to the notification).
@@ -796,7 +796,7 @@ 
 
     $notificationHandler->triggerEvent('global', 0, 'notify_accept_challenge', $extra_tags, [$row['player1_uid']]);
 
-    redirect_header(XOOPS_URL . "/modules/chess/game.php?game_id=$game_id", _CHESS_REDIRECT_DELAY_SUCCESS, _MD_CHESS_GAME_CREATED);
+    redirect_header(XOOPS_URL."/modules/chess/game.php?game_id=$game_id", _CHESS_REDIRECT_DELAY_SUCCESS, _MD_CHESS_GAME_CREATED);
 }
 
 /**
@@ -814,7 +814,7 @@ 
     $result = $xoopsDB->query("SELECT player1_uid FROM $challenges_table WHERE challenge_id = '$challenge_id'");
 
     if ($xoopsDB->getRowsNum($result) < 1) {
-        redirect_header(XOOPS_URL . '/modules/chess/', _CHESS_REDIRECT_DELAY_FAILURE, _MD_CHESS_GAME_NOT_FOUND);
+        redirect_header(XOOPS_URL.'/modules/chess/', _CHESS_REDIRECT_DELAY_FAILURE, _MD_CHESS_GAME_NOT_FOUND);
     }
 
     $row = $xoopsDB->fetchArray($result);
@@ -875,7 +875,7 @@ 
     );
 
     if ($xoopsDB->errno()) {
-        trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
+        trigger_error($xoopsDB->errno().':'.$xoopsDB->error(), E_USER_ERROR);
     }
 
     $challenge_id = $xoopsDB->getInsertId();
@@ -917,7 +917,7 @@ 
     $xoopsDB->query("DELETE FROM $table WHERE challenge_id='$challenge_id'");
 
     if ($xoopsDB->errno()) {
-        trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
+        trigger_error($xoopsDB->errno().':'.$xoopsDB->error(), E_USER_ERROR);
     }
 }
 
@@ -983,7 +983,7 @@ 
     );
 
     if ($xoopsDB->errno()) {
-        trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
+        trigger_error($xoopsDB->errno().':'.$xoopsDB->error(), E_USER_ERROR);
     }
 
     $game_id = $xoopsDB->getInsertId();

index.php

Spacing +7 added lines, -7 removed lines

@@ -39,12 +39,12 @@ 
  */
 
 $GLOBALS['xoopsOption']['template_main'] = 'chess_games.tpl';
-require __DIR__ . '/header.php';
+require __DIR__.'/header.php';
 
-require_once XOOPS_ROOT_PATH . '/class/xoopsformloader.php';
-require_once XOOPS_ROOT_PATH . '/class/pagenav.php';
-require_once XOOPS_ROOT_PATH . '/modules/chess/include/constants.inc.php';
-require_once XOOPS_ROOT_PATH . '/modules/chess/include/functions.php';
+require_once XOOPS_ROOT_PATH.'/class/xoopsformloader.php';
+require_once XOOPS_ROOT_PATH.'/class/pagenav.php';
+require_once XOOPS_ROOT_PATH.'/modules/chess/include/constants.inc.php';
+require_once XOOPS_ROOT_PATH.'/modules/chess/include/functions.php';
 
 #var_dump($_REQUEST);#*#DEBUG#
 
@@ -260,7 +260,7 @@ 
 
     $memberHandler = xoops_getHandler('member');
 
-    $criteria = new \Criteria('uid', '(' . implode(',', array_keys($userids)) . ')', 'IN');
+    $criteria = new \Criteria('uid', '('.implode(',', array_keys($userids)).')', 'IN');
 
     $usernames = $memberHandler->getUserList($criteria);
 
@@ -352,4 +352,4 @@ 
      * $clist = $configHandler->getConfigList(18);
      * var_dump('clist', $clist);
      ***/
-include_once XOOPS_ROOT_PATH . '/footer.php';
+include_once XOOPS_ROOT_PATH.'/footer.php';

ratings.php

Spacing +13 added lines, -13 removed lines

@@ -37,34 +37,34 @@ 
  */
 
 $GLOBALS['xoopsOption']['template_main'] = 'chess_ratings.tpl';
-require __DIR__ . '/header.php';
+require __DIR__.'/header.php';
 
-require_once XOOPS_ROOT_PATH . '/class/xoopsformloader.php';
-require_once XOOPS_ROOT_PATH . '/class/pagenav.php';
-require_once XOOPS_ROOT_PATH . '/modules/chess/include/constants.inc.php';
-require_once XOOPS_ROOT_PATH . '/modules/chess/include/functions.php';
-require_once XOOPS_ROOT_PATH . '/modules/chess/include/ratings.php';
+require_once XOOPS_ROOT_PATH.'/class/xoopsformloader.php';
+require_once XOOPS_ROOT_PATH.'/class/pagenav.php';
+require_once XOOPS_ROOT_PATH.'/modules/chess/include/constants.inc.php';
+require_once XOOPS_ROOT_PATH.'/modules/chess/include/functions.php';
+require_once XOOPS_ROOT_PATH.'/modules/chess/include/ratings.php';
 
 // check whether the rating feature is enabled
 if ('none' == chess_moduleConfig('rating_system')) {
-    redirect_header(XOOPS_URL . '/modules/chess/index.php', _CHESS_REDIRECT_DELAY_FAILURE, _MD_CHESS_RATINGS_OFF);
+    redirect_header(XOOPS_URL.'/modules/chess/index.php', _CHESS_REDIRECT_DELAY_FAILURE, _MD_CHESS_RATINGS_OFF);
 }
 
 $xoopsConfig['module_cache'][$xoopsModule->getVar('mid')] = 0; // disable caching
 
 // user input
 $submit_recalc_ratings  = isset($_POST['submit_recalc_ratings']);
-$confirm_recalc_ratings = (int)@$_POST['confirm_recalc_ratings'];
-$start                  = (int)@$_GET['start']; // for page nav: offset of first row of results to display (default to 0)
+$confirm_recalc_ratings = (int) @$_POST['confirm_recalc_ratings'];
+$start                  = (int) @$_GET['start']; // for page nav: offset of first row of results to display (default to 0)
 
 #var_dump($_REQUEST);#*#DEBUG#
 
 // If form-submit, check security token.
 if ($submit_recalc_ratings && is_object($GLOBALS['xoopsSecurity']) && !$GLOBALS['xoopsSecurity']->check()) {
     redirect_header(
-        XOOPS_URL . '/modules/chess/ratings.php',
+        XOOPS_URL.'/modules/chess/ratings.php',
         _CHESS_REDIRECT_DELAY_FAILURE,
-        _MD_CHESS_TOKEN_ERROR . '<br>' . implode('<br>', $GLOBALS['xoopsSecurity']->getErrors())
+        _MD_CHESS_TOKEN_ERROR.'<br>'.implode('<br>', $GLOBALS['xoopsSecurity']->getErrors())
     );
 }
 
@@ -89,7 +89,7 @@ 
 // Display the ratings.
 chess_ratings($start, $msg, $msg_class);
 
-require_once XOOPS_ROOT_PATH . '/footer.php';
+require_once XOOPS_ROOT_PATH.'/footer.php';
 /**#@-*/
 
 /**
@@ -167,7 +167,7 @@ 
 
     $memberHandler = xoops_getHandler('member');
 
-    $criteria = new \Criteria('uid', '(' . implode(',', $userids) . ')', 'IN');
+    $criteria = new \Criteria('uid', '('.implode(',', $userids).')', 'IN');
 
     $usernames = $memberHandler->getUserList($criteria);