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 . "

Spacing +22 added lines, -22 removed lines

@@ -36,8 +36,8 @@ 
 
 /**#@+
  */
-require_once XOOPS_ROOT_PATH . '/modules/chess/include/constants.inc.php';
-require_once XOOPS_ROOT_PATH . '/modules/chess/include/functions.php';
+require_once XOOPS_ROOT_PATH.'/modules/chess/include/constants.inc.php';
+require_once XOOPS_ROOT_PATH.'/modules/chess/include/functions.php';
 /**#@-*/
 
 /**
@@ -58,7 +58,7 @@ 
 
     $table = $xoopsDB->prefix('chess_games');
 
-    $limit = (int)$options[0]; // sanitize with intval()
+    $limit = (int) $options[0]; // sanitize with intval()
 
     $where = 'white_uid != black_uid';
 
@@ -122,7 +122,7 @@ 
 
     $memberHandler = xoops_getHandler('member');
 
-    $criteria = new \Criteria('uid', '(' . implode(',', array_keys($userids)) . ')', 'IN');
+    $criteria = new \Criteria('uid', '('.implode(',', array_keys($userids)).')', 'IN');
 
     $usernames = $memberHandler->getUserList($criteria);
 
@@ -159,7 +159,7 @@ 
 
     $table = $xoopsDB->prefix('chess_challenges');
 
-    $limit = (int)$options[0]; // sanitize with intval()
+    $limit = (int) $options[0]; // sanitize with intval()
 
     switch ($options[1]) {
         case 1:
@@ -217,7 +217,7 @@ 
 
     $memberHandler = xoops_getHandler('member');
 
-    $criteria = new \Criteria('uid', '(' . implode(',', array_keys($userids)) . ')', 'IN');
+    $criteria = new \Criteria('uid', '('.implode(',', array_keys($userids)).')', 'IN');
 
     $usernames = $memberHandler->getUserList($criteria);
 
@@ -252,7 +252,7 @@ 
         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');
 
@@ -274,7 +274,7 @@ 
 
     $table = $xoopsDB->prefix('chess_ratings');
 
-    $limit = (int)$options[0]; // sanitize with intval()
+    $limit = (int) $options[0]; // sanitize with intval()
 
     switch ($options[1]) {
         case 1:
@@ -327,7 +327,7 @@ 
     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);
     }
@@ -362,16 +362,16 @@ 
     $show_unrated = 2 == $options[2] ? 'checked' : '';
 
     return '
-		' . _MB_CHESS_NUM_GAMES . ": <input type='text' name='options[0]' value='{$options[0]}' size='3' maxlength='3'>
+		' . _MB_CHESS_NUM_GAMES.": <input type='text' name='options[0]' value='{$options[0]}' size='3' maxlength='3'>
 		<br>
 		<br>
-		<input type='radio' name='options[1]' value='1' $show_inplay    > " . _MB_CHESS_SHOW_GAMES_INPLAY . "
-		<input type='radio' name='options[1]' value='2' $show_concluded > " . _MB_CHESS_SHOW_GAMES_CONCLUDED . "
-		<input type='radio' name='options[1]' value='3' $show_both      > " . _MB_CHESS_SHOW_GAMES_BOTH . "
+		<input type='radio' name='options[1]' value='1' $show_inplay    > "._MB_CHESS_SHOW_GAMES_INPLAY."
+		<input type='radio' name='options[1]' value='2' $show_concluded > "._MB_CHESS_SHOW_GAMES_CONCLUDED."
+		<input type='radio' name='options[1]' value='3' $show_both      > "._MB_CHESS_SHOW_GAMES_BOTH."
 		<br>
 		<br>
-		<input type='radio' name='options[2]' value='1' $show_rated_only> " . _MB_CHESS_SHOW_GAMES_RATED . "
-		<input type='radio' name='options[2]' value='2' $show_unrated   > " . _MB_CHESS_SHOW_GAMES_UNRATED . '
+		<input type='radio' name='options[2]' value='1' $show_rated_only> "._MB_CHESS_SHOW_GAMES_RATED."
+		<input type='radio' name='options[2]' value='2' $show_unrated   > "._MB_CHESS_SHOW_GAMES_UNRATED.'
 	';
 }
 
@@ -390,11 +390,11 @@ 
     $show_both = 3 == $options[1] ? 'checked' : '';
 
     return '
-		' . _MB_CHESS_NUM_CHALLENGES . ": <input type='text' name='options[0]' value='{$options[0]}' size='3' maxlength='3'>
+		' . _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 . "
-		<input type='radio' name='options[1]' value='2' $show_user> " . _MB_CHESS_SHOW_CHALLENGES_USER . "
-		<input type='radio' name='options[1]' value='3' $show_both> " . _MB_CHESS_SHOW_CHALLENGES_BOTH . '
+		<input type='radio' name='options[1]' value='1' $show_open> "._MB_CHESS_SHOW_CHALLENGES_OPEN."
+		<input type='radio' name='options[1]' value='2' $show_user> "._MB_CHESS_SHOW_CHALLENGES_USER."
+		<input type='radio' name='options[1]' value='3' $show_both> "._MB_CHESS_SHOW_CHALLENGES_BOTH.'
 	';
 }
 
@@ -411,9 +411,9 @@ 
     $show_all = 2 == $options[1] ? 'checked' : '';
 
     return '
-		' . _MB_CHESS_NUM_PLAYERS . ": <input type='text' name='options[0]' value='{$options[0]}' size='3' maxlength='3'>
+		' . _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 . "
-		<input type='radio' name='options[1]' value='2' $show_all           > " . _MB_CHESS_SHOW_ALL_RATINGS . '
+		<input type='radio' name='options[1]' value='1' $show_nonprovisional> "._MB_CHESS_SHOW_NONPROVISIONAL."
+		<input type='radio' name='options[1]' value='2' $show_all           > "._MB_CHESS_SHOW_ALL_RATINGS.'
 	';
 }

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

Spacing +13 added lines, -13 removed lines

@@ -37,13 +37,13 @@ 
 
 /**#@+
  */
