ggoffy / wggithub

readmes.php

Indentation +32 added lines, -32 removed lines

@@ -45,38 +45,38 @@
 $GLOBALS['xoopsTpl']->assign('showItem', $rmId > 0);
 
 switch ($op) {
-	case 'show':
-	case 'list':
-	default:
-		$crReadmes = new \CriteriaCompo();
-		if ($rmId > 0) {
-			$crReadmes->add(new \Criteria('rm_id', $rmId));
-		}
-		$readmesCount = $readmesHandler->getCount($crReadmes);
-		$GLOBALS['xoopsTpl']->assign('readmesCount', $readmesCount);
-		$crReadmes->setStart($start);
-		$crReadmes->setLimit($limit);
-		$readmesAll = $readmesHandler->getAll($crReadmes);
-		if ($readmesCount > 0) {
-			$readmes = [];
-			// Get All Readmes
-			foreach (\array_keys($readmesAll) as $i) {
-				$readmes[$i] = $readmesAll[$i]->getValuesReadmes();
-				$keywords[$i] = $readmesAll[$i]->getVar('rm_name');
-			}
-			$GLOBALS['xoopsTpl']->assign('readmes', $readmes);
-			unset($readmes);
-			// 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));
-			}
-			$GLOBALS['xoopsTpl']->assign('type', $helper->getConfig('table_type'));
-			$GLOBALS['xoopsTpl']->assign('divideby', $helper->getConfig('divideby'));
-			$GLOBALS['xoopsTpl']->assign('numb_col', $helper->getConfig('numb_col'));
-		}
-		break;
+    case 'show':
+    case 'list':
+    default:
+        $crReadmes = new \CriteriaCompo();
+        if ($rmId > 0) {
+            $crReadmes->add(new \Criteria('rm_id', $rmId));
+        }
+        $readmesCount = $readmesHandler->getCount($crReadmes);
+        $GLOBALS['xoopsTpl']->assign('readmesCount', $readmesCount);
+        $crReadmes->setStart($start);
+        $crReadmes->setLimit($limit);
+        $readmesAll = $readmesHandler->getAll($crReadmes);
+        if ($readmesCount > 0) {
+            $readmes = [];
+            // Get All Readmes
+            foreach (\array_keys($readmesAll) as $i) {
+                $readmes[$i] = $readmesAll[$i]->getValuesReadmes();
+                $keywords[$i] = $readmesAll[$i]->getVar('rm_name');
+            }
+            $GLOBALS['xoopsTpl']->assign('readmes', $readmes);
+            unset($readmes);
+            // 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));
+            }
+            $GLOBALS['xoopsTpl']->assign('type', $helper->getConfig('table_type'));
+            $GLOBALS['xoopsTpl']->assign('divideby', $helper->getConfig('divideby'));
+            $GLOBALS['xoopsTpl']->assign('numb_col', $helper->getConfig('numb_col'));
+        }
+        break;
 }
 
 // Breadcrumbs

Spacing +6 added lines, -6 removed lines

@@ -25,9 +25,9 @@ 
 use XoopsModules\Wggithub\Constants;
 use XoopsModules\Wggithub\Common;
 
-require __DIR__ . '/header.php';
+require __DIR__.'/header.php';
 $GLOBALS['xoopsOption']['template_main'] = 'wggithub_readmes.tpl';
-include_once XOOPS_ROOT_PATH . '/header.php';
+include_once XOOPS_ROOT_PATH.'/header.php';
 
 $op    = Request::getCmd('op', 'list');
 $start = Request::getInt('start', 0);
@@ -68,8 +68,8 @@ 
 			unset($readmes);
 			// Display Navigation
 			if ($readmesCount > $limit) {
-				include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
-				$pagenav = new \XoopsPageNav($readmesCount, $limit, $start, 'start', 'op=list&limit=' . $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));
 			}
 			$GLOBALS['xoopsTpl']->assign('type', $helper->getConfig('table_type'));
