Inspection of "next steps Milo implementation" - ggoffy/wggithub - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 8005c7...9b803d )

by Goffy

created 2020-12-22 11:29 UTC

Status

Indentation +10 added lines, -10 removed lines patch added patch discarded remove patch

@@ -50,26 +50,26 @@
 block discarded – undo
 // Upload Folders
 $configurator = new Common\Configurator();
 if ($configurator->uploadFolders && \is_array($configurator->uploadFolders)) {
-	foreach (\array_keys($configurator->uploadFolders) as $i) {
-		$folder[] = $configurator->uploadFolders[$i];
-	}
+    foreach (\array_keys($configurator->uploadFolders) as $i) {
+        $folder[] = $configurator->uploadFolders[$i];
+    }
 }
 // Uploads Folders Created
 foreach (\array_keys($folder) as $i) {
-	$adminObject->addConfigBoxLine($folder[$i], 'folder');
-	$adminObject->addConfigBoxLine(array($folder[$i], '777'), 'chmod');
+    $adminObject->addConfigBoxLine($folder[$i], 'folder');
+    $adminObject->addConfigBoxLine(array($folder[$i], '777'), 'chmod');
 }
 
 // Render Index
 $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('index.php'));
 // Test Data
 if ($helper->getConfig('displaySampleButton')) {
-	\xoops_loadLanguage('admin/modulesadmin', 'system');
-	include_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 . '_SAVE_SAMPLEDATA'), '__DIR__ . /../../testdata/index.php?op=save', 'add');
+    \xoops_loadLanguage('admin/modulesadmin', 'system');
+    include_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 . '_SAVE_SAMPLEDATA'), '__DIR__ . /../../testdata/index.php?op=save', 'add');
 //	$adminObject->addItemButton(\constant('CO_' . $moduleDirNameUpper . '_EXPORT_SCHEMA'), '__DIR__ . /../../testdata/index.php?op=exportschema', 'add');
-	$adminObject->displayButton('left');
+    $adminObject->displayButton('left');
 }
 $GLOBALS['xoopsTpl']->assign('index', $adminObject->displayIndex());
 // End Test Data

Please login to merge, or discard this patch.

admin/footer.php 1 patch

Indentation +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -20,8 +20,8 @@
 block discarded – undo
  * @author         Goffy - XOOPS Development Team - Email:<[email protected]> - Website:<https://wedega.com>
  */
 if (isset($templateMain)) {
-	$GLOBALS['xoopsTpl']->assign('maintainedby', $helper->getConfig('maintainedby'));
-	$GLOBALS['xoopsTpl']->display("db:{$templateMain}");
+    $GLOBALS['xoopsTpl']->assign('maintainedby', $helper->getConfig('maintainedby'));
+    $GLOBALS['xoopsTpl']->display("db:{$templateMain}");
 }
 
 xoops_cp_footer();

Please login to merge, or discard this patch.

admin/releases.php 1 patch

Indentation +106 added lines, -106 removed lines patch added patch discarded remove patch

@@ -31,111 +31,111 @@
 block discarded – undo
 // Request rel_id
 $relId = Request::getInt('rel_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_releases.tpl';
-		$GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('releases.php'));
-		$adminObject->addItemButton(_AM_WGGITHUB_ADD_RELEASE, 'releases.php?op=new', 'add');
-		$GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
-		$releasesCount = $releasesHandler->getCountReleases();
-		$releasesAll = $releasesHandler->getAllReleases($start, $limit);
-		$GLOBALS['xoopsTpl']->assign('releases_count', $releasesCount);
-		$GLOBALS['xoopsTpl']->assign('wggithub_url', WGGITHUB_URL);
-		$GLOBALS['xoopsTpl']->assign('wggithub_upload_url', WGGITHUB_UPLOAD_URL);
-		// Table view releases
-		if ($releasesCount > 0) {
-			foreach (\array_keys($releasesAll) as $i) {
-				$release = $releasesAll[$i]->getValuesReleases();
-				$GLOBALS['xoopsTpl']->append('releases_list', $release);
-				unset($release);
-			}
-			// Display Navigation
-			if ($releasesCount > $limit) {
-				include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
-				$pagenav = new \XoopsPageNav($releasesCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
-				$GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
-			}
-		} else {
-			$GLOBALS['xoopsTpl']->assign('error', _AM_WGGITHUB_THEREARENT_RELEASES);
-		}
-		break;
-	case 'new':
-		$templateMain = 'wggithub_admin_releases.tpl';
-		$GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('releases.php'));
-		$adminObject->addItemButton(_AM_WGGITHUB_RELEASES_LIST, 'releases.php', 'list');
-		$GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
-		// Form Create
-		$releasesObj = $releasesHandler->create();
-		$form = $releasesObj->getFormReleases();
-		$GLOBALS['xoopsTpl']->assign('form', $form->render());
-		break;
-	case 'save':
-		// Security Check
-		if (!$GLOBALS['xoopsSecurity']->check()) {
-			\redirect_header('releases.php', 3, \implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
-		}
-		if ($relId > 0) {
-			$releasesObj = $releasesHandler->get($relId);
-		} else {
-			$releasesObj = $releasesHandler->create();
-		}
-		// Set Vars
-		$releasesObj->setVar('rel_repoid', Request::getInt('rel_repoid', 0));
-		$releasesObj->setVar('rel_type', Request::getInt('rel_type', 0));
-		$releasesObj->setVar('rel_name', Request::getString('rel_name', ''));
-		$releasePublishedatObj = \DateTime::createFromFormat(_SHORTDATESTRING, Request::getString('rel_publishedat'));
-		$releasesObj->setVar('rel_publishedat', $releasePublishedatObj->getTimestamp());
-		$releasesObj->setVar('rel_tarballurl', Request::getString('rel_tarballurl', ''));
-		$releasesObj->setVar('rel_zipballurl', Request::getString('rel_zipballurl', ''));
-		$releaseDatecreatedObj = \DateTime::createFromFormat(_SHORTDATESTRING, Request::getString('rel_datecreated'));
-		$releasesObj->setVar('rel_datecreated', $releaseDatecreatedObj->getTimestamp());
-		$releasesObj->setVar('rel_submitter', Request::getInt('rel_submitter', 0));
-		// Insert Data
-		if ($releasesHandler->insert($releasesObj)) {
-			\redirect_header('releases.php?op=list', 2, _AM_WGGITHUB_FORM_OK);
-		}
-		// Get Form
-		$GLOBALS['xoopsTpl']->assign('error', $releasesObj->getHtmlErrors());
-		$form = $releasesObj->getFormReleases();
-		$GLOBALS['xoopsTpl']->assign('form', $form->render());
-		break;
-	case 'edit':
-		$templateMain = 'wggithub_admin_releases.tpl';
-		$GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('releases.php'));
-		$adminObject->addItemButton(_AM_WGGITHUB_ADD_RELEASE, 'releases.php?op=new', 'add');
-		$adminObject->addItemButton(_AM_WGGITHUB_RELEASES_LIST, 'releases.php', 'list');
-		$GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
-		// Get Form
-		$releasesObj = $releasesHandler->get($relId);
-		$form = $releasesObj->getFormReleases();
-		$GLOBALS['xoopsTpl']->assign('form', $form->render());
-		break;
-	case 'delete':
-		$templateMain = 'wggithub_admin_releases.tpl';
-		$GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('releases.php'));
-		$releasesObj = $releasesHandler->get($relId);
-		$relName = $releasesObj->getVar('rel_name');
-		if (isset($_REQUEST['ok']) && 1 == $_REQUEST['ok']) {
-			if (!$GLOBALS['xoopsSecurity']->check()) {
-				\redirect_header('releases.php', 3, \implode(', ', $GLOBALS['xoopsSecurity']->getErrors()));
-			}
-			if ($releasesHandler->delete($releasesObj)) {
-				\redirect_header('releases.php', 3, _AM_WGGITHUB_FORM_DELETE_OK);
-			} else {
-				$GLOBALS['xoopsTpl']->assign('error', $releasesObj->getHtmlErrors());
-			}
-		} else {
-			$xoopsconfirm = new Common\XoopsConfirm(
-				['ok' => 1, 'rel_id' => $relId, 'op' => 'delete'],
-				$_SERVER['REQUEST_URI'],
-				\sprintf(_AM_WGGITHUB_FORM_SURE_DELETE, $releasesObj->getVar('rel_name')));
-			$form = $xoopsconfirm->getFormXoopsConfirm();
-			$GLOBALS['xoopsTpl']->assign('form', $form->render());
-		}
-		break;
+    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_releases.tpl';
+        $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('releases.php'));
+        $adminObject->addItemButton(_AM_WGGITHUB_ADD_RELEASE, 'releases.php?op=new', 'add');
+        $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
+        $releasesCount = $releasesHandler->getCountReleases();
+        $releasesAll = $releasesHandler->getAllReleases($start, $limit);
+        $GLOBALS['xoopsTpl']->assign('releases_count', $releasesCount);
+        $GLOBALS['xoopsTpl']->assign('wggithub_url', WGGITHUB_URL);
+        $GLOBALS['xoopsTpl']->assign('wggithub_upload_url', WGGITHUB_UPLOAD_URL);
+        // Table view releases
+        if ($releasesCount > 0) {
+            foreach (\array_keys($releasesAll) as $i) {
+                $release = $releasesAll[$i]->getValuesReleases();
+                $GLOBALS['xoopsTpl']->append('releases_list', $release);
+                unset($release);
+            }
+            // Display Navigation
+            if ($releasesCount > $limit) {
+                include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
+                $pagenav = new \XoopsPageNav($releasesCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
+                $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
+            }
+        } else {
+            $GLOBALS['xoopsTpl']->assign('error', _AM_WGGITHUB_THEREARENT_RELEASES);
+        }
+        break;
+    case 'new':
+        $templateMain = 'wggithub_admin_releases.tpl';
+        $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('releases.php'));
+        $adminObject->addItemButton(_AM_WGGITHUB_RELEASES_LIST, 'releases.php', 'list');
+        $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
+        // Form Create
+        $releasesObj = $releasesHandler->create();
+        $form = $releasesObj->getFormReleases();
+        $GLOBALS['xoopsTpl']->assign('form', $form->render());
+        break;
+    case 'save':
+        // Security Check
+        if (!$GLOBALS['xoopsSecurity']->check()) {
+            \redirect_header('releases.php', 3, \implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
+        }
+        if ($relId > 0) {
+            $releasesObj = $releasesHandler->get($relId);
+        } else {
+            $releasesObj = $releasesHandler->create();
+        }
+        // Set Vars
+        $releasesObj->setVar('rel_repoid', Request::getInt('rel_repoid', 0));
+        $releasesObj->setVar('rel_type', Request::getInt('rel_type', 0));
+        $releasesObj->setVar('rel_name', Request::getString('rel_name', ''));
+        $releasePublishedatObj = \DateTime::createFromFormat(_SHORTDATESTRING, Request::getString('rel_publishedat'));
+        $releasesObj->setVar('rel_publishedat', $releasePublishedatObj->getTimestamp());
+        $releasesObj->setVar('rel_tarballurl', Request::getString('rel_tarballurl', ''));
+        $releasesObj->setVar('rel_zipballurl', Request::getString('rel_zipballurl', ''));
+        $releaseDatecreatedObj = \DateTime::createFromFormat(_SHORTDATESTRING, Request::getString('rel_datecreated'));
+        $releasesObj->setVar('rel_datecreated', $releaseDatecreatedObj->getTimestamp());
+        $releasesObj->setVar('rel_submitter', Request::getInt('rel_submitter', 0));
+        // Insert Data
+        if ($releasesHandler->insert($releasesObj)) {
+            \redirect_header('releases.php?op=list', 2, _AM_WGGITHUB_FORM_OK);
+        }
+        // Get Form
+        $GLOBALS['xoopsTpl']->assign('error', $releasesObj->getHtmlErrors());
+        $form = $releasesObj->getFormReleases();
+        $GLOBALS['xoopsTpl']->assign('form', $form->render());
+        break;
+    case 'edit':
+        $templateMain = 'wggithub_admin_releases.tpl';
+        $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('releases.php'));
+        $adminObject->addItemButton(_AM_WGGITHUB_ADD_RELEASE, 'releases.php?op=new', 'add');
+        $adminObject->addItemButton(_AM_WGGITHUB_RELEASES_LIST, 'releases.php', 'list');
+        $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
+        // Get Form
+        $releasesObj = $releasesHandler->get($relId);
+        $form = $releasesObj->getFormReleases();
+        $GLOBALS['xoopsTpl']->assign('form', $form->render());
+        break;
+    case 'delete':
+        $templateMain = 'wggithub_admin_releases.tpl';
+        $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('releases.php'));
+        $releasesObj = $releasesHandler->get($relId);
+        $relName = $releasesObj->getVar('rel_name');
+        if (isset($_REQUEST['ok']) && 1 == $_REQUEST['ok']) {
+            if (!$GLOBALS['xoopsSecurity']->check()) {
+                \redirect_header('releases.php', 3, \implode(', ', $GLOBALS['xoopsSecurity']->getErrors()));
+            }
+            if ($releasesHandler->delete($releasesObj)) {
+                \redirect_header('releases.php', 3, _AM_WGGITHUB_FORM_DELETE_OK);
+            } else {
+                $GLOBALS['xoopsTpl']->assign('error', $releasesObj->getHtmlErrors());
+            }
+        } else {
+            $xoopsconfirm = new Common\XoopsConfirm(
+                ['ok' => 1, 'rel_id' => $relId, 'op' => 'delete'],
+                $_SERVER['REQUEST_URI'],
+                \sprintf(_AM_WGGITHUB_FORM_SURE_DELETE, $releasesObj->getVar('rel_name')));
+            $form = $xoopsconfirm->getFormXoopsConfirm();
+            $GLOBALS['xoopsTpl']->assign('form', $form->render());
+        }
+        break;
 }
 require __DIR__ . '/footer.php';

