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

Indentation +635 added lines, -635 removed lines

@@ -66,402 +66,402 @@ 
  */
 function chess_game()
 {
-    // user input
+	// user input
 
-    $game_id = (int)@$_GET['game_id'];
+	$game_id = (int)@$_GET['game_id'];
 
-    $submit_move = isset($_POST['submit_move']);
+	$submit_move = isset($_POST['submit_move']);
 
-    $submit_refresh = isset($_POST['submit_refresh']);
+	$submit_refresh = isset($_POST['submit_refresh']);
 
-    $move = chess_sanitize(trim(@$_POST['chessmove']), 'A-Za-z0-9=-');
+	$move = chess_sanitize(trim(@$_POST['chessmove']), 'A-Za-z0-9=-');
 
-    $movetype = chess_sanitize(@$_POST['movetype']);
+	$movetype = chess_sanitize(@$_POST['movetype']);
 
-    $confirm = isset($_POST['confirm']) ? 1 : 0;
+	$confirm = isset($_POST['confirm']) ? 1 : 0;
 
-    $move_explain = chess_sanitize(trim(@$_POST['move_explain']), 'A-Za-z0-9 .,;:=()[]*?!-');
+	$move_explain = chess_sanitize(trim(@$_POST['move_explain']), 'A-Za-z0-9 .,;:=()[]*?!-');
 
-    $arbiter_explain = chess_sanitize(trim(@$_POST['arbiter_explain']), 'A-Za-z0-9 .,;:=()[]*?!-');
+	$arbiter_explain = chess_sanitize(trim(@$_POST['arbiter_explain']), 'A-Za-z0-9 .,;:=()[]*?!-');
 
-    $orientation = chess_sanitize(@$_POST['orientation']);
+	$orientation = chess_sanitize(@$_POST['orientation']);
 
-    $show_arbiter_ctrl = isset($_POST['show_arbiter_ctrl']);
+	$show_arbiter_ctrl = isset($_POST['show_arbiter_ctrl']);
 
-    $submit_arbitrate = isset($_POST['submit_arbitrate']);
+	$submit_arbitrate = isset($_POST['submit_arbitrate']);
 
-    $arbiter_action = chess_sanitize(@$_POST['arbiter_action']);
+	$arbiter_action = chess_sanitize(@$_POST['arbiter_action']);
 
-    // If form-submit, check security token.
+	// If form-submit, check security token.
 
-    if (($submit_move || $submit_refresh || $submit_arbitrate || $show_arbiter_ctrl) && is_object($GLOBALS['xoopsSecurity']) && !$GLOBALS['xoopsSecurity']->check()) {
-        redirect_header(
-            XOOPS_URL . '/modules/chess/',
-            _CHESS_REDIRECT_DELAY_FAILURE,
-            _MD_CHESS_TOKEN_ERROR . '<br>' . implode('<br>', $GLOBALS['xoopsSecurity']->getErrors())
-        );
-    }
+	if (($submit_move || $submit_refresh || $submit_arbitrate || $show_arbiter_ctrl) && is_object($GLOBALS['xoopsSecurity']) && !$GLOBALS['xoopsSecurity']->check()) {
+		redirect_header(
+			XOOPS_URL . '/modules/chess/',
+			_CHESS_REDIRECT_DELAY_FAILURE,
+			_MD_CHESS_TOKEN_ERROR . '<br>' . implode('<br>', $GLOBALS['xoopsSecurity']->getErrors())
+		);
+	}
 
-    // Fetch the game data from the database.
+	// Fetch the game data from the database.
 
-    $gamedata = chess_get_game($game_id);
+	$gamedata = chess_get_game($game_id);
 
-    if (false === $gamedata) {
-        redirect_header(XOOPS_URL . '/modules/chess/', _CHESS_REDIRECT_DELAY_FAILURE, _MD_CHESS_GAME_NOT_FOUND);
-    }
+	if (false === $gamedata) {
+		redirect_header(XOOPS_URL . '/modules/chess/', _CHESS_REDIRECT_DELAY_FAILURE, _MD_CHESS_GAME_NOT_FOUND);
+	}
 
-    $gamedata_updated = false;
+	$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
-    $delete_game           = 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
+	$delete_game           = false;
 
-    global $xoopsUser;
+	global $xoopsUser;
 
-    $uid = $xoopsUser ? $xoopsUser->getVar('uid') : 0;
+	$uid = $xoopsUser ? $xoopsUser->getVar('uid') : 0;
 
-    $selfplay = $gamedata['white_uid'] == $gamedata['black_uid'];
+	$selfplay = $gamedata['white_uid'] == $gamedata['black_uid'];
 
-    if ($selfplay) {
-        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
-        }
-        // not self-play
-    } else {
-        if ($uid == $gamedata['white_uid']) {
-            $user_color = 'w'; // current user is white
-        } elseif ($uid == $gamedata['black_uid']) {
-            $user_color = 'b'; // current user is black
-        } else {
-            $user_color = '';  // current user is not a player
-        }
-    }
+	if ($selfplay) {
+		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
+		}
+		// not self-play
+	} else {
+		if ($uid == $gamedata['white_uid']) {
+			$user_color = 'w'; // current user is white
+		} elseif ($uid == $gamedata['black_uid']) {
+			$user_color = 'b'; // current user is black
+		} else {
+			$user_color = '';  // current user is not a player
+		}
+	}
 
-    // Determine whether current user is a player in the current game, and whether it's his move.
+	// Determine whether current user is a player in the current game, and whether it's his move.
 
-    $user_is_player = 'w' == $user_color || 'b' == $user_color;
+	$user_is_player = 'w' == $user_color || 'b' == $user_color;
 
-    $user_is_player_to_move = $user_color == $gamedata['fen_active_color'];
+	$user_is_player_to_move = $user_color == $gamedata['fen_active_color'];
 
-    $user_color_name = 'w' == $user_color ? _MD_CHESS_WHITE : _MD_CHESS_BLACK;
+	$user_color_name = 'w' == $user_color ? _MD_CHESS_WHITE : _MD_CHESS_BLACK;
 
