mambax7 / chess

player_stats.php

Indentation +169 added lines, -169 removed lines

@@ -57,19 +57,19 @@ 
 
 // If player username provided, map it to a user ID, overriding any provided value of player user ID.
 if (!empty($player_uname)) {
-    $player_uid = chess_uname_to_uid($player_uname);
-    // Otherwise, if player user ID provided, map it to a username.
+	$player_uid = chess_uname_to_uid($player_uname);
+	// Otherwise, if player user ID provided, map it to a username.
 } elseif (0 != $player_uid) {
-    $memberHandler = xoops_getHandler('member');
+	$memberHandler = xoops_getHandler('member');
 
-    $player_user = $memberHandler->getUser($player_uid);
+	$player_user = $memberHandler->getUser($player_uid);
 
-    $player_uname = is_object($player_user) ? $player_user->getVar('uname') : '';
+	$player_uname = is_object($player_user) ? $player_user->getVar('uname') : '';
 }
 
 // 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
@@ -89,96 +89,96 @@ 
  */
 function chess_player_stats($player_uid, $player_uname, $show_option = _CHESS_SHOW_EXCEPT_SELFPLAY, $cstart = 0, $gstart = 0)
 {
-    global $xoopsDB, $xoopsTpl;
+	global $xoopsDB, $xoopsTpl;
 
-    $rating_system = chess_moduleConfig('rating_system');
+	$rating_system = chess_moduleConfig('rating_system');
 
-    $num_provisional_games = chess_ratings_num_provisional_games();
+	$num_provisional_games = chess_ratings_num_provisional_games();
 
-    // set show_option to default if appropriate
+	// set show_option to default if appropriate
 
-    if (!$show_option || ('none' == $rating_system && _CHESS_SHOW_RATED_ONLY == $show_option)) {
-        $show_option = _CHESS_SHOW_EXCEPT_SELFPLAY;
-    }
+	if (!$show_option || ('none' == $rating_system && _CHESS_SHOW_RATED_ONLY == $show_option)) {
+		$show_option = _CHESS_SHOW_EXCEPT_SELFPLAY;
+	}
 
-    // 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);
 
-    $challenges_table = $xoopsDB->prefix('chess_challenges');
+	$challenges_table = $xoopsDB->prefix('chess_challenges');
 
-    $games_table = $xoopsDB->prefix('chess_games');
+	$games_table = $xoopsDB->prefix('chess_games');
 
-    $ratings_table = $xoopsDB->prefix('chess_ratings');
+	$ratings_table = $xoopsDB->prefix('chess_ratings');
 
-    $player = [];
+	$player = [];
 
-    $player['uid'] = $player_uid;
+	$player['uid'] = $player_uid;
 
-    $player['uname'] = $player_uname;
+	$player['uname'] = $player_uname;
 
-    // ---------------------------------------------
+	// ---------------------------------------------
 
-    // form for selecting player and display-options
+	// form for selecting player and display-options
 
-    // ---------------------------------------------
+	// ---------------------------------------------
 
-    // security token not needed for this form
+	// security token not needed for this form
 
-    $form = new XoopsThemeForm(_MD_CHESS_SELECT_PLAYER, 'form1', 'player_stats.php');
+	$form = new XoopsThemeForm(_MD_CHESS_SELECT_PLAYER, 'form1', 'player_stats.php');
 
-    $form->addElement(new XoopsFormText('', 'player_uname', 25, 50, $player_uname));
+	$form->addElement(new XoopsFormText('', 'player_uname', 25, 50, $player_uname));
 
-    $form->addElement(new XoopsFormButton('', 'submit_select_player', _MD_CHESS_SUBMIT_BUTTON, 'submit'));
+	$form->addElement(new XoopsFormButton('', 'submit_select_player', _MD_CHESS_SUBMIT_BUTTON, 'submit'));
 
-    $menu_show_option = new XoopsFormSelect('', 'show_option', $show_option, 1, false);
+	$menu_show_option = new XoopsFormSelect('', 'show_option', $show_option, 1, false);
 
-    $menu_show_option->addOption(_CHESS_SHOW_ALL_GAMES, _MD_CHESS_SHOW_ALL_GAMES);
+	$menu_show_option->addOption(_CHESS_SHOW_ALL_GAMES, _MD_CHESS_SHOW_ALL_GAMES);
 
-    $menu_show_option->addOption(_CHESS_SHOW_EXCEPT_SELFPLAY, _MD_CHESS_SHOW_EXCEPT_SELFPLAY); // default
+	$menu_show_option->addOption(_CHESS_SHOW_EXCEPT_SELFPLAY, _MD_CHESS_SHOW_EXCEPT_SELFPLAY); // default
 
-    if ('none' != $rating_system) {
-        $menu_show_option->addOption(_CHESS_SHOW_RATED_ONLY, _MD_CHESS_SHOW_RATED_ONLY);
-    }
+	if ('none' != $rating_system) {
+		$menu_show_option->addOption(_CHESS_SHOW_RATED_ONLY, _MD_CHESS_SHOW_RATED_ONLY);
+	}
 
-    $form->addElement($menu_show_option);
+	$form->addElement($menu_show_option);
 
-    $form->assign($xoopsTpl);
+	$form->assign($xoopsTpl);
 
-    // user IDs that will require mapping to usernames
+	// user IDs that will require mapping to usernames
 
-    $userids = [];
+	$userids = [];
 
-    // --------------
+	// --------------
 
-    // player's games
+	// player's games
 
-    // --------------
+	// --------------
 
-    // 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.
 
-    $where = "'$player_uid' IN (white_uid, black_uid)";
+	$where = "'$player_uid' IN (white_uid, black_uid)";
 
-    if (_CHESS_SHOW_EXCEPT_SELFPLAY == $show_option) {
-        $where .= ' AND white_uid != black_uid';
-    } elseif (_CHESS_SHOW_RATED_ONLY == $show_option) {
-        $where .= ' AND is_rated = "1" AND white_uid != black_uid';
-    }
+	if (_CHESS_SHOW_EXCEPT_SELFPLAY == $show_option) {
+		$where .= ' AND white_uid != black_uid';
+	} elseif (_CHESS_SHOW_RATED_ONLY == $show_option) {
+		$where .= ' AND is_rated = "1" AND white_uid != black_uid';
+	}
 
-    $result = $xoopsDB->query("SELECT COUNT(*) FROM $games_table WHERE $where");
+	$result = $xoopsDB->query("SELECT COUNT(*) FROM $games_table WHERE $where");
 
-    [$num_items] = $xoopsDB->fetchRow($result);
+	[$num_items] = $xoopsDB->fetchRow($result);
 
-    $xoopsDB->freeRecordSet($result);
+	$xoopsDB->freeRecordSet($result);
 
-    $result = $xoopsDB->query(
-        "
+	$result = $xoopsDB->query(
+		"
         SELECT   game_id, fen_active_color, white_uid, black_uid, pgn_result, is_rated,
             UNIX_TIMESTAMP(GREATEST(create_date,start_date,last_date)) AS last_activity
         FROM      $games_table
@@ -186,191 +186,191 @@ 
         ORDER BY  last_activity DESC
         LIMIT     $gstart, $max_items_to_display
     "
-    );
+	);
 
-    $games = [];
+	$games = [];
 
-    while (false !== ($row = $xoopsDB->fetchArray($result))) {
-        $games[] = [
-            'game_id'          => $row['game_id'],
-            'white_uid'        => $row['white_uid'],
-            'black_uid'        => $row['black_uid'],
-            'fen_active_color' => $row['fen_active_color'],
-            'pgn_result'       => $row['pgn_result'],
-            'last_activity'    => $row['last_activity'],
-            'is_rated'         => $row['is_rated'],
-        ];
+	while (false !== ($row = $xoopsDB->fetchArray($result))) {
+		$games[] = [
+			'game_id'          => $row['game_id'],
+			'white_uid'        => $row['white_uid'],
+			'black_uid'        => $row['black_uid'],
+			'fen_active_color' => $row['fen_active_color'],
+			'pgn_result'       => $row['pgn_result'],
+			'last_activity'    => $row['last_activity'],
+			'is_rated'         => $row['is_rated'],
+		];
 
-        // save user IDs that will require mapping to usernames
+		// save user IDs that will require mapping to usernames
 
-        if ($row['white_uid']) {
-            $userids[$row['white_uid']] = 1;
-        }
+		if ($row['white_uid']) {
+			$userids[$row['white_uid']] = 1;
+		}
 
-        if ($row['black_uid']) {
-            $userids[$row['black_uid']] = 1;
-        }
-    }
+		if ($row['black_uid']) {
+			$userids[$row['black_uid']] = 1;
+		}
+	}
 
-    $xoopsDB->freeRecordSet($result);
+	$xoopsDB->freeRecordSet($result);
 
-    $show_option_urlparam = "&show_option=$show_option";
+	$show_option_urlparam = "&show_option=$show_option";
 
-    $games_pagenav = new XoopsPageNav($num_items, $max_items_to_display, $gstart, 'gstart', "player_uid=$player_uid$show_option_urlparam");
+	$games_pagenav = new XoopsPageNav($num_items, $max_items_to_display, $gstart, 'gstart', "player_uid=$player_uid$show_option_urlparam");
 
-    // -------------------
+	// -------------------
 
-    // player's challenges
+	// player's challenges
 
-    // -------------------
+	// -------------------
 
-    // 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.
 
-    $where = "'$player_uid' IN (player1_uid, player2_uid)";
+	$where = "'$player_uid' IN (player1_uid, player2_uid)";
 
-    if (_CHESS_SHOW_RATED_ONLY == $show_option) {
-        $where .= ' AND is_rated = "1"';
-    }
+	if (_CHESS_SHOW_RATED_ONLY == $show_option) {
+		$where .= ' AND is_rated = "1"';
+	}
 
-    $result = $xoopsDB->query("SELECT COUNT(*) FROM $challenges_table WHERE $where");
+	$result = $xoopsDB->query("SELECT COUNT(*) FROM $challenges_table WHERE $where");
 
-    [$num_items] = $xoopsDB->fetchRow($result);
+	[$num_items] = $xoopsDB->fetchRow($result);
 
-    $xoopsDB->freeRecordSet($result);
+	$xoopsDB->freeRecordSet($result);
 
-    $result = $xoopsDB->query(
-        "
+	$result = $xoopsDB->query(
+		"
         SELECT   challenge_id, game_type, color_option, player1_uid, player2_uid, UNIX_TIMESTAMP(create_date) AS create_date, is_rated
         FROM     $challenges_table
         WHERE    $where
         ORDER BY create_date DESC
         LIMIT    $cstart, $max_items_to_display
     "
-    );
+	);
 
-    $challenges = [];
+	$challenges = [];
 
-    while (false !== ($row = $xoopsDB->fetchArray($result))) {
-        $challenges[] = [
-            'challenge_id' => $row['challenge_id'],
-            'game_type'    => $row['game_type'],
-            'color_option' => $row['color_option'],
-            'player1_uid'  => $row['player1_uid'],
-            'player2_uid'  => $row['player2_uid'],
-            'create_date'  => $row['create_date'],
-            'is_rated'     => $row['is_rated'],
-        ];
+	while (false !== ($row = $xoopsDB->fetchArray($result))) {
+		$challenges[] = [
+			'challenge_id' => $row['challenge_id'],
+			'game_type'    => $row['game_type'],
+			'color_option' => $row['color_option'],
+			'player1_uid'  => $row['player1_uid'],
+			'player2_uid'  => $row['player2_uid'],
+			'create_date'  => $row['create_date'],
+			'is_rated'     => $row['is_rated'],
+		];
 
-        // save user IDs that will require mapping to usernames
+		// save user IDs that will require mapping to usernames
 
-        if ($row['player1_uid']) {
-            $userids[$row['player1_uid']] = 1;
-        }
+		if ($row['player1_uid']) {
+			$userids[$row['player1_uid']] = 1;
+		}
 
-        if ($row['player2_uid']) {
-            $userids[$row['player2_uid']] = 1;
-        }
-    }
+		if ($row['player2_uid']) {
+			$userids[$row['player2_uid']] = 1;
+		}
+	}
 
-    $xoopsDB->freeRecordSet($result);
+	$xoopsDB->freeRecordSet($result);
 
-    $show_option_urlparam = "&show_option=$show_option";
+	$show_option_urlparam = "&show_option=$show_option";
 
-    $challenges_pagenav = new XoopsPageNav($num_items, $max_items_to_display, $cstart, 'cstart', "player_uid=$player_uid$show_option_urlparam");
+	$challenges_pagenav = new XoopsPageNav($num_items, $max_items_to_display, $cstart, 'cstart', "player_uid=$player_uid$show_option_urlparam");
 
-    // ---------
+	// ---------
 
-    // usernames
+	// usernames
 
-    // ---------
+	// ---------
 
-    // 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(',', array_keys($userids)) . ')', 'IN');
+	$criteria = new \Criteria('uid', '(' . implode(',', array_keys($userids)) . ')', 'IN');
 
-    $usernames = $memberHandler->getUserList($criteria);
+	$usernames = $memberHandler->getUserList($criteria);
 
-    // add usernames to $games
+	// add usernames to $games
 
-    foreach ($games as $k => $game) {
-        $games[$k]['white_uname'] = $usernames[$game['white_uid']] ?? '?';
+	foreach ($games as $k => $game) {
+		$games[$k]['white_uname'] = $usernames[$game['white_uid']] ?? '?';
 
-        $games[$k]['black_uname'] = $usernames[$game['black_uid']] ?? '?';
-    }
+		$games[$k]['black_uname'] = $usernames[$game['black_uid']] ?? '?';
+	}
 
-    // add usernames to $challenges
+	// add usernames to $challenges
 
-    foreach ($challenges as $k => $challenge) {
-        $challenges[$k]['player1_uname'] = $usernames[$challenge['player1_uid']] ?? '?';
+	foreach ($challenges as $k => $challenge) {
+		$challenges[$k]['player1_uname'] = $usernames[$challenge['player1_uid']] ?? '?';
 
-        $challenges[$k]['player2_uname'] = $usernames[$challenge['player2_uid']] ?? '?';
-    }
+		$challenges[$k]['player2_uname'] = $usernames[$challenge['player2_uid']] ?? '?';
+	}
 
-    // ---------------------------------------------------
+	// ---------------------------------------------------
 
-    // player's rating info (if rating feature is enabled)
+	// player's rating info (if rating feature is enabled)
 
-    // ---------------------------------------------------
+	// ---------------------------------------------------
 
-    if ('none' != $rating_system) {
-        $result = $xoopsDB->query(
-            "
+	if ('none' != $rating_system) {
+		$result = $xoopsDB->query(
+			"
             SELECT   player_uid, rating, games_won, games_lost, games_drawn, (games_won+games_lost+games_drawn) AS games_played
             FROM     $ratings_table
             ORDER BY rating DESC, player_uid ASC
         "
-        );
+		);
 
-        $ranking = 0;
+		$ranking = 0;
 
-        while (false !== ($row = $xoopsDB->fetchArray($result))) {
-            if ($row['games_played'] >= $num_provisional_games) {
-                ++$ranking;
-            }
+		while (false !== ($row = $xoopsDB->fetchArray($result))) {
+			if ($row['games_played'] >= $num_provisional_games) {
+				++$ranking;
+			}
 
-            if ($row['player_uid'] == $player_uid) {
-                break;
-            }
-        }
+			if ($row['player_uid'] == $player_uid) {
+				break;
+			}
+		}
 
-        $xoopsDB->freeRecordSet($result);
+		$xoopsDB->freeRecordSet($result);
 
-        if ($row['player_uid'] == $player_uid) {
-            $player['ranking'] = $ranking;
+		if ($row['player_uid'] == $player_uid) {
+			$player['ranking'] = $ranking;
 
-            $player['rating'] = $row['rating'];
+			$player['rating'] = $row['rating'];
 
-            $player['games_won'] = $row['games_won'];
+			$player['games_won'] = $row['games_won'];
 
-            $player['games_lost'] = $row['games_lost'];
+			$player['games_lost'] = $row['games_lost'];
 
-            $player['games_drawn'] = $row['games_drawn'];
+			$player['games_drawn'] = $row['games_drawn'];
 
-            $player['games_played'] = $row['games_played'];
-        }
-    }
+			$player['games_played'] = $row['games_played'];
+		}
+	}
 
-    // Template variables
+	// Template variables
 
-    $player['games'] = $games;
+	$player['games'] = $games;
 
-    $player['challenges'] = $challenges;
+	$player['challenges'] = $challenges;
 
-    $xoopsTpl->assign('chess_player', $player);
+	$xoopsTpl->assign('chess_player', $player);
 
-    $xoopsTpl->assign('chess_rating_system', chess_moduleConfig('rating_system'));
+	$xoopsTpl->assign('chess_rating_system', chess_moduleConfig('rating_system'));
 
-    $xoopsTpl->assign('chess_provisional_games', $num_provisional_games);
+	$xoopsTpl->assign('chess_provisional_games', $num_provisional_games);
 
-    $xoopsTpl->assign('chess_show_option_urlparam', $show_option_urlparam);
+	$xoopsTpl->assign('chess_show_option_urlparam', $show_option_urlparam);
 
-    $xoopsTpl->assign('chess_games_pagenav', $games_pagenav->renderNav());
+	$xoopsTpl->assign('chess_games_pagenav', $games_pagenav->renderNav());
 
-    $xoopsTpl->assign('chess_challenges_pagenav', $challenges_pagenav->renderNav());
+	$xoopsTpl->assign('chess_challenges_pagenav', $challenges_pagenav->renderNav());
 }

header.php

Spacing +1 added lines, -1 removed lines

@@ -35,4 +35,4 @@
 /**#@+
  */
 
-require dirname(__DIR__, 2) . '/mainfile.php';
+require dirname(__DIR__, 2).'/mainfile.php';

admin/main.php

Indentation +158 added lines, -158 removed lines

@@ -50,18 +50,18 @@ 
 xoops_cp_header();
 
 switch ($op) {
-    case 'suspended_games':
-        chess_admin_suspended_games();
-        break;
-    case 'active_games':
-        chess_admin_active_games();
-        break;
-    case 'challenges':
-        chess_admin_challenges();
-        break;
-    default:
-        chess_admin_menu();
-        break;
+	case 'suspended_games':
+		chess_admin_suspended_games();
+		break;
+	case 'active_games':
+		chess_admin_active_games();
+		break;
+	case 'challenges':
+		chess_admin_challenges();
+		break;
+	default:
+		chess_admin_menu();
+		break;
 }
 
 xoops_cp_footer();
