mambax7 / chess

blocks/blocks.php

Indentation +186 added lines, -186 removed lines

@@ -48,36 +48,36 @@ 
  */
 function b_chess_games_show($options)
 {
-    global $xoopsModule, $xoopsDB;
+	global $xoopsModule, $xoopsDB;
 
-    // don't display this block within owning module
+	// don't display this block within owning module
 
-    if (is_object($xoopsModule) && 'chess' == $xoopsModule->getVar('dirname')) {
-        return [];
-    }
+	if (is_object($xoopsModule) && 'chess' == $xoopsModule->getVar('dirname')) {
+		return [];
+	}
 
-    $table = $xoopsDB->prefix('chess_games');
+	$table = $xoopsDB->prefix('chess_games');
 
-    $limit = (int)$options[0]; // sanitize with intval()
+	$limit = (int)$options[0]; // sanitize with intval()
 
-    $where = 'white_uid != black_uid';
+	$where = 'white_uid != black_uid';
 
-    switch ($options[1]) {
-        case 1:
-            $where .= " AND pgn_result = '*'";
-            break;
-        case 2:
-            $where .= " AND pgn_result != '*'";
-            break;
-    }
+	switch ($options[1]) {
+		case 1:
+			$where .= " AND pgn_result = '*'";
+			break;
+		case 2:
+			$where .= " AND pgn_result != '*'";
+			break;
+	}
 
-    if (1 == $options[2]) {
-        $where .= " AND is_rated = '1'";
-    }
+	if (1 == $options[2]) {
+		$where .= " AND is_rated = '1'";
+	}
 
-    $result = $xoopsDB->query(
-        trim(
-            "
+	$result = $xoopsDB->query(
+		trim(
+			"
 		SELECT   game_id, fen_active_color, white_uid, black_uid, pgn_result, UNIX_TIMESTAMP(create_date) AS create_date,
 		         UNIX_TIMESTAMP(start_date) AS start_date, UNIX_TIMESTAMP(last_date) AS last_date,
 		         UNIX_TIMESTAMP(GREATEST(create_date,start_date,last_date)) AS most_recent_date
@@ -86,59 +86,59 @@ 
 		ORDER BY most_recent_date DESC
 		LIMIT    $limit
 	"
-        )
-    );
+		)
+	);
 
-    // user IDs that will require mapping to usernames
+	// user IDs that will require mapping to usernames
 
-    $userids = [];
+	$userids = [];
 
-    $games = [];
+	$games = [];
 
-    while (false !== ($row = $xoopsDB->fetchArray($result))) {
-        $games[] = [
-            'game_id'          => $row['game_id'],
-            'white_uid'        => $row['white_uid'],
-            'black_uid'        => $row['black_uid'],
-            'date'             => $row['most_recent_date'],
-            'fen_active_color' => $row['fen_active_color'],
-            'pgn_result'       => $row['pgn_result'],
-        ];
+	while (false !== ($row = $xoopsDB->fetchArray($result))) {
+		$games[] = [
+			'game_id'          => $row['game_id'],
+			'white_uid'        => $row['white_uid'],
+			'black_uid'        => $row['black_uid'],
+			'date'             => $row['most_recent_date'],
+			'fen_active_color' => $row['fen_active_color'],
+			'pgn_result'       => $row['pgn_result'],
+		];
 
-        // 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);
 
-    // 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]['username_white'] = $usernames[$game['white_uid']] ?? '?';
+	foreach ($games as $k => $game) {
+		$games[$k]['username_white'] = $usernames[$game['white_uid']] ?? '?';
 
-        $games[$k]['username_black'] = $usernames[$game['black_uid']] ?? '?';
-    }
+		$games[$k]['username_black'] = $usernames[$game['black_uid']] ?? '?';
+	}
 
-    $block['games'] = $games;
+	$block['games'] = $games;
 
-    $block['date_format'] = _SHORTDATESTRING;
+	$block['date_format'] = _SHORTDATESTRING;
 
-    return $block;
+	return $block;
 }
 
 /**
@@ -149,91 +149,91 @@ 
  */
 function b_chess_challenges_show($options)
 {
-    global $xoopsModule, $xoopsDB;
+	global $xoopsModule, $xoopsDB;
 
-    // don't display this block within owning module
+	// don't display this block within owning module
 
-    if (is_object($xoopsModule) && 'chess' == $xoopsModule->getVar('dirname')) {
-        return [];
-    }
+	if (is_object($xoopsModule) && 'chess' == $xoopsModule->getVar('dirname')) {
+		return [];
+	}
 
-    $table = $xoopsDB->prefix('chess_challenges');
+	$table = $xoopsDB->prefix('chess_challenges');
 
-    $limit = (int)$options[0]; // sanitize with intval()
+	$limit = (int)$options[0]; // sanitize with intval()
 
-    switch ($options[1]) {
-        case 1:
-            $where = "game_type = 'open'";
-            break;
-        case 2:
-            $where = "game_type = 'user'";
-            break;
-        default:
-            $where = 1;
-            break;
-    }
+	switch ($options[1]) {
+		case 1:
+			$where = "game_type = 'open'";
+			break;
+		case 2:
+			$where = "game_type = 'user'";
+			break;
+		default:
+			$where = 1;
+			break;
+	}
 
-    $result = $xoopsDB->query(
-        trim(
-            "
+	$result = $xoopsDB->query(
+		trim(
+			"
 		SELECT   challenge_id, game_type, player1_uid, player2_uid, UNIX_TIMESTAMP(create_date) AS create_date
 		FROM     $table
 		WHERE    $where
 		ORDER BY create_date DESC
 		LIMIT    $limit
 	"
-        )
-    );
+		)
+	);
 
-    // user IDs that will require mapping to usernames
+	// user IDs that will require mapping to usernames
 
-    $userids = [];
+	$userids = [];
 
-    $challenges = [];
+	$challenges = [];
 
-    while (false !== ($row = $xoopsDB->fetchArray($result))) {
-        $challenges[] = [
-            'challenge_id' => $row['challenge_id'],
-            'game_type'    => $row['game_type'],
-            'player1_uid'  => $row['player1_uid'],
-            'player2_uid'  => $row['player2_uid'],
-            'create_date'  => $row['create_date'],
-        ];
+	while (false !== ($row = $xoopsDB->fetchArray($result))) {
+		$challenges[] = [
+			'challenge_id' => $row['challenge_id'],
+			'game_type'    => $row['game_type'],
+			'player1_uid'  => $row['player1_uid'],
+			'player2_uid'  => $row['player2_uid'],
+			'create_date'  => $row['create_date'],
+		];
 
-        // 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);
 
-    // 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 $challenges
+	// add usernames to $challenges
 
-    foreach ($challenges as $k => $challenge) {
-        $challenges[$k]['username_player1'] = $usernames[$challenge['player1_uid']] ?? '?';
+	foreach ($challenges as $k => $challenge) {
+		$challenges[$k]['username_player1'] = $usernames[$challenge['player1_uid']] ?? '?';
 
-        $challenges[$k]['username_player2'] = $usernames[$challenge['player2_uid']] ?? '?';
-    }
+		$challenges[$k]['username_player2'] = $usernames[$challenge['player2_uid']] ?? '?';
+	}
 
-    $block['challenges'] = $challenges;
+	$block['challenges'] = $challenges;
 
-    $block['date_format'] = _SHORTDATESTRING;
+	$block['date_format'] = _SHORTDATESTRING;
 
-    return $block;
+	return $block;
 }
 
 /**
@@ -244,103 +244,103 @@ 
  */
 function b_chess_players_show($options)
 {
-    global $xoopsModule, $xoopsDB;
+	global $xoopsModule, $xoopsDB;
 
-    // don't display this block within owning module
+	// don't display this block within owning module
 
-    if (is_object($xoopsModule) && 'chess' == $xoopsModule->getVar('dirname')) {
-        return [];
-    }
+	if (is_object($xoopsModule) && 'chess' == $xoopsModule->getVar('dirname')) {
+		return [];
+	}
 
-    require_once XOOPS_ROOT_PATH . '/modules/chess/include/ratings.php';
+	require_once XOOPS_ROOT_PATH . '/modules/chess/include/ratings.php';
 
-    $moduleHandler = xoops_getHandler('module');
+	$moduleHandler = xoops_getHandler('module');
 
-    $module = $moduleHandler->getByDirname('chess');
+	$module = $moduleHandler->getByDirname('chess');
 
-    $configHandler = xoops_getHandler('config');
+	$configHandler = xoops_getHandler('config');
 
-    $moduleConfig = $configHandler->getConfigsByCat(0, $module->getVar('mid'));
+	$moduleConfig = $configHandler->getConfigsByCat(0, $module->getVar('mid'));
 
-    $block['rating_system'] = $moduleConfig['rating_system'];
+	$block['rating_system'] = $moduleConfig['rating_system'];
 
-    $block['provisional_games'] = chess_ratings_num_provisional_games();
+	$block['provisional_games'] = chess_ratings_num_provisional_games();
 
-    // if ratings disabled, nothing else to do
+	// if ratings disabled, nothing else to do
 
-    if ('none' == $moduleConfig['rating_system']) {
-        return $block;
-    }
+	if ('none' == $moduleConfig['rating_system']) {
+		return $block;
+	}
 
-    $table = $xoopsDB->prefix('chess_ratings');
+	$table = $xoopsDB->prefix('chess_ratings');
 
-    $limit = (int)$options[0]; // sanitize with intval()
+	$limit = (int)$options[0]; // sanitize with intval()
 
-    switch ($options[1]) {
-        case 1:
-            $block['show_provisional_ratings'] = false;
-            $where                             = "(games_won+games_lost+games_drawn) >= '{$block['provisional_games']}'";
-            break;
-        case 2:
-        default:
-            $block['show_provisional_ratings'] = true;
-            $where                             = 1;
-            break;
-    }
+	switch ($options[1]) {
+		case 1:
+			$block['show_provisional_ratings'] = false;
+			$where                             = "(games_won+games_lost+games_drawn) >= '{$block['provisional_games']}'";
+			break;
+		case 2:
+		default:
+			$block['show_provisional_ratings'] = true;
+			$where                             = 1;
+			break;
+	}
 
-    $result = $xoopsDB->query(
-        trim(
-            "
+	$result = $xoopsDB->query(
+		trim(
+			"
 		SELECT   player_uid, rating, (games_won+games_lost+games_drawn) AS games_played
 		FROM     $table
 		WHERE    $where
 		ORDER BY rating DESC, player_uid ASC
 		LIMIT    $limit
 	"
-        )
-    );
+		)
+	);
 
-    // user IDs that will require mapping to usernames
+	// user IDs that will require mapping to usernames
 
-    $userids = [];
+	$userids = [];
 
-    $players = [];
+	$players = [];
 
-    while (false !== ($row = $xoopsDB->fetchArray($result))) {
-        $players[] = [
-            'player_uid'   => $row['player_uid'],
-            'rating'       => $row['rating'],
-            'games_played' => $row['games_played'],
-        ];
+	while (false !== ($row = $xoopsDB->fetchArray($result))) {
+		$players[] = [
+			'player_uid'   => $row['player_uid'],
+			'rating'       => $row['rating'],
+			'games_played' => $row['games_played'],
+		];
 
-        // save user IDs that will require mapping to usernames
+		// save user IDs that will require mapping to usernames
 
-        if ($row['player_uid']) {
-            $userids[$row['player_uid']] = 1;
-        }
-    }
+		if ($row['player_uid']) {
+			$userids[$row['player_uid']] = 1;
+		}
+	}
 
-    $xoopsDB->freeRecordSet($result);
+	$xoopsDB->freeRecordSet($result);
 
-    // get mapping of user IDs to usernames
+	// get mapping of user IDs to usernames
 
-    if (!empty($userids)) {
-        $memberHandler = xoops_getHandler('member');
+	if (!empty($userids)) {
+		$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 $players
+	// add usernames to $players
 
-    foreach ($players as $k => $player) {
-        $players[$k]['player_uname'] = $usernames[$player['player_uid']] ?? '?';
-    }
+	foreach ($players as $k => $player) {
+		$players[$k]['player_uname'] = $usernames[$player['player_uid']] ?? '?';
+	}
 
-    $block['players'] = $players;
+	$block['players'] = $players;
 
-    return $block;
+	return $block;
 }
 
 /**
@@ -351,17 +351,17 @@ 
  */
 function b_chess_games_edit($options)
 {
-    $show_inplay = 1 == $options[1] ? 'checked' : '';
+	$show_inplay = 1 == $options[1] ? 'checked' : '';
 
-    $show_concluded = 2 == $options[1] ? 'checked' : '';
+	$show_concluded = 2 == $options[1] ? 'checked' : '';
 
-    $show_both = 3 == $options[1] ? 'checked' : '';
+	$show_both = 3 == $options[1] ? 'checked' : '';
 
-    $show_rated_only = 1 == $options[2] ? 'checked' : '';
+	$show_rated_only = 1 == $options[2] ? 'checked' : '';
 
-    $show_unrated = 2 == $options[2] ? 'checked' : '';
+	$show_unrated = 2 == $options[2] ? 'checked' : '';
 
-    return '
+	return '
 		' . _MB_CHESS_NUM_GAMES . ": <input type='text' name='options[0]' value='{$options[0]}' size='3' maxlength='3'>
 		<br>
 		<br>
@@ -383,13 +383,13 @@ 
  */
 function b_chess_challenges_edit($options)
 {
-    $show_open = 1 == $options[1] ? 'checked' : '';
+	$show_open = 1 == $options[1] ? 'checked' : '';
 
-    $show_user = 2 == $options[1] ? 'checked' : '';
+	$show_user = 2 == $options[1] ? 'checked' : '';
 
-    $show_both = 3 == $options[1] ? 'checked' : '';
+	$show_both = 3 == $options[1] ? 'checked' : '';
 
-    return '
+	return '
 		' . _MB_CHESS_NUM_CHALLENGES . ": <input type='text' name='options[0]' value='{$options[0]}' size='3' maxlength='3'>
 		<br>
 		<input type='radio' name='options[1]' value='1' $show_open> " . _MB_CHESS_SHOW_CHALLENGES_OPEN . "
@@ -406,11 +406,11 @@ 
  */
 function b_chess_players_edit($options)
 {
-    $show_nonprovisional = 1 == $options[1] ? 'checked' : '';
+	$show_nonprovisional = 1 == $options[1] ? 'checked' : '';
 
-    $show_all = 2 == $options[1] ? 'checked' : '';
+	$show_all = 2 == $options[1] ? 'checked' : '';
 
-    return '
+	return '
 		' . _MB_CHESS_NUM_PLAYERS . ": <input type='text' name='options[0]' value='{$options[0]}' size='3' maxlength='3'>
 		<br>
 		<input type='radio' name='options[1]' value='1' $show_nonprovisional> " . _MB_CHESS_SHOW_NONPROVISIONAL . "

index.php

Indentation +179 added lines, -179 removed lines

@@ -47,98 +47,98 @@ 
 
 #var_dump($_REQUEST);#*#DEBUG#
 
-    global $xoopsDB, $xoopsTpl;
+	global $xoopsDB, $xoopsTpl;
 
-    // ----------
+	// ----------
 
-    // user input
+	// user input
 
-    // ----------
+	// ----------
 
-    // offset of first row of challenges table to display (default to 0)
+	// offset of first row of challenges table to display (default to 0)
 
-    $cstart = (int)($_POST['cstart'] ?? @$_GET['cstart']);
+	$cstart = (int)($_POST['cstart'] ?? @$_GET['cstart']);
 
-    // offset of first row of games table to display (default to 0)
+	// offset of first row of games table to display (default to 0)
 
-    $gstart = (int)($_POST['gstart'] ?? @$_GET['gstart']);
+	$gstart = (int)($_POST['gstart'] ?? @$_GET['gstart']);
 
-    // challenges display option
+	// challenges display option
 
-    $cshow = (int)($_POST['cshow'] ?? @$_GET['cshow']);
+	$cshow = (int)($_POST['cshow'] ?? @$_GET['cshow']);
 
-    // games display option 1
+	// games display option 1
 
-    $gshow1 = (int)($_POST['gshow1'] ?? @$_GET['gshow1']);
+	$gshow1 = (int)($_POST['gshow1'] ?? @$_GET['gshow1']);
 
-    // games display option 2
+	// games display option 2
 
-    $gshow2 = (int)($_POST['gshow2'] ?? @$_GET['gshow2']);
+	$gshow2 = (int)($_POST['gshow2'] ?? @$_GET['gshow2']);
 
-    // set show-options to default if undefined
+	// set show-options to default if undefined
 
-    if (!$cshow) {
-        $cshow = _CHESS_SHOW_CHALLENGES_BOTH;
-    }
+	if (!$cshow) {
+		$cshow = _CHESS_SHOW_CHALLENGES_BOTH;
+	}
 
-    if (!$gshow1) {
-        $gshow1 = _CHESS_SHOW_GAMES_BOTH;
-    }
+	if (!$gshow1) {
+		$gshow1 = _CHESS_SHOW_GAMES_BOTH;
+	}
 
-    if (!$gshow2) {
-        $gshow2 = _CHESS_SHOW_GAMES_UNRATED;
-    }
+	if (!$gshow2) {
+		$gshow2 = _CHESS_SHOW_GAMES_UNRATED;
+	}
 
-    // 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);
 
-    $xoopsTpl->assign('chess_date_format', _MEDIUMDATESTRING);
+	$xoopsTpl->assign('chess_date_format', _MEDIUMDATESTRING);
 
-    // user IDs that will require mapping to usernames
+	// user IDs that will require mapping to usernames
 
-    $userids = [];
+	$userids = [];
 
-    // -----
+	// -----
 
-    // games
+	// 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.
 
-    $games_table = $xoopsDB->prefix('chess_games');
+	$games_table = $xoopsDB->prefix('chess_games');
 
-    $where = 'white_uid != black_uid';
+	$where = 'white_uid != black_uid';
 
-    switch ($gshow1) {
-        case 1:
-            $where .= " AND pgn_result = '*'";
-            break;
-        case 2:
-            $where .= " AND pgn_result != '*'";
-            break;
-    }
+	switch ($gshow1) {
+		case 1:
+			$where .= " AND pgn_result = '*'";
+			break;
+		case 2:
+			$where .= " AND pgn_result != '*'";
+			break;
+	}
 
-    if (1 == $gshow2) {
-        $where .= " AND is_rated = '1'";
-    }
+	if (1 == $gshow2) {
+		$where .= " AND is_rated = '1'";
+	}
 
-    $result = $xoopsDB->query("SELECT COUNT(*) FROM $games_table WHERE $where");
+	$result = $xoopsDB->query("SELECT COUNT(*) FROM $games_table WHERE $where");
 
-    [$num_games] = $xoopsDB->fetchRow($result);
+	[$num_games] = $xoopsDB->fetchRow($result);
 
-    $xoopsDB->freeRecordSet($result);
+	$xoopsDB->freeRecordSet($result);
 
-    $result = $xoopsDB->query(
-        trim(
-            "
+	$result = $xoopsDB->query(
+		trim(
+			"
 		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
@@ -146,211 +146,211 @@ 
 		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'],
-            'last_activity'    => $row['last_activity'],
-            'fen_active_color' => $row['fen_active_color'],
-            'pgn_result'       => $row['pgn_result'],
-            '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'],
+			'last_activity'    => $row['last_activity'],
+			'fen_active_color' => $row['fen_active_color'],
+			'pgn_result'       => $row['pgn_result'],
+			'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);
 
-    $games_pagenav = new XoopsPageNav($num_games, $max_items_to_display, $gstart, 'gstart', "cstart=$cstart&cshow=$cshow&gshow1=$gshow1&gshow2=$gshow2");
+	$games_pagenav = new XoopsPageNav($num_games, $max_items_to_display, $gstart, 'gstart', "cstart=$cstart&cshow=$cshow&gshow1=$gshow1&gshow2=$gshow2");
 
-    $xoopsTpl->assign('chess_games_pagenav', $games_pagenav->renderNav());
+	$xoopsTpl->assign('chess_games_pagenav', $games_pagenav->renderNav());
 
-    // ----------
+	// ----------
 
-    // challenges
+	// 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.
 
-    $challenges_table = $xoopsDB->prefix('chess_challenges');
+	$challenges_table = $xoopsDB->prefix('chess_challenges');
 
-    switch ($cshow) {
-        case _CHESS_SHOW_CHALLENGES_OPEN:
-            $where = "game_type = 'open'";
-            break;
-        case _CHESS_SHOW_CHALLENGES_USER:
-            $where = "game_type = 'user'";
-            break;
-        default:
-            $where = 1;
-            break;
-    }
+	switch ($cshow) {
+		case _CHESS_SHOW_CHALLENGES_OPEN:
+			$where = "game_type = 'open'";
+			break;
+		case _CHESS_SHOW_CHALLENGES_USER:
+			$where = "game_type = 'user'";
+			break;
+		default:
+			$where = 1;
+			break;
+	}
 
-    $result = $xoopsDB->query("SELECT COUNT(*) FROM $challenges_table WHERE $where");
+	$result = $xoopsDB->query("SELECT COUNT(*) FROM $challenges_table WHERE $where");
 
-    [$num_challenges] = $xoopsDB->fetchRow($result);
+	[$num_challenges] = $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, 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);
 
-    $challenges_pagenav = new XoopsPageNav($num_challenges, $max_items_to_display, $cstart, 'cstart', "gstart=$gstart&cshow=$cshow&gshow1=$gshow1&gshow2=$gshow2");
+	$challenges_pagenav = new XoopsPageNav($num_challenges, $max_items_to_display, $cstart, 'cstart', "gstart=$gstart&cshow=$cshow&gshow1=$gshow1&gshow2=$gshow2");
 
-    $xoopsTpl->assign('chess_challenges_pagenav', $challenges_pagenav->renderNav());
+	$xoopsTpl->assign('chess_challenges_pagenav', $challenges_pagenav->renderNav());
 
-    // ---------
+	// ---------
 
-    // 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]['username_white'] = $usernames[$game['white_uid']] ?? '?';
+	foreach ($games as $k => $game) {
+		$games[$k]['username_white'] = $usernames[$game['white_uid']] ?? '?';
 
-        $games[$k]['username_black'] = $usernames[$game['black_uid']] ?? '?';
-    }
+		$games[$k]['username_black'] = $usernames[$game['black_uid']] ?? '?';
+	}
 
-    // add usernames to $challenges
+	// add usernames to $challenges
 
-    foreach ($challenges as $k => $challenge) {
-        $challenges[$k]['username_player1'] = $usernames[$challenge['player1_uid']] ?? '?';
+	foreach ($challenges as $k => $challenge) {
+		$challenges[$k]['username_player1'] = $usernames[$challenge['player1_uid']] ?? '?';
 
-        $challenges[$k]['username_player2'] = $usernames[$challenge['player2_uid']] ?? '?';
-    }
+		$challenges[$k]['username_player2'] = $usernames[$challenge['player2_uid']] ?? '?';
+	}
 
-    $xoopsTpl->assign('chess_games', $games);
+	$xoopsTpl->assign('chess_games', $games);
 
-    $xoopsTpl->assign('chess_challenges', $challenges);
+	$xoopsTpl->assign('chess_challenges', $challenges);
 
-    $xoopsTpl->assign('chess_rating_system', chess_moduleConfig('rating_system'));
+	$xoopsTpl->assign('chess_rating_system', chess_moduleConfig('rating_system'));
 
-    // -----
+	// -----
 
-    // forms
+	// forms
 
-    // -----
+	// -----
 
-    // security token not needed for this form
+	// security token not needed for this form
 
-    $form1 = new XoopsThemeForm('', 'form1', 'index.php');
+	$form1 = new XoopsThemeForm('', 'form1', 'index.php');
 
-    $form1->addElement(new XoopsFormButton('', 'submit', _MD_CHESS_SUBMIT_BUTTON, 'submit'));
+	$form1->addElement(new XoopsFormButton('', 'submit', _MD_CHESS_SUBMIT_BUTTON, 'submit'));
 
-    $menu_cshow = new XoopsFormSelect('', 'cshow', $cshow, 1, false);
+	$menu_cshow = new XoopsFormSelect('', 'cshow', $cshow, 1, false);
 
-    $menu_cshow->addOption(_CHESS_SHOW_CHALLENGES_OPEN, _MD_CHESS_SHOW_CHALLENGES_OPEN);
+	$menu_cshow->addOption(_CHESS_SHOW_CHALLENGES_OPEN, _MD_CHESS_SHOW_CHALLENGES_OPEN);
 
-    $menu_cshow->addOption(_CHESS_SHOW_CHALLENGES_USER, _MD_CHESS_SHOW_CHALLENGES_USER);
+	$menu_cshow->addOption(_CHESS_SHOW_CHALLENGES_USER, _MD_CHESS_SHOW_CHALLENGES_USER);
 
-    $menu_cshow->addOption(_CHESS_SHOW_CHALLENGES_BOTH, _MD_CHESS_SHOW_CHALLENGES_BOTH);
+	$menu_cshow->addOption(_CHESS_SHOW_CHALLENGES_BOTH, _MD_CHESS_SHOW_CHALLENGES_BOTH);
 
-    $form1->addElement($menu_cshow);
+	$form1->addElement($menu_cshow);
 
-    $form1->addElement(new XoopsFormHidden('gstart', $gstart));
+	$form1->addElement(new XoopsFormHidden('gstart', $gstart));
 
-    $form1->addElement(new XoopsFormHidden('gshow1', $gshow1));
+	$form1->addElement(new XoopsFormHidden('gshow1', $gshow1));
 
-    $form1->addElement(new XoopsFormHidden('gshow2', $gshow2));
+	$form1->addElement(new XoopsFormHidden('gshow2', $gshow2));
 
-    $form1->assign($xoopsTpl);
+	$form1->assign($xoopsTpl);
 
-    // security token not needed for this form
+	// security token not needed for this form
 
-    $form2 = new XoopsThemeForm('', 'form2', 'index.php');
+	$form2 = new XoopsThemeForm('', 'form2', 'index.php');
 
-    $form2->addElement(new XoopsFormButton('', 'submit', _MD_CHESS_SUBMIT_BUTTON, 'submit'));
+	$form2->addElement(new XoopsFormButton('', 'submit', _MD_CHESS_SUBMIT_BUTTON, 'submit'));
 
-    $menu_gshow1 = new XoopsFormSelect('', 'gshow1', $gshow1, 1, false);
+	$menu_gshow1 = new XoopsFormSelect('', 'gshow1', $gshow1, 1, false);
 
-    $menu_gshow1->addOption(_CHESS_SHOW_GAMES_INPLAY, _MD_CHESS_SHOW_GAMES_INPLAY);
+	$menu_gshow1->addOption(_CHESS_SHOW_GAMES_INPLAY, _MD_CHESS_SHOW_GAMES_INPLAY);
 
-    $menu_gshow1->addOption(_CHESS_SHOW_GAMES_CONCLUDED, _MD_CHESS_SHOW_GAMES_CONCLUDED);
+	$menu_gshow1->addOption(_CHESS_SHOW_GAMES_CONCLUDED, _MD_CHESS_SHOW_GAMES_CONCLUDED);
 
-    $menu_gshow1->addOption(_CHESS_SHOW_GAMES_BOTH, _MD_CHESS_SHOW_GAMES_BOTH);
+	$menu_gshow1->addOption(_CHESS_SHOW_GAMES_BOTH, _MD_CHESS_SHOW_GAMES_BOTH);
 
-    $form2->addElement($menu_gshow1);
+	$form2->addElement($menu_gshow1);
 
-    $menu_gshow2 = new XoopsFormSelect('', 'gshow2', $gshow2, 1, false);
+	$menu_gshow2 = new XoopsFormSelect('', 'gshow2', $gshow2, 1, false);
 
-    $menu_gshow2->addOption(_CHESS_SHOW_GAMES_RATED, _MD_CHESS_SHOW_GAMES_RATED);
+	$menu_gshow2->addOption(_CHESS_SHOW_GAMES_RATED, _MD_CHESS_SHOW_GAMES_RATED);
 
-    $menu_gshow2->addOption(_CHESS_SHOW_GAMES_UNRATED, _MD_CHESS_SHOW_GAMES_UNRATED);
+	$menu_gshow2->addOption(_CHESS_SHOW_GAMES_UNRATED, _MD_CHESS_SHOW_GAMES_UNRATED);
 
-    $form2->addElement($menu_gshow2);
+	$form2->addElement($menu_gshow2);
 
-    $form2->addElement(new XoopsFormHidden('cstart', $cstart));
+	$form2->addElement(new XoopsFormHidden('cstart', $cstart));
 
-    $form2->addElement(new XoopsFormHidden('cshow', $cshow));
+	$form2->addElement(new XoopsFormHidden('cshow', $cshow));
 
-    $form2->assign($xoopsTpl);
+	$form2->assign($xoopsTpl);
 
-    #*#DEBUG# - trying something unrelated to the chess module
-    /***
+	#*#DEBUG# - trying something unrelated to the chess module
+	/***
      * $configHandler = xoops_getHandler('config');
      * $clist = $configHandler->getConfigList(18);
      * var_dump('clist', $clist);

help.php

Indentation +5 added lines, -5 removed lines

@@ -40,19 +40,19 @@
 $xoopsConfig['module_cache'][$xoopsModule->getVar('mid')] = 0; // disable caching
 require_once XOOPS_ROOT_PATH . '/modules/' . $xoopsModule->getVar('dirname') . '/include/functions.php';
 if (file_exists(XOOPS_ROOT_PATH . '/modules/' . $xoopsModule->getVar('dirname') . '/language/' . $xoopsConfig['language'] . '/help.php')) {
-    require_once XOOPS_ROOT_PATH . '/modules/' . $xoopsModule->getVar('dirname') . '/language/' . $xoopsConfig['language'] . '/help.php';
+	require_once XOOPS_ROOT_PATH . '/modules/' . $xoopsModule->getVar('dirname') . '/language/' . $xoopsConfig['language'] . '/help.php';
 } else {
-    require_once XOOPS_ROOT_PATH . '/modules/' . $xoopsModule->getVar('dirname') . '/language/english/help.php';
+	require_once XOOPS_ROOT_PATH . '/modules/' . $xoopsModule->getVar('dirname') . '/language/english/help.php';
 }
 
 $rating_system          = chess_moduleConfig('rating_system');
 $rating_system_des_name = '_HE_CHESS_RATINGS_' . mb_strtoupper($rating_system);
 if (defined($rating_system_des_name)) {
-    $rating_system_des = constant($rating_system_des_name);
+	$rating_system_des = constant($rating_system_des_name);
 } else {
-    // missing constant definition - display name of constant for diagnostic use
+	// missing constant definition - display name of constant for diagnostic use
 
-    $rating_system_des = $rating_system_des_name;
+	$rating_system_des = $rating_system_des_name;
 }
 
 $xoopsTpl->assign('chess_allow_setup', chess_moduleConfig('allow_setup'));

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

language/english/modinfo.php

Indentation +8 added lines, -8 removed lines

@@ -17,8 +17,8 @@ 
 define('_MI_CHESS', 'Chess');
 define('_MI_CHESS_DES', 'Allows users to play chess games against each other.');
 define(
-    '_MI_CHESS_CREDITS',
-    '
+	'_MI_CHESS_CREDITS',
+	'
 	Jacques Masscrier and Pierre François Gagnon (french language pack).
 	<br>
 	CXR Rating System used by permission of <a target="_blank" href="http://chess-express.com/">Chess Express Ratings, Inc.</a>
@@ -69,8 +69,8 @@ 
 define('_MI_CHESS_MAX_ITEMS_DES', 'Applies to games, challenges and players.');
 define('_MI_CHESS_RATING_SYSTEM', 'Player rating system');
 define(
-    '_MI_CHESS_RATING_SYSTEM_DES',
-    '
+	'_MI_CHESS_RATING_SYSTEM_DES',
+	'
 	Available rating systems:
 	<br><br>
 	&nbsp;&nbsp;' . _MI_CHESS_RATING_SYSTEM_CXR . '    - Adaptation of the ELO rating system, used by permission of <a target="_blank" href="http://chess-express.com/">Chess Express Ratings, Inc.</a>
@@ -85,8 +85,8 @@ 
 );
 define('_MI_CHESS_INITIAL_RATING', 'Initial player rating');
 define(
-    '_MI_CHESS_INITIAL_RATING_DES',
-    '
+	'_MI_CHESS_INITIAL_RATING_DES',
+	'
 	If the "' . _MI_CHESS_RATING_SYSTEM_CXR . '" rating system is selected, this value should be between 800 and 2000.
 	<br><br>
 	Applies only if a player rating system is selected.
@@ -96,8 +96,8 @@ 
 );
 define('_MI_CHESS_ALLOW_UNRATED', 'Allow unrated games?');
 define(
-    '_MI_CHESS_ALLOW_UNRATED_DES',
-    '
+	'_MI_CHESS_ALLOW_UNRATED_DES',
+	'
 	When offering a challenge, may the player exclude the game from use in rating calculations?
 	<br><br>
 	Applies only if a player rating system is selected.

preloads/core.php

Indentation +8 added lines, -8 removed lines

@@ -22,12 +22,12 @@
  */
 class ChessCorePreload extends \XoopsPreloadItem
 {
-    // to add PSR-4 autoloader
-    /**
-     * @param $args
-     */
-    public static function eventCoreIncludeCommonEnd($args)
-    {
-        require __DIR__ . '/autoloader.php';
-    }
+	// to add PSR-4 autoloader
+	/**
+	 * @param $args
+	 */
+	public static function eventCoreIncludeCommonEnd($args)
+	{
+		require __DIR__ . '/autoloader.php';
+	}
 }

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/functions.php

Indentation +74 added lines, -74 removed lines

@@ -40,9 +40,9 @@ 
  */
 function chess_sanitize($text, $allowed_characters = 'A-Za-z0-9')
 {
-    $char_class = preg_replace('/(\.\\\+\*\?\[\^\]\$\(\)\{\}\=\!\<\>\|\:\#)/', '\\${1}', $allowed_characters);
-    //$char_class = preg_quote($allowed_characters, '/');
-    return preg_replace("/[^$char_class]/i", '_', $text);
+	$char_class = preg_replace('/(\.\\\+\*\?\[\^\]\$\(\)\{\}\=\!\<\>\|\:\#)/', '\\${1}', $allowed_characters);
+	//$char_class = preg_quote($allowed_characters, '/');
+	return preg_replace("/[^$char_class]/i", '_', $text);
 }
 
 /**
@@ -53,29 +53,29 @@ 
  */
 function chess_moduleConfig($option)
 {
-    global $xoopsModule, $xoopsModuleConfig;
+	global $xoopsModule, $xoopsModuleConfig;
 
-    $value = null;
+	$value = null;
 
-    if (is_object($xoopsModule) && 'chess' == $xoopsModule->getVar('dirname') && isset($xoopsModuleConfig[$option])) {
-        $value = $xoopsModuleConfig[$option];
-    } else { // for use within a block
-        $moduleHandler = xoops_getHandler('module');
+	if (is_object($xoopsModule) && 'chess' == $xoopsModule->getVar('dirname') && isset($xoopsModuleConfig[$option])) {
+		$value = $xoopsModuleConfig[$option];
+	} else { // for use within a block
+		$moduleHandler = xoops_getHandler('module');
 
-        $module = $moduleHandler->getByDirname('chess');
+		$module = $moduleHandler->getByDirname('chess');
 
-        $configHandler = xoops_getHandler('config');
+		$configHandler = xoops_getHandler('config');
 
-        $moduleConfig = $configHandler->getConfigsByCat(0, $module->getVar('mid'));
+		$moduleConfig = $configHandler->getConfigsByCat(0, $module->getVar('mid'));
 
-        if (isset($moduleConfig[$option])) {
-            $value = $moduleConfig[$option];
-        } else {
-            trigger_error("configuration option '$option' not found", E_USER_ERROR);
-        }
-    }
+		if (isset($moduleConfig[$option])) {
+			$value = $moduleConfig[$option];
+		} else {
+			trigger_error("configuration option '$option' not found", E_USER_ERROR);
+		}
+	}
 
-    return $value;
+	return $value;
 }
 
 /**
@@ -86,29 +86,29 @@ 
  */
 function chess_can_play($uid = null)
 {
-    global $xoopsUser;
+	global $xoopsUser;
 
-    if (isset($uid)) {
-        $memberHandler = xoops_getHandler('member');
+	if (isset($uid)) {
+		$memberHandler = xoops_getHandler('member');
 
-        $user = $memberHandler->getUser($uid);
-    } elseif (is_object($xoopsUser)) {
-        $user = $xoopsUser;
-    } else {
-        $user = null;
-    }
+		$user = $memberHandler->getUser($uid);
+	} elseif (is_object($xoopsUser)) {
+		$user = $xoopsUser;
+	} else {
+		$user = null;
+	}
 
-    $groups_play = chess_moduleConfig('groups_play');
+	$groups_play = chess_moduleConfig('groups_play');
 
-    $can_play = false;
+	$can_play = false;
 
-    if (in_array(XOOPS_GROUP_ANONYMOUS, $groups_play)) {
-        $can_play = true;
-    } elseif (is_object($user)) {
-        $can_play = count(array_intersect($user->getGroups(), $groups_play)) > 0;
-    }
+	if (in_array(XOOPS_GROUP_ANONYMOUS, $groups_play)) {
+		$can_play = true;
+	} elseif (is_object($user)) {
+		$can_play = count(array_intersect($user->getGroups(), $groups_play)) > 0;
+	}
 
-    return $can_play;
+	return $can_play;
 }
 
 /**
@@ -119,29 +119,29 @@ 
  */
 function chess_can_delete($uid = null)
 {
-    global $xoopsUser;
+	global $xoopsUser;
 
-    if (isset($uid)) {
-        $memberHandler = xoops_getHandler('member');
+	if (isset($uid)) {
+		$memberHandler = xoops_getHandler('member');
 
-        $user = $memberHandler->getUser($uid);
-    } elseif (is_object($xoopsUser)) {
-        $user = $xoopsUser;
-    } else {
-        $user = null;
-    }
+		$user = $memberHandler->getUser($uid);
+	} elseif (is_object($xoopsUser)) {
+		$user = $xoopsUser;
+	} else {
+		$user = null;
+	}
 
-    $groups_delete = chess_moduleConfig('groups_delete');
+	$groups_delete = chess_moduleConfig('groups_delete');
 
-    $can_delete = false;
+	$can_delete = false;
 
-    if (in_array(XOOPS_GROUP_ANONYMOUS, $groups_delete)) {
-        $can_delete = true;
-    } elseif (is_object($user)) {
-        $can_delete = count(array_intersect($user->getGroups(), $groups_delete)) > 0;
-    }
+	if (in_array(XOOPS_GROUP_ANONYMOUS, $groups_delete)) {
+		$can_delete = true;
+	} elseif (is_object($user)) {
+		$can_delete = count(array_intersect($user->getGroups(), $groups_delete)) > 0;
+	}
 
-    return $can_delete;
+	return $can_delete;
 }
 
 /**
@@ -156,19 +156,19 @@ 
  */
 function chess_to_pgn_string($data)
 {
-    #var_dump('chess_to_pgn_string, data=', $data);#*#DEBUG#
+	#var_dump('chess_to_pgn_string, data=', $data);#*#DEBUG#
 
-    if ('0000-00-00 00:00:00' == $data['datetime']) {
-        $datetime = '????.??.?? ??:??:??';
-    } else {
-        $datetime = str_replace('-', '.', $data['datetime']);
-    }
+	if ('0000-00-00 00:00:00' == $data['datetime']) {
+		$datetime = '????.??.?? ??:??:??';
+	} else {
+		$datetime = str_replace('-', '.', $data['datetime']);
+	}
 
-    [$date, $time] = explode(' ', $datetime);
+	[$date, $time] = explode(' ', $datetime);
 
-    $movetext = wordwrap($data['movetext'], 75);
+	$movetext = wordwrap($data['movetext'], 75);
 
-    $rtn = <<<END
+	$rtn = <<<END
 [Event "{$data['event']}"]
 [Site "{$data['site']}"]
 [Date "$date"]
@@ -180,13 +180,13 @@ 
 
 END;
 
-    if ($data['setup'] && $data['fen']) {
-        $rtn .= "[Setup \"1\"]\n[FEN \"{$data['fen']}\"]\n";
-    }
+	if ($data['setup'] && $data['fen']) {
+		$rtn .= "[Setup \"1\"]\n[FEN \"{$data['fen']}\"]\n";
+	}
 
-    $rtn .= "\n$movetext\n";
+	$rtn .= "\n$movetext\n";
 
-    return $rtn;
+	return $rtn;
 }
 
 /**
@@ -197,19 +197,19 @@ 
  */
 function chess_uname_to_uid($uname)
 {
-    $criteria = new \CriteriaCompo();
+	$criteria = new \CriteriaCompo();
 
-    $criteria->add(new \Criteria('uname', MyTextSanitizer::addSlashes($uname)));
+	$criteria->add(new \Criteria('uname', MyTextSanitizer::addSlashes($uname)));
 
-    $criteria->setLimit(1);
+	$criteria->setLimit(1);
 
-    $memberHandler = xoops_getHandler('member');
+	$memberHandler = xoops_getHandler('member');
 
-    $users = $memberHandler->getUserList($criteria);
+	$users = $memberHandler->getUserList($criteria);
 
-    $uids = array_keys($users);
+	$uids = array_keys($users);
 
-    return $uids[0] ?? 0;
+	return $uids[0] ?? 0;
 }
 
 /*** #*#DEBUG# testing something

include/onuninstall.php

Indentation +30 added lines, -30 removed lines

@@ -29,9 +29,9 @@ 
  */
 function xoops_module_pre_uninstall_chess(\XoopsModule $module)
 {
-    // Do some synchronization if needed
+	// Do some synchronization if needed
 
-    return true;
+	return true;
 }
 
 /**
@@ -42,50 +42,50 @@ 
  */
 function xoops_module_uninstall_chess(\XoopsModule $module)
 {
-    require __DIR__ . '/common.php';
+	require __DIR__ . '/common.php';
 
-    $moduleDirName = basename(dirname(__DIR__));
+	$moduleDirName = basename(dirname(__DIR__));
 
-    $moduleDirNameUpper = mb_strtoupper($moduleDirName);
+	$moduleDirNameUpper = mb_strtoupper($moduleDirName);
 
-    $helper = Chess\Helper::getInstance();
+	$helper = Chess\Helper::getInstance();
 
-    $utility = new Chess\Utility();
-    //    $configurator = new Chess\Common\Configurator();
+	$utility = new Chess\Utility();
+	//    $configurator = new Chess\Common\Configurator();
 
-    // Load language files
+	// Load language files
 
-    $helper->loadLanguage('admin');
+	$helper->loadLanguage('admin');
 
-    $helper->loadLanguage('common');
+	$helper->loadLanguage('common');
 
-    $success = true;
+	$success = true;
 
-    //------------------------------------------------------------------
+	//------------------------------------------------------------------
 
-    // Remove uploads folder (and all subfolders) if they exist
+	// Remove uploads folder (and all subfolders) if they exist
 
-    //------------------------------------------------------------------
+	//------------------------------------------------------------------
 
-    $old_directories = [$GLOBALS['xoops']->path("uploads/{$moduleDirName}")];
+	$old_directories = [$GLOBALS['xoops']->path("uploads/{$moduleDirName}")];
 
-    foreach ($old_directories as $old_dir) {
-        $dirInfo = new SplFileInfo($old_dir);
+	foreach ($old_directories as $old_dir) {
+		$dirInfo = new SplFileInfo($old_dir);
 
-        if ($dirInfo->isDir()) {
-            // The directory exists so delete it
+		if ($dirInfo->isDir()) {
+			// The directory exists so delete it
 
-            if (false === $utility::rrmdir($old_dir)) {
-                $module->setErrors(sprintf(constant('CO_' . $moduleDirNameUpper . '_ERROR_BAD_DEL_PATH'), $old_dir));
+			if (false === $utility::rrmdir($old_dir)) {
+				$module->setErrors(sprintf(constant('CO_' . $moduleDirNameUpper . '_ERROR_BAD_DEL_PATH'), $old_dir));
 
-                $success = false;
-            }
-        }
+				$success = false;
+			}
+		}
 
-        unset($dirInfo);
-    }
+		unset($dirInfo);
+	}
 
-    /*
+	/*
     //------------ START ----------------
     //------------------------------------------------------------------
     // Remove xsitemap.xml from XOOPS root folder if it exists
@@ -99,6 +99,6 @@ 
 //    return $success && $delOk; // use this if you're using this routine
 */
 
-    return $success;
-    //------------ END  ----------------
+	return $success;
+	//------------ END  ----------------
 }