-    if ($submit_move && !$gamedata['suspended']) {
-        // If the player has changed his confirm-move preference, it needs to be updated in the database.
+	if ($submit_move && !$gamedata['suspended']) {
+		// If the player has changed his confirm-move preference, it needs to be updated in the database.
 
-        // For a self-play game, the white and black confirm-move preferences are kept the same, to avoid confusion.
+		// For a self-play game, the white and black confirm-move preferences are kept the same, to avoid confusion.
 
-        if ($user_is_player) {
-            if ('w' == $user_color && $gamedata['white_confirm'] != $confirm) {
-                $gamedata['white_confirm'] = $confirm;
+		if ($user_is_player) {
+			if ('w' == $user_color && $gamedata['white_confirm'] != $confirm) {
+				$gamedata['white_confirm'] = $confirm;
 
-                if ($selfplay) {
-                    $gamedata['black_confirm'] = $confirm;
-                }
+				if ($selfplay) {
+					$gamedata['black_confirm'] = $confirm;
+				}
 
-                $gamedata_updated = true;
-            } elseif ('b' == $user_color && $gamedata['black_confirm'] != $confirm) {
-                $gamedata['black_confirm'] = $confirm;
+				$gamedata_updated = true;
+			} elseif ('b' == $user_color && $gamedata['black_confirm'] != $confirm) {
+				$gamedata['black_confirm'] = $confirm;
 
-                if ($selfplay) {
-                    $gamedata['white_confirm'] = $confirm;
-                }
+				if ($selfplay) {
+					$gamedata['white_confirm'] = $confirm;
+				}
 
-                $gamedata_updated = true;
-            }
-        }
+				$gamedata_updated = true;
+			}
+		}
 
-        switch ($movetype) {
-            default:
-            case _CHESS_MOVETYPE_NORMAL:
-                if ($user_is_player_to_move && $gamedata['offer_draw'] != $user_color) {
-                    [$move_performed, $move_result_text] = chess_move($gamedata, $move);
+		switch ($movetype) {
+			default:
+			case _CHESS_MOVETYPE_NORMAL:
+				if ($user_is_player_to_move && $gamedata['offer_draw'] != $user_color) {
+					[$move_performed, $move_result_text] = chess_move($gamedata, $move);
 
-                    if ($move_performed) {
-                        if ($selfplay) { // If self-play, the current user's color switches.
-                            $user_color = $gamedata['fen_active_color'];
-                        }
+					if ($move_performed) {
+						if ($selfplay) { // If self-play, the current user's color switches.
+							$user_color = $gamedata['fen_active_color'];
+						}
 
-                        $gamedata['offer_draw'] = '';
+						$gamedata['offer_draw'] = '';
 
-                        $gamedata_updated = true;
+						$gamedata_updated = true;
 
-                        $notify = "$user_color_name: $move";
-                    }
-                }
-                break;
-            case _CHESS_MOVETYPE_CLAIM_DRAW_3:
-            case _CHESS_MOVETYPE_CLAIM_DRAW_50:
-                if ($user_is_player_to_move && $gamedata['offer_draw'] != $user_color) {
-                    if (!empty($move)) {
-                        [$move_performed, $move_result_text] = chess_move($gamedata, $move);
+						$notify = "$user_color_name: $move";
+					}
+				}
+				break;
+			case _CHESS_MOVETYPE_CLAIM_DRAW_3:
+			case _CHESS_MOVETYPE_CLAIM_DRAW_50:
+				if ($user_is_player_to_move && $gamedata['offer_draw'] != $user_color) {
+					if (!empty($move)) {
+						[$move_performed, $move_result_text] = chess_move($gamedata, $move);
 
-                        #var_dump('chess_game', $move_performed, $move_result_text);#*#DEBUG#
+						#var_dump('chess_game', $move_performed, $move_result_text);#*#DEBUG#
 
-                        if ($move_performed) {
-                            if ($selfplay) { // If self-play, the current user's color switches.
-                                $user_color = $gamedata['fen_active_color'];
-                            }
+						if ($move_performed) {
+							if ($selfplay) { // If self-play, the current user's color switches.
+								$user_color = $gamedata['fen_active_color'];
+							}
 
-                            $gamedata['offer_draw'] = '';
+							$gamedata['offer_draw'] = '';
 
-                            $gamedata_updated = true;
+							$gamedata_updated = true;
 
-                            $notify = "$user_color_name: $move";
-                        } else {
-                            break; // move invalid, so don't bother checking draw-claim
-                        }
-                    }
+							$notify = "$user_color_name: $move";
+						} else {
+							break; // move invalid, so don't bother checking draw-claim
+						}
+					}
 
-                    [$draw_claim_valid, $draw_claim_text] = _CHESS_MOVETYPE_CLAIM_DRAW_3 == $movetype ? chess_is_draw_threefold_repetition($gamedata) : chess_is_draw_50_move_rule($gamedata);
+					[$draw_claim_valid, $draw_claim_text] = _CHESS_MOVETYPE_CLAIM_DRAW_3 == $movetype ? chess_is_draw_threefold_repetition($gamedata) : chess_is_draw_50_move_rule($gamedata);
 
-                    #var_dump('chess_game', $draw_claim_valid, $draw_claim_text);#*#DEBUG#
+					#var_dump('chess_game', $draw_claim_valid, $draw_claim_text);#*#DEBUG#
 
-                    if ($draw_claim_valid) {
-                        $gamedata['offer_draw'] = '';
+					if ($draw_claim_valid) {
+						$gamedata['offer_draw'] = '';
 
-                        $gamedata['pgn_result'] = '1/2-1/2';
+						$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']);
+						$gamedata['pgn_movetext'] = str_replace('*', "{$gamedata['pgn_result']} $comment", $gamedata['pgn_movetext']);
 
-                        $gamedata_updated = true;
+						$gamedata_updated = true;
 
-                        $notify = !empty($move) ? "$user_color_name: $move: $draw_claim_text" : "$user_color_name: $draw_claim_text";
-                    } else {
-                        $draw_claim_error_text = $draw_claim_text;
-                    }
-                }
-                break;
-            case _CHESS_MOVETYPE_RESIGN:
-                if ($user_is_player) {
-                    $gamedata['offer_draw'] = '';
+						$notify = !empty($move) ? "$user_color_name: $move: $draw_claim_text" : "$user_color_name: $draw_claim_text";
+					} else {
+						$draw_claim_error_text = $draw_claim_text;
+					}
+				}
+				break;
+			case _CHESS_MOVETYPE_RESIGN:
+				if ($user_is_player) {
+					$gamedata['offer_draw'] = '';
 
-                    $gamedata['pgn_result'] = 'w' == $user_color ? '0-1' : '1-0';
+					$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['pgn_movetext'] = str_replace('*', "{$gamedata['pgn_result']} $comment", $gamedata['pgn_movetext']);
 
-                    $gamedata_updated = true;
+					$gamedata_updated = true;
 
-                    $notify = "$user_color_name " . _MD_CHESS_RESIGNED;
-                }
-                break;
-            case _CHESS_MOVETYPE_OFFER_DRAW:
-                if ($user_is_player && empty($gamedata['offer_draw']) && !$selfplay) {
-                    $gamedata['offer_draw'] = $user_color;
+					$notify = "$user_color_name " . _MD_CHESS_RESIGNED;
+				}
+				break;
+			case _CHESS_MOVETYPE_OFFER_DRAW:
+				if ($user_is_player && empty($gamedata['offer_draw']) && !$selfplay) {
+					$gamedata['offer_draw'] = $user_color;
 
-                    $gamedata_updated = true;
+					$gamedata_updated = true;
 
-                    $notify = "$user_color_name " . _MD_CHESS_OFFERED_DRAW;
-                }
-                break;
-            case _CHESS_MOVETYPE_ACCEPT_DRAW:
-                if ($user_is_player && !empty($gamedata['offer_draw']) && $gamedata['offer_draw'] != $user_color && !$selfplay) {
-                    $gamedata['offer_draw'] = '';
+					$notify = "$user_color_name " . _MD_CHESS_OFFERED_DRAW;
+				}
+				break;
+			case _CHESS_MOVETYPE_ACCEPT_DRAW:
+				if ($user_is_player && !empty($gamedata['offer_draw']) && $gamedata['offer_draw'] != $user_color && !$selfplay) {
+					$gamedata['offer_draw'] = '';
 
-                    $gamedata['pgn_result'] = '1/2-1/2';
+					$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['pgn_movetext'] = str_replace('*', "{$gamedata['pgn_result']} $comment", $gamedata['pgn_movetext']);
 
-                    $gamedata_updated = true;
+					$gamedata_updated = true;
 
-                    $notify = "$user_color_name " . _MD_CHESS_ACCEPTED_DRAW;
-                }
+					$notify = "$user_color_name " . _MD_CHESS_ACCEPTED_DRAW;
+				}
 
-            // no break
-            case _CHESS_MOVETYPE_REJECT_DRAW:
-                if ($user_is_player && !empty($gamedata['offer_draw']) && $gamedata['offer_draw'] != $user_color && !$selfplay) {
-                    $gamedata['offer_draw'] = '';
+			// no break
+			case _CHESS_MOVETYPE_REJECT_DRAW:
+				if ($user_is_player && !empty($gamedata['offer_draw']) && $gamedata['offer_draw'] != $user_color && !$selfplay) {
+					$gamedata['offer_draw'] = '';
 
-                    $gamedata_updated = true;
+					$gamedata_updated = true;
 
-                    $notify = "$user_color_name " . _MD_CHESS_REJECTED_DRAW;
-                }
-                break;
-            case _CHESS_MOVETYPE_RESTART:
-                if ($user_is_player && $selfplay) {
-                    // instantiate a ChessGame to get a "fresh" gamestate
+					$notify = "$user_color_name " . _MD_CHESS_REJECTED_DRAW;
+				}
+				break;
+			case _CHESS_MOVETYPE_RESTART:
+				if ($user_is_player && $selfplay) {
+					// instantiate a ChessGame to get a "fresh" gamestate
 
-                    $chessgame = new Chess\ChessGame($gamedata['pgn_fen']);
+					$chessgame = new Chess\ChessGame($gamedata['pgn_fen']);
 
-                    $new_gamestate = $chessgame->gamestate();
+					$new_gamestate = $chessgame->gamestate();
 
-                    // update the game data
+					// update the game data
 
-                    $gamedata['fen_piece_placement'] = $new_gamestate['fen_piece_placement'];
+					$gamedata['fen_piece_placement'] = $new_gamestate['fen_piece_placement'];
 
-                    $gamedata['fen_active_color'] = $new_gamestate['fen_active_color'];
+					$gamedata['fen_active_color'] = $new_gamestate['fen_active_color'];
 
-                    $gamedata['fen_castling_availability'] = $new_gamestate['fen_castling_availability'];
+					$gamedata['fen_castling_availability'] = $new_gamestate['fen_castling_availability'];
 
-                    $gamedata['fen_en_passant_target_square'] = $new_gamestate['fen_en_passant_target_square'];
+					$gamedata['fen_en_passant_target_square'] = $new_gamestate['fen_en_passant_target_square'];
 
-                    $gamedata['fen_halfmove_clock'] = $new_gamestate['fen_halfmove_clock'];
+					$gamedata['fen_halfmove_clock'] = $new_gamestate['fen_halfmove_clock'];
 
-                    $gamedata['fen_fullmove_number'] = $new_gamestate['fen_fullmove_number'];
+					$gamedata['fen_fullmove_number'] = $new_gamestate['fen_fullmove_number'];
 
-                    $gamedata['pgn_fen'] = $new_gamestate['pgn_fen'];
+					$gamedata['pgn_fen'] = $new_gamestate['pgn_fen'];
 
-                    $gamedata['pgn_result'] = $new_gamestate['pgn_result'];
+					$gamedata['pgn_result'] = $new_gamestate['pgn_result'];
 
-                    $gamedata['pgn_movetext'] = $new_gamestate['pgn_movetext'];
+					$gamedata['pgn_movetext'] = $new_gamestate['pgn_movetext'];
 
-                    // update user color
+					// update user color
 
-                    $user_color = $gamedata['fen_active_color'];
+					$user_color = $gamedata['fen_active_color'];
 
-                    $gamedata_updated = true;
-                }
-                break;
-            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
-                }
-                break;
-            case _CHESS_MOVETYPE_WANT_ARBITRATION:
-                if ($user_is_player) {
-                    // Ensure that $move_explain does not contain separator $sep.
+					$gamedata_updated = true;
+				}
+				break;
+			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
+				}
+				break;
+			case _CHESS_MOVETYPE_WANT_ARBITRATION:
+				if ($user_is_player) {
+					// Ensure that $move_explain does not contain separator $sep.
 
-                    $sep = '|';
+					$sep = '|';
 
-                    $move_explain = str_replace($sep, '_', $move_explain);
+					$move_explain = str_replace($sep, '_', $move_explain);
 
-                    $gamedata['suspended'] = implode($sep, [date('Y-m-d H:i:s'), $uid, _CHESS_MOVETYPE_WANT_ARBITRATION, $move_explain]);
+					$gamedata['suspended'] = implode($sep, [date('Y-m-d H:i:s'), $uid, _CHESS_MOVETYPE_WANT_ARBITRATION, $move_explain]);
 
-                    $gamedata_updated = true;
+					$gamedata_updated = true;
 
-                    $notify = "$user_color_name " . _MD_CHESS_RQSTED_ARBT . ' ' . _MD_CHESS_BEEN_SUSPENDED;
-                }
-                break;
-        }
-    }
+					$notify = "$user_color_name " . _MD_CHESS_RQSTED_ARBT . ' ' . _MD_CHESS_BEEN_SUSPENDED;
+				}
+				break;
+		}
+	}
 
-    // If board orientation setting uninitialized or invalid, set it to the appropriate default.
+	// If board orientation setting uninitialized or invalid, set it to the appropriate default.
 
-    if (!in_array($orientation, [_CHESS_ORIENTATION_ACTIVE, _CHESS_ORIENTATION_WHITE, _CHESS_ORIENTATION_BLACK])) {
-        if ($user_is_player && 'b' == $user_color) {
-            $orientation = _CHESS_ORIENTATION_BLACK;
-        } else {
-            $orientation = _CHESS_ORIENTATION_WHITE;
-        }
-    }
+	if (!in_array($orientation, [_CHESS_ORIENTATION_ACTIVE, _CHESS_ORIENTATION_WHITE, _CHESS_ORIENTATION_BLACK])) {
+		if ($user_is_player && 'b' == $user_color) {
+			$orientation = _CHESS_ORIENTATION_BLACK;
+		} else {
+			$orientation = _CHESS_ORIENTATION_WHITE;
+		}
+	}
 