@@ -72,30 +72,30 @@ 
  */
 function chess_admin_menu()
 {
-    global $xoopsModule;
+	global $xoopsModule;
 
-    echo '
+	echo '
     <h4> ' . _AM_CHESS_CONF . ' </h4>'
-        //    <table width='100%' border='0' cellspacing='1' class='outer'>
-        //    <tr>
-        //        <td><a href='" . XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . "/admin/index.php?op=suspended_games'>" . _AM_CHESS_SUSPENDED_GAMES . '</a>
-        //        <td>' . _AM_CHESS_SUSPENDED_GAMES_DES . "</td>
-        //    </tr>
-        //    <tr>
-        //        <td><a href='" . XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . "/admin/index.php?op=active_games'>" . _AM_CHESS_ACTIVE_GAMES . '</a>
-        //        <td>' . _AM_CHESS_ACTIVE_GAMES_DES . "</td>
-        //    </tr>
-        //    <tr>
-        //        <td><a href='" . XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . "/admin/index.php?op=challenges'>" . _AM_CHESS_CHALLENGES . '</a>
-        //        <td>' . _AM_CHESS_CHALLENGES_DES . "</td>
-        //    </tr>
-        //    <tr>
-        //        <td><a href='" . XOOPS_URL . '/modules/system/admin.php?fct=preferences&amp;op=showmod&amp;mod=' . $xoopsModule->getVar('mid') . "'>" . _AM_CHESS_PREFS . '</a>
-        //        <td>' . _AM_CHESS_PREFS_DESC . '</td>
-        //    </tr>
-        //    </table>
-        //'
-    ;
+		//    <table width='100%' border='0' cellspacing='1' class='outer'>
+		//    <tr>
+		//        <td><a href='" . XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . "/admin/index.php?op=suspended_games'>" . _AM_CHESS_SUSPENDED_GAMES . '</a>
+		//        <td>' . _AM_CHESS_SUSPENDED_GAMES_DES . "</td>
+		//    </tr>
+		//    <tr>
+		//        <td><a href='" . XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . "/admin/index.php?op=active_games'>" . _AM_CHESS_ACTIVE_GAMES . '</a>
+		//        <td>' . _AM_CHESS_ACTIVE_GAMES_DES . "</td>
+		//    </tr>
+		//    <tr>
+		//        <td><a href='" . XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . "/admin/index.php?op=challenges'>" . _AM_CHESS_CHALLENGES . '</a>
+		//        <td>' . _AM_CHESS_CHALLENGES_DES . "</td>
+		//    </tr>
+		//    <tr>
+		//        <td><a href='" . XOOPS_URL . '/modules/system/admin.php?fct=preferences&amp;op=showmod&amp;mod=' . $xoopsModule->getVar('mid') . "'>" . _AM_CHESS_PREFS . '</a>
+		//        <td>' . _AM_CHESS_PREFS_DESC . '</td>
+		//    </tr>
+		//    </table>
+		//'
+	;
 }
 
 /**
@@ -103,103 +103,103 @@ 
  */
 function chess_admin_suspended_games()
 {
-    global $start, $max_items_to_display, $op, $xoopsDB, $xoopsModule;
+	global $start, $max_items_to_display, $op, $xoopsDB, $xoopsModule;
 
-    $memberHandler = xoops_getHandler('member');
+	$memberHandler = xoops_getHandler('member');
 
-    // 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.
 
-    $games_table = $xoopsDB->prefix('chess_games');
+	$games_table = $xoopsDB->prefix('chess_games');
 
-    $result = $xoopsDB->query("SELECT COUNT(*) FROM $games_table WHERE suspended != ''");
+	$result = $xoopsDB->query("SELECT COUNT(*) FROM $games_table WHERE suspended != ''");
 
-    [$num_rows] = $xoopsDB->fetchRow($result);
+	[$num_rows] = $xoopsDB->fetchRow($result);
 
-    $xoopsDB->freeRecordSet($result);
+	$xoopsDB->freeRecordSet($result);
 
-    // Sort by date-suspended in ascending order, so that games that were suspended the earliest will be displayed
+	// Sort by date-suspended in ascending order, so that games that were suspended the earliest will be displayed
 
-    // at the top, and can more easily be arbitrated on a first-come first-serve basis.
+	// at the top, and can more easily be arbitrated on a first-come first-serve basis.
 
-    // Note that the suspended column begins with the date-suspended in the format 'YYYY-MM-DD HH:MM:SS', so the sorting
+	// Note that the suspended column begins with the date-suspended in the format 'YYYY-MM-DD HH:MM:SS', so the sorting
 
-    // will work as desired.
+	// will work as desired.
 
-    $result = $xoopsDB->query(
-        trim(
-            "
+	$result = $xoopsDB->query(
+		trim(
+			"
         SELECT   game_id, white_uid, black_uid, UNIX_TIMESTAMP(start_date) AS start_date, suspended
         FROM     $games_table
         WHERE    suspended != ''
         ORDER BY suspended
         LIMIT    $start, $max_items_to_display
     "
-        )
-    );
+		)
+	);
 
-    if ($xoopsDB->getRowsNum($result) > 0) {
-        echo '<h3>' . _AM_CHESS_SUSPENDED_GAMES . "</h3>\n";
+	if ($xoopsDB->getRowsNum($result) > 0) {
+		echo '<h3>' . _AM_CHESS_SUSPENDED_GAMES . "</h3>\n";
 
-        while (false !== ($row = $xoopsDB->fetchArray($result))) {
-            $user_white = $memberHandler->getUser($row['white_uid']);
+		while (false !== ($row = $xoopsDB->fetchArray($result))) {
+			$user_white = $memberHandler->getUser($row['white_uid']);
 
-            $username_white = is_object($user_white) ? $user_white->getVar('uname') : '(open)';
+			$username_white = is_object($user_white) ? $user_white->getVar('uname') : '(open)';
 
-            $user_black = $memberHandler->getUser($row['black_uid']);
+			$user_black = $memberHandler->getUser($row['black_uid']);
 
-            $username_black = is_object($user_black) ? $user_black->getVar('uname') : '(open)';
+			$username_black = is_object($user_black) ? $user_black->getVar('uname') : '(open)';
 
-            $date = $row['start_date'] ? date('Y.m.d', $row['start_date']) : 'not yet started';
+			$date = $row['start_date'] ? date('Y.m.d', $row['start_date']) : 'not yet started';
 
-            $title_text = _AM_CHESS_GAME . " #{$row['game_id']}&nbsp;&nbsp;&nbsp;$username_white " . _AM_CHESS_VS . " $username_black&nbsp;&nbsp;&nbsp;($date)";
+			$title_text = _AM_CHESS_GAME . " #{$row['game_id']}&nbsp;&nbsp;&nbsp;$username_white " . _AM_CHESS_VS . " $username_black&nbsp;&nbsp;&nbsp;($date)";
 
-            $form = new XoopsThemeForm($title_text, "game_{$row['game_id']}", XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . "/game.php?game_id={$row['game_id']}", 'post', true);
+			$form = new XoopsThemeForm($title_text, "game_{$row['game_id']}", XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . "/game.php?game_id={$row['game_id']}", 'post', true);
 
-            [$date, $suspender_uid, $type, $explain] = explode('|', $row['suspended']);
+			[$date, $suspender_uid, $type, $explain] = explode('|', $row['suspended']);
 
-            switch ($type) {
-                case 'arbiter_suspend':
-                    $type_display = _AM_CHESS_SUSP_TYPE_ARBITER;
-                    break;
-                case 'want_arbitration':
-                    $type_display = _AM_CHESS_SUSP_TYPE_PLAYER;
-                    break;
-                default:
-                    $type_display = _AM_CHESS_ERROR;
-                    break;
-            }
+			switch ($type) {
+				case 'arbiter_suspend':
+					$type_display = _AM_CHESS_SUSP_TYPE_ARBITER;
+					break;
+				case 'want_arbitration':
+					$type_display = _AM_CHESS_SUSP_TYPE_PLAYER;
+					break;
+				default:
+					$type_display = _AM_CHESS_ERROR;
+					break;
+			}
 
-            $suspender_user = $memberHandler->getUser($suspender_uid);
+			$suspender_user = $memberHandler->getUser($suspender_uid);
 
-            $suspender_username = is_object($suspender_user) ? $suspender_user->getVar('uname') : _AM_CHESS_UNKNOWN_USER;
+			$suspender_username = is_object($suspender_user) ? $suspender_user->getVar('uname') : _AM_CHESS_UNKNOWN_USER;
 
-            $form->addElement(new XoopsFormLabel(_AM_CHESS_WHEN_SUSPENDED . ':', formatTimestamp(strtotime($date))));
+			$form->addElement(new XoopsFormLabel(_AM_CHESS_WHEN_SUSPENDED . ':', formatTimestamp(strtotime($date))));
 
-            $form->addElement(new XoopsFormLabel(_AM_CHESS_SUSPENDED_BY . ':', $suspender_username));
+			$form->addElement(new XoopsFormLabel(_AM_CHESS_SUSPENDED_BY . ':', $suspender_username));
 
-            $form->addElement(new XoopsFormLabel(_AM_CHESS_SUSPENSION_TYPE . ':', $type_display));
+			$form->addElement(new XoopsFormLabel(_AM_CHESS_SUSPENSION_TYPE . ':', $type_display));
 
-            $form->addElement(new XoopsFormLabel(_AM_CHESS_SUSPENSION_REASON . ':', $explain));
+			$form->addElement(new XoopsFormLabel(_AM_CHESS_SUSPENSION_REASON . ':', $explain));
 
-            $form->addElement(new XoopsFormButton('', 'submit', _AM_CHESS_ARBITRATE_SUBMIT, 'submit'));
+			$form->addElement(new XoopsFormButton('', 'submit', _AM_CHESS_ARBITRATE_SUBMIT, 'submit'));
 
-            $form->addElement(new XoopsFormHidden('show_arbiter_ctrl', 1));
+			$form->addElement(new XoopsFormHidden('show_arbiter_ctrl', 1));
 
-            $form->display();
-        }
+			$form->display();
+		}
 
-        $pagenav = new XoopsPageNav($num_rows, $max_items_to_display, $start, 'start', "op=$op");
+		$pagenav = new XoopsPageNav($num_rows, $max_items_to_display, $start, 'start', "op=$op");
 
-        echo '<div align="center">' . $pagenav->renderNav() . "&nbsp;</div>\n";
-    } else {
-        echo '<h3>' . _AM_CHESS_NO_SUSPENDED_GAMES . "</h3>\n";
-    }
+		echo '<div align="center">' . $pagenav->renderNav() . "&nbsp;</div>\n";
+	} else {
+		echo '<h3>' . _AM_CHESS_NO_SUSPENDED_GAMES . "</h3>\n";
+	}
 
-    $xoopsDB->freeRecordSet($result);
+	$xoopsDB->freeRecordSet($result);
 }
 
 /**
@@ -207,69 +207,69 @@ 
  */
 function chess_admin_active_games()
 {
-    global $start, $max_items_to_display, $op, $xoopsDB, $xoopsModule;
+	global $start, $max_items_to_display, $op, $xoopsDB, $xoopsModule;
 
-    $memberHandler = xoops_getHandler('member');
+	$memberHandler = xoops_getHandler('member');
 
-    // 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.
 
-    $games_table = $xoopsDB->prefix('chess_games');
+	$games_table = $xoopsDB->prefix('chess_games');
 
-    $result = $xoopsDB->query("SELECT COUNT(*) FROM $games_table WHERE pgn_result = '*' and suspended = ''");
+	$result = $xoopsDB->query("SELECT COUNT(*) FROM $games_table WHERE pgn_result = '*' and suspended = ''");
 
-    [$num_rows] = $xoopsDB->fetchRow($result);
+	[$num_rows] = $xoopsDB->fetchRow($result);
 
-    $xoopsDB->freeRecordSet($result);
+	$xoopsDB->freeRecordSet($result);
 
-    $result = $xoopsDB->query(
-        trim(
-            "
+	$result = $xoopsDB->query(
+		trim(
+			"
         SELECT   game_id, white_uid, black_uid, UNIX_TIMESTAMP(start_date) AS start_date, GREATEST(create_date,start_date,last_date) AS most_recent_date
         FROM     $games_table
         WHERE    pgn_result = '*' and suspended = ''
         ORDER BY most_recent_date DESC
         LIMIT    $start, $max_items_to_display
     "
-        )
-    );
+		)
+	);
 
-    if ($xoopsDB->getRowsNum($result) > 0) {
-        echo '<h3>' . _AM_CHESS_ACTIVE_GAMES . "</h3>\n";
+	if ($xoopsDB->getRowsNum($result) > 0) {
+		echo '<h3>' . _AM_CHESS_ACTIVE_GAMES . "</h3>\n";
 
-        while (false !== ($row = $xoopsDB->fetchArray($result))) {
-            $user_white = $memberHandler->getUser($row['white_uid']);
+		while (false !== ($row = $xoopsDB->fetchArray($result))) {
+			$user_white = $memberHandler->getUser($row['white_uid']);
 
-            $username_white = is_object($user_white) ? $user_white->getVar('uname') : '(open)';
+			$username_white = is_object($user_white) ? $user_white->getVar('uname') : '(open)';
 
-            $user_black = $memberHandler->getUser($row['black_uid']);
+			$user_black = $memberHandler->getUser($row['black_uid']);
 
-            $username_black = is_object($user_black) ? $user_black->getVar('uname') : '(open)';
+			$username_black = is_object($user_black) ? $user_black->getVar('uname') : '(open)';
 
-            $date = $row['start_date'] ? date('Y.m.d', $row['start_date']) : 'not yet started';
+			$date = $row['start_date'] ? date('Y.m.d', $row['start_date']) : 'not yet started';
 
-            $title_text = _AM_CHESS_GAME . " #{$row['game_id']}&nbsp;&nbsp;&nbsp;$username_white " . _AM_CHESS_VS . " $username_black&nbsp;&nbsp;&nbsp;($date)";
+			$title_text = _AM_CHESS_GAME . " #{$row['game_id']}&nbsp;&nbsp;&nbsp;$username_white " . _AM_CHESS_VS . " $username_black&nbsp;&nbsp;&nbsp;($date)";
 
-            $form = new XoopsThemeForm($title_text, "game_{$row['game_id']}", XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . "/game.php?game_id={$row['game_id']}", 'post', true);
+			$form = new XoopsThemeForm($title_text, "game_{$row['game_id']}", XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . "/game.php?game_id={$row['game_id']}", 'post', true);
 
-            $form->addElement(new XoopsFormButton('', 'submit', _AM_CHESS_ARBITRATE_SUBMIT, 'submit'));
+			$form->addElement(new XoopsFormButton('', 'submit', _AM_CHESS_ARBITRATE_SUBMIT, 'submit'));
 
-            $form->addElement(new XoopsFormHidden('show_arbiter_ctrl', 1));
+			$form->addElement(new XoopsFormHidden('show_arbiter_ctrl', 1));
 
-            $form->display();
-        }
+			$form->display();
+		}
 
-        $pagenav = new XoopsPageNav($num_rows, $max_items_to_display, $start, 'start', "op=$op");
+		$pagenav = new XoopsPageNav($num_rows, $max_items_to_display, $start, 'start', "op=$op");
 
-        echo '<div align="center">' . $pagenav->renderNav() . "&nbsp;</div>\n";
-    } else {
-        echo '<h3>' . _AM_CHESS_NO_ACTIVE_GAMES . "</h3>\n";
-    }
+		echo '<div align="center">' . $pagenav->renderNav() . "&nbsp;</div>\n";
+	} else {
+		echo '<h3>' . _AM_CHESS_NO_ACTIVE_GAMES . "</h3>\n";
+	}
 
-    $xoopsDB->freeRecordSet($result);
+	$xoopsDB->freeRecordSet($result);
 }
 
 /**
@@ -277,66 +277,66 @@ 
  */
 function chess_admin_challenges()
 {
-    global $start, $max_items_to_display, $op, $xoopsDB, $xoopsModule;
+	global $start, $max_items_to_display, $op, $xoopsDB, $xoopsModule;
 
-    $memberHandler = xoops_getHandler('member');
+	$memberHandler = xoops_getHandler('member');
 
-    // 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.
 
-    $challenges_table = $xoopsDB->prefix('chess_challenges');
+	$challenges_table = $xoopsDB->prefix('chess_challenges');
 
-    $result = $xoopsDB->query("SELECT COUNT(*) FROM $challenges_table");
+	$result = $xoopsDB->query("SELECT COUNT(*) FROM $challenges_table");
 
-    [$num_rows] = $xoopsDB->fetchRow($result);
+	[$num_rows] = $xoopsDB->fetchRow($result);
 
-    $xoopsDB->freeRecordSet($result);
+	$xoopsDB->freeRecordSet($result);
 
-    $result = $xoopsDB->query(
-        trim(
-            "
+	$result = $xoopsDB->query(
+		trim(
+			"
         SELECT   challenge_id, game_type, color_option, player1_uid, player2_uid, UNIX_TIMESTAMP(create_date) AS create_date
         FROM     $challenges_table
         ORDER BY create_date DESC
         LIMIT    $start, $max_items_to_display
     "
-        )
-    );
+		)
+	);
 
-    if ($xoopsDB->getRowsNum($result) > 0) {
-        echo '<h3>' . _AM_CHESS_CHALLENGES . "</h3>\n";
+	if ($xoopsDB->getRowsNum($result) > 0) {
+		echo '<h3>' . _AM_CHESS_CHALLENGES . "</h3>\n";
 
-        while (false !== ($row = $xoopsDB->fetchArray($result))) {
-            $user_player1 = $memberHandler->getUser($row['player1_uid']);
+		while (false !== ($row = $xoopsDB->fetchArray($result))) {
+			$user_player1 = $memberHandler->getUser($row['player1_uid']);
 
-            $username_player1 = is_object($user_player1) ? $user_player1->getVar('uname') : '?';
+			$username_player1 = is_object($user_player1) ? $user_player1->getVar('uname') : '?';
 
-            $user_player2 = $memberHandler->getUser($row['player2_uid']);
+			$user_player2 = $memberHandler->getUser($row['player2_uid']);
 
-            $username_player2 = is_object($user_player2) ? $user_player2->getVar('uname') : '(open)';
+			$username_player2 = is_object($user_player2) ? $user_player2->getVar('uname') : '(open)';
 
-            $date = date('Y.m.d', $row['create_date']);
+			$date = date('Y.m.d', $row['create_date']);
 
-            $title_text = _AM_CHESS_CHALLENGE . " #{$row['challenge_id']}&nbsp;&nbsp;&nbsp;$username_player1 " . _AM_CHESS_CHALLENGED . ": $username_player2&nbsp;&nbsp;&nbsp;(" . _AM_CHESS_CREATED . " $date)";
+			$title_text = _AM_CHESS_CHALLENGE . " #{$row['challenge_id']}&nbsp;&nbsp;&nbsp;$username_player1 " . _AM_CHESS_CHALLENGED . ": $username_player2&nbsp;&nbsp;&nbsp;(" . _AM_CHESS_CREATED . " $date)";
 
-            $form = new XoopsThemeForm($title_text, "challenge_{$row['challenge_id']}", XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . "/create.php?challenge_id={$row['challenge_id']}", 'post', true);
+			$form = new XoopsThemeForm($title_text, "challenge_{$row['challenge_id']}", XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . "/create.php?challenge_id={$row['challenge_id']}", 'post', true);
 
-            $form->addElement(new XoopsFormButton('', 'submit', _AM_CHESS_ARBITRATE_SUBMIT, 'submit'));
+			$form->addElement(new XoopsFormButton('', 'submit', _AM_CHESS_ARBITRATE_SUBMIT, 'submit'));
 
-            $form->addElement(new XoopsFormHidden('show_arbiter_ctrl', 1));
+			$form->addElement(new XoopsFormHidden('show_arbiter_ctrl', 1));
 
-            $form->display();
-        }
+			$form->display();
+		}
 
-        $pagenav = new XoopsPageNav($num_rows, $max_items_to_display, $start, 'start', "op=$op");
+		$pagenav = new XoopsPageNav($num_rows, $max_items_to_display, $start, 'start', "op=$op");
 
-        echo '<div align="center">' . $pagenav->renderNav() . "&nbsp;</div>\n";
-    } else {
-        echo '<h3>' . _AM_CHESS_NO_CHALLENGES . "</h3>\n";
-    }
+		echo '<div align="center">' . $pagenav->renderNav() . "&nbsp;</div>\n";
+	} else {
+		echo '<h3>' . _AM_CHESS_NO_CHALLENGES . "</h3>\n";
+	}
 
-    $xoopsDB->freeRecordSet($result);
+	$xoopsDB->freeRecordSet($result);
 }

