mambax7 / chess

player_stats.php

Spacing +19 added lines, -19 removed lines

@@ -35,15 +35,15 @@ 
 /**#@+
  */
 require_once '../../mainfile.php';
-require_once XOOPS_ROOT_PATH . '/class/xoopsformloader.php';
-require_once XOOPS_ROOT_PATH . '/class/pagenav.php';
-require_once XOOPS_ROOT_PATH . '/modules/chess/include/constants.inc.php';
-require_once XOOPS_ROOT_PATH . '/modules/chess/include/functions.inc.php';
-require_once XOOPS_ROOT_PATH . '/modules/chess/include/ratings.inc.php';
+require_once XOOPS_ROOT_PATH.'/class/xoopsformloader.php';
+require_once XOOPS_ROOT_PATH.'/class/pagenav.php';
+require_once XOOPS_ROOT_PATH.'/modules/chess/include/constants.inc.php';
+require_once XOOPS_ROOT_PATH.'/modules/chess/include/functions.inc.php';
+require_once XOOPS_ROOT_PATH.'/modules/chess/include/ratings.inc.php';
 
 $xoopsOption['template_main'] = 'chess_player_stats.html';
 $xoopsConfig['module_cache'][$xoopsModule->getVar('mid')] = 0; // disable caching
-require_once XOOPS_ROOT_PATH . '/header.php';
+require_once XOOPS_ROOT_PATH.'/header.php';
 
 // user input
 $player_uid   = intval(isset($_POST['player_uid']) ? $_POST['player_uid'] : @$_GET['player_uid']);
@@ -60,20 +60,20 @@ 
 
 // Otherwise, if player user ID provided, map it to a username.
 } elseif ($player_uid != 0) {
-	$member_handler =& xoops_gethandler('member');
-	$player_user    =& $member_handler->getUser($player_uid);
-	$player_uname   =  is_object($player_user) ? $player_user->getVar('uname') : '';
+	$member_handler = & xoops_gethandler('member');
+	$player_user    = & $member_handler->getUser($player_uid);
+	$player_uname   = is_object($player_user) ? $player_user->getVar('uname') : '';
 }
 
 // Check that both user ID and username are now defined.
 if ($player_uid == 0 or empty($player_uname)) {
-	redirect_header(XOOPS_URL . '/modules/chess/index.php', _CHESS_REDIRECT_DELAY_FAILURE, _MD_CHESS_PLAYER_NOT_FOUND);
+	redirect_header(XOOPS_URL.'/modules/chess/index.php', _CHESS_REDIRECT_DELAY_FAILURE, _MD_CHESS_PLAYER_NOT_FOUND);
 }
 
 // Display stats
 chess_player_stats($player_uid, $player_uname, $show_option, $cstart, $gstart);
 
-require_once XOOPS_ROOT_PATH . '/footer.php';
+require_once XOOPS_ROOT_PATH.'/footer.php';
 /**#@-*/
 
 /**
@@ -246,9 +246,9 @@ 
 // ---------
 
 	// get mapping of user IDs to usernames
-	$member_handler =& xoops_gethandler('member');
-	$criteria       =  new Criteria('uid', '(' . implode(',', array_keys($userids)) . ')', 'IN');
-	$usernames      =  $member_handler->getUserList($criteria);
+	$member_handler = & xoops_gethandler('member');
+	$criteria       = new Criteria('uid', '('.implode(',', array_keys($userids)).')', 'IN');
+	$usernames      = $member_handler->getUserList($criteria);
 
 	// add usernames to $games
 	foreach ($games as $k => $game) {
@@ -300,12 +300,12 @@ 
 	$player['games']      = $games;
 	$player['challenges'] = $challenges;
 
-	$xoopsTpl->assign('chess_player',               $player);
-	$xoopsTpl->assign('chess_rating_system',        chess_moduleConfig('rating_system'));
-	$xoopsTpl->assign('chess_provisional_games',    $num_provisional_games);
+	$xoopsTpl->assign('chess_player', $player);
+	$xoopsTpl->assign('chess_rating_system', chess_moduleConfig('rating_system'));
+	$xoopsTpl->assign('chess_provisional_games', $num_provisional_games);
 	$xoopsTpl->assign('chess_show_option_urlparam', $show_option_urlparam);
-	$xoopsTpl->assign('chess_games_pagenav',        $games_pagenav->renderNav());
-	$xoopsTpl->assign('chess_challenges_pagenav',   $challenges_pagenav->renderNav());
+	$xoopsTpl->assign('chess_games_pagenav', $games_pagenav->renderNav());
+	$xoopsTpl->assign('chess_challenges_pagenav', $challenges_pagenav->renderNav());
 }
 
 ?>

admin/index.php

Spacing +47 added lines, -47 removed lines

@@ -34,9 +34,9 @@ 
 /**#@+
  */
 require_once 'admin_header.php';
-require_once XOOPS_ROOT_PATH . '/class/xoopsformloader.php';
-require_once XOOPS_ROOT_PATH . '/class/pagenav.php';
-require_once XOOPS_ROOT_PATH . '/modules/chess/include/functions.inc.php';
+require_once XOOPS_ROOT_PATH.'/class/xoopsformloader.php';
+require_once XOOPS_ROOT_PATH.'/class/pagenav.php';
+require_once XOOPS_ROOT_PATH.'/modules/chess/include/functions.inc.php';
 
 // user input
 $op    = chess_sanitize(@$_GET['op']);
@@ -77,20 +77,20 @@ 
 	<h4> "._AM_CHESS_CONF." </h4>
 	<table width='100%' border='0' cellspacing='1' class='outer'>
 	<tr>
-		<td><a href='" .XOOPS_URL. "/modules/" .$xoopsModule->getVar('dirname'). "/admin/index.php?op=suspended_games'>" ._AM_CHESS_SUSPENDED_GAMES. "</a>
-		<td>" ._AM_CHESS_SUSPENDED_GAMES_DES. "</td>
+		<td><a href='" .XOOPS_URL."/modules/".$xoopsModule->getVar('dirname')."/admin/index.php?op=suspended_games'>"._AM_CHESS_SUSPENDED_GAMES."</a>
+		<td>" ._AM_CHESS_SUSPENDED_GAMES_DES."</td>
 	</tr>
 	<tr>
-		<td><a href='" .XOOPS_URL. "/modules/" .$xoopsModule->getVar('dirname'). "/admin/index.php?op=active_games'>" ._AM_CHESS_ACTIVE_GAMES. "</a>
-		<td>" ._AM_CHESS_ACTIVE_GAMES_DES. "</td>
+		<td><a href='" .XOOPS_URL."/modules/".$xoopsModule->getVar('dirname')."/admin/index.php?op=active_games'>"._AM_CHESS_ACTIVE_GAMES."</a>
+		<td>" ._AM_CHESS_ACTIVE_GAMES_DES."</td>
 	</tr>
 	<tr>
-		<td><a href='" .XOOPS_URL. "/modules/" .$xoopsModule->getVar('dirname'). "/admin/index.php?op=challenges'>" ._AM_CHESS_CHALLENGES. "</a>
-		<td>" ._AM_CHESS_CHALLENGES_DES. "</td>
+		<td><a href='" .XOOPS_URL."/modules/".$xoopsModule->getVar('dirname')."/admin/index.php?op=challenges'>"._AM_CHESS_CHALLENGES."</a>
+		<td>" ._AM_CHESS_CHALLENGES_DES."</td>
 	</tr>
 	<tr>
-		<td><a href='" .XOOPS_URL. "/modules/system/admin.php?fct=preferences&amp;op=showmod&amp;mod=" .$xoopsModule->getVar('mid'). "'>" ._AM_CHESS_PREFS. "</a>
-		<td>" ._AM_CHESS_PREFS_DESC. "</td>
+		<td><a href='" .XOOPS_URL."/modules/system/admin.php?fct=preferences&amp;op=showmod&amp;mod=".$xoopsModule->getVar('mid')."'>"._AM_CHESS_PREFS."</a>
+		<td>" ._AM_CHESS_PREFS_DESC."</td>
 	</tr>
 	</table>
 ";
@@ -103,7 +103,7 @@ 
 {
 	global $start, $max_items_to_display, $op, $xoopsDB, $xoopsModule;
 
-	$member_handler =& xoops_gethandler('member');
+	$member_handler = & xoops_gethandler('member');
 
 	// 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.
@@ -129,20 +129,20 @@ 
 
 	if ($xoopsDB->getRowsNum($result) > 0) {
 
-		echo '<h3>' ._AM_CHESS_SUSPENDED_GAMES. "</h3>\n";
+		echo '<h3>'._AM_CHESS_SUSPENDED_GAMES."</h3>\n";
 
 	 	while ($row = $xoopsDB->fetchArray($result)) {
 
-			$user_white     =& $member_handler->getUser($row['white_uid']);
-			$username_white =  is_object($user_white) ? $user_white->getVar('uname') : '(open)';
+			$user_white     = & $member_handler->getUser($row['white_uid']);
+			$username_white = is_object($user_white) ? $user_white->getVar('uname') : '(open)';
 
-			$user_black     =& $member_handler->getUser($row['black_uid']);
-			$username_black =  is_object($user_black) ? $user_black->getVar('uname') : '(open)';
+			$user_black     = & $member_handler->getUser($row['black_uid']);
+			$username_black = is_object($user_black) ? $user_black->getVar('uname') : '(open)';
 
 			$date = $row['start_date'] ? date('Y.m.d', $row['start_date']) : 'not yet started';
 
-			$title_text = _AM_CHESS_GAME. " #{$row['game_id']}&nbsp;&nbsp;&nbsp;$username_white " ._AM_CHESS_VS. " $username_black&nbsp;&nbsp;&nbsp;($date)";
-			$form = new XoopsThemeForm($title_text, "game_{$row['game_id']}", XOOPS_URL. '/modules/' .$xoopsModule->getVar('dirname'). "/game.php?game_id={$row['game_id']}", 'post', true);
+			$title_text = _AM_CHESS_GAME." #{$row['game_id']}&nbsp;&nbsp;&nbsp;$username_white "._AM_CHESS_VS." $username_black&nbsp;&nbsp;&nbsp;($date)";
+			$form = new XoopsThemeForm($title_text, "game_{$row['game_id']}", XOOPS_URL.'/modules/'.$xoopsModule->getVar('dirname')."/game.php?game_id={$row['game_id']}", 'post', true);
 
 			list($date, $suspender_uid, $type, $explain) = explode('|', $row['suspended']);
 
@@ -158,13 +158,13 @@ 
 					break;
 			}
 
-			$suspender_user     =& $member_handler->getUser($suspender_uid);
-			$suspender_username =  is_object($suspender_user) ? $suspender_user->getVar('uname') : _AM_CHESS_UNKNOWN_USER;
+			$suspender_user     = & $member_handler->getUser($suspender_uid);
+			$suspender_username = is_object($suspender_user) ? $suspender_user->getVar('uname') : _AM_CHESS_UNKNOWN_USER;
 
-			$form->addElement(new XoopsFormLabel(_AM_CHESS_WHEN_SUSPENDED    . ':', formatTimestamp(strtotime($date))));
-			$form->addElement(new XoopsFormLabel(_AM_CHESS_SUSPENDED_BY      . ':', $suspender_username));
-			$form->addElement(new XoopsFormLabel(_AM_CHESS_SUSPENSION_TYPE   . ':', $type_display));
-			$form->addElement(new XoopsFormLabel(_AM_CHESS_SUSPENSION_REASON . ':', $explain));
+			$form->addElement(new XoopsFormLabel(_AM_CHESS_WHEN_SUSPENDED.':', formatTimestamp(strtotime($date))));
+			$form->addElement(new XoopsFormLabel(_AM_CHESS_SUSPENDED_BY.':', $suspender_username));
+			$form->addElement(new XoopsFormLabel(_AM_CHESS_SUSPENSION_TYPE.':', $type_display));
+			$form->addElement(new XoopsFormLabel(_AM_CHESS_SUSPENSION_REASON.':', $explain));
 
 			$form->addElement(new XoopsFormButton('', 'submit', _AM_CHESS_ARBITRATE_SUBMIT, 'submit'));
 
@@ -174,11 +174,11 @@ 
 		}
 
 		$pagenav = new XoopsPageNav($num_rows, $max_items_to_display, $start, 'start', "op=$op");
-		echo '<div align="center">' . $pagenav->renderNav() . "&nbsp;</div>\n";
+		echo '<div align="center">'.$pagenav->renderNav()."&nbsp;</div>\n";
 
 	} else {
 
-		echo '<h3>' ._AM_CHESS_NO_SUSPENDED_GAMES. "</h3>\n";
+		echo '<h3>'._AM_CHESS_NO_SUSPENDED_GAMES."</h3>\n";
 	}
 
 	$xoopsDB->freeRecordSet($result);