-    // Determine if user is a valid arbiter.
+	// Determine if user is a valid arbiter.
 
-    global $xoopsModule;
+	global $xoopsModule;
 
-    $is_arbiter = is_object($xoopsUser) && $xoopsUser->isAdmin($xoopsModule->getVar('mid'));
+	$is_arbiter = is_object($xoopsUser) && $xoopsUser->isAdmin($xoopsModule->getVar('mid'));
 
-    // If arbiter action was submitted, and user is a valid arbiter, process the action.
+	// If arbiter action was submitted, and user is a valid arbiter, process the action.
 
-    if ($submit_arbitrate && $is_arbiter) {
-        $username = $xoopsUser ? $xoopsUser->getVar('uname') : '*unknown*';
+	if ($submit_arbitrate && $is_arbiter) {
+		$username = $xoopsUser ? $xoopsUser->getVar('uname') : '*unknown*';
 
-        switch ($arbiter_action) {
-            case _CHESS_ARBITER_RESUME:
-                if ($gamedata['suspended'] && '*' == $gamedata['pgn_result']) {
-                    $gamedata['suspended'] = '';
+		switch ($arbiter_action) {
+			case _CHESS_ARBITER_RESUME:
+				if ($gamedata['suspended'] && '*' == $gamedata['pgn_result']) {
+					$gamedata['suspended'] = '';
 
-                    $gamedata_updated = true;
+					$gamedata_updated = true;
 
-                    $notify = eval('return \'' . _MD_CHESS_RESUMED_PLAY . '\';'); // eval references $username
-                }
-                break;
-            case _CHESS_ARBITER_DRAW:
-                if ($gamedata['suspended'] && '*' == $gamedata['pgn_result']) {
-                    $gamedata['offer_draw'] = '';
+					$notify = eval('return \'' . _MD_CHESS_RESUMED_PLAY . '\';'); // eval references $username
+				}
+				break;
+			case _CHESS_ARBITER_DRAW:
+				if ($gamedata['suspended'] && '*' == $gamedata['pgn_result']) {
+					$gamedata['offer_draw'] = '';
 
-                    $gamedata['pgn_result'] = '1/2-1/2';
+					$gamedata['pgn_result'] = '1/2-1/2';
 
-                    $arbiter_explain = str_replace('{', '(', $arbiter_explain);
+					$arbiter_explain = str_replace('{', '(', $arbiter_explain);
 
-                    $arbiter_explain = str_replace('}', ')', $arbiter_explain);
+					$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']);
+					$gamedata['pgn_movetext'] = str_replace('*', "{$gamedata['pgn_result']} $comment", $gamedata['pgn_movetext']);
 
-                    $gamedata['suspended'] = '';
+					$gamedata['suspended'] = '';
 
-                    $gamedata_updated = true;
+					$gamedata_updated = true;
 
-                    $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
-                }
-                break;
-            case _CHESS_ARBITER_SUSPEND:
-                if (!$gamedata['suspended'] && '*' == $gamedata['pgn_result']) {
-                    // Ensure that $arbiter_explain does not contain separator $sep.
+					$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
+				}
+				break;
+			case _CHESS_ARBITER_SUSPEND:
+				if (!$gamedata['suspended'] && '*' == $gamedata['pgn_result']) {
+					// Ensure that $arbiter_explain does not contain separator $sep.
 
-                    $sep = '|';
+					$sep = '|';
 
-                    $arbiter_explain = str_replace($sep, '_', $arbiter_explain);
+					$arbiter_explain = str_replace($sep, '_', $arbiter_explain);
 
-                    $gamedata['suspended'] = implode('|', [date('Y-m-d H:i:s'), $uid, _CHESS_MOVETYPE_ARBITER_SUSPEND, $arbiter_explain]);
+					$gamedata['suspended'] = implode('|', [date('Y-m-d H:i:s'), $uid, _CHESS_MOVETYPE_ARBITER_SUSPEND, $arbiter_explain]);
 
-                    $gamedata_updated = true;
+					$gamedata_updated = true;
 
-                    $notify = eval('return \'' . _MD_CHESS_SUSPENDED_PLAY . '\';'); // eval references $username
-                }
-                break;
-            default:
-                break;
-        }
-    }
+					$notify = eval('return \'' . _MD_CHESS_SUSPENDED_PLAY . '\';'); // eval references $username
+				}
+				break;
+			default:
+				break;
+		}
+	}
 
-    // Store the updated game data in the database.
+	// Store the updated game data in the database.
 
-    if ($gamedata_updated) {
-        chess_put_game($game_id, $gamedata);
-    }
+	if ($gamedata_updated) {
+		chess_put_game($game_id, $gamedata);
+	}
 
-    // Display the (possibly updated) board.
+	// Display the (possibly updated) board.
 
-    if (!$delete_game) {
-        chess_show_board($gamedata, $orientation, $user_color, $move_performed, $move_result_text, $draw_claim_error_text, $show_arbiter_ctrl && $is_arbiter);
-    }
+	if (!$delete_game) {
+		chess_show_board($gamedata, $orientation, $user_color, $move_performed, $move_result_text, $draw_claim_error_text, $show_arbiter_ctrl && $is_arbiter);
+	}
 
-    // If a move (or other action) was made, notify any subscribers.
+	// If a move (or other action) was made, notify any subscribers.
 
-    if (!empty($notify)) {
-        $notificationHandler = xoops_getHandler('notification');
+	if (!empty($notify)) {
+		$notificationHandler = xoops_getHandler('notification');
 
-        $notificationHandler->triggerEvent('game', $game_id, 'notify_game_move', ['CHESS_ACTION' => $notify]);
+		$notificationHandler->triggerEvent('game', $game_id, 'notify_game_move', ['CHESS_ACTION' => $notify]);
 
-        // admin notifications
+		// admin notifications
 
-        if (_CHESS_MOVETYPE_WANT_ARBITRATION == $movetype) {
-            $event = 'notify_request_arbitration';
+		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];
+			$extra_tags = ['CHESS_REQUESTOR' => $username, 'CHESS_GAME_ID' => $game_id, 'CHESS_EXPLAIN' => $move_explain];
 
-            $notificationHandler->triggerEvent('global', 0, $event, $extra_tags);
-        }
-    }
+			$notificationHandler->triggerEvent('global', 0, $event, $extra_tags);
+		}
+	}
 
-    // Handle delete-game action.
+	// Handle delete-game action.
 
