mambax7 / chess

include/ratings_linear.php

Indentation +15 added lines, -15 removed lines

@@ -47,22 +47,22 @@ 
  */
 function chess_ratings_adj_linear($white_rating, $white_games, $black_rating, $black_games, $pgn_result)
 {
-    // compute score: +1 for win, 0 for draw, -1 for loss
+	// compute score: +1 for win, 0 for draw, -1 for loss
 
-    switch ($pgn_result) {
-        case '1-0':
-            $S = 1;
-            break;
-        case '1/2-1/2':
-        default: // should not occur
-            $S = 0;
-            break;
-        case '0-1':
-            $S = -1;
-            break;
-    }
+	switch ($pgn_result) {
+		case '1-0':
+			$S = 1;
+			break;
+		case '1/2-1/2':
+		default: // should not occur
+			$S = 0;
+			break;
+		case '0-1':
+			$S = -1;
+			break;
+	}
 
-    return [$white_rating + $S * 10, $black_rating - $S * 10];
+	return [$white_rating + $S * 10, $black_rating - $S * 10];
 }
 
 /**
@@ -72,5 +72,5 @@ 
  */
 function chess_ratings_num_provisional_games_linear()
 {
-    return 2;
+	return 2;
 }

include/common.php

Indentation +31 added lines, -31 removed lines

@@ -42,47 +42,47 @@ 
 $pathIcon16 = \Xmf\Module\Admin::iconUrl('', 16);
 $pathIcon32 = \Xmf\Module\Admin::iconUrl('', 32);
 if (is_object($helper->getModule())) {
-    $pathModIcon16 = $helper->getModule()->getInfo('modicons16');
+	$pathModIcon16 = $helper->getModule()->getInfo('modicons16');
 
-    $pathModIcon32 = $helper->getModule()->getInfo('modicons32');
+	$pathModIcon32 = $helper->getModule()->getInfo('modicons32');
 }
 
 if (!defined($moduleDirNameUpper . '_CONSTANTS_DEFINED')) {
-    define($moduleDirNameUpper . '_DIRNAME', basename(dirname(__DIR__)));
+	define($moduleDirNameUpper . '_DIRNAME', basename(dirname(__DIR__)));
 
-    define($moduleDirNameUpper . '_ROOT_PATH', XOOPS_ROOT_PATH . '/modules/' . $moduleDirName . '/');
+	define($moduleDirNameUpper . '_ROOT_PATH', XOOPS_ROOT_PATH . '/modules/' . $moduleDirName . '/');
 
-    define($moduleDirNameUpper . '_PATH', XOOPS_ROOT_PATH . '/modules/' . $moduleDirName . '/');
+	define($moduleDirNameUpper . '_PATH', XOOPS_ROOT_PATH . '/modules/' . $moduleDirName . '/');
 
-    define($moduleDirNameUpper . '_URL', XOOPS_URL . '/modules/' . $moduleDirName . '/');
+	define($moduleDirNameUpper . '_URL', XOOPS_URL . '/modules/' . $moduleDirName . '/');
 
-    define($moduleDirNameUpper . '_IMAGE_URL', constant($moduleDirNameUpper . '_URL') . '/assets/images/');
+	define($moduleDirNameUpper . '_IMAGE_URL', constant($moduleDirNameUpper . '_URL') . '/assets/images/');
 
-    define($moduleDirNameUpper . '_IMAGE_PATH', constant($moduleDirNameUpper . '_ROOT_PATH') . '/assets/images');
+	define($moduleDirNameUpper . '_IMAGE_PATH', constant($moduleDirNameUpper . '_ROOT_PATH') . '/assets/images');
 
-    define($moduleDirNameUpper . '_ADMIN_URL', constant($moduleDirNameUpper . '_URL') . '/admin/');
+	define($moduleDirNameUpper . '_ADMIN_URL', constant($moduleDirNameUpper . '_URL') . '/admin/');
 
-    define($moduleDirNameUpper . '_ADMIN_PATH', constant($moduleDirNameUpper . '_ROOT_PATH') . '/admin/');
+	define($moduleDirNameUpper . '_ADMIN_PATH', constant($moduleDirNameUpper . '_ROOT_PATH') . '/admin/');
 
-    define($moduleDirNameUpper . '_ADMIN', constant($moduleDirNameUpper . '_URL') . '/admin/index.php');
-    //    define($moduleDirNameUpper . '_AUTHOR_LOGOIMG', constant($moduleDirNameUpper . '_URL') . '/assets/images/logoModule.png');
-    define($moduleDirNameUpper . '_UPLOAD_URL', XOOPS_UPLOAD_URL . '/' . $moduleDirName); // WITHOUT Trailing slash
-    define($moduleDirNameUpper . '_UPLOAD_PATH', XOOPS_UPLOAD_PATH . '/' . $moduleDirName); // WITHOUT Trailing slash
-    define($moduleDirNameUpper . '_AUTHOR_LOGOIMG', $pathIcon32 . '/xoopsmicrobutton.gif');
+	define($moduleDirNameUpper . '_ADMIN', constant($moduleDirNameUpper . '_URL') . '/admin/index.php');
+	//    define($moduleDirNameUpper . '_AUTHOR_LOGOIMG', constant($moduleDirNameUpper . '_URL') . '/assets/images/logoModule.png');
+	define($moduleDirNameUpper . '_UPLOAD_URL', XOOPS_UPLOAD_URL . '/' . $moduleDirName); // WITHOUT Trailing slash
+	define($moduleDirNameUpper . '_UPLOAD_PATH', XOOPS_UPLOAD_PATH . '/' . $moduleDirName); // WITHOUT Trailing slash
+	define($moduleDirNameUpper . '_AUTHOR_LOGOIMG', $pathIcon32 . '/xoopsmicrobutton.gif');
 
-    define($moduleDirNameUpper . '_CONSTANTS_DEFINED', 1);
+	define($moduleDirNameUpper . '_CONSTANTS_DEFINED', 1);
 }
 
 $icons = [
-    'edit'    => "<img src='" . $pathIcon16 . "/edit.png'  alt=" . _EDIT . "' align='middle'>",
-    'delete'  => "<img src='" . $pathIcon16 . "/delete.png' alt='" . _DELETE . "' align='middle'>",
-    'clone'   => "<img src='" . $pathIcon16 . "/editcopy.png' alt='" . _CLONE . "' align='middle'>",
-    'preview' => "<img src='" . $pathIcon16 . "/view.png' alt='" . _PREVIEW . "' align='middle'>",
-    'print'   => "<img src='" . $pathIcon16 . "/printer.png' alt='" . _CLONE . "' align='middle'>",
-    'pdf'     => "<img src='" . $pathIcon16 . "/pdf.png' alt='" . _CLONE . "' align='middle'>",
-    'add'     => "<img src='" . $pathIcon16 . "/add.png' alt='" . _ADD . "' align='middle'>",
-    '0'       => "<img src='" . $pathIcon16 . "/0.png' alt='" . 0 . "' align='middle'>",
-    '1'       => "<img src='" . $pathIcon16 . "/1.png' alt='" . 1 . "' align='middle'>",
+	'edit'    => "<img src='" . $pathIcon16 . "/edit.png'  alt=" . _EDIT . "' align='middle'>",
+	'delete'  => "<img src='" . $pathIcon16 . "/delete.png' alt='" . _DELETE . "' align='middle'>",
+	'clone'   => "<img src='" . $pathIcon16 . "/editcopy.png' alt='" . _CLONE . "' align='middle'>",
+	'preview' => "<img src='" . $pathIcon16 . "/view.png' alt='" . _PREVIEW . "' align='middle'>",
+	'print'   => "<img src='" . $pathIcon16 . "/printer.png' alt='" . _CLONE . "' align='middle'>",
+	'pdf'     => "<img src='" . $pathIcon16 . "/pdf.png' alt='" . _CLONE . "' align='middle'>",
+	'add'     => "<img src='" . $pathIcon16 . "/add.png' alt='" . _ADD . "' align='middle'>",
+	'0'       => "<img src='" . $pathIcon16 . "/0.png' alt='" . 0 . "' align='middle'>",
+	'1'       => "<img src='" . $pathIcon16 . "/1.png' alt='" . 1 . "' align='middle'>",
 ];
 
 $debug = false;