Please login to merge, or discard this patch.

admin/header.php 1 patch

Indentation +4 added lines, -4 removed lines patch added patch discarded remove patch

@@ -40,8 +40,8 @@  discard block
 block discarded – undo
 $myts = MyTextSanitizer::getInstance();
 // 
 if (!isset($xoopsTpl) || !\is_object($xoopsTpl)) {
-	include_once XOOPS_ROOT_PATH . '/class/template.php';
-	$xoopsTpl = new \XoopsTpl();
+    include_once XOOPS_ROOT_PATH . '/class/template.php';
+    $xoopsTpl = new \XoopsTpl();
 }
 
 // Load languages
@@ -50,9 +50,9 @@  discard block
 block discarded – undo
 
 // Local admin menu class
 if (\file_exists($GLOBALS['xoops']->path($pathModuleAdmin.'/moduleadmin.php'))) {
-	include_once $GLOBALS['xoops']->path($pathModuleAdmin.'/moduleadmin.php');
+    include_once $GLOBALS['xoops']->path($pathModuleAdmin.'/moduleadmin.php');
 } else {
-	\redirect_header('../../../admin.php', 5, _AM_MODULEADMIN_MISSING);
+    \redirect_header('../../../admin.php', 5, _AM_MODULEADMIN_MISSING);
 }
 
 xoops_cp_header();

Please login to merge, or discard this patch.

admin/readmes.php 1 patch

Indentation +105 added lines, -105 removed lines patch added patch discarded remove patch