-    if ($delete_game) {
-        chess_delete_game($game_id);
+	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);
+	}
 }
 
 /**
@@ -472,19 +472,19 @@ 
  */
 function chess_get_game($game_id)
 {
-    global $xoopsDB;
+	global $xoopsDB;
 
-    $games_table = $xoopsDB->prefix('chess_games');
+	$games_table = $xoopsDB->prefix('chess_games');
 
-    $result = $xoopsDB->query("SELECT * FROM $games_table WHERE game_id = '$game_id'");
+	$result = $xoopsDB->query("SELECT * FROM $games_table WHERE game_id = '$game_id'");
 
-    $gamedata = $xoopsDB->fetchArray($result);
+	$gamedata = $xoopsDB->fetchArray($result);
 
-    #var_dump('chess_get_game, gamedata', $gamedata);#*#DEBUG#
+	#var_dump('chess_get_game, gamedata', $gamedata);#*#DEBUG#
 
-    $xoopsDB->freeRecordSet($result);
+	$xoopsDB->freeRecordSet($result);
 
-    return $gamedata;
+	return $gamedata;
 }
 
 /**
@@ -495,21 +495,21 @@ 
  */
 function chess_put_game($game_id, $gamedata)
 {
-    global $xoopsDB;
+	global $xoopsDB;
 
-    $myts = MyTextSanitizer::getInstance();
+	$myts = MyTextSanitizer::getInstance();
 
-    $suspended_q = $myts->addSlashes($gamedata['suspended']);
+	$suspended_q = $myts->addSlashes($gamedata['suspended']);
 
-    $movetext_q = $myts->addSlashes($gamedata['pgn_movetext']);
+	$movetext_q = $myts->addSlashes($gamedata['pgn_movetext']);
 
-    $table = $xoopsDB->prefix('chess_games');
+	$table = $xoopsDB->prefix('chess_games');
 
-    #echo "updating database table $table<br>\n";#*#DEBUG#
+	#echo "updating database table $table<br>\n";#*#DEBUG#
 
-    $xoopsDB->query(
-        trim(
-            "
+	$xoopsDB->query(
+		trim(
+			"
         UPDATE $table
         SET
             start_date                   = '{$gamedata['start_date']}',
@@ -528,18 +528,18 @@ 
             black_confirm                = '{$gamedata['black_confirm']}'
         WHERE  game_id = '$game_id'
     "
-        )
-    );
+		)
+	);
 
-    if ($xoopsDB->errno()) {
-        trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
-    }
+	if ($xoopsDB->errno()) {
+		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';
+	if ('*' != $gamedata['pgn_result'] && '1' == $gamedata['is_rated']) {
+		require_once XOOPS_ROOT_PATH . '/modules/chess/include/ratings.php';
 
-        chess_ratings_adj($game_id);
-    }
+		chess_ratings_adj($game_id);
+	}
 }
 
 /**
@@ -549,19 +549,19 @@ 
  */
 function chess_delete_game($game_id)
 {
-    global $xoopsModule, $xoopsDB;
+	global $xoopsModule, $xoopsDB;
 
-    // delete notifications associated with this game
+	// delete notifications associated with this game
 
-    xoops_notification_deletebyitem($xoopsModule->getVar('mid'), 'game', $game_id);
+	xoops_notification_deletebyitem($xoopsModule->getVar('mid'), 'game', $game_id);
 
-    $table = $xoopsDB->prefix('chess_games');
+	$table = $xoopsDB->prefix('chess_games');
 
-    $xoopsDB->query("DELETE FROM $table WHERE game_id='$game_id'");
+	$xoopsDB->query("DELETE FROM $table WHERE game_id='$game_id'");
 
-    if ($xoopsDB->errno()) {
-        trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
-    }
+	if ($xoopsDB->errno()) {
+		trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
+	}
 }
 
 /**
@@ -575,71 +575,71 @@ 
  */
 function chess_move(&$gamedata, $move)
 {
-    $gamestate = [
-        'fen_piece_placement'          => $gamedata['fen_piece_placement'],
-        'fen_active_color'             => $gamedata['fen_active_color'],
-        'fen_castling_availability'    => $gamedata['fen_castling_availability'],
-        'fen_en_passant_target_square' => $gamedata['fen_en_passant_target_square'],
-        'fen_halfmove_clock'           => $gamedata['fen_halfmove_clock'],
-        'fen_fullmove_number'          => $gamedata['fen_fullmove_number'],
-        'pgn_fen'                      => $gamedata['pgn_fen'],
-        'pgn_result'                   => $gamedata['pgn_result'],
-        'pgn_movetext'                 => $gamedata['pgn_movetext'],
-    ];
+	$gamestate = [
+		'fen_piece_placement'          => $gamedata['fen_piece_placement'],
+		'fen_active_color'             => $gamedata['fen_active_color'],
+		'fen_castling_availability'    => $gamedata['fen_castling_availability'],
+		'fen_en_passant_target_square' => $gamedata['fen_en_passant_target_square'],
+		'fen_halfmove_clock'           => $gamedata['fen_halfmove_clock'],
+		'fen_fullmove_number'          => $gamedata['fen_fullmove_number'],
+		'pgn_fen'                      => $gamedata['pgn_fen'],
+		'pgn_result'                   => $gamedata['pgn_result'],
+		'pgn_movetext'                 => $gamedata['pgn_movetext'],
+	];
 
-    $chessgame = new Chess\ChessGame($gamestate);
+	$chessgame = new Chess\ChessGame($gamestate);
 
-    #echo "Performing move: '$move'<br>\n";#*#DEBUG#
+	#echo "Performing move: '$move'<br>\n";#*#DEBUG#
 
-    [$move_performed, $move_result_text] = $chessgame->move($move);
+	[$move_performed, $move_result_text] = $chessgame->move($move);
 
-    #echo "Move result: '$move_result_text'<br>\n";#*#DEBUG#
+	#echo "Move result: '$move_result_text'<br>\n";#*#DEBUG#
 
-    if ($move_performed) {
-        // The move was valid - update the game data.
+	if ($move_performed) {
+		// The move was valid - update the game data.
 
-        $new_gamestate = $chessgame->gamestate();
+		$new_gamestate = $chessgame->gamestate();
 
-        #var_dump('new_gamestate', $new_gamestate);#*#DEBUG#
+		#var_dump('new_gamestate', $new_gamestate);#*#DEBUG#
 
-        #*#DEBUG# - start
+		#*#DEBUG# - start
 
-        #if ($new_gamestate['fen_castling_availability'] != $gamedata['fen_castling_availability']) {
+		#if ($new_gamestate['fen_castling_availability'] != $gamedata['fen_castling_availability']) {
 
-        #    echo "*** castling_availability changed from '{$gamedata['fen_castling_availability']}' to '{$new_gamestate['fen_castling_availability']}' ***<br>\n";
+		#    echo "*** castling_availability changed from '{$gamedata['fen_castling_availability']}' to '{$new_gamestate['fen_castling_availability']}' ***<br>\n";
 
-        #}
+		#}
 
-        #*#DEBUG# - end
+		#*#DEBUG# - end
 
-        $gamedata['fen_piece_placement'] = $new_gamestate['fen_piece_placement'];
+		$gamedata['fen_piece_placement'] = $new_gamestate['fen_piece_placement'];
 
-        $gamedata['fen_active_color'] = $new_gamestate['fen_active_color'];
+		$gamedata['fen_active_color'] = $new_gamestate['fen_active_color'];
 
-        $gamedata['fen_castling_availability'] = $new_gamestate['fen_castling_availability'];
+		$gamedata['fen_castling_availability'] = $new_gamestate['fen_castling_availability'];
 
-        $gamedata['fen_en_passant_target_square'] = $new_gamestate['fen_en_passant_target_square'];
+		$gamedata['fen_en_passant_target_square'] = $new_gamestate['fen_en_passant_target_square'];
 
-        $gamedata['fen_halfmove_clock'] = $new_gamestate['fen_halfmove_clock'];
+		$gamedata['fen_halfmove_clock'] = $new_gamestate['fen_halfmove_clock'];
 
-        $gamedata['fen_fullmove_number'] = $new_gamestate['fen_fullmove_number'];
+		$gamedata['fen_fullmove_number'] = $new_gamestate['fen_fullmove_number'];
 
-        $gamedata['pgn_fen'] = $new_gamestate['pgn_fen'];
+		$gamedata['pgn_fen'] = $new_gamestate['pgn_fen'];
 
-        $gamedata['pgn_result'] = $new_gamestate['pgn_result'];
+		$gamedata['pgn_result'] = $new_gamestate['pgn_result'];
 
-        $gamedata['pgn_movetext'] = $new_gamestate['pgn_movetext'];
+		$gamedata['pgn_movetext'] = $new_gamestate['pgn_movetext'];
 
-        $gamedata['last_date'] = date('Y-m-d H:i:s');
+		$gamedata['last_date'] = date('Y-m-d H:i:s');
 
-        // if start_date undefined (first move), initialize it
+		// if start_date undefined (first move), initialize it
 
-        if ('0000-00-00 00:00:00' == $gamedata['start_date']) {
-            $gamedata['start_date'] = $gamedata['last_date'];
-        }
-    }
+		if ('0000-00-00 00:00:00' == $gamedata['start_date']) {
+			$gamedata['start_date'] = $gamedata['last_date'];
+		}
+	}
 
-    return [$move_performed, $move_result_text];
+	return [$move_performed, $move_result_text];
 }
 
 /**
@@ -652,19 +652,19 @@ 
  */
 function chess_is_draw_50_move_rule($gamedata)
 {
-    #var_dump('gamedata', $gamedata);#*#DEBUG#
+	#var_dump('gamedata', $gamedata);#*#DEBUG#
 
-    if ($gamedata['fen_halfmove_clock'] >= 100) {
-        $draw_claim_valid = true;
+	if ($gamedata['fen_halfmove_clock'] >= 100) {
+		$draw_claim_valid = true;
 
-        $draw_claim_text = _MD_CHESS_DRAW_50;
-    } else {
-        $draw_claim_valid = false;
+		$draw_claim_text = _MD_CHESS_DRAW_50;
+	} else {
+		$draw_claim_valid = false;
 
-        $draw_claim_text = _MD_CHESS_NO_DRAW_50;
-    }
+		$draw_claim_text = _MD_CHESS_NO_DRAW_50;
+	}
 
-    return [$draw_claim_valid, $draw_claim_text];
+	return [$draw_claim_valid, $draw_claim_text];
 }
 
 /**
@@ -680,41 +680,41 @@ 
  */
 function chess_is_draw_threefold_repetition($gamedata)
 {
-    #var_dump('gamedata', $gamedata);#*#DEBUG#
+	#var_dump('gamedata', $gamedata);#*#DEBUG#
 
-    // Define this constant as true to output a log file containing a move analysis.
+	// Define this constant as true to output a log file containing a move analysis.
 
-    define('CHESS_LOG_3FOLD', false);
+	define('CHESS_LOG_3FOLD', false);
 
-    if (CHESS_LOG_3FOLD) {
-        $log = [];
-    }
+	if (CHESS_LOG_3FOLD) {
+		$log = [];
+	}
 
-    $chessgame = new Chess\ChessGame($gamedata);
+	$chessgame = new Chess\ChessGame($gamedata);
 
-    // board position against which to check for repetitions
+	// board position against which to check for repetitions
 
-    $last_board_state = "{$gamedata['fen_piece_placement']} {$gamedata['fen_active_color']} {$gamedata['fen_castling_availability']} {$gamedata['fen_en_passant_target_square']}";
+	$last_board_state = "{$gamedata['fen_piece_placement']} {$gamedata['fen_active_color']} {$gamedata['fen_castling_availability']} {$gamedata['fen_en_passant_target_square']}";
 
-    $last_move_number = $gamedata['fen_fullmove_number'];
+	$last_move_number = $gamedata['fen_fullmove_number'];
 
-    #echo "last_board_state='$last_board_state'<br>\n";#*#DEBUG#
+	#echo "last_board_state='$last_board_state'<br>\n";#*#DEBUG#
 
-    if (CHESS_LOG_3FOLD) {
-        $log[] = sprintf('%08x %03d%1s %s', crc32($last_board_state), $gamedata['fen_fullmove_number'], $gamedata['fen_active_color'], $last_board_state);
-    }
+	if (CHESS_LOG_3FOLD) {
+		$log[] = sprintf('%08x %03d%1s %s', crc32($last_board_state), $gamedata['fen_fullmove_number'], $gamedata['fen_active_color'], $last_board_state);
+	}
 
-    $chessgame = new Chess\ChessGame($gamedata['pgn_fen']);
+	$chessgame = new Chess\ChessGame($gamedata['pgn_fen']);
 
-    empty($chessgame->error) or trigger_error('chessgame invalid', E_USER_ERROR);
+	empty($chessgame->error) or trigger_error('chessgame invalid', E_USER_ERROR);
 
-    $tmp_gamedata = $chessgame->gamestate();
+	$tmp_gamedata = $chessgame->gamestate();
 
-    is_array($tmp_gamedata) or trigger_error('gamestate invalid', E_USER_ERROR);
+	is_array($tmp_gamedata) or trigger_error('gamestate invalid', E_USER_ERROR);
 
-    #*#DEBUG# - start
+	#*#DEBUG# - start
 
-    /***
+	/***
      * if (function_exists('posix_times')) {
      * $posix_times = posix_times();
      * var_dump('posix_times', $posix_times);
@@ -725,92 +725,92 @@ 
      * }
      ***/
 
-    #*#DEBUG# - end
+	#*#DEBUG# - end
 
-    // $repeats is the list of moves for which the board positions are identical.
+	// $repeats is the list of moves for which the board positions are identical.
 
-    // For example, '6w' would represent the board position immediately before white's sixth move.
+	// For example, '6w' would represent the board position immediately before white's sixth move.
 
-    // The current position is included, since that's the position against which the other positions will be compared.
+	// 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.
+	// Compare initial board position with last board position, unless the move number is the same, meaning that there haven't been any moves.
 
-    #echo "FEN: '{$tmp_gamedata['fen_piece_placement']} {$tmp_gamedata['fen_active_color']} {$tmp_gamedata['fen_castling_availability']} {$tmp_gamedata['fen_en_passant_target_square']} {$tmp_gamedata['fen_halfmove_clock']} {$tmp_gamedata['fen_fullmove_number']}'<br>\n";#*#DEBUG#
+	#echo "FEN: '{$tmp_gamedata['fen_piece_placement']} {$tmp_gamedata['fen_active_color']} {$tmp_gamedata['fen_castling_availability']} {$tmp_gamedata['fen_en_passant_target_square']} {$tmp_gamedata['fen_halfmove_clock']} {$tmp_gamedata['fen_fullmove_number']}'<br>\n";#*#DEBUG#
 
-    $board_state = "{$tmp_gamedata['fen_piece_placement']} {$tmp_gamedata['fen_active_color']} {$tmp_gamedata['fen_castling_availability']} {$tmp_gamedata['fen_en_passant_target_square']}";
+	$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'];
+	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'];
 
-        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);
-        }
+		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);
+		}
 