Spacing +25 added lines, -25 removed lines

@@ -34,14 +34,14 @@ 
 
 /**#@+
  */
-require_once __DIR__ . '/admin_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/functions.php';
+require_once __DIR__.'/admin_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/functions.php';
 
 // user input
 $op    = chess_sanitize(@$_GET['op']);
-$start = (int)@$_GET['start']; // offset of first row of table to display (default to 0)
+$start = (int) @$_GET['start']; // offset of first row of table to display (default to 0)
 
 // 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');
@@ -75,7 +75,7 @@ 
     global $xoopsModule;
 
     echo '
-    <h4> ' . _AM_CHESS_CONF . ' </h4>'
+    <h4> ' . _AM_CHESS_CONF.' </h4>'
         //    <table width='100%' border='0' cellspacing='1' class='outer'>
         //    <tr>
         //        <td><a href='" . XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . "/admin/index.php?op=suspended_games'>" . _AM_CHESS_SUSPENDED_GAMES . '</a>
@@ -142,7 +142,7 @@ 
     );
 
     if ($xoopsDB->getRowsNum($result) > 0) {
-        echo '<h3>' . _AM_CHESS_SUSPENDED_GAMES . "</h3>\n";
+        echo '<h3>'._AM_CHESS_SUSPENDED_GAMES."</h3>\n";
 
         while (false !== ($row = $xoopsDB->fetchArray($result))) {
             $user_white = $memberHandler->getUser($row['white_uid']);
@@ -155,9 +155,9 @@ 
 
             $date = $row['start_date'] ? date('Y.m.d', $row['start_date']) : 'not yet started';
 
-            $title_text = _AM_CHESS_GAME . " #{$row['game_id']}&nbsp;&nbsp;&nbsp;$username_white " . _AM_CHESS_VS . " $username_black&nbsp;&nbsp;&nbsp;($date)";
+            $title_text = _AM_CHESS_GAME." #{$row['game_id']}&nbsp;&nbsp;&nbsp;$username_white "._AM_CHESS_VS." $username_black&nbsp;&nbsp;&nbsp;($date)";
 
-            $form = new XoopsThemeForm($title_text, "game_{$row['game_id']}", XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . "/game.php?game_id={$row['game_id']}", 'post', true);
+            $form = new XoopsThemeForm($title_text, "game_{$row['game_id']}", XOOPS_URL.'/modules/'.$xoopsModule->getVar('dirname')."/game.php?game_id={$row['game_id']}", 'post', true);
 
             [$date, $suspender_uid, $type, $explain] = explode('|', $row['suspended']);
 
@@ -177,13 +177,13 @@ 
 
             $suspender_username = is_object($suspender_user) ? $suspender_user->getVar('uname') : _AM_CHESS_UNKNOWN_USER;
 
-            $form->addElement(new XoopsFormLabel(_AM_CHESS_WHEN_SUSPENDED . ':', formatTimestamp(strtotime($date))));
+            $form->addElement(new XoopsFormLabel(_AM_CHESS_WHEN_SUSPENDED.':', formatTimestamp(strtotime($date))));
 
-            $form->addElement(new XoopsFormLabel(_AM_CHESS_SUSPENDED_BY . ':', $suspender_username));
+            $form->addElement(new XoopsFormLabel(_AM_CHESS_SUSPENDED_BY.':', $suspender_username));
 
-            $form->addElement(new XoopsFormLabel(_AM_CHESS_SUSPENSION_TYPE . ':', $type_display));
+            $form->addElement(new XoopsFormLabel(_AM_CHESS_SUSPENSION_TYPE.':', $type_display));
 
-            $form->addElement(new XoopsFormLabel(_AM_CHESS_SUSPENSION_REASON . ':', $explain));
+            $form->addElement(new XoopsFormLabel(_AM_CHESS_SUSPENSION_REASON.':', $explain));
 
             $form->addElement(new XoopsFormButton('', 'submit', _AM_CHESS_ARBITRATE_SUBMIT, 'submit'));
 
@@ -194,9 +194,9 @@ 
 
         $pagenav = new XoopsPageNav($num_rows, $max_items_to_display, $start, 'start', "op=$op");
 
-        echo '<div align="center">' . $pagenav->renderNav() . "&nbsp;</div>\n";
+        echo '<div align="center">'.$pagenav->renderNav()."&nbsp;</div>\n";
     } else {
-        echo '<h3>' . _AM_CHESS_NO_SUSPENDED_GAMES . "</h3>\n";
+        echo '<h3>'._AM_CHESS_NO_SUSPENDED_GAMES."</h3>\n";
     }
 
     $xoopsDB->freeRecordSet($result);
@@ -238,7 +238,7 @@ 
     );
 
     if ($xoopsDB->getRowsNum($result) > 0) {
-        echo '<h3>' . _AM_CHESS_ACTIVE_GAMES . "</h3>\n";
+        echo '<h3>'._AM_CHESS_ACTIVE_GAMES."</h3>\n";
 
         while (false !== ($row = $xoopsDB->fetchArray($result))) {
             $user_white = $memberHandler->getUser($row['white_uid']);
@@ -251,9 +251,9 @@ 
 
             $date = $row['start_date'] ? date('Y.m.d', $row['start_date']) : 'not yet started';
 
-            $title_text = _AM_CHESS_GAME . " #{$row['game_id']}&nbsp;&nbsp;&nbsp;$username_white " . _AM_CHESS_VS . " $username_black&nbsp;&nbsp;&nbsp;($date)";
+            $title_text = _AM_CHESS_GAME." #{$row['game_id']}&nbsp;&nbsp;&nbsp;$username_white "._AM_CHESS_VS." $username_black&nbsp;&nbsp;&nbsp;($date)";
 
-            $form = new XoopsThemeForm($title_text, "game_{$row['game_id']}", XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . "/game.php?game_id={$row['game_id']}", 'post', true);
+            $form = new XoopsThemeForm($title_text, "game_{$row['game_id']}", XOOPS_URL.'/modules/'.$xoopsModule->getVar('dirname')."/game.php?game_id={$row['game_id']}", 'post', true);
 
             $form->addElement(new XoopsFormButton('', 'submit', _AM_CHESS_ARBITRATE_SUBMIT, 'submit'));
 
@@ -264,9 +264,9 @@ 
 
         $pagenav = new XoopsPageNav($num_rows, $max_items_to_display, $start, 'start', "op=$op");
 
-        echo '<div align="center">' . $pagenav->renderNav() . "&nbsp;</div>\n";
+        echo '<div align="center">'.$pagenav->renderNav()."&nbsp;</div>\n";
     } else {
-        echo '<h3>' . _AM_CHESS_NO_ACTIVE_GAMES . "</h3>\n";
+        echo '<h3>'._AM_CHESS_NO_ACTIVE_GAMES."</h3>\n";
     }
 
     $xoopsDB->freeRecordSet($result);
@@ -307,7 +307,7 @@ 
     );
 
     if ($xoopsDB->getRowsNum($result) > 0) {
-        echo '<h3>' . _AM_CHESS_CHALLENGES . "</h3>\n";
+        echo '<h3>'._AM_CHESS_CHALLENGES."</h3>\n";
 
         while (false !== ($row = $xoopsDB->fetchArray($result))) {
             $user_player1 = $memberHandler->getUser($row['player1_uid']);
@@ -320,9 +320,9 @@ 
 
             $date = date('Y.m.d', $row['create_date']);
 
-            $title_text = _AM_CHESS_CHALLENGE . " #{$row['challenge_id']}&nbsp;&nbsp;&nbsp;$username_player1 " . _AM_CHESS_CHALLENGED . ": $username_player2&nbsp;&nbsp;&nbsp;(" . _AM_CHESS_CREATED . " $date)";
+            $title_text = _AM_CHESS_CHALLENGE." #{$row['challenge_id']}&nbsp;&nbsp;&nbsp;$username_player1 "._AM_CHESS_CHALLENGED.": $username_player2&nbsp;&nbsp;&nbsp;("._AM_CHESS_CREATED." $date)";
 
-            $form = new XoopsThemeForm($title_text, "challenge_{$row['challenge_id']}", XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . "/create.php?challenge_id={$row['challenge_id']}", 'post', true);
+            $form = new XoopsThemeForm($title_text, "challenge_{$row['challenge_id']}", XOOPS_URL.'/modules/'.$xoopsModule->getVar('dirname')."/create.php?challenge_id={$row['challenge_id']}", 'post', true);
 
             $form->addElement(new XoopsFormButton('', 'submit', _AM_CHESS_ARBITRATE_SUBMIT, 'submit'));
 
@@ -333,9 +333,9 @@ 
 
         $pagenav = new XoopsPageNav($num_rows, $max_items_to_display, $start, 'start', "op=$op");
 
-        echo '<div align="center">' . $pagenav->renderNav() . "&nbsp;</div>\n";
+        echo '<div align="center">'.$pagenav->renderNav()."&nbsp;</div>\n";
     } else {
-        echo '<h3>' . _AM_CHESS_NO_CHALLENGES . "</h3>\n";
+        echo '<h3>'._AM_CHESS_NO_CHALLENGES."</h3>\n";
     }
 
     $xoopsDB->freeRecordSet($result);

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,31 +725,31 @@ 
      * }
      ***/
 
-    #*#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) {
@@ -757,61 +757,61 @@ 
          * }
          ***/
 
-        #*#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) {
@@ -819,18 +819,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);
@@ -841,27 +841,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];
 }
 
 /**
@@ -872,31 +872,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;
 }
 
 /**
@@ -912,252 +912,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 +41 added lines, -41 removed lines

@@ -40,19 +40,19 @@ 
 
 /**#@+
  */
-require_once dirname(__DIR__, 2) . '/mainfile.php';
-require_once XOOPS_ROOT_PATH . '/class/xoopsformloader.php';
+require_once dirname(__DIR__, 2).'/mainfile.php';
+require_once XOOPS_ROOT_PATH.'/class/xoopsformloader.php';
 
 $GLOBALS['xoopsOption']['template_main']                  = 'chess_game_main.tpl';
 $xoopsConfig['module_cache'][$xoopsModule->getVar('mid')] = 0; // disable caching
-require_once XOOPS_ROOT_PATH . '/header.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.'/header.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);
@@ -807,7 +807,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
 
@@ -854,7 +854,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);
 
@@ -917,7 +917,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">
     '
     );
 