-require dirname(__DIR__, 2) . '/mainfile.php';
+require dirname(__DIR__, 2).'/mainfile.php';
 $GLOBALS['xoopsOption']['template_main'] = 'chess_games.tpl';
-require_once XOOPS_ROOT_PATH . '/header.php';
-require_once XOOPS_ROOT_PATH . '/class/xoopsformloader.php';
-require_once XOOPS_ROOT_PATH . '/class/pagenav.php';
-require_once XOOPS_ROOT_PATH . '/modules/chess/include/constants.inc.php';
-require_once XOOPS_ROOT_PATH . '/modules/chess/include/functions.php';
+require_once XOOPS_ROOT_PATH.'/header.php';
+require_once XOOPS_ROOT_PATH.'/class/xoopsformloader.php';
+require_once XOOPS_ROOT_PATH.'/class/pagenav.php';
+require_once XOOPS_ROOT_PATH.'/modules/chess/include/constants.inc.php';
+require_once XOOPS_ROOT_PATH.'/modules/chess/include/functions.php';
 
 #var_dump($_REQUEST);#*#DEBUG#
 
@@ -57,23 +57,23 @@ 
 
     // 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)
 
-    $gstart = (int)($_POST['gstart'] ?? @$_GET['gstart']);
+    $gstart = (int) ($_POST['gstart'] ?? @$_GET['gstart']);
 
     // challenges display option
 
-    $cshow = (int)($_POST['cshow'] ?? @$_GET['cshow']);
+    $cshow = (int) ($_POST['cshow'] ?? @$_GET['cshow']);
 
     // games display option 1
 
-    $gshow1 = (int)($_POST['gshow1'] ?? @$_GET['gshow1']);
+    $gshow1 = (int) ($_POST['gshow1'] ?? @$_GET['gshow1']);
 
     // games display option 2
 
-    $gshow2 = (int)($_POST['gshow2'] ?? @$_GET['gshow2']);
+    $gshow2 = (int) ($_POST['gshow2'] ?? @$_GET['gshow2']);
 
     // set show-options to default if undefined
 
@@ -263,7 +263,7 @@ 
 
     $memberHandler = xoops_getHandler('member');
 
-    $criteria = new \Criteria('uid', '(' . implode(',', array_keys($userids)) . ')', 'IN');
+    $criteria = new \Criteria('uid', '('.implode(',', array_keys($userids)).')', 'IN');
 
     $usernames = $memberHandler->getUserList($criteria);
 
@@ -355,4 +355,4 @@ 
      * $clist = $configHandler->getConfigList(18);
      * var_dump('clist', $clist);
      ***/
-include_once XOOPS_ROOT_PATH . '/footer.php';
+include_once XOOPS_ROOT_PATH.'/footer.php';

