ggoffy / wggithub

preloads/autoloader.php

Spacing +4 added lines, -4 removed lines

@@ -4,12 +4,12 @@ 
  * @see http://www.php-fig.org/psr/psr-4/examples/
  */
 spl_autoload_register(
-    static function ($class) {
+    static function($class) {
         // project-specific namespace prefix
-        $prefix = 'XoopsModules\\' . \ucfirst(\basename(\dirname(__DIR__)));
+        $prefix = 'XoopsModules\\'.\ucfirst(\basename(\dirname(__DIR__)));
 
         // base directory for the namespace prefix
-        $baseDir = __DIR__ . '/../class/';
+        $baseDir = __DIR__.'/../class/';
 
         // does the class use the namespace prefix?
         $len = mb_strlen($prefix);
@@ -24,7 +24,7 @@ 
         // replace the namespace prefix with the base directory, replace namespace
         // separators with directory separators in the relative class name, append
         // with .php
-        $file = $baseDir . \str_replace('\\', '/', $relativeClass) . '.php';
+        $file = $baseDir.\str_replace('\\', '/', $relativeClass).'.php';
 
         // if the file exists, require it
         if (\file_exists($file)) {

preloads/core.php

Spacing +1 added lines, -1 removed lines

@@ -28,6 +28,6 @@
      */
     public static function eventCoreIncludeCommonEnd($args)
     {
-        include __DIR__ . '/autoloader.php';
+        include __DIR__.'/autoloader.php';
     }
 }

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';

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;

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);

admin/logs.php

Indentation +101 added lines, -101 removed lines

@@ -31,107 +31,107 @@
 // Request log_id
 $logId = Request::getInt('log_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_logs.tpl';
-		$GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('logs.php'));
-		$adminObject->addItemButton(_AM_WGGITHUB_ADD_LOG, 'logs.php?op=new', 'add');
-		$GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
-		$logsCount = $logsHandler->getCountLogs();
-		$logsAll = $logsHandler->getAllLogs($start, $limit);
-		$GLOBALS['xoopsTpl']->assign('logs_count', $logsCount);
-		$GLOBALS['xoopsTpl']->assign('wggithub_url', WGGITHUB_URL);
-		$GLOBALS['xoopsTpl']->assign('wggithub_upload_url', WGGITHUB_UPLOAD_URL);
-		// Table view logs
-		if ($logsCount > 0) {
-			foreach (\array_keys($logsAll) as $i) {
-				$log = $logsAll[$i]->getValuesLogs();
-				$GLOBALS['xoopsTpl']->append('logs_list', $log);
-				unset($log);
-			}
-			// Display Navigation
-			if ($logsCount > $limit) {
-				include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
-				$pagenav = new \XoopsPageNav($logsCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
-				$GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
-			}
-		} else {
-			$GLOBALS['xoopsTpl']->assign('error', _AM_WGGITHUB_THEREARENT_LOGS);
-		}
-		break;
-	case 'new':
-		$templateMain = 'wggithub_admin_logs.tpl';
-		$GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('logs.php'));
-		$adminObject->addItemButton(_AM_WGGITHUB_LOGS_LIST, 'logs.php', 'list');
-		$GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
-		// Form Create
-		$logsObj = $logsHandler->create();
-		$form = $logsObj->getFormLogs();
-		$GLOBALS['xoopsTpl']->assign('form', $form->render());
-		break;
-	case 'save':
-		// Security Check
-		if (!$GLOBALS['xoopsSecurity']->check()) {
-			\redirect_header('logs.php', 3, \implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
-		}
-		if ($logId > 0) {
-			$logsObj = $logsHandler->get($logId);
-		} else {
-			$logsObj = $logsHandler->create();
-		}
-		// Set Vars
+    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_logs.tpl';
+        $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('logs.php'));
+        $adminObject->addItemButton(_AM_WGGITHUB_ADD_LOG, 'logs.php?op=new', 'add');
+        $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
+        $logsCount = $logsHandler->getCountLogs();
+        $logsAll = $logsHandler->getAllLogs($start, $limit);
+        $GLOBALS['xoopsTpl']->assign('logs_count', $logsCount);
+        $GLOBALS['xoopsTpl']->assign('wggithub_url', WGGITHUB_URL);
+        $GLOBALS['xoopsTpl']->assign('wggithub_upload_url', WGGITHUB_UPLOAD_URL);
+        // Table view logs
+        if ($logsCount > 0) {
+            foreach (\array_keys($logsAll) as $i) {
+                $log = $logsAll[$i]->getValuesLogs();
+                $GLOBALS['xoopsTpl']->append('logs_list', $log);
+                unset($log);
+            }
+            // Display Navigation
+            if ($logsCount > $limit) {
+                include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
+                $pagenav = new \XoopsPageNav($logsCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
+                $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
+            }
+        } else {
+            $GLOBALS['xoopsTpl']->assign('error', _AM_WGGITHUB_THEREARENT_LOGS);
+        }
+        break;
+    case 'new':
+        $templateMain = 'wggithub_admin_logs.tpl';
+        $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('logs.php'));
+        $adminObject->addItemButton(_AM_WGGITHUB_LOGS_LIST, 'logs.php', 'list');
+        $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
+        // Form Create
+        $logsObj = $logsHandler->create();
+        $form = $logsObj->getFormLogs();
+        $GLOBALS['xoopsTpl']->assign('form', $form->render());
+        break;
+    case 'save':
+        // Security Check
+        if (!$GLOBALS['xoopsSecurity']->check()) {
+            \redirect_header('logs.php', 3, \implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
+        }
+        if ($logId > 0) {
+            $logsObj = $logsHandler->get($logId);
+        } else {
+            $logsObj = $logsHandler->create();
+        }
+        // Set Vars
         $logsObj->setVar('log_type', Request::getInt('log_type', 0));
-		$logsObj->setVar('log_details', Request::getString('log_details', ''));
-		$logsObj->setVar('log_result', Request::getString('log_result', ''));
-		$logDatecreatedObj = \DateTime::createFromFormat(_SHORTDATESTRING, Request::getString('log_datecreated'));
-		$logsObj->setVar('log_datecreated', $logDatecreatedObj->getTimestamp());
-		$logsObj->setVar('log_submitter', Request::getInt('log_submitter', 0));
-		// Insert Data
-		if ($logsHandler->insert($logsObj)) {
-			\redirect_header('logs.php?op=list', 2, _AM_WGGITHUB_FORM_OK);
-		}
-		// Get Form
-		$GLOBALS['xoopsTpl']->assign('error', $logsObj->getHtmlErrors());
-		$form = $logsObj->getFormLogs();
-		$GLOBALS['xoopsTpl']->assign('form', $form->render());
-		break;
-	case 'edit':
-		$templateMain = 'wggithub_admin_logs.tpl';
-		$GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('logs.php'));
-		$adminObject->addItemButton(_AM_WGGITHUB_ADD_LOG, 'logs.php?op=new', 'add');
-		$adminObject->addItemButton(_AM_WGGITHUB_LOGS_LIST, 'logs.php', 'list');
-		$GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
-		// Get Form
-		$logsObj = $logsHandler->get($logId);
-		$form = $logsObj->getFormLogs();
-		$GLOBALS['xoopsTpl']->assign('form', $form->render());
-		break;
-	case 'delete':
-		$templateMain = 'wggithub_admin_logs.tpl';
-		$GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('logs.php'));
-		$logsObj = $logsHandler->get($logId);
-		$reqRequest = $logsObj->getVar('log_details');
-		if (isset($_REQUEST['ok']) && 1 == $_REQUEST['ok']) {
-			if (!$GLOBALS['xoopsSecurity']->check()) {
-				\redirect_header('logs.php', 3, \implode(', ', $GLOBALS['xoopsSecurity']->getErrors()));
-			}
-			if ($logsHandler->delete($logsObj)) {
-				\redirect_header('logs.php', 3, _AM_WGGITHUB_FORM_DELETE_OK);
-			} else {
-				$GLOBALS['xoopsTpl']->assign('error', $logsObj->getHtmlErrors());
-			}
-		} else {
-			$xoopsconfirm = new Common\XoopsConfirm(
-				['ok' => 1, 'log_id' => $logId, 'op' => 'delete'],
-				$_SERVER['REQUEST_URI'],
-				\sprintf(_AM_WGGITHUB_FORM_SURE_DELETE, $logsObj->getVar('log_details')));
-			$form = $xoopsconfirm->getFormXoopsConfirm();
-			$GLOBALS['xoopsTpl']->assign('form', $form->render());
-		}
-		break;
+        $logsObj->setVar('log_details', Request::getString('log_details', ''));
+        $logsObj->setVar('log_result', Request::getString('log_result', ''));
+        $logDatecreatedObj = \DateTime::createFromFormat(_SHORTDATESTRING, Request::getString('log_datecreated'));
+        $logsObj->setVar('log_datecreated', $logDatecreatedObj->getTimestamp());
+        $logsObj->setVar('log_submitter', Request::getInt('log_submitter', 0));
+        // Insert Data
+        if ($logsHandler->insert($logsObj)) {
+            \redirect_header('logs.php?op=list', 2, _AM_WGGITHUB_FORM_OK);
+        }
+        // Get Form
+        $GLOBALS['xoopsTpl']->assign('error', $logsObj->getHtmlErrors());
+        $form = $logsObj->getFormLogs();
+        $GLOBALS['xoopsTpl']->assign('form', $form->render());
+        break;
+    case 'edit':
+        $templateMain = 'wggithub_admin_logs.tpl';
+        $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('logs.php'));
+        $adminObject->addItemButton(_AM_WGGITHUB_ADD_LOG, 'logs.php?op=new', 'add');
+        $adminObject->addItemButton(_AM_WGGITHUB_LOGS_LIST, 'logs.php', 'list');
+        $GLOBALS['xoopsTpl']->assign('buttons', $adminObject->displayButton('left'));
+        // Get Form
+        $logsObj = $logsHandler->get($logId);
+        $form = $logsObj->getFormLogs();
+        $GLOBALS['xoopsTpl']->assign('form', $form->render());
+        break;
+    case 'delete':
+        $templateMain = 'wggithub_admin_logs.tpl';
+        $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('logs.php'));
+        $logsObj = $logsHandler->get($logId);
+        $reqRequest = $logsObj->getVar('log_details');
+        if (isset($_REQUEST['ok']) && 1 == $_REQUEST['ok']) {
+            if (!$GLOBALS['xoopsSecurity']->check()) {
+                \redirect_header('logs.php', 3, \implode(', ', $GLOBALS['xoopsSecurity']->getErrors()));
+            }
+            if ($logsHandler->delete($logsObj)) {
+                \redirect_header('logs.php', 3, _AM_WGGITHUB_FORM_DELETE_OK);
+            } else {
+                $GLOBALS['xoopsTpl']->assign('error', $logsObj->getHtmlErrors());
+            }
+        } else {
+            $xoopsconfirm = new Common\XoopsConfirm(
+                ['ok' => 1, 'log_id' => $logId, 'op' => 'delete'],
+                $_SERVER['REQUEST_URI'],
+                \sprintf(_AM_WGGITHUB_FORM_SURE_DELETE, $logsObj->getVar('log_details')));
+            $form = $xoopsconfirm->getFormXoopsConfirm();
+            $GLOBALS['xoopsTpl']->assign('form', $form->render());
+        }
+        break;
 }
 require __DIR__ . '/footer.php';

Spacing +4 added lines, -4 removed lines

@@ -25,7 +25,7 @@ 
 use XoopsModules\Wggithub\Constants;
 use XoopsModules\Wggithub\Common;
 
-require __DIR__ . '/header.php';
+require __DIR__.'/header.php';
 // It recovered the value of argument op in URL$
 $op = Request::getCmd('op', 'list');
 // Request log_id
@@ -55,8 +55,8 @@ 
 			}
 			// Display Navigation
 			if ($logsCount > $limit) {
-				include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
-				$pagenav = new \XoopsPageNav($logsCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
+				include_once XOOPS_ROOT_PATH.'/class/pagenav.php';
+				$pagenav = new \XoopsPageNav($logsCount, $limit, $start, 'start', 'op=list&limit='.$limit);
 				$GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
 			}
 		} else {
@@ -134,4 +134,4 @@ 
 		}
 		break;
 }
-require __DIR__ . '/footer.php';
+require __DIR__.'/footer.php';