ggoffy / wggithub

admin/permissions.php

Spacing +3 added lines, -3 removed lines

@@ -24,7 +24,7 @@ 
 use XoopsModules\Wggithub;
 use XoopsModules\Wggithub\Constants;
 
-require __DIR__ . '/header.php';
+require __DIR__.'/header.php';
 
 // Template Index
 $templateMain = 'wggithub_admin_permissions.tpl';
@@ -33,7 +33,7 @@ 
 $op = Request::getCmd('op', 'global');
 
 // Get Form
-include_once XOOPS_ROOT_PATH . '/class/xoopsform/grouppermform.php';
+include_once XOOPS_ROOT_PATH.'/class/xoopsform/grouppermform.php';
 \xoops_load('XoopsFormLoader');
 
 $formTitle = \_AM_WGGITHUB_PERMISSIONS_GLOBAL;
@@ -61,4 +61,4 @@ 
     exit();
 }
 
-require __DIR__ . '/footer.php';
+require __DIR__.'/footer.php';

admin/directories.php

Indentation +103 added lines, -103 removed lines

@@ -31,109 +31,109 @@
 // Request dir_id
 $dirId = Request::getInt('dir_id');
 switch ($op) {
-	case 'list':
-	default:
-		// Define Stylesheet
-		$GLOBALS['xoTheme']->addStylesheet($style, null);
-		$start = Request::getInt('start', 0);
-		$limit = Request::getInt('limit', $helper->getConfig('adminpager'));
-		$templateMain = 'wggithub_admin_directories.tpl';
-		$GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('directories.php'));
-		$adminObject->addItemButton(_AM_WGGITHUB_ADD_DIRECTORY, 'directories.php?op=new', 'add');
-		$GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
-		$directoriesCount = $directoriesHandler->getCountDirectories();
-		$directoriesAll = $directoriesHandler->getAllDirectories($start, $limit);
-		$GLOBALS['xoopsTpl']->assign('directories_count', $directoriesCount);
-		$GLOBALS['xoopsTpl']->assign('wggithub_url', WGGITHUB_URL);
-		$GLOBALS['xoopsTpl']->assign('wggithub_upload_url', WGGITHUB_UPLOAD_URL);
-		// Table view directories
-		if ($directoriesCount > 0) {
-			foreach (\array_keys($directoriesAll) as $i) {
-				$directory = $directoriesAll[$i]->getValuesDirectories();
-				$GLOBALS['xoopsTpl']->append('directories_list', $directory);
-				unset($directory);
-			}
-			// Display Navigation
-			if ($directoriesCount > $limit) {
-				include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
-				$pagenav = new \XoopsPageNav($directoriesCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
-				$GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
-			}
-		} else {
-			$GLOBALS['xoopsTpl']->assign('error', _AM_WGGITHUB_THEREARENT_DIRECTORIES);
-		}
-		break;
-	case 'new':
-		$templateMain = 'wggithub_admin_directories.tpl';
-		$GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('directories.php'));
-		$adminObject->addItemButton(_AM_WGGITHUB_DIRECTORIES_LIST, 'directories.php', 'list');
-		$GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
-		// Form Create
-		$directoriesObj = $directoriesHandler->create();
-		$form = $directoriesObj->getFormDirectories();
-		$GLOBALS['xoopsTpl']->assign('form', $form->render());
-		break;
-	case 'save':
-		// Security Check
-		if (!$GLOBALS['xoopsSecurity']->check()) {
-			\redirect_header('directories.php', 3, \implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
-		}
-		if ($dirId > 0) {
-			$directoriesObj = $directoriesHandler->get($dirId);
-		} else {
-			$directoriesObj = $directoriesHandler->create();
-		}
-		// Set Vars
-		$directoriesObj->setVar('dir_name', Request::getString('dir_name', ''));
-		$directoriesObj->setVar('dir_type', Request::getInt('dir_type', 0));
-		$directoriesObj->setVar('dir_autoupdate', Request::getInt('dir_autoupdate', 0));
-		$directoriesObj->setVar('dir_online', Request::getInt('dir_online', 0));
+    case 'list':
+    default:
+        // Define Stylesheet
+        $GLOBALS['xoTheme']->addStylesheet($style, null);
+        $start = Request::getInt('start', 0);
+        $limit = Request::getInt('limit', $helper->getConfig('adminpager'));
+        $templateMain = 'wggithub_admin_directories.tpl';
+        $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('directories.php'));
+        $adminObject->addItemButton(_AM_WGGITHUB_ADD_DIRECTORY, 'directories.php?op=new', 'add');
+        $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
+        $directoriesCount = $directoriesHandler->getCountDirectories();
+        $directoriesAll = $directoriesHandler->getAllDirectories($start, $limit);
+        $GLOBALS['xoopsTpl']->assign('directories_count', $directoriesCount);
+        $GLOBALS['xoopsTpl']->assign('wggithub_url', WGGITHUB_URL);
+        $GLOBALS['xoopsTpl']->assign('wggithub_upload_url', WGGITHUB_UPLOAD_URL);
+        // Table view directories
+        if ($directoriesCount > 0) {
+            foreach (\array_keys($directoriesAll) as $i) {
+                $directory = $directoriesAll[$i]->getValuesDirectories();
+                $GLOBALS['xoopsTpl']->append('directories_list', $directory);
+                unset($directory);
+            }
+            // Display Navigation
+            if ($directoriesCount > $limit) {
+                include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
+                $pagenav = new \XoopsPageNav($directoriesCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
+                $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
+            }
+        } else {
+            $GLOBALS['xoopsTpl']->assign('error', _AM_WGGITHUB_THEREARENT_DIRECTORIES);
+        }
+        break;
+    case 'new':
+        $templateMain = 'wggithub_admin_directories.tpl';
+        $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('directories.php'));
+        $adminObject->addItemButton(_AM_WGGITHUB_DIRECTORIES_LIST, 'directories.php', 'list');
+        $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
+        // Form Create
+        $directoriesObj = $directoriesHandler->create();
+        $form = $directoriesObj->getFormDirectories();
+        $GLOBALS['xoopsTpl']->assign('form', $form->render());
+        break;
+    case 'save':
+        // Security Check
+        if (!$GLOBALS['xoopsSecurity']->check()) {
+            \redirect_header('directories.php', 3, \implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
+        }
+        if ($dirId > 0) {
+            $directoriesObj = $directoriesHandler->get($dirId);
+        } else {
+            $directoriesObj = $directoriesHandler->create();
+        }
+        // Set Vars
+        $directoriesObj->setVar('dir_name', Request::getString('dir_name', ''));
+        $directoriesObj->setVar('dir_type', Request::getInt('dir_type', 0));
+        $directoriesObj->setVar('dir_autoupdate', Request::getInt('dir_autoupdate', 0));
+        $directoriesObj->setVar('dir_online', Request::getInt('dir_online', 0));
         $directoriesObj->setVar('dir_filterrelease', Request::getInt('dir_filterrelease', 0));
-		$directoryDatecreatedObj = \DateTime::createFromFormat(_SHORTDATESTRING, Request::getString('dir_datecreated'));
-		$directoriesObj->setVar('dir_datecreated', $directoryDatecreatedObj->getTimestamp());
-		$directoriesObj->setVar('dir_submitter', Request::getInt('dir_submitter', 0));
-		// Insert Data
-		if ($directoriesHandler->insert($directoriesObj)) {
-			\redirect_header('directories.php?op=list', 2, _AM_WGGITHUB_FORM_OK);
-		}
-		// Get Form
-		$GLOBALS['xoopsTpl']->assign('error', $directoriesObj->getHtmlErrors());
-		$form = $directoriesObj->getFormDirectories();
-		$GLOBALS['xoopsTpl']->assign('form', $form->render());
-		break;
-	case 'edit':
-		$templateMain = 'wggithub_admin_directories.tpl';
-		$GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('directories.php'));
-		$adminObject->addItemButton(_AM_WGGITHUB_ADD_DIRECTORY, 'directories.php?op=new', 'add');
-		$adminObject->addItemButton(_AM_WGGITHUB_DIRECTORIES_LIST, 'directories.php', 'list');
-		$GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
-		// Get Form
-		$directoriesObj = $directoriesHandler->get($dirId);
-		$form = $directoriesObj->getFormDirectories();
-		$GLOBALS['xoopsTpl']->assign('form', $form->render());
-		break;
-	case 'delete':
-		$templateMain = 'wggithub_admin_directories.tpl';
-		$GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('directories.php'));
-		$directoriesObj = $directoriesHandler->get($dirId);
-		$dirName = $directoriesObj->getVar('dir_name');
-		if (isset($_REQUEST['ok']) && 1 == $_REQUEST['ok']) {
-			if (!$GLOBALS['xoopsSecurity']->check()) {
-				\redirect_header('directories.php', 3, \implode(', ', $GLOBALS['xoopsSecurity']->getErrors()));
-			}
-			if ($directoriesHandler->delete($directoriesObj)) {
-				\redirect_header('directories.php', 3, _AM_WGGITHUB_FORM_DELETE_OK);
-			} else {
-				$GLOBALS['xoopsTpl']->assign('error', $directoriesObj->getHtmlErrors());
-			}
-		} else {
-			$xoopsconfirm = new Common\XoopsConfirm(
-				['ok' => 1, 'dir_id' => $dirId, 'op' => 'delete'],
-				$_SERVER['REQUEST_URI'],
-				\sprintf(_AM_WGGITHUB_FORM_SURE_DELETE, $directoriesObj->getVar('dir_name')));
-			$form = $xoopsconfirm->getFormXoopsConfirm();
-			$GLOBALS['xoopsTpl']->assign('form', $form->render());
-		}
-		break;
+        $directoryDatecreatedObj = \DateTime::createFromFormat(_SHORTDATESTRING, Request::getString('dir_datecreated'));
+        $directoriesObj->setVar('dir_datecreated', $directoryDatecreatedObj->getTimestamp());
+        $directoriesObj->setVar('dir_submitter', Request::getInt('dir_submitter', 0));
+        // Insert Data
+        if ($directoriesHandler->insert($directoriesObj)) {
+            \redirect_header('directories.php?op=list', 2, _AM_WGGITHUB_FORM_OK);
+        }
+        // Get Form
+        $GLOBALS['xoopsTpl']->assign('error', $directoriesObj->getHtmlErrors());
+        $form = $directoriesObj->getFormDirectories();
+        $GLOBALS['xoopsTpl']->assign('form', $form->render());
+        break;
+    case 'edit':
+        $templateMain = 'wggithub_admin_directories.tpl';
+        $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('directories.php'));
+        $adminObject->addItemButton(_AM_WGGITHUB_ADD_DIRECTORY, 'directories.php?op=new', 'add');
+        $adminObject->addItemButton(_AM_WGGITHUB_DIRECTORIES_LIST, 'directories.php', 'list');
+        $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
+        // Get Form
+        $directoriesObj = $directoriesHandler->get($dirId);
+        $form = $directoriesObj->getFormDirectories();
+        $GLOBALS['xoopsTpl']->assign('form', $form->render());
+        break;
+    case 'delete':
+        $templateMain = 'wggithub_admin_directories.tpl';
+        $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('directories.php'));
+        $directoriesObj = $directoriesHandler->get($dirId);
+        $dirName = $directoriesObj->getVar('dir_name');
+        if (isset($_REQUEST['ok']) && 1 == $_REQUEST['ok']) {
+            if (!$GLOBALS['xoopsSecurity']->check()) {
+                \redirect_header('directories.php', 3, \implode(', ', $GLOBALS['xoopsSecurity']->getErrors()));
+            }
+            if ($directoriesHandler->delete($directoriesObj)) {
+                \redirect_header('directories.php', 3, _AM_WGGITHUB_FORM_DELETE_OK);
+            } else {
+                $GLOBALS['xoopsTpl']->assign('error', $directoriesObj->getHtmlErrors());
+            }
+        } else {
+            $xoopsconfirm = new Common\XoopsConfirm(
+                ['ok' => 1, 'dir_id' => $dirId, 'op' => 'delete'],
+                $_SERVER['REQUEST_URI'],
+                \sprintf(_AM_WGGITHUB_FORM_SURE_DELETE, $directoriesObj->getVar('dir_name')));
+            $form = $xoopsconfirm->getFormXoopsConfirm();
+            $GLOBALS['xoopsTpl']->assign('form', $form->render());
+        }
+        break;
 }
 require __DIR__ . '/footer.php';