@@ -31,110 +31,110 @@
 block discarded – undo
 // Request rm_id
 $rmId = Request::getInt('rm_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_readmes.tpl';
-		$GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('readmes.php'));
-		$adminObject->addItemButton(_AM_WGGITHUB_ADD_README, 'readmes.php?op=new', 'add');
-		$GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
-		$readmesCount = $readmesHandler->getCountReadmes();
-		$readmesAll = $readmesHandler->getAllReadmes($start, $limit);
-		$GLOBALS['xoopsTpl']->assign('readmes_count', $readmesCount);
-		$GLOBALS['xoopsTpl']->assign('wggithub_url', WGGITHUB_URL);
-		$GLOBALS['xoopsTpl']->assign('wggithub_upload_url', WGGITHUB_UPLOAD_URL);
-		// Table view readmes
-		if ($readmesCount > 0) {
-			foreach (\array_keys($readmesAll) as $i) {
-				$readme = $readmesAll[$i]->getValuesReadmes();
-				$GLOBALS['xoopsTpl']->append('readmes_list', $readme);
-				unset($readme);
-			}
-			// Display Navigation
-			if ($readmesCount > $limit) {
-				include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
-				$pagenav = new \XoopsPageNav($readmesCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
-				$GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
-			}
-		} else {
-			$GLOBALS['xoopsTpl']->assign('error', _AM_WGGITHUB_THEREARENT_READMES);
-		}
-		break;
-	case 'new':
-		$templateMain = 'wggithub_admin_readmes.tpl';
-		$GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('readmes.php'));
-		$adminObject->addItemButton(_AM_WGGITHUB_READMES_LIST, 'readmes.php', 'list');
-		$GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
-		// Form Create
-		$readmesObj = $readmesHandler->create();
-		$form = $readmesObj->getFormReadmes();
-		$GLOBALS['xoopsTpl']->assign('form', $form->render());
-		break;
-	case 'save':
-		// Security Check
-		if (!$GLOBALS['xoopsSecurity']->check()) {
-			\redirect_header('readmes.php', 3, \implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
-		}
-		if ($rmId > 0) {
-			$readmesObj = $readmesHandler->get($rmId);
-		} else {
-			$readmesObj = $readmesHandler->create();
-		}
-		// Set Vars
-		$readmesObj->setVar('rm_repoid', Request::getInt('rm_repoid', 0));
-		$readmesObj->setVar('rm_name', Request::getString('rm_name', ''));
-		$readmesObj->setVar('rm_type', Request::getString('rm_type', ''));
-		$readmesObj->setVar('rm_content', Request::getString('rm_content', ''));
-		$readmesObj->setVar('rm_encoding', Request::getString('rm_encoding', ''));
-		$readmesObj->setVar('rm_downloadurl', Request::getString('rm_downloadurl', ''));
-		$readmeDatecreatedObj = \DateTime::createFromFormat(_SHORTDATESTRING, Request::getString('rm_datecreated'));
-		$readmesObj->setVar('rm_datecreated', $readmeDatecreatedObj->getTimestamp());
-		$readmesObj->setVar('rm_submitter', Request::getInt('rm_submitter', 0));
-		// Insert Data
-		if ($readmesHandler->insert($readmesObj)) {
-			\redirect_header('readmes.php?op=list', 2, _AM_WGGITHUB_FORM_OK);
-		}
-		// Get Form
-		$GLOBALS['xoopsTpl']->assign('error', $readmesObj->getHtmlErrors());
-		$form = $readmesObj->getFormReadmes();
-		$GLOBALS['xoopsTpl']->assign('form', $form->render());
-		break;
-	case 'edit':
-		$templateMain = 'wggithub_admin_readmes.tpl';
-		$GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('readmes.php'));
-		$adminObject->addItemButton(_AM_WGGITHUB_ADD_README, 'readmes.php?op=new', 'add');
-		$adminObject->addItemButton(_AM_WGGITHUB_READMES_LIST, 'readmes.php', 'list');
-		$GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
-		// Get Form
-		$readmesObj = $readmesHandler->get($rmId);
-		$form = $readmesObj->getFormReadmes();
-		$GLOBALS['xoopsTpl']->assign('form', $form->render());
-		break;
-	case 'delete':
-		$templateMain = 'wggithub_admin_readmes.tpl';
-		$GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('readmes.php'));
-		$readmesObj = $readmesHandler->get($rmId);
-		$rmName = $readmesObj->getVar('rm_name');
-		if (isset($_REQUEST['ok']) && 1 == $_REQUEST['ok']) {
-			if (!$GLOBALS['xoopsSecurity']->check()) {
-				\redirect_header('readmes.php', 3, \implode(', ', $GLOBALS['xoopsSecurity']->getErrors()));
-			}
-			if ($readmesHandler->delete($readmesObj)) {
-				\redirect_header('readmes.php', 3, _AM_WGGITHUB_FORM_DELETE_OK);
-			} else {
-				$GLOBALS['xoopsTpl']->assign('error', $readmesObj->getHtmlErrors());
-			}
-		} else {
-			$xoopsconfirm = new Common\XoopsConfirm(
-				['ok' => 1, 'rm_id' => $rmId, 'op' => 'delete'],
-				$_SERVER['REQUEST_URI'],
-				\sprintf(_AM_WGGITHUB_FORM_SURE_DELETE, $readmesObj->getVar('rm_name')));
-			$form = $xoopsconfirm->getFormXoopsConfirm();
-			$GLOBALS['xoopsTpl']->assign('form', $form->render());
-		}
-		break;
+    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_readmes.tpl';
+        $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('readmes.php'));
+        $adminObject->addItemButton(_AM_WGGITHUB_ADD_README, 'readmes.php?op=new', 'add');
+        $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
+        $readmesCount = $readmesHandler->getCountReadmes();
+        $readmesAll = $readmesHandler->getAllReadmes($start, $limit);
+        $GLOBALS['xoopsTpl']->assign('readmes_count', $readmesCount);
+        $GLOBALS['xoopsTpl']->assign('wggithub_url', WGGITHUB_URL);
+        $GLOBALS['xoopsTpl']->assign('wggithub_upload_url', WGGITHUB_UPLOAD_URL);
+        // Table view readmes
+        if ($readmesCount > 0) {
+            foreach (\array_keys($readmesAll) as $i) {
+                $readme = $readmesAll[$i]->getValuesReadmes();
+                $GLOBALS['xoopsTpl']->append('readmes_list', $readme);
+                unset($readme);
+            }
+            // Display Navigation
+            if ($readmesCount > $limit) {
+                include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
+                $pagenav = new \XoopsPageNav($readmesCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
+                $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
+            }
+        } else {
+            $GLOBALS['xoopsTpl']->assign('error', _AM_WGGITHUB_THEREARENT_READMES);
+        }
+        break;
+    case 'new':
+        $templateMain = 'wggithub_admin_readmes.tpl';
+        $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('readmes.php'));
+        $adminObject->addItemButton(_AM_WGGITHUB_READMES_LIST, 'readmes.php', 'list');
+        $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
+        // Form Create
+        $readmesObj = $readmesHandler->create();
+        $form = $readmesObj->getFormReadmes();
+        $GLOBALS['xoopsTpl']->assign('form', $form->render());
+        break;
+    case 'save':
+        // Security Check
+        if (!$GLOBALS['xoopsSecurity']->check()) {
+            \redirect_header('readmes.php', 3, \implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
+        }
+        if ($rmId > 0) {
+            $readmesObj = $readmesHandler->get($rmId);
+        } else {
+            $readmesObj = $readmesHandler->create();
+        }
+        // Set Vars
+        $readmesObj->setVar('rm_repoid', Request::getInt('rm_repoid', 0));
+        $readmesObj->setVar('rm_name', Request::getString('rm_name', ''));
+        $readmesObj->setVar('rm_type', Request::getString('rm_type', ''));
+        $readmesObj->setVar('rm_content', Request::getString('rm_content', ''));
+        $readmesObj->setVar('rm_encoding', Request::getString('rm_encoding', ''));
+        $readmesObj->setVar('rm_downloadurl', Request::getString('rm_downloadurl', ''));
+        $readmeDatecreatedObj = \DateTime::createFromFormat(_SHORTDATESTRING, Request::getString('rm_datecreated'));
+        $readmesObj->setVar('rm_datecreated', $readmeDatecreatedObj->getTimestamp());
+        $readmesObj->setVar('rm_submitter', Request::getInt('rm_submitter', 0));
+        // Insert Data
+        if ($readmesHandler->insert($readmesObj)) {
+            \redirect_header('readmes.php?op=list', 2, _AM_WGGITHUB_FORM_OK);
+        }
+        // Get Form
+        $GLOBALS['xoopsTpl']->assign('error', $readmesObj->getHtmlErrors());
+        $form = $readmesObj->getFormReadmes();
+        $GLOBALS['xoopsTpl']->assign('form', $form->render());
+        break;
+    case 'edit':
+        $templateMain = 'wggithub_admin_readmes.tpl';
+        $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('readmes.php'));
+        $adminObject->addItemButton(_AM_WGGITHUB_ADD_README, 'readmes.php?op=new', 'add');
+        $adminObject->addItemButton(_AM_WGGITHUB_READMES_LIST, 'readmes.php', 'list');
+        $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
+        // Get Form
+        $readmesObj = $readmesHandler->get($rmId);
+        $form = $readmesObj->getFormReadmes();
+        $GLOBALS['xoopsTpl']->assign('form', $form->render());
+        break;
+    case 'delete':
+        $templateMain = 'wggithub_admin_readmes.tpl';
+        $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('readmes.php'));
+        $readmesObj = $readmesHandler->get($rmId);
+        $rmName = $readmesObj->getVar('rm_name');
+        if (isset($_REQUEST['ok']) && 1 == $_REQUEST['ok']) {
+            if (!$GLOBALS['xoopsSecurity']->check()) {
+                \redirect_header('readmes.php', 3, \implode(', ', $GLOBALS['xoopsSecurity']->getErrors()));
+            }
+            if ($readmesHandler->delete($readmesObj)) {
+                \redirect_header('readmes.php', 3, _AM_WGGITHUB_FORM_DELETE_OK);
+            } else {
+                $GLOBALS['xoopsTpl']->assign('error', $readmesObj->getHtmlErrors());
+            }
+        } else {
+            $xoopsconfirm = new Common\XoopsConfirm(
+                ['ok' => 1, 'rm_id' => $rmId, 'op' => 'delete'],
+                $_SERVER['REQUEST_URI'],
+                \sprintf(_AM_WGGITHUB_FORM_SURE_DELETE, $readmesObj->getVar('rm_name')));
+            $form = $xoopsconfirm->getFormXoopsConfirm();
+            $GLOBALS['xoopsTpl']->assign('form', $form->render());
+        }
+        break;
 }
 require __DIR__ . '/footer.php';

Please login to merge, or discard this patch.

include/search.inc.php 1 patch

Indentation +50 added lines, -50 removed lines patch added patch discarded remove patch