testdata/index.php

Spacing +16 added lines, -16 removed lines

@@ -19,8 +19,8 @@ 
 use XoopsModules\Chess\Common;
 use XoopsModules\Chess\Utility;
 
-require_once dirname(__DIR__, 3) . '/include/cp_header.php';
-require dirname(__DIR__) . '/preloads/autoloader.php';
+require_once dirname(__DIR__, 3).'/include/cp_header.php';
+require dirname(__DIR__).'/preloads/autoloader.php';
 
 $op = \Xmf\Request::getCmd('op', '');
 
@@ -42,7 +42,7 @@ 
         } else {
             xoops_cp_header();
 
-            xoops_confirm(['ok' => 1, 'op' => 'load'], 'index.php', sprintf(constant('CO_' . $moduleDirNameUpper . '_' . 'ADD_SAMPLEDATA_OK')), constant('CO_' . $moduleDirNameUpper . '_' . 'CONFIRM'), true);
+            xoops_confirm(['ok' => 1, 'op' => 'load'], 'index.php', sprintf(constant('CO_'.$moduleDirNameUpper.'_'.'ADD_SAMPLEDATA_OK')), constant('CO_'.$moduleDirNameUpper.'_'.'CONFIRM'), true);
 
             xoops_cp_footer();
         }
@@ -70,12 +70,12 @@ 
 
     $language = 'english/';
 
-    if (is_dir(__DIR__ . '/' . $xoopsConfig['language'])) {
-        $language = $xoopsConfig['language'] . '/';
+    if (is_dir(__DIR__.'/'.$xoopsConfig['language'])) {
+        $language = $xoopsConfig['language'].'/';
     }
 
     foreach ($tables as $table) {
-        $tabledata = \Xmf\Yaml::readWrapped($language . $table . '.yml');
+        $tabledata = \Xmf\Yaml::readWrapped($language.$table.'.yml');
 
         if (is_array($tabledata)) {
             \Xmf\Database\TableLoad::truncateTable($table);
@@ -98,7 +98,7 @@ 
         }
     }
 
-    redirect_header('../admin/index.php', 1, constant('CO_' . $moduleDirNameUpper . '_' . 'SAMPLEDATA_SUCCESS'));
+    redirect_header('../admin/index.php', 1, constant('CO_'.$moduleDirNameUpper.'_'.'SAMPLEDATA_SUCCESS'));
 }
 
 function saveSampleData()
@@ -113,25 +113,25 @@ 
 
     $language = 'english/';
 
-    if (is_dir(__DIR__ . '/' . $xoopsConfig['language'])) {
-        $language = $xoopsConfig['language'] . '/';
+    if (is_dir(__DIR__.'/'.$xoopsConfig['language'])) {
+        $language = $xoopsConfig['language'].'/';
     }
 
-    $languageFolder = __DIR__ . '/' . $language;
+    $languageFolder = __DIR__.'/'.$language;
 
-    if (!file_exists($languageFolder . '/')) {
-        Utility::createFolder($languageFolder . '/');
+    if (!file_exists($languageFolder.'/')) {
+        Utility::createFolder($languageFolder.'/');
     }
 
-    $exportFolder = $languageFolder . '/Exports-' . date('Y-m-d-H-i-s') . '/';
+    $exportFolder = $languageFolder.'/Exports-'.date('Y-m-d-H-i-s').'/';
 
     Utility::createFolder($exportFolder);
 
     foreach ($tables as $table) {
-        \Xmf\Database\TableLoad::saveTableToYamlFile($table, $exportFolder . $table . '.yml');
+        \Xmf\Database\TableLoad::saveTableToYamlFile($table, $exportFolder.$table.'.yml');
     }
 
-    redirect_header('../admin/index.php', 1, constant('CO_' . $moduleDirNameUpper . '_' . 'SAMPLEDATA_SUCCESS'));
+    redirect_header('../admin/index.php', 1, constant('CO_'.$moduleDirNameUpper.'_'.'SAMPLEDATA_SUCCESS'));
 }
 
 function exportSchema()
@@ -147,6 +147,6 @@ 
         //
         //        redirect_header('../admin/index.php', 1, constant('CO_' . $moduleDirNameUpper . '_' . 'EXPORT_SCHEMA_SUCCESS'));
     } catch (\Throwable $e) {
-        exit(constant('CO_' . $moduleDirNameUpper . '_' . 'EXPORT_SCHEMA_ERROR'));
+        exit(constant('CO_'.$moduleDirNameUpper.'_'.'EXPORT_SCHEMA_ERROR'));
     }
 }