-        #*#DEBUG# - start
-        /***
+		#*#DEBUG# - start
+		/***
          * if (count($repeats) >= 3) {
          * echo "*** Three repetitions! {$repeats[1]},{$repeats[2]},{$repeats[0]} ***<br>\n";
          * } elseif (count($repeats) >= 2) {
          * echo "*** Two repetitions!  {$repeats[1]},{$repeats[0]} ***<br>\n";
          * }
          ***/
-        #*#DEBUG# - end
-    }
+		#*#DEBUG# - end
+	}
 
-    // Convert pgn_movetext into Nx3 array $movelist.
+	// Convert pgn_movetext into Nx3 array $movelist.
 
-    $movelist = chess_make_movelist($gamedata['pgn_movetext']);
+	$movelist = chess_make_movelist($gamedata['pgn_movetext']);
 
-    #var_dump('movelist', $movelist);#*#DEBUG#
+	#var_dump('movelist', $movelist);#*#DEBUG#
 
-    // Compare board positions after each move with last board position.
+	// Compare board positions after each move with last board position.
 
-    foreach ($movelist as $fullmove) {
-        #echo "'{$fullmove[0]}' '{$fullmove[1]}' '{$fullmove[2]}'<br>\n";#*#DEBUG#
+	foreach ($movelist as $fullmove) {
+		#echo "'{$fullmove[0]}' '{$fullmove[1]}' '{$fullmove[2]}'<br>\n";#*#DEBUG#
 
-        if (CHESS_LOG_3FOLD) {
-            #$log[] = "'{$fullmove[0]}' '{$fullmove[1]}' '{$fullmove[2]}'";#*#LOG_3FOLD# #*#DEBUG#
-        }
+		if (CHESS_LOG_3FOLD) {
+			#$log[] = "'{$fullmove[0]}' '{$fullmove[1]}' '{$fullmove[2]}'";#*#LOG_3FOLD# #*#DEBUG#
+		}
 
-        for ($i = 1; $i <= 2; ++$i) {
-            if (!empty($fullmove[$i])) {
-                $move = $fullmove[$i];
-            } else {
-                continue; // $fullmove[$i] can be empty if last move was white's, or if game was setup with black to move first.
-            }
+		for ($i = 1; $i <= 2; ++$i) {
+			if (!empty($fullmove[$i])) {
+				$move = $fullmove[$i];
+			} else {
+				continue; // $fullmove[$i] can be empty if last move was white's, or if game was setup with black to move first.
+			}
 
-            // Remove check/checkmate annotation, if present.
+			// Remove check/checkmate annotation, if present.
 
-            $move = str_replace('+', '', $move);
+			$move = str_replace('+', '', $move);
 
-            $move = str_replace('#', '', $move);
+			$move = str_replace('#', '', $move);
 
-            #echo "Performing move: '$move'<br>\n";#*#DEBUG#
+			#echo "Performing move: '$move'<br>\n";#*#DEBUG#
 
-            [$tmp_move_performed, $tmp_move_result_text] = $chessgame->move($move);
+			[$tmp_move_performed, $tmp_move_result_text] = $chessgame->move($move);
 
-            #echo "Move result: '$tmp_move_result_text'<br>\n";#*#DEBUG#
+			#echo "Move result: '$tmp_move_result_text'<br>\n";#*#DEBUG#
 
-            $tmp_move_performed or trigger_error("Failed to perform move $move: $tmp_move_result_text", E_USER_ERROR);
+			$tmp_move_performed or trigger_error("Failed to perform move $move: $tmp_move_result_text", E_USER_ERROR);
 
-            $tmp_gamedata = $chessgame->gamestate();
+			$tmp_gamedata = $chessgame->gamestate();
 
-            #echo "FEN: '{$tmp_gamedata['fen_piece_placement']} {$tmp_gamedata['fen_active_color']} {$tmp_gamedata['fen_castling_availability']} {$tmp_gamedata['fen_en_passant_target_square']} {$tmp_gamedata['fen_halfmove_clock']} {$tmp_gamedata['fen_fullmove_number']}'<br>\n";#*#DEBUG#
+			#echo "FEN: '{$tmp_gamedata['fen_piece_placement']} {$tmp_gamedata['fen_active_color']} {$tmp_gamedata['fen_castling_availability']} {$tmp_gamedata['fen_en_passant_target_square']} {$tmp_gamedata['fen_halfmove_clock']} {$tmp_gamedata['fen_fullmove_number']}'<br>\n";#*#DEBUG#
 
-            $board_state = "{$tmp_gamedata['fen_piece_placement']} {$tmp_gamedata['fen_active_color']} {$tmp_gamedata['fen_castling_availability']} {$tmp_gamedata['fen_en_passant_target_square']}";
+			$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 (CHESS_LOG_3FOLD) {
-                $log[] = sprintf('%08x %03d%1s %s', crc32($board_state), $tmp_gamedata['fen_fullmove_number'], $tmp_gamedata['fen_active_color'], $board_state);
-            }
+			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);
+			}
 
-            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'];
+			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'];
 
-                #*#DEBUG# - start
+				#*#DEBUG# - start
 
-                /***
+				/***
                  * if (count($repeats) >= 3) {
                  * echo "*** Three repetitions! {$repeats[1]},{$repeats[2]},{$repeats[0]} ***<br>\n";
                  * } elseif (count($repeats) >= 2) {
@@ -818,18 +818,18 @@ 
                  * }
                  ***/
 
-                #*#DEBUG# - end
+				#*#DEBUG# - end
 
-                if (count($repeats) >= 3) {
-                    break 2;
-                }
-            }
-        }
-    }
+				if (count($repeats) >= 3) {
+					break 2;
+				}
+			}
+		}
+	}
 
-    #*#DEBUG# - start
+	#*#DEBUG# - start
 
-    /***
+	/***
      * if (function_exists('posix_times')) {
      * $posix_times = posix_times();
      * var_dump('posix_times', $posix_times);
@@ -840,27 +840,27 @@ 
      * }
      ***/
 
-    #*#DEBUG# - end
+	#*#DEBUG# - end
 
-    if (count($repeats) >= 3) {
-        $draw_claim_valid = true;
+	if (count($repeats) >= 3) {
+		$draw_claim_valid = true;
 
-        $draw_claim_text = sprintf(_MD_CHESS_DRAW_3, "{$repeats[1]},{$repeats[2]},{$repeats[0]}");
-    } else {
-        $draw_claim_valid = false;
+		$draw_claim_text = sprintf(_MD_CHESS_DRAW_3, "{$repeats[1]},{$repeats[2]},{$repeats[0]}");
+	} else {
+		$draw_claim_valid = false;
 
-        $draw_claim_text = _MD_CHESS_NO_DRAW_3;
-    }
+		$draw_claim_text = _MD_CHESS_NO_DRAW_3;
+	}
 
-    if (CHESS_LOG_3FOLD) {
-        $logfile = XOOPS_ROOT_PATH . '/cache/' . date('Ymd_His') . '_3fold.log';
+	if (CHESS_LOG_3FOLD) {
+		$logfile = XOOPS_ROOT_PATH . '/cache/' . date('Ymd_His') . '_3fold.log';
 
-        sort($log);
+		sort($log);
 
-        error_log(implode("\n", $log), 3, $logfile);
-    }
+		error_log(implode("\n", $log), 3, $logfile);
+	}
 