@@ -35,60 +35,60 @@
 block discarded – undo
  */
 function wggithub_search($queryarray, $andor, $limit, $offset, $userid)
 {
-	$ret = [];
-	$helper = \XoopsModules\Wggithub\Helper::getInstance();
+    $ret = [];
+    $helper = \XoopsModules\Wggithub\Helper::getInstance();
 
-	// search in table repositories
-	// search keywords
-	$elementCount = 0;
-	$repositoriesHandler = $helper->getHandler('Repositories');
-	if (\is_array($queryarray)) {
-		$elementCount = \count($queryarray);
-	}
-	if ($elementCount > 0) {
-		$crKeywords = new \CriteriaCompo();
-		for ($i = 0; $i  <  $elementCount; $i++) {
-			$crKeyword = new \CriteriaCompo();
-			unset($crKeyword);
-		}
-	}
-	// search user(s)
-	if ($userid && \is_array($userid)) {
-		$userid = array_map('intval', $userid);
-		$crUser = new \CriteriaCompo();
-		$crUser->add(new \Criteria('repo_submitter', '(' . \implode(',', $userid) . ')', 'IN'), 'OR');
-	} elseif (is_numeric($userid) && $userid > 0) {
-		$crUser = new \CriteriaCompo();
-		$crUser->add(new \Criteria('repo_submitter', $userid), 'OR');
-	}
-	$crSearch = new \CriteriaCompo();
-	if (isset($crKeywords)) {
-		$crSearch->add($crKeywords, 'AND');
-	}
-	if (isset($crUser)) {
-		$crSearch->add($crUser, 'AND');
-	}
-	$crSearch->setStart($offset);
-	$crSearch->setLimit($limit);
-	$crSearch->setSort('repo_datecreated');
-	$crSearch->setOrder('DESC');
-	$repositoriesAll = $repositoriesHandler->getAll($crSearch);
-	foreach (\array_keys($repositoriesAll) as $i) {
-		$ret[] = [
-			'image'  => 'assets/icons/16/repositories.png',
-			'link'   => 'repositories.php?op=show&amp;repo_id=' . $repositoriesAll[$i]->getVar('repo_id'),
-			'title'  => $repositoriesAll[$i]->getVar('repo_name'),
-			'time'   => $repositoriesAll[$i]->getVar('repo_datecreated')
-		];
-	}
-	unset($crKeywords);
-	unset($crKeyword);
-	unset($crUser);
-	unset($crSearch);
+    // search in table repositories
+    // search keywords
+    $elementCount = 0;
+    $repositoriesHandler = $helper->getHandler('Repositories');
+    if (\is_array($queryarray)) {
+        $elementCount = \count($queryarray);
+    }
+    if ($elementCount > 0) {
+        $crKeywords = new \CriteriaCompo();
+        for ($i = 0; $i  <  $elementCount; $i++) {
+            $crKeyword = new \CriteriaCompo();
+            unset($crKeyword);
+        }
+    }
+    // search user(s)
+    if ($userid && \is_array($userid)) {
+        $userid = array_map('intval', $userid);
+        $crUser = new \CriteriaCompo();
+        $crUser->add(new \Criteria('repo_submitter', '(' . \implode(',', $userid) . ')', 'IN'), 'OR');
+    } elseif (is_numeric($userid) && $userid > 0) {
+        $crUser = new \CriteriaCompo();
+        $crUser->add(new \Criteria('repo_submitter', $userid), 'OR');
+    }
+    $crSearch = new \CriteriaCompo();
+    if (isset($crKeywords)) {
+        $crSearch->add($crKeywords, 'AND');
+    }
+    if (isset($crUser)) {
+        $crSearch->add($crUser, 'AND');
+    }
+    $crSearch->setStart($offset);
+    $crSearch->setLimit($limit);
+    $crSearch->setSort('repo_datecreated');
+    $crSearch->setOrder('DESC');
+    $repositoriesAll = $repositoriesHandler->getAll($crSearch);
+    foreach (\array_keys($repositoriesAll) as $i) {
+        $ret[] = [
+            'image'  => 'assets/icons/16/repositories.png',
+            'link'   => 'repositories.php?op=show&amp;repo_id=' . $repositoriesAll[$i]->getVar('repo_id'),
+            'title'  => $repositoriesAll[$i]->getVar('repo_name'),
+            'time'   => $repositoriesAll[$i]->getVar('repo_datecreated')
+        ];
+    }
+    unset($crKeywords);
+    unset($crKeyword);
+    unset($crUser);
+    unset($crSearch);
 
 
 
 
-	return $ret;
+    return $ret;
 
 }

Please login to merge, or discard this patch.

include/update.php 1 patch

Indentation +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -39,7 +39,7 @@  discard block
 block discarded – undo
     $ret = wggithub_check_db($module);
 
     //check upload directory
-	include_once __DIR__ . '/install.php';
+    include_once __DIR__ . '/install.php';
     $ret = xoops_module_install_wggithub($module);
 
     $errors = $module->getErrors();
@@ -113,7 +113,7 @@  discard block
 block discarded – undo
 function wggithub_check_db($module)
 {
     $ret = true;
-	//insert here code for database check
+    //insert here code for database check
 
     // Example: update table (add new field)
     $table   = $GLOBALS['xoopsDB']->prefix('wggithub_repositories');

Please login to merge, or discard this patch.

include/notification.inc.php 1 patch

Indentation +20 added lines, -20 removed lines patch added patch discarded remove patch

@@ -29,26 +29,26 @@
 block discarded – undo
  */
 function wggithub_notify_iteminfo($category, $item_id)
 {
-	global $xoopsDB;
+    global $xoopsDB;
 
-	if (!\defined('WGGITHUB_URL')) {
-		\define('WGGITHUB_URL', \XOOPS_URL . '/modules/wggithub');
-	}
+    if (!\defined('WGGITHUB_URL')) {
+        \define('WGGITHUB_URL', \XOOPS_URL . '/modules/wggithub');
+    }
 
-	switch ($category) {
-		case 'global':
-			$item['name'] = '';
-			$item['url']  = '';
-			return $item;
-			break;
-		case 'repositories':
-			$sql          = 'SELECT repo_name FROM ' . $xoopsDB->prefix('wggithub_repositories') . ' WHERE repo_id = '. $item_id;
-			$result       = $xoopsDB->query($sql);
-			$result_array = $xoopsDB->fetchArray($result);
-			$item['name'] = $result_array['repo_name'];
-			$item['url']  = WGGITHUB_URL . '/repositories.php?repo_id=' . $item_id;
-			return $item;
-			break;
-	}
-	return null;
+    switch ($category) {
+        case 'global':
+            $item['name'] = '';
+            $item['url']  = '';
+            return $item;
+            break;
+        case 'repositories':
+            $sql          = 'SELECT repo_name FROM ' . $xoopsDB->prefix('wggithub_repositories') . ' WHERE repo_id = '. $item_id;
+            $result       = $xoopsDB->query($sql);
+            $result_array = $xoopsDB->fetchArray($result);
+            $item['name'] = $result_array['repo_name'];
+            $item['url']  = WGGITHUB_URL . '/repositories.php?repo_id=' . $item_id;
+            return $item;
+            break;
+    }
+    return null;
 }

Please login to merge, or discard this patch.

include/install.php 1 patch

Indentation +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -84,7 +84,7 @@
 block discarded – undo
             $dest = $configurator->copyBlankFiles[$i] . '/blank.gif';
             $utility::copyFile($file, $dest);
         }
-		$file = \dirname(__DIR__) . '/assets/images/blank.png';
+        $file = \dirname(__DIR__) . '/assets/images/blank.png';
         foreach (\array_keys($configurator->copyBlankFiles) as $i) {
             $dest = $configurator->copyBlankFiles[$i] . '/blank.png';
             $utility::copyFile($file, $dest);

Please login to merge, or discard this patch.

		@@ -50,26 +50,26 @@
		block discarded – undo
50	50	// Upload Folders
51	51	$configurator = new Common\Configurator();
52	52	if ($configurator->uploadFolders && \is_array($configurator->uploadFolders)) {
53		- foreach (\array_keys($configurator->uploadFolders) as $i) {
54		- $folder[] = $configurator->uploadFolders[$i];
55		- }
	53	+ foreach (\array_keys($configurator->uploadFolders) as $i) {
	54	+ $folder[] = $configurator->uploadFolders[$i];
	55	+ }
56	56	}
57	57	// Uploads Folders Created
58	58	foreach (\array_keys($folder) as $i) {
59		- $adminObject->addConfigBoxLine($folder[$i], 'folder');
60		- $adminObject->addConfigBoxLine(array($folder[$i], '777'), 'chmod');
	59	+ $adminObject->addConfigBoxLine($folder[$i], 'folder');
	60	+ $adminObject->addConfigBoxLine(array($folder[$i], '777'), 'chmod');
61	61	}
62	62
63	63	// Render Index
64	64	$GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('index.php'));
65	65	// Test Data
66	66	if ($helper->getConfig('displaySampleButton')) {
67		- \xoops_loadLanguage('admin/modulesadmin', 'system');
68		- include_once \dirname(__DIR__) . '/testdata/index.php';
69		- $adminObject->addItemButton(\constant('CO_' . $moduleDirNameUpper . '_ADD_SAMPLEDATA'), '__DIR__ . /../../testdata/index.php?op=load', 'add');
70		- $adminObject->addItemButton(\constant('CO_' . $moduleDirNameUpper . '_SAVE_SAMPLEDATA'), '__DIR__ . /../../testdata/index.php?op=save', 'add');
	67	+ \xoops_loadLanguage('admin/modulesadmin', 'system');
	68	+ include_once \dirname(__DIR__) . '/testdata/index.php';
	69	+ $adminObject->addItemButton(\constant('CO_' . $moduleDirNameUpper . '_ADD_SAMPLEDATA'), '__DIR__ . /../../testdata/index.php?op=load', 'add');
	70	+ $adminObject->addItemButton(\constant('CO_' . $moduleDirNameUpper . '_SAVE_SAMPLEDATA'), '__DIR__ . /../../testdata/index.php?op=save', 'add');
71	71	// $adminObject->addItemButton(\constant('CO_' . $moduleDirNameUpper . '_EXPORT_SCHEMA'), '__DIR__ . /../../testdata/index.php?op=exportschema', 'add');
72		- $adminObject->displayButton('left');
	72	+ $adminObject->displayButton('left');
73	73	}
74	74	$GLOBALS['xoopsTpl']->assign('index', $adminObject->displayIndex());
75	75	// End Test Data

		@@ -40,8 +40,8 @@ discard block
		block discarded – undo