@@ -191,7 +191,7 @@ 
 {
 	global $start, $max_items_to_display, $op, $xoopsDB, $xoopsModule;
 
-	$member_handler =& xoops_gethandler('member');
+	$member_handler = & xoops_gethandler('member');
 
 	// 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.
@@ -213,20 +213,20 @@ 
 
 	if ($xoopsDB->getRowsNum($result) > 0) {
 
-		echo '<h3>' ._AM_CHESS_ACTIVE_GAMES. "</h3>\n";
+		echo '<h3>'._AM_CHESS_ACTIVE_GAMES."</h3>\n";
 
 	 	while ($row = $xoopsDB->fetchArray($result)) {
 
-			$user_white     =& $member_handler->getUser($row['white_uid']);
-			$username_white =  is_object($user_white) ? $user_white->getVar('uname') : '(open)';
+			$user_white     = & $member_handler->getUser($row['white_uid']);
+			$username_white = is_object($user_white) ? $user_white->getVar('uname') : '(open)';
 
-			$user_black     =& $member_handler->getUser($row['black_uid']);
-			$username_black =  is_object($user_black) ? $user_black->getVar('uname') : '(open)';
+			$user_black     = & $member_handler->getUser($row['black_uid']);
+			$username_black = is_object($user_black) ? $user_black->getVar('uname') : '(open)';
 
 			$date = $row['start_date'] ? date('Y.m.d', $row['start_date']) : 'not yet started';
 
-			$title_text = _AM_CHESS_GAME. " #{$row['game_id']}&nbsp;&nbsp;&nbsp;$username_white " ._AM_CHESS_VS. " $username_black&nbsp;&nbsp;&nbsp;($date)";
-			$form = new XoopsThemeForm($title_text, "game_{$row['game_id']}", XOOPS_URL. '/modules/' .$xoopsModule->getVar('dirname'). "/game.php?game_id={$row['game_id']}", 'post', true);
+			$title_text = _AM_CHESS_GAME." #{$row['game_id']}&nbsp;&nbsp;&nbsp;$username_white "._AM_CHESS_VS." $username_black&nbsp;&nbsp;&nbsp;($date)";
+			$form = new XoopsThemeForm($title_text, "game_{$row['game_id']}", XOOPS_URL.'/modules/'.$xoopsModule->getVar('dirname')."/game.php?game_id={$row['game_id']}", 'post', true);
 
 			$form->addElement(new XoopsFormButton('', 'submit', _AM_CHESS_ARBITRATE_SUBMIT, 'submit'));
 
@@ -236,11 +236,11 @@ 
 		}
 
 		$pagenav = new XoopsPageNav($num_rows, $max_items_to_display, $start, 'start', "op=$op");
-		echo '<div align="center">' . $pagenav->renderNav() . "&nbsp;</div>\n";
+		echo '<div align="center">'.$pagenav->renderNav()."&nbsp;</div>\n";
 
 	} else {
 
-		echo '<h3>' ._AM_CHESS_NO_ACTIVE_GAMES. "</h3>\n";
+		echo '<h3>'._AM_CHESS_NO_ACTIVE_GAMES."</h3>\n";
 	}
 
 	$xoopsDB->freeRecordSet($result);
@@ -253,7 +253,7 @@ 
 {
 	global $start, $max_items_to_display, $op, $xoopsDB, $xoopsModule;
 
-	$member_handler =& xoops_gethandler('member');
+	$member_handler = & xoops_gethandler('member');
 
 	// 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.
@@ -274,20 +274,20 @@ 
 
 	if ($xoopsDB->getRowsNum($result) > 0) {
 
-		echo '<h3>' ._AM_CHESS_CHALLENGES. "</h3>\n";
+		echo '<h3>'._AM_CHESS_CHALLENGES."</h3>\n";
 
 	 	while ($row = $xoopsDB->fetchArray($result)) {
 
-			$user_player1     =& $member_handler->getUser($row['player1_uid']);
-			$username_player1 =  is_object($user_player1) ? $user_player1->getVar('uname') : '?';
+			$user_player1     = & $member_handler->getUser($row['player1_uid']);
+			$username_player1 = is_object($user_player1) ? $user_player1->getVar('uname') : '?';
 
-			$user_player2     =& $member_handler->getUser($row['player2_uid']);
-			$username_player2 =  is_object($user_player2) ? $user_player2->getVar('uname') : '(open)';
+			$user_player2     = & $member_handler->getUser($row['player2_uid']);
+			$username_player2 = is_object($user_player2) ? $user_player2->getVar('uname') : '(open)';
 
 			$date = date('Y.m.d', $row['create_date']);
 
-			$title_text = _AM_CHESS_CHALLENGE. " #{$row['challenge_id']}&nbsp;&nbsp;&nbsp;$username_player1 " ._AM_CHESS_CHALLENGED. ": $username_player2&nbsp;&nbsp;&nbsp;(" ._AM_CHESS_CREATED. " $date)";
-			$form = new XoopsThemeForm($title_text, "challenge_{$row['challenge_id']}", XOOPS_URL. '/modules/' .$xoopsModule->getVar('dirname'). "/create.php?challenge_id={$row['challenge_id']}", 'post', true);
+			$title_text = _AM_CHESS_CHALLENGE." #{$row['challenge_id']}&nbsp;&nbsp;&nbsp;$username_player1 "._AM_CHESS_CHALLENGED.": $username_player2&nbsp;&nbsp;&nbsp;("._AM_CHESS_CREATED." $date)";
+			$form = new XoopsThemeForm($title_text, "challenge_{$row['challenge_id']}", XOOPS_URL.'/modules/'.$xoopsModule->getVar('dirname')."/create.php?challenge_id={$row['challenge_id']}", 'post', true);
 
 			$form->addElement(new XoopsFormButton('', 'submit', _AM_CHESS_ARBITRATE_SUBMIT, 'submit'));
 
@@ -297,11 +297,11 @@ 
 		}
 
 		$pagenav = new XoopsPageNav($num_rows, $max_items_to_display, $start, 'start', "op=$op");
-		echo '<div align="center">' . $pagenav->renderNav() . "&nbsp;</div>\n";
+		echo '<div align="center">'.$pagenav->renderNav()."&nbsp;</div>\n";
 
 	} else {
 
-		echo '<h3>' ._AM_CHESS_NO_CHALLENGES. "</h3>\n";
+		echo '<h3>'._AM_CHESS_NO_CHALLENGES."</h3>\n";
 	}
 
 	$xoopsDB->freeRecordSet($result);

game.php

Spacing +76 added lines, -76 removed lines

@@ -39,19 +39,19 @@ 
 /**#@+
  */
 require_once '../../mainfile.php';
-require_once XOOPS_ROOT_PATH . '/class/xoopsformloader.php';
+require_once XOOPS_ROOT_PATH.'/class/xoopsformloader.php';
 
 $xoopsOption['template_main'] = 'chess_game_main.html';
 $xoopsConfig['module_cache'][$xoopsModule->getVar('mid')] = 0; // disable caching
-require_once XOOPS_ROOT_PATH . '/header.php';
-require_once XOOPS_ROOT_PATH . '/modules/chess/class/chessgame.inc.php';
-require_once XOOPS_ROOT_PATH . '/modules/chess/include/constants.inc.php';
-require_once XOOPS_ROOT_PATH . '/modules/chess/include/functions.inc.php';
+require_once XOOPS_ROOT_PATH.'/header.php';
+require_once XOOPS_ROOT_PATH.'/modules/chess/class/chessgame.inc.php';
+require_once XOOPS_ROOT_PATH.'/modules/chess/include/constants.inc.php';
+require_once XOOPS_ROOT_PATH.'/modules/chess/include/functions.inc.php';
 
 chess_game();
 
-require_once XOOPS_ROOT_PATH . '/include/comment_view.php';
-require_once XOOPS_ROOT_PATH . '/footer.php';
+require_once XOOPS_ROOT_PATH.'/include/comment_view.php';
+require_once XOOPS_ROOT_PATH.'/footer.php';
 /**#@-*/
 
 /**
@@ -72,7 +72,7 @@ 
 	$move              = chess_sanitize(trim(@$_POST['chessmove']), 'A-Za-z0-9=-');
 	$movetype          = chess_sanitize(@$_POST['movetype']);
 	$confirm           = isset($_POST['confirm']) ? 1 : 0;
-	$move_explain      = chess_sanitize(trim(@$_POST['move_explain']),    'A-Za-z0-9 .,;:=()[]*?!-');
+	$move_explain      = chess_sanitize(trim(@$_POST['move_explain']), 'A-Za-z0-9 .,;:=()[]*?!-');
 	$arbiter_explain   = chess_sanitize(trim(@$_POST['arbiter_explain']), 'A-Za-z0-9 .,;:=()[]*?!-');
 	$orientation       = chess_sanitize(@$_POST['orientation']);
 	$show_arbiter_ctrl = isset($_POST['show_arbiter_ctrl']);
@@ -81,8 +81,8 @@ 
 
 	// If form-submit, check security token.
 	if (($submit_move or $submit_refresh or $submit_arbitrate or $show_arbiter_ctrl) and is_object($GLOBALS['xoopsSecurity']) and !$GLOBALS['xoopsSecurity']->check()) {
-		redirect_header(XOOPS_URL . '/modules/chess/', _CHESS_REDIRECT_DELAY_FAILURE,
-			_MD_CHESS_TOKEN_ERROR . '<br />' . implode('<br />', $GLOBALS['xoopsSecurity']->getErrors()));
+		redirect_header(XOOPS_URL.'/modules/chess/', _CHESS_REDIRECT_DELAY_FAILURE,
+			_MD_CHESS_TOKEN_ERROR.'<br />'.implode('<br />', $GLOBALS['xoopsSecurity']->getErrors()));
 	}
 
 	// Fetch the game data from the database.
@@ -93,13 +93,13 @@ 
 
 	$gamedata_updated      = false;
 	$move_performed        = false; // status result from move-handler
-	$move_result_text      = '';    // text result from move-handler
-	$draw_claim_error_text = '';    // text describing invalid draw-claim
-	$notify                = '';    // text description of action for notification
+	$move_result_text      = ''; // text result from move-handler
+	$draw_claim_error_text = ''; // text describing invalid draw-claim
+	$notify                = ''; // text description of action for notification
 	$delete_game           = false;
 
 	global $xoopsUser;
-	$uid =  $xoopsUser ? $xoopsUser->getVar('uid') : 0;
+	$uid = $xoopsUser ? $xoopsUser->getVar('uid') : 0;
 
 	$selfplay = $gamedata['white_uid'] == $gamedata['black_uid'];
 
@@ -107,7 +107,7 @@ 
 		if ($uid == $gamedata['white_uid']) {
 			$user_color = $gamedata['fen_active_color']; // current user is either player, so consider him active player
 		} else {
-			$user_color = '';                            // current user is not a player
+			$user_color = ''; // current user is not a player
 		}
 
 	// not self-play
@@ -117,7 +117,7 @@ 
 		} elseif ($uid == $gamedata['black_uid']) {
 			$user_color = 'b'; // current user is black
 		} else {
-			$user_color = '';  // current user is not a player
+			$user_color = ''; // current user is not a player
 		}
 	}
 
@@ -187,7 +187,7 @@ 
 					if ($draw_claim_valid) {
 						$gamedata['offer_draw']   = '';
 						$gamedata['pgn_result']   = '1/2-1/2';
-						$comment                  = '{' . $draw_claim_text . '}';
+						$comment                  = '{'.$draw_claim_text.'}';
 						$gamedata['pgn_movetext'] = str_replace('*', "{$gamedata['pgn_result']} $comment", $gamedata['pgn_movetext']);
 						$gamedata_updated         = true;
 						$notify                   = !empty($move) ? "$user_color_name: $move: $draw_claim_text" : "$user_color_name: $draw_claim_text";
@@ -201,10 +201,10 @@ 
 				if ($user_is_player) {
 					$gamedata['offer_draw']   = '';
 					$gamedata['pgn_result']   = $user_color == 'w' ? '0-1' : '1-0';
-					$comment                  = '{' . $user_color_name . ' ' . _MD_CHESS_RESIGNED . '}';
-					$gamedata['pgn_movetext'] = str_replace('*', "{$gamedata['pgn_result']} $comment" , $gamedata['pgn_movetext']);
+					$comment                  = '{'.$user_color_name.' '._MD_CHESS_RESIGNED.'}';
+					$gamedata['pgn_movetext'] = str_replace('*', "{$gamedata['pgn_result']} $comment", $gamedata['pgn_movetext']);
 					$gamedata_updated         = true;
-					$notify                   = "$user_color_name " . _MD_CHESS_RESIGNED;
+					$notify                   = "$user_color_name "._MD_CHESS_RESIGNED;
 				}
 				break;
 
@@ -212,7 +212,7 @@ 
 				if ($user_is_player and empty($gamedata['offer_draw']) and !$selfplay) {
 					$gamedata['offer_draw'] = $user_color;
 					$gamedata_updated       = true;
-					$notify                 = "$user_color_name " . _MD_CHESS_OFFERED_DRAW;
+					$notify                 = "$user_color_name "._MD_CHESS_OFFERED_DRAW;
 				}
 				break;
 
@@ -220,17 +220,17 @@ 
 				if ($user_is_player and !empty($gamedata['offer_draw']) and $gamedata['offer_draw'] != $user_color and !$selfplay) {
 					$gamedata['offer_draw']   = '';
 					$gamedata['pgn_result']   = '1/2-1/2';
-					$comment                  = '{' . _MD_CHESS_DRAW_BY_AGREEMENT . '}';
-					$gamedata['pgn_movetext'] = str_replace('*', "{$gamedata['pgn_result']} $comment" , $gamedata['pgn_movetext']);
+					$comment                  = '{'._MD_CHESS_DRAW_BY_AGREEMENT.'}';
+					$gamedata['pgn_movetext'] = str_replace('*', "{$gamedata['pgn_result']} $comment", $gamedata['pgn_movetext']);
 					$gamedata_updated         = true;
-					$notify                   = "$user_color_name " . _MD_CHESS_ACCEPTED_DRAW;
+					$notify                   = "$user_color_name "._MD_CHESS_ACCEPTED_DRAW;
 				}
 
 			case _CHESS_MOVETYPE_REJECT_DRAW:
 				if ($user_is_player and !empty($gamedata['offer_draw']) and $gamedata['offer_draw'] != $user_color and !$selfplay) {
 					$gamedata['offer_draw']   = '';
 					$gamedata_updated         = true;
-					$notify                   = "$user_color_name " . _MD_CHESS_REJECTED_DRAW;
+					$notify                   = "$user_color_name "._MD_CHESS_REJECTED_DRAW;
 				}
 				break;
 
@@ -263,7 +263,7 @@ 
 				if ($user_is_player and ($selfplay or chess_can_delete())) {
 
 					$delete_game = true; // must defer actual deletion until after notifications are sent
-					$notify      = eval('return "' ._MD_CHESS_DELETED_GAME. '";'); // eval references $username
+					$notify      = eval('return "'._MD_CHESS_DELETED_GAME.'";'); // eval references $username
 				}
 				break;
 
@@ -276,7 +276,7 @@ 
 
 					$gamedata['suspended'] = implode($sep, array(date('Y-m-d H:i:s'), $uid, _CHESS_MOVETYPE_WANT_ARBITRATION, $move_explain));
 					$gamedata_updated      = true;
-					$notify = "$user_color_name " . _MD_CHESS_RQSTED_ARBT . ' ' . _MD_CHESS_BEEN_SUSPENDED;
+					$notify = "$user_color_name "._MD_CHESS_RQSTED_ARBT.' '._MD_CHESS_BEEN_SUSPENDED;
 				}
 				break;
 		}
@@ -298,15 +298,15 @@ 
 	// If arbiter action was submitted, and user is a valid arbiter, process the action.
 	if ($submit_arbitrate and $is_arbiter) {
 
-		$username =  $xoopsUser ? $xoopsUser->getVar('uname') : '*unknown*';
+		$username = $xoopsUser ? $xoopsUser->getVar('uname') : '*unknown*';
 
-		switch($arbiter_action) {
+		switch ($arbiter_action) {
 
 			case _CHESS_ARBITER_RESUME:
 				if ($gamedata['suspended'] and $gamedata['pgn_result'] == '*') {
 					$gamedata['suspended'] = '';
 					$gamedata_updated      = true;
-					$notify                = eval('return "' ._MD_CHESS_RESUMED_PLAY. '";'); // eval references $username
+					$notify                = eval('return "'._MD_CHESS_RESUMED_PLAY.'";'); // eval references $username
 				}
 				break;
 
@@ -316,18 +316,18 @@ 
 					$gamedata['pgn_result']   = '1/2-1/2';
 					$arbiter_explain          = str_replace('{', '(', $arbiter_explain);
 					$arbiter_explain          = str_replace('}', ')', $arbiter_explain);
-					$comment                  = '{' . sprintf(_MD_CHESS_DRAW_DECLARED, $arbiter_explain) . '}';
-					$gamedata['pgn_movetext'] = str_replace('*', "{$gamedata['pgn_result']} $comment" , $gamedata['pgn_movetext']);
+					$comment                  = '{'.sprintf(_MD_CHESS_DRAW_DECLARED, $arbiter_explain).'}';
+					$gamedata['pgn_movetext'] = str_replace('*', "{$gamedata['pgn_result']} $comment", $gamedata['pgn_movetext']);
 					$gamedata['suspended']    = '';
 					$gamedata_updated         = true;
-					$notify                   = eval('return "' ._MD_CHESS_DECLARED_DRAW. '";'); // eval references $username
+					$notify                   = eval('return "'._MD_CHESS_DECLARED_DRAW.'";'); // eval references $username
 				}
 				break;
 
 			case _CHESS_ARBITER_DELETE:
 				if ($gamedata['suspended']) {
 					$delete_game = true; // must defer actual deletion until after notifications are sent
-					$notify      = eval('return "' ._MD_CHESS_DELETED_GAME. '";'); // eval references $username
+					$notify      = eval('return "'._MD_CHESS_DELETED_GAME.'";'); // eval references $username
 				}
 				break;
 
@@ -340,7 +340,7 @@ 
 
 					$gamedata['suspended'] = implode('|', array(date('Y-m-d H:i:s'), $uid, _CHESS_MOVETYPE_ARBITER_SUSPEND, $arbiter_explain));
 					$gamedata_updated      = true;
-					$notify                = eval('return "' ._MD_CHESS_SUSPENDED_PLAY. '";'); // eval references $username
+					$notify                = eval('return "'._MD_CHESS_SUSPENDED_PLAY.'";'); // eval references $username
 				}
 				break;
 
@@ -362,7 +362,7 @@ 
 	// If a move (or other action) was made, notify any subscribers.
 	if (!empty($notify)) {
 
-		$notification_handler =& xoops_gethandler('notification');
+		$notification_handler = & xoops_gethandler('notification');
 
 		$notification_handler->triggerEvent('game', $game_id, 'notify_game_move', array('CHESS_ACTION' => $notify));
 
@@ -370,7 +370,7 @@ 
 		if ($movetype == _CHESS_MOVETYPE_WANT_ARBITRATION) {
 
 			$event      = 'notify_request_arbitration';
-			$username   =  $xoopsUser ? $xoopsUser->getVar('uname') : '(' ._MD_CHESS_UNKNOWN. ')';
+			$username   = $xoopsUser ? $xoopsUser->getVar('uname') : '('._MD_CHESS_UNKNOWN.')';
 			$extra_tags = array('CHESS_REQUESTOR' => $username, 'CHESS_GAME_ID' => $game_id, 'CHESS_EXPLAIN' => $move_explain);
 			$notification_handler->triggerEvent('global', 0, $event, $extra_tags);
 		}
@@ -416,7 +416,7 @@ 
 	$suspended_q = $myts->addslashes($gamedata['suspended']);
 	$movetext_q  = $myts->addslashes($gamedata['pgn_movetext']);
 
-	$table  = $xoopsDB->prefix('chess_games');
+	$table = $xoopsDB->prefix('chess_games');
 	#echo "updating database table $table<br />\n";#*#DEBUG#
 	$xoopsDB->query(trim("
 		UPDATE $table
@@ -438,11 +438,11 @@ 
 		WHERE  game_id = '$game_id'
 	"));
 	if ($xoopsDB->errno()) {
-		trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
+		trigger_error($xoopsDB->errno().':'.$xoopsDB->error(), E_USER_ERROR);
 	}
 
 	if ($gamedata['pgn_result'] != '*' and $gamedata['is_rated'] == '1') {
-		require_once XOOPS_ROOT_PATH . '/modules/chess/include/ratings.inc.php';
+		require_once XOOPS_ROOT_PATH.'/modules/chess/include/ratings.inc.php';
 		chess_ratings_adj($game_id);
 	}
 }
@@ -462,7 +462,7 @@ 
 	$table = $xoopsDB->prefix('chess_games');
 	$xoopsDB->query("DELETE FROM $table WHERE game_id='$game_id'");
 	if ($xoopsDB->errno()) {
-		trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
+		trigger_error($xoopsDB->errno().':'.$xoopsDB->error(), E_USER_ERROR);
 	}
 }
 
@@ -568,7 +568,7 @@ 
 	#var_dump('gamedata', $gamedata);#*#DEBUG#
 
 	// Define this constant as true to output a log file containing a move analysis.
-	define ('CHESS_LOG_3FOLD', false);
+	define('CHESS_LOG_3FOLD', false);
 	if (CHESS_LOG_3FOLD) {
 		$log = array();
 	}
@@ -605,13 +605,13 @@ 
 	// $repeats is the list of moves for which the board positions are identical.
 	// For example, '6w' would represent the board position immediately before white's sixth move.
 	// The current position is included, since that's the position against which the other positions will be compared.
-	$repeats[] = $gamedata['fen_fullmove_number'] . $gamedata['fen_active_color'];
+	$repeats[] = $gamedata['fen_fullmove_number'].$gamedata['fen_active_color'];
 
 	// Compare initial board position with last board position, unless the move number is the same, meaning that there haven't been any moves.
 	#echo "FEN: '{$tmp_gamedata['fen_piece_placement']} {$tmp_gamedata['fen_active_color']} {$tmp_gamedata['fen_castling_availability']} {$tmp_gamedata['fen_en_passant_target_square']} {$tmp_gamedata['fen_halfmove_clock']} {$tmp_gamedata['fen_fullmove_number']}'<br />\n";#*#DEBUG#
 	$board_state = "{$tmp_gamedata['fen_piece_placement']} {$tmp_gamedata['fen_active_color']} {$tmp_gamedata['fen_castling_availability']} {$tmp_gamedata['fen_en_passant_target_square']}";
 	if ($tmp_gamedata['fen_fullmove_number'] != $last_move_number and $board_state == $last_board_state) {
-		$repeats[] = $tmp_gamedata['fen_fullmove_number'] . $tmp_gamedata['fen_active_color'];
+		$repeats[] = $tmp_gamedata['fen_fullmove_number'].$tmp_gamedata['fen_active_color'];
 		if (CHESS_LOG_3FOLD) {
 			$log[] = sprintf("%08x %03d%1s %s", crc32($board_state), $tmp_gamedata['fen_fullmove_number'], $tmp_gamedata['fen_active_color'], $board_state);
 		}
@@ -631,7 +631,7 @@ 
 	#var_dump('movelist', $movelist);#*#DEBUG#
 
 	// Compare board positions after each move with last board position.
-	foreach($movelist as $fullmove) {
+	foreach ($movelist as $fullmove) {
 
 		#echo "'{$fullmove[0]}' '{$fullmove[1]}' '{$fullmove[2]}'<br />\n";#*#DEBUG#
 		if (CHESS_LOG_3FOLD) {
@@ -661,7 +661,7 @@ 
 				$log[] = sprintf("%08x %03d%1s %s", crc32($board_state), $tmp_gamedata['fen_fullmove_number'], $tmp_gamedata['fen_active_color'], $board_state);
 			}
 			if ($tmp_gamedata['fen_fullmove_number'] != $last_move_number and $board_state == $last_board_state) {
-				$repeats[] = $tmp_gamedata['fen_fullmove_number'] . $tmp_gamedata['fen_active_color'];
+				$repeats[] = $tmp_gamedata['fen_fullmove_number'].$tmp_gamedata['fen_active_color'];
 #*#DEBUG# - start
 /***
 				if (count($repeats) >= 3) {
@@ -700,7 +700,7 @@ 
 	}
 
 	if (CHESS_LOG_3FOLD) {
-		$logfile = XOOPS_ROOT_PATH . '/cache/' . date('Ymd_His') . '_3fold.log';
+		$logfile = XOOPS_ROOT_PATH.'/cache/'.date('Ymd_His').'_3fold.log';
 		sort($log);
 		error_log(implode("\n", $log), 3, $logfile);
 	}
@@ -753,14 +753,14 @@ 
 	global $xoopsTpl;
 
 	$xoopsTpl->assign('xoops_module_header', '
-		<link rel="stylesheet" type="text/css" media="screen" href="' .XOOPS_URL. '/modules/chess/include/style.css" />
+		<link rel="stylesheet" type="text/css" media="screen" href="' .XOOPS_URL.'/modules/chess/include/style.css" />
 	');
 
-	$member_handler =& xoops_gethandler('member');
-	$white_user     =& $member_handler->getUser($gamedata['white_uid']);
-	$white_username =  is_object($white_user) ? $white_user->getVar('uname') : '?';
-	$black_user     =& $member_handler->getUser($gamedata['black_uid']);
-	$black_username =  is_object($black_user) ? $black_user->getVar('uname') : '?';
+	$member_handler = & xoops_gethandler('member');
+	$white_user     = & $member_handler->getUser($gamedata['white_uid']);
+	$white_username = is_object($white_user) ? $white_user->getVar('uname') : '?';
+	$black_user     = & $member_handler->getUser($gamedata['black_uid']);
+	$black_username = is_object($black_user) ? $black_user->getVar('uname') : '?';
 
 	// Determine whether board is flipped (black at bottom) or "normal" (white at bottom).
 	switch ($orientation) {
@@ -815,11 +815,11 @@ 
 	}
 
 	$xoopsTpl->assign('chess_create_date', $gamedata['create_date'] != '0000-00-00 00:00:00' ? strtotime($gamedata['create_date']) : 0);
-	$xoopsTpl->assign('chess_start_date',  $gamedata['start_date']  != '0000-00-00 00:00:00' ? strtotime($gamedata['start_date'])  : 0);
-	$xoopsTpl->assign('chess_last_date',   $gamedata['last_date']   != '0000-00-00 00:00:00' ? strtotime($gamedata['last_date'])   : 0);
-	$xoopsTpl->assign('chess_white_user',  $white_username);
-	$xoopsTpl->assign('chess_black_user',  $black_username);
-	$xoopsTpl->assign('chess_tiles',       $tiles);
+	$xoopsTpl->assign('chess_start_date', $gamedata['start_date'] != '0000-00-00 00:00:00' ? strtotime($gamedata['start_date']) : 0);
+	$xoopsTpl->assign('chess_last_date', $gamedata['last_date'] != '0000-00-00 00:00:00' ? strtotime($gamedata['last_date']) : 0);
+	$xoopsTpl->assign('chess_white_user', $white_username);
+	$xoopsTpl->assign('chess_black_user', $black_username);
+	$xoopsTpl->assign('chess_tiles', $tiles);
 	$xoopsTpl->assign('chess_file_labels', $file_labels);
 
 	$xoopsTpl->assign('chess_pgn_string',
@@ -839,40 +839,40 @@ 
 		)
 	);
 
-	$xoopsTpl->assign('chess_movelist',    $movelist);
+	$xoopsTpl->assign('chess_movelist', $movelist);
 	$xoopsTpl->assign('chess_date_format', _MEDIUMDATESTRING);
 
 	#if (empty($move_result_text)) {$move_result_text = 'test';}#*#DEBUG#
 	$xoopsTpl->assign('chess_move_performed', $move_performed);
-	$xoopsTpl->assign('chess_move_result',    $move_result_text);
+	$xoopsTpl->assign('chess_move_result', $move_result_text);
 
 	$xoopsTpl->assign('chess_draw_claim_error_text', $draw_claim_error_text);
 
 	$xoopsTpl->assign('chess_user_color', $user_color);
 
-	$xoopsTpl->assign('chess_confirm',     $gamedata[$user_color == 'w' ? 'white_confirm' : 'black_confirm']);
+	$xoopsTpl->assign('chess_confirm', $gamedata[$user_color == 'w' ? 'white_confirm' : 'black_confirm']);
 	$xoopsTpl->assign('chess_orientation', $orientation);
 
-	$xoopsTpl->assign('chess_rank_start',     $flip ? 1      : 8);
-	$xoopsTpl->assign('chess_rank_direction', $flip ? 'up'   : 'down');
+	$xoopsTpl->assign('chess_rank_start', $flip ? 1 : 8);
+	$xoopsTpl->assign('chess_rank_direction', $flip ? 'up' : 'down');
 
-	$xoopsTpl->assign('chess_file_start',     $flip ? 8      : 1);
+	$xoopsTpl->assign('chess_file_start', $flip ? 8 : 1);
 	$xoopsTpl->assign('chess_file_direction', $flip ? 'down' : 'up');
 
 	static $pieces = array(
-		'K' => array('color' => 'w', 'name' => 'wking',   'alt' => _MD_CHESS_ALT_WKING),
-		'Q' => array('color' => 'w', 'name' => 'wqueen',  'alt' => _MD_CHESS_ALT_WQUEEN),
-		'R' => array('color' => 'w', 'name' => 'wrook',   'alt' => _MD_CHESS_ALT_WROOK),
+		'K' => array('color' => 'w', 'name' => 'wking', 'alt' => _MD_CHESS_ALT_WKING),
+		'Q' => array('color' => 'w', 'name' => 'wqueen', 'alt' => _MD_CHESS_ALT_WQUEEN),
+		'R' => array('color' => 'w', 'name' => 'wrook', 'alt' => _MD_CHESS_ALT_WROOK),
 		'B' => array('color' => 'w', 'name' => 'wbishop', 'alt' => _MD_CHESS_ALT_WBISHOP),
 		'N' => array('color' => 'w', 'name' => 'wknight', 'alt' => _MD_CHESS_ALT_WKNIGHT),
-		'P' => array('color' => 'w', 'name' => 'wpawn',   'alt' => _MD_CHESS_ALT_WPAWN),
-		'k' => array('color' => 'b', 'name' => 'bking',   'alt' => _MD_CHESS_ALT_BKING),
-		'q' => array('color' => 'b', 'name' => 'bqueen',  'alt' => _MD_CHESS_ALT_BQUEEN),
-		'r' => array('color' => 'b', 'name' => 'brook',   'alt' => _MD_CHESS_ALT_BROOK),
+		'P' => array('color' => 'w', 'name' => 'wpawn', 'alt' => _MD_CHESS_ALT_WPAWN),
+		'k' => array('color' => 'b', 'name' => 'bking', 'alt' => _MD_CHESS_ALT_BKING),
+		'q' => array('color' => 'b', 'name' => 'bqueen', 'alt' => _MD_CHESS_ALT_BQUEEN),
+		'r' => array('color' => 'b', 'name' => 'brook', 'alt' => _MD_CHESS_ALT_BROOK),
 		'b' => array('color' => 'b', 'name' => 'bbishop', 'alt' => _MD_CHESS_ALT_BBISHOP),
 		'n' => array('color' => 'b', 'name' => 'bknight', 'alt' => _MD_CHESS_ALT_BKNIGHT),
-		'p' => array('color' => 'b', 'name' => 'bpawn',   'alt' => _MD_CHESS_ALT_BPAWN),
-		'x' => array('color' => 'x', 'name' => 'empty',   'alt' => _MD_CHESS_ALT_EMPTY),
+		'p' => array('color' => 'b', 'name' => 'bpawn', 'alt' => _MD_CHESS_ALT_BPAWN),
+		'x' => array('color' => 'x', 'name' => 'empty', 'alt' => _MD_CHESS_ALT_EMPTY),
 	);
 	$xoopsTpl->assign('chess_pieces', $pieces);
 
@@ -891,8 +891,8 @@ 
 				$susp_type_display = _MD_CHESS_LABEL_ERROR;
 				break;
 		}
-		$susp_user     =& $member_handler->getUser($susp_uid);
-		$susp_username =  is_object($susp_user) ? $susp_user->getVar('uname') : _MD_CHESS_UNKNOWN;
+		$susp_user     = & $member_handler->getUser($susp_uid);
+		$susp_username = is_object($susp_user) ? $susp_user->getVar('uname') : _MD_CHESS_UNKNOWN;
 		$suspend_info = array(
 			'date'   => strtotime($susp_date),
 			'user'   => $susp_username,
@@ -929,7 +929,7 @@ 
 	#var_dump('captured_pieces', $captured_pieces);#*#DEBUG#
 	$xoopsTpl->assign('chess_captured_pieces', $captured_pieces);
 
-	$xoopsTpl->assign('chess_pawn_promote_choices', $gamedata['fen_active_color'] == 'w' ? array('Q','R','B','N') : array('q','r','b','n'));
+	$xoopsTpl->assign('chess_pawn_promote_choices', $gamedata['fen_active_color'] == 'w' ? array('Q', 'R', 'B', 'N') : array('q', 'r', 'b', 'n'));
 
 	$xoopsTpl->assign('chess_allow_delete', chess_can_delete());
 

docs/maketestdata.php

Spacing +5 added lines, -5 removed lines

@@ -18,10 +18,10 @@ 
 define('DBUSER', 'root');
 define('DBPASS', '');
 
-define('NUM_USERS',      3);
+define('NUM_USERS', 3);
 define('NUM_CHALLENGES', 1000);
-define('NUM_GAMES',      10000);
-define('NUM_RATINGS',    NUM_USERS / 2);
+define('NUM_GAMES', 10000);
+define('NUM_RATINGS', NUM_USERS / 2);
 /**#@-*/
 
 perform();
@@ -35,8 +35,8 @@ 
 	$games_table      = 'chess_games';
 	$ratings_table    = 'chess_ratings';
 
-	@mysql_connect(DBHOST, DBUSER, DBPASS) or trigger_error('[' .mysql_errno(). '] ' .mysql_error(), E_USER_ERROR);
-	mysql_select_db(DBNAME)                or trigger_error('[' .mysql_errno(). '] ' .mysql_error(), E_USER_ERROR);
+	@mysql_connect(DBHOST, DBUSER, DBPASS) or trigger_error('['.mysql_errno().'] '.mysql_error(), E_USER_ERROR);
+	mysql_select_db(DBNAME) or trigger_error('['.mysql_errno().'] '.mysql_error(), E_USER_ERROR);
 
 // For safety, don't generate test data unless the tables are empty.
 

EXTRA/htdocs/class/smarty/plugins/modifier.xoops_format_date.php

Spacing +4 added lines, -4 removed lines

@@ -31,7 +31,7 @@ 
  * -------------------------------------------------------------
  */
 
-function smarty_modifier_xoops_format_date($timestamp, $format, $nojavascript=null)
+function smarty_modifier_xoops_format_date($timestamp, $format, $nojavascript = null)
 {
 	static $javascript_include_generated = false;
 
@@ -63,7 +63,7 @@ 
 
 	if ($use_javascript) {
 		if (!$javascript_include_generated) {
-			$file = XOOPS_URL . '/include/phpdate.js';
+			$file = XOOPS_URL.'/include/phpdate.js';
 			$rtn .= "<script type='text/javascript' src='$file'></script>";
 			$javascript_include_generated = true;
 		}
@@ -72,10 +72,10 @@ 
 	if ($use_javascript) {
 
 		// backslashes have to be encoded (replaced with ASCII code) for Javascript
-		$format_encoded = str_replace('\\', '\\x' . dechex(ord('\\')), $format);
+		$format_encoded = str_replace('\\', '\\x'.dechex(ord('\\')), $format);
 
 		// ampersands have to be encoded for XHTML compliancy
-		$format_encoded = str_replace('&', '\x' . dechex(ord('&')), $format_encoded);
+		$format_encoded = str_replace('&', '\x'.dechex(ord('&')), $format_encoded);
 
 		// Javascript-formatted date
 		#$jdate = "'J' + phpDate('$format_encoded', $timestamp)"; #*#DEBUG# - 'J' only for debugging purposes

xoops_version.php

Spacing +19 added lines, -19 removed lines

@@ -149,7 +149,7 @@ 
 // Display my-games menu item if current user is logged in.
 	if (is_object($xoopsUser)) {
 		$modversion['sub'][5]['name'] = _MI_CHESS_SMNAME5;
-		$modversion['sub'][5]['url']  = 'player_stats.php?player_uid=' . $xoopsUser->getVar('uid');
+		$modversion['sub'][5]['url']  = 'player_stats.php?player_uid='.$xoopsUser->getVar('uid');
 	}
 }
 
@@ -166,24 +166,24 @@ 
 $modversion['pages'][5]['url'] = "player_stats.php";
 
 // Templates
-$modversion['templates'][ 1]['file']        = 'chess_games.html';
-$modversion['templates'][ 1]['description'] = _MI_CHESS_INDEX;
-$modversion['templates'][ 2]['file']        = 'chess_game_main.html';
-$modversion['templates'][ 2]['description'] = _MI_CHESS_GAME;
-$modversion['templates'][ 3]['file']        = 'chess_game_moveform.html';
-$modversion['templates'][ 3]['description'] = _MI_CHESS_MOVE_FORM;
-$modversion['templates'][ 4]['file']        = 'chess_game_promote_popup.html';
-$modversion['templates'][ 4]['description'] = _MI_CHESS_PROMOTE_POPUP;
-$modversion['templates'][ 5]['file']        = 'chess_game_board.html';
-$modversion['templates'][ 5]['description'] = _MI_CHESS_BOARD;
-$modversion['templates'][ 6]['file']        = 'chess_game_prefsform.html';
-$modversion['templates'][ 6]['description'] = _MI_CHESS_PREFS_FORM;
-$modversion['templates'][ 7]['file']        = 'chess_game_arbitrateform.html';
-$modversion['templates'][ 7]['description'] = _MI_CHESS_ARBITER_FORM;
-$modversion['templates'][ 8]['file']        = 'chess_help.html';
-$modversion['templates'][ 8]['description'] = _MI_CHESS_HELP;
-$modversion['templates'][ 9]['file']        = 'chess_ratings.html';
-$modversion['templates'][ 9]['description'] = _MI_CHESS_RATINGS;
+$modversion['templates'][1]['file']        = 'chess_games.html';
+$modversion['templates'][1]['description'] = _MI_CHESS_INDEX;
+$modversion['templates'][2]['file']        = 'chess_game_main.html';
+$modversion['templates'][2]['description'] = _MI_CHESS_GAME;
+$modversion['templates'][3]['file']        = 'chess_game_moveform.html';
+$modversion['templates'][3]['description'] = _MI_CHESS_MOVE_FORM;
+$modversion['templates'][4]['file']        = 'chess_game_promote_popup.html';
+$modversion['templates'][4]['description'] = _MI_CHESS_PROMOTE_POPUP;
+$modversion['templates'][5]['file']        = 'chess_game_board.html';
+$modversion['templates'][5]['description'] = _MI_CHESS_BOARD;
+$modversion['templates'][6]['file']        = 'chess_game_prefsform.html';
+$modversion['templates'][6]['description'] = _MI_CHESS_PREFS_FORM;
+$modversion['templates'][7]['file']        = 'chess_game_arbitrateform.html';
+$modversion['templates'][7]['description'] = _MI_CHESS_ARBITER_FORM;
+$modversion['templates'][8]['file']        = 'chess_help.html';
+$modversion['templates'][8]['description'] = _MI_CHESS_HELP;
+$modversion['templates'][9]['file']        = 'chess_ratings.html';
+$modversion['templates'][9]['description'] = _MI_CHESS_RATINGS;
 $modversion['templates'][10]['file']        = 'chess_player_stats.html';
 $modversion['templates'][10]['description'] = _MI_CHESS_PLAYER_STATS;
 

help.php

Spacing +12 added lines, -12 removed lines

@@ -38,16 +38,16 @@ 
 include '../../mainfile.php';
 $xoopsOption['template_main'] = 'chess_help.html';
 $xoopsConfig['module_cache'][$xoopsModule->getVar('mid')] = 0; // disable caching
-include_once XOOPS_ROOT_PATH . '/header.php';
-require_once XOOPS_ROOT_PATH . '/modules/'.$xoopsModule->getVar('dirname').'/include/functions.inc.php';
-if (file_exists(XOOPS_ROOT_PATH . '/modules/'.$xoopsModule->getVar('dirname').'/language/'.$xoopsConfig['language'].'/help.php')) {
-	include_once XOOPS_ROOT_PATH . '/modules/'.$xoopsModule->getVar('dirname').'/language/'.$xoopsConfig['language'].'/help.php';
+include_once XOOPS_ROOT_PATH.'/header.php';
+require_once XOOPS_ROOT_PATH.'/modules/'.$xoopsModule->getVar('dirname').'/include/functions.inc.php';
+if (file_exists(XOOPS_ROOT_PATH.'/modules/'.$xoopsModule->getVar('dirname').'/language/'.$xoopsConfig['language'].'/help.php')) {
+	include_once XOOPS_ROOT_PATH.'/modules/'.$xoopsModule->getVar('dirname').'/language/'.$xoopsConfig['language'].'/help.php';
 } else {
-	include_once XOOPS_ROOT_PATH . '/modules/'.$xoopsModule->getVar('dirname').'/language/english/help.php';
+	include_once XOOPS_ROOT_PATH.'/modules/'.$xoopsModule->getVar('dirname').'/language/english/help.php';
 }
 
 $rating_system          = chess_moduleConfig('rating_system');
-$rating_system_des_name = '_HE_CHESS_RATINGS_' . strtoupper($rating_system);
+$rating_system_des_name = '_HE_CHESS_RATINGS_'.strtoupper($rating_system);
 if (defined($rating_system_des_name)) {
 	$rating_system_des = constant($rating_system_des_name);
 } else {
@@ -55,14 +55,14 @@ 
 	$rating_system_des = $rating_system_des_name;
 }
 
-$xoopsTpl->assign('chess_allow_setup',         chess_moduleConfig('allow_setup'));
-$xoopsTpl->assign('chess_rating_system',       $rating_system);
-$xoopsTpl->assign('chess_rating_system_des',   $rating_system_des);
+$xoopsTpl->assign('chess_allow_setup', chess_moduleConfig('allow_setup'));
+$xoopsTpl->assign('chess_rating_system', $rating_system);
+$xoopsTpl->assign('chess_rating_system_des', $rating_system_des);
 $xoopsTpl->assign('chess_allow_unrated_games', chess_moduleConfig('allow_unrated_games'));
-$xoopsTpl->assign('chess_allow_delete',        chess_can_delete());
-$xoopsTpl->assign('chess_is_admin',            is_object($xoopsUser) and $xoopsUser->isAdmin($xoopsModule->getVar('mid')));
+$xoopsTpl->assign('chess_allow_delete', chess_can_delete());
+$xoopsTpl->assign('chess_is_admin', is_object($xoopsUser) and $xoopsUser->isAdmin($xoopsModule->getVar('mid')));
 
-include_once XOOPS_ROOT_PATH . '/footer.php';
+include_once XOOPS_ROOT_PATH.'/footer.php';
 
 /**#@-*/
 

create.php

Spacing +88 added lines, -88 removed lines

@@ -44,10 +44,10 @@ 
  */
 require_once '../../mainfile.php';
 $xoopsConfig['module_cache'][$xoopsModule->getVar('mid')] = 0; // disable caching
-require_once XOOPS_ROOT_PATH . '/header.php';
-require_once XOOPS_ROOT_PATH . '/class/xoopsformloader.php';
-require_once XOOPS_ROOT_PATH . '/modules/chess/include/constants.inc.php';
-require_once XOOPS_ROOT_PATH . '/modules/chess/include/functions.inc.php';
+require_once XOOPS_ROOT_PATH.'/header.php';
+require_once XOOPS_ROOT_PATH.'/class/xoopsformloader.php';
+require_once XOOPS_ROOT_PATH.'/modules/chess/include/constants.inc.php';
+require_once XOOPS_ROOT_PATH.'/modules/chess/include/functions.inc.php';
 
 #var_dump($_REQUEST);#*#DEBUG#
 
@@ -79,8 +79,8 @@ 
 
 // If form-submit, check security token.
 if (($submit_challenge1 or $submit_challenge2 or $submit_challenge3 or $submit_accept or $submit_delete or $show_arbiter_ctrl) and is_object($GLOBALS['xoopsSecurity']) and !$GLOBALS['xoopsSecurity']->check()) {
-	redirect_header(XOOPS_URL . '/modules/chess/', _CHESS_REDIRECT_DELAY_FAILURE,
-		_MD_CHESS_TOKEN_ERROR . '<br />' . implode('<br />', $GLOBALS['xoopsSecurity']->getErrors()));
+	redirect_header(XOOPS_URL.'/modules/chess/', _CHESS_REDIRECT_DELAY_FAILURE,
+		_MD_CHESS_TOKEN_ERROR.'<br />'.implode('<br />', $GLOBALS['xoopsSecurity']->getErrors()));
 }
 
 // If rating feature disabled, force ratings to off.
@@ -117,7 +117,7 @@ 
 
 	$fen_error = chess_fen_error($fen);
 	if (!empty($fen_error)) {
-		chess_show_create_form1($gametype, $fen, _MD_CHESS_FEN_INVALID . ': ' . $fen_error);
+		chess_show_create_form1($gametype, $fen, _MD_CHESS_FEN_INVALID.': '.$fen_error);
 	} elseif ($gametype == _CHESS_GAMETYPE_OPEN or $gametype == _CHESS_GAMETYPE_USER) {
 		chess_show_create_form2($gametype, $fen);
 	} else {
@@ -158,7 +158,7 @@ 
 		}
 	} elseif ($gametype == _CHESS_GAMETYPE_SELF) {
 		$game_id = chess_create_game($uid, $uid, $fen, $rated);
-		redirect_header(XOOPS_URL . "/modules/chess/game.php?game_id=$game_id", _CHESS_REDIRECT_DELAY_SUCCESS, _MD_CHESS_GAME_CREATED);
+		redirect_header(XOOPS_URL."/modules/chess/game.php?game_id=$game_id", _CHESS_REDIRECT_DELAY_SUCCESS, _MD_CHESS_GAME_CREATED);
 	} else {
 		chess_show_create_form1($gametype, $fen, _MD_CHESS_GAMETYPE_INVALID);
 	}
@@ -208,10 +208,10 @@ 
 	$form = new XoopsThemeForm(_MD_CHESS_CREATE_FORM, 'create_form1', 'create.php', 'post', true);
 
 	if ($error_msg) {
-		$form->addElement(new XoopsFormLabel(_MD_CHESS_ERROR . ': ', '<div class="errorMsg">' . $error_msg . '</div>'));
+		$form->addElement(new XoopsFormLabel(_MD_CHESS_ERROR.': ', '<div class="errorMsg">'.$error_msg.'</div>'));
 	}
 
-	$menu_gametype = new XoopsFormSelect(_MD_CHESS_LABEL_GAMETYPE . ':', 'gametype', $gametype, 1, false);
+	$menu_gametype = new XoopsFormSelect(_MD_CHESS_LABEL_GAMETYPE.':', 'gametype', $gametype, 1, false);
 	$menu_gametype->addOption(_CHESS_GAMETYPE_OPEN, _MD_CHESS_MENU_GAMETYPE_OPEN);
 	$menu_gametype->addOption(_CHESS_GAMETYPE_USER, _MD_CHESS_MENU_GAMETYPE_USER);
 	$menu_gametype->addOption(_CHESS_GAMETYPE_SELF, _MD_CHESS_MENU_GAMETYPE_SELF);
@@ -220,7 +220,7 @@ 
 	if (chess_moduleConfig('allow_setup')) {
 		$form->addElement(new XoopsFormLabel('', '&nbsp;'));
 		$form->addElement(new XoopsFormLabel('', _MD_CHESS_LABEL_FEN_EXPLAIN));
-		$form->addElement(new XoopsFormText(_MD_CHESS_LABEL_FEN_SETUP . ':', 'fen', 80, _CHESS_TEXTBOX_FEN_MAXLEN, $fen));
+		$form->addElement(new XoopsFormText(_MD_CHESS_LABEL_FEN_SETUP.':', 'fen', 80, _CHESS_TEXTBOX_FEN_MAXLEN, $fen));
 	}
 
 	$form->addElement(new XoopsFormLabel('&nbsp;', '&nbsp;'));
@@ -248,32 +248,32 @@ 
 	$form = new XoopsThemeForm(_MD_CHESS_CREATE_FORM, 'create_form2', 'create.php', 'post', true);
 
 	$form->addElement(new XoopsFormHidden('gametype', $gametype));
-	$form->addElement(new XoopsFormHidden('fen',      $fen));
+	$form->addElement(new XoopsFormHidden('fen', $fen));
 
 	if ($error_msg) {
-		$form->addElement(new XoopsFormLabel(_MD_CHESS_ERROR . ':', '<div class="errorMsg">' . $error_msg . '</div>'));
+		$form->addElement(new XoopsFormLabel(_MD_CHESS_ERROR.':', '<div class="errorMsg">'.$error_msg.'</div>'));
 	}
 
-	$member_handler    =& xoops_gethandler('member');
-	$opponent_user     =& $member_handler->getUser($opponent_uid);
-	$opponent_username =  is_object($opponent_user) ? $opponent_user->getVar('uname') : '';
+	$member_handler    = & xoops_gethandler('member');
+	$opponent_user     = & $member_handler->getUser($opponent_uid);
+	$opponent_username = is_object($opponent_user) ? $opponent_user->getVar('uname') : '';
 
 	if ($gametype == _CHESS_GAMETYPE_USER) {
-		$form->addElement(new XoopsFormText(_MD_CHESS_LABEL_OPPONENT . ':', 'opponent', _CHESS_TEXTBOX_OPPONENT_SIZE,
+		$form->addElement(new XoopsFormText(_MD_CHESS_LABEL_OPPONENT.':', 'opponent', _CHESS_TEXTBOX_OPPONENT_SIZE,
 			_CHESS_TEXTBOX_OPPONENT_MAXLEN, $opponent_username));
 	}
 
-	$radio_color = new XoopsFormRadio(_MD_CHESS_LABEL_COLOR . ':', 'coloroption', $coloroption);
+	$radio_color = new XoopsFormRadio(_MD_CHESS_LABEL_COLOR.':', 'coloroption', $coloroption);
 	$radio_color->addOption(_CHESS_COLOROPTION_OPPONENT, _MD_CHESS_RADIO_COLOR_OPPONENT);
-	$radio_color->addOption(_CHESS_COLOROPTION_RANDOM,   _MD_CHESS_RADIO_COLOR_RANDOM);
-	$radio_color->addOption(_CHESS_COLOROPTION_WHITE,    _MD_CHESS_RADIO_COLOR_WHITE);
-	$radio_color->addOption(_CHESS_COLOROPTION_BLACK,    _MD_CHESS_RADIO_COLOR_BLACK);
+	$radio_color->addOption(_CHESS_COLOROPTION_RANDOM, _MD_CHESS_RADIO_COLOR_RANDOM);
+	$radio_color->addOption(_CHESS_COLOROPTION_WHITE, _MD_CHESS_RADIO_COLOR_WHITE);
+	$radio_color->addOption(_CHESS_COLOROPTION_BLACK, _MD_CHESS_RADIO_COLOR_BLACK);
 	$form->addElement($radio_color);
 
 	if (chess_moduleConfig('rating_system') !== 'none') {
 		if ($gametype == _CHESS_GAMETYPE_OPEN or $gametype == _CHESS_GAMETYPE_USER) {
 			if (chess_moduleConfig('allow_unrated_games')) {
-				$radio_rated = new XoopsFormRadio(_MD_CHESS_RATED_GAME . ':', 'rated', $rated);
+				$radio_rated = new XoopsFormRadio(_MD_CHESS_RATED_GAME.':', 'rated', $rated);
 				$radio_rated->addOption(1, _YES);
 				$radio_rated->addOption(0, _NO);
 				$form->addElement($radio_rated);
@@ -304,21 +304,21 @@ 
  */
 function chess_show_create_form3($gametype, $fen, $coloroption, $opponent_uid, $rated)
 {
-	$member_handler    =& xoops_gethandler('member');
-	$opponent_user     =& $member_handler->getUser($opponent_uid);
-	$opponent_username =  is_object($opponent_user) ? $opponent_user->getVar('uname') : '';
+	$member_handler    = & xoops_gethandler('member');
+	$opponent_user     = & $member_handler->getUser($opponent_uid);
+	$opponent_username = is_object($opponent_user) ? $opponent_user->getVar('uname') : '';
 
 	$form = new XoopsThemeForm(_MD_CHESS_CREATE_FORM, 'create_form3', 'create.php', 'post', true);
 
-	$form->addElement(new XoopsFormHidden('gametype',    $gametype));
-	$form->addElement(new XoopsFormHidden('fen',         $fen));
-	$form->addElement(new XoopsFormHidden('opponent',    $opponent_username));
+	$form->addElement(new XoopsFormHidden('gametype', $gametype));
+	$form->addElement(new XoopsFormHidden('fen', $fen));
+	$form->addElement(new XoopsFormHidden('opponent', $opponent_username));
 	$form->addElement(new XoopsFormHidden('coloroption', $coloroption));
-	$form->addElement(new XoopsFormHidden('rated',       $rated));
+	$form->addElement(new XoopsFormHidden('rated', $rated));
 
-	$form->addElement(new XoopsFormLabel('', '<div class="confirmMsg">' . _MD_CHESS_GAME_CONFIRM . '</div>'));
+	$form->addElement(new XoopsFormLabel('', '<div class="confirmMsg">'._MD_CHESS_GAME_CONFIRM.'</div>'));
 
-	switch($gametype) {
+	switch ($gametype) {
 		case _CHESS_GAMETYPE_OPEN:
 			$label_gametype = _MD_CHESS_LABEL_GAMETYPE_OPEN;
 			break;
@@ -332,24 +332,24 @@ 
 			$label_gametype = _MD_CHESS_LABEL_ERROR;
 			break;
 	}
-	$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_GAMETYPE . ':', $label_gametype));
+	$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_GAMETYPE.':', $label_gametype));
 
 	if (!empty($fen)) {
-		$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_FEN_SETUP . ':', $fen));
+		$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_FEN_SETUP.':', $fen));
 	}
 
 	if ($gametype == _CHESS_GAMETYPE_USER) {
 
-		$member_handler    =& xoops_gethandler('member');
-		$opponent_user     =& $member_handler->getUser($opponent_uid);
-		$opponent_username =  is_object($opponent_user) ? $opponent_user->getVar('uname') : '';
+		$member_handler    = & xoops_gethandler('member');
+		$opponent_user     = & $member_handler->getUser($opponent_uid);
+		$opponent_username = is_object($opponent_user) ? $opponent_user->getVar('uname') : '';
 
-		$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_OPPONENT . ':', $opponent_username));
+		$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_OPPONENT.':', $opponent_username));
 	}
 
 	if ($gametype == _CHESS_GAMETYPE_OPEN or $gametype == _CHESS_GAMETYPE_USER) {
 
-		switch($coloroption) {
+		switch ($coloroption) {
 			case _CHESS_COLOROPTION_OPPONENT:
 				$label_coloroption = _MD_CHESS_RADIO_COLOR_OPPONENT;
 				break;
@@ -366,17 +366,17 @@ 
 				$label_coloroption = _MD_CHESS_LABEL_ERROR;
 				break;
 		}
-		$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_COLOR . ':', $label_coloroption));
+		$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_COLOR.':', $label_coloroption));
 
 		if (chess_moduleConfig('rating_system') != 'none') {
-			$form->addElement(new XoopsFormLabel(_MD_CHESS_RATED_GAME . ':', $rated ? _YES : _NO));
+			$form->addElement(new XoopsFormLabel(_MD_CHESS_RATED_GAME.':', $rated ? _YES : _NO));
 		}
 
 		// Determine whether current user is subscribed to receive a notification when his challenge is accepted.
 		global $xoopsModule, $xoopsUser;
 		$uid = $xoopsUser->getVar('uid');
 		$mid = $xoopsModule->getVar('mid');