-    return [$draw_claim_valid, $draw_claim_text];
+	return [$draw_claim_valid, $draw_claim_text];
 }
 
 /**
@@ -871,31 +871,31 @@ 
  */
 function chess_make_movelist($movetext)
 {
-    $movelist = [];
+	$movelist = [];
 
-    $move_tokens = explode(' ', preg_replace('/\{.*\}/', '', $movetext));
+	$move_tokens = explode(' ', preg_replace('/\{.*\}/', '', $movetext));
 
-    $index = -1;
+	$index = -1;
 
-    while ($move_tokens) {
-        $move_token = array_shift($move_tokens);
+	while ($move_tokens) {
+		$move_token = array_shift($move_tokens);
 
-        if (in_array($move_token, ['1-0', '0-1', '1/2-1/2', '*'])) {
-            break;
-        } elseif (preg_match('/^\d+(\.|\.\.\.)$/', $move_token, $matches)) {
-            ++$index;
+		if (in_array($move_token, ['1-0', '0-1', '1/2-1/2', '*'])) {
+			break;
+		} elseif (preg_match('/^\d+(\.|\.\.\.)$/', $move_token, $matches)) {
+			++$index;
 
-            $movelist[$index][] = $move_token;
+			$movelist[$index][] = $move_token;
 
-            if ('...' == $matches[1]) { // setup-game with initial black move - add padding for white's move
-                $movelist[$index][] = '';
-            }
-        } else {
-            $movelist[$index][] = $move_token;
-        }
-    }
+			if ('...' == $matches[1]) { // setup-game with initial black move - add padding for white's move
+				$movelist[$index][] = '';
+			}
+		} else {
+			$movelist[$index][] = $move_token;
+		}
+	}
 
-    return $movelist;
+	return $movelist;
 }
 
 /**
@@ -911,252 +911,252 @@ 
  */
 function chess_show_board($gamedata, $orientation, $user_color, $move_performed, $move_result_text = '', $draw_claim_error_text = '', $show_arbiter_ctrl = false)
 {
-    global $xoopsTpl;
+	global $xoopsTpl;
 
-    $xoopsTpl->assign(
-        'xoops_module_header',
-        '
+	$xoopsTpl->assign(
+		'xoops_module_header',
+		'
         <link rel="stylesheet" type="text/css" media="screen" href="' . XOOPS_URL . '/modules/chess/assets/css/style.css">
     '
-    );
+	);
 
-    $memberHandler = xoops_getHandler('member');
+	$memberHandler = xoops_getHandler('member');
 
-    $white_user = $memberHandler->getUser($gamedata['white_uid']);
+	$white_user = $memberHandler->getUser($gamedata['white_uid']);
 
-    $white_username = is_object($white_user) ? $white_user->getVar('uname') : '?';
+	$white_username = is_object($white_user) ? $white_user->getVar('uname') : '?';
 
-    $black_user = $memberHandler->getUser($gamedata['black_uid']);
+	$black_user = $memberHandler->getUser($gamedata['black_uid']);
 
-    $black_username = is_object($black_user) ? $black_user->getVar('uname') : '?';
+	$black_username = is_object($black_user) ? $black_user->getVar('uname') : '?';
 
-    // Determine whether board is flipped (black at bottom) or "normal" (white at bottom).
+	// Determine whether board is flipped (black at bottom) or "normal" (white at bottom).
 
-    switch ($orientation) {
-        default:
-        case _CHESS_ORIENTATION_ACTIVE:
-            $flip = 'b' == $gamedata['fen_active_color'];
-            break;
-        case _CHESS_ORIENTATION_WHITE:
-            $flip = false;
-            break;
-        case _CHESS_ORIENTATION_BLACK:
-            $flip = true;
-            break;
-    }
+	switch ($orientation) {
+		default:
+		case _CHESS_ORIENTATION_ACTIVE:
+			$flip = 'b' == $gamedata['fen_active_color'];
+			break;
+		case _CHESS_ORIENTATION_WHITE:
+			$flip = false;
+			break;
+		case _CHESS_ORIENTATION_BLACK:
+			$flip = true;
+			break;
+	}
 
-    // Convert fen_piece_placement string into 8x8-array $tiles.
+	// Convert fen_piece_placement string into 8x8-array $tiles.
 
-    $tiles = [];
+	$tiles = [];
 
-    $ranks = explode('/', $gamedata['fen_piece_placement']);
+	$ranks = explode('/', $gamedata['fen_piece_placement']);
 
-    if ($flip) {
-        $ranks = array_reverse($ranks);
-    }
+	if ($flip) {
+		$ranks = array_reverse($ranks);
+	}
 
-    foreach ($ranks as $rank) {
-        $expanded_row = preg_replace_callback(
-            '/(\d)/',
+	foreach ($ranks as $rank) {
+		$expanded_row = preg_replace_callback(
+			'/(\d)/',
 
-            static function ($matches) {
-                return str_repeat('x', $matches[1]);
-            },
-            $rank
-        );
+			static function ($matches) {
+				return str_repeat('x', $matches[1]);
+			},
+			$rank
+		);
 
-        $rank_tiles = preg_split('//', $expanded_row, -1, PREG_SPLIT_NO_EMPTY);
+		$rank_tiles = preg_split('//', $expanded_row, -1, PREG_SPLIT_NO_EMPTY);
 
-        $tiles[] = $flip ? array_reverse($rank_tiles) : $rank_tiles;
-    }
+		$tiles[] = $flip ? array_reverse($rank_tiles) : $rank_tiles;
+	}
 
-    #var_dump('tiles', $tiles);#*#DEBUG#
+	#var_dump('tiles', $tiles);#*#DEBUG#
 
-    // Convert pgn_movetext into Nx3 array $movelist.
+	// Convert pgn_movetext into Nx3 array $movelist.
 
-    $movelist = chess_make_movelist($gamedata['pgn_movetext']);
+	$movelist = chess_make_movelist($gamedata['pgn_movetext']);
 
-    static $file_labels = ['a', 'b', 'c', 'd', 'e', 'f', 'g', 'h'];
+	static $file_labels = ['a', 'b', 'c', 'd', 'e', 'f', 'g', 'h'];
 
-    if ($flip) {
-        $file_labels = array_reverse($file_labels);
-    }
+	if ($flip) {
+		$file_labels = array_reverse($file_labels);
+	}
 
-    $xoopsTpl->assign('chess_gamedata', $gamedata);
+	$xoopsTpl->assign('chess_gamedata', $gamedata);
 
-    // comment at end of movetext
+	// comment at end of movetext
 
-    if (preg_match('/\{(.*?)\}\s*$/', $gamedata['pgn_movetext'], $matches)) {
-        $xoopsTpl->assign('chess_result_comment', $matches[1]);
-    } else {
-        $xoopsTpl->assign('chess_result_comment', '');
-    }
+	if (preg_match('/\{(.*?)\}\s*$/', $gamedata['pgn_movetext'], $matches)) {
+		$xoopsTpl->assign('chess_result_comment', $matches[1]);
+	} else {
+		$xoopsTpl->assign('chess_result_comment', '');
+	}
 
-    $xoopsTpl->assign('chess_create_date', '0000-00-00 00:00:00' != $gamedata['create_date'] ? strtotime($gamedata['create_date']) : 0);
+	$xoopsTpl->assign('chess_create_date', '0000-00-00 00:00:00' != $gamedata['create_date'] ? strtotime($gamedata['create_date']) : 0);
 
-    $xoopsTpl->assign('chess_start_date', '0000-00-00 00:00:00' != $gamedata['start_date'] ? strtotime($gamedata['start_date']) : 0);
+	$xoopsTpl->assign('chess_start_date', '0000-00-00 00:00:00' != $gamedata['start_date'] ? strtotime($gamedata['start_date']) : 0);
 
-    $xoopsTpl->assign('chess_last_date', '0000-00-00 00:00:00' != $gamedata['last_date'] ? strtotime($gamedata['last_date']) : 0);
+	$xoopsTpl->assign('chess_last_date', '0000-00-00 00:00:00' != $gamedata['last_date'] ? strtotime($gamedata['last_date']) : 0);
 
-    $xoopsTpl->assign('chess_white_user', $white_username);
+	$xoopsTpl->assign('chess_white_user', $white_username);
 
-    $xoopsTpl->assign('chess_black_user', $black_username);
+	$xoopsTpl->assign('chess_black_user', $black_username);
 
-    $xoopsTpl->assign('chess_tiles', $tiles);
+	$xoopsTpl->assign('chess_tiles', $tiles);
 
-    $xoopsTpl->assign('chess_file_labels', $file_labels);
+	$xoopsTpl->assign('chess_file_labels', $file_labels);
 
-    $xoopsTpl->assign(
-        'chess_pgn_string',
-        chess_to_pgn_string(
-            [
-                'event'    => '?',
-                'site'     => $_SERVER['SERVER_NAME'],
-                'datetime' => $gamedata['start_date'],
-                'round'    => '?',
-                'white'    => $white_username,
-                'black'    => $black_username,
-                'result'   => $gamedata['pgn_result'],
-                'setup'    => !empty($gamedata['pgn_fen']) ? 1 : 0,
-                'fen'      => $gamedata['pgn_fen'],
-                'movetext' => $gamedata['pgn_movetext'],
-            ]
-        )
-    );
+	$xoopsTpl->assign(
+		'chess_pgn_string',
+		chess_to_pgn_string(
+			[
+				'event'    => '?',
+				'site'     => $_SERVER['SERVER_NAME'],
+				'datetime' => $gamedata['start_date'],
+				'round'    => '?',
+				'white'    => $white_username,
+				'black'    => $black_username,
+				'result'   => $gamedata['pgn_result'],
+				'setup'    => !empty($gamedata['pgn_fen']) ? 1 : 0,
+				'fen'      => $gamedata['pgn_fen'],
+				'movetext' => $gamedata['pgn_movetext'],
+			]
+		)
+	);
 
-    $xoopsTpl->assign('chess_movelist', $movelist);
+	$xoopsTpl->assign('chess_movelist', $movelist);
 
-    $xoopsTpl->assign('chess_date_format', _MEDIUMDATESTRING);
+	$xoopsTpl->assign('chess_date_format', _MEDIUMDATESTRING);
 
-    #if (empty($move_result_text)) {$move_result_text = 'test';}#*#DEBUG#
+	#if (empty($move_result_text)) {$move_result_text = 'test';}#*#DEBUG#
 
-    $xoopsTpl->assign('chess_move_performed', $move_performed);
+	$xoopsTpl->assign('chess_move_performed', $move_performed);
 
-    $xoopsTpl->assign('chess_move_result', $move_result_text);
+	$xoopsTpl->assign('chess_move_result', $move_result_text);
 
-    $xoopsTpl->assign('chess_draw_claim_error_text', $draw_claim_error_text);
+	$xoopsTpl->assign('chess_draw_claim_error_text', $draw_claim_error_text);
 
-    $xoopsTpl->assign('chess_user_color', $user_color);
+	$xoopsTpl->assign('chess_user_color', $user_color);
 
-    $xoopsTpl->assign('chess_confirm', $gamedata['w' == $user_color ? 'white_confirm' : 'black_confirm']);
+	$xoopsTpl->assign('chess_confirm', $gamedata['w' == $user_color ? 'white_confirm' : 'black_confirm']);
 
-    $xoopsTpl->assign('chess_orientation', $orientation);
+	$xoopsTpl->assign('chess_orientation', $orientation);
 
-    $xoopsTpl->assign('chess_rank_start', $flip ? 1 : 8);
+	$xoopsTpl->assign('chess_rank_start', $flip ? 1 : 8);
 
-    $xoopsTpl->assign('chess_rank_direction', $flip ? 'up' : 'down');
+	$xoopsTpl->assign('chess_rank_direction', $flip ? 'up' : 'down');
 
-    $xoopsTpl->assign('chess_file_start', $flip ? 8 : 1);
+	$xoopsTpl->assign('chess_file_start', $flip ? 8 : 1);
 
-    $xoopsTpl->assign('chess_file_direction', $flip ? 'down' : 'up');
+	$xoopsTpl->assign('chess_file_direction', $flip ? 'down' : 'up');
 
-    static $pieces = [
-        'K' => ['color' => 'w', 'name' => 'wking', 'alt' => _MD_CHESS_ALT_WKING],
-        'Q' => ['color' => 'w', 'name' => 'wqueen', 'alt' => _MD_CHESS_ALT_WQUEEN],
-        'R' => ['color' => 'w', 'name' => 'wrook', 'alt' => _MD_CHESS_ALT_WROOK],
-        'B' => ['color' => 'w', 'name' => 'wbishop', 'alt' => _MD_CHESS_ALT_WBISHOP],
-        'N' => ['color' => 'w', 'name' => 'wknight', 'alt' => _MD_CHESS_ALT_WKNIGHT],
-        'P' => ['color' => 'w', 'name' => 'wpawn', 'alt' => _MD_CHESS_ALT_WPAWN],
-        'k' => ['color' => 'b', 'name' => 'bking', 'alt' => _MD_CHESS_ALT_BKING],
-        'q' => ['color' => 'b', 'name' => 'bqueen', 'alt' => _MD_CHESS_ALT_BQUEEN],
-        'r' => ['color' => 'b', 'name' => 'brook', 'alt' => _MD_CHESS_ALT_BROOK],
-        'b' => ['color' => 'b', 'name' => 'bbishop', 'alt' => _MD_CHESS_ALT_BBISHOP],
-        'n' => ['color' => 'b', 'name' => 'bknight', 'alt' => _MD_CHESS_ALT_BKNIGHT],
-        'p' => ['color' => 'b', 'name' => 'bpawn', 'alt' => _MD_CHESS_ALT_BPAWN],
-        'x' => ['color' => 'x', 'name' => 'empty', 'alt' => _MD_CHESS_ALT_EMPTY],
-    ];
+	static $pieces = [
+		'K' => ['color' => 'w', 'name' => 'wking', 'alt' => _MD_CHESS_ALT_WKING],
+		'Q' => ['color' => 'w', 'name' => 'wqueen', 'alt' => _MD_CHESS_ALT_WQUEEN],
+		'R' => ['color' => 'w', 'name' => 'wrook', 'alt' => _MD_CHESS_ALT_WROOK],
+		'B' => ['color' => 'w', 'name' => 'wbishop', 'alt' => _MD_CHESS_ALT_WBISHOP],
+		'N' => ['color' => 'w', 'name' => 'wknight', 'alt' => _MD_CHESS_ALT_WKNIGHT],
+		'P' => ['color' => 'w', 'name' => 'wpawn', 'alt' => _MD_CHESS_ALT_WPAWN],
+		'k' => ['color' => 'b', 'name' => 'bking', 'alt' => _MD_CHESS_ALT_BKING],
+		'q' => ['color' => 'b', 'name' => 'bqueen', 'alt' => _MD_CHESS_ALT_BQUEEN],
+		'r' => ['color' => 'b', 'name' => 'brook', 'alt' => _MD_CHESS_ALT_BROOK],
+		'b' => ['color' => 'b', 'name' => 'bbishop', 'alt' => _MD_CHESS_ALT_BBISHOP],
+		'n' => ['color' => 'b', 'name' => 'bknight', 'alt' => _MD_CHESS_ALT_BKNIGHT],
+		'p' => ['color' => 'b', 'name' => 'bpawn', 'alt' => _MD_CHESS_ALT_BPAWN],
+		'x' => ['color' => 'x', 'name' => 'empty', 'alt' => _MD_CHESS_ALT_EMPTY],
+	];
 
-    $xoopsTpl->assign('chess_pieces', $pieces);
+	$xoopsTpl->assign('chess_pieces', $pieces);
 
-    $xoopsTpl->assign('chess_show_arbitration_controls', $show_arbiter_ctrl);
+	$xoopsTpl->assign('chess_show_arbitration_controls', $show_arbiter_ctrl);
 
-    if ($show_arbiter_ctrl && $gamedata['suspended']) {
-        [$susp_date, $susp_uid, $susp_type, $susp_explain] = explode('|', $gamedata['suspended']);
+	if ($show_arbiter_ctrl && $gamedata['suspended']) {
+		[$susp_date, $susp_uid, $susp_type, $susp_explain] = explode('|', $gamedata['suspended']);
 
-        switch ($susp_type) {
-            case 'arbiter_suspend':
-                $susp_type_display = _MD_CHESS_SUSP_TYPE_ARBITER;
-                break;
-            case 'want_arbitration':
-                $susp_type_display = _MD_CHESS_SUSP_TYPE_PLAYER;
-                break;
-            default:
-                $susp_type_display = _MD_CHESS_LABEL_ERROR;
-                break;
-        }
+		switch ($susp_type) {
+			case 'arbiter_suspend':
+				$susp_type_display = _MD_CHESS_SUSP_TYPE_ARBITER;
+				break;
+			case 'want_arbitration':
+				$susp_type_display = _MD_CHESS_SUSP_TYPE_PLAYER;
+				break;
+			default:
+				$susp_type_display = _MD_CHESS_LABEL_ERROR;
+				break;
+		}
 
-        $susp_user = $memberHandler->getUser($susp_uid);
+		$susp_user = $memberHandler->getUser($susp_uid);
 
-        $susp_username = is_object($susp_user) ? $susp_user->getVar('uname') : _MD_CHESS_UNKNOWN;
+		$susp_username = is_object($susp_user) ? $susp_user->getVar('uname') : _MD_CHESS_UNKNOWN;
 
-        $suspend_info = [
-            'date'   => strtotime($susp_date),
-            'user'   => $susp_username,
-            'type'   => $susp_type_display,
-            'reason' => $susp_explain,
-        ];
+		$suspend_info = [
+			'date'   => strtotime($susp_date),
+			'user'   => $susp_username,
+			'type'   => $susp_type_display,
+			'reason' => $susp_explain,
+		];
 
-        $xoopsTpl->assign('chess_suspend_info', $suspend_info);
-    }
+		$xoopsTpl->assign('chess_suspend_info', $suspend_info);
+	}
 
-    // Initialize $captured_pieces_all to indicate all pieces captured, then subtract off the pieces remaining on the board.
+	// Initialize $captured_pieces_all to indicate all pieces captured, then subtract off the pieces remaining on the board.
 
-    $captured_pieces_all = [
-        'Q' => 1,
-        'R' => 2,
-        'B' => 2,
-        'N' => 2,
-        'P' => 8,
-        'q' => 1,
-        'r' => 2,
-        'b' => 2,
-        'n' => 2,
-        'p' => 8,
-    ];
+	$captured_pieces_all = [
+		'Q' => 1,
+		'R' => 2,
+		'B' => 2,
+		'N' => 2,
+		'P' => 8,
+		'q' => 1,
+		'r' => 2,
+		'b' => 2,
+		'n' => 2,
+		'p' => 8,
+	];
 
-    for ($i = 0, $iMax = mb_strlen($gamedata['fen_piece_placement']); $i < $iMax; ++$i) {
-        $piece = $gamedata['fen_piece_placement'][$i];
+	for ($i = 0, $iMax = mb_strlen($gamedata['fen_piece_placement']); $i < $iMax; ++$i) {
+		$piece = $gamedata['fen_piece_placement'][$i];
 
-        if (!empty($captured_pieces_all[$piece])) {
-            --$captured_pieces_all[$piece];
-        }
-    }
+		if (!empty($captured_pieces_all[$piece])) {
+			--$captured_pieces_all[$piece];
+		}
+	}
 
-    // Construct lists of white's and black's captured pieces.
+	// Construct lists of white's and black's captured pieces.
 
-    $captured_pieces = ['white' => [], 'black' => []];
+	$captured_pieces = ['white' => [], 'black' => []];
 
-    foreach ($captured_pieces_all as $piece => $count) {
-        if ($count > 0) {
-            if (ctype_upper($piece)) {
-                $captured_pieces['white'] = array_merge($captured_pieces['white'], array_pad([], $count, $piece));
-            } elseif (ctype_lower($piece)) {
-                $captured_pieces['black'] = array_merge($captured_pieces['black'], array_pad([], $count, $piece));
-            }
-        }
-    }
+	foreach ($captured_pieces_all as $piece => $count) {
+		if ($count > 0) {
+			if (ctype_upper($piece)) {
+				$captured_pieces['white'] = array_merge($captured_pieces['white'], array_pad([], $count, $piece));
+			} elseif (ctype_lower($piece)) {
+				$captured_pieces['black'] = array_merge($captured_pieces['black'], array_pad([], $count, $piece));
+			}
+		}
+	}
 
-    #var_dump('captured_pieces_all', $captured_pieces_all);#*#DEBUG#
+	#var_dump('captured_pieces_all', $captured_pieces_all);#*#DEBUG#
 
-    #var_dump('captured_pieces', $captured_pieces);#*#DEBUG#
+	#var_dump('captured_pieces', $captured_pieces);#*#DEBUG#
 
-    $xoopsTpl->assign('chess_captured_pieces', $captured_pieces);
+	$xoopsTpl->assign('chess_captured_pieces', $captured_pieces);
 
-    $xoopsTpl->assign('chess_pawn_promote_choices', 'w' == $gamedata['fen_active_color'] ? ['Q', 'R', 'B', 'N'] : ['q', 'r', 'b', 'n']);
+	$xoopsTpl->assign('chess_pawn_promote_choices', 'w' == $gamedata['fen_active_color'] ? ['Q', 'R', 'B', 'N'] : ['q', 'r', 'b', 'n']);
 
-    $xoopsTpl->assign('chess_allow_delete', chess_can_delete());
+	$xoopsTpl->assign('chess_allow_delete', chess_can_delete());
 
-    // popup window contents for selecting piece to which pawn is promoted
+	// popup window contents for selecting piece to which pawn is promoted
 
-    // (Note that template is compiled here by fetch(), so any template variables it uses must already be defined.)
+	// (Note that template is compiled here by fetch(), so any template variables it uses must already be defined.)
 
-    $xoopsTpl->assign('chess_pawn_promote_popup', $user_color ? $xoopsTpl->fetch('db:chess_game_promote_popup.tpl') : '');
+	$xoopsTpl->assign('chess_pawn_promote_popup', $user_color ? $xoopsTpl->fetch('db:chess_game_promote_popup.tpl') : '');
 
-    $xoopsTpl->assign('chess_ratings_enabled', 'none' != chess_moduleConfig('rating_system'));
+	$xoopsTpl->assign('chess_ratings_enabled', 'none' != chess_moduleConfig('rating_system'));
 
-    // security token
+	// security token
 
-    $xoopsTpl->assign('chess_xoops_request_token', is_object($GLOBALS['xoopsSecurity']) ? $GLOBALS['xoopsSecurity']->getTokenHTML() : '');
+	$xoopsTpl->assign('chess_xoops_request_token', is_object($GLOBALS['xoopsSecurity']) ? $GLOBALS['xoopsSecurity']->getTokenHTML() : '');
 }
 
 ?>

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