40	40	$myts = MyTextSanitizer::getInstance();
41	41	//
42	42	if (!isset($xoopsTpl) \|\| !\is_object($xoopsTpl)) {
43		- include_once XOOPS_ROOT_PATH . '/class/template.php';
44		- $xoopsTpl = new \XoopsTpl();
	43	+ include_once XOOPS_ROOT_PATH . '/class/template.php';
	44	+ $xoopsTpl = new \XoopsTpl();
45	45	}
46	46
47	47	// Load languages
		@@ -50,9 +50,9 @@ discard block
		block discarded – undo
50	50
51	51	// Local admin menu class
52	52	if (\file_exists($GLOBALS['xoops']->path($pathModuleAdmin.'/moduleadmin.php'))) {
53		- include_once $GLOBALS['xoops']->path($pathModuleAdmin.'/moduleadmin.php');
	53	+ include_once $GLOBALS['xoops']->path($pathModuleAdmin.'/moduleadmin.php');
54	54	} else {
55		- \redirect_header('../../../admin.php', 5, _AM_MODULEADMIN_MISSING);
	55	+ \redirect_header('../../../admin.php', 5, _AM_MODULEADMIN_MISSING);
56	56	}
57	57
58	58	xoops_cp_header();

		@@ -84,7 +84,7 @@
		block discarded – undo
84	84	$dest = $configurator->copyBlankFiles[$i] . '/blank.gif';
85	85	$utility::copyFile($file, $dest);
86	86	}
87		- $file = \dirname(__DIR__) . '/assets/images/blank.png';
	87	+ $file = \dirname(__DIR__) . '/assets/images/blank.png';
88	88	foreach (\array_keys($configurator->copyBlankFiles) as $i) {
89	89	$dest = $configurator->copyBlankFiles[$i] . '/blank.png';
90	90	$utility::copyFile($file, $dest);

		@@ -20,8 +20,8 @@
		block discarded – undo
20	20	* @author Goffy - XOOPS Development Team - Email:<[email protected]> - Website:<https://wedega.com>
21	21	*/
22	22	if (isset($templateMain)) {
23		- $GLOBALS['xoopsTpl']->assign('maintainedby', $helper->getConfig('maintainedby'));
24		- $GLOBALS['xoopsTpl']->display("db:{$templateMain}");
	23	+ $GLOBALS['xoopsTpl']->assign('maintainedby', $helper->getConfig('maintainedby'));
	24	+ $GLOBALS['xoopsTpl']->display("db:{$templateMain}");
25	25	}
26	26
27	27	xoops_cp_footer();

		@@ -31,111 +31,111 @@
		block discarded – undo
31	31	// Request rel_id
32	32	$relId = Request::getInt('rel_id');
33	33	switch ($op) {
34		- case 'list':
35		- default:
36		- // Define Stylesheet
37		- $GLOBALS['xoTheme']->addStylesheet($style, null);
38		- $start = Request::getInt('start', 0);
39		- $limit = Request::getInt('limit', $helper->getConfig('adminpager'));
40		- $templateMain = 'wggithub_admin_releases.tpl';
41		- $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('releases.php'));
42		- $adminObject->addItemButton(_AM_WGGITHUB_ADD_RELEASE, 'releases.php?op=new', 'add');
43		- $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
44		- $releasesCount = $releasesHandler->getCountReleases();
45		- $releasesAll = $releasesHandler->getAllReleases($start, $limit);
46		- $GLOBALS['xoopsTpl']->assign('releases_count', $releasesCount);
47		- $GLOBALS['xoopsTpl']->assign('wggithub_url', WGGITHUB_URL);
48		- $GLOBALS['xoopsTpl']->assign('wggithub_upload_url', WGGITHUB_UPLOAD_URL);
49		- // Table view releases
50		- if ($releasesCount > 0) {
51		- foreach (\array_keys($releasesAll) as $i) {
52		- $release = $releasesAll[$i]->getValuesReleases();
53		- $GLOBALS['xoopsTpl']->append('releases_list', $release);
54		- unset($release);
55		- }
56		- // Display Navigation
57		- if ($releasesCount > $limit) {
58		- include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
59		- $pagenav = new \XoopsPageNav($releasesCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
60		- $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
61		- }
62		- } else {
63		- $GLOBALS['xoopsTpl']->assign('error', _AM_WGGITHUB_THEREARENT_RELEASES);
64		- }
65		- break;
66		- case 'new':
67		- $templateMain = 'wggithub_admin_releases.tpl';
68		- $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('releases.php'));
69		- $adminObject->addItemButton(_AM_WGGITHUB_RELEASES_LIST, 'releases.php', 'list');
70		- $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
71		- // Form Create
72		- $releasesObj = $releasesHandler->create();
73		- $form = $releasesObj->getFormReleases();
74		- $GLOBALS['xoopsTpl']->assign('form', $form->render());
75		- break;
76		- case 'save':
77		- // Security Check
78		- if (!$GLOBALS['xoopsSecurity']->check()) {
79		- \redirect_header('releases.php', 3, \implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
80		- }
81		- if ($relId > 0) {
82		- $releasesObj = $releasesHandler->get($relId);
83		- } else {
84		- $releasesObj = $releasesHandler->create();
85		- }
86		- // Set Vars
87		- $releasesObj->setVar('rel_repoid', Request::getInt('rel_repoid', 0));
88		- $releasesObj->setVar('rel_type', Request::getInt('rel_type', 0));
89		- $releasesObj->setVar('rel_name', Request::getString('rel_name', ''));
90		- $releasePublishedatObj = \DateTime::createFromFormat(_SHORTDATESTRING, Request::getString('rel_publishedat'));
91		- $releasesObj->setVar('rel_publishedat', $releasePublishedatObj->getTimestamp());
92		- $releasesObj->setVar('rel_tarballurl', Request::getString('rel_tarballurl', ''));
93		- $releasesObj->setVar('rel_zipballurl', Request::getString('rel_zipballurl', ''));
94		- $releaseDatecreatedObj = \DateTime::createFromFormat(_SHORTDATESTRING, Request::getString('rel_datecreated'));
95		- $releasesObj->setVar('rel_datecreated', $releaseDatecreatedObj->getTimestamp());
96		- $releasesObj->setVar('rel_submitter', Request::getInt('rel_submitter', 0));
97		- // Insert Data
98		- if ($releasesHandler->insert($releasesObj)) {
99		- \redirect_header('releases.php?op=list', 2, _AM_WGGITHUB_FORM_OK);
100		- }
101		- // Get Form
102		- $GLOBALS['xoopsTpl']->assign('error', $releasesObj->getHtmlErrors());
103		- $form = $releasesObj->getFormReleases();
104		- $GLOBALS['xoopsTpl']->assign('form', $form->render());
105		- break;
106		- case 'edit':
107		- $templateMain = 'wggithub_admin_releases.tpl';
108		- $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('releases.php'));
109		- $adminObject->addItemButton(_AM_WGGITHUB_ADD_RELEASE, 'releases.php?op=new', 'add');
110		- $adminObject->addItemButton(_AM_WGGITHUB_RELEASES_LIST, 'releases.php', 'list');
111		- $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
112		- // Get Form
113		- $releasesObj = $releasesHandler->get($relId);
114		- $form = $releasesObj->getFormReleases();
115		- $GLOBALS['xoopsTpl']->assign('form', $form->render());
116		- break;
117		- case 'delete':
118		- $templateMain = 'wggithub_admin_releases.tpl';
119		- $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('releases.php'));
120		- $releasesObj = $releasesHandler->get($relId);
121		- $relName = $releasesObj->getVar('rel_name');
122		- if (isset($_REQUEST['ok']) && 1 == $_REQUEST['ok']) {
123		- if (!$GLOBALS['xoopsSecurity']->check()) {
124		- \redirect_header('releases.php', 3, \implode(', ', $GLOBALS['xoopsSecurity']->getErrors()));
125		- }
126		- if ($releasesHandler->delete($releasesObj)) {
127		- \redirect_header('releases.php', 3, _AM_WGGITHUB_FORM_DELETE_OK);
128		- } else {
129		- $GLOBALS['xoopsTpl']->assign('error', $releasesObj->getHtmlErrors());
130		- }
131		- } else {
132		- $xoopsconfirm = new Common\XoopsConfirm(
133		- ['ok' => 1, 'rel_id' => $relId, 'op' => 'delete'],
134		- $_SERVER['REQUEST_URI'],
135		- \sprintf(_AM_WGGITHUB_FORM_SURE_DELETE, $releasesObj->getVar('rel_name')));
136		- $form = $xoopsconfirm->getFormXoopsConfirm();
137		- $GLOBALS['xoopsTpl']->assign('form', $form->render());
138		- }
139		- break;
	34	+ case 'list':
	35	+ default:
	36	+ // Define Stylesheet
	37	+ $GLOBALS['xoTheme']->addStylesheet($style, null);
	38	+ $start = Request::getInt('start', 0);
	39	+ $limit = Request::getInt('limit', $helper->getConfig('adminpager'));
	40	+ $templateMain = 'wggithub_admin_releases.tpl';
	41	+ $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('releases.php'));
	42	+ $adminObject->addItemButton(_AM_WGGITHUB_ADD_RELEASE, 'releases.php?op=new', 'add');
	43	+ $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
	44	+ $releasesCount = $releasesHandler->getCountReleases();
	45	+ $releasesAll = $releasesHandler->getAllReleases($start, $limit);
	46	+ $GLOBALS['xoopsTpl']->assign('releases_count', $releasesCount);
	47	+ $GLOBALS['xoopsTpl']->assign('wggithub_url', WGGITHUB_URL);
	48	+ $GLOBALS['xoopsTpl']->assign('wggithub_upload_url', WGGITHUB_UPLOAD_URL);
	49	+ // Table view releases
	50	+ if ($releasesCount > 0) {
	51	+ foreach (\array_keys($releasesAll) as $i) {
	52	+ $release = $releasesAll[$i]->getValuesReleases();
	53	+ $GLOBALS['xoopsTpl']->append('releases_list', $release);
	54	+ unset($release);
	55	+ }
	56	+ // Display Navigation
	57	+ if ($releasesCount > $limit) {
	58	+ include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
	59	+ $pagenav = new \XoopsPageNav($releasesCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
	60	+ $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
	61	+ }
	62	+ } else {
	63	+ $GLOBALS['xoopsTpl']->assign('error', _AM_WGGITHUB_THEREARENT_RELEASES);
	64	+ }
	65	+ break;
	66	+ case 'new':
	67	+ $templateMain = 'wggithub_admin_releases.tpl';
	68	+ $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('releases.php'));
	69	+ $adminObject->addItemButton(_AM_WGGITHUB_RELEASES_LIST, 'releases.php', 'list');
	70	+ $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
	71	+ // Form Create
	72	+ $releasesObj = $releasesHandler->create();
	73	+ $form = $releasesObj->getFormReleases();
	74	+ $GLOBALS['xoopsTpl']->assign('form', $form->render());
	75	+ break;
	76	+ case 'save':
	77	+ // Security Check
	78	+ if (!$GLOBALS['xoopsSecurity']->check()) {
	79	+ \redirect_header('releases.php', 3, \implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
	80	+ }
	81	+ if ($relId > 0) {
	82	+ $releasesObj = $releasesHandler->get($relId);
	83	+ } else {
	84	+ $releasesObj = $releasesHandler->create();
	85	+ }
	86	+ // Set Vars
	87	+ $releasesObj->setVar('rel_repoid', Request::getInt('rel_repoid', 0));
	88	+ $releasesObj->setVar('rel_type', Request::getInt('rel_type', 0));
	89	+ $releasesObj->setVar('rel_name', Request::getString('rel_name', ''));
	90	+ $releasePublishedatObj = \DateTime::createFromFormat(_SHORTDATESTRING, Request::getString('rel_publishedat'));
	91	+ $releasesObj->setVar('rel_publishedat', $releasePublishedatObj->getTimestamp());
	92	+ $releasesObj->setVar('rel_tarballurl', Request::getString('rel_tarballurl', ''));
	93	+ $releasesObj->setVar('rel_zipballurl', Request::getString('rel_zipballurl', ''));
	94	+ $releaseDatecreatedObj = \DateTime::createFromFormat(_SHORTDATESTRING, Request::getString('rel_datecreated'));
	95	+ $releasesObj->setVar('rel_datecreated', $releaseDatecreatedObj->getTimestamp());
	96	+ $releasesObj->setVar('rel_submitter', Request::getInt('rel_submitter', 0));
	97	+ // Insert Data
	98	+ if ($releasesHandler->insert($releasesObj)) {
	99	+ \redirect_header('releases.php?op=list', 2, _AM_WGGITHUB_FORM_OK);
	100	+ }
	101	+ // Get Form
	102	+ $GLOBALS['xoopsTpl']->assign('error', $releasesObj->getHtmlErrors());
	103	+ $form = $releasesObj->getFormReleases();
	104	+ $GLOBALS['xoopsTpl']->assign('form', $form->render());
	105	+ break;
	106	+ case 'edit':
	107	+ $templateMain = 'wggithub_admin_releases.tpl';
	108	+ $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('releases.php'));
	109	+ $adminObject->addItemButton(_AM_WGGITHUB_ADD_RELEASE, 'releases.php?op=new', 'add');
	110	+ $adminObject->addItemButton(_AM_WGGITHUB_RELEASES_LIST, 'releases.php', 'list');
	111	+ $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
	112	+ // Get Form
	113	+ $releasesObj = $releasesHandler->get($relId);
	114	+ $form = $releasesObj->getFormReleases();
	115	+ $GLOBALS['xoopsTpl']->assign('form', $form->render());
	116	+ break;
	117	+ case 'delete':
	118	+ $templateMain = 'wggithub_admin_releases.tpl';
	119	+ $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('releases.php'));
	120	+ $releasesObj = $releasesHandler->get($relId);
	121	+ $relName = $releasesObj->getVar('rel_name');
	122	+ if (isset($_REQUEST['ok']) && 1 == $_REQUEST['ok']) {
	123	+ if (!$GLOBALS['xoopsSecurity']->check()) {
	124	+ \redirect_header('releases.php', 3, \implode(', ', $GLOBALS['xoopsSecurity']->getErrors()));
	125	+ }
	126	+ if ($releasesHandler->delete($releasesObj)) {
	127	+ \redirect_header('releases.php', 3, _AM_WGGITHUB_FORM_DELETE_OK);
	128	+ } else {
	129	+ $GLOBALS['xoopsTpl']->assign('error', $releasesObj->getHtmlErrors());
	130	+ }
	131	+ } else {
	132	+ $xoopsconfirm = new Common\XoopsConfirm(
	133	+ ['ok' => 1, 'rel_id' => $relId, 'op' => 'delete'],
	134	+ $_SERVER['REQUEST_URI'],
	135	+ \sprintf(_AM_WGGITHUB_FORM_SURE_DELETE, $releasesObj->getVar('rel_name')));
	136	+ $form = $xoopsconfirm->getFormXoopsConfirm();
	137	+ $GLOBALS['xoopsTpl']->assign('form', $form->render());
	138	+ }
	139	+ break;