-		$notification_handler =& xoops_gethandler('notification');
+		$notification_handler = & xoops_gethandler('notification');
 		$is_subscribed = $notification_handler->isSubscribed('global', 0, 'notify_accept_challenge', $mid, $uid) != 0;
 
 		// Display checkbox with checked-state reflecting subscription status.
@@ -427,20 +427,20 @@ 
 
 	$form = new XoopsThemeForm(_MD_CHESS_ACCEPT_FORM, 'accept_form', 'create.php', 'post', true);
 
-	$form->addElement(new XoopsFormHidden('challenge_id',   $challenge_id));
+	$form->addElement(new XoopsFormHidden('challenge_id', $challenge_id));
 
-	$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_DATE_CREATED . ':', formatTimestamp($row['create_date'], "m")));
+	$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_DATE_CREATED.':', formatTimestamp($row['create_date'], "m")));
 
-	$member_handler =& xoops_gethandler('member');
+	$member_handler = & xoops_gethandler('member');
 
-	switch($row['game_type']) {
+	switch ($row['game_type']) {
 		case _CHESS_GAMETYPE_OPEN:
 			$label_gametype = _MD_CHESS_LABEL_GAMETYPE_OPEN;
 			break;
 		case _CHESS_GAMETYPE_USER:
-			$player2_user     =& $member_handler->getUser($row['player2_uid']);
-			$player2_username =  is_object($player2_user) ? $player2_user->getVar('uname') : '?';
-			$label_gametype = _MD_CHESS_LABEL_GAMETYPE_USER . ': ' . $player2_username;
+			$player2_user     = & $member_handler->getUser($row['player2_uid']);
+			$player2_username = is_object($player2_user) ? $player2_user->getVar('uname') : '?';
+			$label_gametype = _MD_CHESS_LABEL_GAMETYPE_USER.': '.$player2_username;
 			break;
 		case _CHESS_GAMETYPE_SELF:
 			$label_gametype = _MD_CHESS_LABEL_GAMETYPE_SELF;
@@ -449,26 +449,26 @@ 
 			$label_gametype = _MD_CHESS_LABEL_ERROR;
 			break;
 	}
-	$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_GAMETYPE . ':', $label_gametype));
+	$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_GAMETYPE.':', $label_gametype));
 