test.php

Spacing +3 added lines, -3 removed lines

@@ -30,9 +30,9 @@ 
     Github\Releases,
 };
 
-require __DIR__ . '/header.php';
+require __DIR__.'/header.php';
 $GLOBALS['xoopsOption']['template_main'] = 'wggithub_index.tpl';
-include_once \XOOPS_ROOT_PATH . '/header.php';
+include_once \XOOPS_ROOT_PATH.'/header.php';
 // Define Stylesheet
 $GLOBALS['xoTheme']->addStylesheet($style, null);
 $keywords = [];
@@ -165,4 +165,4 @@ 
 var_dump($result);
 */
 
-require __DIR__ . '/footer.php';
+require __DIR__.'/footer.php';

testdata/index.php

Spacing +16 added lines, -16 removed lines

@@ -24,8 +24,8 @@ 
     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', '');
 
@@ -45,7 +45,7 @@ 
             loadSampleData();
         } 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();
         }
         break;
@@ -68,20 +68,20 @@ 
     $tables = Helper::getHelper($moduleDirName)->getModule()->getInfo('tables');
 
     $language = 'english/';
-    if (\is_dir(__DIR__ . '/' . $xoopsConfig['language'])) {
-        $language = $xoopsConfig['language'] . '/';
+    if (\is_dir(__DIR__.'/'.$xoopsConfig['language'])) {
+        $language = $xoopsConfig['language'].'/';
     }
 
     // load module tables
     foreach ($tables as $table) {
-        $tabledata = Yaml::readWrapped($language . $table . '.yml');
+        $tabledata = Yaml::readWrapped($language.$table.'.yml');
         TableLoad::truncateTable($table);
         TableLoad::loadTableFromArray($table, $tabledata);
     }
 
     // load permissions
     $table     = 'group_permission';
-    $tabledata = Yaml::readWrapped($language . $table . '.yml');
+    $tabledata = Yaml::readWrapped($language.$table.'.yml');
     $mid       = Helper::getHelper($moduleDirName)->getModule()->getVar('mid');
     loadTableFromArrayWithReplace($table, $tabledata, 'gperm_modid', $mid);
 
@@ -94,7 +94,7 @@ 
             $utility::rcopy($src, $dest);
         }
     }
-    \redirect_header('../admin/index.php', 1, \constant('CO_' . $moduleDirNameUpper . '_' . 'SAMPLEDATA_SUCCESS'));
+    \redirect_header('../admin/index.php', 1, \constant('CO_'.$moduleDirNameUpper.'_'.'SAMPLEDATA_SUCCESS'));
 }
 
 function saveSampleData()
@@ -108,23 +108,23 @@ 
 
     $tables = Helper::getHelper($moduleDirName)->getModule()->getInfo('tables');
 