@@ -91,19 +91,19 @@ 
 $myts = \MyTextSanitizer::getInstance();
 
 if (!isset($GLOBALS['xoopsTpl']) || !($GLOBALS['xoopsTpl'] instanceof \XoopsTpl)) {
-    require_once $GLOBALS['xoops']->path('class/template.php');
+	require_once $GLOBALS['xoops']->path('class/template.php');
 
-    $GLOBALS['xoopsTpl'] = new \XoopsTpl();
+	$GLOBALS['xoopsTpl'] = new \XoopsTpl();
 }
 
 $GLOBALS['xoopsTpl']->assign('mod_url', XOOPS_URL . '/modules/' . $moduleDirName);
 // Local icons path
 if (is_object($helper->getModule())) {
-    $pathModIcon16 = $helper->getModule()->getInfo('modicons16');
+	$pathModIcon16 = $helper->getModule()->getInfo('modicons16');
 
-    $pathModIcon32 = $helper->getModule()->getInfo('modicons32');
+	$pathModIcon32 = $helper->getModule()->getInfo('modicons32');
 
-    $GLOBALS['xoopsTpl']->assign('pathModIcon16', XOOPS_URL . '/modules/' . $moduleDirName . '/' . $pathModIcon16);
+	$GLOBALS['xoopsTpl']->assign('pathModIcon16', XOOPS_URL . '/modules/' . $moduleDirName . '/' . $pathModIcon16);
 
-    $GLOBALS['xoopsTpl']->assign('pathModIcon32', $pathModIcon32);
+	$GLOBALS['xoopsTpl']->assign('pathModIcon32', $pathModIcon32);
 }

Spacing +26 added lines, -26 removed lines

@@ -20,7 +20,7 @@ 
 
 use XoopsModules\Chess;
 
-include dirname(__DIR__) . '/preloads/autoloader.php';
+include dirname(__DIR__).'/preloads/autoloader.php';
 
 $moduleDirName      = basename(dirname(__DIR__));
 $moduleDirNameUpper = mb_strtoupper($moduleDirName); //$capsDirName
@@ -47,42 +47,42 @@ 
     $pathModIcon32 = $helper->getModule()->getInfo('modicons32');
 }
 
