mambax7 / chess

include/ratings.php

Indentation +139 added lines, -139 removed lines

@@ -41,28 +41,28 @@ 
  */
 function chess_ratings_adj($gid)
 {
-    global $xoopsDB;
+	global $xoopsDB;
 
-    $rating_system = chess_moduleConfig('rating_system');
+	$rating_system = chess_moduleConfig('rating_system');
 
-    $init_rating = chess_moduleConfig('initial_rating');
+	$init_rating = chess_moduleConfig('initial_rating');
 
-    if ('none' == $rating_system) {
-        return false;
-    }
+	if ('none' == $rating_system) {
+		return false;
+	}
 
-    // determine function for calculating new ratings using configured rating system
+	// determine function for calculating new ratings using configured rating system
 
-    $func = chess_ratings_get_func_adj($rating_system);
+	$func = chess_ratings_get_func_adj($rating_system);
 
-    $games_table = $xoopsDB->prefix('chess_games');
+	$games_table = $xoopsDB->prefix('chess_games');
 
-    $ratings_table = $xoopsDB->prefix('chess_ratings');
+	$ratings_table = $xoopsDB->prefix('chess_ratings');
 
-    // get the game info
+	// get the game info
 
-    $result = $xoopsDB->query(
-        "
+	$result = $xoopsDB->query(
+		"
         SELECT
             g.white_uid AS white_uid, g.black_uid AS black_uid, g.pgn_result AS pgn_result, w.rating AS white_rating, b.rating AS black_rating,
             (w.games_won+w.games_lost+w.games_drawn) AS white_games, (b.games_won+b.games_lost+b.games_drawn) AS black_games
@@ -72,77 +72,77 @@ 
         WHERE     g.game_id = '$gid' AND g.is_rated = '1' AND g.pgn_result != '*'
             AND (w.player_uid IS NULL OR b.player_uid IS NULL OR w.player_uid != b.player_uid)
     "
-    );
+	);
 
-    // check that game exists and is rated
+	// check that game exists and is rated
 
-    if (1 != $xoopsDB->getRowsNum($result)) {
-        return false;
-    }
+	if (1 != $xoopsDB->getRowsNum($result)) {
+		return false;
+	}
 
-    $row = $xoopsDB->fetchArray($result);
+	$row = $xoopsDB->fetchArray($result);
 
-    $xoopsDB->freeRecordSet($result);
+	$xoopsDB->freeRecordSet($result);
 
-    #var_dump($row);#*#DEBUG#
+	#var_dump($row);#*#DEBUG#
 
-    // make sure the users are in the players' table
+	// make sure the users are in the players' table
 
-    $value_list = [];
+	$value_list = [];
 
-    if (!isset($row['white_rating'])) {
-        $row['white_rating'] = $init_rating;
+	if (!isset($row['white_rating'])) {
+		$row['white_rating'] = $init_rating;
 
-        $row['white_games'] = 0;
+		$row['white_games'] = 0;
 
-        $value_list[] = "('{$row['white_uid']}','{$row['white_rating']}')";
-    }
+		$value_list[] = "('{$row['white_uid']}','{$row['white_rating']}')";
+	}
 
-    if (!isset($row['black_rating'])) {
-        $row['black_rating'] = $init_rating;
+	if (!isset($row['black_rating'])) {
+		$row['black_rating'] = $init_rating;
 
-        $row['black_games'] = 0;
+		$row['black_games'] = 0;
 
-        $value_list[] = "('{$row['black_uid']}','{$row['black_rating']}')";
-    }
+		$value_list[] = "('{$row['black_uid']}','{$row['black_rating']}')";
+	}
 
-    if (!empty($value_list)) {
-        $values = implode(',', $value_list);
+	if (!empty($value_list)) {
+		$values = implode(',', $value_list);
 
-        $xoopsDB->query("INSERT INTO $ratings_table (player_uid, rating) VALUES $values");
+		$xoopsDB->query("INSERT INTO $ratings_table (player_uid, rating) VALUES $values");
 
-        $xoopsDB->errno() and trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
-    }
+		$xoopsDB->errno() and trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
+	}
 
-    // calculate new ratings using configured rating system
+	// calculate new ratings using configured rating system
 
-    [$white_rating_new, $black_rating_new] = $func($row['white_rating'], $row['white_games'], $row['black_rating'], $row['black_games'], $row['pgn_result']);
+	[$white_rating_new, $black_rating_new] = $func($row['white_rating'], $row['white_games'], $row['black_rating'], $row['black_games'], $row['pgn_result']);
 
-    // determine game-count columns to increment
+	// determine game-count columns to increment
 
-    [$white_col, $black_col] = chess_ratings_get_columns($row['pgn_result']);
+	[$white_col, $black_col] = chess_ratings_get_columns($row['pgn_result']);
 
-    $xoopsDB->query(
-        "
+	$xoopsDB->query(
+		"
         UPDATE $ratings_table
         SET    rating = '$white_rating_new', $white_col = $white_col + 1
         WHERE  player_uid = '{$row['white_uid']}'
     "
-    );
+	);
 
-    $xoopsDB->errno() and trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
+	$xoopsDB->errno() and trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
 
-    $xoopsDB->query(
-        "
+	$xoopsDB->query(
+		"
         UPDATE $ratings_table
         SET    rating = '$black_rating_new', $black_col = $black_col + 1
         WHERE  player_uid = '{$row['black_uid']}'
     "
-    );
+	);
 
-    $xoopsDB->errno() and trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
+	$xoopsDB->errno() and trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
 
-    return true;
+	return true;
 }
 
 /**
@@ -152,98 +152,98 @@ 
  */
 function chess_recalc_ratings()
 {
-    global $xoopsDB;
+	global $xoopsDB;
 
-    $rating_system = chess_moduleConfig('rating_system');
+	$rating_system = chess_moduleConfig('rating_system');
 
-    $init_rating = chess_moduleConfig('initial_rating');
+	$init_rating = chess_moduleConfig('initial_rating');
 
-    if ('none' == $rating_system) {
-        return false;
-    }
+	if ('none' == $rating_system) {
+		return false;
+	}
 
-    // determine function for calculating new ratings using configured rating system
+	// determine function for calculating new ratings using configured rating system
 
-    $func = chess_ratings_get_func_adj($rating_system);
+	$func = chess_ratings_get_func_adj($rating_system);
 
-    $games_table = $xoopsDB->prefix('chess_games');
+	$games_table = $xoopsDB->prefix('chess_games');
 
-    $ratings_table = $xoopsDB->prefix('chess_ratings');
+	$ratings_table = $xoopsDB->prefix('chess_ratings');
 
-    // Nuke the current ratings.  #*#TBD# - don't want to empty this table, since there will be other info in it besides ratings (?)
+	// Nuke the current ratings.  #*#TBD# - don't want to empty this table, since there will be other info in it besides ratings (?)
 
-    $xoopsDB->query("DELETE FROM $ratings_table");
+	$xoopsDB->query("DELETE FROM $ratings_table");
 
-    // get all games
+	// get all games
 
-    $result = $xoopsDB->query(
-        "
+	$result = $xoopsDB->query(
+		"
         SELECT    white_uid, black_uid, pgn_result
         FROM      $games_table
         WHERE     is_rated = '1' AND pgn_result != '*' AND white_uid != black_uid
         ORDER BY  last_date ASC
     "
-    );
+	);
 
-    $players = [];
+	$players = [];
 
-    // process the games
+	// process the games
 
-    while (false !== ($row = $xoopsDB->fetchArray($result))) {
-        #var_dump($row);#*#DEBUG#
+	while (false !== ($row = $xoopsDB->fetchArray($result))) {
+		#var_dump($row);#*#DEBUG#
 
-        if (!isset($players[$row['white_uid']])) {
-            $players[$row['white_uid']] = ['rating' => $init_rating, 'games_won' => 0, 'games_lost' => 0, 'games_drawn' => 0];
-        }
+		if (!isset($players[$row['white_uid']])) {
+			$players[$row['white_uid']] = ['rating' => $init_rating, 'games_won' => 0, 'games_lost' => 0, 'games_drawn' => 0];
+		}
 
-        if (!isset($players[$row['black_uid']])) {
-            $players[$row['black_uid']] = ['rating' => $init_rating, 'games_won' => 0, 'games_lost' => 0, 'games_drawn' => 0];
-        }
+		if (!isset($players[$row['black_uid']])) {
+			$players[$row['black_uid']] = ['rating' => $init_rating, 'games_won' => 0, 'games_lost' => 0, 'games_drawn' => 0];
+		}
 
-        $player_white = &$players[$row['white_uid']];
+		$player_white = &$players[$row['white_uid']];
 
-        $player_black = &$players[$row['black_uid']];
+		$player_black = &$players[$row['black_uid']];
 
-        // calculate new ratings using configured rating system
+		// calculate new ratings using configured rating system
 
-        [$white_rating_new, $black_rating_new] = $func(
-            $player_white['rating'],
-            $player_white['games_won'] + $player_white['games_lost'] + $player_white['games_drawn'],
-            $player_black['rating'],
-            $player_black['games_won'] + $player_black['games_lost'] + $player_black['games_drawn'],
-            $row['pgn_result']
-        );
+		[$white_rating_new, $black_rating_new] = $func(
+			$player_white['rating'],
+			$player_white['games_won'] + $player_white['games_lost'] + $player_white['games_drawn'],
+			$player_black['rating'],
+			$player_black['games_won'] + $player_black['games_lost'] + $player_black['games_drawn'],
+			$row['pgn_result']
+		);
 
-        // determine game-count columns to increment
+		// determine game-count columns to increment
 
-        [$white_col, $black_col] = chess_ratings_get_columns($row['pgn_result']);
+		[$white_col, $black_col] = chess_ratings_get_columns($row['pgn_result']);
 
-        $player_white['rating'] = $white_rating_new;
+		$player_white['rating'] = $white_rating_new;
 
-        ++$player_white[$white_col];
+		++$player_white[$white_col];
 
-        $player_black['rating'] = $black_rating_new;
+		$player_black['rating'] = $black_rating_new;
 
-        ++$player_black[$black_col];
-    }
+		++$player_black[$black_col];
+	}
 
-    $xoopsDB->freeRecordSet($result);
+	$xoopsDB->freeRecordSet($result);
 
-    if (!empty($players)) {
-        $value_list = [];
+	if (!empty($players)) {
+		$value_list = [];
 
-        foreach ($players as $player_uid => $player) {
-            $value_list[] = "('$player_uid', '{$player['rating']}', '{$player['games_won']}', '{$player['games_lost']}', '{$player['games_drawn']}')";
-        }
+		foreach ($players as $player_uid => $player) {
+			$value_list[] = "('$player_uid', '{$player['rating']}', '{$player['games_won']}', '{$player['games_lost']}', '{$player['games_drawn']}')";
+		}
 
-        $values = implode(',', $value_list);
+		$values = implode(',', $value_list);
 
-        $xoopsDB->query("INSERT INTO $ratings_table (player_uid, rating, games_won, games_lost, games_drawn) VALUES $values");
+		$xoopsDB->query("INSERT INTO $ratings_table (player_uid, rating, games_won, games_lost, games_drawn) VALUES $values");
 
-        $xoopsDB->errno() and trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
-    }
+		$xoopsDB->errno() and trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
+	}
 
-    return true;
+	return true;
 }
 
 /**
@@ -253,25 +253,25 @@ 
  */
 function chess_ratings_num_provisional_games()
 {
-    $rating_system = chess_moduleConfig('rating_system');
+	$rating_system = chess_moduleConfig('rating_system');
 
-    if ('none' == $rating_system) {
-        return 0;
-    }
+	if ('none' == $rating_system) {
+		return 0;
+	}
 
-    // determine function for getting number of provisional games using configured rating system
+	// determine function for getting number of provisional games using configured rating system
 
-    $file = XOOPS_ROOT_PATH . "/modules/chess/include/ratings_{$rating_system}.php";
+	$file = XOOPS_ROOT_PATH . "/modules/chess/include/ratings_{$rating_system}.php";
 
-    file_exists($file) or trigger_error("missing file '$file' for rating system '$rating_system'", E_USER_ERROR);
+	file_exists($file) or trigger_error("missing file '$file' for rating system '$rating_system'", E_USER_ERROR);
 
-    require_once $file;
+	require_once $file;
 
-    $func = "chess_ratings_num_provisional_games_{$rating_system}";
+	$func = "chess_ratings_num_provisional_games_{$rating_system}";
 
-    function_exists($func) or trigger_error("missing function '$func' for rating system '$rating_system'", E_USER_ERROR);
+	function_exists($func) or trigger_error("missing function '$func' for rating system '$rating_system'", E_USER_ERROR);
 
-    return $func();
+	return $func();
 }
 
 /**
@@ -282,17 +282,17 @@ 
  */
 function chess_ratings_get_func_adj($rating_system)
 {
-    $file = XOOPS_ROOT_PATH . "/modules/chess/include/ratings_{$rating_system}.php";
+	$file = XOOPS_ROOT_PATH . "/modules/chess/include/ratings_{$rating_system}.php";
 
-    file_exists($file) or trigger_error("missing file '$file' for rating system '$rating_system'", E_USER_ERROR);
+	file_exists($file) or trigger_error("missing file '$file' for rating system '$rating_system'", E_USER_ERROR);
 
-    require_once $file;
+	require_once $file;
 
-    $func = "chess_ratings_adj_{$rating_system}";
+	$func = "chess_ratings_adj_{$rating_system}";
 
-    function_exists($func) or trigger_error("missing function '$func' for rating system '$rating_system'", E_USER_ERROR);
+	function_exists($func) or trigger_error("missing function '$func' for rating system '$rating_system'", E_USER_ERROR);
 
-    return $func;
+	return $func;
 }
 
 /**
@@ -307,21 +307,21 @@ 
  */
 function chess_ratings_get_columns($pgn_result)
 {
-    switch ($pgn_result) {
-        case '1-0':
-            $white_col = 'games_won';
-            $black_col = 'games_lost';
-            break;
-        case '1/2-1/2':
-        default: // should not occur
-            $white_col = 'games_drawn';
-            $black_col = 'games_drawn';
-            break;
-        case '0-1':
-            $white_col = 'games_lost';
-            $black_col = 'games_won';
-            break;
-    }
-
-    return [$white_col, $black_col];
+	switch ($pgn_result) {
+		case '1-0':
+			$white_col = 'games_won';
+			$black_col = 'games_lost';
+			break;
+		case '1/2-1/2':
+		default: // should not occur
+			$white_col = 'games_drawn';
+			$black_col = 'games_drawn';
+			break;
+		case '0-1':
+			$white_col = 'games_lost';
+			$black_col = 'games_won';
+			break;
+	}
+
+	return [$white_col, $black_col];
 }