@@ -83,7 +83,7 @@ 
 $xoBreadcrumbs[] = ['title' => _MA_WGGITHUB_READMES];
 
 // Keywords
-wggithubMetaKeywords($helper->getConfig('keywords') . ', ' . \implode(',', $keywords));
+wggithubMetaKeywords($helper->getConfig('keywords').', '.\implode(',', $keywords));
 unset($keywords);
 
 // Description
@@ -91,4 +91,4 @@ 
 $GLOBALS['xoopsTpl']->assign('xoops_mpageurl', WGGITHUB_URL.'/readmes.php');
 $GLOBALS['xoopsTpl']->assign('wggithub_upload_url', WGGITHUB_UPLOAD_URL);
 
-require __DIR__ . '/footer.php';
+require __DIR__.'/footer.php';

admin/permissions.php

Spacing +3 added lines, -3 removed lines

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

test.php

Spacing +3 added lines, -3 removed lines

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

class/Constants.php

Indentation +9 added lines, -9 removed lines

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

class/ReadmesHandler.php

Indentation +84 added lines, -84 removed lines

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

Spacing +6 added lines, -6 removed lines

@@ -170,7 +170,7 @@ 
                     $readmesObj->setVar('rm_content', $readme['content']);
                     $readmesObj->setVar('rm_encoding', $readme['encoding']);
                     $readmesObj->setVar('rm_downloadurl', $readme['download_url']);