Indentation +75 added lines, -75 removed lines

@@ -47,7 +47,7 @@ 
 
 // 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
@@ -61,11 +61,11 @@ 
 
 // 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',
-        _CHESS_REDIRECT_DELAY_FAILURE,
-        _MD_CHESS_TOKEN_ERROR . '<br>' . implode('<br>', $GLOBALS['xoopsSecurity']->getErrors())
-    );
+	redirect_header(
+		XOOPS_URL . '/modules/chess/ratings.php',
+		_CHESS_REDIRECT_DELAY_FAILURE,
+		_MD_CHESS_TOKEN_ERROR . '<br>' . implode('<br>', $GLOBALS['xoopsSecurity']->getErrors())
+	);
 }
 
 $msg       = '';
@@ -73,17 +73,17 @@ 
 
 // If arbiter requested recalculation of ratings, do it.
 if ($submit_recalc_ratings && is_object($xoopsUser) && $xoopsUser->isAdmin($xoopsModule->getVar('mid'))) {
-    if ($confirm_recalc_ratings) {
-        chess_recalc_ratings();
+	if ($confirm_recalc_ratings) {
+		chess_recalc_ratings();
 
-        $msg = _MD_CHESS_RECALC_DONE;
+		$msg = _MD_CHESS_RECALC_DONE;
 
-        $msg_class = 'resultMsg';
-    } else {
-        $msg = _MD_CHESS_RECALC_NOT_DONE;
+		$msg_class = 'resultMsg';
+	} else {
+		$msg = _MD_CHESS_RECALC_NOT_DONE;
 
-        $msg_class = 'errorMsg';
-    }
+		$msg_class = 'errorMsg';
+	}
 }
 
 // Display the ratings.