-if (!defined($moduleDirNameUpper . '_CONSTANTS_DEFINED')) {
-    define($moduleDirNameUpper . '_DIRNAME', basename(dirname(__DIR__)));
+if (!defined($moduleDirNameUpper.'_CONSTANTS_DEFINED')) {
+    define($moduleDirNameUpper.'_DIRNAME', basename(dirname(__DIR__)));
 
-    define($moduleDirNameUpper . '_ROOT_PATH', XOOPS_ROOT_PATH . '/modules/' . $moduleDirName . '/');
+    define($moduleDirNameUpper.'_ROOT_PATH', XOOPS_ROOT_PATH.'/modules/'.$moduleDirName.'/');
 
-    define($moduleDirNameUpper . '_PATH', XOOPS_ROOT_PATH . '/modules/' . $moduleDirName . '/');
+    define($moduleDirNameUpper.'_PATH', XOOPS_ROOT_PATH.'/modules/'.$moduleDirName.'/');
 
-    define($moduleDirNameUpper . '_URL', XOOPS_URL . '/modules/' . $moduleDirName . '/');
+    define($moduleDirNameUpper.'_URL', XOOPS_URL.'/modules/'.$moduleDirName.'/');
 
-    define($moduleDirNameUpper . '_IMAGE_URL', constant($moduleDirNameUpper . '_URL') . '/assets/images/');
+    define($moduleDirNameUpper.'_IMAGE_URL', constant($moduleDirNameUpper.'_URL').'/assets/images/');
 
-    define($moduleDirNameUpper . '_IMAGE_PATH', constant($moduleDirNameUpper . '_ROOT_PATH') . '/assets/images');
+    define($moduleDirNameUpper.'_IMAGE_PATH', constant($moduleDirNameUpper.'_ROOT_PATH').'/assets/images');
 
-    define($moduleDirNameUpper . '_ADMIN_URL', constant($moduleDirNameUpper . '_URL') . '/admin/');
+    define($moduleDirNameUpper.'_ADMIN_URL', constant($moduleDirNameUpper.'_URL').'/admin/');
 
-    define($moduleDirNameUpper . '_ADMIN_PATH', constant($moduleDirNameUpper . '_ROOT_PATH') . '/admin/');
+    define($moduleDirNameUpper.'_ADMIN_PATH', constant($moduleDirNameUpper.'_ROOT_PATH').'/admin/');
 
-    define($moduleDirNameUpper . '_ADMIN', constant($moduleDirNameUpper . '_URL') . '/admin/index.php');
+    define($moduleDirNameUpper.'_ADMIN', constant($moduleDirNameUpper.'_URL').'/admin/index.php');
     //    define($moduleDirNameUpper . '_AUTHOR_LOGOIMG', constant($moduleDirNameUpper . '_URL') . '/assets/images/logoModule.png');
-    define($moduleDirNameUpper . '_UPLOAD_URL', XOOPS_UPLOAD_URL . '/' . $moduleDirName); // WITHOUT Trailing slash
-    define($moduleDirNameUpper . '_UPLOAD_PATH', XOOPS_UPLOAD_PATH . '/' . $moduleDirName); // WITHOUT Trailing slash
-    define($moduleDirNameUpper . '_AUTHOR_LOGOIMG', $pathIcon32 . '/xoopsmicrobutton.gif');
+    define($moduleDirNameUpper.'_UPLOAD_URL', XOOPS_UPLOAD_URL.'/'.$moduleDirName); // WITHOUT Trailing slash
+    define($moduleDirNameUpper.'_UPLOAD_PATH', XOOPS_UPLOAD_PATH.'/'.$moduleDirName); // WITHOUT Trailing slash
+    define($moduleDirNameUpper.'_AUTHOR_LOGOIMG', $pathIcon32.'/xoopsmicrobutton.gif');
 
-    define($moduleDirNameUpper . '_CONSTANTS_DEFINED', 1);
+    define($moduleDirNameUpper.'_CONSTANTS_DEFINED', 1);
 }
 
 $icons = [
-    'edit'    => "<img src='" . $pathIcon16 . "/edit.png'  alt=" . _EDIT . "' align='middle'>",
-    'delete'  => "<img src='" . $pathIcon16 . "/delete.png' alt='" . _DELETE . "' align='middle'>",
-    'clone'   => "<img src='" . $pathIcon16 . "/editcopy.png' alt='" . _CLONE . "' align='middle'>",
-    'preview' => "<img src='" . $pathIcon16 . "/view.png' alt='" . _PREVIEW . "' align='middle'>",
-    'print'   => "<img src='" . $pathIcon16 . "/printer.png' alt='" . _CLONE . "' align='middle'>",
-    'pdf'     => "<img src='" . $pathIcon16 . "/pdf.png' alt='" . _CLONE . "' align='middle'>",
-    'add'     => "<img src='" . $pathIcon16 . "/add.png' alt='" . _ADD . "' align='middle'>",
-    '0'       => "<img src='" . $pathIcon16 . "/0.png' alt='" . 0 . "' align='middle'>",
-    '1'       => "<img src='" . $pathIcon16 . "/1.png' alt='" . 1 . "' align='middle'>",
+    'edit'    => "<img src='".$pathIcon16."/edit.png'  alt="._EDIT."' align='middle'>",
+    'delete'  => "<img src='".$pathIcon16."/delete.png' alt='"._DELETE."' align='middle'>",
+    'clone'   => "<img src='".$pathIcon16."/editcopy.png' alt='"._CLONE."' align='middle'>",
+    'preview' => "<img src='".$pathIcon16."/view.png' alt='"._PREVIEW."' align='middle'>",
+    'print'   => "<img src='".$pathIcon16."/printer.png' alt='"._CLONE."' align='middle'>",
+    'pdf'     => "<img src='".$pathIcon16."/pdf.png' alt='"._CLONE."' align='middle'>",
+    'add'     => "<img src='".$pathIcon16."/add.png' alt='"._ADD."' align='middle'>",
+    '0'       => "<img src='".$pathIcon16."/0.png' alt='".0."' align='middle'>",
+    '1'       => "<img src='".$pathIcon16."/1.png' alt='".1."' align='middle'>",
 ];
 
 $debug = false;
@@ -96,14 +96,14 @@ 
     $GLOBALS['xoopsTpl'] = new \XoopsTpl();
 }
 