-	$player1_user     =& $member_handler->getUser($row['player1_uid']);
-	$player1_username =  is_object($player1_user) ? $player1_user->getVar('uname') : '?';
-	$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_CHALLENGER . ':',
-		"<a href='" .XOOPS_URL. "/modules/chess/player_stats.php?player_uid={$row['player1_uid']}'>$player1_username</a>"
+	$player1_user     = & $member_handler->getUser($row['player1_uid']);
+	$player1_username = is_object($player1_user) ? $player1_user->getVar('uname') : '?';
+	$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_CHALLENGER.':',
+		"<a href='".XOOPS_URL."/modules/chess/player_stats.php?player_uid={$row['player1_uid']}'>$player1_username</a>"
 	));
 
-	$player2_username =  $xoopsUser ? $xoopsUser->getVar('uname') : '?';
-	$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_OPPONENT . ':', $player2_username));
+	$player2_username = $xoopsUser ? $xoopsUser->getVar('uname') : '?';
+	$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_OPPONENT.':', $player2_username));
 
 	if (!empty($row['fen'])) {
-		$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_FEN_SETUP . ':', $row['fen']));
+		$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_FEN_SETUP.':', $row['fen']));
 	}
 
 	if ($row['color_option'] == _CHESS_COLOROPTION_OPPONENT) {
-		$radio_color = new XoopsFormRadio(_MD_CHESS_LABEL_COLOR . ':', 'coloroption', _CHESS_COLOROPTION_RANDOM);
-		$radio_color->addOption(_CHESS_COLOROPTION_RANDOM,   _MD_CHESS_RADIO_COLOR_RANDOM);
-		$radio_color->addOption(_CHESS_COLOROPTION_WHITE,    _MD_CHESS_RADIO_COLOR_WHITE);
-		$radio_color->addOption(_CHESS_COLOROPTION_BLACK,    _MD_CHESS_RADIO_COLOR_BLACK);
+		$radio_color = new XoopsFormRadio(_MD_CHESS_LABEL_COLOR.':', 'coloroption', _CHESS_COLOROPTION_RANDOM);
+		$radio_color->addOption(_CHESS_COLOROPTION_RANDOM, _MD_CHESS_RADIO_COLOR_RANDOM);
+		$radio_color->addOption(_CHESS_COLOROPTION_WHITE, _MD_CHESS_RADIO_COLOR_WHITE);
+		$radio_color->addOption(_CHESS_COLOROPTION_BLACK, _MD_CHESS_RADIO_COLOR_BLACK);
 		$form->addElement($radio_color);
 	} else {
 		switch ($row['color_option']) {
@@ -485,11 +485,11 @@ 
 				$label_coloroption = _MD_CHESS_LABEL_ERROR;
 				break;
 		}
-		$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_COLOR . ':', $label_coloroption));
+		$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_COLOR.':', $label_coloroption));
 	}
 
 	if (chess_moduleConfig('rating_system') != 'none') {
-		$form->addElement(new XoopsFormLabel(_MD_CHESS_RATED_GAME . ':', $row['is_rated'] ? _YES : _NO));
+		$form->addElement(new XoopsFormLabel(_MD_CHESS_RATED_GAME.':', $row['is_rated'] ? _YES : _NO));
 	}
 
 	// Display notification-subscribe checkbox, initially checked.