@@ -960,7 +960,7 @@ 
         $expanded_row = preg_replace_callback(
             '/(\d)/',
 
-            static function ($matches) {
+            static function($matches) {
                 return str_repeat('x', $matches[1]);
             },
             $rank

docs/maketestdata.php

Indentation +85 added lines, -85 removed lines

@@ -30,63 +30,63 @@ 
  */
 function perform()
 {
-    $challenges_table = 'chess_challenges';
+	$challenges_table = 'chess_challenges';
 
-    $games_table = 'chess_games';
+	$games_table = 'chess_games';
 
-    $ratings_table = 'chess_ratings';
+	$ratings_table = 'chess_ratings';
 
-    mysqli_connect(DBHOST, DBUSER, DBPASS) or trigger_error('[' . $GLOBALS['xoopsDB']->errno() . '] ' . $GLOBALS['xoopsDB']->error(), E_USER_ERROR);
+	mysqli_connect(DBHOST, DBUSER, DBPASS) or trigger_error('[' . $GLOBALS['xoopsDB']->errno() . '] ' . $GLOBALS['xoopsDB']->error(), E_USER_ERROR);
 
-    mysqli_select_db($GLOBALS['xoopsDB']->conn, DBNAME) or trigger_error('[' . $GLOBALS['xoopsDB']->errno() . '] ' . $GLOBALS['xoopsDB']->error(), E_USER_ERROR);
+	mysqli_select_db($GLOBALS['xoopsDB']->conn, DBNAME) or trigger_error('[' . $GLOBALS['xoopsDB']->errno() . '] ' . $GLOBALS['xoopsDB']->error(), E_USER_ERROR);
 
-    // For safety, don't generate test data unless the tables are empty.
+	// For safety, don't generate test data unless the tables are empty.
 
-    if (!table_empty($challenges_table) || !table_empty($games_table) || !table_empty($ratings_table)) {
-        echo "Tables already contain data - no action performed.\n";
+	if (!table_empty($challenges_table) || !table_empty($games_table) || !table_empty($ratings_table)) {
+		echo "Tables already contain data - no action performed.\n";
 
-        exit;
-    }
+		exit;
+	}
 
-    // Generate the challenges table
+	// Generate the challenges table
 
-    $game_types = ['open', 'user'];
+	$game_types = ['open', 'user'];
 
-    $color_options = ['player2', 'random', 'white', 'black'];
+	$color_options = ['player2', 'random', 'white', 'black'];
 
-    for ($i = 0; $i < NUM_CHALLENGES; ++$i) {
-        $game_type = rand_array_value($game_types);
+	for ($i = 0; $i < NUM_CHALLENGES; ++$i) {
+		$game_type = rand_array_value($game_types);
 
-        $fen_index = random_int(1, 10);
+		$fen_index = random_int(1, 10);
 
-        $fen = 10 == $fen_index ? 'rnbqkbnr/pppppppp/8/8/8/8/PPPPPPPP/RNBQKBNR w KQkq - 0 1' : '';
+		$fen = 10 == $fen_index ? 'rnbqkbnr/pppppppp/8/8/8/8/PPPPPPPP/RNBQKBNR w KQkq - 0 1' : '';
 
-        $color_option = rand_array_value($color_options);
+		$color_option = rand_array_value($color_options);
 
-        $notify_move_player1 = random_int(0, 1);
+		$notify_move_player1 = random_int(0, 1);
 
-        $player1_uid = random_int(1, NUM_USERS);
+		$player1_uid = random_int(1, NUM_USERS);
 
-        if ('open' == $game_type) {
-            $player2_uid = 0;
-        } else {
-            // select $player2_uid != $player1_uid
+		if ('open' == $game_type) {
+			$player2_uid = 0;
+		} else {
+			// select $player2_uid != $player1_uid
 
-            do {
-                $player2_uid = random_int(1, NUM_USERS);
-            } while ($player2_uid == $player1_uid);
-        }
+			do {
+				$player2_uid = random_int(1, NUM_USERS);
+			} while ($player2_uid == $player1_uid);
+		}
 
-        $create_date_max = time();
+		$create_date_max = time();
 
-        $create_date_min = $create_date_max - 30 * 24 * 3600;
+		$create_date_min = $create_date_max - 30 * 24 * 3600;
 
-        $create_date = date('Y-m-d H:i:s', random_int($create_date_min, $create_date_max));
+		$create_date = date('Y-m-d H:i:s', random_int($create_date_min, $create_date_max));
 
-        $is_rated = random_int(0, 1);
+		$is_rated = random_int(0, 1);
 
-        do_query(
-            "
+		do_query(
+			"
             INSERT INTO $challenges_table
             SET
                 game_type           = '$game_type',
@@ -98,68 +98,68 @@ 
                 create_date         = '$create_date',
                 is_rated            = '$is_rated'
         "
-        );
-    }
+		);
+	}
 
-    // Generate the games table
+	// Generate the games table
 
-    $pgn_results = ['*', '0-1', '1-0', '1/2-1/2'];
+	$pgn_results = ['*', '0-1', '1-0', '1/2-1/2'];
 
-    $suspended_explains = ['foo', 'bar', 'baz', 'quux'];
+	$suspended_explains = ['foo', 'bar', 'baz', 'quux'];
 
-    for ($i = 0; $i < NUM_GAMES; ++$i) {
-        $white_uid = random_int(1, NUM_USERS);
+	for ($i = 0; $i < NUM_GAMES; ++$i) {
+		$white_uid = random_int(1, NUM_USERS);
 
-        $black_uid = random_int(1, NUM_USERS);
+		$black_uid = random_int(1, NUM_USERS);
 
-        // Force some games to be self-play.
+		// Force some games to be self-play.
 
-        if (10 == random_int(1, 10)) {
-            $black_uid = $white_uid;
-        }
+		if (10 == random_int(1, 10)) {
+			$black_uid = $white_uid;
+		}
 
-        $create_date_max = time();
+		$create_date_max = time();
 
-        $create_date_min = $create_date_max - 365 * 24 * 3600;
+		$create_date_min = $create_date_max - 365 * 24 * 3600;
 
-        $create_date_sec = random_int($create_date_min, $create_date_max);
+		$create_date_sec = random_int($create_date_min, $create_date_max);
 
-        $create_date = date('Y-m-d H:i:s', $create_date_sec);
+		$create_date = date('Y-m-d H:i:s', $create_date_sec);
 
-        $is_started = random_int(1, 4) < 4;
+		$is_started = random_int(1, 4) < 4;
 
-        $start_date_sec = $is_started ? $create_date_sec + random_int(3600, 3 * 24 * 3600) : 0;
+		$start_date_sec = $is_started ? $create_date_sec + random_int(3600, 3 * 24 * 3600) : 0;
 
-        $start_date = $is_started ? date('Y-m-d H:i:s', $start_date_sec) : '0000-00-00 00:00:00';
+		$start_date = $is_started ? date('Y-m-d H:i:s', $start_date_sec) : '0000-00-00 00:00:00';
 
-        $multiple_moves = $is_started && random_int(1, 10) < 10;
+		$multiple_moves = $is_started && random_int(1, 10) < 10;
 
-        $last_date_sec = $multiple_moves ? $start_date_sec + random_int(3600, 90 * 24 * 3600) : 0;
+		$last_date_sec = $multiple_moves ? $start_date_sec + random_int(3600, 90 * 24 * 3600) : 0;
 
-        $last_date = $multiple_moves ? date('Y-m-d H:i:s', $last_date_sec) : '0000-00-00 00:00:00';
+		$last_date = $multiple_moves ? date('Y-m-d H:i:s', $last_date_sec) : '0000-00-00 00:00:00';
 
-        $pgn_result = $multiple_moves ? rand_array_value($pgn_results) : '*';
+		$pgn_result = $multiple_moves ? rand_array_value($pgn_results) : '*';
 
-        if ($multiple_moves && '*' == $pgn_result && 5 == random_int(1, 5)) {
-            $suspended_date = date('Y-m-d H:i:s', $last_date_sec + random_int(60, 72 * 3600));
+		if ($multiple_moves && '*' == $pgn_result && 5 == random_int(1, 5)) {
+			$suspended_date = date('Y-m-d H:i:s', $last_date_sec + random_int(60, 72 * 3600));
 
-            $suspended_uids = [1, $white_uid, $black_uid];
+			$suspended_uids = [1, $white_uid, $black_uid];
 
-            $suspended_uid = rand_array_value($suspended_uids);
+			$suspended_uid = rand_array_value($suspended_uids);
 
-            $suspended_type = 1 == $suspended_uid ? 'arbiter_suspend' : 'want_arbitration';
+			$suspended_type = 1 == $suspended_uid ? 'arbiter_suspend' : 'want_arbitration';
 
-            $suspended_explain = rand_array_value($suspended_explains);
+			$suspended_explain = rand_array_value($suspended_explains);
 
-            $suspended = "$suspended_date|$suspended_uid|$suspended_type|$suspended_explain";
-        } else {
-            $suspended = '';
-        }
+			$suspended = "$suspended_date|$suspended_uid|$suspended_type|$suspended_explain";
+		} else {
+			$suspended = '';
+		}
 
-        $is_rated = $white_uid != $black_uid ? random_int(0, 1) : 0;
+		$is_rated = $white_uid != $black_uid ? random_int(0, 1) : 0;
 
-        do_query(
-            "
+		do_query(
+			"
             INSERT INTO $games_table
             SET
                 white_uid   = '$white_uid',
@@ -171,10 +171,10 @@ 
                 suspended   = '$suspended',
                 is_rated    = '$is_rated'
         "
-        );
-    }
+		);
+	}
 
-    $GLOBALS['xoopsDB']->close();
+	$GLOBALS['xoopsDB']->close();
 }
 
 /**
@@ -185,13 +185,13 @@ 
  */
 function table_empty($table)
 {
-    $result = do_query("SELECT COUNT(*) FROM $table");
+	$result = do_query("SELECT COUNT(*) FROM $table");
 
-    [$num_rows] = $GLOBALS['xoopsDB']->fetchRow($result);
+	[$num_rows] = $GLOBALS['xoopsDB']->fetchRow($result);
 
-    $GLOBALS['xoopsDB']->freeRecordSet($result);
+	$GLOBALS['xoopsDB']->freeRecordSet($result);
 
-    return 0 == $num_rows;
+	return 0 == $num_rows;
 }
 
 /**
@@ -204,17 +204,17 @@ 
  */
 function do_query($query)
 {
-    $result = $GLOBALS['xoopsDB']->queryF($query);
+	$result = $GLOBALS['xoopsDB']->queryF($query);
 
-    if (false === $result) {
-        $errno = $GLOBALS['xoopsDB']->errno();
+	if (false === $result) {
+		$errno = $GLOBALS['xoopsDB']->errno();
 
-        $error = $GLOBALS['xoopsDB']->error();
+		$error = $GLOBALS['xoopsDB']->error();
 
-        trigger_error("[$errno] $error\n$query", E_USER_ERROR);
-    }
+		trigger_error("[$errno] $error\n$query", E_USER_ERROR);
+	}
 
-    return $result;
+	return $result;
 }
 
 /**
@@ -223,5 +223,5 @@ 
  */
 function rand_array_value($array)
 {
-    return $array[array_rand($array)];
+	return $array[array_rand($array)];
 }

xoops_version.php

Indentation +239 added lines, -239 removed lines

@@ -30,98 +30,98 @@ 
 
 // ------------------- Informations ------------------- //
 $modversion = [
-    'version'       => 2.01,
-    'module_status' => 'Alpha 1',
-    'release_date'  => '2020/05/25',
-    'name'          => _MI_CHESS,
-    'description'   => _MI_CHESS_DES,
-    'official'      => 0,
-    //1 indicates official XOOPS module supported by XOOPS Dev Team, 0 means 3rd party supported
-    'author'        => '<a target="_blank" href="http://Dave-L.com/">Dave Lerner</a>, Mamba',
-
-    'credits'             => 'XOOPS Development Team',
-    'author_mail'         => 'author-email',
-    'author_website_url'  => 'https://xoops.org',
-    'author_website_name' => 'XOOPS',
-    'license'             => 'GPL 2.0 or later',
-    'license_url'         => 'www.gnu.org/licenses/gpl-2.0.html/',
-    // ------------------- Folders & Files -------------------
-    'release_info'        => 'Changelog',
-    'release_file'        => XOOPS_URL . "/modules/$moduleDirName/docs/changelog.txt",
-
-    'manual'              => 'link to manual file',
-    'manual_file'         => XOOPS_URL . "/modules/$moduleDirName/docs/install.txt",
-    // images
-    'image'               => 'assets/images/logoModule.png',
-    'iconsmall'           => 'assets/images/iconsmall.png',
-    'iconbig'             => 'assets/images/iconbig.png',
-    'dirname'             => $moduleDirName,
-    //Frameworks
-    //    'dirmoduleadmin'      => 'Frameworks/moduleclasses/moduleadmin',
-    //    'sysicons16'          => 'Frameworks/moduleclasses/icons/16',
-    //    'sysicons32'          => 'Frameworks/moduleclasses/icons/32',
-    // Local path icons
-    'modicons16'          => 'assets/images/icons/16',
-    'modicons32'          => 'assets/images/icons/32',
-    //About
-    'demo_site_url'       => 'https://xoops.org',
-    'demo_site_name'      => 'XOOPS Demo Site',
-    'support_url'         => 'https://xoops.org/modules/newbb/viewforum.php?forum=28/',
-    'support_name'        => 'Support Forum',
-    'submit_bug'          => 'https://github.com/XoopsModules25x/' . $moduleDirName . '/issues',
-    'module_website_url'  => 'www.xoops.org',
-    'module_website_name' => 'XOOPS Project',
-    // ------------------- Min Requirements -------------------
-    'min_php'             => '7.2',
-    'min_xoops'           => '2.5.10',
-    'min_admin'           => '1.2',
-    'min_db'              => ['mysql' => '5.5'],
-    // ------------------- Admin Menu -------------------
-    'system_menu'         => 1,
-    'hasAdmin'            => 1,
-    'adminindex'          => 'admin/index.php',
-    'adminmenu'           => 'admin/menu.php',
-    // ------------------- Install/Update -------------------
-    'onInstall'           => 'include/install.php',
-    //    'onInstall' => 'include/oninstall.php',
-    //    'onUpdate' => 'include/onupdate.php',
-    //  'onUninstall'         => 'include/onuninstall.php',
-    // -------------------  PayPal ---------------------------
-    'paypal'              => [
-        'business'      => 'xoopsfoundation@gmail.com',
-        'item_name'     => 'Donation : ' . _MI_CHESS_NAME,
-        'amount'        => 0,
-        'currency_code' => 'USD',
-    ],
-    // ------------------- Search ---------------------------
-    'hasSearch'           => 0,
-    //    'search'              => [
-    //        'file' => 'include/search.inc.php',
-    //        'func' => 'pedigree_search',
-    //    ],
-    // ------------------- Comments -------------------------
-    'hasComments'         => 1,
-    'comments'            => [
-        'itemName' => 'game_id',
-        'pageName' => 'game.php',
-    ],
-    // ------------------- Mysql -----------------------------
-    'sqlfile'             => ['mysql' => 'sql/mysql.sql'],
-    // ------------------- Tables ----------------------------
-    'tables'              => [
-        $moduleDirName . '_' . 'games',
-        $moduleDirName . '_' . 'challenges',
-        $moduleDirName . '_' . 'ratings',
-    ],
+	'version'       => 2.01,
+	'module_status' => 'Alpha 1',
+	'release_date'  => '2020/05/25',
+	'name'          => _MI_CHESS,
+	'description'   => _MI_CHESS_DES,
+	'official'      => 0,
+	//1 indicates official XOOPS module supported by XOOPS Dev Team, 0 means 3rd party supported
+	'author'        => '<a target="_blank" href="http://Dave-L.com/">Dave Lerner</a>, Mamba',
+
+	'credits'             => 'XOOPS Development Team',
+	'author_mail'         => 'author-email',
+	'author_website_url'  => 'https://xoops.org',
+	'author_website_name' => 'XOOPS',
+	'license'             => 'GPL 2.0 or later',
+	'license_url'         => 'www.gnu.org/licenses/gpl-2.0.html/',
+	// ------------------- Folders & Files -------------------
+	'release_info'        => 'Changelog',
+	'release_file'        => XOOPS_URL . "/modules/$moduleDirName/docs/changelog.txt",
+
+	'manual'              => 'link to manual file',
+	'manual_file'         => XOOPS_URL . "/modules/$moduleDirName/docs/install.txt",
+	// images
+	'image'               => 'assets/images/logoModule.png',
+	'iconsmall'           => 'assets/images/iconsmall.png',
+	'iconbig'             => 'assets/images/iconbig.png',
+	'dirname'             => $moduleDirName,
+	//Frameworks
+	//    'dirmoduleadmin'      => 'Frameworks/moduleclasses/moduleadmin',
+	//    'sysicons16'          => 'Frameworks/moduleclasses/icons/16',
+	//    'sysicons32'          => 'Frameworks/moduleclasses/icons/32',
+	// Local path icons
+	'modicons16'          => 'assets/images/icons/16',
+	'modicons32'          => 'assets/images/icons/32',
+	//About
+	'demo_site_url'       => 'https://xoops.org',
+	'demo_site_name'      => 'XOOPS Demo Site',
+	'support_url'         => 'https://xoops.org/modules/newbb/viewforum.php?forum=28/',
+	'support_name'        => 'Support Forum',
+	'submit_bug'          => 'https://github.com/XoopsModules25x/' . $moduleDirName . '/issues',
+	'module_website_url'  => 'www.xoops.org',
+	'module_website_name' => 'XOOPS Project',
+	// ------------------- Min Requirements -------------------
+	'min_php'             => '7.2',
+	'min_xoops'           => '2.5.10',
+	'min_admin'           => '1.2',
+	'min_db'              => ['mysql' => '5.5'],
+	// ------------------- Admin Menu -------------------
+	'system_menu'         => 1,
+	'hasAdmin'            => 1,
+	'adminindex'          => 'admin/index.php',
+	'adminmenu'           => 'admin/menu.php',
+	// ------------------- Install/Update -------------------
+	'onInstall'           => 'include/install.php',
+	//    'onInstall' => 'include/oninstall.php',
+	//    'onUpdate' => 'include/onupdate.php',
+	//  'onUninstall'         => 'include/onuninstall.php',
+	// -------------------  PayPal ---------------------------
+	'paypal'              => [
+		'business'      => 'xoopsfoundation@gmail.com',
+		'item_name'     => 'Donation : ' . _MI_CHESS_NAME,
+		'amount'        => 0,
+		'currency_code' => 'USD',
+	],
+	// ------------------- Search ---------------------------
+	'hasSearch'           => 0,
+	//    'search'              => [
+	//        'file' => 'include/search.inc.php',
+	//        'func' => 'pedigree_search',
+	//    ],
+	// ------------------- Comments -------------------------
+	'hasComments'         => 1,
+	'comments'            => [
+		'itemName' => 'game_id',
+		'pageName' => 'game.php',
+	],
+	// ------------------- Mysql -----------------------------
+	'sqlfile'             => ['mysql' => 'sql/mysql.sql'],
+	// ------------------- Tables ----------------------------
+	'tables'              => [
+		$moduleDirName . '_' . 'games',
+		$moduleDirName . '_' . 'challenges',
+		$moduleDirName . '_' . 'ratings',
+	],
 ];
 
 // ------------------- Help files ------------------- //
 $modversion['help']        = 'page=help';
 $modversion['helpsection'] = [
-    ['name' => _MI_CHESS_OVERVIEW, 'link' => 'page=help'],
-    ['name' => _MI_CHESS_DISCLAIMER, 'link' => 'page=disclaimer'],
-    ['name' => _MI_CHESS_LICENSE, 'link' => 'page=license'],
-    ['name' => _MI_CHESS_SUPPORT, 'link' => 'page=support'],
+	['name' => _MI_CHESS_OVERVIEW, 'link' => 'page=help'],
+	['name' => _MI_CHESS_DISCLAIMER, 'link' => 'page=disclaimer'],
+	['name' => _MI_CHESS_LICENSE, 'link' => 'page=license'],
+	['name' => _MI_CHESS_SUPPORT, 'link' => 'page=support'],
 ];
 
 // Config
@@ -129,77 +129,77 @@ 
 $memberHandler = xoops_getHandler('member');
 $groups        = $memberHandler->getGroups();
 foreach ($groups as $group) {
-    if (XOOPS_GROUP_ANONYMOUS != $group->getVar('groupid')) {
-        $grouparray[$group->getVar('name')] = $group->getVar('groupid');
-    }
+	if (XOOPS_GROUP_ANONYMOUS != $group->getVar('groupid')) {
+		$grouparray[$group->getVar('name')] = $group->getVar('groupid');
+	}
 }
 
 $modversion['config'][] = [
-    'name'        => 'groups_play',
-    'title'       => '_MI_CHESS_GROUPS_PLAY',
-    'description' => '_MI_CHESS_GROUPS_PLAY_DES',
-    'formtype'    => 'group_multi',
-    'valuetype'   => 'array',
-    'default'     => [XOOPS_GROUP_ADMIN, XOOPS_GROUP_USERS],
+	'name'        => 'groups_play',
+	'title'       => '_MI_CHESS_GROUPS_PLAY',
+	'description' => '_MI_CHESS_GROUPS_PLAY_DES',
+	'formtype'    => 'group_multi',
+	'valuetype'   => 'array',
+	'default'     => [XOOPS_GROUP_ADMIN, XOOPS_GROUP_USERS],
 ];
 
 $modversion['config'][] = [
-    'name'        => 'groups_delete',
-    'title'       => '_MI_CHESS_GROUPS_DELETE',
-    'description' => '_MI_CHESS_GROUPS_DELETE_DES',
-    'formtype'    => 'group_multi',
-    'valuetype'   => 'array',
-    'default'     => [],
+	'name'        => 'groups_delete',
+	'title'       => '_MI_CHESS_GROUPS_DELETE',
+	'description' => '_MI_CHESS_GROUPS_DELETE_DES',
+	'formtype'    => 'group_multi',
+	'valuetype'   => 'array',
+	'default'     => [],
 ];
 
 $modversion['config'][] = [
-    'name'        => 'allow_setup',
-    'title'       => '_MI_CHESS_ALLOW_SETUP',
-    'description' => '_MI_CHESS_ALLOW_SETUP_DES',
-    'formtype'    => 'yesno',
-    'valuetype'   => 'int',
-    'default'     => 0,
+	'name'        => 'allow_setup',
+	'title'       => '_MI_CHESS_ALLOW_SETUP',
+	'description' => '_MI_CHESS_ALLOW_SETUP_DES',
+	'formtype'    => 'yesno',
+	'valuetype'   => 'int',
+	'default'     => 0,
 ];
 
 $modversion['config'][] = [
-    'name'        => 'max_items',
-    'title'       => '_MI_CHESS_MAX_ITEMS',
-    'description' => '_MI_CHESS_MAX_ITEMS_DES',
-    'formtype'    => 'textbox',
-    'valuetype'   => 'int',
-    'default'     => 10,
+	'name'        => 'max_items',
+	'title'       => '_MI_CHESS_MAX_ITEMS',
+	'description' => '_MI_CHESS_MAX_ITEMS_DES',
+	'formtype'    => 'textbox',
+	'valuetype'   => 'int',
+	'default'     => 10,
 ];
 
 $modversion['config'][] = [
-    'name'        => 'rating_system',
-    'title'       => '_MI_CHESS_RATING_SYSTEM',
-    'description' => '_MI_CHESS_RATING_SYSTEM_DES',
-    'formtype'    => 'select',
-    'valuetype'   => 'text',
-    'options'     => [
-        _MI_CHESS_RATING_SYSTEM_NONE   => 'none',
-        _MI_CHESS_RATING_SYSTEM_CXR    => 'cxr',
-        _MI_CHESS_RATING_SYSTEM_LINEAR => 'linear',
-    ],
-    'default'     => 'cxr',
+	'name'        => 'rating_system',
+	'title'       => '_MI_CHESS_RATING_SYSTEM',
+	'description' => '_MI_CHESS_RATING_SYSTEM_DES',
+	'formtype'    => 'select',
+	'valuetype'   => 'text',
+	'options'     => [
+		_MI_CHESS_RATING_SYSTEM_NONE   => 'none',
+		_MI_CHESS_RATING_SYSTEM_CXR    => 'cxr',
+		_MI_CHESS_RATING_SYSTEM_LINEAR => 'linear',
+	],
+	'default'     => 'cxr',
 ];
 
 $modversion['config'][] = [
-    'name'        => 'initial_rating',
-    'title'       => '_MI_CHESS_INITIAL_RATING',
-    'description' => '_MI_CHESS_INITIAL_RATING_DES',
-    'formtype'    => 'textbox',
-    'valuetype'   => 'int',
-    'default'     => 1200,
+	'name'        => 'initial_rating',
+	'title'       => '_MI_CHESS_INITIAL_RATING',
+	'description' => '_MI_CHESS_INITIAL_RATING_DES',
+	'formtype'    => 'textbox',
+	'valuetype'   => 'int',
+	'default'     => 1200,
 ];
 
 $modversion['config'][] = [
-    'name'        => 'allow_unrated_games',
-    'title'       => '_MI_CHESS_ALLOW_UNRATED',
-    'description' => '_MI_CHESS_ALLOW_UNRATED_DES',
-    'formtype'    => 'yesno',
-    'valuetype'   => 'int',
-    'default'     => 1,
+	'name'        => 'allow_unrated_games',
+	'title'       => '_MI_CHESS_ALLOW_UNRATED',
+	'description' => '_MI_CHESS_ALLOW_UNRATED_DES',
+	'formtype'    => 'yesno',
+	'valuetype'   => 'int',
+	'default'     => 1,
 ];
 
 // Menu
@@ -214,83 +214,83 @@ 
 // Conditional menu items
 global $xoopsModule, $xoopsModuleConfig, $xoopsUser;
 if (is_object($xoopsModule) && 'chess' == $xoopsModule->getVar('dirname')) {
-    // Display create-game menu item if current user has the play-chess right.
+	// Display create-game menu item if current user has the play-chess right.
 
-    if (!empty($xoopsModuleConfig['groups_play']) && is_array($xoopsModuleConfig['groups_play'])
-        && (in_array(XOOPS_GROUP_ANONYMOUS, $xoopsModuleConfig['groups_play'])
-            || (is_object($xoopsUser) && count(array_intersect($xoopsUser->getGroups(), $xoopsModuleConfig['groups_play'])) > 0))) {
-        $modversion['sub'][3]['name'] = _MI_CHESS_SMNAME3;
-        $modversion['sub'][3]['url']  = 'create.php';
-    }
+	if (!empty($xoopsModuleConfig['groups_play']) && is_array($xoopsModuleConfig['groups_play'])
+		&& (in_array(XOOPS_GROUP_ANONYMOUS, $xoopsModuleConfig['groups_play'])
+			|| (is_object($xoopsUser) && count(array_intersect($xoopsUser->getGroups(), $xoopsModuleConfig['groups_play'])) > 0))) {
+		$modversion['sub'][3]['name'] = _MI_CHESS_SMNAME3;
+		$modversion['sub'][3]['url']  = 'create.php';
+	}
 
-    // Display ratings menu item if ratings system is enabled.
+	// Display ratings menu item if ratings system is enabled.
 
-    if ('none' != $xoopsModuleConfig['rating_system']) {
-        $modversion['sub'][4]['name'] = _MI_CHESS_SMNAME4;
-        $modversion['sub'][4]['url']  = 'ratings.php';
-    }
+	if ('none' != $xoopsModuleConfig['rating_system']) {
+		$modversion['sub'][4]['name'] = _MI_CHESS_SMNAME4;
+		$modversion['sub'][4]['url']  = 'ratings.php';
+	}
 
-    // Display my-games menu item if current user is logged in.
+	// Display my-games menu item if current user is logged in.
 
-    if (is_object($xoopsUser)) {
-        $modversion['sub'][5]['name'] = _MI_CHESS_SMNAME5;
-        $modversion['sub'][5]['url']  = 'player_stats.php?player_uid=' . $xoopsUser->getVar('uid');
-    }
+	if (is_object($xoopsUser)) {
+		$modversion['sub'][5]['name'] = _MI_CHESS_SMNAME5;
+		$modversion['sub'][5]['url']  = 'player_stats.php?player_uid=' . $xoopsUser->getVar('uid');
+	}
 }
 
 // Page Awareness
 $modversion['pages'][] = [
-    ['name' => _MI_CHESS_SMNAME1, 'url' => 'help.php'],
-    ['name' => _MI_CHESS_SMNAME2, 'url' => 'index.php'],
-    ['name' => _MI_CHESS_SMNAME3, 'url' => 'create.php'],
-    ['name' => _MI_CHESS_SMNAME4, 'url' => 'ratings.php'],
-    ['name' => _MI_CHESS_SMNAME5, 'url' => 'player_stats.php'],
+	['name' => _MI_CHESS_SMNAME1, 'url' => 'help.php'],
+	['name' => _MI_CHESS_SMNAME2, 'url' => 'index.php'],
+	['name' => _MI_CHESS_SMNAME3, 'url' => 'create.php'],
+	['name' => _MI_CHESS_SMNAME4, 'url' => 'ratings.php'],
+	['name' => _MI_CHESS_SMNAME5, 'url' => 'player_stats.php'],
 ];
 
 // Templates
 $modversion['templates'] = [
-    ['file' => 'chess_games.tpl', 'description' => _MI_CHESS_INDEX],
-    ['file' => 'chess_game_main.tpl', 'description' => _MI_CHESS_GAME],
-    ['file' => 'chess_game_moveform.tpl', 'description' => _MI_CHESS_MOVE_FORM],
-    ['file' => 'chess_game_promote_popup.tpl', 'description' => _MI_CHESS_PROMOTE_POPUP],
-    ['file' => 'chess_game_board.tpl', 'description' => _MI_CHESS_BOARD],
-    ['file' => 'chess_game_prefsform.tpl', 'description' => _MI_CHESS_PREFS_FORM],
-    ['file' => 'chess_game_arbitrateform.tpl', 'description' => _MI_CHESS_ARBITER_FORM],
-    ['file' => 'chess_help.tpl', 'description' => _MI_CHESS_HELP],
-    ['file' => 'chess_ratings.tpl', 'description' => _MI_CHESS_RATINGS],
-    ['file' => 'chess_player_stats.tpl', 'description' => _MI_CHESS_PLAYER_STATS],
+	['file' => 'chess_games.tpl', 'description' => _MI_CHESS_INDEX],
+	['file' => 'chess_game_main.tpl', 'description' => _MI_CHESS_GAME],
+	['file' => 'chess_game_moveform.tpl', 'description' => _MI_CHESS_MOVE_FORM],
+	['file' => 'chess_game_promote_popup.tpl', 'description' => _MI_CHESS_PROMOTE_POPUP],
+	['file' => 'chess_game_board.tpl', 'description' => _MI_CHESS_BOARD],
+	['file' => 'chess_game_prefsform.tpl', 'description' => _MI_CHESS_PREFS_FORM],
+	['file' => 'chess_game_arbitrateform.tpl', 'description' => _MI_CHESS_ARBITER_FORM],
+	['file' => 'chess_help.tpl', 'description' => _MI_CHESS_HELP],
+	['file' => 'chess_ratings.tpl', 'description' => _MI_CHESS_RATINGS],
+	['file' => 'chess_player_stats.tpl', 'description' => _MI_CHESS_PLAYER_STATS],
 ];
 // Blocks
 $modversion['blocks'][] = [
-    'file'        => 'blocks.php',
-    'name'        => _MI_CHESS_GAMES,
-    'description' => _MI_CHESS_GAMES_DES,
-    'show_func'   => 'b_chess_games_show',
-    // options: maximum number of games to display | 1=show games in play only/2=show concluded games only/3=show both | 1=show rated games only/2=show rated and unrated games
-    'options'     => '4|3',
-    'edit_func'   => 'b_chess_games_edit',
-    'template'    => 'chess_block_games.tpl',
+	'file'        => 'blocks.php',
+	'name'        => _MI_CHESS_GAMES,
+	'description' => _MI_CHESS_GAMES_DES,
+	'show_func'   => 'b_chess_games_show',
+	// options: maximum number of games to display | 1=show games in play only/2=show concluded games only/3=show both | 1=show rated games only/2=show rated and unrated games
+	'options'     => '4|3',
+	'edit_func'   => 'b_chess_games_edit',
+	'template'    => 'chess_block_games.tpl',
 ];
 
 $modversion['blocks'][] = [
-    'file'        => 'blocks.php',
-    'name'        => _MI_CHESS_CHALLENGES,
-    'description' => _MI_CHESS_CHALLENGES_DES,
-    'show_func'   => 'b_chess_challenges_show',
-    // options: maximum number of challenges to display | 1=show open challenges only/2=show individual challenges only/3=show both
-    'options'     => '4|3',
-    'edit_func'   => 'b_chess_challenges_edit',
-    'template'    => 'chess_block_challenges.tpl',
+	'file'        => 'blocks.php',
+	'name'        => _MI_CHESS_CHALLENGES,
+	'description' => _MI_CHESS_CHALLENGES_DES,
+	'show_func'   => 'b_chess_challenges_show',
+	// options: maximum number of challenges to display | 1=show open challenges only/2=show individual challenges only/3=show both
+	'options'     => '4|3',
+	'edit_func'   => 'b_chess_challenges_edit',
+	'template'    => 'chess_block_challenges.tpl',
 ];
 $modversion['blocks'][] = [
-    'file'        => 'blocks.php',
-    'name'        => _MI_CHESS_PLAYERS,
-    'description' => _MI_CHESS_PLAYERS_DES,
-    'show_func'   => 'b_chess_players_show',
-    // options: maximum number of players to display | 1=show non-provisional ratings only/2=show all ratings
-    'options'     => '4|1',
-    'edit_func'   => 'b_chess_players_edit',
-    'template'    => 'chess_block_players.tpl',
+	'file'        => 'blocks.php',
+	'name'        => _MI_CHESS_PLAYERS,
+	'description' => _MI_CHESS_PLAYERS_DES,
+	'show_func'   => 'b_chess_players_show',
+	// options: maximum number of players to display | 1=show non-provisional ratings only/2=show all ratings
+	'options'     => '4|1',
+	'edit_func'   => 'b_chess_players_edit',
+	'template'    => 'chess_block_players.tpl',
 ];
 
 // Notification
@@ -299,68 +299,68 @@ 
 $modversion['notification']['lookup_func'] = 'chess_notify_item_info';
 
 $modversion['notification']['category'][] = [
-    'name'           => 'game',
-    'title'          => _MI_CHESS_NCAT_GAME,
-    'description'    => _MI_CHESS_NCAT_GAME_DES,
-    'subscribe_from' => 'game.php',
-    'item_name'      => 'game_id',
-    'allow_bookmark' => 1,
+	'name'           => 'game',
+	'title'          => _MI_CHESS_NCAT_GAME,
+	'description'    => _MI_CHESS_NCAT_GAME_DES,
+	'subscribe_from' => 'game.php',
+	'item_name'      => 'game_id',
+	'allow_bookmark' => 1,
 ];
 
 $modversion['notification']['category'][] = [
-    'name'           => 'global',
-    'title'          => _MI_CHESS_NCAT_GLOBAL,
-    'description'    => _MI_CHESS_NCAT_GLOBAL_DES,
-    'subscribe_from' => ['index.php', 'game.php'],
+	'name'           => 'global',
+	'title'          => _MI_CHESS_NCAT_GLOBAL,
+	'description'    => _MI_CHESS_NCAT_GLOBAL_DES,
+	'subscribe_from' => ['index.php', 'game.php'],
 ];
 
 $modversion['notification']['event'][] = [
-    'name'          => 'notify_game_move',
-    'category'      => 'game',
-    'title'         => _MI_CHESS_NEVT_MOVE,
-    'caption'       => _MI_CHESS_NEVT_MOVE_CAP,
-    'description'   => _MI_CHESS_NEVT_MOVE_DES,
-    'mail_template' => 'notify_game_move',
-    'mail_subject'  => _MI_CHESS_NEVT_MOVE_SUB,
+	'name'          => 'notify_game_move',
+	'category'      => 'game',
+	'title'         => _MI_CHESS_NEVT_MOVE,
+	'caption'       => _MI_CHESS_NEVT_MOVE_CAP,
+	'description'   => _MI_CHESS_NEVT_MOVE_DES,
+	'mail_template' => 'notify_game_move',
+	'mail_subject'  => _MI_CHESS_NEVT_MOVE_SUB,
 ];
 
 $modversion['notification']['event'][] = [
-    'name'          => 'notify_challenge_user',
-    'category'      => 'global',
-    'title'         => _MI_CHESS_NEVT_CHAL_USER,
-    'caption'       => _MI_CHESS_NEVT_CHAL_USER_CAP,
-    'description'   => _MI_CHESS_NEVT_CHAL_USER_DES,
-    'mail_template' => 'notify_challenge_user',
-    'mail_subject'  => _MI_CHESS_NEVT_CHAL_USER_SUB,
+	'name'          => 'notify_challenge_user',
+	'category'      => 'global',
+	'title'         => _MI_CHESS_NEVT_CHAL_USER,
+	'caption'       => _MI_CHESS_NEVT_CHAL_USER_CAP,
+	'description'   => _MI_CHESS_NEVT_CHAL_USER_DES,
+	'mail_template' => 'notify_challenge_user',
+	'mail_subject'  => _MI_CHESS_NEVT_CHAL_USER_SUB,
 ];
 
 $modversion['notification']['event'][] = [
-    'name'          => 'notify_challenge_open',
-    'category'      => 'global',
-    'title'         => _MI_CHESS_NEVT_CHAL_OPEN,
-    'caption'       => _MI_CHESS_NEVT_CHAL_OPEN_CAP,
-    'description'   => _MI_CHESS_NEVT_CHAL_OPEN_DES,
-    'mail_template' => 'notify_challenge_open',
-    'mail_subject'  => _MI_CHESS_NEVT_CHAL_OPEN_SUB,
+	'name'          => 'notify_challenge_open',
+	'category'      => 'global',
+	'title'         => _MI_CHESS_NEVT_CHAL_OPEN,
+	'caption'       => _MI_CHESS_NEVT_CHAL_OPEN_CAP,
+	'description'   => _MI_CHESS_NEVT_CHAL_OPEN_DES,
+	'mail_template' => 'notify_challenge_open',
+	'mail_subject'  => _MI_CHESS_NEVT_CHAL_OPEN_SUB,
 ];
 
 $modversion['notification']['event'][] = [
-    'name'          => 'notify_accept_challenge',
-    'category'      => 'global',
-    'title'         => _MI_CHESS_NEVT_ACPT_CHAL,
-    'caption'       => _MI_CHESS_NEVT_ACPT_CHAL_CAP,
-    'description'   => _MI_CHESS_NEVT_ACPT_CHAL_DES,
-    'mail_template' => 'notify_accept_challenge',
-    'mail_subject'  => _MI_CHESS_NEVT_ACPT_CHAL_SUB,
+	'name'          => 'notify_accept_challenge',
+	'category'      => 'global',
+	'title'         => _MI_CHESS_NEVT_ACPT_CHAL,
+	'caption'       => _MI_CHESS_NEVT_ACPT_CHAL_CAP,
+	'description'   => _MI_CHESS_NEVT_ACPT_CHAL_DES,
+	'mail_template' => 'notify_accept_challenge',
+	'mail_subject'  => _MI_CHESS_NEVT_ACPT_CHAL_SUB,
 ];
 
 $modversion['notification']['event'][] = [
-    'name'          => 'notify_request_arbitration',
-    'category'      => 'global',
-    'title'         => _MI_CHESS_NEVT_RQST_ARBT,
-    'caption'       => _MI_CHESS_NEVT_RQST_ARBT_CAP,
-    'description'   => _MI_CHESS_NEVT_RQST_ARBT_DES,
-    'mail_template' => 'notify_request_arbitration',
-    'mail_subject'  => _MI_CHESS_NEVT_RQST_ARBT_SUB,
-    'admin_only'    => 1,
+	'name'          => 'notify_request_arbitration',
+	'category'      => 'global',
+	'title'         => _MI_CHESS_NEVT_RQST_ARBT,
+	'caption'       => _MI_CHESS_NEVT_RQST_ARBT_CAP,
+	'description'   => _MI_CHESS_NEVT_RQST_ARBT_DES,
+	'mail_template' => 'notify_request_arbitration',
+	'mail_subject'  => _MI_CHESS_NEVT_RQST_ARBT_SUB,
+	'admin_only'    => 1,
 ];

create.php

Indentation +551 added lines, -551 removed lines

@@ -56,7 +56,7 @@ 
 #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
@@ -87,16 +87,16 @@ 
 
 // 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/',
-        _CHESS_REDIRECT_DELAY_FAILURE,
-        _MD_CHESS_TOKEN_ERROR . '<br>' . implode('<br>', $GLOBALS['xoopsSecurity']->getErrors())
-    );
+	redirect_header(
+		XOOPS_URL . '/modules/chess/',
+		_CHESS_REDIRECT_DELAY_FAILURE,
+		_MD_CHESS_TOKEN_ERROR . '<br>' . implode('<br>', $GLOBALS['xoopsSecurity']->getErrors())
+	);
 }
 
 // If rating feature disabled, force ratings to off.
 if ('none' == chess_moduleConfig('rating_system')) {
-    $rated = 0;
+	$rated = 0;
 }
 
 $uid = is_object($xoopsUser) ? $xoopsUser->getVar('uid') : 0;