-$GLOBALS['xoopsTpl']->assign('mod_url', XOOPS_URL . '/modules/' . $moduleDirName);
+$GLOBALS['xoopsTpl']->assign('mod_url', XOOPS_URL.'/modules/'.$moduleDirName);
 // Local icons path
 if (is_object($helper->getModule())) {
     $pathModIcon16 = $helper->getModule()->getInfo('modicons16');
 
     $pathModIcon32 = $helper->getModule()->getInfo('modicons32');
 
-    $GLOBALS['xoopsTpl']->assign('pathModIcon16', XOOPS_URL . '/modules/' . $moduleDirName . '/' . $pathModIcon16);
+    $GLOBALS['xoopsTpl']->assign('pathModIcon16', XOOPS_URL.'/modules/'.$moduleDirName.'/'.$pathModIcon16);
 
     $GLOBALS['xoopsTpl']->assign('pathModIcon32', $pathModIcon32);
 }

include/install.inc.php

Spacing +5 added lines, -5 removed lines

@@ -56,8 +56,8 @@ 
 
     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);
+        $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.
 
@@ -76,7 +76,7 @@ 
     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);
+        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) {
@@ -104,7 +104,7 @@ 
     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);
+        chess_set_message($module, sprintf(_MI_CHESS_GAMES_TABLE_2, $games_table, (string) $mysql_errno, $mysql_error), true);
         return false;
     }
     [$count] = $xoopsDB->fetchRow($result);
@@ -260,7 +260,7 @@ 
 function chess_load_lang_file($filename, $module = '', $default = 'english')
 {
     $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");
     }

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";
+	}
 }
 
 ?>

config/icons.php

Indentation +12 added lines, -12 removed lines

@@ -4,16 +4,16 @@
 $moduleDirName = basename(dirname(__DIR__));
 
 return (object)[
-    'name'  => mb_strtoupper($moduleDirName) . ' IconConfigurator',
-    'icons' => [
-        'edit'    => "<img src='" . $pathIcon16 . "/edit.png'  alt=" . _EDIT . "' align='middle'>",
-        'delete'  => "<img src='" . $pathIcon16 . "/delete.png' alt='" . _DELETE . "' align='middle'>",
-        'clone'   => "<img src='" . $pathIcon16 . "/editcopy.png' alt='" . _CLONE . "' align='middle'>",
-        'preview' => "<img src='" . $pathIcon16 . "/view.png' alt='" . _PREVIEW . "' align='middle'>",
-        'print'   => "<img src='" . $pathIcon16 . "/printer.png' alt='" . _CLONE . "' align='middle'>",
-        'pdf'     => "<img src='" . $pathIcon16 . "/pdf.png' alt='" . _CLONE . "' align='middle'>",
-        'add'     => "<img src='" . $pathIcon16 . "/add.png' alt='" . _ADD . "' align='middle'>",
-        '0'       => "<img src='" . $pathIcon16 . "/0.png' alt='" . 0 . "' align='middle'>",
-        '1'       => "<img src='" . $pathIcon16 . "/1.png' alt='" . 1 . "' align='middle'>",
-    ],
+	'name'  => mb_strtoupper($moduleDirName) . ' IconConfigurator',
+	'icons' => [
+		'edit'    => "<img src='" . $pathIcon16 . "/edit.png'  alt=" . _EDIT . "' align='middle'>",
+		'delete'  => "<img src='" . $pathIcon16 . "/delete.png' alt='" . _DELETE . "' align='middle'>",
+		'clone'   => "<img src='" . $pathIcon16 . "/editcopy.png' alt='" . _CLONE . "' align='middle'>",
+		'preview' => "<img src='" . $pathIcon16 . "/view.png' alt='" . _PREVIEW . "' align='middle'>",
+		'print'   => "<img src='" . $pathIcon16 . "/printer.png' alt='" . _CLONE . "' align='middle'>",
+		'pdf'     => "<img src='" . $pathIcon16 . "/pdf.png' alt='" . _CLONE . "' align='middle'>",
+		'add'     => "<img src='" . $pathIcon16 . "/add.png' alt='" . _ADD . "' align='middle'>",
+		'0'       => "<img src='" . $pathIcon16 . "/0.png' alt='" . 0 . "' align='middle'>",
+		'1'       => "<img src='" . $pathIcon16 . "/1.png' alt='" . 1 . "' align='middle'>",
+	],
 ];