comment_edit.php

Spacing +2 added lines, -2 removed lines

@@ -35,6 +35,6 @@
 
 /**#@+
  */
-require dirname(__DIR__, 2) . '/mainfile.php';
-require XOOPS_ROOT_PATH . '/include/comment_edit.php';
+require dirname(__DIR__, 2).'/mainfile.php';
+require XOOPS_ROOT_PATH.'/include/comment_edit.php';
 /**#@-*/

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

Spacing +7 added lines, -7 removed lines

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

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

Spacing +2 added lines, -2 removed lines

@@ -36,9 +36,9 @@
 
     $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.
 

comment_new.php

Spacing +2 added lines, -2 removed lines

@@ -35,6 +35,6 @@
 
 /**#@+
  */
-require dirname(__DIR__, 2) . '/mainfile.php';
-require XOOPS_ROOT_PATH . '/include/comment_new.php';
+require dirname(__DIR__, 2).'/mainfile.php';
+require XOOPS_ROOT_PATH.'/include/comment_new.php';
 /**#@-*/

notification_update.php

Spacing +2 added lines, -2 removed lines

@@ -9,6 +9,6 @@
 
 /**#@+
  */
-require dirname(__DIR__, 2) . '/mainfile.php';
-require XOOPS_ROOT_PATH . '/include/notification_update.php';
+require dirname(__DIR__, 2).'/mainfile.php';
+require XOOPS_ROOT_PATH.'/include/notification_update.php';
 /**#@-*/

language/english/filechecker.php

Spacing +10 added lines, -10 removed lines

@@ -14,15 +14,15 @@
  * @ignore
  */
 
-define('CO_' . $moduleDirNameUpper . '_' . 'FC_AVAILABLE', "<span style='color: green;'>Available</span>");
-define('CO_' . $moduleDirNameUpper . '_' . 'FC_NOTAVAILABLE', "<span style='color: red;'>Not available</span>");
-define('CO_' . $moduleDirNameUpper . '_' . 'FC_NOTWRITABLE', "<span style='color: red;'>Should have permission ( %d ), but it has ( %d )</span>");
-define('CO_' . $moduleDirNameUpper . '_' . 'FC_COPYTHEFILE', 'Copy it');
-define('CO_' . $moduleDirNameUpper . '_' . 'FC_CREATETHEFILE', 'Create it');
-define('CO_' . $moduleDirNameUpper . '_' . 'FC_SETMPERM', 'Set the permission');
-define('CO_' . $moduleDirNameUpper . '_' . 'FC_FILECOPIED', 'The file has been copied');
-define('CO_' . $moduleDirNameUpper . '_' . 'FC_FILENOTCOPIED', 'The file cannot be copied');
-define('CO_' . $moduleDirNameUpper . '_' . 'FC_PERMSET', 'The permission has been set');
-define('CO_' . $moduleDirNameUpper . '_' . 'FC_PERMNOTSET', 'The permission cannot be set');
+define('CO_'.$moduleDirNameUpper.'_'.'FC_AVAILABLE', "<span style='color: green;'>Available</span>");
+define('CO_'.$moduleDirNameUpper.'_'.'FC_NOTAVAILABLE', "<span style='color: red;'>Not available</span>");
+define('CO_'.$moduleDirNameUpper.'_'.'FC_NOTWRITABLE', "<span style='color: red;'>Should have permission ( %d ), but it has ( %d )</span>");
+define('CO_'.$moduleDirNameUpper.'_'.'FC_COPYTHEFILE', 'Copy it');
+define('CO_'.$moduleDirNameUpper.'_'.'FC_CREATETHEFILE', 'Create it');
+define('CO_'.$moduleDirNameUpper.'_'.'FC_SETMPERM', 'Set the permission');
+define('CO_'.$moduleDirNameUpper.'_'.'FC_FILECOPIED', 'The file has been copied');
+define('CO_'.$moduleDirNameUpper.'_'.'FC_FILENOTCOPIED', 'The file cannot be copied');
+define('CO_'.$moduleDirNameUpper.'_'.'FC_PERMSET', 'The permission has been set');
+define('CO_'.$moduleDirNameUpper.'_'.'FC_PERMNOTSET', 'The permission cannot be set');
 
 /**#@-*/