@@ -535,28 +535,28 @@ 
 
 	$form = new XoopsThemeForm(_MD_CHESS_DELETE_FORM, 'delete_form', 'create.php', 'post', true);
 
-	$form->addElement(new XoopsFormHidden('challenge_id',   $challenge_id));
+	$form->addElement(new XoopsFormHidden('challenge_id', $challenge_id));
 
 	if ($show_arbiter_ctrl) {
 		$form->addElement(new XoopsFormHidden('show_arbiter_ctrl', $show_arbiter_ctrl));
 	}
 
 	if ($error_msg) {
-		$form->addElement(new XoopsFormLabel(_MD_CHESS_ERROR . ': ', '<div class="errorMsg">' . $error_msg . '</div>'));
+		$form->addElement(new XoopsFormLabel(_MD_CHESS_ERROR.': ', '<div class="errorMsg">'.$error_msg.'</div>'));
 	}
 
-	$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_DATE_CREATED . ':', formatTimestamp($row['create_date'], "m")));
+	$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_DATE_CREATED.':', formatTimestamp($row['create_date'], "m")));
 
-	$member_handler =& xoops_gethandler('member');
+	$member_handler = & xoops_gethandler('member');
 
-	switch($row['game_type']) {
+	switch ($row['game_type']) {
 		case _CHESS_GAMETYPE_OPEN:
 			$label_gametype = _MD_CHESS_LABEL_GAMETYPE_OPEN;
 			break;
 		case _CHESS_GAMETYPE_USER:
-			$player2_user     =& $member_handler->getUser($row['player2_uid']);
-			$player2_username =  is_object($player2_user) ? $player2_user->getVar('uname') : '?';
-			$label_gametype = _MD_CHESS_LABEL_GAMETYPE_USER . ': ' . $player2_username;
+			$player2_user     = & $member_handler->getUser($row['player2_uid']);
+			$player2_username = is_object($player2_user) ? $player2_user->getVar('uname') : '?';
+			$label_gametype = _MD_CHESS_LABEL_GAMETYPE_USER.': '.$player2_username;
 			break;
 		case _CHESS_GAMETYPE_SELF:
 			$label_gametype = _MD_CHESS_LABEL_GAMETYPE_SELF;
@@ -565,17 +565,17 @@ 
 			$label_gametype = _MD_CHESS_LABEL_ERROR;
 			break;
 	}
-	$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_GAMETYPE . ':', $label_gametype));
+	$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_GAMETYPE.':', $label_gametype));
 