Spacing +7 added lines, -7 removed lines

@@ -30,7 +30,7 @@ 
 
 /**#@+
  */
-require_once XOOPS_ROOT_PATH . '/modules/chess/include/functions.php';
+require_once XOOPS_ROOT_PATH.'/modules/chess/include/functions.php';
 /**#@-*/
 
 /**
@@ -111,7 +111,7 @@ 
 
         $xoopsDB->query("INSERT INTO $ratings_table (player_uid, rating) VALUES $values");
 
-        $xoopsDB->errno() and trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
+        $xoopsDB->errno() and trigger_error($xoopsDB->errno().':'.$xoopsDB->error(), E_USER_ERROR);
     }
 
     // calculate new ratings using configured rating system
@@ -130,7 +130,7 @@ 
     "
     );
 
-    $xoopsDB->errno() and trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
+    $xoopsDB->errno() and trigger_error($xoopsDB->errno().':'.$xoopsDB->error(), E_USER_ERROR);
 
     $xoopsDB->query(
         "
@@ -140,7 +140,7 @@ 
     "
     );
 
-    $xoopsDB->errno() and trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
+    $xoopsDB->errno() and trigger_error($xoopsDB->errno().':'.$xoopsDB->error(), E_USER_ERROR);
 
     return true;
 }
@@ -240,7 +240,7 @@ 
 
         $xoopsDB->query("INSERT INTO $ratings_table (player_uid, rating, games_won, games_lost, games_drawn) VALUES $values");
 
-        $xoopsDB->errno() and trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
+        $xoopsDB->errno() and trigger_error($xoopsDB->errno().':'.$xoopsDB->error(), E_USER_ERROR);
     }
 
     return true;
@@ -261,7 +261,7 @@ 
 
     // determine function for getting number of provisional games using configured rating system
 
-    $file = XOOPS_ROOT_PATH . "/modules/chess/include/ratings_{$rating_system}.php";
+    $file = XOOPS_ROOT_PATH."/modules/chess/include/ratings_{$rating_system}.php";
 
     file_exists($file) or trigger_error("missing file '$file' for rating system '$rating_system'", E_USER_ERROR);
 
@@ -282,7 +282,7 @@ 
  */
 function chess_ratings_get_func_adj($rating_system)
 {
-    $file = XOOPS_ROOT_PATH . "/modules/chess/include/ratings_{$rating_system}.php";
+    $file = XOOPS_ROOT_PATH."/modules/chess/include/ratings_{$rating_system}.php";
 
     file_exists($file) or trigger_error("missing file '$file' for rating system '$rating_system'", E_USER_ERROR);
 

include/install.inc.php

Indentation +156 added lines, -156 removed lines

@@ -49,74 +49,74 @@ 
  */
 function xoops_module_pre_update_chess($module, $oldversion)
 {
-    global $xoopsDB;
-
-    // For downward-compatiblity, in case this function doesn't get called by the module handler.
-    $GLOBALS['chess_module_pre_update_called'] = true;
-
-    if ($oldversion < 102) { // old version < 1.02: direct update not supported.
-
-        $docfile = XOOPS_ROOT_PATH . '/modules/chess/docs/INSTALL.TXT';
-        chess_set_message($module, sprintf(_MI_CHESS_OLD_VERSION, (string)$oldversion, $docfile), true);
-        return false;
-    } elseif ($oldversion >= 107) { // old version >= 1.07:  no action needed.
-
-        return true;
-    }
-
-    // 1.02 <= old version < 1.07: perform update.
-
-    $ratings_table    = $xoopsDB->prefix('chess_ratings');
-    $challenges_table = $xoopsDB->prefix('chess_challenges');
-    $games_table      = $xoopsDB->prefix('chess_games');
-
-    // Check that ratings table does not already exist.
-    chess_set_message($module, sprintf(_MI_CHESS_RATINGS_TABLE_1, $ratings_table));
-    $result = $xoopsDB->query("SHOW TABLES LIKE '$ratings_table'");
-    if (!$result) {
-        $mysql_errno = $xoopsDB->errno();
-        $mysql_error = $xoopsDB->error();
-        chess_set_message($module, sprintf(_MI_CHESS_RATINGS_TABLE_2, $ratings_table, (string)$mysql_errno, $mysql_error), true);
-        return false;
-    }
-    if ($xoopsDB->getRowsNum($result) > 0) {
-        chess_set_message($module, sprintf(_MI_CHESS_RATINGS_TABLE_3, $ratings_table), true);
-        return false;
-    }
-    $xoopsDB->freeRecordSet($result);
-    chess_set_message($module, _MI_CHESS_OK);
-
-    // Check database tables.
-    chess_set_message($module, _MI_CHESS_CHK_DB_TABLES);
-    $table_check_messages = chess_check_tables([$challenges_table, $games_table]);
-    if (!empty($table_check_messages)) {
-        foreach ($table_check_messages as $message) {
-            chess_set_message($module, $message, true);
-        }
-        return false;
-    }
-    chess_set_message($module, _MI_CHESS_OK);
-
-    // Check that values in column pgn_result of games table are in range.
-    $pgn_result_values = "'*','1-0','0-1','1/2-1/2'";
-    chess_set_message($module, sprintf(_MI_CHESS_GAMES_TABLE_1, $games_table));
-    $result = $xoopsDB->query("SELECT COUNT(*) FROM `$games_table` WHERE `pgn_result` NOT IN ($pgn_result_values)");
-    if (!$result) {
-        $mysql_errno = $xoopsDB->errno();
-        $mysql_error = $xoopsDB->error();
-        chess_set_message($module, sprintf(_MI_CHESS_GAMES_TABLE_2, $games_table, (string)$mysql_errno, $mysql_error), true);
-        return false;
-    }
-    [$count] = $xoopsDB->fetchRow($result);
-    if ($count > 0) {
-        chess_set_message($module, sprintf(_MI_CHESS_GAMES_TABLE_3, 'pgn_result', $games_table, $pgn_result_values), true);
-        chess_set_message($module, _MI_CHESS_GAMES_TABLE_4, true);
-        return false;
-    }
-    $xoopsDB->freeRecordSet($result);
-    chess_set_message($module, _MI_CHESS_OK);
-
-    return true; // successful
+	global $xoopsDB;
+
+	// For downward-compatiblity, in case this function doesn't get called by the module handler.
+	$GLOBALS['chess_module_pre_update_called'] = true;
+
+	if ($oldversion < 102) { // old version < 1.02: direct update not supported.
+
+		$docfile = XOOPS_ROOT_PATH . '/modules/chess/docs/INSTALL.TXT';
+		chess_set_message($module, sprintf(_MI_CHESS_OLD_VERSION, (string)$oldversion, $docfile), true);
+		return false;
+	} elseif ($oldversion >= 107) { // old version >= 1.07:  no action needed.
+
+		return true;
+	}
+
+	// 1.02 <= old version < 1.07: perform update.
+
+	$ratings_table    = $xoopsDB->prefix('chess_ratings');
+	$challenges_table = $xoopsDB->prefix('chess_challenges');
+	$games_table      = $xoopsDB->prefix('chess_games');
+
+	// Check that ratings table does not already exist.
+	chess_set_message($module, sprintf(_MI_CHESS_RATINGS_TABLE_1, $ratings_table));
+	$result = $xoopsDB->query("SHOW TABLES LIKE '$ratings_table'");
+	if (!$result) {
+		$mysql_errno = $xoopsDB->errno();
+		$mysql_error = $xoopsDB->error();
+		chess_set_message($module, sprintf(_MI_CHESS_RATINGS_TABLE_2, $ratings_table, (string)$mysql_errno, $mysql_error), true);
+		return false;
+	}
+	if ($xoopsDB->getRowsNum($result) > 0) {
+		chess_set_message($module, sprintf(_MI_CHESS_RATINGS_TABLE_3, $ratings_table), true);
+		return false;
+	}
+	$xoopsDB->freeRecordSet($result);
+	chess_set_message($module, _MI_CHESS_OK);
+
+	// Check database tables.
+	chess_set_message($module, _MI_CHESS_CHK_DB_TABLES);
+	$table_check_messages = chess_check_tables([$challenges_table, $games_table]);
+	if (!empty($table_check_messages)) {
+		foreach ($table_check_messages as $message) {
+			chess_set_message($module, $message, true);
+		}
+		return false;
+	}
+	chess_set_message($module, _MI_CHESS_OK);
+
+	// Check that values in column pgn_result of games table are in range.
+	$pgn_result_values = "'*','1-0','0-1','1/2-1/2'";
+	chess_set_message($module, sprintf(_MI_CHESS_GAMES_TABLE_1, $games_table));
+	$result = $xoopsDB->query("SELECT COUNT(*) FROM `$games_table` WHERE `pgn_result` NOT IN ($pgn_result_values)");
+	if (!$result) {
+		$mysql_errno = $xoopsDB->errno();
+		$mysql_error = $xoopsDB->error();
+		chess_set_message($module, sprintf(_MI_CHESS_GAMES_TABLE_2, $games_table, (string)$mysql_errno, $mysql_error), true);
+		return false;
+	}
+	[$count] = $xoopsDB->fetchRow($result);
+	if ($count > 0) {
+		chess_set_message($module, sprintf(_MI_CHESS_GAMES_TABLE_3, 'pgn_result', $games_table, $pgn_result_values), true);
+		chess_set_message($module, _MI_CHESS_GAMES_TABLE_4, true);
+		return false;
+	}
+	$xoopsDB->freeRecordSet($result);
+	chess_set_message($module, _MI_CHESS_OK);
+
+	return true; // successful
 }
 
 /**
@@ -128,25 +128,25 @@ 
  */
 function xoops_module_update_chess(&$module, $oldversion)
 {
-    global $xoopsDB;
+	global $xoopsDB;
 
-    // Before proceeding, ensure that pre-update processing has been done, and that all the checks pass.
-    // For downward-compatiblity, in case the "pre_update" function doesn't get called by the module handler.
-    if (!@$GLOBALS['chess_module_pre_update_called'] && !xoops_module_pre_update_chess($module, $oldversion)) {
-        return false;
-    }
+	// Before proceeding, ensure that pre-update processing has been done, and that all the checks pass.
+	// For downward-compatiblity, in case the "pre_update" function doesn't get called by the module handler.
+	if (!@$GLOBALS['chess_module_pre_update_called'] && !xoops_module_pre_update_chess($module, $oldversion)) {
+		return false;
+	}
 
-    if ($oldversion >= 107) { // old version >= 1.07:  no action needed.
-        return true;
-    }
+	if ($oldversion >= 107) { // old version >= 1.07:  no action needed.
+		return true;
+	}
 
-    $ratings_table    = $xoopsDB->prefix('chess_ratings');
-    $challenges_table = $xoopsDB->prefix('chess_challenges');
-    $games_table      = $xoopsDB->prefix('chess_games');
+	$ratings_table    = $xoopsDB->prefix('chess_ratings');
+	$challenges_table = $xoopsDB->prefix('chess_challenges');
+	$games_table      = $xoopsDB->prefix('chess_games');
 
-    $queries = [
+	$queries = [
 
-        "CREATE TABLE `$ratings_table` (
+		"CREATE TABLE `$ratings_table` (
             `player_uid` mediumint(8) unsigned NOT NULL default '0',
             `rating` smallint(6) unsigned NOT NULL default '1200',
             `games_won` smallint(6) unsigned NOT NULL default '0',
@@ -157,40 +157,40 @@ 
             KEY `games` (`games_won`,`games_lost`,`games_drawn`)
             ) TYPE=MyISAM",
 
-        "ALTER TABLE `$challenges_table` ADD `is_rated` ENUM('1','0') DEFAULT '1' NOT NULL",
-        "ALTER TABLE `$challenges_table` ADD INDEX `game_type` (`game_type`)",
-        "ALTER TABLE `$challenges_table` ADD INDEX `player1_uid` (`player1_uid`)",
-        "ALTER TABLE `$challenges_table` ADD INDEX `player2_uid` (`player2_uid`)",
-        "ALTER TABLE `$challenges_table` ADD INDEX `create_date` (`create_date`)",
-        "ALTER TABLE `$challenges_table` ADD INDEX `is_rated` (`is_rated`)",
-
-        "ALTER TABLE `$games_table` CHANGE `pgn_result` `pgn_result` ENUM('*','0-1','1-0','1/2-1/2') DEFAULT '*' NOT NULL",
-        "ALTER TABLE `$games_table` ADD `is_rated` ENUM('1','0') DEFAULT '1' NOT NULL",
-        "ALTER TABLE `$games_table` ADD INDEX `white_uid` (`white_uid`)",
-        "ALTER TABLE `$games_table` ADD INDEX `black_uid` (`black_uid`)",
-        "ALTER TABLE `$games_table` ADD INDEX `date` (`create_date`,`start_date`,`last_date`)",
-        "ALTER TABLE `$games_table` ADD INDEX `pgn_result` (`pgn_result`)",
-        "ALTER TABLE `$games_table` ADD INDEX `suspended_date` (`suspended`(19))",
-        "ALTER TABLE `$games_table` ADD INDEX `is_rated` (`is_rated`)",
-
-        "UPDATE `$games_table` SET `is_rated` = '0' WHERE `white_uid` = `black_uid`",
-    ];
-
-    // Update database tables.
-    chess_set_message($module, _MI_CHESS_UPDATING_DATABASE);
-    foreach ($queries as $query) {
-        chess_set_message($module, "> $query");
-        $result = $xoopsDB->query($query);
-        if (!$result) {
-            $mysql_errno = $xoopsDB->errno();
-            $mysql_error = $xoopsDB->error();
-            chess_set_message($module, " ... ($mysql_errno) $mysql_error");
-            return false;
-        }
-        chess_set_message($module, _MI_CHESS_OK);
-    }
-
-    /***
+		"ALTER TABLE `$challenges_table` ADD `is_rated` ENUM('1','0') DEFAULT '1' NOT NULL",
+		"ALTER TABLE `$challenges_table` ADD INDEX `game_type` (`game_type`)",
+		"ALTER TABLE `$challenges_table` ADD INDEX `player1_uid` (`player1_uid`)",
+		"ALTER TABLE `$challenges_table` ADD INDEX `player2_uid` (`player2_uid`)",
+		"ALTER TABLE `$challenges_table` ADD INDEX `create_date` (`create_date`)",
+		"ALTER TABLE `$challenges_table` ADD INDEX `is_rated` (`is_rated`)",
+
+		"ALTER TABLE `$games_table` CHANGE `pgn_result` `pgn_result` ENUM('*','0-1','1-0','1/2-1/2') DEFAULT '*' NOT NULL",
+		"ALTER TABLE `$games_table` ADD `is_rated` ENUM('1','0') DEFAULT '1' NOT NULL",
+		"ALTER TABLE `$games_table` ADD INDEX `white_uid` (`white_uid`)",
+		"ALTER TABLE `$games_table` ADD INDEX `black_uid` (`black_uid`)",
+		"ALTER TABLE `$games_table` ADD INDEX `date` (`create_date`,`start_date`,`last_date`)",
+		"ALTER TABLE `$games_table` ADD INDEX `pgn_result` (`pgn_result`)",
+		"ALTER TABLE `$games_table` ADD INDEX `suspended_date` (`suspended`(19))",
+		"ALTER TABLE `$games_table` ADD INDEX `is_rated` (`is_rated`)",
+
+		"UPDATE `$games_table` SET `is_rated` = '0' WHERE `white_uid` = `black_uid`",
+	];
+
+	// Update database tables.
+	chess_set_message($module, _MI_CHESS_UPDATING_DATABASE);
+	foreach ($queries as $query) {
+		chess_set_message($module, "> $query");
+		$result = $xoopsDB->query($query);
+		if (!$result) {
+			$mysql_errno = $xoopsDB->errno();
+			$mysql_error = $xoopsDB->error();
+			chess_set_message($module, " ... ($mysql_errno) $mysql_error");
+			return false;
+		}
+		chess_set_message($module, _MI_CHESS_OK);
+	}
+
+	/***
      * #*#TODO# - Leave this here for now, in case I think of a way to get it to work.
      * # This causes an error about the rating_system module configuration parameter not being defined,
      * # so I added a note in INSTALL.TXT about manually recalculating the ratings after install.
@@ -201,9 +201,9 @@ 
      * chess_recalc_ratings();
      ***/
 
-    chess_set_message($module, _MI_CHESS_UPDATE_SUCCESSFUL);
+	chess_set_message($module, _MI_CHESS_UPDATE_SUCCESSFUL);
 
-    return true; // successful
+	return true; // successful
 }
 
 /**
@@ -214,37 +214,37 @@ 
  */
 function chess_check_tables($table_names)
 {
-    global $xoopsDB;
+	global $xoopsDB;
 
-    $messages = [];
+	$messages = [];
 
-    foreach ($table_names as $table_name) {
-        $query  = "CHECK TABLE `$table_name`";
-        $result = $xoopsDB->query($query);
-        if (!$result) {
-            $mysql_errno = $xoopsDB->errno();
-            $mysql_error = $xoopsDB->error();
-            $messages[]  = $query;
-            $messages[]  = " ... ($mysql_errno) $mysql_error";
-            continue;
-        }
+	foreach ($table_names as $table_name) {
+		$query  = "CHECK TABLE `$table_name`";
+		$result = $xoopsDB->query($query);
+		if (!$result) {
+			$mysql_errno = $xoopsDB->errno();
+			$mysql_error = $xoopsDB->error();
+			$messages[]  = $query;
+			$messages[]  = " ... ($mysql_errno) $mysql_error";
+			continue;
+		}
 
-        // Initialize, in case the real table status fails to get retrieved.
-        $table_status = '*** STATUS UNKNOWN ***';
+		// Initialize, in case the real table status fails to get retrieved.
+		$table_status = '*** STATUS UNKNOWN ***';
 
-        // The query may return multiple rows.  Only the last row is normally of interest, so only that row is saved.
-        while ($row = $xoopsDB->fetchArray($result)) {
-            $table_status = $row['Msg_text'];
-        }
+		// The query may return multiple rows.  Only the last row is normally of interest, so only that row is saved.
+		while ($row = $xoopsDB->fetchArray($result)) {
+			$table_status = $row['Msg_text'];
+		}
 
-        $xoopsDB->freeRecordSet($result);
+		$xoopsDB->freeRecordSet($result);
 
-        if ('OK' != $table_status) {
-            $messages[] = " ... $table_name: $table_status";
-        }
-    }
+		if ('OK' != $table_status) {
+			$messages[] = " ... $table_name: $table_status";
+		}
+	}
 
-    return $messages;
+	return $messages;
 }
 
 /**
@@ -259,12 +259,12 @@ 
  */
 function chess_load_lang_file($filename, $module = '', $default = 'english')
 {
-    $lang = $GLOBALS['xoopsConfig']['language'];
-    $path = XOOPS_ROOT_PATH . (empty($module) ? '/' : "/modules/$module/") . 'language';
-    if (!($ret = @include_once("$path/$lang/$filename.php"))) {
-        $ret = include_once("$path/$default/$filename.php");
-    }
-    return $ret;
+	$lang = $GLOBALS['xoopsConfig']['language'];
+	$path = XOOPS_ROOT_PATH . (empty($module) ? '/' : "/modules/$module/") . 'language';
+	if (!($ret = @include_once("$path/$lang/$filename.php"))) {
+		$ret = include_once("$path/$default/$filename.php");
+	}
+	return $ret;
 }
 
 /**
@@ -276,14 +276,14 @@ 
  */
 function chess_set_message($module, $text = '', $error = false)
 {
-    $text = $error ? "<span style='color:#ff0000;background-color:#ffffff;font-weight:bold;'>$text</span>" : $text;
-
-    // For downward compatibility with XOOPS versions that don't have the method XoopsModule::setMessage.
-    if (is_object($module) && method_exists($module, 'setMessage')) {
-        $module->setMessage($text);
-    } else {
-        echo "<code>$text</code><br />\n";
-    }
+	$text = $error ? "<span style='color:#ff0000;background-color:#ffffff;font-weight:bold;'>$text</span>" : $text;
+
+	// For downward compatibility with XOOPS versions that don't have the method XoopsModule::setMessage.
+	if (is_object($module) && method_exists($module, 'setMessage')) {
+		$module->setMessage($text);
+	} else {
+		echo "<code>$text</code><br />\n";
+	}
 }
 
 ?>

include/install.php

Indentation +141 added lines, -141 removed lines

@@ -50,105 +50,105 @@ 
  */
 function xoops_module_pre_update_chess($module, $oldversion)
 {
-    global $xoopsDB;
+	global $xoopsDB;
 
-    // For downward-compatiblity, in case this function doesn't get called by the module handler.
+	// For downward-compatiblity, in case this function doesn't get called by the module handler.
 
-    $GLOBALS['chess_module_pre_update_called'] = true;
+	$GLOBALS['chess_module_pre_update_called'] = true;
 
-    if ($oldversion < 102) { // old version < 1.02: direct update not supported.
-        $docfile = XOOPS_ROOT_PATH . '/modules/chess/docs/INSTALL.TXT';
+	if ($oldversion < 102) { // old version < 1.02: direct update not supported.
+		$docfile = XOOPS_ROOT_PATH . '/modules/chess/docs/INSTALL.TXT';
 
-        chess_set_message($module, sprintf(_MI_CHESS_OLD_VERSION, (string)$oldversion, $docfile), true);
+		chess_set_message($module, sprintf(_MI_CHESS_OLD_VERSION, (string)$oldversion, $docfile), true);
 
-        return false;
-    } elseif ($oldversion >= 107) { // old version >= 1.07:  no action needed.
-        return true;
-    }
+		return false;
+	} elseif ($oldversion >= 107) { // old version >= 1.07:  no action needed.
+		return true;
+	}
 
-    // 1.02 <= old version < 1.07: perform update.
+	// 1.02 <= old version < 1.07: perform update.
 
-    $ratings_table = $xoopsDB->prefix('chess_ratings');
+	$ratings_table = $xoopsDB->prefix('chess_ratings');
 
-    $challenges_table = $xoopsDB->prefix('chess_challenges');
+	$challenges_table = $xoopsDB->prefix('chess_challenges');
 
-    $games_table = $xoopsDB->prefix('chess_games');
+	$games_table = $xoopsDB->prefix('chess_games');
 
-    // Check that ratings table does not already exist.
+	// Check that ratings table does not already exist.
 
-    chess_set_message($module, sprintf(_MI_CHESS_RATINGS_TABLE_1, $ratings_table));
+	chess_set_message($module, sprintf(_MI_CHESS_RATINGS_TABLE_1, $ratings_table));
 
-    $result = $xoopsDB->query("SHOW TABLES LIKE '$ratings_table'");
+	$result = $xoopsDB->query("SHOW TABLES LIKE '$ratings_table'");
 
-    if (!$result) {
-        $mysql_errno = $xoopsDB->errno();
+	if (!$result) {
+		$mysql_errno = $xoopsDB->errno();
 
-        $mysql_error = $xoopsDB->error();
+		$mysql_error = $xoopsDB->error();
 
-        chess_set_message($module, sprintf(_MI_CHESS_RATINGS_TABLE_2, $ratings_table, (string)$mysql_errno, $mysql_error), true);
+		chess_set_message($module, sprintf(_MI_CHESS_RATINGS_TABLE_2, $ratings_table, (string)$mysql_errno, $mysql_error), true);
 
-        return false;
-    }
+		return false;
+	}
 
-    if ($xoopsDB->getRowsNum($result) > 0) {
-        chess_set_message($module, sprintf(_MI_CHESS_RATINGS_TABLE_3, $ratings_table), true);
+	if ($xoopsDB->getRowsNum($result) > 0) {
+		chess_set_message($module, sprintf(_MI_CHESS_RATINGS_TABLE_3, $ratings_table), true);
 
-        return false;
-    }
+		return false;
+	}
 
-    $xoopsDB->freeRecordSet($result);
+	$xoopsDB->freeRecordSet($result);
 
-    chess_set_message($module, _MI_CHESS_OK);
+	chess_set_message($module, _MI_CHESS_OK);
 
-    // Check database tables.
+	// Check database tables.
 
-    chess_set_message($module, _MI_CHESS_CHK_DB_TABLES);
+	chess_set_message($module, _MI_CHESS_CHK_DB_TABLES);
 
-    $table_check_messages = chess_check_tables([$challenges_table, $games_table]);
+	$table_check_messages = chess_check_tables([$challenges_table, $games_table]);
 
-    if (!empty($table_check_messages)) {
-        foreach ($table_check_messages as $message) {
-            chess_set_message($module, $message, true);
-        }
+	if (!empty($table_check_messages)) {
+		foreach ($table_check_messages as $message) {
+			chess_set_message($module, $message, true);
+		}
 
-        return false;
-    }
+		return false;
+	}
 
-    chess_set_message($module, _MI_CHESS_OK);
+	chess_set_message($module, _MI_CHESS_OK);
 
-    // Check that values in column pgn_result of games table are in range.
+	// Check that values in column pgn_result of games table are in range.
 
-    $pgn_result_values = "'*','1-0','0-1','1/2-1/2'";
+	$pgn_result_values = "'*','1-0','0-1','1/2-1/2'";
 
-    chess_set_message($module, sprintf(_MI_CHESS_GAMES_TABLE_1, $games_table));
+	chess_set_message($module, sprintf(_MI_CHESS_GAMES_TABLE_1, $games_table));
 
-    $result = $xoopsDB->query("SELECT COUNT(*) FROM `$games_table` WHERE `pgn_result` NOT IN ($pgn_result_values)");
+	$result = $xoopsDB->query("SELECT COUNT(*) FROM `$games_table` WHERE `pgn_result` NOT IN ($pgn_result_values)");
 
-    if (!$result) {
-        $mysql_errno = $xoopsDB->errno();
+	if (!$result) {
+		$mysql_errno = $xoopsDB->errno();
 
-        $mysql_error = $xoopsDB->error();
+		$mysql_error = $xoopsDB->error();
 
-        chess_set_message($module, sprintf(_MI_CHESS_GAMES_TABLE_2, $games_table, (string)$mysql_errno, $mysql_error), true);
+		chess_set_message($module, sprintf(_MI_CHESS_GAMES_TABLE_2, $games_table, (string)$mysql_errno, $mysql_error), true);
 
-        return false;
-    }
+		return false;
+	}
 
-    [$count] = $xoopsDB->fetchRow($result);
+	[$count] = $xoopsDB->fetchRow($result);
 
-    if ($count > 0) {
-        chess_set_message($module, sprintf(_MI_CHESS_GAMES_TABLE_3, 'pgn_result', $games_table, $pgn_result_values), true);
+	if ($count > 0) {
+		chess_set_message($module, sprintf(_MI_CHESS_GAMES_TABLE_3, 'pgn_result', $games_table, $pgn_result_values), true);
 
-        chess_set_message($module, _MI_CHESS_GAMES_TABLE_4, true);
+		chess_set_message($module, _MI_CHESS_GAMES_TABLE_4, true);
 
-        return false;
-    }
+		return false;
+	}
 
-    $xoopsDB->freeRecordSet($result);
+	$xoopsDB->freeRecordSet($result);
 
-    chess_set_message($module, _MI_CHESS_OK);
+	chess_set_message($module, _MI_CHESS_OK);
 
-    return true; // successful
+	return true; // successful
 }
 
 /**
@@ -160,28 +160,28 @@ 
  */
 function xoops_module_update_chess($module, $oldversion)
 {
-    global $xoopsDB;
+	global $xoopsDB;
 
-    // Before proceeding, ensure that pre-update processing has been done, and that all the checks pass.
+	// Before proceeding, ensure that pre-update processing has been done, and that all the checks pass.
 
-    // For downward-compatiblity, in case the "pre_update" function doesn't get called by the module handler.
+	// For downward-compatiblity, in case the "pre_update" function doesn't get called by the module handler.
 
-    if (!@$GLOBALS['chess_module_pre_update_called'] && !xoops_module_pre_update_chess($module, $oldversion)) {
-        return false;
-    }
+	if (!@$GLOBALS['chess_module_pre_update_called'] && !xoops_module_pre_update_chess($module, $oldversion)) {
+		return false;
+	}
 
-    if ($oldversion >= 107) { // old version >= 1.07:  no action needed.
-        return true;
-    }
+	if ($oldversion >= 107) { // old version >= 1.07:  no action needed.
+		return true;
+	}
 
-    $ratings_table = $xoopsDB->prefix('chess_ratings');
+	$ratings_table = $xoopsDB->prefix('chess_ratings');
 
-    $challenges_table = $xoopsDB->prefix('chess_challenges');
+	$challenges_table = $xoopsDB->prefix('chess_challenges');
 
-    $games_table = $xoopsDB->prefix('chess_games');
+	$games_table = $xoopsDB->prefix('chess_games');
 
-    $queries = [
-        "CREATE TABLE `$ratings_table` (
+	$queries = [
+		"CREATE TABLE `$ratings_table` (
             `player_uid` mediumint(8) unsigned NOT NULL default '0',
             `rating` smallint(6) unsigned NOT NULL default '1200',
             `games_won` smallint(6) unsigned NOT NULL default '0',
@@ -192,48 +192,48 @@ 
             KEY `games` (`games_won`,`games_lost`,`games_drawn`)
             ) ENGINE=MyISAM",
 
-        "ALTER TABLE `$challenges_table` ADD `is_rated` ENUM('1','0') DEFAULT '1' NOT NULL",
-        "ALTER TABLE `$challenges_table` ADD INDEX `game_type` (`game_type`)",
-        "ALTER TABLE `$challenges_table` ADD INDEX `player1_uid` (`player1_uid`)",
-        "ALTER TABLE `$challenges_table` ADD INDEX `player2_uid` (`player2_uid`)",
-        "ALTER TABLE `$challenges_table` ADD INDEX `create_date` (`create_date`)",
-        "ALTER TABLE `$challenges_table` ADD INDEX `is_rated` (`is_rated`)",
+		"ALTER TABLE `$challenges_table` ADD `is_rated` ENUM('1','0') DEFAULT '1' NOT NULL",
+		"ALTER TABLE `$challenges_table` ADD INDEX `game_type` (`game_type`)",
+		"ALTER TABLE `$challenges_table` ADD INDEX `player1_uid` (`player1_uid`)",
+		"ALTER TABLE `$challenges_table` ADD INDEX `player2_uid` (`player2_uid`)",
+		"ALTER TABLE `$challenges_table` ADD INDEX `create_date` (`create_date`)",
+		"ALTER TABLE `$challenges_table` ADD INDEX `is_rated` (`is_rated`)",
 
-        "ALTER TABLE `$games_table` CHANGE `pgn_result` `pgn_result` ENUM('*','0-1','1-0','1/2-1/2') DEFAULT '*' NOT NULL",
-        "ALTER TABLE `$games_table` ADD `is_rated` ENUM('1','0') DEFAULT '1' NOT NULL",
-        "ALTER TABLE `$games_table` ADD INDEX `white_uid` (`white_uid`)",
-        "ALTER TABLE `$games_table` ADD INDEX `black_uid` (`black_uid`)",
-        "ALTER TABLE `$games_table` ADD INDEX `date` (`create_date`,`start_date`,`last_date`)",
-        "ALTER TABLE `$games_table` ADD INDEX `pgn_result` (`pgn_result`)",
-        "ALTER TABLE `$games_table` ADD INDEX `suspended_date` (`suspended`(19))",
-        "ALTER TABLE `$games_table` ADD INDEX `is_rated` (`is_rated`)",
+		"ALTER TABLE `$games_table` CHANGE `pgn_result` `pgn_result` ENUM('*','0-1','1-0','1/2-1/2') DEFAULT '*' NOT NULL",
+		"ALTER TABLE `$games_table` ADD `is_rated` ENUM('1','0') DEFAULT '1' NOT NULL",
+		"ALTER TABLE `$games_table` ADD INDEX `white_uid` (`white_uid`)",
+		"ALTER TABLE `$games_table` ADD INDEX `black_uid` (`black_uid`)",
+		"ALTER TABLE `$games_table` ADD INDEX `date` (`create_date`,`start_date`,`last_date`)",
+		"ALTER TABLE `$games_table` ADD INDEX `pgn_result` (`pgn_result`)",
+		"ALTER TABLE `$games_table` ADD INDEX `suspended_date` (`suspended`(19))",
+		"ALTER TABLE `$games_table` ADD INDEX `is_rated` (`is_rated`)",
 
-        "UPDATE `$games_table` SET `is_rated` = '0' WHERE `white_uid` = `black_uid`",
-    ];
+		"UPDATE `$games_table` SET `is_rated` = '0' WHERE `white_uid` = `black_uid`",
+	];
 
-    // Update database tables.
+	// Update database tables.
 
-    chess_set_message($module, _MI_CHESS_UPDATING_DATABASE);
+	chess_set_message($module, _MI_CHESS_UPDATING_DATABASE);
 
-    foreach ($queries as $query) {
-        chess_set_message($module, "> $query");
+	foreach ($queries as $query) {
+		chess_set_message($module, "> $query");
 
-        $result = $xoopsDB->query($query);
+		$result = $xoopsDB->query($query);
 
-        if (!$result) {
-            $mysql_errno = $xoopsDB->errno();
+		if (!$result) {
+			$mysql_errno = $xoopsDB->errno();
 
-            $mysql_error = $xoopsDB->error();
+			$mysql_error = $xoopsDB->error();
 
-            chess_set_message($module, " ... ($mysql_errno) $mysql_error");
+			chess_set_message($module, " ... ($mysql_errno) $mysql_error");
 
-            return false;
-        }
+			return false;
+		}
 
-        chess_set_message($module, _MI_CHESS_OK);
-    }
+		chess_set_message($module, _MI_CHESS_OK);
+	}
 
-    /***
+	/***
      * #*#TODO# - Leave this here for now, in case I think of a way to get it to work.
      * # This causes an error about the rating_system module configuration parameter not being defined,
      * # so I added a note in INSTALL.TXT about manually recalculating the ratings after install.
@@ -244,9 +244,9 @@ 
      * chess_recalc_ratings();
      ***/
 
-    chess_set_message($module, _MI_CHESS_UPDATE_SUCCESSFUL);
+	chess_set_message($module, _MI_CHESS_UPDATE_SUCCESSFUL);
 
-    return true; // successful
+	return true; // successful
 }
 
 /**
@@ -257,45 +257,45 @@ 
  */
 function chess_check_tables($table_names)
 {
-    global $xoopsDB;
+	global $xoopsDB;
 
-    $messages = [];
+	$messages = [];
 
-    foreach ($table_names as $table_name) {
-        $query = "CHECK TABLE `$table_name`";
+	foreach ($table_names as $table_name) {
+		$query = "CHECK TABLE `$table_name`";
 
-        $result = $xoopsDB->query($query);
+		$result = $xoopsDB->query($query);
 
-        if (!$result) {
-            $mysql_errno = $xoopsDB->errno();
+		if (!$result) {
+			$mysql_errno = $xoopsDB->errno();
 
-            $mysql_error = $xoopsDB->error();
+			$mysql_error = $xoopsDB->error();
 
-            $messages[] = $query;
+			$messages[] = $query;
 
-            $messages[] = " ... ($mysql_errno) $mysql_error";
+			$messages[] = " ... ($mysql_errno) $mysql_error";
 
-            continue;
-        }
+			continue;
+		}
 
-        // Initialize, in case the real table status fails to get retrieved.
+		// Initialize, in case the real table status fails to get retrieved.
 
-        $table_status = '*** STATUS UNKNOWN ***';
+		$table_status = '*** STATUS UNKNOWN ***';
 
-        // The query may return multiple rows.  Only the last row is normally of interest, so only that row is saved.
+		// The query may return multiple rows.  Only the last row is normally of interest, so only that row is saved.
 
-        while (false !== ($row = $xoopsDB->fetchArray($result))) {
-            $table_status = $row['Msg_text'];
-        }
+		while (false !== ($row = $xoopsDB->fetchArray($result))) {
+			$table_status = $row['Msg_text'];
+		}
 
-        $xoopsDB->freeRecordSet($result);
+		$xoopsDB->freeRecordSet($result);
 
-        if ('OK' != $table_status) {
-            $messages[] = " ... $table_name: $table_status";
-        }
-    }
+		if ('OK' != $table_status) {
+			$messages[] = " ... $table_name: $table_status";
+		}
+	}
 
-    return $messages;
+	return $messages;
 }
 
 /**
@@ -310,15 +310,15 @@ 
  */
 function chess_load_lang_file($filename, $module = '', $default = 'english')
 {
-    $lang = $GLOBALS['xoopsConfig']['language'];
+	$lang = $GLOBALS['xoopsConfig']['language'];
 
-    $path = XOOPS_ROOT_PATH . (empty($module) ? '/' : "/modules/$module/") . 'language';
+	$path = XOOPS_ROOT_PATH . (empty($module) ? '/' : "/modules/$module/") . 'language';
 
-    if (!($ret = @include_once("$path/$lang/$filename.php"))) {
-        $ret = include_once("$path/$default/$filename.php");
-    }
+	if (!($ret = @include_once("$path/$lang/$filename.php"))) {
+		$ret = include_once("$path/$default/$filename.php");
+	}
 
-    return $ret;
+	return $ret;
 }
 
 /**
@@ -330,13 +330,13 @@ 
  */
 function chess_set_message($module, $text = '', $error = false)
 {
-    $text = $error ? "<span style='color:#ff0000;background-color:#ffffff;font-weight:bold;'>$text</span>" : $text;
+	$text = $error ? "<span style='color:#ff0000;background-color:#ffffff;font-weight:bold;'>$text</span>" : $text;
 
-    // For downward compatibility with XOOPS versions that don't have the method XoopsModule::setMessage.
+	// For downward compatibility with XOOPS versions that don't have the method XoopsModule::setMessage.
 
-    if (is_object($module) && method_exists($module, 'setMessage')) {
-        $module->setMessage($text);
-    } else {
-        echo "<code>$text</code><br>\n";
-    }
+	if (is_object($module) && method_exists($module, 'setMessage')) {
+		$module->setMessage($text);
+	} else {
+		echo "<code>$text</code><br>\n";
+	}
 }

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 +169 added lines, -169 removed lines