@@ -105,87 +105,87 @@ 
 $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);
+	chess_show_create_form1($gametype);
 } elseif ($cancel_challenge3) {
-    if (_CHESS_GAMETYPE_OPEN == $gametype || _CHESS_GAMETYPE_USER == $gametype) {
-        chess_show_create_form2($gametype, $fen);
-    } else {
-        chess_show_create_form1($gametype, $fen);
-    }
+	if (_CHESS_GAMETYPE_OPEN == $gametype || _CHESS_GAMETYPE_USER == $gametype) {
+		chess_show_create_form2($gametype, $fen);
+	} else {
+		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);
-    } elseif (_CHESS_GAMETYPE_OPEN == $gametype || _CHESS_GAMETYPE_USER == $gametype) {
-        chess_show_create_form2($gametype, $fen);
-    } else {
-        chess_show_create_form3($gametype, $fen, $coloroption, $opponent_uid, $rated);
-    }
+	$fen_error = chess_fen_error($fen);
+
+	if (!empty($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 {
+		chess_show_create_form3($gametype, $fen, $coloroption, $opponent_uid, $rated);
+	}
 } elseif ($submit_challenge2) {
-    if (_CHESS_GAMETYPE_USER == $gametype) {
-        if (empty($opponent)) {
-            chess_show_create_form2($gametype, $fen, $coloroption, $opponent_uid, $rated, _MD_CHESS_OPPONENT_MISSING);
-        } elseif (!$opponent_uid) {
-            chess_show_create_form2($gametype, $fen, $coloroption, $opponent_uid, $rated, _MD_CHESS_OPPONENT_INVALID);
-        } elseif ($opponent_uid == $uid) {
-            chess_show_create_form2($gametype, $fen, $coloroption, $opponent_uid, $rated, _MD_CHESS_OPPONENT_SELF);
-        } else {
-            chess_show_create_form3($gametype, $fen, $coloroption, $opponent_uid, $rated);
-        }
-    } else {
-        chess_show_create_form3($gametype, $fen, $coloroption, $opponent_uid, $rated);
-    }
+	if (_CHESS_GAMETYPE_USER == $gametype) {
+		if (empty($opponent)) {
+			chess_show_create_form2($gametype, $fen, $coloroption, $opponent_uid, $rated, _MD_CHESS_OPPONENT_MISSING);
+		} elseif (!$opponent_uid) {
+			chess_show_create_form2($gametype, $fen, $coloroption, $opponent_uid, $rated, _MD_CHESS_OPPONENT_INVALID);
+		} elseif ($opponent_uid == $uid) {
+			chess_show_create_form2($gametype, $fen, $coloroption, $opponent_uid, $rated, _MD_CHESS_OPPONENT_SELF);
+		} else {
+			chess_show_create_form3($gametype, $fen, $coloroption, $opponent_uid, $rated);
+		}
+	} else {
+		chess_show_create_form3($gametype, $fen, $coloroption, $opponent_uid, $rated);
+	}
 } elseif ($submit_challenge3) {
-    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);
-    } elseif (_CHESS_GAMETYPE_USER == $gametype) {
-        if (empty($opponent)) {
-            chess_show_create_form2($gametype, $fen, $coloroption, $opponent_uid, $rated, _MD_CHESS_OPPONENT_MISSING);
-        } elseif (!$opponent_uid) {
-            chess_show_create_form2($gametype, $fen, $coloroption, $opponent_uid, $rated, _MD_CHESS_OPPONENT_INVALID);
-        } elseif ($opponent_uid == $uid) {
-            chess_show_create_form2($gametype, $fen, $coloroption, $opponent_uid, $rated, _MD_CHESS_OPPONENT_SELF);
-        } 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);
-        }
-    } 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);
-    } else {
-        chess_show_create_form1($gametype, $fen, _MD_CHESS_GAMETYPE_INVALID);
-    }
+	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);
+	} elseif (_CHESS_GAMETYPE_USER == $gametype) {
+		if (empty($opponent)) {
+			chess_show_create_form2($gametype, $fen, $coloroption, $opponent_uid, $rated, _MD_CHESS_OPPONENT_MISSING);
+		} elseif (!$opponent_uid) {
+			chess_show_create_form2($gametype, $fen, $coloroption, $opponent_uid, $rated, _MD_CHESS_OPPONENT_INVALID);
+		} elseif ($opponent_uid == $uid) {
+			chess_show_create_form2($gametype, $fen, $coloroption, $opponent_uid, $rated, _MD_CHESS_OPPONENT_SELF);
+		} 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);
+		}
+	} 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);
+	} else {
+		chess_show_create_form1($gametype, $fen, _MD_CHESS_GAMETYPE_INVALID);
+	}
 } elseif ($submit_accept) {
-    chess_accept_challenge($challenge_id, $coloroption, $notify_move);
+	chess_accept_challenge($challenge_id, $coloroption, $notify_move);
 } elseif ($submit_delete) {
-    if ($confirm_delete) {
-        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);
-        } else {
-            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);
-    }
+	if ($confirm_delete) {
+		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);
+		} else {
+			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);
+	}
 } elseif ($challenge_id) {
-    if (($show_arbiter_ctrl && $is_arbiter) || chess_is_challenger($challenge_id)) {
-        chess_show_delete_form($challenge_id, $show_arbiter_ctrl && $is_arbiter);
-    } else {
-        chess_show_accept_form($challenge_id);
-    }
+	if (($show_arbiter_ctrl && $is_arbiter) || chess_is_challenger($challenge_id)) {
+		chess_show_delete_form($challenge_id, $show_arbiter_ctrl && $is_arbiter);
+	} else {
+		chess_show_accept_form($challenge_id);
+	}
 } else {
-    chess_show_create_form1($gametype);
+	chess_show_create_form1($gametype);
 }
 
 require_once XOOPS_ROOT_PATH . '/footer.php';