-	$player1_user     =& $member_handler->getUser($row['player1_uid']);
-	$player1_username =  is_object($player1_user) ? $player1_user->getVar('uname') : '?';
-	$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_CHALLENGER . ':',  $player1_username));
+	$player1_user     = & $member_handler->getUser($row['player1_uid']);
+	$player1_username = is_object($player1_user) ? $player1_user->getVar('uname') : '?';
+	$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_CHALLENGER.':', $player1_username));
 
 	if (!empty($row['fen'])) {
-		$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_FEN_SETUP . ':', $row['fen']));
+		$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_FEN_SETUP.':', $row['fen']));
 	}
 
-	switch($row['color_option']) {
+	switch ($row['color_option']) {
 		case _CHESS_COLOROPTION_OPPONENT:
 			$label_coloroption = _MD_CHESS_RADIO_COLOR_OPPONENT;
 			break;
@@ -592,10 +592,10 @@ 
 			$label_coloroption = _MD_CHESS_LABEL_ERROR;
 			break;
 	}
-	$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_COLOR . ':', $label_coloroption));
+	$form->addElement(new XoopsFormLabel(_MD_CHESS_LABEL_COLOR.':', $label_coloroption));
 
 	if (chess_moduleConfig('rating_system') != 'none') {
-		$form->addElement(new XoopsFormLabel(_MD_CHESS_RATED_GAME . ':', $row['is_rated'] ? _YES : _NO));
+		$form->addElement(new XoopsFormLabel(_MD_CHESS_RATED_GAME.':', $row['is_rated'] ? _YES : _NO));
 	}
 
 	// Display confirm-delete checkbox, initially unchecked.