@@ -47,13 +47,13 @@ 
 
 #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)
 $cstart = Request::getInt('cstart', 0);
@@ -71,70 +71,70 @@ 
 $gshow2 = Request::getInt('gshow2', 0);
 //$gshow2 = intval(isset($_POST['gshow2']) ? $_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
@@ -142,211 +142,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);

language/english/help.php

Indentation +94 added lines, -94 removed lines

@@ -22,8 +22,8 @@ 
 define('_HE_CHESS_INTRO_TITLE', 'Introduction');
 
 define(
-    '_HE_CHESS_INTRO_010',
-    '
+	'_HE_CHESS_INTRO_010',
+	'
 This software is intended to comply with the <a href="http://fide.com/official/handbook.asp?level=EE1">Laws of Chess</a>
 as stipulated by the F&eacute;d&eacute;ration Internationale des &Eacute;checs (International Chess Federation),
 commonly known as FIDE.
@@ -35,8 +35,8 @@ 
 define('_HE_CHESS_CREATE_TITLE', 'Creating new games');
 
 define(
-    '_HE_CHESS_CREATE_010',
-    '
+	'_HE_CHESS_CREATE_010',
+	'
 There are three types of games that you may create:
 '
 );
@@ -45,8 +45,8 @@ 
 define('_HE_CHESS_CREATE_010_3', 'Self-play');
 
 define(
-    '_HE_CHESS_CREATE_020',
-    '
+	'_HE_CHESS_CREATE_020',
+	'
 An <em>open challenge</em> may be accepted by anyone.
 An <em>individual challenge</em> may only be accepted by a specified user.
 A <em>self-play</em> game may be created if you wish to experiment with this software.
@@ -54,15 +54,15 @@ 
 );
 
 define(
-    '_HE_CHESS_CREATE_030',
-    '
+	'_HE_CHESS_CREATE_030',
+	'
     You may optionally provide an initial board setup position using Forsyth-Edwards Notation (FEN).
 '
 );
 
 define(
-    '_HE_CHESS_CREATE_040',
-    '
+	'_HE_CHESS_CREATE_040',
+	'
 You may specify that your opponent will choose the colors,
 that the colors will be assigned randomly,
 or you may choose a specific color.
@@ -70,23 +70,23 @@ 
 );
 
 define(
-    '_HE_CHESS_CREATE_050',
-    '
+	'_HE_CHESS_CREATE_050',
+	'
     You may specify whether the game will be rated.
 '
 );
 
 define(
-    '_HE_CHESS_CREATE_060',
-    '
+	'_HE_CHESS_CREATE_060',
+	'
 A challenge game will be created as soon as another user accepts the challenge.
 A self-play game will be created immediately.
 '
 );
 
 define(
-    '_HE_CHESS_CREATE_070',
-    '
+	'_HE_CHESS_CREATE_070',
+	'
 A challenger may delete his challenge at any time prior to its acceptance.
 '
 );
@@ -96,15 +96,15 @@ 
 define('_HE_CHESS_RATINGS_TITLE', 'Player rating system');
 
 define(
-    '_HE_CHESS_RATINGS_CXR',
-    '
+	'_HE_CHESS_RATINGS_CXR',
+	'
     CXR - An adaptation of the ELO rating system, used by permission of <a target="_blank" href="http://chess-express.com/">Chess Express Ratings, Inc.</a>
 '
 );
 
 define(
-    '_HE_CHESS_RATINGS_LINEAR',
-    '
+	'_HE_CHESS_RATINGS_LINEAR',
+	'
     Linear - A very simple system that adds (subtracts) a fixed number of points for a win (loss).
 '
 );
@@ -114,8 +114,8 @@ 
 define('_HE_CHESS_PLAY_TITLE', 'Playing a game');
 
 define(
-    '_HE_CHESS_PLAY_010',
-    '
+	'_HE_CHESS_PLAY_010',
+	'
 A player can choose from the following actions.
 Not all of these actions are available all of the time.
 '
@@ -131,16 +131,16 @@ 
 define('_HE_CHESS_PLAY_010_9', 'Request arbitration');
 
 define(
-    '_HE_CHESS_PLAY_020',
-    '
+	'_HE_CHESS_PLAY_020',
+	'
 You may make a <em>normal move</em> if it is your turn to move.
 Moving is discussed in detail <a href="#moving">below</a>.
 '
 );
 
 define(
-    '_HE_CHESS_PLAY_030',
-    '
+	'_HE_CHESS_PLAY_030',
+	'
 You may <em>claim a draw</em> if the conditions are satisfied for the 50-move rule or the threefold repetition rule,
 and it is your turn to move.
 You may optionally enter a (legal) move that will cause the conditions to be satisfied.
@@ -150,8 +150,8 @@ 
 );
 
 define(
-    '_HE_CHESS_PLAY_040',
-    '
+	'_HE_CHESS_PLAY_040',
+	'
 You may <em>offer a draw</em> either on your turn or on your opponent\'s turn.
 You may not withdraw the offer.
 Your opponent must either <em>accept</em> it or <em>reject</em> it.
@@ -164,40 +164,40 @@ 
 );
 
 define(
-    '_HE_CHESS_PLAY_050',
-    '
+	'_HE_CHESS_PLAY_050',
+	'
 You may <em>resign</em> either on your turn or on your opponent\'s turn.
 This will immediately end the game, and your opponent will be the winner.
 '
 );
 
 define(
-    '_HE_CHESS_PLAY_060',
-    '
+	'_HE_CHESS_PLAY_060',
+	'
 In a self-play game, you may <em>restart</em> the game at any time.
 This will delete all moves made in the game and restart the game at its initial position.
 '
 );
 
 define(
-    '_HE_CHESS_PLAY_070',
-    '
+	'_HE_CHESS_PLAY_070',
+	'
 You may <em>delete</em> the game at any time.
 This will permanently remove the game from the database.
 '
 );
 
 define(
-    '_HE_CHESS_PLAY_075',
-    '
+	'_HE_CHESS_PLAY_075',
+	'
 In a self-play game, you may <em>delete</em> the game at any time.
 This will permanently remove the game from the database.
 '
 );
 
 define(
-    '_HE_CHESS_PLAY_080',
-    '
+	'_HE_CHESS_PLAY_080',
+	'
 You may <em>request arbitration</em> either on your turn or on your opponent\'s turn.
 This will immediately suspend the game so that an arbiter may review it.
 This action may be used for any situation that warrants it, including software bugs.
@@ -207,8 +207,8 @@ 
 );
 
 define(
-    '_HE_CHESS_PLAY_090',
-    '
+	'_HE_CHESS_PLAY_090',
+	'
 If your browser is Javascript-enabled, a dialog will popup to <em>confirm</em> your move or other action.
 If you prefer, you may disable that dialog by unchecking the Confirm checkbox.
 (But remember, just like in "real" chess, once you make a move, you can\'t withdraw it.)
@@ -222,8 +222,8 @@ 
 define('_HE_CHESS_ARBITRATION_TITLE', 'Suspended games and arbitration');
 
 define(
-    '_HE_CHESS_ARBITRATION_010',
-    '
+	'_HE_CHESS_ARBITRATION_010',
+	'
 While a game is suspended, no moves or other player actions can be made.
 An arbiter will review the game and make a decision in compliance with the accepted Laws of Chess.
 If necessary, the arbiter will consult the player(s) and/or seek advice elsewhere prior to making a decision.
@@ -236,8 +236,8 @@ 
 define('_HE_CHESS_ARBITRATION_010_3', 'Delete game');
 
 define(
-    '_HE_CHESS_ARBITRATION_020',
-    '
+	'_HE_CHESS_ARBITRATION_020',
+	'
 When appropriate, an arbiter may also suspend a game that\'s in normal play.
 '
 );
@@ -247,8 +247,8 @@ 
 define('_HE_CHESS_DISPLAY_TITLE', 'Board orientation and refreshing the display');
 
 define(
-    '_HE_CHESS_DISPLAY_010',
-    '
+	'_HE_CHESS_DISPLAY_010',
+	'
 There are three choices for how the chess board is oriented:
 '
 );
@@ -258,24 +258,24 @@ 
 define('_HE_CHESS_DISPLAY_010_3', 'Black at bottom');
 
 define(
-    '_HE_CHESS_DISPLAY_020',
-    '
+	'_HE_CHESS_DISPLAY_020',
+	'
 When you view a game in which you\'re playing, your color is at the bottom by default.
 Otherwise white is at the bottom by default.
 '
 );
 
 define(
-    '_HE_CHESS_DISPLAY_030',
-    '
+	'_HE_CHESS_DISPLAY_030',
+	'
 If you change the orientation setting, click the <em>Refresh</em> button to effect the change.
 The orientation setting will revert to the default if you leave the page and later return to it.
 '
 );
 
 define(
-    '_HE_CHESS_DISPLAY_040',
-    '
+	'_HE_CHESS_DISPLAY_040',
+	'
 Note that if your opponent moves while you\'re viewing the board, your view will not automatically update.
 You would need to use the Refresh button to update the board to see his move.
 '
@@ -286,16 +286,16 @@ 
 define('_HE_CHESS_NOTIFY_TITLE', 'Notifications and comments');
 
 define(
-    '_HE_CHESS_NOTIFY_010',
-    '
+	'_HE_CHESS_NOTIFY_010',
+	'
 You can subscribe to any of several <em>Notification Options</em> related to chess games,
 such as notification when a new move is made and notification when a new challenge game is created.
 '
 );
 
 define(
-    '_HE_CHESS_NOTIFY_020',
-    '
+	'_HE_CHESS_NOTIFY_020',
+	'
 <em>Comments</em> may be posted for each game.
 '
 );
@@ -305,15 +305,15 @@ 
 define('_HE_CHESS_MOUSE_TITLE', 'Moving (using the mouse)');
 
 define(
-    '_HE_CHESS_MOUSE_010',
-    '
+	'_HE_CHESS_MOUSE_010',
+	'
 Moves are made either by using a mouse (recommended), or by typing the notation for the move in the text box.
 '
 );
 
 define(
-    '_HE_CHESS_MOUSE_020',
-    '
+	'_HE_CHESS_MOUSE_020',
+	'
 To use the mouse, click on the piece to move, and then click on the destination tile.
 (To castle, click on the king and its destination tile.)
 Each tile clicked will be highlighted.
@@ -324,8 +324,8 @@ 
 );
 
 define(
-    '_HE_CHESS_MOUSE_030',
-    '
+	'_HE_CHESS_MOUSE_030',
+	'
 Moving with the mouse requires that your browser be Javascript-enabled.
 '
 );
@@ -335,15 +335,15 @@ 
 define('_HE_CHESS_NOTATION_TITLE', 'Moving (using notation)');
 
 define(
-    '_HE_CHESS_NOTATION_010',
-    '
+	'_HE_CHESS_NOTATION_010',
+	'
 As an alternative to using the mouse, you may enter moves by typing the notation in the text box.
 '
 );
 
 define(
-    '_HE_CHESS_NOTATION_020',
-    '
+	'_HE_CHESS_NOTATION_020',
+	'
 The notation for a move is composed of four parts (and an optional fifth part for pawn promotion):
 '
 );
@@ -365,8 +365,8 @@ 
 define('_HE_CHESS_NOTATION_020_5_A', 'Possible values: =Q, =R, =B, =N');
 
 define(
-    '_HE_CHESS_NOTATION_030',
-    '
+	'_HE_CHESS_NOTATION_030',
+	'
 Examples of complete moves:
 '
 );
@@ -379,8 +379,8 @@ 
 define('_HE_CHESS_NOTATION_030_7', 'Pe7-e8=Q  (pawn moves from e7 to e8 and is promoted to queen)');
 
 define(
-    '_HE_CHESS_NOTATION_040',
-    '
+	'_HE_CHESS_NOTATION_040',
+	'
 Instead of the above notation, you may use Standard Algebraic Notation (SAN). Examples:
 '
 );
@@ -393,22 +393,22 @@ 
 define('_HE_CHESS_NOTATION_040_7', 'e8Q  (pawn moves to e8 and is promoted to queen)');
 
 define(
-    '_HE_CHESS_NOTATION_050',
-    '
+	'_HE_CHESS_NOTATION_050',
+	'
 The notation for castling uses the uppercase letter "O" (oh), not the digit "0" (zero).
 '
 );
 
 define(
-    '_HE_CHESS_NOTATION_060',
-    '
+	'_HE_CHESS_NOTATION_060',
+	'
 SAN is described in more detail at the <a href="#fide">FIDE</a> site.
 '
 );
 
 define(
-    '_HE_CHESS_NOTATION_070',
-    '
+	'_HE_CHESS_NOTATION_070',
+	'
 <em>The piece identifier must be upper case, and the file coordinate must be lower case.
 Only valid, non-ambiguous moves will be accepted.</em>
 '
@@ -419,8 +419,8 @@ 
 define('_HE_CHESS_EXPORT_TITLE', 'Exporting games');
 
 define(
-    '_HE_CHESS_EXPORT_010',
-    '
+	'_HE_CHESS_EXPORT_010',
+	'
 The current state of the game is displayed in both Portable Game Notation (PGN) and Forsyth-Edwards Notation (FEN).
 These are widely used notations for representing chess games.
 You may save the PGN or FEN representation of the game for your own records,
@@ -433,8 +433,8 @@ 
 define('_HE_CHESS_MISC_TITLE', 'Miscellaneous');
 
 define(
-    '_HE_CHESS_MISC_010',
-    '
+	'_HE_CHESS_MISC_010',
+	'
 The stalemate test does not recognize whether a piece, other than the king,
 is bound by check (unable to move because it would place the king in check).
 So although the player won\'t be able to make any moves, the situation will not be detected as stalemate.
@@ -446,8 +446,8 @@ 
 define('_HE_CHESS_MISC_010_IMAGE', 'stalemate example');
 
 define(
-    '_HE_CHESS_MISC_020',
-    '
+	'_HE_CHESS_MISC_020',
+	'
 Article 9.6 of the FIDE Laws of Chess states that
 <em>The game is drawn when a position is reached from which a checkmate cannot occur by any possible series of legal moves,
 even with the most unskilled play. This immediately ends the game.</em>
@@ -459,8 +459,8 @@ 
 );
 
 define(
-    '_HE_CHESS_MISC_030',
-    '
+	'_HE_CHESS_MISC_030',
+	'
 There are no time constraints on play.
 None of the FIDE rules concerning clocks, time limits or flags are applicable here.
 '
@@ -471,29 +471,29 @@ 
 define('_HE_CHESS_ADMIN_TITLE', 'Administration and arbitration');
 
 define(
-    '_HE_CHESS_ADMIN_010',
-    '
+	'_HE_CHESS_ADMIN_010',
+	'
 An <em>arbiter</em> is a user with module admin rights for the chess module.
 '
 );
 
 define(
-    '_HE_CHESS_ADMIN_020',
-    '
+	'_HE_CHESS_ADMIN_020',
+	'
 From the Xoops control panel, an arbiter may view or edit the chess module preferences, and may arbitrate games and challenges.
 '
 );
 
 define(
-    '_HE_CHESS_ADMIN_030',
-    '
+	'_HE_CHESS_ADMIN_030',
+	'
 When an arbiter accesses a challenge from the control panel, he has the option of deleting that challenge.
 '
 );
 
 define(
-    '_HE_CHESS_ADMIN_040',
-    '
+	'_HE_CHESS_ADMIN_040',
+	'
 When an arbiter accesses an active or suspended game from the control panel,
 he will see the normal game page, with the addition of <em>arbiter controls</em>.
 If a game is active, the arbiter controls allow the arbiter to suspend the game.
@@ -504,8 +504,8 @@ 
 );
 
 define(
-    '_HE_CHESS_ADMIN_050',
-    '
+	'_HE_CHESS_ADMIN_050',
+	'
 An arbiter may subscribe to be notified when a player requests arbitration in a chess game.
 '
 );

language/english/filechecker.php

Spacing +10 added lines, -10 removed lines

@@ -14,15 +14,15 @@
  * @ignore
  */
 
-define('CO_' . $moduleDirNameUpper . '_' . 'FC_AVAILABLE', "<span style='color: #008000;'>Available</span>");
-define('CO_' . $moduleDirNameUpper . '_' . 'FC_NOTAVAILABLE', "<span style='color: #ff0000;'>Not available</span>");
-define('CO_' . $moduleDirNameUpper . '_' . 'FC_NOTWRITABLE', "<span style='color: #ff0000;'>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: #008000;'>Available</span>");
+define('CO_'.$moduleDirNameUpper.'_'.'FC_NOTAVAILABLE', "<span style='color: #ff0000;'>Not available</span>");
+define('CO_'.$moduleDirNameUpper.'_'.'FC_NOTWRITABLE', "<span style='color: #ff0000;'>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');
 
 /**#@-*/

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.

Spacing +7 added lines, -7 removed lines

@@ -73,25 +73,25 @@ 
     '
     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>
+    &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>
     <br><br>
-    &nbsp;&nbsp;' . _MI_CHESS_RATING_SYSTEM_LINEAR . ' - A very simple system that adds (subtracts) a fixed number of points for a win (loss).
+    &nbsp;&nbsp;' . _MI_CHESS_RATING_SYSTEM_LINEAR.' - A very simple system that adds (subtracts) a fixed number of points for a win (loss).
     <br><br>
-    Select "' . _MI_CHESS_RATING_SYSTEM_NONE . '" to disable the rating feature.
+    Select "' . _MI_CHESS_RATING_SYSTEM_NONE.'" to disable the rating feature.
     <br><br>
     After changing this setting, you should rebuild the player ratings data from Main Menu >> Chess >>
-    ' . _MI_CHESS_SMNAME4 . '.
+    ' . _MI_CHESS_SMNAME4.'.
 '
 );
 define('_MI_CHESS_INITIAL_RATING', 'Initial player rating');
 define(
     '_MI_CHESS_INITIAL_RATING_DES',
     '
-    If the "' . _MI_CHESS_RATING_SYSTEM_CXR . '" rating system is selected, this value should be between 800 and 2000.
+    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.
     <br><br>
-    If you change this value, you should rebuild the player ratings data from Main Menu >> Chess >> ' . _MI_CHESS_SMNAME4 . '.
+    If you change this value, you should rebuild the player ratings data from Main Menu >> Chess >> ' . _MI_CHESS_SMNAME4.'.
 '
 );
 define('_MI_CHESS_ALLOW_UNRATED', 'Allow unrated games?');
@@ -170,7 +170,7 @@ 
 
 //Help
 define('_MI_CHESS_DIRNAME', basename(dirname(__DIR__, 2)));
-define('_MI_CHESS_HELP_HEADER', __DIR__ . '/help/helpheader.tpl');
+define('_MI_CHESS_HELP_HEADER', __DIR__.'/help/helpheader.tpl');
 define('_MI_CHESS_BACK_2_ADMIN', 'Back to Administration of ');
 define('_MI_CHESS_OVERVIEW', 'Overview');
 

language/english/common.php

Spacing +124 added lines, -124 removed lines

@@ -25,132 +25,132 @@ 
 $moduleDirName      = basename(dirname(__DIR__, 2));
 $moduleDirNameUpper = mb_strtoupper($moduleDirName);
 
-define('CO_' . $moduleDirNameUpper . '_GDLIBSTATUS', 'GD library support: ');
-define('CO_' . $moduleDirNameUpper . '_GDLIBVERSION', 'GD Library version: ');
-define('CO_' . $moduleDirNameUpper . '_GDOFF', "<span style='font-weight: bold;'>Disabled</span> (No thumbnails available)");
-define('CO_' . $moduleDirNameUpper . '_GDON', "<span style='font-weight: bold;'>Enabled</span> (Thumbsnails available)");
-define('CO_' . $moduleDirNameUpper . '_IMAGEINFO', 'Server status');
-define('CO_' . $moduleDirNameUpper . '_MAXPOSTSIZE', 'Max post size permitted (post_max_size directive in php.ini): ');
-define('CO_' . $moduleDirNameUpper . '_MAXUPLOADSIZE', 'Max upload size permitted (upload_max_filesize directive in php.ini): ');
-define('CO_' . $moduleDirNameUpper . '_MEMORYLIMIT', 'Memory limit (memory_limit directive in php.ini): ');
-define('CO_' . $moduleDirNameUpper . '_METAVERSION', "<span style='font-weight: bold;'>Downloads meta version:</span> ");
-define('CO_' . $moduleDirNameUpper . '_OFF', "<span style='font-weight: bold;'>OFF</span>");
-define('CO_' . $moduleDirNameUpper . '_ON', "<span style='font-weight: bold;'>ON</span>");
-define('CO_' . $moduleDirNameUpper . '_SERVERPATH', 'Server path to XOOPS root: ');
-define('CO_' . $moduleDirNameUpper . '_SERVERUPLOADSTATUS', 'Server uploads status: ');
-define('CO_' . $moduleDirNameUpper . '_SPHPINI', "<span style='font-weight: bold;'>Information taken from PHP ini file:</span>");
-define('CO_' . $moduleDirNameUpper . '_UPLOADPATHDSC', 'Note. Upload path *MUST* contain the full server path of your upload folder.');
-
-define('CO_' . $moduleDirNameUpper . '_PRINT', "<span style='font-weight: bold;'>Print</span>");
-define('CO_' . $moduleDirNameUpper . '_PDF', "<span style='font-weight: bold;'>Create PDF</span>");
-
-define('CO_' . $moduleDirNameUpper . '_UPGRADEFAILED0', "Update failed - couldn't rename field '%s'");
-define('CO_' . $moduleDirNameUpper . '_UPGRADEFAILED1', "Update failed - couldn't add new fields");
-define('CO_' . $moduleDirNameUpper . '_UPGRADEFAILED2', "Update failed - couldn't rename table '%s'");
-define('CO_' . $moduleDirNameUpper . '_ERROR_COLUMN', 'Could not create column in database : %s');
-define('CO_' . $moduleDirNameUpper . '_ERROR_BAD_XOOPS', 'This module requires XOOPS %s+ (%s installed)');
-define('CO_' . $moduleDirNameUpper . '_ERROR_BAD_PHP', 'This module requires PHP version %s+ (%s installed)');
-define('CO_' . $moduleDirNameUpper . '_ERROR_TAG_REMOVAL', 'Could not remove tags from Tag Module');
-
-define('CO_' . $moduleDirNameUpper . '_FOLDERS_DELETED_OK', 'Upload Folders have been deleted');
+define('CO_'.$moduleDirNameUpper.'_GDLIBSTATUS', 'GD library support: ');
+define('CO_'.$moduleDirNameUpper.'_GDLIBVERSION', 'GD Library version: ');
+define('CO_'.$moduleDirNameUpper.'_GDOFF', "<span style='font-weight: bold;'>Disabled</span> (No thumbnails available)");
+define('CO_'.$moduleDirNameUpper.'_GDON', "<span style='font-weight: bold;'>Enabled</span> (Thumbsnails available)");
+define('CO_'.$moduleDirNameUpper.'_IMAGEINFO', 'Server status');
+define('CO_'.$moduleDirNameUpper.'_MAXPOSTSIZE', 'Max post size permitted (post_max_size directive in php.ini): ');
+define('CO_'.$moduleDirNameUpper.'_MAXUPLOADSIZE', 'Max upload size permitted (upload_max_filesize directive in php.ini): ');
+define('CO_'.$moduleDirNameUpper.'_MEMORYLIMIT', 'Memory limit (memory_limit directive in php.ini): ');
+define('CO_'.$moduleDirNameUpper.'_METAVERSION', "<span style='font-weight: bold;'>Downloads meta version:</span> ");
+define('CO_'.$moduleDirNameUpper.'_OFF', "<span style='font-weight: bold;'>OFF</span>");
+define('CO_'.$moduleDirNameUpper.'_ON', "<span style='font-weight: bold;'>ON</span>");
+define('CO_'.$moduleDirNameUpper.'_SERVERPATH', 'Server path to XOOPS root: ');
+define('CO_'.$moduleDirNameUpper.'_SERVERUPLOADSTATUS', 'Server uploads status: ');
+define('CO_'.$moduleDirNameUpper.'_SPHPINI', "<span style='font-weight: bold;'>Information taken from PHP ini file:</span>");
+define('CO_'.$moduleDirNameUpper.'_UPLOADPATHDSC', 'Note. Upload path *MUST* contain the full server path of your upload folder.');
+
+define('CO_'.$moduleDirNameUpper.'_PRINT', "<span style='font-weight: bold;'>Print</span>");
+define('CO_'.$moduleDirNameUpper.'_PDF', "<span style='font-weight: bold;'>Create PDF</span>");
+
+define('CO_'.$moduleDirNameUpper.'_UPGRADEFAILED0', "Update failed - couldn't rename field '%s'");
+define('CO_'.$moduleDirNameUpper.'_UPGRADEFAILED1', "Update failed - couldn't add new fields");
+define('CO_'.$moduleDirNameUpper.'_UPGRADEFAILED2', "Update failed - couldn't rename table '%s'");
+define('CO_'.$moduleDirNameUpper.'_ERROR_COLUMN', 'Could not create column in database : %s');
+define('CO_'.$moduleDirNameUpper.'_ERROR_BAD_XOOPS', 'This module requires XOOPS %s+ (%s installed)');
+define('CO_'.$moduleDirNameUpper.'_ERROR_BAD_PHP', 'This module requires PHP version %s+ (%s installed)');
+define('CO_'.$moduleDirNameUpper.'_ERROR_TAG_REMOVAL', 'Could not remove tags from Tag Module');
+
+define('CO_'.$moduleDirNameUpper.'_FOLDERS_DELETED_OK', 'Upload Folders have been deleted');
 
 // Error Msgs
-define('CO_' . $moduleDirNameUpper . '_ERROR_BAD_DEL_PATH', 'Could not delete %s directory');
-define('CO_' . $moduleDirNameUpper . '_ERROR_BAD_REMOVE', 'Could not delete %s');
-define('CO_' . $moduleDirNameUpper . '_ERROR_NO_PLUGIN', 'Could not load plugin');
+define('CO_'.$moduleDirNameUpper.'_ERROR_BAD_DEL_PATH', 'Could not delete %s directory');
+define('CO_'.$moduleDirNameUpper.'_ERROR_BAD_REMOVE', 'Could not delete %s');
+define('CO_'.$moduleDirNameUpper.'_ERROR_NO_PLUGIN', 'Could not load plugin');
 
 //Help
-define('CO_' . $moduleDirNameUpper . '_DIRNAME', basename(dirname(__DIR__, 2)));
-define('CO_' . $moduleDirNameUpper . '_HELP_HEADER', __DIR__ . '/help/helpheader.tpl');
-define('CO_' . $moduleDirNameUpper . '_BACK_2_ADMIN', 'Back to Administration of ');
-define('CO_' . $moduleDirNameUpper . '_OVERVIEW', 'Overview');
+define('CO_'.$moduleDirNameUpper.'_DIRNAME', basename(dirname(__DIR__, 2)));
+define('CO_'.$moduleDirNameUpper.'_HELP_HEADER', __DIR__.'/help/helpheader.tpl');
+define('CO_'.$moduleDirNameUpper.'_BACK_2_ADMIN', 'Back to Administration of ');
+define('CO_'.$moduleDirNameUpper.'_OVERVIEW', 'Overview');
 
 //define('CO_' . $moduleDirNameUpper . '_HELP_DIR', __DIR__);
 
 //help multi-page
-define('CO_' . $moduleDirNameUpper . '_DISCLAIMER', 'Disclaimer');
-define('CO_' . $moduleDirNameUpper . '_LICENSE', 'License');
-define('CO_' . $moduleDirNameUpper . '_SUPPORT', 'Support');
+define('CO_'.$moduleDirNameUpper.'_DISCLAIMER', 'Disclaimer');
+define('CO_'.$moduleDirNameUpper.'_LICENSE', 'License');
+define('CO_'.$moduleDirNameUpper.'_SUPPORT', 'Support');
 
 //Sample Data
-define('CO_' . $moduleDirNameUpper . '_' . 'ADD_SAMPLEDATA', 'Import Sample Data (will delete ALL current data)');
-define('CO_' . $moduleDirNameUpper . '_' . 'SAMPLEDATA_SUCCESS', 'Sample Date uploaded successfully');
-define('CO_' . $moduleDirNameUpper . '_' . 'SAVE_SAMPLEDATA', 'Export Tables to YAML');
-define('CO_' . $moduleDirNameUpper . '_' . 'SHOW_SAMPLE_BUTTON', 'Show Sample Button?');
-define('CO_' . $moduleDirNameUpper . '_' . 'SHOW_SAMPLE_BUTTON_DESC', 'If yes, the "Add Sample Data" button will be visible to the Admin. It is Yes as a default for first installation.');
-define('CO_' . $moduleDirNameUpper . '_' . 'EXPORT_SCHEMA', 'Export DB Schema to YAML');
-define('CO_' . $moduleDirNameUpper . '_' . 'EXPORT_SCHEMA_SUCCESS', 'Export DB Schema to YAML was a success');
-define('CO_' . $moduleDirNameUpper . '_' . 'EXPORT_SCHEMA_ERROR', 'ERROR: Export of DB Schema to YAML failed');
-define('CO_' . $moduleDirNameUpper . '_' . 'ADD_SAMPLEDATA_OK', 'Are you sure to Import Sample Data? (It will delete ALL current data)');
-define('CO_' . $moduleDirNameUpper . '_' . 'HIDE_SAMPLEDATA_BUTTONS', 'Hide the Import buttons)');
-define('CO_' . $moduleDirNameUpper . '_' . 'SHOW_SAMPLEDATA_BUTTONS', 'Show the Import buttons)');
-define('CO_' . $moduleDirNameUpper . '_' . 'CONFIRM', 'Confirm');
+define('CO_'.$moduleDirNameUpper.'_'.'ADD_SAMPLEDATA', 'Import Sample Data (will delete ALL current data)');
+define('CO_'.$moduleDirNameUpper.'_'.'SAMPLEDATA_SUCCESS', 'Sample Date uploaded successfully');
+define('CO_'.$moduleDirNameUpper.'_'.'SAVE_SAMPLEDATA', 'Export Tables to YAML');
+define('CO_'.$moduleDirNameUpper.'_'.'SHOW_SAMPLE_BUTTON', 'Show Sample Button?');
+define('CO_'.$moduleDirNameUpper.'_'.'SHOW_SAMPLE_BUTTON_DESC', 'If yes, the "Add Sample Data" button will be visible to the Admin. It is Yes as a default for first installation.');
+define('CO_'.$moduleDirNameUpper.'_'.'EXPORT_SCHEMA', 'Export DB Schema to YAML');
+define('CO_'.$moduleDirNameUpper.'_'.'EXPORT_SCHEMA_SUCCESS', 'Export DB Schema to YAML was a success');
+define('CO_'.$moduleDirNameUpper.'_'.'EXPORT_SCHEMA_ERROR', 'ERROR: Export of DB Schema to YAML failed');
+define('CO_'.$moduleDirNameUpper.'_'.'ADD_SAMPLEDATA_OK', 'Are you sure to Import Sample Data? (It will delete ALL current data)');
+define('CO_'.$moduleDirNameUpper.'_'.'HIDE_SAMPLEDATA_BUTTONS', 'Hide the Import buttons)');
+define('CO_'.$moduleDirNameUpper.'_'.'SHOW_SAMPLEDATA_BUTTONS', 'Show the Import buttons)');
+define('CO_'.$moduleDirNameUpper.'_'.'CONFIRM', 'Confirm');
 
 //letter choice
-define('CO_' . $moduleDirNameUpper . '_' . 'BROWSETOTOPIC', "<span style='font-weight: bold;'>Browse items alphabetically</span>");
-define('CO_' . $moduleDirNameUpper . '_' . 'OTHER', 'Other');
-define('CO_' . $moduleDirNameUpper . '_' . 'ALL', 'All');
+define('CO_'.$moduleDirNameUpper.'_'.'BROWSETOTOPIC', "<span style='font-weight: bold;'>Browse items alphabetically</span>");
+define('CO_'.$moduleDirNameUpper.'_'.'OTHER', 'Other');
+define('CO_'.$moduleDirNameUpper.'_'.'ALL', 'All');
 
 // block defines
-define('CO_' . $moduleDirNameUpper . '_' . 'ACCESSRIGHTS', 'Access Rights');
-define('CO_' . $moduleDirNameUpper . '_' . 'ACTION', 'Action');
-define('CO_' . $moduleDirNameUpper . '_' . 'ACTIVERIGHTS', 'Active Rights');
-define('CO_' . $moduleDirNameUpper . '_' . 'BADMIN', 'Block Administration');
-define('CO_' . $moduleDirNameUpper . '_' . 'BLKDESC', 'Description');
-define('CO_' . $moduleDirNameUpper . '_' . 'CBCENTER', 'Center Middle');
-define('CO_' . $moduleDirNameUpper . '_' . 'CBLEFT', 'Center Left');
-define('CO_' . $moduleDirNameUpper . '_' . 'CBRIGHT', 'Center Right');
-define('CO_' . $moduleDirNameUpper . '_' . 'SBLEFT', 'Left');
-define('CO_' . $moduleDirNameUpper . '_' . 'SBRIGHT', 'Right');
-define('CO_' . $moduleDirNameUpper . '_' . 'SIDE', 'Alignment');
-define('CO_' . $moduleDirNameUpper . '_' . 'TITLE', 'Title');
-define('CO_' . $moduleDirNameUpper . '_' . 'VISIBLE', 'Visible');
-define('CO_' . $moduleDirNameUpper . '_' . 'VISIBLEIN', 'Visible In');
-define('CO_' . $moduleDirNameUpper . '_' . 'WEIGHT', 'Weight');
-
-define('CO_' . $moduleDirNameUpper . '_' . 'PERMISSIONS', 'Permissions');
-define('CO_' . $moduleDirNameUpper . '_' . 'BLOCKS', 'Blocks Admin');
-define('CO_' . $moduleDirNameUpper . '_' . 'BLOCKS_DESC', 'Blocks/Group Admin');
-
-define('CO_' . $moduleDirNameUpper . '_' . 'BLOCKS_MANAGMENT', 'Manage');
-define('CO_' . $moduleDirNameUpper . '_' . 'BLOCKS_ADDBLOCK', 'Add a new block');
-define('CO_' . $moduleDirNameUpper . '_' . 'BLOCKS_EDITBLOCK', 'Edit a block');
-define('CO_' . $moduleDirNameUpper . '_' . 'BLOCKS_CLONEBLOCK', 'Clone a block');
+define('CO_'.$moduleDirNameUpper.'_'.'ACCESSRIGHTS', 'Access Rights');
+define('CO_'.$moduleDirNameUpper.'_'.'ACTION', 'Action');
+define('CO_'.$moduleDirNameUpper.'_'.'ACTIVERIGHTS', 'Active Rights');
+define('CO_'.$moduleDirNameUpper.'_'.'BADMIN', 'Block Administration');
+define('CO_'.$moduleDirNameUpper.'_'.'BLKDESC', 'Description');
+define('CO_'.$moduleDirNameUpper.'_'.'CBCENTER', 'Center Middle');
+define('CO_'.$moduleDirNameUpper.'_'.'CBLEFT', 'Center Left');
+define('CO_'.$moduleDirNameUpper.'_'.'CBRIGHT', 'Center Right');
+define('CO_'.$moduleDirNameUpper.'_'.'SBLEFT', 'Left');
+define('CO_'.$moduleDirNameUpper.'_'.'SBRIGHT', 'Right');
+define('CO_'.$moduleDirNameUpper.'_'.'SIDE', 'Alignment');
+define('CO_'.$moduleDirNameUpper.'_'.'TITLE', 'Title');
+define('CO_'.$moduleDirNameUpper.'_'.'VISIBLE', 'Visible');
+define('CO_'.$moduleDirNameUpper.'_'.'VISIBLEIN', 'Visible In');
+define('CO_'.$moduleDirNameUpper.'_'.'WEIGHT', 'Weight');
+
+define('CO_'.$moduleDirNameUpper.'_'.'PERMISSIONS', 'Permissions');
+define('CO_'.$moduleDirNameUpper.'_'.'BLOCKS', 'Blocks Admin');
+define('CO_'.$moduleDirNameUpper.'_'.'BLOCKS_DESC', 'Blocks/Group Admin');
+
+define('CO_'.$moduleDirNameUpper.'_'.'BLOCKS_MANAGMENT', 'Manage');
+define('CO_'.$moduleDirNameUpper.'_'.'BLOCKS_ADDBLOCK', 'Add a new block');
+define('CO_'.$moduleDirNameUpper.'_'.'BLOCKS_EDITBLOCK', 'Edit a block');
+define('CO_'.$moduleDirNameUpper.'_'.'BLOCKS_CLONEBLOCK', 'Clone a block');
 
 //myblocksadmin
-define('CO_' . $moduleDirNameUpper . '_' . 'AGDS', 'Admin Groups');
-define('CO_' . $moduleDirNameUpper . '_' . 'BCACHETIME', 'Cache Time');
-define('CO_' . $moduleDirNameUpper . '_' . 'BLOCKS_ADMIN', 'Blocks Admin');
+define('CO_'.$moduleDirNameUpper.'_'.'AGDS', 'Admin Groups');
+define('CO_'.$moduleDirNameUpper.'_'.'BCACHETIME', 'Cache Time');
+define('CO_'.$moduleDirNameUpper.'_'.'BLOCKS_ADMIN', 'Blocks Admin');
 
 //Template Admin
-define('CO_' . $moduleDirNameUpper . '_' . 'TPLSETS', 'Template Management');
-define('CO_' . $moduleDirNameUpper . '_' . 'GENERATE', 'Generate');
-define('CO_' . $moduleDirNameUpper . '_' . 'FILENAME', 'File Name');
+define('CO_'.$moduleDirNameUpper.'_'.'TPLSETS', 'Template Management');
+define('CO_'.$moduleDirNameUpper.'_'.'GENERATE', 'Generate');
+define('CO_'.$moduleDirNameUpper.'_'.'FILENAME', 'File Name');
 
 //Menu
-define('CO_' . $moduleDirNameUpper . '_' . 'ADMENU_MIGRATE', 'Migrate');
-define('CO_' . $moduleDirNameUpper . '_' . 'FOLDER_YES', 'Folder "%s" exist');
-define('CO_' . $moduleDirNameUpper . '_' . 'FOLDER_NO', 'Folder "%s" does not exist. Create the specified folder with CHMOD 777.');
-define('CO_' . $moduleDirNameUpper . '_' . 'SHOW_DEV_TOOLS', 'Show Development Tools Button?');
-define('CO_' . $moduleDirNameUpper . '_' . 'SHOW_DEV_TOOLS_DESC', 'If yes, the "Migrate" Tab and other Development tools will be visible to the Admin.');
-define('CO_' . $moduleDirNameUpper . '_' . 'ADMENU_FEEDBACK', 'Feedback');
-define('CO_' . $moduleDirNameUpper . '_' . 'MIGRATE_OK', 'Database migrated to current schema.');
-define('CO_' . $moduleDirNameUpper . '_' . 'MIGRATE_WARNING', 'Warning! This is intended for developers only. Confirm write schema file from current database.');
-define('CO_' . $moduleDirNameUpper . '_' . 'MIGRATE_SCHEMA_OK', 'Current schema file written');
+define('CO_'.$moduleDirNameUpper.'_'.'ADMENU_MIGRATE', 'Migrate');
+define('CO_'.$moduleDirNameUpper.'_'.'FOLDER_YES', 'Folder "%s" exist');
+define('CO_'.$moduleDirNameUpper.'_'.'FOLDER_NO', 'Folder "%s" does not exist. Create the specified folder with CHMOD 777.');
+define('CO_'.$moduleDirNameUpper.'_'.'SHOW_DEV_TOOLS', 'Show Development Tools Button?');
+define('CO_'.$moduleDirNameUpper.'_'.'SHOW_DEV_TOOLS_DESC', 'If yes, the "Migrate" Tab and other Development tools will be visible to the Admin.');
+define('CO_'.$moduleDirNameUpper.'_'.'ADMENU_FEEDBACK', 'Feedback');
+define('CO_'.$moduleDirNameUpper.'_'.'MIGRATE_OK', 'Database migrated to current schema.');
+define('CO_'.$moduleDirNameUpper.'_'.'MIGRATE_WARNING', 'Warning! This is intended for developers only. Confirm write schema file from current database.');
+define('CO_'.$moduleDirNameUpper.'_'.'MIGRATE_SCHEMA_OK', 'Current schema file written');
 
 //Latest Version Check
-define('CO_' . $moduleDirNameUpper . '_' . 'NEW_VERSION', 'New Version: ');
+define('CO_'.$moduleDirNameUpper.'_'.'NEW_VERSION', 'New Version: ');
 
 //DirectoryChecker
-define('CO_' . $moduleDirNameUpper . '_' . 'AVAILABLE', "<span style='color: #008000;'>Available</span>");
-define('CO_' . $moduleDirNameUpper . '_' . 'NOTAVAILABLE', "<span style='color: #ff0000;'>Not available</span>");
-define('CO_' . $moduleDirNameUpper . '_' . 'NOTWRITABLE', "<span style='color: #ff0000;'>Should have permission ( %d ), but it has ( %d )</span>");
-define('CO_' . $moduleDirNameUpper . '_' . 'CREATETHEDIR', 'Create it');
-define('CO_' . $moduleDirNameUpper . '_' . 'SETMPERM', 'Set the permission');
-define('CO_' . $moduleDirNameUpper . '_' . 'DIRCREATED', 'The directory has been created');
-define('CO_' . $moduleDirNameUpper . '_' . 'DIRNOTCREATED', 'The directory cannot be created');
-define('CO_' . $moduleDirNameUpper . '_' . 'PERMSET', 'The permission has been set');
-define('CO_' . $moduleDirNameUpper . '_' . 'PERMNOTSET', 'The permission cannot be set');
+define('CO_'.$moduleDirNameUpper.'_'.'AVAILABLE', "<span style='color: #008000;'>Available</span>");
+define('CO_'.$moduleDirNameUpper.'_'.'NOTAVAILABLE', "<span style='color: #ff0000;'>Not available</span>");
+define('CO_'.$moduleDirNameUpper.'_'.'NOTWRITABLE', "<span style='color: #ff0000;'>Should have permission ( %d ), but it has ( %d )</span>");
+define('CO_'.$moduleDirNameUpper.'_'.'CREATETHEDIR', 'Create it');
+define('CO_'.$moduleDirNameUpper.'_'.'SETMPERM', 'Set the permission');
+define('CO_'.$moduleDirNameUpper.'_'.'DIRCREATED', 'The directory has been created');
+define('CO_'.$moduleDirNameUpper.'_'.'DIRNOTCREATED', 'The directory cannot be created');
+define('CO_'.$moduleDirNameUpper.'_'.'PERMSET', 'The permission has been set');
+define('CO_'.$moduleDirNameUpper.'_'.'PERMNOTSET', 'The permission cannot be set');
 
 //FileChecker
 //define('CO_' . $moduleDirNameUpper . '_' . 'AVAILABLE', "<span style='color: green;'>Available</span>");
@@ -160,36 +160,36 @@ 
 //define('CO_' . $moduleDirNameUpper . '_' . 'CREATETHEFILE', 'Create it');
 //define('CO_' . $moduleDirNameUpper . '_' . 'SETMPERM', 'Set the permission');
 
-define('CO_' . $moduleDirNameUpper . '_' . 'FILECOPIED', 'The file has been copied');
-define('CO_' . $moduleDirNameUpper . '_' . 'FILENOTCOPIED', 'The file cannot be copied');
+define('CO_'.$moduleDirNameUpper.'_'.'FILECOPIED', 'The file has been copied');
+define('CO_'.$moduleDirNameUpper.'_'.'FILENOTCOPIED', 'The file cannot be copied');
 
 //define('CO_' . $moduleDirNameUpper . '_' . 'PERMSET', 'The permission has been set');
 //define('CO_' . $moduleDirNameUpper . '_' . 'PERMNOTSET', 'The permission cannot be set');
 
 //image config
-define('CO_' . $moduleDirNameUpper . '_' . 'IMAGE_WIDTH', 'Image Display Width');
-define('CO_' . $moduleDirNameUpper . '_' . 'IMAGE_WIDTH_DSC', 'Display width for image');
-define('CO_' . $moduleDirNameUpper . '_' . 'IMAGE_HEIGHT', 'Image Display Height');
-define('CO_' . $moduleDirNameUpper . '_' . 'IMAGE_HEIGHT_DSC', 'Display height for image');
-define('CO_' . $moduleDirNameUpper . '_' . 'IMAGE_CONFIG', '<span style="color: #FF0000; font-size: Small;  font-weight: bold;">--- EXTERNAL Image configuration ---</span> ');
-define('CO_' . $moduleDirNameUpper . '_' . 'IMAGE_CONFIG_DSC', '');
-define('CO_' . $moduleDirNameUpper . '_' . 'IMAGE_UPLOAD_PATH', 'Image Upload path');
-define('CO_' . $moduleDirNameUpper . '_' . 'IMAGE_UPLOAD_PATH_DSC', 'Path for uploading images');
-
-define('CO_' . $moduleDirNameUpper . '_' . 'IMAGE_FILE_SIZE', 'Image File Size (in Bytes)');
-define('CO_' . $moduleDirNameUpper . '_' . 'IMAGE_FILE_SIZE_DSC', 'The maximum file size of the image file (in Bytes)');
+define('CO_'.$moduleDirNameUpper.'_'.'IMAGE_WIDTH', 'Image Display Width');
+define('CO_'.$moduleDirNameUpper.'_'.'IMAGE_WIDTH_DSC', 'Display width for image');
+define('CO_'.$moduleDirNameUpper.'_'.'IMAGE_HEIGHT', 'Image Display Height');
+define('CO_'.$moduleDirNameUpper.'_'.'IMAGE_HEIGHT_DSC', 'Display height for image');
+define('CO_'.$moduleDirNameUpper.'_'.'IMAGE_CONFIG', '<span style="color: #FF0000; font-size: Small;  font-weight: bold;">--- EXTERNAL Image configuration ---</span> ');
+define('CO_'.$moduleDirNameUpper.'_'.'IMAGE_CONFIG_DSC', '');
+define('CO_'.$moduleDirNameUpper.'_'.'IMAGE_UPLOAD_PATH', 'Image Upload path');
+define('CO_'.$moduleDirNameUpper.'_'.'IMAGE_UPLOAD_PATH_DSC', 'Path for uploading images');
+
+define('CO_'.$moduleDirNameUpper.'_'.'IMAGE_FILE_SIZE', 'Image File Size (in Bytes)');
+define('CO_'.$moduleDirNameUpper.'_'.'IMAGE_FILE_SIZE_DSC', 'The maximum file size of the image file (in Bytes)');
 
 //Preferences
-define('CO_' . $moduleDirNameUpper . '_' . 'TRUNCATE_LENGTH', 'Number of Characters to truncate to the long text field');
-define('CO_' . $moduleDirNameUpper . '_' . 'TRUNCATE_LENGTH_DESC', 'Set the maximum number of characters to truncate the long text fields');
+define('CO_'.$moduleDirNameUpper.'_'.'TRUNCATE_LENGTH', 'Number of Characters to truncate to the long text field');
+define('CO_'.$moduleDirNameUpper.'_'.'TRUNCATE_LENGTH_DESC', 'Set the maximum number of characters to truncate the long text fields');
 
 //Module Stats
-define('CO_' . $moduleDirNameUpper . '_' . 'STATS_SUMMARY', 'Module Statistics');
-define('CO_' . $moduleDirNameUpper . '_' . 'TOTAL_CATEGORIES', 'Categories:');
-define('CO_' . $moduleDirNameUpper . '_' . 'TOTAL_ITEMS', 'Items');
-define('CO_' . $moduleDirNameUpper . '_' . 'TOTAL_OFFLINE', 'Offline');
-define('CO_' . $moduleDirNameUpper . '_' . 'TOTAL_PUBLISHED', 'Published');
-define('CO_' . $moduleDirNameUpper . '_' . 'TOTAL_REJECTED', 'Rejected');
-define('CO_' . $moduleDirNameUpper . '_' . 'TOTAL_SUBMITTED', 'Submitted');
+define('CO_'.$moduleDirNameUpper.'_'.'STATS_SUMMARY', 'Module Statistics');
+define('CO_'.$moduleDirNameUpper.'_'.'TOTAL_CATEGORIES', 'Categories:');
+define('CO_'.$moduleDirNameUpper.'_'.'TOTAL_ITEMS', 'Items');
+define('CO_'.$moduleDirNameUpper.'_'.'TOTAL_OFFLINE', 'Offline');
+define('CO_'.$moduleDirNameUpper.'_'.'TOTAL_PUBLISHED', 'Published');
+define('CO_'.$moduleDirNameUpper.'_'.'TOTAL_REJECTED', 'Rejected');
+define('CO_'.$moduleDirNameUpper.'_'.'TOTAL_SUBMITTED', 'Submitted');
 
 /**#@-*/