Spacing +11 added lines, -11 removed lines

@@ -3,17 +3,17 @@
 $pathIcon16    = \Xmf\Module\Admin::iconUrl('', 16);
 $moduleDirName = basename(dirname(__DIR__));
 
-return (object)[
-    'name'  => mb_strtoupper($moduleDirName) . ' IconConfigurator',
+return (object) [
+    'name'  => mb_strtoupper($moduleDirName).' IconConfigurator',
     'icons' => [
-        'edit'    => "<img src='" . $pathIcon16 . "/edit.png'  alt=" . _EDIT . "' align='middle'>",
-        'delete'  => "<img src='" . $pathIcon16 . "/delete.png' alt='" . _DELETE . "' align='middle'>",
-        'clone'   => "<img src='" . $pathIcon16 . "/editcopy.png' alt='" . _CLONE . "' align='middle'>",
-        'preview' => "<img src='" . $pathIcon16 . "/view.png' alt='" . _PREVIEW . "' align='middle'>",
-        'print'   => "<img src='" . $pathIcon16 . "/printer.png' alt='" . _CLONE . "' align='middle'>",
-        'pdf'     => "<img src='" . $pathIcon16 . "/pdf.png' alt='" . _CLONE . "' align='middle'>",
-        'add'     => "<img src='" . $pathIcon16 . "/add.png' alt='" . _ADD . "' align='middle'>",
-        '0'       => "<img src='" . $pathIcon16 . "/0.png' alt='" . 0 . "' align='middle'>",
-        '1'       => "<img src='" . $pathIcon16 . "/1.png' alt='" . 1 . "' align='middle'>",
+        'edit'    => "<img src='".$pathIcon16."/edit.png'  alt="._EDIT."' align='middle'>",
+        'delete'  => "<img src='".$pathIcon16."/delete.png' alt='"._DELETE."' align='middle'>",
+        'clone'   => "<img src='".$pathIcon16."/editcopy.png' alt='"._CLONE."' align='middle'>",
+        'preview' => "<img src='".$pathIcon16."/view.png' alt='"._PREVIEW."' align='middle'>",
+        'print'   => "<img src='".$pathIcon16."/printer.png' alt='"._CLONE."' align='middle'>",
+        'pdf'     => "<img src='".$pathIcon16."/pdf.png' alt='"._CLONE."' align='middle'>",
+        'add'     => "<img src='".$pathIcon16."/add.png' alt='"._ADD."' align='middle'>",
+        '0'       => "<img src='".$pathIcon16."/0.png' alt='".0."' align='middle'>",
+        '1'       => "<img src='".$pathIcon16."/1.png' alt='".1."' align='middle'>",
     ],
 ];

admin/index.php

Indentation +28 added lines, -28 removed lines

@@ -42,31 +42,31 @@ 
 //------------- Test Data ----------------------------
 
 if ($helper->getConfig('displaySampleButton')) {
-    $yamlFile = dirname(__DIR__) . '/config/admin.yml';
+	$yamlFile = dirname(__DIR__) . '/config/admin.yml';
 
-    $config = loadAdminConfig($yamlFile);
+	$config = loadAdminConfig($yamlFile);
 
-    $displaySampleButton = $config['displaySampleButton'];
+	$displaySampleButton = $config['displaySampleButton'];
 
-    if (1 == $displaySampleButton) {
-        xoops_loadLanguage('admin/modulesadmin', 'system');
+	if (1 == $displaySampleButton) {
+		xoops_loadLanguage('admin/modulesadmin', 'system');
 
-        require_once dirname(__DIR__) . '/testdata/index.php';
+		require_once dirname(__DIR__) . '/testdata/index.php';
 
-        $adminObject->addItemButton(constant('CO_' . $moduleDirNameUpper . '_' . 'ADD_SAMPLEDATA'), '__DIR__ . /../../testdata/index.php?op=load', 'add');
+		$adminObject->addItemButton(constant('CO_' . $moduleDirNameUpper . '_' . 'ADD_SAMPLEDATA'), '__DIR__ . /../../testdata/index.php?op=load', 'add');
 
-        $adminObject->addItemButton(constant('CO_' . $moduleDirNameUpper . '_' . 'SAVE_SAMPLEDATA'), '__DIR__ . /../../testdata/index.php?op=save', 'add');
+		$adminObject->addItemButton(constant('CO_' . $moduleDirNameUpper . '_' . 'SAVE_SAMPLEDATA'), '__DIR__ . /../../testdata/index.php?op=save', 'add');
 
-        //    $adminObject->addItemButton(constant('CO_' . $moduleDirNameUpper . '_' . 'EXPORT_SCHEMA'), '__DIR__ . /../../testdata/index.php?op=exportschema', 'add');
+		//    $adminObject->addItemButton(constant('CO_' . $moduleDirNameUpper . '_' . 'EXPORT_SCHEMA'), '__DIR__ . /../../testdata/index.php?op=exportschema', 'add');
 
-        $adminObject->addItemButton(constant('CO_' . $moduleDirNameUpper . '_' . 'HIDE_SAMPLEDATA_BUTTONS'), '?op=hide_buttons', 'delete');
-    } else {
-        $adminObject->addItemButton(constant('CO_' . $moduleDirNameUpper . '_' . 'SHOW_SAMPLEDATA_BUTTONS'), '?op=show_buttons', 'add');
+		$adminObject->addItemButton(constant('CO_' . $moduleDirNameUpper . '_' . 'HIDE_SAMPLEDATA_BUTTONS'), '?op=hide_buttons', 'delete');
+	} else {
+		$adminObject->addItemButton(constant('CO_' . $moduleDirNameUpper . '_' . 'SHOW_SAMPLEDATA_BUTTONS'), '?op=show_buttons', 'add');
 
-        $displaySampleButton = $config['displaySampleButton'];
-    }
+		$displaySampleButton = $config['displaySampleButton'];
+	}
 
-    $adminObject->displayButton('left', '');
+	$adminObject->displayButton('left', '');
 }
 
 //------------- End Test Data ----------------------------
@@ -79,7 +79,7 @@ 
  */
 function loadAdminConfig($yamlFile)
 {
-    return \Xmf\Yaml::readWrapped($yamlFile);
+	return \Xmf\Yaml::readWrapped($yamlFile);
 }
 
 /**
@@ -87,11 +87,11 @@ 
  */
 function hideButtons($yamlFile)
 {
-    $app['displaySampleButton'] = 0;
+	$app['displaySampleButton'] = 0;
 
-    \Xmf\Yaml::save($app, $yamlFile);
+	\Xmf\Yaml::save($app, $yamlFile);
 
-    redirect_header('index.php', 0, '');
+	redirect_header('index.php', 0, '');
 }
 
 /**
@@ -99,22 +99,22 @@ 
  */
 function showButtons($yamlFile)
 {
-    $app['displaySampleButton'] = 1;
+	$app['displaySampleButton'] = 1;
 
-    \Xmf\Yaml::save($app, $yamlFile);
+	\Xmf\Yaml::save($app, $yamlFile);
 
-    redirect_header('index.php', 0, '');
+	redirect_header('index.php', 0, '');
 }
 
 $op = \Xmf\Request::getString('op', 0, 'GET');
 
 switch ($op) {
-    case 'hide_buttons':
-        hideButtons($yamlFile);
-        break;
-    case 'show_buttons':
-        showButtons($yamlFile);
-        break;
+	case 'hide_buttons':
+		hideButtons($yamlFile);
+		break;
+	case 'show_buttons':
+		showButtons($yamlFile);
+		break;
 }
 
 echo $utility::getServerStats();

Spacing +8 added lines, -8 removed lines

@@ -19,7 +19,7 @@ 
 
 use XoopsModules\Chess\Common;
 
-require __DIR__ . '/admin_header.php';
+require __DIR__.'/admin_header.php';
 // Display Admin header
 xoops_cp_header();
 $adminObject = \Xmf\Module\Admin::getInstance();
@@ -42,7 +42,7 @@ 
 //------------- Test Data ----------------------------
 
 if ($helper->getConfig('displaySampleButton')) {
-    $yamlFile = dirname(__DIR__) . '/config/admin.yml';
+    $yamlFile = dirname(__DIR__).'/config/admin.yml';
 
     $config = loadAdminConfig($yamlFile);
 
@@ -51,17 +51,17 @@ 
     if (1 == $displaySampleButton) {
         xoops_loadLanguage('admin/modulesadmin', 'system');
 
-        require_once dirname(__DIR__) . '/testdata/index.php';
+        require_once dirname(__DIR__).'/testdata/index.php';
 
-        $adminObject->addItemButton(constant('CO_' . $moduleDirNameUpper . '_' . 'ADD_SAMPLEDATA'), '__DIR__ . /../../testdata/index.php?op=load', 'add');
+        $adminObject->addItemButton(constant('CO_'.$moduleDirNameUpper.'_'.'ADD_SAMPLEDATA'), '__DIR__ . /../../testdata/index.php?op=load', 'add');
 
-        $adminObject->addItemButton(constant('CO_' . $moduleDirNameUpper . '_' . 'SAVE_SAMPLEDATA'), '__DIR__ . /../../testdata/index.php?op=save', 'add');
+        $adminObject->addItemButton(constant('CO_'.$moduleDirNameUpper.'_'.'SAVE_SAMPLEDATA'), '__DIR__ . /../../testdata/index.php?op=save', 'add');
 
         //    $adminObject->addItemButton(constant('CO_' . $moduleDirNameUpper . '_' . 'EXPORT_SCHEMA'), '__DIR__ . /../../testdata/index.php?op=exportschema', 'add');
 
-        $adminObject->addItemButton(constant('CO_' . $moduleDirNameUpper . '_' . 'HIDE_SAMPLEDATA_BUTTONS'), '?op=hide_buttons', 'delete');
+        $adminObject->addItemButton(constant('CO_'.$moduleDirNameUpper.'_'.'HIDE_SAMPLEDATA_BUTTONS'), '?op=hide_buttons', 'delete');
     } else {
-        $adminObject->addItemButton(constant('CO_' . $moduleDirNameUpper . '_' . 'SHOW_SAMPLEDATA_BUTTONS'), '?op=show_buttons', 'add');
+        $adminObject->addItemButton(constant('CO_'.$moduleDirNameUpper.'_'.'SHOW_SAMPLEDATA_BUTTONS'), '?op=show_buttons', 'add');
 
         $displaySampleButton = $config['displaySampleButton'];
     }
@@ -119,4 +119,4 @@ 
 
 echo $utility::getServerStats();
 
-require __DIR__ . '/admin_footer.php';
+require __DIR__.'/admin_footer.php';

admin/menu.php

Indentation +22 added lines, -22 removed lines

@@ -34,45 +34,45 @@
 
 $pathIcon32 = \Xmf\Module\Admin::menuIconPath('');
 if (is_object($helper->getModule())) {
-    //    $pathModIcon32 = $helper->getModule()->getInfo('modicons32');
+	//    $pathModIcon32 = $helper->getModule()->getInfo('modicons32');
 
-    $pathModIcon32 = $helper->url($helper->getModule()->getInfo('modicons32'));
+	$pathModIcon32 = $helper->url($helper->getModule()->getInfo('modicons32'));
 }
 
 $adminmenu[] = [
-    'title' => _MI_CHESS_MENU_HOME,
-    'link'  => 'admin/index.php',
-    'icon'  => $pathIcon32 . '/home.png',
+	'title' => _MI_CHESS_MENU_HOME,
+	'link'  => 'admin/index.php',
+	'icon'  => $pathIcon32 . '/home.png',
 ];
 
 $adminmenu[] = [
-    'title' => _MI_CHESS_ADMENU1,
-    'link'  => 'admin/main.php?op=suspended_games',
-    'icon'  => $pathIcon32 . '/alert.png',
+	'title' => _MI_CHESS_ADMENU1,
+	'link'  => 'admin/main.php?op=suspended_games',
+	'icon'  => $pathIcon32 . '/alert.png',
 ];
 
 $adminmenu[] = [
-    'title' => _MI_CHESS_ADMENU2,
-    'link'  => 'admin/main.php?op=active_games',
-    'icon'  => $pathIcon32 . '/button_ok.png',
+	'title' => _MI_CHESS_ADMENU2,
+	'link'  => 'admin/main.php?op=active_games',
+	'icon'  => $pathIcon32 . '/button_ok.png',
 ];
 
 $adminmenu[] = [
-    'title' => _MI_CHESS_ADMENU3,
-    'link'  => 'admin/main.php?op=challenges',
-    'icon'  => $pathIcon32 . '/cash_stack.png',
+	'title' => _MI_CHESS_ADMENU3,
+	'link'  => 'admin/main.php?op=challenges',
+	'icon'  => $pathIcon32 . '/cash_stack.png',
 ];
 
 if (is_object($helper->getModule()) && $helper->getConfig('displayDeveloperTools')) {
-    $adminmenu[] = [
-        'title' => constant('CO_' . $moduleDirNameUpper . '_' . 'ADMENU_MIGRATE'),
-        'link'  => 'admin/migrate.php',
-        'icon'  => $pathIcon32 . '/database_go.png',
-    ];
+	$adminmenu[] = [
+		'title' => constant('CO_' . $moduleDirNameUpper . '_' . 'ADMENU_MIGRATE'),
+		'link'  => 'admin/migrate.php',
+		'icon'  => $pathIcon32 . '/database_go.png',
+	];
 }
 
 $adminmenu[] = [
-    'title' => _MI_CHESS_MENU_ABOUT,
-    'link'  => 'admin/about.php',
-    'icon'  => $pathIcon32 . '/about.png',
+	'title' => _MI_CHESS_MENU_ABOUT,
+	'link'  => 'admin/about.php',
+	'icon'  => $pathIcon32 . '/about.png',
 ];

Spacing +8 added lines, -8 removed lines

@@ -25,7 +25,7 @@ 
 //  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307 USA //
 //  ------------------------------------------------------------------------ //
 
-include dirname(__DIR__) . '/preloads/autoloader.php';
+include dirname(__DIR__).'/preloads/autoloader.php';
 
 $moduleDirName      = basename(dirname(__DIR__));
 $moduleDirNameUpper = mb_strtoupper($moduleDirName);
@@ -42,37 +42,37 @@ 
 $adminmenu[] = [
     'title' => _MI_CHESS_MENU_HOME,
     'link'  => 'admin/index.php',
-    'icon'  => $pathIcon32 . '/home.png',
+    'icon'  => $pathIcon32.'/home.png',
 ];
 
 $adminmenu[] = [
     'title' => _MI_CHESS_ADMENU1,
     'link'  => 'admin/main.php?op=suspended_games',
-    'icon'  => $pathIcon32 . '/alert.png',
+    'icon'  => $pathIcon32.'/alert.png',
 ];
 
 $adminmenu[] = [
     'title' => _MI_CHESS_ADMENU2,
     'link'  => 'admin/main.php?op=active_games',
-    'icon'  => $pathIcon32 . '/button_ok.png',
+    'icon'  => $pathIcon32.'/button_ok.png',
 ];
 
 $adminmenu[] = [
     'title' => _MI_CHESS_ADMENU3,
     'link'  => 'admin/main.php?op=challenges',
-    'icon'  => $pathIcon32 . '/cash_stack.png',
+    'icon'  => $pathIcon32.'/cash_stack.png',
 ];
 
 if (is_object($helper->getModule()) && $helper->getConfig('displayDeveloperTools')) {
     $adminmenu[] = [
-        'title' => constant('CO_' . $moduleDirNameUpper . '_' . 'ADMENU_MIGRATE'),
+        'title' => constant('CO_'.$moduleDirNameUpper.'_'.'ADMENU_MIGRATE'),
         'link'  => 'admin/migrate.php',
-        'icon'  => $pathIcon32 . '/database_go.png',
+        'icon'  => $pathIcon32.'/database_go.png',
     ];
 }
 
 $adminmenu[] = [
     'title' => _MI_CHESS_MENU_ABOUT,
     'link'  => 'admin/about.php',
-    'icon'  => $pathIcon32 . '/about.png',
+    'icon'  => $pathIcon32.'/about.png',
 ];

admin/admin_footer.php

Spacing +1 added lines, -1 removed lines

@@ -19,6 +19,6 @@
  */
 $pathIcon32 = Xmf\Module\Admin::iconUrl('', 32);
 
-echo "<div class='adminfooter'>\n" . "  <div style='text-align: center;'>\n" . "    <a href='https://xoops.org' rel='external'><img src='{$pathIcon32}/xoopsmicrobutton.gif' alt='XOOPS' title='XOOPS'></a>\n" . "  </div>\n" . '  ' . _AM_MODULEADMIN_ADMIN_FOOTER . "\n" . '</div>';
+echo "<div class='adminfooter'>\n"."  <div style='text-align: center;'>\n"."    <a href='https://xoops.org' rel='external'><img src='{$pathIcon32}/xoopsmicrobutton.gif' alt='XOOPS' title='XOOPS'></a>\n"."  </div>\n".'  '._AM_MODULEADMIN_ADMIN_FOOTER."\n".'</div>';
 
 xoops_cp_footer();

class/Helper.php

Indentation +51 added lines, -51 removed lines

@@ -26,72 +26,72 @@
  */
 class Helper extends \Xmf\Module\Helper
 {
-    public $debug;
+	public $debug;
 
-    /**
-     * @param bool $debug
-     */
-    public function __construct($debug = false)
-    {
-        $this->debug = $debug;
+	/**
+	 * @param bool $debug
+	 */
+	public function __construct($debug = false)
+	{
+		$this->debug = $debug;
 
-        $moduleDirName = \basename(\dirname(__DIR__));
+		$moduleDirName = \basename(\dirname(__DIR__));
 
-        parent::__construct($moduleDirName);
-    }
+		parent::__construct($moduleDirName);
+	}
 
-    /**
-     * @param bool $debug
-     *
-     * @return \XoopsModules\Chess\Helper
-     */
-    public static function getInstance($debug = false)
-    {
-        static $instance;
+	/**
+	 * @param bool $debug
+	 *
+	 * @return \XoopsModules\Chess\Helper
+	 */
+	public static function getInstance($debug = false)
+	{
+		static $instance;
 
-        if (null === $instance) {
-            $instance = new static($debug);
-        }
+		if (null === $instance) {
+			$instance = new static($debug);
+		}
 
-        return $instance;
-    }
+		return $instance;
+	}
 
-    /**
-     * @return string
-     */
-    public function getDirname()
-    {
-        return $this->dirname;
-    }
+	/**
+	 * @return string
+	 */
+	public function getDirname()
+	{
+		return $this->dirname;
+	}
 
-    /**
-     * Get an Object Handler
-     *
-     * @param string $name name of handler to load
-     *
-     * @return bool|\XoopsObjectHandler|\XoopsPersistableObjectHandler
-     */
-    public function getHandler($name)
-    {
-        $ret = false;
+	/**
+	 * Get an Object Handler
+	 *
+	 * @param string $name name of handler to load
+	 *
+	 * @return bool|\XoopsObjectHandler|\XoopsPersistableObjectHandler
+	 */
+	public function getHandler($name)
+	{
+		$ret = false;
 
-        $class = __NAMESPACE__ . '\\' . $name . 'Handler';
+		$class = __NAMESPACE__ . '\\' . $name . 'Handler';
 
-        if (!\class_exists($class)) {
-            throw new \RuntimeException("Class '$class' not found");
-        }
+		if (!\class_exists($class)) {
+			throw new \RuntimeException("Class '$class' not found");
+		}
 
-        /** @var \XoopsMySQLDatabase $db */
+		/** @var \XoopsMySQLDatabase $db */
 
-        $db = \XoopsDatabaseFactory::getDatabaseConnection();
+		$db = \XoopsDatabaseFactory::getDatabaseConnection();
 
-        $helper = self::getInstance();
+		$helper = self::getInstance();
 
-        $ret = new $class($db, $helper);
+		$ret = new $class($db, $helper);
 
-        $this->addLog("Getting handler '{$name}'");
+		$this->addLog("Getting handler '{$name}'");
 
-        return $ret;
-    }
+		return $ret;
+	}
 }
 //require  dirname(dirname(__DIR__)) . '/mainfile.php';

Spacing +1 added lines, -1 removed lines

@@ -75,7 +75,7 @@
     {
         $ret = false;
 
-        $class = __NAMESPACE__ . '\\' . $name . 'Handler';
+        $class = __NAMESPACE__.'\\'.$name.'Handler';
 
         if (!\class_exists($class)) {
             throw new \RuntimeException("Class '$class' not found");

class/Utility.php

Indentation +1 added lines, -1 removed lines

@@ -27,5 +27,5 @@
  */
 class Utility extends Common\SysUtility
 {
-    //--------------- Custom module methods -----------------------------
+	//--------------- Custom module methods -----------------------------
 }