140	140	}
141	141	require __DIR__ . '/footer.php';

		@@ -31,110 +31,110 @@
		block discarded – undo
31	31	// Request rm_id
32	32	$rmId = Request::getInt('rm_id');
33	33	switch ($op) {
34		- case 'list':
35		- default:
36		- // Define Stylesheet
37		- $GLOBALS['xoTheme']->addStylesheet($style, null);
38		- $start = Request::getInt('start', 0);
39		- $limit = Request::getInt('limit', $helper->getConfig('adminpager'));
40		- $templateMain = 'wggithub_admin_readmes.tpl';
41		- $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('readmes.php'));
42		- $adminObject->addItemButton(_AM_WGGITHUB_ADD_README, 'readmes.php?op=new', 'add');
43		- $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
44		- $readmesCount = $readmesHandler->getCountReadmes();
45		- $readmesAll = $readmesHandler->getAllReadmes($start, $limit);
46		- $GLOBALS['xoopsTpl']->assign('readmes_count', $readmesCount);
47		- $GLOBALS['xoopsTpl']->assign('wggithub_url', WGGITHUB_URL);
48		- $GLOBALS['xoopsTpl']->assign('wggithub_upload_url', WGGITHUB_UPLOAD_URL);
49		- // Table view readmes
50		- if ($readmesCount > 0) {
51		- foreach (\array_keys($readmesAll) as $i) {
52		- $readme = $readmesAll[$i]->getValuesReadmes();
53		- $GLOBALS['xoopsTpl']->append('readmes_list', $readme);
54		- unset($readme);
55		- }
56		- // Display Navigation
57		- if ($readmesCount > $limit) {
58		- include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
59		- $pagenav = new \XoopsPageNav($readmesCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
60		- $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
61		- }
62		- } else {
63		- $GLOBALS['xoopsTpl']->assign('error', _AM_WGGITHUB_THEREARENT_READMES);
64		- }
65		- break;
66		- case 'new':
67		- $templateMain = 'wggithub_admin_readmes.tpl';
68		- $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('readmes.php'));
69		- $adminObject->addItemButton(_AM_WGGITHUB_READMES_LIST, 'readmes.php', 'list');
70		- $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
71		- // Form Create
72		- $readmesObj = $readmesHandler->create();
73		- $form = $readmesObj->getFormReadmes();
74		- $GLOBALS['xoopsTpl']->assign('form', $form->render());
75		- break;
76		- case 'save':
77		- // Security Check
78		- if (!$GLOBALS['xoopsSecurity']->check()) {
79		- \redirect_header('readmes.php', 3, \implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
80		- }
81		- if ($rmId > 0) {
82		- $readmesObj = $readmesHandler->get($rmId);
83		- } else {
84		- $readmesObj = $readmesHandler->create();
85		- }
86		- // Set Vars
87		- $readmesObj->setVar('rm_repoid', Request::getInt('rm_repoid', 0));
88		- $readmesObj->setVar('rm_name', Request::getString('rm_name', ''));
89		- $readmesObj->setVar('rm_type', Request::getString('rm_type', ''));
90		- $readmesObj->setVar('rm_content', Request::getString('rm_content', ''));
91		- $readmesObj->setVar('rm_encoding', Request::getString('rm_encoding', ''));
92		- $readmesObj->setVar('rm_downloadurl', Request::getString('rm_downloadurl', ''));
93		- $readmeDatecreatedObj = \DateTime::createFromFormat(_SHORTDATESTRING, Request::getString('rm_datecreated'));
94		- $readmesObj->setVar('rm_datecreated', $readmeDatecreatedObj->getTimestamp());
95		- $readmesObj->setVar('rm_submitter', Request::getInt('rm_submitter', 0));
96		- // Insert Data
97		- if ($readmesHandler->insert($readmesObj)) {
98		- \redirect_header('readmes.php?op=list', 2, _AM_WGGITHUB_FORM_OK);
99		- }
100		- // Get Form
101		- $GLOBALS['xoopsTpl']->assign('error', $readmesObj->getHtmlErrors());
102		- $form = $readmesObj->getFormReadmes();
103		- $GLOBALS['xoopsTpl']->assign('form', $form->render());
104		- break;
105		- case 'edit':
106		- $templateMain = 'wggithub_admin_readmes.tpl';
107		- $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('readmes.php'));
108		- $adminObject->addItemButton(_AM_WGGITHUB_ADD_README, 'readmes.php?op=new', 'add');
109		- $adminObject->addItemButton(_AM_WGGITHUB_READMES_LIST, 'readmes.php', 'list');
110		- $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
111		- // Get Form
112		- $readmesObj = $readmesHandler->get($rmId);
113		- $form = $readmesObj->getFormReadmes();
114		- $GLOBALS['xoopsTpl']->assign('form', $form->render());
115		- break;
116		- case 'delete':
117		- $templateMain = 'wggithub_admin_readmes.tpl';
118		- $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('readmes.php'));
119		- $readmesObj = $readmesHandler->get($rmId);
120		- $rmName = $readmesObj->getVar('rm_name');
121		- if (isset($_REQUEST['ok']) && 1 == $_REQUEST['ok']) {
122		- if (!$GLOBALS['xoopsSecurity']->check()) {
123		- \redirect_header('readmes.php', 3, \implode(', ', $GLOBALS['xoopsSecurity']->getErrors()));
124		- }
125		- if ($readmesHandler->delete($readmesObj)) {
126		- \redirect_header('readmes.php', 3, _AM_WGGITHUB_FORM_DELETE_OK);
127		- } else {
128		- $GLOBALS['xoopsTpl']->assign('error', $readmesObj->getHtmlErrors());
129		- }
130		- } else {
131		- $xoopsconfirm = new Common\XoopsConfirm(
132		- ['ok' => 1, 'rm_id' => $rmId, 'op' => 'delete'],
133		- $_SERVER['REQUEST_URI'],
134		- \sprintf(_AM_WGGITHUB_FORM_SURE_DELETE, $readmesObj->getVar('rm_name')));
135		- $form = $xoopsconfirm->getFormXoopsConfirm();
136		- $GLOBALS['xoopsTpl']->assign('form', $form->render());
137		- }
138		- break;
	34	+ case 'list':
	35	+ default:
	36	+ // Define Stylesheet
	37	+ $GLOBALS['xoTheme']->addStylesheet($style, null);
	38	+ $start = Request::getInt('start', 0);
	39	+ $limit = Request::getInt('limit', $helper->getConfig('adminpager'));
	40	+ $templateMain = 'wggithub_admin_readmes.tpl';
	41	+ $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('readmes.php'));
	42	+ $adminObject->addItemButton(_AM_WGGITHUB_ADD_README, 'readmes.php?op=new', 'add');
	43	+ $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
	44	+ $readmesCount = $readmesHandler->getCountReadmes();
	45	+ $readmesAll = $readmesHandler->getAllReadmes($start, $limit);
	46	+ $GLOBALS['xoopsTpl']->assign('readmes_count', $readmesCount);
	47	+ $GLOBALS['xoopsTpl']->assign('wggithub_url', WGGITHUB_URL);
	48	+ $GLOBALS['xoopsTpl']->assign('wggithub_upload_url', WGGITHUB_UPLOAD_URL);
	49	+ // Table view readmes
	50	+ if ($readmesCount > 0) {
	51	+ foreach (\array_keys($readmesAll) as $i) {
	52	+ $readme = $readmesAll[$i]->getValuesReadmes();
	53	+ $GLOBALS['xoopsTpl']->append('readmes_list', $readme);
	54	+ unset($readme);
	55	+ }
	56	+ // Display Navigation
	57	+ if ($readmesCount > $limit) {
	58	+ include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
	59	+ $pagenav = new \XoopsPageNav($readmesCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
	60	+ $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
	61	+ }
	62	+ } else {
	63	+ $GLOBALS['xoopsTpl']->assign('error', _AM_WGGITHUB_THEREARENT_READMES);
	64	+ }
	65	+ break;
	66	+ case 'new':
	67	+ $templateMain = 'wggithub_admin_readmes.tpl';
	68	+ $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('readmes.php'));
	69	+ $adminObject->addItemButton(_AM_WGGITHUB_READMES_LIST, 'readmes.php', 'list');
	70	+ $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
	71	+ // Form Create
	72	+ $readmesObj = $readmesHandler->create();
	73	+ $form = $readmesObj->getFormReadmes();
	74	+ $GLOBALS['xoopsTpl']->assign('form', $form->render());
	75	+ break;
	76	+ case 'save':
	77	+ // Security Check
	78	+ if (!$GLOBALS['xoopsSecurity']->check()) {
	79	+ \redirect_header('readmes.php', 3, \implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
	80	+ }
	81	+ if ($rmId > 0) {
	82	+ $readmesObj = $readmesHandler->get($rmId);
	83	+ } else {
	84	+ $readmesObj = $readmesHandler->create();
	85	+ }
	86	+ // Set Vars
	87	+ $readmesObj->setVar('rm_repoid', Request::getInt('rm_repoid', 0));
	88	+ $readmesObj->setVar('rm_name', Request::getString('rm_name', ''));
	89	+ $readmesObj->setVar('rm_type', Request::getString('rm_type', ''));
	90	+ $readmesObj->setVar('rm_content', Request::getString('rm_content', ''));
	91	+ $readmesObj->setVar('rm_encoding', Request::getString('rm_encoding', ''));
	92	+ $readmesObj->setVar('rm_downloadurl', Request::getString('rm_downloadurl', ''));
	93	+ $readmeDatecreatedObj = \DateTime::createFromFormat(_SHORTDATESTRING, Request::getString('rm_datecreated'));
	94	+ $readmesObj->setVar('rm_datecreated', $readmeDatecreatedObj->getTimestamp());
	95	+ $readmesObj->setVar('rm_submitter', Request::getInt('rm_submitter', 0));
	96	+ // Insert Data
	97	+ if ($readmesHandler->insert($readmesObj)) {
	98	+ \redirect_header('readmes.php?op=list', 2, _AM_WGGITHUB_FORM_OK);
	99	+ }
	100	+ // Get Form
	101	+ $GLOBALS['xoopsTpl']->assign('error', $readmesObj->getHtmlErrors());
	102	+ $form = $readmesObj->getFormReadmes();
	103	+ $GLOBALS['xoopsTpl']->assign('form', $form->render());
	104	+ break;
	105	+ case 'edit':
	106	+ $templateMain = 'wggithub_admin_readmes.tpl';
	107	+ $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('readmes.php'));
	108	+ $adminObject->addItemButton(_AM_WGGITHUB_ADD_README, 'readmes.php?op=new', 'add');
	109	+ $adminObject->addItemButton(_AM_WGGITHUB_READMES_LIST, 'readmes.php', 'list');
	110	+ $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
	111	+ // Get Form
	112	+ $readmesObj = $readmesHandler->get($rmId);
	113	+ $form = $readmesObj->getFormReadmes();
	114	+ $GLOBALS['xoopsTpl']->assign('form', $form->render());
	115	+ break;
	116	+ case 'delete':
	117	+ $templateMain = 'wggithub_admin_readmes.tpl';
	118	+ $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('readmes.php'));
	119	+ $readmesObj = $readmesHandler->get($rmId);
	120	+ $rmName = $readmesObj->getVar('rm_name');
	121	+ if (isset($_REQUEST['ok']) && 1 == $_REQUEST['ok']) {
	122	+ if (!$GLOBALS['xoopsSecurity']->check()) {
	123	+ \redirect_header('readmes.php', 3, \implode(', ', $GLOBALS['xoopsSecurity']->getErrors()));
	124	+ }
	125	+ if ($readmesHandler->delete($readmesObj)) {
	126	+ \redirect_header('readmes.php', 3, _AM_WGGITHUB_FORM_DELETE_OK);
	127	+ } else {
	128	+ $GLOBALS['xoopsTpl']->assign('error', $readmesObj->getHtmlErrors());
	129	+ }
	130	+ } else {
	131	+ $xoopsconfirm = new Common\XoopsConfirm(
	132	+ ['ok' => 1, 'rm_id' => $rmId, 'op' => 'delete'],
	133	+ $_SERVER['REQUEST_URI'],
	134	+ \sprintf(_AM_WGGITHUB_FORM_SURE_DELETE, $readmesObj->getVar('rm_name')));
	135	+ $form = $xoopsconfirm->getFormXoopsConfirm();
	136	+ $GLOBALS['xoopsTpl']->assign('form', $form->render());
	137	+ }
	138	+ break;
