ggoffy / wggithub

include/search.inc.php

Indentation +50 added lines, -50 removed lines

@@ -35,60 +35,60 @@
  */
 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;
 
 }

Spacing +3 added lines, -3 removed lines

@@ -47,7 +47,7 @@ 
 	}
 	if ($elementCount > 0) {
 		$crKeywords = new \CriteriaCompo();
-		for ($i = 0; $i  <  $elementCount; $i++) {
+		for ($i = 0; $i < $elementCount; $i++) {
 			$crKeyword = new \CriteriaCompo();
 			unset($crKeyword);
 		}
@@ -56,7 +56,7 @@ 
 	if ($userid && \is_array($userid)) {
 		$userid = array_map('intval', $userid);
 		$crUser = new \CriteriaCompo();
-		$crUser->add(new \Criteria('repo_submitter', '(' . \implode(',', $userid) . ')', 'IN'), 'OR');
+		$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');
@@ -76,7 +76,7 @@ 
 	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'),
+			'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')
 		];

include/update.php

Indentation +2 added lines, -2 removed lines

@@ -39,7 +39,7 @@ 
     $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 @@ 
 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');

Spacing +13 added lines, -13 removed lines

@@ -39,7 +39,7 @@ 
     $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();
@@ -61,7 +61,7 @@ 
 {
     global $xoopsDB;
     $result = $xoopsDB->query(
-        'SELECT t1.tpl_id FROM ' . $xoopsDB->prefix('tplfile') . ' t1, ' . $xoopsDB->prefix('tplfile') . ' t2 WHERE t1.tpl_refid = t2.tpl_refid AND t1.tpl_module = t2.tpl_module AND t1.tpl_tplset=t2.tpl_tplset AND t1.tpl_file = t2.tpl_file AND t1.tpl_type = t2.tpl_type AND t1.tpl_id > t2.tpl_id'
+        'SELECT t1.tpl_id FROM '.$xoopsDB->prefix('tplfile').' t1, '.$xoopsDB->prefix('tplfile').' t2 WHERE t1.tpl_refid = t2.tpl_refid AND t1.tpl_module = t2.tpl_module AND t1.tpl_tplset=t2.tpl_tplset AND t1.tpl_file = t2.tpl_file AND t1.tpl_type = t2.tpl_type AND t1.tpl_id > t2.tpl_id'
     );
     $tplids = [];
     while (false !== (list($tplid) = $xoopsDB->fetchRow($result))) {
@@ -69,7 +69,7 @@ 
     }
     if (\count($tplids) > 0) {
         $tplfileHandler  = \xoops_getHandler('tplfile');
-        $duplicate_files = $tplfileHandler->getObjects(new \Criteria('tpl_id', '(' . \implode(',', $tplids) . ')', 'IN'));
+        $duplicate_files = $tplfileHandler->getObjects(new \Criteria('tpl_id', '('.\implode(',', $tplids).')', 'IN'));
 
         if (\count($duplicate_files) > 0) {
             foreach (\array_keys($duplicate_files) as $i) {
@@ -77,9 +77,9 @@ 
             }
         }
     }
-    $sql = 'SHOW INDEX FROM ' . $xoopsDB->prefix('tplfile') . " WHERE KEY_NAME = 'tpl_refid_module_set_file_type'";
+    $sql = 'SHOW INDEX FROM '.$xoopsDB->prefix('tplfile')." WHERE KEY_NAME = 'tpl_refid_module_set_file_type'";
     if (!$result = $xoopsDB->queryF($sql)) {
-        xoops_error($xoopsDB->error() . '<br>' . $sql);
+        xoops_error($xoopsDB->error().'<br>'.$sql);
 
         return false;
     }
@@ -92,9 +92,9 @@ 
 
         return true;
     }
-    $sql = 'ALTER TABLE ' . $xoopsDB->prefix('tplfile') . ' ADD UNIQUE tpl_refid_module_set_file_type ( tpl_refid, tpl_module, tpl_tplset, tpl_file, tpl_type )';
+    $sql = 'ALTER TABLE '.$xoopsDB->prefix('tplfile').' ADD UNIQUE tpl_refid_module_set_file_type ( tpl_refid, tpl_module, tpl_tplset, tpl_file, tpl_type )';
     if (!$result = $xoopsDB->queryF($sql)) {
-        xoops_error($xoopsDB->error() . '<br>' . $sql);
+        xoops_error($xoopsDB->error().'<br>'.$sql);
         $module->setErrors("'tpl_refid_module_set_file_type' unique index is not added to 'tplfile' table. Warning: do not use XOOPS until you add this unique index.");
 
         return false;
@@ -118,12 +118,12 @@ 
     // Example: update table (add new field)
     $table   = $GLOBALS['xoopsDB']->prefix('wggithub_repositories');
     $field   = 'repo_release';
-    $check   = $GLOBALS['xoopsDB']->queryF('SHOW COLUMNS FROM `' . $table . "` LIKE '" . $field . "'");
+    $check   = $GLOBALS['xoopsDB']->queryF('SHOW COLUMNS FROM `'.$table."` LIKE '".$field."'");
     $numRows = $GLOBALS['xoopsDB']->getRowsNum($check);
     if (!$numRows) {
         $sql = "ALTER TABLE `$table` ADD `$field` INT(1) NOT NULL DEFAULT '0' AFTER `repo_htmlurl`;";
         if (!$result = $GLOBALS['xoopsDB']->queryF($sql)) {
-            xoops_error($GLOBALS['xoopsDB']->error() . '<br>' . $sql);
+            xoops_error($GLOBALS['xoopsDB']->error().'<br>'.$sql);
             $module->setErrors("Error when adding '$field' to table '$table'.");
             $ret = false;
         }
@@ -132,12 +132,12 @@ 
     // Example: update table (add new field)
     $table   = $GLOBALS['xoopsDB']->prefix('wggithub_repositories');
     $field   = 'repo_prerelease';
-    $check   = $GLOBALS['xoopsDB']->queryF('SHOW COLUMNS FROM `' . $table . "` LIKE '" . $field . "'");
+    $check   = $GLOBALS['xoopsDB']->queryF('SHOW COLUMNS FROM `'.$table."` LIKE '".$field."'");
     $numRows = $GLOBALS['xoopsDB']->getRowsNum($check);
     if (!$numRows) {
         $sql = "ALTER TABLE `$table` ADD `$field` INT(1) NOT NULL DEFAULT '0' AFTER `repo_htmlurl`;";
         if (!$result = $GLOBALS['xoopsDB']->queryF($sql)) {
-            xoops_error($GLOBALS['xoopsDB']->error() . '<br>' . $sql);
+            xoops_error($GLOBALS['xoopsDB']->error().'<br>'.$sql);
             $module->setErrors("Error when adding '$field' to table '$table'.");
             $ret = false;
         }
@@ -146,12 +146,12 @@ 
 // Example: update table (add new field)
     $table   = $GLOBALS['xoopsDB']->prefix('wggithub_directories');
     $field   = 'dir_filterrelease';
-    $check   = $GLOBALS['xoopsDB']->queryF('SHOW COLUMNS FROM `' . $table . "` LIKE '" . $field . "'");
+    $check   = $GLOBALS['xoopsDB']->queryF('SHOW COLUMNS FROM `'.$table."` LIKE '".$field."'");
     $numRows = $GLOBALS['xoopsDB']->getRowsNum($check);
     if (!$numRows) {
         $sql = "ALTER TABLE `$table` ADD `$field` INT(1) NOT NULL DEFAULT '0' AFTER `dir_online`;";
         if (!$result = $GLOBALS['xoopsDB']->queryF($sql)) {
-            xoops_error($GLOBALS['xoopsDB']->error() . '<br>' . $sql);
+            xoops_error($GLOBALS['xoopsDB']->error().'<br>'.$sql);
             $module->setErrors("Error when adding '$field' to table '$table'.");
             $ret = false;
         }

include/notification.inc.php

Indentation +20 added lines, -20 removed lines

@@ -29,26 +29,26 @@
  */
 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;
 }

Spacing +3 added lines, -3 removed lines

@@ -32,7 +32,7 @@ 
 	global $xoopsDB;
 
 	if (!\defined('WGGITHUB_URL')) {
-		\define('WGGITHUB_URL', \XOOPS_URL . '/modules/wggithub');
+		\define('WGGITHUB_URL', \XOOPS_URL.'/modules/wggithub');
 	}
 
 	switch ($category) {
@@ -42,11 +42,11 @@ 
 			return $item;
 			break;
 		case 'repositories':
-			$sql          = 'SELECT repo_name FROM ' . $xoopsDB->prefix('wggithub_repositories') . ' WHERE repo_id = '. $item_id;
+			$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;
+			$item['url']  = WGGITHUB_URL.'/repositories.php?repo_id='.$item_id;
 			return $item;
 			break;
 	}

include/install.php

Indentation +1 added lines, -1 removed lines

@@ -84,7 +84,7 @@
             $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);

Spacing +7 added lines, -7 removed lines

@@ -29,7 +29,7 @@ 
  */
 function xoops_module_pre_install_wggithub(\XoopsModule $module)
 {
-    require \dirname(__DIR__) . '/preloads/autoloader.php';
+    require \dirname(__DIR__).'/preloads/autoloader.php';
 
     $utility = new Wggithub\Utility();
 
@@ -42,7 +42,7 @@ 
     if ($xoopsSuccess && $phpSuccess) {
         $moduleTables = &$module->getInfo('tables');
         foreach ($moduleTables as $table) {
-            $GLOBALS['xoopsDB']->queryF('DROP TABLE IF EXISTS ' . $GLOBALS['xoopsDB']->prefix($table) . ';');
+            $GLOBALS['xoopsDB']->queryF('DROP TABLE IF EXISTS '.$GLOBALS['xoopsDB']->prefix($table).';');
         }
     }
 
@@ -55,7 +55,7 @@ 
  */
 function xoops_module_install_wggithub(\XoopsModule $module)
 {
-    require \dirname(__DIR__) . '/preloads/autoloader.php';
+    require \dirname(__DIR__).'/preloads/autoloader.php';
 
     /** @var Wggithub\Helper $helper */ 
     /** @var Wggithub\Utility $utility */
@@ -79,14 +79,14 @@ 
 
     //  ---  COPY blank.gif FILES ---------------
     if ($configurator->copyBlankFiles && \is_array($configurator->copyBlankFiles)) {
-        $file = \dirname(__DIR__) . '/assets/images/blank.gif';
+        $file = \dirname(__DIR__).'/assets/images/blank.gif';
         foreach (\array_keys($configurator->copyBlankFiles) as $i) {
-            $dest = $configurator->copyBlankFiles[$i] . '/blank.gif';
+            $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';
+            $dest = $configurator->copyBlankFiles[$i].'/blank.png';
             $utility::copyFile($file, $dest);
         }
     }

include/common.php

Indentation +2 added lines, -2 removed lines

@@ -20,10 +20,10 @@
  * @author         Goffy - XOOPS Development Team - Email:<goffy@wedega.com> - Website:<https://wedega.com>
  */
 if (!\defined('XOOPS_ICONS32_PATH')) {
-	\define('XOOPS_ICONS32_PATH', XOOPS_ROOT_PATH . '/Frameworks/moduleclasses/icons/32');
+    \define('XOOPS_ICONS32_PATH', XOOPS_ROOT_PATH . '/Frameworks/moduleclasses/icons/32');
 }
 if (!\defined('XOOPS_ICONS32_URL')) {
-	\define('XOOPS_ICONS32_URL', XOOPS_URL . '/Frameworks/moduleclasses/icons/32');
+    \define('XOOPS_ICONS32_URL', XOOPS_URL . '/Frameworks/moduleclasses/icons/32');
 }
 \define('WGGITHUB_DIRNAME', 'wggithub');
 \define('WGGITHUB_PATH', XOOPS_ROOT_PATH . '/modules/' . WGGITHUB_DIRNAME);

Spacing +21 added lines, -21 removed lines

@@ -20,29 +20,29 @@
  * @author         Goffy - XOOPS Development Team - Email:<goffy@wedega.com> - Website:<https://wedega.com>
  */
 if (!\defined('XOOPS_ICONS32_PATH')) {
-	\define('XOOPS_ICONS32_PATH', XOOPS_ROOT_PATH . '/Frameworks/moduleclasses/icons/32');
+	\define('XOOPS_ICONS32_PATH', XOOPS_ROOT_PATH.'/Frameworks/moduleclasses/icons/32');
 }
 if (!\defined('XOOPS_ICONS32_URL')) {
-	\define('XOOPS_ICONS32_URL', XOOPS_URL . '/Frameworks/moduleclasses/icons/32');
+	\define('XOOPS_ICONS32_URL', XOOPS_URL.'/Frameworks/moduleclasses/icons/32');
 }
 \define('WGGITHUB_DIRNAME', 'wggithub');
-\define('WGGITHUB_PATH', XOOPS_ROOT_PATH . '/modules/' . WGGITHUB_DIRNAME);
-\define('WGGITHUB_URL', XOOPS_URL . '/modules/' . WGGITHUB_DIRNAME);
-\define('WGGITHUB_ICONS_PATH', WGGITHUB_PATH . '/assets/icons');
-\define('WGGITHUB_ICONS_URL', WGGITHUB_URL . '/assets/icons');
-\define('WGGITHUB_IMAGE_PATH', WGGITHUB_PATH . '/assets/images');
-\define('WGGITHUB_IMAGE_URL', WGGITHUB_URL . '/assets/images');
-\define('WGGITHUB_UPLOAD_PATH', XOOPS_UPLOAD_PATH . '/' . WGGITHUB_DIRNAME);
-\define('WGGITHUB_UPLOAD_URL', XOOPS_UPLOAD_URL . '/' . WGGITHUB_DIRNAME);
-\define('WGGITHUB_UPLOAD_FILES_PATH', WGGITHUB_UPLOAD_PATH . '/files');
-\define('WGGITHUB_UPLOAD_FILES_URL', WGGITHUB_UPLOAD_URL . '/files');
-\define('WGGITHUB_UPLOAD_IMAGE_PATH', WGGITHUB_UPLOAD_PATH . '/images');
-\define('WGGITHUB_UPLOAD_IMAGE_URL', WGGITHUB_UPLOAD_URL . '/images');
-\define('WGGITHUB_UPLOAD_SHOTS_PATH', WGGITHUB_UPLOAD_PATH . '/images/shots');
-\define('WGGITHUB_UPLOAD_SHOTS_URL', WGGITHUB_UPLOAD_URL . '/images/shots');
-\define('WGGITHUB_ADMIN', WGGITHUB_URL . '/admin/index.php');
-$localLogo = WGGITHUB_IMAGE_URL . '/tdmxoops_logo.png';
+\define('WGGITHUB_PATH', XOOPS_ROOT_PATH.'/modules/'.WGGITHUB_DIRNAME);
+\define('WGGITHUB_URL', XOOPS_URL.'/modules/'.WGGITHUB_DIRNAME);
+\define('WGGITHUB_ICONS_PATH', WGGITHUB_PATH.'/assets/icons');
+\define('WGGITHUB_ICONS_URL', WGGITHUB_URL.'/assets/icons');
+\define('WGGITHUB_IMAGE_PATH', WGGITHUB_PATH.'/assets/images');
+\define('WGGITHUB_IMAGE_URL', WGGITHUB_URL.'/assets/images');
+\define('WGGITHUB_UPLOAD_PATH', XOOPS_UPLOAD_PATH.'/'.WGGITHUB_DIRNAME);
+\define('WGGITHUB_UPLOAD_URL', XOOPS_UPLOAD_URL.'/'.WGGITHUB_DIRNAME);
+\define('WGGITHUB_UPLOAD_FILES_PATH', WGGITHUB_UPLOAD_PATH.'/files');
+\define('WGGITHUB_UPLOAD_FILES_URL', WGGITHUB_UPLOAD_URL.'/files');
+\define('WGGITHUB_UPLOAD_IMAGE_PATH', WGGITHUB_UPLOAD_PATH.'/images');
+\define('WGGITHUB_UPLOAD_IMAGE_URL', WGGITHUB_UPLOAD_URL.'/images');
+\define('WGGITHUB_UPLOAD_SHOTS_PATH', WGGITHUB_UPLOAD_PATH.'/images/shots');
+\define('WGGITHUB_UPLOAD_SHOTS_URL', WGGITHUB_UPLOAD_URL.'/images/shots');
+\define('WGGITHUB_ADMIN', WGGITHUB_URL.'/admin/index.php');
+$localLogo = WGGITHUB_IMAGE_URL.'/tdmxoops_logo.png';
 // Module Information
-$copyright = "<a href='https://wedega.com' title='XOOPS on Wedega' target='_blank'><img src='" . $localLogo . "' alt='XOOPS on Wedega' /></a>";
-include_once XOOPS_ROOT_PATH . '/class/xoopsrequest.php';
-include_once WGGITHUB_PATH . '/include/functions.php';
+$copyright = "<a href='https://wedega.com' title='XOOPS on Wedega' target='_blank'><img src='".$localLogo."' alt='XOOPS on Wedega' /></a>";
+include_once XOOPS_ROOT_PATH.'/class/xoopsrequest.php';
+include_once WGGITHUB_PATH.'/include/functions.php';

include/uninstall.php

Spacing +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
         if ($dirInfo->isDir()) {
             // The directory exists so delete it
             if (!$utility::rrmdir($old_dir)) {
-                $module->setErrors(\sprintf(\constant('CO_' . $moduleDirNameUpper . '_ERROR_BAD_DEL_PATH'), $old_dir));
+                $module->setErrors(\sprintf(\constant('CO_'.$moduleDirNameUpper.'_ERROR_BAD_DEL_PATH'), $old_dir));
                 $success = false;
             }
         }

include/functions.php

Indentation +23 added lines, -23 removed lines

@@ -28,16 +28,16 @@ 
  */
 function wggithub_block_addCatSelect($cats)
 {
-	$cat_sql = '(';
-	if (\is_array($cats)) {
-		$cat_sql .= current($cats);
-		array_shift($cats);
-		foreach ($cats as $cat) {
-			$cat_sql .= ',' . $cat;
-		}
-	}
-	$cat_sql .= ')';
-	return $cat_sql;
+    $cat_sql = '(';
+    if (\is_array($cats)) {
+        $cat_sql .= current($cats);
+        array_shift($cats);
+        foreach ($cats as $cat) {
+            $cat_sql .= ',' . $cat;
+        }
+    }
+    $cat_sql .= ')';
+    return $cat_sql;
 }
 
 /**
@@ -49,17 +49,17 @@ 
  */
 function wggithubGetMyItemIds($permtype, $dirname)
 {
-	global $xoopsUser;
-	static $permissions = [];
-	if (\is_array($permissions) && \array_key_exists($permtype, $permissions)) {
-		return $permissions[$permtype];
-	}
-	$moduleHandler = \xoops_getHandler('module');
-	$wggithubModule = $moduleHandler->getByDirname($dirname);
-	$groups = \is_object($xoopsUser) ? $xoopsUser->getGroups() : XOOPS_GROUP_ANONYMOUS;
-	$grouppermHandler = \xoops_getHandler('groupperm');
-	$itemIds = $grouppermHandler->getItemIds($permtype, $groups, $wggithubModule->getVar('mid'));
-	return $itemIds;
+    global $xoopsUser;
+    static $permissions = [];
+    if (\is_array($permissions) && \array_key_exists($permtype, $permissions)) {
+        return $permissions[$permtype];
+    }
+    $moduleHandler = \xoops_getHandler('module');
+    $wggithubModule = $moduleHandler->getByDirname($dirname);
+    $groups = \is_object($xoopsUser) ? $xoopsUser->getGroups() : XOOPS_GROUP_ANONYMOUS;
+    $grouppermHandler = \xoops_getHandler('groupperm');
+    $itemIds = $grouppermHandler->getItemIds($permtype, $groups, $wggithubModule->getVar('mid'));
+    return $itemIds;
 }
 
 /**
@@ -133,7 +133,7 @@ 
 
         case 'none':
             if($topic_name) {
-                 $topic_name = 'topic=' . $topic_name . '&';
+                    $topic_name = 'topic=' . $topic_name . '&';
             }
             $rewrite_base = '/modules/';
             $page = 'page=' . $array['content_alias'];
@@ -163,7 +163,7 @@ 
             return XOOPS_URL . $rewrite_base . $module_name . $type . $topic_name  . $id . $page . $rewrite_ext;
             break;
 
-         case 'short':
+            case 'short':
             if($topic_name) {
                 $topic_name .= '/';
             }

Spacing +22 added lines, -22 removed lines

@@ -33,7 +33,7 @@ 
 		$cat_sql .= current($cats);
 		array_shift($cats);
 		foreach ($cats as $cat) {
-			$cat_sql .= ',' . $cat;
+			$cat_sql .= ','.$cat;
 		}
 	}
 	$cat_sql .= ')';
@@ -72,9 +72,9 @@ 
 {
     global $xoopsTpl, $xoTheme;
     $myts = MyTextSanitizer::getInstance();
-    $content= $myts->undoHtmlSpecialChars($myts->displayTarea($content));
-    if(isset($xoTheme) && \is_object($xoTheme)) {
-        $xoTheme->addMeta( 'meta', 'keywords', \strip_tags($content));
+    $content = $myts->undoHtmlSpecialChars($myts->displayTarea($content));
+    if (isset($xoTheme) && \is_object($xoTheme)) {
+        $xoTheme->addMeta('meta', 'keywords', \strip_tags($content));
     } else {    // Compatibility for old Xoops versions
         $xoopsTpl->assign('xoops_meta_keywords', \strip_tags($content));
     }
@@ -91,8 +91,8 @@ 
     global $xoopsTpl, $xoTheme;
     $myts = MyTextSanitizer::getInstance();
     $content = $myts->undoHtmlSpecialChars($myts->displayTarea($content));
-    if(isset($xoTheme) && \is_object($xoTheme)) {
-        $xoTheme->addMeta( 'meta', 'description', \strip_tags($content));
+    if (isset($xoTheme) && \is_object($xoTheme)) {
+        $xoTheme->addMeta('meta', 'description', \strip_tags($content));
     } else {    // Compatibility for old Xoops versions
         $xoopsTpl->assign('xoops_meta_description', \strip_tags($content));
     }
@@ -117,7 +117,7 @@ 
     if (0 != $lenght_id) {
         $id = $array['content_id'];
         while (\strlen($id) < $lenght_id) {
-            $id = '0' . $id;
+            $id = '0'.$id;
         }
     } else {
         $id = $array['content_id'];
@@ -132,23 +132,23 @@ 
     switch ($rewrite_url) {
 
         case 'none':
-            if($topic_name) {
-                 $topic_name = 'topic=' . $topic_name . '&amp;';
+            if ($topic_name) {
+                 $topic_name = 'topic='.$topic_name.'&amp;';
             }
             $rewrite_base = '/modules/';
-            $page = 'page=' . $array['content_alias'];
-            return XOOPS_URL . $rewrite_base . $module . '/' . $type . '.php?' . $topic_name . 'id=' . $id . '&amp;' . $page . $comment;
+            $page = 'page='.$array['content_alias'];
+            return XOOPS_URL.$rewrite_base.$module.'/'.$type.'.php?'.$topic_name.'id='.$id.'&amp;'.$page.$comment;
             break;
 
         case 'rewrite':
-            if($topic_name) {
+            if ($topic_name) {
                 $topic_name .= '/';
             }
             $rewrite_base = xoops_getModuleOption('rewrite_mode', $module);
             $rewrite_ext = xoops_getModuleOption('rewrite_ext', $module);
             $module_name = '';
-            if(xoops_getModuleOption('rewrite_name', $module)) {
-                $module_name = xoops_getModuleOption('rewrite_name', $module) . '/';
+            if (xoops_getModuleOption('rewrite_name', $module)) {
+                $module_name = xoops_getModuleOption('rewrite_name', $module).'/';
             }
             $page = $array['content_alias'];
             $type .= '/';
@@ -157,21 +157,21 @@ 
                 $type = '';
             }
             if ('comment-edit/' === $type || 'comment-reply/' === $type || 'comment-delete/' === $type) {
-                return XOOPS_URL . $rewrite_base . $module_name . $type . $id . '/';
+                return XOOPS_URL.$rewrite_base.$module_name.$type.$id.'/';
             }
 
-            return XOOPS_URL . $rewrite_base . $module_name . $type . $topic_name  . $id . $page . $rewrite_ext;
+            return XOOPS_URL.$rewrite_base.$module_name.$type.$topic_name.$id.$page.$rewrite_ext;
             break;
 
          case 'short':
-            if($topic_name) {
+            if ($topic_name) {
                 $topic_name .= '/';
             }
             $rewrite_base = xoops_getModuleOption('rewrite_mode', $module);
             $rewrite_ext = xoops_getModuleOption('rewrite_ext', $module);
             $module_name = '';
-            if(xoops_getModuleOption('rewrite_name', $module)) {
-                $module_name = xoops_getModuleOption('rewrite_name', $module) . '/';
+            if (xoops_getModuleOption('rewrite_name', $module)) {
+                $module_name = xoops_getModuleOption('rewrite_name', $module).'/';
             }
             $page = $array['content_alias'];
             $type .= '/';
@@ -179,10 +179,10 @@ 
                 $type = '';
             }
             if ('comment-edit/' === $type || 'comment-reply/' === $type || 'comment-delete/' === $type) {
-                return XOOPS_URL . $rewrite_base . $module_name . $type . $id . '/';
+                return XOOPS_URL.$rewrite_base.$module_name.$type.$id.'/';
             }
 
-            return XOOPS_URL . $rewrite_base . $module_name . $type . $topic_name . $page . $rewrite_ext;
+            return XOOPS_URL.$rewrite_base.$module_name.$type.$topic_name.$page.$rewrite_ext;
             break;
     }
     return null;
@@ -207,6 +207,6 @@ 
     $url .= htmlentities($url, ENT_COMPAT, 'utf-8');
     $url .= \preg_replace('`&([a-z])(acute|uml|circ|grave|ring|cedil|slash|tilde|caron|lig);`i', "\1", $url);
     $url .= \preg_replace(array($regular_expression, '`[-]+`'), '-', $url);
-    $url = ($url == '') ? $type : strtolower(	rim($url, '-'));
+    $url = ($url == '') ? $type : strtolower(rim($url, '-'));
     return $url;
 }
\ No newline at end of file

testdata/index.php

Spacing +14 added lines, -14 removed lines

@@ -18,8 +18,8 @@ 
 use XoopsModules\Wggithub\Common;
 use XoopsModules\Wggithub\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', '');
 
@@ -39,7 +39,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;
@@ -63,12 +63,12 @@ 
     $tables = \Xmf\Module\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'].'/';
     }
 
     foreach ($tables as $table) {
-        $tabledata = \Xmf\Yaml::readWrapped($language . $table . '.yml');
+        $tabledata = \Xmf\Yaml::readWrapped($language.$table.'.yml');
         \Xmf\Database\TableLoad::truncateTable($table);
         \Xmf\Database\TableLoad::loadTableFromArray($table, $tabledata);
     }
@@ -82,7 +82,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()
@@ -96,15 +96,15 @@ 
 
     $tables = \Xmf\Module\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);
 
     foreach ($tables as $table) {
-        \Xmf\Database\TableLoad::saveTableToYamlFile($table, $exportFolder . $table . '.yml');
+        \Xmf\Database\TableLoad::saveTableToYamlFile($table, $exportFolder.$table.'.yml');
     }
 
     //  ---  COPY test folder files ---------------
@@ -115,7 +115,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()
@@ -131,7 +131,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

Braces +1 added lines, -2 removed lines

@@ -129,8 +129,7 @@
         //        $migrate->saveCurrentSchema();
         //
         //        \redirect_header('../admin/index.php', 1, \constant('CO_' . $moduleDirNameUpper . '_' . 'EXPORT_SCHEMA_SUCCESS'));
-    }
-    catch (\Exception $e) {
+    } catch (\Exception $e) {
         exit(\constant('CO_' . $moduleDirNameUpper . '_' . 'EXPORT_SCHEMA_ERROR'));
     }
 

lib/Github/HttpClient/Plugin/GithubExceptionThrower.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
      */
     public function doHandleRequest(RequestInterface $request, callable $next, callable $first)
     {
-        return $next($request)->then(function (ResponseInterface $response) use ($request) {
+        return $next($request)->then(function(ResponseInterface $response) use ($request) {
             if ($response->getStatusCode() < 400 || $response->getStatusCode() > 600) {
                 return $response;
             }