-    $languageFolder = __DIR__ . '/' . $xoopsConfig['language'];
-    if (!\file_exists($languageFolder . '/')) {
-        Utility::createFolder($languageFolder . '/');
+    $languageFolder = __DIR__.'/'.$xoopsConfig['language'];
+    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);
 
     // save module tables
     foreach ($tables as $table) {
-        TableLoad::saveTableToYamlFile($table, $exportFolder . $table . '.yml');
+        TableLoad::saveTableToYamlFile($table, $exportFolder.$table.'.yml');
     }
 
     // save permissions
     $criteria = new \CriteriaCompo();
     $criteria->add(new \Criteria('gperm_modid', Helper::getHelper($moduleDirName)->getModule()->getVar('mid')));
     $skipColumns[] = 'gperm_id';
-    TableLoad::saveTableToYamlFile('group_permission', $exportFolder . 'group_permission.yml', $criteria, $skipColumns);
+    TableLoad::saveTableToYamlFile('group_permission', $exportFolder.'group_permission.yml', $criteria, $skipColumns);
     unset($criteria);
 
     //  ---  COPY test folder files ---------------
@@ -135,7 +135,7 @@ 
             Utility::rcopy($src, $dest);
         }
     }
-    \redirect_header('../admin/index.php', 1, \constant('CO_' . $moduleDirNameUpper . '_' . 'SAVE_SAMPLEDATA_SUCCESS'));
+    \redirect_header('../admin/index.php', 1, \constant('CO_'.$moduleDirNameUpper.'_'.'SAVE_SAMPLEDATA_SUCCESS'));
 }
 
 function exportSchema()
@@ -150,7 +150,7 @@ 
         //
         //        \redirect_header('../admin/index.php', 1, \constant('CO_' . $moduleDirNameUpper . '_' . 'EXPORT_SCHEMA_SUCCESS'));
     } catch (\Exception $e) {
-        exit(\constant('CO_' . $moduleDirNameUpper . '_' . 'EXPORT_SCHEMA_ERROR'));
+        exit(\constant('CO_'.$moduleDirNameUpper.'_'.'EXPORT_SCHEMA_ERROR'));
     }
 
 }
\ No newline at end of file

index.php

Spacing +15 added lines, -15 removed lines

@@ -30,15 +30,15 @@ 
     Github\Releases
 };
 
-require __DIR__ . '/header.php';
+require __DIR__.'/header.php';
 $GLOBALS['xoopsOption']['template_main'] = 'wggithub_index.tpl';
-include_once \XOOPS_ROOT_PATH . '/header.php';
+include_once \XOOPS_ROOT_PATH.'/header.php';
 
 // Permissions
 $permGlobalView = $permissionsHandler->getPermGlobalView();
 if (!$permGlobalView) {
     $GLOBALS['xoopsTpl']->assign('error', _NOPERM);
-    require __DIR__ . '/footer.php';
+    require __DIR__.'/footer.php';
 }
 $permGlobalRead = $permissionsHandler->getPermGlobalRead();
 $permReadmeUpdate = $permissionsHandler->getPermReadmeUpdate();
@@ -52,7 +52,7 @@ 
 
 // Define Stylesheet
 $GLOBALS['xoTheme']->addStylesheet($style, null);
-$GLOBALS['xoTheme']->addStylesheet(WGGITHUB_URL . '/assets/css/tabs.css', null);
+$GLOBALS['xoTheme']->addStylesheet(WGGITHUB_URL.'/assets/css/tabs.css', null);
 $keywords = [];
 // 
 $GLOBALS['xoopsTpl']->assign('xoops_icons32_url', XOOPS_ICONS32_URL);