-                    $readmesObj->setVar('rm_datecreated',time());
+                    $readmesObj->setVar('rm_datecreated', time());
                     $readmesObj->setVar('rm_submitter', $submitter);
                     // Insert Data
                     if (!$readmesHandler->insert($readmesObj)) {
@@ -240,7 +240,7 @@ 
             $readmesObj->setVar('rm_content', $readme['content']);
             $readmesObj->setVar('rm_encoding', $readme['encoding']);
             $readmesObj->setVar('rm_downloadurl', $readme['download_url']);
-            $readmesObj->setVar('rm_datecreated',time());
+            $readmesObj->setVar('rm_datecreated', time());
             $readmesObj->setVar('rm_submitter', $submitter);
             // Insert Data
             if (!$readmesHandler->insert($readmesObj)) {
@@ -278,10 +278,10 @@ 
     public function updateRepoReadme()
     {
         // update repo_readme
-        $sql = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('wggithub_repositories') . ' INNER JOIN ' . $GLOBALS['xoopsDB']->prefix('wggithub_readmes');
-        $sql .= ' ON ' . $GLOBALS['xoopsDB']->prefix('wggithub_repositories') . '.repo_id = ' . $GLOBALS['xoopsDB']->prefix('wggithub_readmes') . '.rm_repoid ';
-        $sql .= 'SET ' . $GLOBALS['xoopsDB']->prefix('wggithub_repositories') . '.repo_readme = 1 ';
-        $sql .= 'WHERE (((' . $GLOBALS['xoopsDB']->prefix('wggithub_readmes') . '.rm_id)>0));';
+        $sql = 'UPDATE '.$GLOBALS['xoopsDB']->prefix('wggithub_repositories').' INNER JOIN '.$GLOBALS['xoopsDB']->prefix('wggithub_readmes');
+        $sql .= ' ON '.$GLOBALS['xoopsDB']->prefix('wggithub_repositories').'.repo_id = '.$GLOBALS['xoopsDB']->prefix('wggithub_readmes').'.rm_repoid ';
+        $sql .= 'SET '.$GLOBALS['xoopsDB']->prefix('wggithub_repositories').'.repo_readme = 1 ';
+        $sql .= 'WHERE ((('.$GLOBALS['xoopsDB']->prefix('wggithub_readmes').'.rm_id)>0));';
         $GLOBALS['xoopsDB']->queryF($sql);
 
         return true;

class/PermissionsHandler.php

Indentation +98 added lines, -98 removed lines

@@ -33,110 +33,110 @@
  */
 class PermissionsHandler extends \XoopsPersistableObjectHandler
 {
-	/**
-	 * Constructor
-	 *
-	 * @param null
-	 */
-	public function __construct()
-	{
-	}
+    /**
+     * Constructor
+     *
+     * @param null
+     */
+    public function __construct()
+    {
+    }
 
-	/**
-	 * @public function getPermGlobalRead
-	 * returns global right for reading from github
-	 *
-	 * @param null
-	 * @return bool
-	 */
-	public function getPermGlobalRead()
-	{
-		global $xoopsUser, $xoopsModule;
-		$currentuid = 0;
-		if (isset($xoopsUser) && \is_object($xoopsUser)) {
-			if ($xoopsUser->isAdmin($xoopsModule->mid())) {
-				return true;
-			}
-			$currentuid = $xoopsUser->uid();
-		}
-		$grouppermHandler = \xoops_getHandler('groupperm');
-		$mid = $xoopsModule->mid();
-		$memberHandler = \xoops_getHandler('member');
-		if (0 == $currentuid) {
-			$my_group_ids = [XOOPS_GROUP_ANONYMOUS];
-		} else {
-			$my_group_ids = $memberHandler->getGroupsByUser($currentuid);
-		}
-		if ($grouppermHandler->checkRight('wggithub_ac', Constants::PERM_GLOBAL_READ, $my_group_ids, $mid)) {
-			return true;
-		}
-		return false;
-	}
+    /**
+     * @public function getPermGlobalRead
+     * returns global right for reading from github
+     *
+     * @param null
+     * @return bool
+     */
+    public function getPermGlobalRead()
+    {
+        global $xoopsUser, $xoopsModule;
+        $currentuid = 0;
+        if (isset($xoopsUser) && \is_object($xoopsUser)) {
+            if ($xoopsUser->isAdmin($xoopsModule->mid())) {
+                return true;
+            }
+            $currentuid = $xoopsUser->uid();
+        }
+        $grouppermHandler = \xoops_getHandler('groupperm');
+        $mid = $xoopsModule->mid();
+        $memberHandler = \xoops_getHandler('member');
+        if (0 == $currentuid) {
+            $my_group_ids = [XOOPS_GROUP_ANONYMOUS];
+        } else {
+            $my_group_ids = $memberHandler->getGroupsByUser($currentuid);
+        }
+        if ($grouppermHandler->checkRight('wggithub_ac', Constants::PERM_GLOBAL_READ, $my_group_ids, $mid)) {
+            return true;
+        }
+        return false;
+    }
 
-	/**
-	 * @public function permGlobalSubmit
-	 * returns right for global view
-	 *
-	 * @param null
-	 * @return bool
-	 */
-	public function getPermGlobalView()
-	{
-		global $xoopsUser, $xoopsModule;
+    /**
+     * @public function permGlobalSubmit
+     * returns right for global view
+     *
+     * @param null
+     * @return bool
+     */
+    public function getPermGlobalView()
+    {
+        global $xoopsUser, $xoopsModule;
 
         if ($this->getPermGlobalRead()) {
             return true;
         }
 
-		$currentuid = 0;
-		if (isset($xoopsUser) && \is_object($xoopsUser)) {
-			if ($xoopsUser->isAdmin($xoopsModule->mid())) {
-				return true;
-			}
-			$currentuid = $xoopsUser->uid();
-		}
-		$grouppermHandler = \xoops_getHandler('groupperm');
-		$mid = $xoopsModule->mid();
-		$memberHandler = \xoops_getHandler('member');
-		if (0 == $currentuid) {
-			$my_group_ids = [XOOPS_GROUP_ANONYMOUS];
-		} else {
-			$my_group_ids = $memberHandler->getGroupsByUser($currentuid);
-		}
-		if ($grouppermHandler->checkRight('wggithub_ac', Constants::PERM_GLOBAL_VIEW, $my_group_ids, $mid)) {
-			return true;
-		}
-		return false;
-	}
+        $currentuid = 0;
+        if (isset($xoopsUser) && \is_object($xoopsUser)) {
+            if ($xoopsUser->isAdmin($xoopsModule->mid())) {
+                return true;
+            }
+            $currentuid = $xoopsUser->uid();
+        }
+        $grouppermHandler = \xoops_getHandler('groupperm');
+        $mid = $xoopsModule->mid();
+        $memberHandler = \xoops_getHandler('member');
+        if (0 == $currentuid) {
+            $my_group_ids = [XOOPS_GROUP_ANONYMOUS];
+        } else {
+            $my_group_ids = $memberHandler->getGroupsByUser($currentuid);
+        }
+        if ($grouppermHandler->checkRight('wggithub_ac', Constants::PERM_GLOBAL_VIEW, $my_group_ids, $mid)) {
+            return true;
+        }
+        return false;
+    }
 
-	/**
-	 * @public function getPermReadmeUpdate
-	 * returns right for updating existing readme
-	 *
-	 * @param null
-	 * @return bool
-	 */
-	public function getPermReadmeUpdate()
-	{
-		global $xoopsUser, $xoopsModule;
-		$currentuid = 0;
-		if (isset($xoopsUser) && \is_object($xoopsUser)) {
-			if ($xoopsUser->isAdmin($xoopsModule->mid())) {
-				return true;
-			}
-			$currentuid = $xoopsUser->uid();
-		}
-		$grouppermHandler = \xoops_getHandler('groupperm');
-		$mid = $xoopsModule->mid();
-		$memberHandler = \xoops_getHandler('member');
-		if (0 == $currentuid) {
-			$my_group_ids = [XOOPS_GROUP_ANONYMOUS];
-		} else {
-			$my_group_ids = $memberHandler->getGroupsByUser($currentuid);
-		}
-		if ($grouppermHandler->checkRight('wggithub_ac', Constants::PERM_README_UPDATE, $my_group_ids, $mid)) {
-			return true;
-		}
-		return false;
-	}
+    /**
+     * @public function getPermReadmeUpdate
+     * returns right for updating existing readme
+     *
+     * @param null
+     * @return bool
+     */
+    public function getPermReadmeUpdate()
+    {
+        global $xoopsUser, $xoopsModule;
+        $currentuid = 0;
+        if (isset($xoopsUser) && \is_object($xoopsUser)) {
+            if ($xoopsUser->isAdmin($xoopsModule->mid())) {
+                return true;
+            }
+            $currentuid = $xoopsUser->uid();
+        }
+        $grouppermHandler = \xoops_getHandler('groupperm');
+        $mid = $xoopsModule->mid();
+        $memberHandler = \xoops_getHandler('member');
+        if (0 == $currentuid) {
+            $my_group_ids = [XOOPS_GROUP_ANONYMOUS];
+        } else {
+            $my_group_ids = $memberHandler->getGroupsByUser($currentuid);
+        }
+        if ($grouppermHandler->checkRight('wggithub_ac', Constants::PERM_README_UPDATE, $my_group_ids, $mid)) {
+            return true;
+        }
+        return false;
+    }
 }

testdata/index.php

Spacing +19 added lines, -19 removed lines

@@ -24,8 +24,8 @@ 
     Utility
 };
 
-require_once dirname(__DIR__, 3) . '/include/cp_header.php';
-require \dirname(__DIR__) . '/preloads/autoloader.php';
+require_once dirname(__DIR__, 3).'/include/cp_header.php';
+require \dirname(__DIR__).'/preloads/autoloader.php';
 
 $op = \Xmf\Request::getCmd('op', '');
 
@@ -45,7 +45,7 @@ 
             loadSampleData();
         } else {
             xoops_cp_header();
-            xoops_confirm(['ok' => 1, 'op' => 'load'], 'index.php', \sprintf(\constant('CO_' . $moduleDirNameUpper . '_' . 'ADD_SAMPLEDATA_OK')), \constant('CO_' . $moduleDirNameUpper . '_' . 'CONFIRM'), true);
+            xoops_confirm(['ok' => 1, 'op' => 'load'], 'index.php', \sprintf(\constant('CO_'.$moduleDirNameUpper.'_'.'ADD_SAMPLEDATA_OK')), \constant('CO_'.$moduleDirNameUpper.'_'.'CONFIRM'), true);
             xoops_cp_footer();
         }
         break;
@@ -68,20 +68,20 @@ 
     $tables = Helper::getHelper($moduleDirName)->getModule()->getInfo('tables');
 
     $language = 'english/';
-    if (\is_dir(__DIR__ . '/' . $xoopsConfig['language'])) {
-        $language = $xoopsConfig['language'] . '/';
+    if (\is_dir(__DIR__.'/'.$xoopsConfig['language'])) {
+        $language = $xoopsConfig['language'].'/';
     }
 
     // load module tables
     foreach ($tables as $table) {
-        $tabledata = Yaml::readWrapped($language . $table . '.yml');
+        $tabledata = Yaml::readWrapped($language.$table.'.yml');
         TableLoad::truncateTable($table);
         TableLoad::loadTableFromArray($table, $tabledata);
     }
 
     // load permissions
     $table     = 'group_permission';
-    $tabledata = Yaml::readWrapped($language . $table . '.yml');
+    $tabledata = Yaml::readWrapped($language.$table.'.yml');
     $mid       = Helper::getHelper($moduleDirName)->getModule()->getVar('mid');
     loadTableFromArrayWithReplace($table, $tabledata, 'gperm_modid', $mid);
 
@@ -94,7 +94,7 @@ 
             $utility::rcopy($src, $dest);
         }
     }