@@ -200,41 +200,41 @@ 
  */
 function chess_show_create_form1($gametype = _CHESS_GAMETYPE_OPEN, $fen = '', $error_msg = '')
 {
-    $form = new XoopsThemeForm(_MD_CHESS_CREATE_FORM, 'create_form1', 'create.php', 'post', true);
+	$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>'));
-    }
+	if ($error_msg) {
+		$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);
+	$menu_gametype->addOption(_CHESS_GAMETYPE_OPEN, _MD_CHESS_MENU_GAMETYPE_OPEN);
 
-    $menu_gametype->addOption(_CHESS_GAMETYPE_USER, _MD_CHESS_MENU_GAMETYPE_USER);
+	$menu_gametype->addOption(_CHESS_GAMETYPE_USER, _MD_CHESS_MENU_GAMETYPE_USER);
 
-    $menu_gametype->addOption(_CHESS_GAMETYPE_SELF, _MD_CHESS_MENU_GAMETYPE_SELF);
+	$menu_gametype->addOption(_CHESS_GAMETYPE_SELF, _MD_CHESS_MENU_GAMETYPE_SELF);
 
-    $form->addElement($menu_gametype);
+	$form->addElement($menu_gametype);
 
-    if (chess_moduleConfig('allow_setup')) {
-        $form->addElement(new XoopsFormLabel('', '&nbsp;'));
+	if (chess_moduleConfig('allow_setup')) {
+		$form->addElement(new XoopsFormLabel('', '&nbsp;'));
 
-        $form->addElement(new XoopsFormLabel('', _MD_CHESS_LABEL_FEN_EXPLAIN));
+		$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;'));
+	$form->addElement(new XoopsFormLabel('&nbsp;', '&nbsp;'));
 
-    $buttons = new XoopsFormElementTray('');
+	$buttons = new XoopsFormElementTray('');
 
-    $buttons->addElement(new XoopsFormButton('', 'submit_challenge1', _MD_CHESS_CREATE_SUBMIT, 'submit'));
+	$buttons->addElement(new XoopsFormButton('', 'submit_challenge1', _MD_CHESS_CREATE_SUBMIT, 'submit'));
 
-    $buttons->addElement(new XoopsFormButton('', 'cancel_challenge1', _MD_CHESS_CREATE_CANCEL, 'submit'));
+	$buttons->addElement(new XoopsFormButton('', 'cancel_challenge1', _MD_CHESS_CREATE_CANCEL, 'submit'));
 
-    $form->addElement($buttons);
+	$form->addElement($buttons);
 
-    $form->display();
+	$form->display();
 }
 
 /**
@@ -249,69 +249,69 @@ 
  */
 function chess_show_create_form2($gametype, $fen, $coloroption = _CHESS_COLOROPTION_OPPONENT, $opponent_uid = 0, $rated = 1, $error_msg = '')
 {
-    $form = new XoopsThemeForm(_MD_CHESS_CREATE_FORM, 'create_form2', 'create.php', 'post', true);
+	$form = new XoopsThemeForm(_MD_CHESS_CREATE_FORM, 'create_form2', 'create.php', 'post', true);
 
-    $form->addElement(new XoopsFormHidden('gametype', $gametype));
+	$form->addElement(new XoopsFormHidden('gametype', $gametype));
 
-    $form->addElement(new XoopsFormHidden('fen', $fen));
+	$form->addElement(new XoopsFormHidden('fen', $fen));
 
-    if ($error_msg) {
-        $form->addElement(new XoopsFormLabel(_MD_CHESS_ERROR . ':', '<div class="errorMsg">' . $error_msg . '</div>'));
-    }
+	if ($error_msg) {
+		$form->addElement(new XoopsFormLabel(_MD_CHESS_ERROR . ':', '<div class="errorMsg">' . $error_msg . '</div>'));
+	}
 
-    $memberHandler = xoops_getHandler('member');
+	$memberHandler = xoops_getHandler('member');
 
-    $opponent_user = $memberHandler->getUser($opponent_uid);
+	$opponent_user = $memberHandler->getUser($opponent_uid);
 
-    $opponent_username = is_object($opponent_user) ? $opponent_user->getVar('uname') : '';
+	$opponent_username = is_object($opponent_user) ? $opponent_user->getVar('uname') : '';
 
-    if (_CHESS_GAMETYPE_USER == $gametype) {
-        $form->addElement(
-            new XoopsFormText(
-                _MD_CHESS_LABEL_OPPONENT . ':', 'opponent', _CHESS_TEXTBOX_OPPONENT_SIZE, _CHESS_TEXTBOX_OPPONENT_MAXLEN, $opponent_username
-            )
-        );
-    }
+	if (_CHESS_GAMETYPE_USER == $gametype) {
+		$form->addElement(
+			new XoopsFormText(
+				_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);
+	$radio_color->addOption(_CHESS_COLOROPTION_OPPONENT, _MD_CHESS_RADIO_COLOR_OPPONENT);
 
-    $radio_color->addOption(_CHESS_COLOROPTION_RANDOM, _MD_CHESS_RADIO_COLOR_RANDOM);
+	$radio_color->addOption(_CHESS_COLOROPTION_RANDOM, _MD_CHESS_RADIO_COLOR_RANDOM);
 
-    $radio_color->addOption(_CHESS_COLOROPTION_WHITE, _MD_CHESS_RADIO_COLOR_WHITE);
+	$radio_color->addOption(_CHESS_COLOROPTION_WHITE, _MD_CHESS_RADIO_COLOR_WHITE);
 
-    $radio_color->addOption(_CHESS_COLOROPTION_BLACK, _MD_CHESS_RADIO_COLOR_BLACK);
+	$radio_color->addOption(_CHESS_COLOROPTION_BLACK, _MD_CHESS_RADIO_COLOR_BLACK);
 
-    $form->addElement($radio_color);
+	$form->addElement($radio_color);
 
-    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);
+	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->addOption(1, _YES);
+				$radio_rated->addOption(1, _YES);
 
-                $radio_rated->addOption(0, _NO);
+				$radio_rated->addOption(0, _NO);
 
-                $form->addElement($radio_rated);
-            } else {
-                $form->addElement(new XoopsFormHidden('rated', $rated));
-            }
-        }
-    } else {
-        $form->addElement(new XoopsFormHidden('rated', 0));
-    }
+				$form->addElement($radio_rated);
+			} else {
+				$form->addElement(new XoopsFormHidden('rated', $rated));
+			}
+		}
+	} else {
+		$form->addElement(new XoopsFormHidden('rated', 0));
+	}
 
-    $buttons = new XoopsFormElementTray('');
+	$buttons = new XoopsFormElementTray('');
 
-    $buttons->addElement(new XoopsFormButton('', 'submit_challenge2', _MD_CHESS_CREATE_SUBMIT, 'submit'));
+	$buttons->addElement(new XoopsFormButton('', 'submit_challenge2', _MD_CHESS_CREATE_SUBMIT, 'submit'));
 
-    $buttons->addElement(new XoopsFormButton('', 'cancel_challenge2', _MD_CHESS_CREATE_CANCEL, 'submit'));
+	$buttons->addElement(new XoopsFormButton('', 'cancel_challenge2', _MD_CHESS_CREATE_CANCEL, 'submit'));
 
-    $form->addElement($buttons);
+	$form->addElement($buttons);
 
-    $form->display();
+	$form->display();
 }
 
 /**
@@ -325,124 +325,124 @@ 
  */
 function chess_show_create_form3($gametype, $fen, $coloroption, $opponent_uid, $rated)
 {
-    $memberHandler = xoops_getHandler('member');
+	$memberHandler = xoops_getHandler('member');
 
-    $opponent_user = $memberHandler->getUser($opponent_uid);
+	$opponent_user = $memberHandler->getUser($opponent_uid);
 
-    $opponent_username = is_object($opponent_user) ? $opponent_user->getVar('uname') : '';
+	$opponent_username = is_object($opponent_user) ? $opponent_user->getVar('uname') : '';
 
-    $form = new XoopsThemeForm(_MD_CHESS_CREATE_FORM, 'create_form3', 'create.php', 'post', true);
+	$form = new XoopsThemeForm(_MD_CHESS_CREATE_FORM, 'create_form3', 'create.php', 'post', true);
 
-    $form->addElement(new XoopsFormHidden('gametype', $gametype));
+	$form->addElement(new XoopsFormHidden('gametype', $gametype));
 
-    $form->addElement(new XoopsFormHidden('fen', $fen));
+	$form->addElement(new XoopsFormHidden('fen', $fen));
 
-    $form->addElement(new XoopsFormHidden('opponent', $opponent_username));
+	$form->addElement(new XoopsFormHidden('opponent', $opponent_username));
 
-    $form->addElement(new XoopsFormHidden('coloroption', $coloroption));
+	$form->addElement(new XoopsFormHidden('coloroption', $coloroption));
 
-    $form->addElement(new XoopsFormHidden('rated', $rated));
+	$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:
-            $label_gametype = _MD_CHESS_LABEL_GAMETYPE_OPEN;
-            break;
-        case _CHESS_GAMETYPE_USER:
-            $label_gametype = _MD_CHESS_LABEL_GAMETYPE_USER;
-            break;
-        case _CHESS_GAMETYPE_SELF:
-            $label_gametype = _MD_CHESS_LABEL_GAMETYPE_SELF;
-            break;
-        default:
-            $label_gametype = _MD_CHESS_LABEL_ERROR;
-            break;
-    }
+	switch ($gametype) {
+		case _CHESS_GAMETYPE_OPEN:
+			$label_gametype = _MD_CHESS_LABEL_GAMETYPE_OPEN;
+			break;
+		case _CHESS_GAMETYPE_USER:
+			$label_gametype = _MD_CHESS_LABEL_GAMETYPE_USER;
+			break;
+		case _CHESS_GAMETYPE_SELF:
+			$label_gametype = _MD_CHESS_LABEL_GAMETYPE_SELF;
+			break;
+		default:
+			$label_gametype = _MD_CHESS_LABEL_ERROR;
+			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));
-    }
+	if (!empty($fen)) {
+		$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_FEN_SETUP . ':', $fen));
+	}
 
-    if (_CHESS_GAMETYPE_USER == $gametype) {
-        $memberHandler = xoops_getHandler('member');
+	if (_CHESS_GAMETYPE_USER == $gametype) {
+		$memberHandler = xoops_getHandler('member');
 
-        $opponent_user = $memberHandler->getUser($opponent_uid);
+		$opponent_user = $memberHandler->getUser($opponent_uid);
 
-        $opponent_username = is_object($opponent_user) ? $opponent_user->getVar('uname') : '';
+		$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) {
-        switch ($coloroption) {
-            case _CHESS_COLOROPTION_OPPONENT:
-                $label_coloroption = _MD_CHESS_RADIO_COLOR_OPPONENT;
-                break;
-            case _CHESS_COLOROPTION_RANDOM:
-                $label_coloroption = _MD_CHESS_RADIO_COLOR_RANDOM;
-                break;
-            case _CHESS_COLOROPTION_WHITE:
-                $label_coloroption = _MD_CHESS_RADIO_COLOR_WHITE;
-                break;
-            case _CHESS_COLOROPTION_BLACK:
-                $label_coloroption = _MD_CHESS_RADIO_COLOR_BLACK;
-                break;
-            default:
-                $label_coloroption = _MD_CHESS_LABEL_ERROR;
-                break;
-        }
+	if (_CHESS_GAMETYPE_OPEN == $gametype || _CHESS_GAMETYPE_USER == $gametype) {
+		switch ($coloroption) {
+			case _CHESS_COLOROPTION_OPPONENT:
+				$label_coloroption = _MD_CHESS_RADIO_COLOR_OPPONENT;
+				break;
+			case _CHESS_COLOROPTION_RANDOM:
+				$label_coloroption = _MD_CHESS_RADIO_COLOR_RANDOM;
+				break;
+			case _CHESS_COLOROPTION_WHITE:
+				$label_coloroption = _MD_CHESS_RADIO_COLOR_WHITE;
+				break;
+			case _CHESS_COLOROPTION_BLACK:
+				$label_coloroption = _MD_CHESS_RADIO_COLOR_BLACK;
+				break;
+			default:
+				$label_coloroption = _MD_CHESS_LABEL_ERROR;
+				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));
-        }
+		if ('none' != chess_moduleConfig('rating_system')) {
+			$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.
+		// Determine whether current user is subscribed to receive a notification when his challenge is accepted.
 
-        global $xoopsModule, $xoopsUser;
+		global $xoopsModule, $xoopsUser;
 
-        $uid = $xoopsUser->getVar('uid');
+		$uid = $xoopsUser->getVar('uid');
 
-        $mid = $xoopsModule->getVar('mid');
+		$mid = $xoopsModule->getVar('mid');
 
-        $notificationHandler = xoops_getHandler('notification');
+		$notificationHandler = xoops_getHandler('notification');
 
-        $is_subscribed = 0 != $notificationHandler->isSubscribed('global', 0, 'notify_accept_challenge', $mid, $uid);
+		$is_subscribed = 0 != $notificationHandler->isSubscribed('global', 0, 'notify_accept_challenge', $mid, $uid);
 
-        // Display checkbox with checked-state reflecting subscription status.
+		// Display checkbox with checked-state reflecting subscription status.
 
-        $checked_value = 1;
+		$checked_value = 1;
 
-        $checkbox_notify_acpt_chal = new XoopsFormCheckBox('', 'notify_accept', $is_subscribed ? $checked_value : null);
+		$checkbox_notify_acpt_chal = new XoopsFormCheckBox('', 'notify_accept', $is_subscribed ? $checked_value : null);
 
-        $checkbox_notify_acpt_chal->addOption($checked_value, _MD_CHESS_NEVT_ACPT_CHAL_CAP);
+		$checkbox_notify_acpt_chal->addOption($checked_value, _MD_CHESS_NEVT_ACPT_CHAL_CAP);
 
-        $form->addElement($checkbox_notify_acpt_chal);
+		$form->addElement($checkbox_notify_acpt_chal);
 
-        // Display checkbox, initially checked.
+		// Display checkbox, initially checked.
 
-        $checked_value = 1;
+		$checked_value = 1;
 
-        $checkbox_notify_move = new XoopsFormCheckBox('', 'notify_move', $checked_value);
+		$checkbox_notify_move = new XoopsFormCheckBox('', 'notify_move', $checked_value);
 
-        $checkbox_notify_move->addOption($checked_value, _MD_CHESS_NEVT_MOVE_CAP);
+		$checkbox_notify_move->addOption($checked_value, _MD_CHESS_NEVT_MOVE_CAP);
 
-        $form->addElement($checkbox_notify_move);
-    }
+		$form->addElement($checkbox_notify_move);
+	}
 
-    $buttons = new XoopsFormElementTray('');
+	$buttons = new XoopsFormElementTray('');
 
-    $buttons->addElement(new XoopsFormButton('', 'submit_challenge3', _MD_CHESS_CREATE_SUBMIT, 'submit'));
+	$buttons->addElement(new XoopsFormButton('', 'submit_challenge3', _MD_CHESS_CREATE_SUBMIT, 'submit'));
 
-    $buttons->addElement(new XoopsFormButton('', 'cancel_challenge3', _MD_CHESS_CREATE_CANCEL, 'submit'));
+	$buttons->addElement(new XoopsFormButton('', 'cancel_challenge3', _MD_CHESS_CREATE_CANCEL, 'submit'));
 
-    $form->addElement($buttons);
+	$form->addElement($buttons);
 
-    $form->display();
+	$form->display();
 }
 
 /**
@@ -452,125 +452,125 @@ 
  */
 function chess_show_accept_form($challenge_id)
 {
-    global $xoopsDB, $xoopsUser;
+	global $xoopsDB, $xoopsUser;
 
-    $challenges_table = $xoopsDB->prefix('chess_challenges');
+	$challenges_table = $xoopsDB->prefix('chess_challenges');
 
-    $result = $xoopsDB->query(
-        trim(
-            "
+	$result = $xoopsDB->query(
+		trim(
+			"
         SELECT game_type, fen, color_option, player1_uid, player2_uid, UNIX_TIMESTAMP(create_date) as create_date, is_rated
         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);
-    }
+	if ($xoopsDB->getRowsNum($result) < 1) {
+		redirect_header(XOOPS_URL . '/modules/chess/', _CHESS_REDIRECT_DELAY_FAILURE, _MD_CHESS_GAME_NOT_FOUND);
+	}
 
-    $row = $xoopsDB->fetchArray($result);
+	$row = $xoopsDB->fetchArray($result);
 
-    $xoopsDB->freeRecordSet($result);
+	$xoopsDB->freeRecordSet($result);
 
-    $form = new XoopsThemeForm(_MD_CHESS_ACCEPT_FORM, 'accept_form', 'create.php', 'post', true);
+	$form = new XoopsThemeForm(_MD_CHESS_ACCEPT_FORM, 'accept_form', 'create.php', 'post', true);
 
-    $form->addElement(new XoopsFormHidden('challenge_id', $challenge_id));
+	$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');
+	$memberHandler = xoops_getHandler('member');
 
-    switch ($row['game_type']) {
-        case _CHESS_GAMETYPE_OPEN:
-            $label_gametype = _MD_CHESS_LABEL_GAMETYPE_OPEN;
-            break;
-        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;
-            break;
-        case _CHESS_GAMETYPE_SELF:
-            $label_gametype = _MD_CHESS_LABEL_GAMETYPE_SELF;
-            break;
-        default:
-            $label_gametype = _MD_CHESS_LABEL_ERROR;
-            break;
-    }
+	switch ($row['game_type']) {
+		case _CHESS_GAMETYPE_OPEN:
+			$label_gametype = _MD_CHESS_LABEL_GAMETYPE_OPEN;
+			break;
+		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;
+			break;
+		case _CHESS_GAMETYPE_SELF:
+			$label_gametype = _MD_CHESS_LABEL_GAMETYPE_SELF;
+			break;
+		default:
+			$label_gametype = _MD_CHESS_LABEL_ERROR;
+			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_user = $memberHandler->getUser($row['player1_uid']);
 
-    $player1_username = is_object($player1_user) ? $player1_user->getVar('uname') : '?';
+	$player1_username = is_object($player1_user) ? $player1_user->getVar('uname') : '?';
 
-    $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>"
-        )
-    );
+	$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>"
+		)
+	);
 