139	139	}
140	140	require __DIR__ . '/footer.php';

		@@ -35,60 +35,60 @@
		block discarded – undo
35	35	*/
36	36	function wggithub_search($queryarray, $andor, $limit, $offset, $userid)
37	37	{
38		- $ret = [];
39		- $helper = \XoopsModules\Wggithub\Helper::getInstance();
	38	+ $ret = [];
	39	+ $helper = \XoopsModules\Wggithub\Helper::getInstance();
40	40
41		- // search in table repositories
42		- // search keywords
43		- $elementCount = 0;
44		- $repositoriesHandler = $helper->getHandler('Repositories');
45		- if (\is_array($queryarray)) {
46		- $elementCount = \count($queryarray);
47		- }
48		- if ($elementCount > 0) {
49		- $crKeywords = new \CriteriaCompo();
50		- for ($i = 0; $i < $elementCount; $i++) {
51		- $crKeyword = new \CriteriaCompo();
52		- unset($crKeyword);
53		- }
54		- }
55		- // search user(s)
56		- if ($userid && \is_array($userid)) {
57		- $userid = array_map('intval', $userid);
58		- $crUser = new \CriteriaCompo();
59		- $crUser->add(new \Criteria('repo_submitter', '(' . \implode(',', $userid) . ')', 'IN'), 'OR');
60		- } elseif (is_numeric($userid) && $userid > 0) {
61		- $crUser = new \CriteriaCompo();
62		- $crUser->add(new \Criteria('repo_submitter', $userid), 'OR');
63		- }
64		- $crSearch = new \CriteriaCompo();
65		- if (isset($crKeywords)) {
66		- $crSearch->add($crKeywords, 'AND');
67		- }
68		- if (isset($crUser)) {
69		- $crSearch->add($crUser, 'AND');
70		- }
71		- $crSearch->setStart($offset);
72		- $crSearch->setLimit($limit);
73		- $crSearch->setSort('repo_datecreated');
74		- $crSearch->setOrder('DESC');
75		- $repositoriesAll = $repositoriesHandler->getAll($crSearch);
76		- foreach (\array_keys($repositoriesAll) as $i) {
77		- $ret[] = [
78		- 'image' => 'assets/icons/16/repositories.png',
79		- 'link' => 'repositories.php?op=show&repo_id=' . $repositoriesAll[$i]->getVar('repo_id'),
80		- 'title' => $repositoriesAll[$i]->getVar('repo_name'),
81		- 'time' => $repositoriesAll[$i]->getVar('repo_datecreated')
82		- ];
83		- }
84		- unset($crKeywords);
85		- unset($crKeyword);
86		- unset($crUser);
87		- unset($crSearch);
	41	+ // search in table repositories
	42	+ // search keywords
	43	+ $elementCount = 0;
	44	+ $repositoriesHandler = $helper->getHandler('Repositories');
	45	+ if (\is_array($queryarray)) {
	46	+ $elementCount = \count($queryarray);
	47	+ }
	48	+ if ($elementCount > 0) {
	49	+ $crKeywords = new \CriteriaCompo();
	50	+ for ($i = 0; $i < $elementCount; $i++) {
	51	+ $crKeyword = new \CriteriaCompo();
	52	+ unset($crKeyword);
	53	+ }
	54	+ }
	55	+ // search user(s)
	56	+ if ($userid && \is_array($userid)) {
	57	+ $userid = array_map('intval', $userid);
	58	+ $crUser = new \CriteriaCompo();
	59	+ $crUser->add(new \Criteria('repo_submitter', '(' . \implode(',', $userid) . ')', 'IN'), 'OR');
	60	+ } elseif (is_numeric($userid) && $userid > 0) {
	61	+ $crUser = new \CriteriaCompo();
	62	+ $crUser->add(new \Criteria('repo_submitter', $userid), 'OR');
	63	+ }
	64	+ $crSearch = new \CriteriaCompo();
	65	+ if (isset($crKeywords)) {
	66	+ $crSearch->add($crKeywords, 'AND');
	67	+ }
	68	+ if (isset($crUser)) {
	69	+ $crSearch->add($crUser, 'AND');
	70	+ }
	71	+ $crSearch->setStart($offset);
	72	+ $crSearch->setLimit($limit);
	73	+ $crSearch->setSort('repo_datecreated');
	74	+ $crSearch->setOrder('DESC');
	75	+ $repositoriesAll = $repositoriesHandler->getAll($crSearch);
	76	+ foreach (\array_keys($repositoriesAll) as $i) {
	77	+ $ret[] = [
	78	+ 'image' => 'assets/icons/16/repositories.png',
	79	+ 'link' => 'repositories.php?op=show&repo_id=' . $repositoriesAll[$i]->getVar('repo_id'),
	80	+ 'title' => $repositoriesAll[$i]->getVar('repo_name'),
	81	+ 'time' => $repositoriesAll[$i]->getVar('repo_datecreated')
	82	+ ];
	83	+ }
	84	+ unset($crKeywords);
	85	+ unset($crKeyword);
	86	+ unset($crUser);
	87	+ unset($crSearch);
88	88
89	89
90	90
91	91
92		- return $ret;
	92	+ return $ret;
93	93
94	94	}

		@@ -39,7 +39,7 @@ discard block
		block discarded – undo