-    \redirect_header('../admin/index.php', 1, \constant('CO_' . $moduleDirNameUpper . '_' . 'SAMPLEDATA_SUCCESS'));
+    \redirect_header('../admin/index.php', 1, \constant('CO_'.$moduleDirNameUpper.'_'.'SAMPLEDATA_SUCCESS'));
 }
 
 function saveSampleData()
@@ -108,23 +108,23 @@ 
 
     $tables = Helper::getHelper($moduleDirName)->getModule()->getInfo('tables');
 
-    $languageFolder = __DIR__ . '/' . $xoopsConfig['language'];
-    if (!\file_exists($languageFolder . '/')) {
-        Utility::createFolder($languageFolder . '/');
+    $languageFolder = __DIR__.'/'.$xoopsConfig['language'];
+    if (!\file_exists($languageFolder.'/')) {
+        Utility::createFolder($languageFolder.'/');
     }
-    $exportFolder = $languageFolder . '/Exports-' . date('Y-m-d-H-i-s') . '/';
+    $exportFolder = $languageFolder.'/Exports-'.date('Y-m-d-H-i-s').'/';
     Utility::createFolder($exportFolder);
 
     // save module tables
     foreach ($tables as $table) {
-        TableLoad::saveTableToYamlFile($table, $exportFolder . $table . '.yml');
+        TableLoad::saveTableToYamlFile($table, $exportFolder.$table.'.yml');
     }
 
     // save permissions
     $criteria = new \CriteriaCompo();
     $criteria->add(new \Criteria('gperm_modid', Helper::getHelper($moduleDirName)->getModule()->getVar('mid')));
     $skipColumns[] = 'gperm_id';
-    TableLoad::saveTableToYamlFile('group_permission', $exportFolder . 'group_permission.yml', $criteria, $skipColumns);
+    TableLoad::saveTableToYamlFile('group_permission', $exportFolder.'group_permission.yml', $criteria, $skipColumns);
     unset($criteria);
 
     //  ---  COPY test folder files ---------------
@@ -135,7 +135,7 @@ 
             Utility::rcopy($src, $dest);
         }
     }