-    $player2_username = $xoopsUser ? $xoopsUser->getVar('uname') : '?';
+	$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']));
-    }
+	if (!empty($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);
+	if (_CHESS_COLOROPTION_OPPONENT == $row['color_option']) {
+		$radio_color = new XoopsFormRadio(_MD_CHESS_LABEL_COLOR . ':', 'coloroption', _CHESS_COLOROPTION_RANDOM);
 
-        $radio_color->addOption(_CHESS_COLOROPTION_RANDOM, _MD_CHESS_RADIO_COLOR_RANDOM);
+		$radio_color->addOption(_CHESS_COLOROPTION_RANDOM, _MD_CHESS_RADIO_COLOR_RANDOM);
 
-        $radio_color->addOption(_CHESS_COLOROPTION_WHITE, _MD_CHESS_RADIO_COLOR_WHITE);
+		$radio_color->addOption(_CHESS_COLOROPTION_WHITE, _MD_CHESS_RADIO_COLOR_WHITE);
 
-        $radio_color->addOption(_CHESS_COLOROPTION_BLACK, _MD_CHESS_RADIO_COLOR_BLACK);
+		$radio_color->addOption(_CHESS_COLOROPTION_BLACK, _MD_CHESS_RADIO_COLOR_BLACK);
 
-        $form->addElement($radio_color);
-    } else {
-        switch ($row['color_option']) {
-            case _CHESS_COLOROPTION_RANDOM:
-                $label_coloroption = _MD_CHESS_RADIO_COLOR_RANDOM;
-                break;
-            case _CHESS_COLOROPTION_WHITE:
-                $label_coloroption = _MD_CHESS_RADIO_COLOR_BLACK; // player1 white, player2 black
-                break;
-            case _CHESS_COLOROPTION_BLACK:
-                $label_coloroption = _MD_CHESS_RADIO_COLOR_WHITE; // player1 black, player2 white
-                break;
-            default:
-                $label_coloroption = _MD_CHESS_LABEL_ERROR;
-                break;
-        }
+		$form->addElement($radio_color);
+	} else {
+		switch ($row['color_option']) {
+			case _CHESS_COLOROPTION_RANDOM:
+				$label_coloroption = _MD_CHESS_RADIO_COLOR_RANDOM;
+				break;
+			case _CHESS_COLOROPTION_WHITE:
+				$label_coloroption = _MD_CHESS_RADIO_COLOR_BLACK; // player1 white, player2 black
+				break;
+			case _CHESS_COLOROPTION_BLACK:
+				$label_coloroption = _MD_CHESS_RADIO_COLOR_WHITE; // player1 black, player2 white
+				break;
+			default:
+				$label_coloroption = _MD_CHESS_LABEL_ERROR;
+				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));
-    }
+	if ('none' != chess_moduleConfig('rating_system')) {
+		$form->addElement(new XoopsFormLabel(_MD_CHESS_RATED_GAME . ':', $row['is_rated'] ? _YES : _NO));
+	}
 
-    // Display notification-subscribe checkbox, initially checked.
+	// Display notification-subscribe checkbox, initially checked.
 
-    $checked_value = 1;
+	$checked_value = 1;
 
-    $checkbox_notify_move = new XoopsFormCheckBox('', 'notify_move', $checked_value);
+	$checkbox_notify_move = new XoopsFormCheckBox('', 'notify_move', $checked_value);
 
-    $checkbox_notify_move->addOption($checked_value, _MD_CHESS_NEVT_MOVE_CAP);
+	$checkbox_notify_move->addOption($checked_value, _MD_CHESS_NEVT_MOVE_CAP);
 
-    $form->addElement($checkbox_notify_move);
+	$form->addElement($checkbox_notify_move);
 
-    $tray = new XoopsFormElementTray('');
+	$tray = new XoopsFormElementTray('');
 
-    $tray->addElement(new XoopsFormButton('', 'submit_accept', _MD_CHESS_CREATE_ACCEPT, 'submit'));
+	$tray->addElement(new XoopsFormButton('', 'submit_accept', _MD_CHESS_CREATE_ACCEPT, 'submit'));
 
-    $tray->addElement(new XoopsFormButton('', 'cancel_accept', _MD_CHESS_CREATE_CANCEL, 'submit'));
+	$tray->addElement(new XoopsFormButton('', 'cancel_accept', _MD_CHESS_CREATE_CANCEL, 'submit'));
 
-    $form->addElement($tray);
+	$form->addElement($tray);
 
-    $form->display();
+	$form->display();
 }
 
 /**
@@ -582,114 +582,114 @@ 
  */
 function chess_show_delete_form($challenge_id, $show_arbiter_ctrl, $error_msg = '')
 {
-    global $xoopsDB;
+	global $xoopsDB;
 
-    $challenges_table = $xoopsDB->prefix('chess_challenges');
+	$challenges_table = $xoopsDB->prefix('chess_challenges');
 
-    $result = $xoopsDB->query(
-        trim(
-            "
+	$result = $xoopsDB->query(
+		trim(
+			"
         SELECT game_type, fen, color_option, player1_uid, player2_uid, UNIX_TIMESTAMP(create_date) as create_date, is_rated
         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);
-    }
+	if ($xoopsDB->getRowsNum($result) < 1) {
+		redirect_header(XOOPS_URL . '/modules/chess/', _CHESS_REDIRECT_DELAY_FAILURE, _MD_CHESS_GAME_NOT_FOUND);
+	}
 
-    $row = $xoopsDB->fetchArray($result);
+	$row = $xoopsDB->fetchArray($result);
 
-    $xoopsDB->freeRecordSet($result);
+	$xoopsDB->freeRecordSet($result);
 
-    $form = new XoopsThemeForm(_MD_CHESS_DELETE_FORM, 'delete_form', 'create.php', 'post', true);
+	$form = new XoopsThemeForm(_MD_CHESS_DELETE_FORM, 'delete_form', 'create.php', 'post', true);
 
-    $form->addElement(new XoopsFormHidden('challenge_id', $challenge_id));
+	$form->addElement(new XoopsFormHidden('challenge_id', $challenge_id));
 
-    if ($show_arbiter_ctrl) {
-        $form->addElement(new XoopsFormHidden('show_arbiter_ctrl', $show_arbiter_ctrl));
-    }
+	if ($show_arbiter_ctrl) {
+		$form->addElement(new XoopsFormHidden('show_arbiter_ctrl', $show_arbiter_ctrl));
+	}
 
-    if ($error_msg) {
-        $form->addElement(new XoopsFormLabel(_MD_CHESS_ERROR . ': ', '<div class="errorMsg">' . $error_msg . '</div>'));
-    }
+	if ($error_msg) {
+		$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');
+	$memberHandler = xoops_getHandler('member');
 
-    switch ($row['game_type']) {
-        case _CHESS_GAMETYPE_OPEN:
-            $label_gametype = _MD_CHESS_LABEL_GAMETYPE_OPEN;
-            break;
-        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;
-            break;
-        case _CHESS_GAMETYPE_SELF:
-            $label_gametype = _MD_CHESS_LABEL_GAMETYPE_SELF;
-            break;
-        default:
-            $label_gametype = _MD_CHESS_LABEL_ERROR;
-            break;
-    }
+	switch ($row['game_type']) {
+		case _CHESS_GAMETYPE_OPEN:
+			$label_gametype = _MD_CHESS_LABEL_GAMETYPE_OPEN;
+			break;
+		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;
+			break;
+		case _CHESS_GAMETYPE_SELF:
+			$label_gametype = _MD_CHESS_LABEL_GAMETYPE_SELF;
+			break;
+		default:
+			$label_gametype = _MD_CHESS_LABEL_ERROR;
+			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_user = $memberHandler->getUser($row['player1_uid']);
 
-    $player1_username = is_object($player1_user) ? $player1_user->getVar('uname') : '?';
+	$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']));
-    }
+	if (!empty($row['fen'])) {
+		$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_FEN_SETUP . ':', $row['fen']));
+	}
 
-    switch ($row['color_option']) {
-        case _CHESS_COLOROPTION_OPPONENT:
-            $label_coloroption = _MD_CHESS_RADIO_COLOR_OPPONENT;
-            break;
-        case _CHESS_COLOROPTION_RANDOM:
-            $label_coloroption = _MD_CHESS_RADIO_COLOR_RANDOM;
-            break;
-        case _CHESS_COLOROPTION_WHITE:
-            $label_coloroption = _MD_CHESS_RADIO_COLOR_WHITE;
-            break;
-        case _CHESS_COLOROPTION_BLACK:
-            $label_coloroption = _MD_CHESS_RADIO_COLOR_BLACK;
-            break;
-        default:
-            $label_coloroption = _MD_CHESS_LABEL_ERROR;
-            break;
-    }
+	switch ($row['color_option']) {
+		case _CHESS_COLOROPTION_OPPONENT:
+			$label_coloroption = _MD_CHESS_RADIO_COLOR_OPPONENT;
+			break;
+		case _CHESS_COLOROPTION_RANDOM:
+			$label_coloroption = _MD_CHESS_RADIO_COLOR_RANDOM;
+			break;
+		case _CHESS_COLOROPTION_WHITE:
+			$label_coloroption = _MD_CHESS_RADIO_COLOR_WHITE;
+			break;
+		case _CHESS_COLOROPTION_BLACK:
+			$label_coloroption = _MD_CHESS_RADIO_COLOR_BLACK;
+			break;
+		default:
+			$label_coloroption = _MD_CHESS_LABEL_ERROR;
+			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));
-    }
+	if ('none' != chess_moduleConfig('rating_system')) {
+		$form->addElement(new XoopsFormLabel(_MD_CHESS_RATED_GAME . ':', $row['is_rated'] ? _YES : _NO));
+	}
 
-    // Display confirm-delete checkbox, initially unchecked.
+	// Display confirm-delete checkbox, initially unchecked.
 
-    $checked_value = 1;
+	$checked_value = 1;
 
-    $checkbox_confirm_delete = new XoopsFormCheckBox('', 'confirm_delete', !$checked_value);
+	$checkbox_confirm_delete = new XoopsFormCheckBox('', 'confirm_delete', !$checked_value);
 
-    $checkbox_confirm_delete->addOption($checked_value, _MD_CHESS_CONFIRM_DELETE);
+	$checkbox_confirm_delete->addOption($checked_value, _MD_CHESS_CONFIRM_DELETE);
 
-    $tray = new XoopsFormElementTray('');
+	$tray = new XoopsFormElementTray('');
 
-    $tray->addElement(new XoopsFormButton('', 'submit_delete', _MD_CHESS_CREATE_DELETE, 'submit'));
+	$tray->addElement(new XoopsFormButton('', 'submit_delete', _MD_CHESS_CREATE_DELETE, 'submit'));
 
-    $tray->addElement($checkbox_confirm_delete);
+	$tray->addElement($checkbox_confirm_delete);
 
-    $form->addElement($tray);
+	$form->addElement($tray);
 
-    $form->display();
+	$form->display();
 }
 
 /**
@@ -702,100 +702,100 @@ 
  */
 function chess_accept_challenge($challenge_id, $coloroption, $notify_move_player2 = false)
 {
-    global $xoopsDB, $xoopsUser;
+	global $xoopsDB, $xoopsUser;
 
-    $challenges_table = $xoopsDB->prefix('chess_challenges');
+	$challenges_table = $xoopsDB->prefix('chess_challenges');
 
-    $result = $xoopsDB->query(
-        trim(
-            "
+	$result = $xoopsDB->query(
+		trim(
+			"
         SELECT game_type, fen, color_option, notify_move_player1, player1_uid, player2_uid, UNIX_TIMESTAMP(create_date) as create_date, is_rated
         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);
-    }
-
-    $row = $xoopsDB->fetchArray($result);
-
-    $xoopsDB->freeRecordSet($result);
-
-    $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);
-    } elseif ($uid == $row['player1_uid']) {
-        redirect_header(XOOPS_URL . '/modules/chess/', _CHESS_REDIRECT_DELAY_FAILURE, _MD_CHESS_SAME_PLAYER2);
-    }
-
-    switch ($row['color_option']) {
-        case _CHESS_COLOROPTION_OPPONENT:
-            switch ($coloroption) {
-                default:
-                case _CHESS_COLOROPTION_RANDOM:
-                    if (1 == random_int(1, 2)) {
-                        $white_uid = $row['player1_uid'];
-                        $black_uid = $uid;
-                    } else {
-                        $white_uid = $uid;
-                        $black_uid = $row['player1_uid'];
-                    }
-                    break;
-                case _CHESS_COLOROPTION_WHITE:
-                    $white_uid = $uid;
-                    $black_uid = $row['player1_uid'];
-                    break;
-                case _CHESS_COLOROPTION_BLACK:
-                    $white_uid = $row['player1_uid'];
-                    $black_uid = $uid;
-                    break;
-            }
-            break;
-        default:
-        case _CHESS_COLOROPTION_RANDOM:
-            if (1 == random_int(1, 2)) {
-                $white_uid = $row['player1_uid'];
-
-                $black_uid = $uid;
-            } else {
-                $white_uid = $uid;
-
-                $black_uid = $row['player1_uid'];
-            }
-            break;
-        case _CHESS_COLOROPTION_WHITE:
-            $white_uid = $row['player1_uid'];
-            $black_uid = $uid;
-            break;
-        case _CHESS_COLOROPTION_BLACK:
-            $white_uid = $uid;
-            $black_uid = $row['player1_uid'];
-            break;
-    }
-
-    $game_id = chess_create_game($white_uid, $black_uid, $row['fen'], $row['is_rated'], $row['notify_move_player1'] ? $row['player1_uid'] : 0, $notify_move_player2);
-
-    $xoopsDB->query("DELETE FROM $challenges_table WHERE challenge_id = '$challenge_id'");
-
-    if ($xoopsDB->errno()) {
-        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).
-
-    $player2_username = $xoopsUser ? $xoopsUser->getVar('uname') : '*unknown*';
-
-    $notificationHandler = xoops_getHandler('notification');
-
-    $extra_tags = ['CHESS_PLAYER' => $player2_username, 'CHESS_GAME_ID' => $game_id];
-
-    $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);
+		)
+	);
+
+	if ($xoopsDB->getRowsNum($result) < 1) {
+		redirect_header(XOOPS_URL . '/modules/chess/', _CHESS_REDIRECT_DELAY_FAILURE, _MD_CHESS_GAME_NOT_FOUND);
+	}
+
+	$row = $xoopsDB->fetchArray($result);
+
+	$xoopsDB->freeRecordSet($result);
+
+	$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);
+	} elseif ($uid == $row['player1_uid']) {
+		redirect_header(XOOPS_URL . '/modules/chess/', _CHESS_REDIRECT_DELAY_FAILURE, _MD_CHESS_SAME_PLAYER2);
+	}
+
+	switch ($row['color_option']) {
+		case _CHESS_COLOROPTION_OPPONENT:
+			switch ($coloroption) {
+				default:
+				case _CHESS_COLOROPTION_RANDOM:
+					if (1 == random_int(1, 2)) {
+						$white_uid = $row['player1_uid'];
+						$black_uid = $uid;
+					} else {
+						$white_uid = $uid;
+						$black_uid = $row['player1_uid'];
+					}
+					break;
+				case _CHESS_COLOROPTION_WHITE:
+					$white_uid = $uid;
+					$black_uid = $row['player1_uid'];
+					break;
+				case _CHESS_COLOROPTION_BLACK:
+					$white_uid = $row['player1_uid'];
+					$black_uid = $uid;
+					break;
+			}
+			break;
+		default:
+		case _CHESS_COLOROPTION_RANDOM:
+			if (1 == random_int(1, 2)) {
+				$white_uid = $row['player1_uid'];
+
+				$black_uid = $uid;
+			} else {
+				$white_uid = $uid;
+
+				$black_uid = $row['player1_uid'];
+			}
+			break;
+		case _CHESS_COLOROPTION_WHITE:
+			$white_uid = $row['player1_uid'];
+			$black_uid = $uid;
+			break;
+		case _CHESS_COLOROPTION_BLACK:
+			$white_uid = $uid;
+			$black_uid = $row['player1_uid'];
+			break;
+	}
+
+	$game_id = chess_create_game($white_uid, $black_uid, $row['fen'], $row['is_rated'], $row['notify_move_player1'] ? $row['player1_uid'] : 0, $notify_move_player2);
+
+	$xoopsDB->query("DELETE FROM $challenges_table WHERE challenge_id = '$challenge_id'");
+
+	if ($xoopsDB->errno()) {
+		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).
+
+	$player2_username = $xoopsUser ? $xoopsUser->getVar('uname') : '*unknown*';
+
+	$notificationHandler = xoops_getHandler('notification');
+
+	$extra_tags = ['CHESS_PLAYER' => $player2_username, 'CHESS_GAME_ID' => $game_id];
+
+	$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);
 }
 
 /**
@@ -806,23 +806,23 @@ 
  */
 function chess_is_challenger($challenge_id)
 {
-    global $xoopsDB, $xoopsUser;
+	global $xoopsDB, $xoopsUser;
 
-    $challenges_table = $xoopsDB->prefix('chess_challenges');
+	$challenges_table = $xoopsDB->prefix('chess_challenges');
 
-    $result = $xoopsDB->query("SELECT player1_uid FROM $challenges_table WHERE challenge_id = '$challenge_id'");
+	$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);
-    }
+	if ($xoopsDB->getRowsNum($result) < 1) {
+		redirect_header(XOOPS_URL . '/modules/chess/', _CHESS_REDIRECT_DELAY_FAILURE, _MD_CHESS_GAME_NOT_FOUND);
+	}
 
-    $row = $xoopsDB->fetchArray($result);
+	$row = $xoopsDB->fetchArray($result);
 
-    $xoopsDB->freeRecordSet($result);
+	$xoopsDB->freeRecordSet($result);
 
-    $uid = is_object($xoopsUser) ? $xoopsUser->getVar('uid') : 0;
+	$uid = is_object($xoopsUser) ? $xoopsUser->getVar('uid') : 0;
 