@@ -701,16 +701,16 @@ 
 
 	$xoopsDB->query("DELETE FROM $challenges_table WHERE challenge_id = '$challenge_id'");
 	if ($xoopsDB->errno()) {
-		trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
+		trigger_error($xoopsDB->errno().':'.$xoopsDB->error(), E_USER_ERROR);
 	}
 
 	// Notify player 1 that his challenge has been accepted (if he has subscribed to the notification).
 	$player2_username = $xoopsUser ? $xoopsUser->getVar('uname') : '*unknown*';
-	$notification_handler =& xoops_gethandler('notification');
+	$notification_handler = & xoops_gethandler('notification');
 	$extra_tags = array('CHESS_PLAYER' => $player2_username, 'CHESS_GAME_ID' => $game_id);
 	$notification_handler->triggerEvent('global', 0, 'notify_accept_challenge', $extra_tags, array($row['player1_uid']));
 
-	redirect_header(XOOPS_URL . "/modules/chess/game.php?game_id=$game_id", _CHESS_REDIRECT_DELAY_SUCCESS, _MD_CHESS_GAME_CREATED);
+	redirect_header(XOOPS_URL."/modules/chess/game.php?game_id=$game_id", _CHESS_REDIRECT_DELAY_SUCCESS, _MD_CHESS_GAME_CREATED);
 }
 
 /**
@@ -781,16 +781,16 @@ 
 			is_rated            = '$rated'
 	"));
 	if ($xoopsDB->errno()) {
-		trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
+		trigger_error($xoopsDB->errno().':'.$xoopsDB->error(), E_USER_ERROR);
 	}
 
 	$challenge_id = $xoopsDB->getInsertId();
 
-	$notification_handler =& xoops_gethandler('notification');
+	$notification_handler = & xoops_gethandler('notification');
 
 	// Update the challenger's subscription for receiving a notification when the challenge is accepted.
 	if ($notify_accept) {
-		$notification_handler->subscribe  ('global', 0, 'notify_accept_challenge');
+		$notification_handler->subscribe('global', 0, 'notify_accept_challenge');
 	} else {
 		$notification_handler->unsubscribe('global', 0, 'notify_accept_challenge');
 	}
@@ -818,7 +818,7 @@ 
 
 	$xoopsDB->query("DELETE FROM $table WHERE challenge_id='$challenge_id'");
 	if ($xoopsDB->errno()) {
-		trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
+		trigger_error($xoopsDB->errno().':'.$xoopsDB->error(), E_USER_ERROR);
 	}
 }
 
@@ -874,12 +874,12 @@ 
 			is_rated                     = '$rated'
 	"));
 	if ($xoopsDB->errno()) {
-		trigger_error($xoopsDB->errno() . ':' . $xoopsDB->error(), E_USER_ERROR);
+		trigger_error($xoopsDB->errno().':'.$xoopsDB->error(), E_USER_ERROR);
 	}
 
 	$game_id = $xoopsDB->getInsertId();
 
-	$notification_handler =& xoops_gethandler('notification');
+	$notification_handler = & xoops_gethandler('notification');
 
 	// If requested, subscribe the challenger to receive a notification when a new move is made.
 	if ($notify_move_player1_uid) {
@@ -903,7 +903,7 @@ 
 function chess_fen_error($fen)
 {
 	if (!empty($fen)) {
-		require_once XOOPS_ROOT_PATH . '/modules/chess/class/chessgame.inc.php';
+		require_once XOOPS_ROOT_PATH.'/modules/chess/class/chessgame.inc.php';
 		$chessgame = new ChessGame($fen);
 		$fen_error = $chessgame->error;
 	} else {

include/notification.inc.php

Spacing +11 added lines, -11 removed lines

@@ -38,10 +38,10 @@ 
 // The language file gets automatically included within this module,
 // but not when viewing the notifications page via notifications.php.
 global $xoopsConfig;
-if (file_exists(XOOPS_ROOT_PATH . "/modules/chess/language/{$xoopsConfig['language']}/main.php")) {
-	include_once XOOPS_ROOT_PATH . "/modules/chess/language/{$xoopsConfig['language']}/main.php";
+if (file_exists(XOOPS_ROOT_PATH."/modules/chess/language/{$xoopsConfig['language']}/main.php")) {
+	include_once XOOPS_ROOT_PATH."/modules/chess/language/{$xoopsConfig['language']}/main.php";
 } else {
-	include_once XOOPS_ROOT_PATH . '/modules/chess/language/english/main.php';
+	include_once XOOPS_ROOT_PATH.'/modules/chess/language/english/main.php';
 }
 /**#@-*/
 