@@ -101,119 +101,119 @@ 
  */
 function chess_ratings($start = 0, $msg = '', $msg_class = 'errorMsg')
 {
-    global $xoopsDB, $xoopsUser, $xoopsModule, $xoopsTpl;
+	global $xoopsDB, $xoopsUser, $xoopsModule, $xoopsTpl;
 
-    // Display ratings.
+	// Display ratings.
 
-    // Get maximum number of items to display on a page, and constrain it to a reasonable value.
+	// Get maximum number of items to display on a page, and constrain it to a reasonable value.
 
-    $max_items_to_display = chess_moduleConfig('max_items');
+	$max_items_to_display = chess_moduleConfig('max_items');
 
-    $max_items_to_display = min(max($max_items_to_display, 1), 1000);
+	$max_items_to_display = min(max($max_items_to_display, 1), 1000);
 
-    $games_table = $xoopsDB->prefix('chess_games');
+	$games_table = $xoopsDB->prefix('chess_games');
 
-    $ratings_table = $xoopsDB->prefix('chess_ratings');
+	$ratings_table = $xoopsDB->prefix('chess_ratings');
 
-    // Two queries are performed, one without a limit clause to count the total number of rows for the page navigator,
+	// Two queries are performed, one without a limit clause to count the total number of rows for the page navigator,
 
-    // and one with a limit clause to get the data for display on the current page.
+	// and one with a limit clause to get the data for display on the current page.
 
-    // SQL_CALC_FOUND_ROWS and FOUND_ROWS(), available in MySQL 4.0.0, provide a more efficient way of doing this.
+	// SQL_CALC_FOUND_ROWS and FOUND_ROWS(), available in MySQL 4.0.0, provide a more efficient way of doing this.
 
-    $result = $xoopsDB->query(
-        "
+	$result = $xoopsDB->query(
+		"
         SELECT    COUNT(*)
         FROM      $ratings_table AS p
     "
-    );
+	);
 
-    [$num_items] = $xoopsDB->fetchRow($result);
+	[$num_items] = $xoopsDB->fetchRow($result);
 
-    $xoopsDB->freeRecordSet($result);
+	$xoopsDB->freeRecordSet($result);
 
-    $pagenav = new XoopsPageNav($num_items, $max_items_to_display, $start, 'start');
+	$pagenav = new XoopsPageNav($num_items, $max_items_to_display, $start, 'start');
 
-    $result = $xoopsDB->query(
-        "
+	$result = $xoopsDB->query(
+		"
         SELECT    player_uid, rating, (games_won+games_lost+games_drawn) AS games_played
         FROM      $ratings_table
         ORDER BY  rating DESC, player_uid ASC
         LIMIT     $start, $max_items_to_display
     "
-    );
+	);
 
-    // user IDs that will require mapping to usernames
+	// user IDs that will require mapping to usernames
 
-    $userids = [];
+	$userids = [];
 
-    $players = [];
+	$players = [];
 
-    while (false !== ($row = $xoopsDB->fetchArray($result))) {
-        // save user IDs that will require mapping to usernames
+	while (false !== ($row = $xoopsDB->fetchArray($result))) {
+		// save user IDs that will require mapping to usernames
 
-        $userids[] = $row['player_uid'];
+		$userids[] = $row['player_uid'];
 
-        $players[] = [
-            'player_uid'   => $row['player_uid'],
-            'rating'       => $row['rating'],
-            'games_played' => $row['games_played'],
-        ];
-    }
+		$players[] = [
+			'player_uid'   => $row['player_uid'],
+			'rating'       => $row['rating'],
+			'games_played' => $row['games_played'],
+		];
+	}
 
-    $xoopsDB->freeRecordSet($result);
+	$xoopsDB->freeRecordSet($result);
 
-    // get mapping of user IDs to usernames
+	// get mapping of user IDs to usernames
 
-    $memberHandler = xoops_getHandler('member');
+	$memberHandler = xoops_getHandler('member');
 
-    $criteria = new \Criteria('uid', '(' . implode(',', $userids) . ')', 'IN');
+	$criteria = new \Criteria('uid', '(' . implode(',', $userids) . ')', 'IN');
 
-    $usernames = $memberHandler->getUserList($criteria);
+	$usernames = $memberHandler->getUserList($criteria);
 
-    // add usernames to $players
+	// add usernames to $players
 
-    foreach ($players as $k => $player) {
-        $players[$k]['player_uname'] = $usernames[$player['player_uid']] ?? '?';
-    }
+	foreach ($players as $k => $player) {
+		$players[$k]['player_uname'] = $usernames[$player['player_uid']] ?? '?';
+	}
 
-    // Display form for arbiter to recalculate ratings.
+	// Display form for arbiter to recalculate ratings.
 
-    if (is_object($xoopsUser) && $xoopsUser->isAdmin($xoopsModule->getVar('mid'))) {
-        // security token added below
+	if (is_object($xoopsUser) && $xoopsUser->isAdmin($xoopsModule->getVar('mid'))) {
+		// security token added below
 
-        $form = new XoopsThemeForm(_MD_CHESS_RECALC_RATINGS, 'form1', 'ratings.php');
+		$form = new XoopsThemeForm(_MD_CHESS_RECALC_RATINGS, 'form1', 'ratings.php');
 
-        // checkbox (initially unchecked)
+		// checkbox (initially unchecked)
 
-        $checked_value = 1;
+		$checked_value = 1;
 
-        $checkbox_confirm_recalc_ratings = new XoopsFormCheckBox('', 'confirm_recalc_ratings', !$checked_value);
+		$checkbox_confirm_recalc_ratings = new XoopsFormCheckBox('', 'confirm_recalc_ratings', !$checked_value);
 
-        $checkbox_confirm_recalc_ratings->addOption($checked_value, _MD_CHESS_RECALC_CONFIRM);
+		$checkbox_confirm_recalc_ratings->addOption($checked_value, _MD_CHESS_RECALC_CONFIRM);
 
-        $form->addElement($checkbox_confirm_recalc_ratings);
+		$form->addElement($checkbox_confirm_recalc_ratings);
 
-        $form->addElement(new XoopsFormButton('', 'submit_recalc_ratings', _MD_CHESS_SUBMIT_BUTTON, 'submit'));
+		$form->addElement(new XoopsFormButton('', 'submit_recalc_ratings', _MD_CHESS_SUBMIT_BUTTON, 'submit'));
 
-        $form->assign($xoopsTpl);
+		$form->assign($xoopsTpl);
 
-        // security token
+		// security token
 
-        // This method is used because form is templated.
+		// This method is used because form is templated.
 
-        $xoopsTpl->assign('chess_xoops_request_token', is_object($GLOBALS['xoopsSecurity']) ? $GLOBALS['xoopsSecurity']->getTokenHTML() : '');
-    }
+		$xoopsTpl->assign('chess_xoops_request_token', is_object($GLOBALS['xoopsSecurity']) ? $GLOBALS['xoopsSecurity']->getTokenHTML() : '');
+	}
 
-    // Template variables
+	// Template variables
 
-    $xoopsTpl->assign('chess_provisional_games', chess_ratings_num_provisional_games());
+	$xoopsTpl->assign('chess_provisional_games', chess_ratings_num_provisional_games());
 
-    $xoopsTpl->assign('chess_msg', $msg);
+	$xoopsTpl->assign('chess_msg', $msg);
 
-    $xoopsTpl->assign('chess_msg_class', $msg_class);
+	$xoopsTpl->assign('chess_msg_class', $msg_class);
 
-    $xoopsTpl->assign('chess_players_pagenav', $pagenav->renderNav());
+	$xoopsTpl->assign('chess_players_pagenav', $pagenav->renderNav());
 
-    $xoopsTpl->assign('chess_players', $players);
+	$xoopsTpl->assign('chess_players', $players);
 }

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);