-    \redirect_header('../admin/index.php', 1, \constant('CO_' . $moduleDirNameUpper . '_' . 'SAVE_SAMPLEDATA_SUCCESS'));
+    \redirect_header('../admin/index.php', 1, \constant('CO_'.$moduleDirNameUpper.'_'.'SAVE_SAMPLEDATA_SUCCESS'));
 }
 
 function exportSchema()
@@ -150,7 +150,7 @@ 
         //
         //        \redirect_header('../admin/index.php', 1, \constant('CO_' . $moduleDirNameUpper . '_' . 'EXPORT_SCHEMA_SUCCESS'));
     } catch (\Exception $e) {
-        exit(\constant('CO_' . $moduleDirNameUpper . '_' . 'EXPORT_SCHEMA_ERROR'));
+        exit(\constant('CO_'.$moduleDirNameUpper.'_'.'EXPORT_SCHEMA_ERROR'));
     }
 
 }
@@ -173,10 +173,10 @@ 
     $db = \XoopsDatabaseFactory::getDatabaseConnection();
     $prefixedTable = $db->prefix($table);
     $count = 0;
-    $sql = 'DELETE FROM ' . $prefixedTable . ' WHERE `' . $search . '`=' . $db->quote($replace);
+    $sql = 'DELETE FROM '.$prefixedTable.' WHERE `'.$search.'`='.$db->quote($replace);
     $db->queryF($sql);
     foreach ($data as $row) {
-        $insertInto = 'INSERT INTO ' . $prefixedTable . ' (';
+        $insertInto = 'INSERT INTO '.$prefixedTable.' (';
         $valueClause = ' VALUES (';
         $first = true;
         foreach ($row as $column => $value) {
@@ -193,7 +193,7 @@ 
                 $valueClause .= $db->quote($value);
             }
         }