-    return $uid == $row['player1_uid'];
+	return $uid == $row['player1_uid'];
 }
 
 /**
@@ -838,27 +838,27 @@ 
  */
 function chess_create_challenge($gametype, $fen, $coloroption, $rated, $notify_accept, $notify_move_player1, $opponent_uid = 0)
 {
-    #$where = __CLASS__ . '::' . __FUNCTION__;#*#DEBUG#
+	#$where = __CLASS__ . '::' . __FUNCTION__;#*#DEBUG#
 
-    #echo "In $where\n";#*#DEBUG#
+	#echo "In $where\n";#*#DEBUG#
 
-    global $xoopsUser;
+	global $xoopsUser;
 
-    $uid = is_object($xoopsUser) ? $xoopsUser->getVar('uid') : 0;
+	$uid = is_object($xoopsUser) ? $xoopsUser->getVar('uid') : 0;
 
-    $myts = MyTextSanitizer::getInstance();
+	$myts = MyTextSanitizer::getInstance();
 
-    $fen_q = $myts->addSlashes($fen);
+	$fen_q = $myts->addSlashes($fen);
 
-    #trigger_error("inserting new game'", E_USER_NOTICE);#*#DEBUG#
+	#trigger_error("inserting new game'", E_USER_NOTICE);#*#DEBUG#
 
-    global $xoopsDB;
+	global $xoopsDB;
 
-    $table = $xoopsDB->prefix('chess_challenges');
+	$table = $xoopsDB->prefix('chess_challenges');
 
-    $xoopsDB->query(
-        trim(
-            "
+	$xoopsDB->query(
+		trim(
+			"
         INSERT INTO $table
         SET
             game_type           = '$gametype',
@@ -870,36 +870,36 @@ 
             create_date         = NOW(),
             is_rated            = '$rated'
     "
-        )
-    );
+		)
+	);
 
-    if ($xoopsDB->errno()) {
-        trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
-    }
+	if ($xoopsDB->errno()) {
+		trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
+	}
 
-    $challenge_id = $xoopsDB->getInsertId();
+	$challenge_id = $xoopsDB->getInsertId();
 
-    $notificationHandler = xoops_getHandler('notification');
+	$notificationHandler = xoops_getHandler('notification');
 
-    // Update the challenger's subscription for receiving a notification when the challenge is accepted.
+	// Update the challenger's subscription for receiving a notification when the challenge is accepted.
 
-    if ($notify_accept) {
-        $notificationHandler->subscribe('global', 0, 'notify_accept_challenge');
-    } else {
-        $notificationHandler->unsubscribe('global', 0, 'notify_accept_challenge');
-    }
+	if ($notify_accept) {
+		$notificationHandler->subscribe('global', 0, 'notify_accept_challenge');
+	} else {
+		$notificationHandler->unsubscribe('global', 0, 'notify_accept_challenge');
+	}
 
-    // Notify any subscribers that a challenge has been offered.
+	// Notify any subscribers that a challenge has been offered.
 
-    $player1_username = $xoopsUser ? $xoopsUser->getVar('uname') : '*unknown*';
+	$player1_username = $xoopsUser ? $xoopsUser->getVar('uname') : '*unknown*';
 
-    $extra_tags = ['CHESS_CHALLENGER' => $player1_username, 'CHESS_CHALLENGE_ID' => $challenge_id];
+	$extra_tags = ['CHESS_CHALLENGER' => $player1_username, 'CHESS_CHALLENGE_ID' => $challenge_id];
 
-    if (_CHESS_GAMETYPE_OPEN == $gametype) {
-        $notificationHandler->triggerEvent('global', 0, 'notify_challenge_open', $extra_tags);
-    } elseif (_CHESS_GAMETYPE_USER == $gametype) {
-        $notificationHandler->triggerEvent('global', 0, 'notify_challenge_user', $extra_tags, [$opponent_uid]);
-    }
+	if (_CHESS_GAMETYPE_OPEN == $gametype) {
+		$notificationHandler->triggerEvent('global', 0, 'notify_challenge_open', $extra_tags);
+	} elseif (_CHESS_GAMETYPE_USER == $gametype) {
+		$notificationHandler->triggerEvent('global', 0, 'notify_challenge_user', $extra_tags, [$opponent_uid]);
+	}
 }
 
 /**
@@ -909,15 +909,15 @@ 
  */
 function chess_delete_challenge($challenge_id)
 {
-    global $xoopsDB;
+	global $xoopsDB;
 
-    $table = $xoopsDB->prefix('chess_challenges');
+	$table = $xoopsDB->prefix('chess_challenges');
 
-    $xoopsDB->query("DELETE FROM $table WHERE challenge_id='$challenge_id'");
+	$xoopsDB->query("DELETE FROM $table WHERE challenge_id='$challenge_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);
+	}
 }
 
 /**
@@ -933,33 +933,33 @@ 
  */
 function chess_create_game($white_uid, $black_uid, $fen, $rated, $notify_move_player1_uid = 0, $notify_move_player2 = false)
 {
-    #$where = __CLASS__ . '::' . __FUNCTION__;#*#DEBUG#
+	#$where = __CLASS__ . '::' . __FUNCTION__;#*#DEBUG#
 
-    #echo "In $where\n";#*#DEBUG#
+	#echo "In $where\n";#*#DEBUG#
 
-    #var_dump('white_uid', $white_uid, 'black_uid', $black_uid);#*#DEBUG#
+	#var_dump('white_uid', $white_uid, 'black_uid', $black_uid);#*#DEBUG#
 
-    $chessgame = new Chess\ChessGame($fen);
+	$chessgame = new Chess\ChessGame($fen);
 
-    empty($chessgame->error) or trigger_error('chessgame invalid', E_USER_ERROR);
+	empty($chessgame->error) or trigger_error('chessgame invalid', E_USER_ERROR);
 
-    $gamestate = $chessgame->gamestate();
+	$gamestate = $chessgame->gamestate();
 
-    is_array($gamestate) or trigger_error('gamestate invalid', E_USER_ERROR);
+	is_array($gamestate) or trigger_error('gamestate invalid', E_USER_ERROR);
 
-    #trigger_error("inserting new game'", E_USER_NOTICE);#*#DEBUG#
+	#trigger_error("inserting new game'", E_USER_NOTICE);#*#DEBUG#
 
-    $myts = MyTextSanitizer::getInstance();
+	$myts = MyTextSanitizer::getInstance();
 
-    $fen_q = $myts->addSlashes($fen);
+	$fen_q = $myts->addSlashes($fen);
 
-    global $xoopsDB;
+	global $xoopsDB;
 
-    $table = $xoopsDB->prefix('chess_games');
+	$table = $xoopsDB->prefix('chess_games');
 
-    $xoopsDB->query(
-        trim(
-            "
+	$xoopsDB->query(
+		trim(
+			"
         INSERT INTO $table
         SET
             white_uid                    = '$white_uid',
@@ -978,30 +978,30 @@ 
             pgn_movetext                 = '{$gamestate['pgn_movetext']}',
             is_rated                     = '$rated'
     "
-        )
-    );
+		)
+	);
 
-    if ($xoopsDB->errno()) {
-        trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
-    }
+	if ($xoopsDB->errno()) {
+		trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
+	}
 
-    $game_id = $xoopsDB->getInsertId();
+	$game_id = $xoopsDB->getInsertId();
 
-    $notificationHandler = xoops_getHandler('notification');
+	$notificationHandler = xoops_getHandler('notification');
 
-    // If requested, subscribe the challenger to receive a notification when a new move is made.
+	// If requested, subscribe the challenger to receive a notification when a new move is made.
 
-    if ($notify_move_player1_uid) {
-        $notificationHandler->subscribe('game', $game_id, 'notify_game_move', null, null, $notify_move_player1_uid);
-    }
+	if ($notify_move_player1_uid) {
+		$notificationHandler->subscribe('game', $game_id, 'notify_game_move', null, null, $notify_move_player1_uid);
+	}
 
-    // If requested, subscribe the accepter to receive a notification when a new move is made.
+	// If requested, subscribe the accepter to receive a notification when a new move is made.
 
-    if ($notify_move_player2) {
-        $notificationHandler->subscribe('game', $game_id, 'notify_game_move');
-    }
+	if ($notify_move_player2) {
+		$notificationHandler->subscribe('game', $game_id, 'notify_game_move');
+	}
 
-    return $game_id;
+	return $game_id;
 }
 
 /**
@@ -1012,15 +1012,15 @@ 
  */
 function chess_fen_error($fen)
 {
-    if (!empty($fen)) {
-        $chessgame = new Chess\ChessGame($fen);
+	if (!empty($fen)) {
+		$chessgame = new Chess\ChessGame($fen);
 
-        $fen_error = $chessgame->error;
-    } else {
-        $fen_error = '';
-    }
+		$fen_error = $chessgame->error;
+	} else {
+		$fen_error = '';
+	}
 
-    return $fen_error;
+	return $fen_error;
 }
 
 /**
@@ -1033,9 +1033,9 @@ 
  */
 function chess_opponent_uid($username)
 {
-    $uid = chess_uname_to_uid($username);
+	$uid = chess_uname_to_uid($username);
 
-    $can_play = $uid > 0 ? chess_can_play($uid) : false;
+	$can_play = $uid > 0 ? chess_can_play($uid) : false;
 
-    return $can_play ? $uid : 0;
+	return $can_play ? $uid : 0;
 }

include/notification.inc.php

Indentation +33 added lines, -33 removed lines

@@ -40,9 +40,9 @@ 
 // but not when viewing the notifications page via notifications.php.
 global $xoopsConfig;
 if (file_exists(XOOPS_ROOT_PATH . "/modules/chess/language/{$xoopsConfig['language']}/main.php")) {
-    require_once XOOPS_ROOT_PATH . "/modules/chess/language/{$xoopsConfig['language']}/main.php";
+	require_once XOOPS_ROOT_PATH . "/modules/chess/language/{$xoopsConfig['language']}/main.php";
 } else {
-    require_once XOOPS_ROOT_PATH . '/modules/chess/language/english/main.php';
+	require_once XOOPS_ROOT_PATH . '/modules/chess/language/english/main.php';
 }
 /**#@-*/
 
@@ -57,57 +57,57 @@ 
  */
 function chess_notify_item_info($category, $item_id)
 {
-    if ('global' == $category) {
-        $item['name'] = 'Chess';
+	if ('global' == $category) {
+		$item['name'] = 'Chess';
 
-        $item['url'] = XOOPS_URL . '/modules/chess/';
+		$item['url'] = XOOPS_URL . '/modules/chess/';
 
-        return $item;
-    } elseif ('game' == $category) {
-        global $xoopsDB;
+		return $item;
+	} elseif ('game' == $category) {
+		global $xoopsDB;
 
-        $table = $xoopsDB->prefix('chess_games');
+		$table = $xoopsDB->prefix('chess_games');
 
-        $result = $xoopsDB->query(
-            trim(
-                "
+		$result = $xoopsDB->query(
+			trim(
+				"
             SELECT white_uid, black_uid, UNIX_TIMESTAMP(start_date) AS start_date
             FROM   $table
             WHERE  game_id = '$item_id'
         "
-            )
-        );
+			)
+		);
 
-        $gamedata = $xoopsDB->fetchArray($result);
+		$gamedata = $xoopsDB->fetchArray($result);
 
-        $xoopsDB->freeRecordSet($result);
+		$xoopsDB->freeRecordSet($result);
 
-        if (false !== $gamedata) {
-            // get mapping of user IDs to usernames
+		if (false !== $gamedata) {
+			// get mapping of user IDs to usernames
 
-            $criteria = new \Criteria('uid', "({$gamedata['white_uid']}, {$gamedata['black_uid']})", 'IN');
+			$criteria = new \Criteria('uid', "({$gamedata['white_uid']}, {$gamedata['black_uid']})", 'IN');
 
-            $memberHandler = xoops_getHandler('member');
+			$memberHandler = xoops_getHandler('member');
 
-            $usernames = $memberHandler->getUserList($criteria);
+			$usernames = $memberHandler->getUserList($criteria);
 
-            $username_white = $usernames[$gamedata['white_uid']] ?? _MD_CHESS_NA;
+			$username_white = $usernames[$gamedata['white_uid']] ?? _MD_CHESS_NA;
 
-            $username_black = $usernames[$gamedata['black_uid']] ?? _MD_CHESS_NA;
+			$username_black = $usernames[$gamedata['black_uid']] ?? _MD_CHESS_NA;
 
-            $date = $gamedata['start_date'] ? date('Y.m.d', $gamedata['start_date']) : _MD_CHESS_NA;
-        } else {
-            $username_white = _MD_CHESS_NA;
+			$date = $gamedata['start_date'] ? date('Y.m.d', $gamedata['start_date']) : _MD_CHESS_NA;
+		} else {
+			$username_white = _MD_CHESS_NA;
 
-            $username_black = _MD_CHESS_NA;
+			$username_black = _MD_CHESS_NA;
 
-            $date = _MD_CHESS_NA;
-        }
+			$date = _MD_CHESS_NA;
+		}
 
-        $item['name'] = "$username_white " . _MD_CHESS_LABEL_VS . " $username_black ($date)";
+		$item['name'] = "$username_white " . _MD_CHESS_LABEL_VS . " $username_black ($date)";
 
-        $item['url'] = XOOPS_URL . '/modules/chess/game.php?game_id=' . $item_id;
+		$item['url'] = XOOPS_URL . '/modules/chess/game.php?game_id=' . $item_id;
 
-        return $item;
-    }
+		return $item;
+	}
 }

include/ratings_cxr.inc.php

Indentation +60 added lines, -60 removed lines

@@ -44,72 +44,72 @@ 
  */
 function chess_ratings_adj_cxr($white_rating, $white_games, $black_rating, $black_games, $pgn_result)
 {
-    // compute score: +1 for win, 0 for draw, -1 for loss
-    switch ($pgn_result) {
-        case '1-0':
-            $S = 1;
-            break;
-        case '1/2-1/2':
-        default: // should not occur
-            $S = 0;
-            break;
-        case '0-1':
-            $S = -1;
-            break;
-    }
+	// compute score: +1 for win, 0 for draw, -1 for loss
+	switch ($pgn_result) {
+		case '1-0':
+			$S = 1;
+			break;
+		case '1/2-1/2':
+		default: // should not occur
+			$S = 0;
+			break;
+		case '0-1':
+			$S = -1;
+			break;
+	}
 
-    if (($white_games < 5 && $black_games < 5) || ($white_games > 5 && $black_games > 5)) {
-        // Formula 1: Rnew = Rold + (S x 21) + (Ropponent - Rold) / 25
-        $w_new = ($S * 21) + ($black_rating - $white_rating) / 25;
-        $b_new = (-$S * 21) + ($white_rating - $black_rating) / 25;
-    } elseif ($white_games > 5 && $black_games < 5) {
-        // Formula 2: Rnew = Rold + (S x 6) + (Ropponent - Rold) / 100
-        $w_new = ($S * 6) + ($black_rating - $white_rating) / 100;
+	if (($white_games < 5 && $black_games < 5) || ($white_games > 5 && $black_games > 5)) {
+		// Formula 1: Rnew = Rold + (S x 21) + (Ropponent - Rold) / 25
+		$w_new = ($S * 21) + ($black_rating - $white_rating) / 25;
+		$b_new = (-$S * 21) + ($white_rating - $black_rating) / 25;
+	} elseif ($white_games > 5 && $black_games < 5) {
+		// Formula 2: Rnew = Rold + (S x 6) + (Ropponent - Rold) / 100
+		$w_new = ($S * 6) + ($black_rating - $white_rating) / 100;
 
-        // Formula 3: Rnew = (4/5) x Rold + (1/5) x Ropponent + (S x 80)
-        $b_new = ($white_rating / 5) + ($S * -80) - ($black_rating / 5);
-    } else {
-        // Formula 2: Rnew = Rold + (S x 6) + (Ropponent - Rold) / 100
-        $b_new = ($S * 6) + ($white_rating - $black_rating) / 100;
+		// Formula 3: Rnew = (4/5) x Rold + (1/5) x Ropponent + (S x 80)
+		$b_new = ($white_rating / 5) + ($S * -80) - ($black_rating / 5);
+	} else {
+		// Formula 2: Rnew = Rold + (S x 6) + (Ropponent - Rold) / 100
+		$b_new = ($S * 6) + ($white_rating - $black_rating) / 100;
 
-        // Formula 3: Rnew = (4/5) x Rold + (1/5) x Ropponent + (S x 80)
-        $w_new = ($black_rating / 5) + ($S * -80) - ($white_rating / 5);
-    }
+		// Formula 3: Rnew = (4/5) x Rold + (1/5) x Ropponent + (S x 80)
+		$w_new = ($black_rating / 5) + ($S * -80) - ($white_rating / 5);
+	}
 
-    // Rule R1: The winning rated player must gain at least two points.
-    // Rule R2: The losing rated player must lose at least two points.
-    if (abs($w_new) < 2) {
-        $w_new = $S * 2;
-    }
-    if (abs($b_new) < 2) {
-        $b_new = $S * -2;
-    }
+	// Rule R1: The winning rated player must gain at least two points.
+	// Rule R2: The losing rated player must lose at least two points.
+	if (abs($w_new) < 2) {
+		$w_new = $S * 2;
+	}
+	if (abs($b_new) < 2) {
+		$b_new = $S * -2;
+	}
 
-    // Rule R3: The rated player must not gain nor lose more than 41 points.
-    if (abs($w_new) > 41) {
-        $w_new = $S * 41;
-    }
-    if (abs($b_new) > 41) {
-        $b_new = $S * -41;
-    }
+	// Rule R3: The rated player must not gain nor lose more than 41 points.
+	if (abs($w_new) > 41) {
+		$w_new = $S * 41;
+	}
+	if (abs($b_new) > 41) {
+		$b_new = $S * -41;
+	}
 
-    if (1 == $S) {
-        if ($white_games < 5 && $w_new < 0) {
-            $w_new = 2;
-        }
-        if ($black_games < 5 && $b_new > 0) {
-            $b_new = -2;
-        }
-    } elseif (-1 == $S) {
-        if ($white_games < 5 && $w_new > 0) {
-            $w_new = -2;
-        }
-        if ($black_games < 5 && $b_new < 0) {
-            $b_new = 2;
-        }
-    }
+	if (1 == $S) {
+		if ($white_games < 5 && $w_new < 0) {
+			$w_new = 2;
+		}
+		if ($black_games < 5 && $b_new > 0) {
+			$b_new = -2;
+		}
+	} elseif (-1 == $S) {
+		if ($white_games < 5 && $w_new > 0) {
+			$w_new = -2;
+		}
+		if ($black_games < 5 && $b_new < 0) {
+			$b_new = 2;
+		}
+	}
 
-    return [$white_rating + $w_new, $black_rating + $b_new];
+	return [$white_rating + $w_new, $black_rating + $b_new];
 }
 
 /**
@@ -119,7 +119,7 @@ 
  */
 function chess_ratings_num_provisional_games_cxr()
 {
-    return 5;
+	return 5;
 }
 
 ?>