@@ -119,7 +119,7 @@ 
             foreach (\array_keys($directoriesAll) as $i) {
                 $directories[$i] = $directoriesAll[$i]->getValuesDirectories();
                 $dirName = $directoriesAll[$i]->getVar('dir_name');
-                $dirFilterRelease = (bool)$directoriesAll[$i]->getVar('dir_filterrelease');
+                $dirFilterRelease = (bool) $directoriesAll[$i]->getVar('dir_filterrelease');
                 $repos = [];
                 if (Constants::DIRECTORY_TYPE_ORG == $directoriesAll[$i]->getVar('dir_type')) {
                     //$github->readOrgRepositories($dirName, 100, 1);
@@ -131,7 +131,7 @@ 
                 $crRepositories->add(new \Criteria('repo_user', $dirName));
                 $repositoriesCountTotal = $repositoriesHandler->getCount($crRepositories);
                 if ('any' === $filterRelease && $dirFilterRelease) {
-                    $crRepositories->add(new \Criteria('repo_prerelease', 1) );
+                    $crRepositories->add(new \Criteria('repo_prerelease', 1));
                     $crRepositories->add(new \Criteria('repo_release', 1), 'OR');
                 }
                 if ('final' === $filterRelease && $dirFilterRelease) {
@@ -188,9 +188,9 @@ 
                 }
                 $directories[$i]['repos'] = $repos;
                 $directories[$i]['previousRepos'] = $start > 0;
-                $directories[$i]['previousOp'] = '&start=' . ($start - $limit) . '&limit=' . $limit . '&release=' . $filterRelease . '&sortby=' . $filterSortby;
+                $directories[$i]['previousOp'] = '&start='.($start - $limit).'&limit='.$limit.'&release='.$filterRelease.'&sortby='.$filterSortby;
                 $directories[$i]['nextRepos'] = ($repositoriesCount - $start) > $limit;
-                $directories[$i]['nextOp'] = '&start=' . ($start + $limit) . '&limit=' . $limit . '&release=' . $filterRelease . '&sortby=' . $filterSortby;
+                $directories[$i]['nextOp'] = '&start='.($start + $limit).'&limit='.$limit.'&release='.$filterRelease.'&sortby='.$filterSortby;
                 $GLOBALS['xoopsTpl']->assign('start', $start);
                 $GLOBALS['xoopsTpl']->assign('limit', $limit);
                 $GLOBALS['xoopsTpl']->assign('menu', $menu);
@@ -201,8 +201,8 @@ 
             unset($crDirectories, $directories);
             // Display Navigation
             if ($directoriesCount > $limit) {
-                include_once \XOOPS_ROOT_PATH . '/class/pagenav.php';
-                $pagenav = new \XoopsPageNav($directoriesCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
+                include_once \XOOPS_ROOT_PATH.'/class/pagenav.php';
+                $pagenav = new \XoopsPageNav($directoriesCount, $limit, $start, 'start', 'op=list&limit='.$limit);
                 $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
             }
             $GLOBALS['xoopsTpl']->assign('lang_thereare', \sprintf(\_MA_WGGITHUB_INDEX_THEREARE, $directoriesCount));
@@ -215,7 +215,7 @@ 
         // Permissions
         if (!$permGlobalRead) {
             $GLOBALS['xoopsTpl']->assign('error', _NOPERM);
-            require __DIR__ . '/footer.php';
+            require __DIR__.'/footer.php';
         }
         executeUpdate();
         break;
@@ -223,7 +223,7 @@ 
         // Permissions
         if (!$permReadmeUpdate) {
             $GLOBALS['xoopsTpl']->assign('error', _NOPERM);
-            require __DIR__ . '/footer.php';
+            require __DIR__.'/footer.php';
         }
         $repoId = Request::getInt('repo_id', 0);
         $repoUser  = Request::getString('repo_user', 'none');
@@ -236,14 +236,14 @@ 
 // Breadcrumbs
 $xoBreadcrumbs[] = ['title' => \_MA_WGGITHUB_INDEX];
 // Keywords
-wggithubMetaKeywords($helper->getConfig('keywords') . ', ' . \implode(',', $keywords));
+wggithubMetaKeywords($helper->getConfig('keywords').', '.\implode(',', $keywords));
 unset($keywords);
 // Description
 wggithubMetaDescription(\_MA_WGGITHUB_INDEX_DESC);
 $GLOBALS['xoopsTpl']->assign('xoops_mpageurl', WGGITHUB_URL.'/index.php');
 $GLOBALS['xoopsTpl']->assign('xoops_icons32_url', XOOPS_ICONS32_URL);
 $GLOBALS['xoopsTpl']->assign('wggithub_upload_url', WGGITHUB_UPLOAD_URL);
-require __DIR__ . '/footer.php';
+require __DIR__.'/footer.php';
 
 
 
@@ -252,7 +252,7 @@ 
  *
  * @return bool
  */
-function executeUpdate(){
+function executeUpdate() {
 
     $github = Github::getInstance();
     $helper = Helper::getInstance();

class/Directories.php

Indentation +102 added lines, -102 removed lines

@@ -33,74 +33,74 @@ 
  */
 class Directories extends \XoopsObject
 {
-	/**
-	 * Constructor
-	 *
-	 * @param null
-	 */
-	public function __construct()
-	{
-		$this->initVar('dir_id', \XOBJ_DTYPE_INT);
-		$this->initVar('dir_name', \XOBJ_DTYPE_TXTBOX);
-		$this->initVar('dir_type', \XOBJ_DTYPE_INT);
+    /**
+     * Constructor
+     *
+     * @param null
+     */
+    public function __construct()
+    {
+        $this->initVar('dir_id', \XOBJ_DTYPE_INT);
+        $this->initVar('dir_name', \XOBJ_DTYPE_TXTBOX);
+        $this->initVar('dir_type', \XOBJ_DTYPE_INT);
         $this->initVar('dir_autoupdate', XOBJ_DTYPE_INT);
         $this->initVar('dir_online', XOBJ_DTYPE_INT);
         $this->initVar('dir_filterrelease', XOBJ_DTYPE_INT);
-		$this->initVar('dir_datecreated', \XOBJ_DTYPE_INT);
-		$this->initVar('dir_submitter', \XOBJ_DTYPE_INT);
-	}
+        $this->initVar('dir_datecreated', \XOBJ_DTYPE_INT);
+        $this->initVar('dir_submitter', \XOBJ_DTYPE_INT);
+    }
 
-	/**
-	 * @static function &getInstance
-	 *
-	 * @param null
-	 */
-	public static function getInstance()
-	{
-		static $instance = false;
-		if (!$instance) {
-			$instance = new self();
-		}
-	}
+    /**
+     * @static function &getInstance
+     *
+     * @param null
+     */
+    public static function getInstance()
+    {
+        static $instance = false;
+        if (!$instance) {
+            $instance = new self();
+        }
+    }
 
-	/**
-	 * The new inserted $Id
-	 * @return inserted id
-	 */
-	public function getNewInsertedIdDirectories()
-	{
-		$newInsertedId = $GLOBALS['xoopsDB']->getInsertId();
-		return $newInsertedId;
-	}
+    /**
+     * The new inserted $Id
+     * @return inserted id
+     */
+    public function getNewInsertedIdDirectories()
+    {
+        $newInsertedId = $GLOBALS['xoopsDB']->getInsertId();
+        return $newInsertedId;
+    }
 
-	/**
-	 * @public function getForm
-	 * @param bool $action
-	 * @return \XoopsThemeForm
-	 */
-	public function getFormDirectories($action = false)
-	{
-		$helper = \XoopsModules\Wggithub\Helper::getInstance();
-		if (!$action) {
-			$action = $_SERVER['REQUEST_URI'];
-		}
-		$isAdmin = $GLOBALS['xoopsUser']->isAdmin($GLOBALS['xoopsModule']->mid());
+    /**
+     * @public function getForm
+     * @param bool $action
+     * @return \XoopsThemeForm
+     */
+    public function getFormDirectories($action = false)
+    {
+        $helper = \XoopsModules\Wggithub\Helper::getInstance();
+        if (!$action) {
+            $action = $_SERVER['REQUEST_URI'];
+        }
+        $isAdmin = $GLOBALS['xoopsUser']->isAdmin($GLOBALS['xoopsModule']->mid());
 
-		// Title
-		$title = $this->isNew() ? \sprintf(\_AM_WGGITHUB_DIRECTORY_ADD) : \sprintf(\_AM_WGGITHUB_DIRECTORY_EDIT);
-		// Get Theme Form
-		\xoops_load('XoopsFormLoader');
-		$form = new \XoopsThemeForm($title, 'form', $action, 'post', true);
-		$form->setExtra('enctype="multipart/form-data"');
-		// Form Text dirName
-		$form->addElement(new \XoopsFormText(\_AM_WGGITHUB_DIRECTORY_NAME, 'dir_name', 50, 255, $this->getVar('dir_name')), true);
-		// Directories Handler
-		$directoriesHandler = $helper->getHandler('Directories');
-		// Form Select dirType
-		$dirTypeSelect = new \XoopsFormSelect(\_AM_WGGITHUB_DIRECTORY_TYPE, 'dir_type', $this->getVar('dir_type'), 5);
-		$dirTypeSelect->addOption(Constants::DIRECTORY_TYPE_USER, \_AM_WGGITHUB_DIRECTORY_TYPE_USER);
-		$dirTypeSelect->addOption(Constants::DIRECTORY_TYPE_ORG, \_AM_WGGITHUB_DIRECTORY_TYPE_ORG);
-		$form->addElement($dirTypeSelect, true);
+        // Title
+        $title = $this->isNew() ? \sprintf(\_AM_WGGITHUB_DIRECTORY_ADD) : \sprintf(\_AM_WGGITHUB_DIRECTORY_EDIT);
+        // Get Theme Form
+        \xoops_load('XoopsFormLoader');
+        $form = new \XoopsThemeForm($title, 'form', $action, 'post', true);
+        $form->setExtra('enctype="multipart/form-data"');
+        // Form Text dirName
+        $form->addElement(new \XoopsFormText(\_AM_WGGITHUB_DIRECTORY_NAME, 'dir_name', 50, 255, $this->getVar('dir_name')), true);
+        // Directories Handler
+        $directoriesHandler = $helper->getHandler('Directories');
+        // Form Select dirType
+        $dirTypeSelect = new \XoopsFormSelect(\_AM_WGGITHUB_DIRECTORY_TYPE, 'dir_type', $this->getVar('dir_type'), 5);
+        $dirTypeSelect->addOption(Constants::DIRECTORY_TYPE_USER, \_AM_WGGITHUB_DIRECTORY_TYPE_USER);
+        $dirTypeSelect->addOption(Constants::DIRECTORY_TYPE_ORG, \_AM_WGGITHUB_DIRECTORY_TYPE_ORG);
+        $form->addElement($dirTypeSelect, true);
         // Form Radio Yes/No dirAutoupdate
         $dirAutoupdate = $this->isNew() ?: $this->getVar('dir_autoupdate');
         $form->addElement(new \XoopsFormRadioYN(_AM_WGGITHUB_DIRECTORY_AUTOUPDATE, 'dir_autoupdate', $dirAutoupdate));
@@ -110,51 +110,51 @@ 
         // Form Radio Yes/No dirFilterrelease
         $dirFilterrelease = $this->isNew() ?: $this->getVar('dir_filterrelease');
         $form->addElement(new \XoopsFormRadioYN(_AM_WGGITHUB_DIRECTORY_FILTERRELEASE, 'dir_filterrelease', $dirFilterrelease));
-		// Form Text Date Select dirDatecreated
-		$dirDatecreated = $this->isNew() ?: $this->getVar('dir_datecreated');
-		$form->addElement(new \XoopsFormTextDateSelect(\_AM_WGGITHUB_DIRECTORY_DATECREATED, 'dir_datecreated', '', $dirDatecreated));
-		// Form Select User dirSubmitter
-		$form->addElement(new \XoopsFormSelectUser(\_AM_WGGITHUB_DIRECTORY_SUBMITTER, 'dir_submitter', false, $this->getVar('dir_submitter')));
-		// To Save
-		$form->addElement(new \XoopsFormHidden('op', 'save'));
-		$form->addElement(new \XoopsFormButtonTray('', _SUBMIT, 'submit', '', false));
-		return $form;
-	}
+        // Form Text Date Select dirDatecreated
+        $dirDatecreated = $this->isNew() ?: $this->getVar('dir_datecreated');
+        $form->addElement(new \XoopsFormTextDateSelect(\_AM_WGGITHUB_DIRECTORY_DATECREATED, 'dir_datecreated', '', $dirDatecreated));
+        // Form Select User dirSubmitter
+        $form->addElement(new \XoopsFormSelectUser(\_AM_WGGITHUB_DIRECTORY_SUBMITTER, 'dir_submitter', false, $this->getVar('dir_submitter')));
+        // To Save
+        $form->addElement(new \XoopsFormHidden('op', 'save'));
+        $form->addElement(new \XoopsFormButtonTray('', _SUBMIT, 'submit', '', false));
+        return $form;
+    }
 
-	/**
-	 * Get Values
-	 * @param null $keys
-	 * @param null $format
-	 * @param null $maxDepth
-	 * @return array
-	 */
-	public function getValuesDirectories($keys = null, $format = null, $maxDepth = null)
-	{
-		$ret = $this->getValues($keys, $format, $maxDepth);
-		$ret['id']          = $this->getVar('dir_id');
-		$ret['name']        = $this->getVar('dir_name');
-		$ret['type']        = $this->getVar('dir_type');
+    /**
+     * Get Values
+     * @param null $keys
+     * @param null $format
+     * @param null $maxDepth
+     * @return array
+     */
+    public function getValuesDirectories($keys = null, $format = null, $maxDepth = null)
+    {
+        $ret = $this->getValues($keys, $format, $maxDepth);
+        $ret['id']          = $this->getVar('dir_id');
+        $ret['name']        = $this->getVar('dir_name');
+        $ret['type']        = $this->getVar('dir_type');
         $ret['type_text']   = Constants::DIRECTORY_TYPE_USER == $this->getVar('dir_type') ? \_AM_WGGITHUB_DIRECTORY_TYPE_USER : \_AM_WGGITHUB_DIRECTORY_TYPE_ORG;
         $ret['autoupdate']  = (int)$this->getVar('dir_autoupdate') > 0 ? _YES : _NO;
         $ret['online']      = (int)$this->getVar('dir_online') > 0 ? _YES : _NO;
         $ret['filterrelease'] = (int)$this->getVar('dir_filterrelease') > 0 ? _YES : _NO;
-		$ret['datecreated'] = \formatTimestamp($this->getVar('dir_datecreated'), 's');
-		$ret['submitter']   = \XoopsUser::getUnameFromId($this->getVar('dir_submitter'));
-		return $ret;
-	}
+        $ret['datecreated'] = \formatTimestamp($this->getVar('dir_datecreated'), 's');
+        $ret['submitter']   = \XoopsUser::getUnameFromId($this->getVar('dir_submitter'));
+        return $ret;
+    }
 
-	/**
-	 * Returns an array representation of the object
-	 *
-	 * @return array
-	 */
-	public function toArrayDirectories()
-	{
-		$ret = [];
-		$vars = $this->getVars();
-		foreach (\array_keys($vars) as $var) {
-			$ret[$var] = $this->getVar('"{$var}"');
-		}
-		return $ret;
-	}
+    /**
+     * Returns an array representation of the object
+     *
+     * @return array
+     */
+    public function toArrayDirectories()
+    {
+        $ret = [];
+        $vars = $this->getVars();
+        foreach (\array_keys($vars) as $var) {
+            $ret[$var] = $this->getVar('"{$var}"');
+        }
+        return $ret;
+    }
 }

class/Constants.php

Indentation +9 added lines, -9 removed lines

@@ -28,7 +28,7 @@ 
  */
 interface Constants
 {
-	// Constants for tables
+    // Constants for tables
     public const TABLE_SETTINGS = 0;
     public const TABLE_REPOSITORIES = 1;
     public const TABLE_DIRECTORIES = 2;
@@ -39,16 +39,16 @@ 
     public const DIRECTORY_TYPE_USER = 1;
     public const DIRECTORY_TYPE_ORG  = 2;
 
-	// Constants for status
-	public const STATUS_NONE     = 0;
+    // Constants for status
+    public const STATUS_NONE     = 0;
     public const STATUS_NEW      = 1;
     public const STATUS_UPDATED  = 2;
-	public const STATUS_UPTODATE = 3;
+    public const STATUS_UPTODATE = 3;
 
-	// Constants for permissions
-	public const PERM_GLOBAL_NONE   = 0;
-	public const PERM_GLOBAL_VIEW   = 1;
-	public const PERM_GLOBAL_READ   = 2;
-	public const PERM_README_UPDATE = 3;
+    // Constants for permissions
+    public const PERM_GLOBAL_NONE   = 0;
+    public const PERM_GLOBAL_VIEW   = 1;
+    public const PERM_GLOBAL_READ   = 2;
+    public const PERM_README_UPDATE = 3;
 
 }

class/MDParser/Parsedown.php

Spacing +59 added lines, -59 removed lines

@@ -209,7 +209,7 @@ 
 
             if (isset($CurrentBlock['continuable']))
             {
-                $methodName = 'block' . $CurrentBlock['type'] . 'Continue';
+                $methodName = 'block'.$CurrentBlock['type'].'Continue';
                 $Block = $this->$methodName($Line, $CurrentBlock);
 
                 if (isset($Block))
@@ -220,7 +220,7 @@ 
                 } else {
                     if ($this->isBlockCompletable($CurrentBlock['type']))
                     {
-                        $methodName = 'block' . $CurrentBlock['type'] . 'Complete';
+                        $methodName = 'block'.$CurrentBlock['type'].'Complete';
                         $CurrentBlock = $this->$methodName($CurrentBlock);
                     }
                 }
@@ -238,7 +238,7 @@ 
             {
                 foreach ($this->BlockTypes[$marker] as $blockType)
                 {
-                    $blockTypes []= $blockType;
+                    $blockTypes [] = $blockType;
                 }
             }
 
@@ -253,7 +253,7 @@ 
                 {
                     $Block['type'] = $blockType;
 
-                    if ( ! isset($Block['identified']))
+                    if (!isset($Block['identified']))
                     {
                         if (isset($CurrentBlock))
                         {
@@ -300,7 +300,7 @@ 
 
         if (isset($CurrentBlock['continuable']) and $this->isBlockCompletable($CurrentBlock['type']))
         {
-            $methodName = 'block' . $CurrentBlock['type'] . 'Complete';
+            $methodName = 'block'.$CurrentBlock['type'].'Complete';
             $CurrentBlock = $this->$methodName($CurrentBlock);
         }
 
@@ -318,7 +318,7 @@ 
 
     protected function extractElement(array $Component)
     {
-        if ( ! isset($Component['element']))
+        if (!isset($Component['element']))
         {
             if (isset($Component['markup']))
             {
@@ -333,12 +333,12 @@ 
 
     protected function isBlockContinuable($Type)
     {
-        return method_exists($this, 'block' . $Type . 'Continue');
+        return method_exists($this, 'block'.$Type.'Continue');
     }
 
     protected function isBlockCompletable($Type)
     {
-        return method_exists($this, 'block' . $Type . 'Complete');
+        return method_exists($this, 'block'.$Type.'Complete');
     }
 
     #
@@ -346,7 +346,7 @@ 
 
     protected function blockCode($Line, $Block = null)
     {
-        if (isset($Block) and $Block['type'] === 'Paragraph' and ! isset($Block['interrupted']))
+        if (isset($Block) and $Block['type'] === 'Paragraph' and !isset($Block['interrupted']))
         {
             return;
         }
@@ -430,7 +430,7 @@ 
             return;
         }
 
-        $Block['element']['rawHtml'] .= "\n" . $Line['body'];
+        $Block['element']['rawHtml'] .= "\n".$Line['body'];
 
         if (strpos($Line['text'], '-->') !== false)
         {
@@ -521,7 +521,7 @@ 
             return $Block;
         }
 
-        $Block['element']['element']['text'] .= "\n" . $Line['body'];
+        $Block['element']['element']['text'] .= "\n".$Line['body'];
 
         return $Block;
     }
@@ -554,7 +554,7 @@ 
 
         $Block = array(
             'element' => array(
-                'name' => 'h' . $level,
+                'name' => 'h'.$level,
                 'handler' => array(
                     'function' => 'lineElements',
                     'argument' => $text,
@@ -581,7 +581,7 @@ 
             {
                 $contentIndent -= 1;
                 $matches[1] = substr($matches[1], 0, -$contentIndent);
-                $matches[3] = str_repeat(' ', $contentIndent) . $matches[3];
+                $matches[3] = str_repeat(' ', $contentIndent).$matches[3];
             } elseif ($contentIndent === 0) {
                 $matches[1] .= ' ';
             }
@@ -612,7 +612,7 @@ 
                     if (
                         isset($CurrentBlock)
                         and $CurrentBlock['type'] === 'Paragraph'
-                        and ! isset($CurrentBlock['interrupted'])
+                        and !isset($CurrentBlock['interrupted'])
                     ) {
                         return;
                     }
@@ -630,7 +630,7 @@ 
                 )
             );
 
-            $Block['element']['elements'] []= & $Block['li'];
+            $Block['element']['elements'] [] = & $Block['li'];
 
             return $Block;
         }
@@ -657,7 +657,7 @@ 
             )
         ) {
             if (isset($Block['interrupted'])) {
-                $Block['li']['handler']['argument'] []= '';
+                $Block['li']['handler']['argument'] [] = '';
 
                 $Block['loose'] = true;
 
@@ -679,7 +679,7 @@ 
                 )
             );
 
-            $Block['element']['elements'] []= & $Block['li'];
+            $Block['element']['elements'] [] = & $Block['li'];
 
             return $Block;
         } elseif ($Line['indent'] < $requiredIndent and $this->blockList($Line)) {
@@ -693,7 +693,7 @@ 
         if ($Line['indent'] >= $requiredIndent) {
             if (isset($Block['interrupted']))
             {
-                $Block['li']['handler']['argument'] []= '';
+                $Block['li']['handler']['argument'] [] = '';
 
                 $Block['loose'] = true;
 
@@ -702,15 +702,15 @@ 
 
             $text = substr($Line['body'], $requiredIndent);
 
-            $Block['li']['handler']['argument'] []= $text;
+            $Block['li']['handler']['argument'] [] = $text;
 
             return $Block;
         }
 
-        if ( ! isset($Block['interrupted'])) {
+        if (!isset($Block['interrupted'])) {
             $text = preg_replace('/^[ ]{0,'.$requiredIndent.'}+/', '', $Line['body']);
 
-            $Block['li']['handler']['argument'] []= $text;
+            $Block['li']['handler']['argument'] [] = $text;
 
             return $Block;
         }
@@ -723,7 +723,7 @@ 
             {
                 if (end($li['handler']['argument']) !== '')
                 {
-                    $li['handler']['argument'] []= '';
+                    $li['handler']['argument'] [] = '';
                 }
             }
         }
@@ -759,13 +759,13 @@ 
         }
 
         if ($Line['text'][0] === '>' and preg_match('/^>[ ]?+(.*+)/', $Line['text'], $matches)) {
-            $Block['element']['handler']['argument'] []= $matches[1];
+            $Block['element']['handler']['argument'] [] = $matches[1];
 
             return $Block;
         }
 
-        if ( ! isset($Block['interrupted'])) {
-            $Block['element']['handler']['argument'] []= $Line['text'];
+        if (!isset($Block['interrupted'])) {
+            $Block['element']['handler']['argument'] [] = $Line['text'];
 
             return $Block;
         }
@@ -794,7 +794,7 @@ 
 
     protected function blockSetextHeader($Line, array $Block = null)
     {
-        if ( ! isset($Block) or $Block['type'] !== 'Paragraph' or isset($Block['interrupted'])) {
+        if (!isset($Block) or $Block['type'] !== 'Paragraph' or isset($Block['interrupted'])) {
             return;
         }
 
@@ -839,7 +839,7 @@ 
             return;
         }
 
-        $Block['element']['rawHtml'] .= "\n" . $Line['body'];
+        $Block['element']['rawHtml'] .= "\n".$Line['body'];
 
         return $Block;
     }
@@ -874,7 +874,7 @@ 
 
     protected function blockTable($Line, array $Block = null)
     {
-        if ( ! isset($Block) or $Block['type'] !== 'Paragraph' or isset($Block['interrupted'])) {
+        if (!isset($Block) or $Block['type'] !== 'Paragraph' or isset($Block['interrupted'])) {
             return;
         }
 
@@ -917,7 +917,7 @@ 
                 $alignment = $alignment === 'left' ? 'center' : 'right';
             }
 
-            $alignments []= $alignment;
+            $alignments [] = $alignment;
         }
 
         # ~
@@ -955,7 +955,7 @@ 
                 );
             }
 
-            $HeaderElements []= $HeaderElement;
+            $HeaderElements [] = $HeaderElement;
         }
 
         # ~
@@ -969,16 +969,16 @@ 
             ),
         );
 
-        $Block['element']['elements'] []= array(
+        $Block['element']['elements'] [] = array(
             'name' => 'thead',
         );
 
-        $Block['element']['elements'] []= array(
+        $Block['element']['elements'] [] = array(
             'name' => 'tbody',
             'elements' => array(),
         );
 
-        $Block['element']['elements'][0]['elements'] []= array(
+        $Block['element']['elements'][0]['elements'] [] = array(
             'name' => 'tr',
             'elements' => $HeaderElements,
         );
@@ -1021,11 +1021,11 @@ 
                 if (isset($Block['alignments'][$index]))
                 {
                     $Element['attributes'] = array(
-                        'style' => 'text-align: ' . $Block['alignments'][$index] . ';',
+                        'style' => 'text-align: '.$Block['alignments'][$index].';',
                     );
                 }
 
-                $Elements []= $Element;
+                $Elements [] = $Element;
             }
 
             $Element = array(
@@ -1033,7 +1033,7 @@ 
                 'elements' => $Elements,
             );
 
-            $Block['element']['elements'][1]['elements'] []= $Element;
+            $Block['element']['elements'][1]['elements'] [] = $Element;
 
             return $Block;
         }
@@ -1133,7 +1133,7 @@ 
 
                 $Inline = $this->{"inline$inlineType"}($Excerpt);
 
-                if ( ! isset($Inline))
+                if (!isset($Inline))
                 {
                     continue;
                 }
@@ -1147,7 +1147,7 @@ 
 
                 # sets a default inline position
 
-                if ( ! isset($Inline['position']))
+                if (!isset($Inline['position']))
                 {
                     $Inline['position'] = $markerPosition;
                 }
@@ -1191,7 +1191,7 @@ 
 
         foreach ($Elements as &$Element)
         {
-            if ( ! isset($Element['autobreak']))
+            if (!isset($Element['autobreak']))
             {
                 $Element['autobreak'] = false;
             }
@@ -1247,14 +1247,14 @@ 
         $hostnameLabel = '[a-zA-Z0-9](?:[a-zA-Z0-9-]{0,61}[a-zA-Z0-9])?';
 
         $commonMarkEmail = '[a-zA-Z0-9.!#$%&\'*+\/=?^_`{|}~-]++@'
-            . $hostnameLabel . '(?:\.' . $hostnameLabel . ')*';
+            . $hostnameLabel.'(?:\.'.$hostnameLabel.')*';
 
         if (strpos($Excerpt['text'], '>') !== false
             and preg_match("/^<((mailto:)?$commonMarkEmail)>/i", $Excerpt['text'], $matches)
-        ){
+        ) {
             $url = $matches[1];
 
-            if ( ! isset($matches[2]))
+            if (!isset($matches[2]))
             {
                 $url = "mailto:$url";
             }
@@ -1274,7 +1274,7 @@ 
 
     protected function inlineEmphasis($Excerpt)
     {
-        if ( ! isset($Excerpt['text'][1]))
+        if (!isset($Excerpt['text'][1]))
         {
             return;
         }
@@ -1320,12 +1320,12 @@ 
 
     protected function inlineImage($Excerpt)
     {
-        if ( ! isset($Excerpt['text'][1]) or $Excerpt['text'][1] !== '[')
+        if (!isset($Excerpt['text'][1]) or $Excerpt['text'][1] !== '[')
         {
             return;
         }
 
-        $Excerpt['text']= substr($Excerpt['text'], 1);
+        $Excerpt['text'] = substr($Excerpt['text'], 1);
 
         $Link = $this->inlineLink($Excerpt);
 
@@ -1411,7 +1411,7 @@ 
                 $definition = strtolower($Element['handler']['argument']);
             }
 
-            if ( ! isset($this->DefinitionData['Reference'][$definition]))
+            if (!isset($this->DefinitionData['Reference'][$definition]))
             {
                 return;
             }
@@ -1466,7 +1466,7 @@ 
             and preg_match('/^&(#?+[0-9a-zA-Z]++);/', $Excerpt['text'], $matches)
         ) {
             return array(
-                'element' => array('rawHtml' => '&' . $matches[1] . ';'),
+                'element' => array('rawHtml' => '&'.$matches[1].';'),
                 'extent' => strlen($matches[0]),
             );
         }
@@ -1476,7 +1476,7 @@ 
 
     protected function inlineStrikethrough($Excerpt)
     {
-        if ( ! isset($Excerpt['text'][1]))
+        if (!isset($Excerpt['text'][1]))
         {
             return;
         }
@@ -1499,7 +1499,7 @@ 
 
     protected function inlineUrl($Excerpt)
     {
-        if ($this->urlsLinked !== true or ! isset($Excerpt['text'][2]) or $Excerpt['text'][2] !== '/')
+        if ($this->urlsLinked !== true or !isset($Excerpt['text'][2]) or $Excerpt['text'][2] !== '/')
         {
             return;
         }
@@ -1670,7 +1670,7 @@ 
 
         if ($hasName)
         {
-            $markup .= '<' . $Element['name'];
+            $markup .= '<'.$Element['name'];
 
             if (isset($Element['attributes']))
             {
@@ -1728,7 +1728,7 @@ 
                 }
             }
 
-            $markup .= $hasName ? '</' . $Element['name'] . '>' : '';
+            $markup .= $hasName ? '</'.$Element['name'].'>' : '';
         }
         elseif ($hasName)
         {
@@ -1757,7 +1757,7 @@ 
             // (autobreak === false) covers both sides of an element
             $autoBreak = !$autoBreak ? $autoBreak : $autoBreakNext;
 
-            $markup .= ($autoBreak ? "\n" : '') . $this->element($Element);
+            $markup .= ($autoBreak ? "\n" : '').$this->element($Element);
             $autoBreak = $autoBreakNext;
         }
 
@@ -1772,7 +1772,7 @@ 
     {
         $Elements = $this->linesElements($lines);
 
-        if ( ! in_array('', $lines)
+        if (!in_array('', $lines)
             and isset($Elements[0]) and isset($Elements[0]['name'])
             and $Elements[0]['name'] === 'p'
         ) {
@@ -1829,12 +1829,12 @@ 
     protected function sanitiseElement(array $Element)
     {
         static $goodAttribute = '/^[a-zA-Z0-9][a-zA-Z0-9-_]*+$/';
-        static $safeUrlNameToAtt  = array(
+        static $safeUrlNameToAtt = array(
             'a'   => 'href',
             'img' => 'src',
         );
 
-        if ( ! isset($Element['name']))
+        if (!isset($Element['name']))
         {
             unset($Element['attributes']);
             return $Element;
@@ -1845,12 +1845,12 @@ 
             $Element = $this->filterUnsafeUrlInAttribute($Element, $safeUrlNameToAtt[$Element['name']]);
         }
 
-        if ( ! empty($Element['attributes']))
+        if (!empty($Element['attributes']))
         {
             foreach ($Element['attributes'] as $att => $val)
             {
                 # filter out badly parsed attribute
-                if ( ! preg_match($goodAttribute, $att))
+                if (!preg_match($goodAttribute, $att))
                 {
                     unset($Element['attributes'][$att]);
                 }
@@ -1951,8 +1951,8 @@ 
     protected $textLevelElements = array(
         'a', 'br', 'bdo', 'abbr', 'blink', 'nextid', 'acronym', 'basefont',
         'b', 'em', 'big', 'cite', 'small', 'spacer', 'listing',
-        'i', 'rp', 'del', 'code',          'strike', 'marquee',
-        'q', 'rt', 'ins', 'font',          'strong',
+        'i', 'rp', 'del', 'code', 'strike', 'marquee',
+        'q', 'rt', 'ins', 'font', 'strong',
         's', 'tt', 'kbd', 'mark',
         'u', 'xm', 'sub', 'nobr',
                    'sup', 'ruby',

Braces +13 added lines, -26 removed lines

@@ -1284,12 +1284,10 @@ 
         if ($Excerpt['text'][1] === $marker and preg_match($this->StrongRegex[$marker], $Excerpt['text'], $matches))
         {
             $emphasis = 'strong';
-        }
-        elseif (preg_match($this->EmRegex[$marker], $Excerpt['text'], $matches))
+        } elseif (preg_match($this->EmRegex[$marker], $Excerpt['text'], $matches))
         {
             $emphasis = 'em';
-        }
-        else
+        } else
         {
             return;
         }
@@ -1380,8 +1378,7 @@ 
             $extent += strlen($matches[0]);
 
             $remainder = substr($remainder, $extent);
-        }
-        else
+        } else
         {
             return;
         }
@@ -1396,8 +1393,7 @@ 
             }
 
             $extent += strlen($matches[0]);
-        }
-        else
+        } else
         {
             if (preg_match('/^\s*\[(.*?)\]/', $remainder, $matches))
             {
@@ -1405,8 +1401,7 @@ 
                 $definition = strtolower($definition);
 
                 $extent += strlen($matches[0]);
-            }
-            else
+            } else
             {
                 $definition = strtolower($Element['handler']['argument']);
             }
@@ -1571,8 +1566,7 @@ 
                 $argument = $Element['text'];
                 unset($Element['text']);
                 $destination = 'rawHtml';
-            }
-            else
+            } else
             {
                 $function = $Element['handler']['function'];
                 $argument = $Element['handler']['argument'];
@@ -1609,8 +1603,7 @@ 
         if (isset($Element['elements']))
         {
             $Element['elements'] = $this->elementsApplyRecursive($closure, $Element['elements']);
-        }
-        elseif (isset($Element['element']))
+        } elseif (isset($Element['element']))
         {
             $Element['element'] = $this->elementApplyRecursive($closure, $Element['element']);
         }
@@ -1623,8 +1616,7 @@ 
         if (isset($Element['elements']))
         {
             $Element['elements'] = $this->elementsApplyRecursiveDepthFirst($closure, $Element['elements']);
-        }
-        elseif (isset($Element['element']))
+        } elseif (isset($Element['element']))
         {
             $Element['element'] = $this->elementsApplyRecursiveDepthFirst($closure, $Element['element']);
         }
@@ -1711,26 +1703,22 @@ 
             if (isset($Element['elements']))
             {
                 $markup .= $this->elements($Element['elements']);
-            }
-            elseif (isset($Element['element']))
+            } elseif (isset($Element['element']))
             {
                 $markup .= $this->element($Element['element']);
-            }
-            else
+            } else
             {
                 if (!$permitRawHtml)
                 {
                     $markup .= self::escape($text, true);
-                }
-                else
+                } else
                 {
                     $markup .= $text;
                 }
             }
 
             $markup .= $hasName ? '</' . $Element['name'] . '>' : '';
-        }
-        elseif ($hasName)
+        } elseif ($hasName)
         {
             $markup .= ' />';
         }
@@ -1896,8 +1884,7 @@ 
         if ($len > strlen($string))
         {
             return false;
-        }
-        else
+        } else
         {
             return strtolower(substr($string, 0, $len)) === strtolower($needle);
         }