@@ -59,7 +59,7 @@ 
 	if ($category == 'global') {
 
 		$item['name'] = 'Chess';
-		$item['url']  = XOOPS_URL . '/modules/chess/';
+		$item['url']  = XOOPS_URL.'/modules/chess/';
 		return $item;
 
 	} elseif ($category == 'game') {
@@ -78,12 +78,12 @@ 
 	 	if ($gamedata !== false) {
 
 			// get mapping of user IDs to usernames
-			$criteria       =  new Criteria('uid', "({$gamedata['white_uid']}, {$gamedata['black_uid']})", 'IN');
-			$member_handler =& xoops_gethandler('member');
-			$usernames      =  $member_handler->getUserList($criteria);
+			$criteria       = new Criteria('uid', "({$gamedata['white_uid']}, {$gamedata['black_uid']})", 'IN');
+			$member_handler = & xoops_gethandler('member');
+			$usernames      = $member_handler->getUserList($criteria);
 
-			$username_white =  isset($usernames[$gamedata['white_uid']]) ? $usernames[$gamedata['white_uid']] : _MD_CHESS_NA;
-			$username_black =  isset($usernames[$gamedata['black_uid']]) ? $usernames[$gamedata['black_uid']] : _MD_CHESS_NA;
+			$username_white = isset($usernames[$gamedata['white_uid']]) ? $usernames[$gamedata['white_uid']] : _MD_CHESS_NA;
+			$username_black = isset($usernames[$gamedata['black_uid']]) ? $usernames[$gamedata['black_uid']] : _MD_CHESS_NA;
 
 			$date = $gamedata['start_date'] ? date('Y.m.d', $gamedata['start_date']) : _MD_CHESS_NA;
 
@@ -94,8 +94,8 @@ 
 			$date           = _MD_CHESS_NA;
 		}
 
-		$item['name'] = "$username_white " ._MD_CHESS_LABEL_VS. " $username_black ($date)";
-		$item['url']  = XOOPS_URL . '/modules/chess/game.php?game_id=' . $item_id;
+		$item['name'] = "$username_white "._MD_CHESS_LABEL_VS." $username_black ($date)";
+		$item['url']  = XOOPS_URL.'/modules/chess/game.php?game_id='.$item_id;
 		return $item;
 	}
 }