mambax7 / chess

blocks/blocks.php

Spacing +22 added lines, -22 removed lines

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

index.php

Spacing +13 added lines, -13 removed lines

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

testdata/index.php

Spacing +16 added lines, -16 removed lines

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

comment_edit.php

Spacing +2 added lines, -2 removed lines

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

help.php

Spacing +7 added lines, -7 removed lines

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

docs/maketestdata.php

Spacing +2 added lines, -2 removed lines

@@ -36,9 +36,9 @@
 
     $ratings_table = 'chess_ratings';
 
-    mysqli_connect(DBHOST, DBUSER, DBPASS) or trigger_error('[' . $GLOBALS['xoopsDB']->errno() . '] ' . $GLOBALS['xoopsDB']->error(), E_USER_ERROR);
+    mysqli_connect(DBHOST, DBUSER, DBPASS) or trigger_error('['.$GLOBALS['xoopsDB']->errno().'] '.$GLOBALS['xoopsDB']->error(), E_USER_ERROR);
 
-    mysqli_select_db($GLOBALS['xoopsDB']->conn, DBNAME) or trigger_error('[' . $GLOBALS['xoopsDB']->errno() . '] ' . $GLOBALS['xoopsDB']->error(), E_USER_ERROR);
+    mysqli_select_db($GLOBALS['xoopsDB']->conn, DBNAME) or trigger_error('['.$GLOBALS['xoopsDB']->errno().'] '.$GLOBALS['xoopsDB']->error(), E_USER_ERROR);
 
     // For safety, don't generate test data unless the tables are empty.
 

comment_new.php

Spacing +2 added lines, -2 removed lines

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

notification_update.php

Spacing +2 added lines, -2 removed lines

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

language/english/filechecker.php

Spacing +10 added lines, -10 removed lines

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