class/ReadmesHandler.php

Indentation +84 added lines, -84 removed lines

@@ -35,96 +35,96 @@
  */
 class ReadmesHandler extends \XoopsPersistableObjectHandler
 {
-	/**
-	 * Constructor
-	 *
-	 * @param \XoopsDatabase $db
-	 */
-	public function __construct(\XoopsDatabase $db)
-	{
-		parent::__construct($db, 'wggithub_readmes', Readmes::class, 'rm_id', 'rm_name');
-	}
+    /**
+     * Constructor
+     *
+     * @param \XoopsDatabase $db
+     */
+    public function __construct(\XoopsDatabase $db)
+    {
+        parent::__construct($db, 'wggithub_readmes', Readmes::class, 'rm_id', 'rm_name');
+    }
 
-	/**
-	 * @param bool $isNew
-	 *
-	 * @return object
-	 */
-	public function create($isNew = true)
-	{
-		return parent::create($isNew);
-	}
+    /**
+     * @param bool $isNew
+     *
+     * @return object
+     */
+    public function create($isNew = true)
+    {
+        return parent::create($isNew);
+    }
 
-	/**
-	 * retrieve a field
-	 *
-	 * @param int $i field id
-	 * @param null fields
-	 * @return mixed reference to the {@link Get} object
-	 */
-	public function get($i = null, $fields = null)
-	{
-		return parent::get($i, $fields);
-	}
+    /**
+     * retrieve a field
+     *
+     * @param int $i field id
+     * @param null fields
+     * @return mixed reference to the {@link Get} object
+     */
+    public function get($i = null, $fields = null)
+    {
+        return parent::get($i, $fields);
+    }
 
-	/**
-	 * get inserted id
-	 *
-	 * @param null
-	 * @return int reference to the {@link Get} object
-	 */
-	public function getInsertId()
-	{
-		return $this->db->getInsertId();
-	}
+    /**
+     * get inserted id
+     *
+     * @param null
+     * @return int reference to the {@link Get} object
+     */
+    public function getInsertId()
+    {
+        return $this->db->getInsertId();
+    }
 
-	/**
-	 * Get Count Readmes in the database
-	 * @param int    $start
-	 * @param int    $limit
-	 * @param string $sort
-	 * @param string $order
-	 * @return int
-	 */
-	public function getCountReadmes($start = 0, $limit = 0, $sort = 'rm_id ASC, rm_name', $order = 'ASC')
-	{
-		$crCountReadmes = new \CriteriaCompo();
-		$crCountReadmes = $this->getReadmesCriteria($crCountReadmes, $start, $limit, $sort, $order);
-		return $this->getCount($crCountReadmes);
-	}
+    /**
+     * Get Count Readmes in the database
+     * @param int    $start
+     * @param int    $limit
+     * @param string $sort
+     * @param string $order
+     * @return int
+     */
+    public function getCountReadmes($start = 0, $limit = 0, $sort = 'rm_id ASC, rm_name', $order = 'ASC')
+    {
+        $crCountReadmes = new \CriteriaCompo();
+        $crCountReadmes = $this->getReadmesCriteria($crCountReadmes, $start, $limit, $sort, $order);
+        return $this->getCount($crCountReadmes);
+    }
 
-	/**
-	 * Get All Readmes in the database
-	 * @param int    $start
-	 * @param int    $limit
-	 * @param string $sort
-	 * @param string $order
-	 * @return array
-	 */
-	public function getAllReadmes($start = 0, $limit = 0, $sort = 'rm_id ASC, rm_name', $order = 'ASC')
-	{
-		$crAllReadmes = new \CriteriaCompo();
-		$crAllReadmes = $this->getReadmesCriteria($crAllReadmes, $start, $limit, $sort, $order);
-		return $this->getAll($crAllReadmes);
-	}
+    /**
+     * Get All Readmes in the database
+     * @param int    $start
+     * @param int    $limit
+     * @param string $sort
+     * @param string $order
+     * @return array
+     */
+    public function getAllReadmes($start = 0, $limit = 0, $sort = 'rm_id ASC, rm_name', $order = 'ASC')
+    {
+        $crAllReadmes = new \CriteriaCompo();
+        $crAllReadmes = $this->getReadmesCriteria($crAllReadmes, $start, $limit, $sort, $order);
+        return $this->getAll($crAllReadmes);
+    }
 
-	/**
-	 * Get Criteria Readmes
-	 * @param        $crReadmes
-	 * @param int    $start
-	 * @param int    $limit
-	 * @param string $sort
-	 * @param string $order
-	 * @return int
-	 */
-	private function getReadmesCriteria($crReadmes, $start, $limit, $sort, $order)
-	{
-		$crReadmes->setStart($start);
-		$crReadmes->setLimit($limit);
-		$crReadmes->setSort($sort);
-		$crReadmes->setOrder($order);
-		return $crReadmes;
-	}
+    /**
+     * Get Criteria Readmes
+     * @param        $crReadmes
+     * @param int    $start
+     * @param int    $limit
+     * @param string $sort
+     * @param string $order
+     * @return int
+     */
+    private function getReadmesCriteria($crReadmes, $start, $limit, $sort, $order)
+    {
+        $crReadmes->setStart($start);
+        $crReadmes->setLimit($limit);
+        $crReadmes->setSort($sort);
+        $crReadmes->setOrder($order);
+        return $crReadmes;
+    }
 
     /**
      * Update table requests