39	39	$ret = wggithub_check_db($module);
40	40
41	41	//check upload directory
42		- include_once __DIR__ . '/install.php';
	42	+ include_once __DIR__ . '/install.php';
43	43	$ret = xoops_module_install_wggithub($module);
44	44
45	45	$errors = $module->getErrors();
		@@ -113,7 +113,7 @@ discard block
		block discarded – undo
113	113	function wggithub_check_db($module)
114	114	{
115	115	$ret = true;
116		- //insert here code for database check
	116	+ //insert here code for database check
117	117
118	118	// Example: update table (add new field)
119	119	$table = $GLOBALS['xoopsDB']->prefix('wggithub_repositories');

		@@ -29,26 +29,26 @@
		block discarded – undo
29	29	*/
30	30	function wggithub_notify_iteminfo($category, $item_id)
31	31	{
32		- global $xoopsDB;
	32	+ global $xoopsDB;
33	33
34		- if (!\defined('WGGITHUB_URL')) {
35		- \define('WGGITHUB_URL', \XOOPS_URL . '/modules/wggithub');
36		- }
	34	+ if (!\defined('WGGITHUB_URL')) {
	35	+ \define('WGGITHUB_URL', \XOOPS_URL . '/modules/wggithub');
	36	+ }
37	37
38		- switch ($category) {
39		- case 'global':
40		- $item['name'] = '';
41		- $item['url'] = '';
42		- return $item;
43		- break;
44		- case 'repositories':
45		- $sql = 'SELECT repo_name FROM ' . $xoopsDB->prefix('wggithub_repositories') . ' WHERE repo_id = '. $item_id;
46		- $result = $xoopsDB->query($sql);
47		- $result_array = $xoopsDB->fetchArray($result);
48		- $item['name'] = $result_array['repo_name'];
49		- $item['url'] = WGGITHUB_URL . '/repositories.php?repo_id=' . $item_id;
50		- return $item;
51		- break;
52		- }
53		- return null;
	38	+ switch ($category) {
	39	+ case 'global':
	40	+ $item['name'] = '';
	41	+ $item['url'] = '';
	42	+ return $item;
	43	+ break;
	44	+ case 'repositories':
	45	+ $sql = 'SELECT repo_name FROM ' . $xoopsDB->prefix('wggithub_repositories') . ' WHERE repo_id = '. $item_id;
	46	+ $result = $xoopsDB->query($sql);
	47	+ $result_array = $xoopsDB->fetchArray($result);
	48	+ $item['name'] = $result_array['repo_name'];
	49	+ $item['url'] = WGGITHUB_URL . '/repositories.php?repo_id=' . $item_id;
	50	+ return $item;
	51	+ break;
	52	+ }
	53	+ return null;
54	54	}

ggoffy / wggithub

Push — master ( 8005c7...9b803d )

Status

Category

Indentation +10 added lines, -10 removed lines patch added patch discarded remove patch

Indentation +2 added lines, -2 removed lines patch added patch discarded remove patch

Indentation +106 added lines, -106 removed lines patch added patch discarded remove patch

Indentation +4 added lines, -4 removed lines patch added patch discarded remove patch

Indentation +105 added lines, -105 removed lines patch added patch discarded remove patch

Indentation +50 added lines, -50 removed lines patch added patch discarded remove patch

Indentation +2 added lines, -2 removed lines patch added patch discarded remove patch

Indentation +20 added lines, -20 removed lines patch added patch discarded remove patch

Indentation +1 added lines, -1 removed lines patch added patch discarded remove patch