-        $sql = $insertInto . ') ' . $valueClause . ')';
+        $sql = $insertInto.') '.$valueClause.')';
         $result = $db->queryF($sql);
         if (false !== $result) {
             ++$count;

index.php

Spacing +15 added lines, -15 removed lines

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

class/SettingsHandler.php

Indentation +90 added lines, -90 removed lines

@@ -31,96 +31,96 @@
  */
 class SettingsHandler extends \XoopsPersistableObjectHandler
 {
-	/**
-	 * Constructor
-	 *
-	 * @param \XoopsDatabase $db
-	 */
-	public function __construct(\XoopsDatabase $db)
-	{
-		parent::__construct($db, 'wggithub_settings', Settings::class, 'set_id', 'set_token');
-	}
-
-	/**
-	 * @param bool $isNew
-	 *
-	 * @return object
-	 */
-	public function create($isNew = true)
-	{
-		return parent::create($isNew);
-	}
-
-	/**
-	 * retrieve a field
-	 *
-	 * @param int $i field id
-	 * @param null fields
-	 * @return mixed reference to the {@link Get} object
-	 */
-	public function get($i = null, $fields = null)
-	{
-		return parent::get($i, $fields);
-	}
-
-	/**
-	 * get inserted id
-	 *
-	 * @param null
-	 * @return int reference to the {@link Get} object
-	 */
-	public function getInsertId()
-	{
-		return $this->db->getInsertId();
-	}
-
-	/**
-	 * Get Count Settings in the database
-	 * @param int    $start
-	 * @param int    $limit
-	 * @param string $sort
-	 * @param string $order
-	 * @return int
-	 */
-	public function getCountSettings($start = 0, $limit = 0, $sort = 'set_id ASC, set_token', $order = 'ASC')
-	{
-		$crCountSettings = new \CriteriaCompo();
-		$crCountSettings = $this->getSettingsCriteria($crCountSettings, $start, $limit, $sort, $order);
-		return $this->getCount($crCountSettings);
-	}
-
-	/**
-	 * Get All Settings in the database
-	 * @param int    $start
-	 * @param int    $limit
-	 * @param string $sort
-	 * @param string $order
-	 * @return array
-	 */
-	public function getAllSettings($start = 0, $limit = 0, $sort = 'set_id ASC, set_token', $order = 'ASC')
-	{
-		$crAllSettings = new \CriteriaCompo();
-		$crAllSettings = $this->getSettingsCriteria($crAllSettings, $start, $limit, $sort, $order);
-		return $this->getAll($crAllSettings);
-	}
-
-	/**
-	 * Get Criteria Settings
-	 * @param        $crSettings
-	 * @param int    $start
-	 * @param int    $limit
-	 * @param string $sort
-	 * @param string $order
-	 * @return int
-	 */
-	private function getSettingsCriteria($crSettings, $start, $limit, $sort, $order)
-	{
-		$crSettings->setStart($start);
-		$crSettings->setLimit($limit);
-		$crSettings->setSort($sort);
-		$crSettings->setOrder($order);
-		return $crSettings;
-	}
+    /**
+     * Constructor
+     *
+     * @param \XoopsDatabase $db
+     */
+    public function __construct(\XoopsDatabase $db)
+    {
+        parent::__construct($db, 'wggithub_settings', Settings::class, 'set_id', 'set_token');
+    }
+
+    /**
+     * @param bool $isNew
+     *
+     * @return object
+     */
+    public function create($isNew = true)
+    {
+        return parent::create($isNew);
+    }
+
+    /**
+     * retrieve a field
+     *
+     * @param int $i field id
+     * @param null fields
+     * @return mixed reference to the {@link Get} object
+     */
+    public function get($i = null, $fields = null)
+    {
+        return parent::get($i, $fields);
+    }
+
+    /**
+     * get inserted id
+     *
+     * @param null
+     * @return int reference to the {@link Get} object
+     */
+    public function getInsertId()
+    {
+        return $this->db->getInsertId();
+    }
+
+    /**
+     * Get Count Settings in the database
+     * @param int    $start
+     * @param int    $limit
+     * @param string $sort
+     * @param string $order
+     * @return int
+     */
+    public function getCountSettings($start = 0, $limit = 0, $sort = 'set_id ASC, set_token', $order = 'ASC')
+    {
+        $crCountSettings = new \CriteriaCompo();
+        $crCountSettings = $this->getSettingsCriteria($crCountSettings, $start, $limit, $sort, $order);
+        return $this->getCount($crCountSettings);
+    }
+
+    /**
+     * Get All Settings in the database
+     * @param int    $start
+     * @param int    $limit
+     * @param string $sort
+     * @param string $order
+     * @return array
+     */
+    public function getAllSettings($start = 0, $limit = 0, $sort = 'set_id ASC, set_token', $order = 'ASC')
+    {
+        $crAllSettings = new \CriteriaCompo();
+        $crAllSettings = $this->getSettingsCriteria($crAllSettings, $start, $limit, $sort, $order);
+        return $this->getAll($crAllSettings);
+    }
+
+    /**
+     * Get Criteria Settings
+     * @param        $crSettings
+     * @param int    $start
+     * @param int    $limit
+     * @param string $sort
+     * @param string $order
+     * @return int
+     */
+    private function getSettingsCriteria($crSettings, $start, $limit, $sort, $order)
+    {
+        $crSettings->setStart($start);
+        $crSettings->setLimit($limit);
+        $crSettings->setSort($sort);
+        $crSettings->setOrder($order);
+        return $crSettings;
+    }
 
     /**
      * Get Primary Setting

Spacing +2 added lines, -2 removed lines

@@ -148,7 +148,7 @@ 
      */
     public function setPrimarySetting($setId)
     {
-        $helper  = \XoopsModules\Wggithub\Helper::getInstance();
+        $helper = \XoopsModules\Wggithub\Helper::getInstance();
         $settingsObj = null;
         $settingsHandler = $helper->getHandler('Settings');
         if (isset($setId)) {
@@ -158,7 +158,7 @@ 
         }
 
         // reset all
-        $strSQL = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('wggithub_settings') . ' SET ' . $GLOBALS['xoopsDB']->prefix('wggithub_settings') . '.set_primary = 0';
+        $strSQL = 'UPDATE '.$GLOBALS['xoopsDB']->prefix('wggithub_settings').' SET '.$GLOBALS['xoopsDB']->prefix('wggithub_settings').'.set_primary = 0';
         $GLOBALS['xoopsDB']->queryF($strSQL);
         // Set Vars
         $settingsObj->setVar('set_primary', 1);