ggoffy / wggithub

class/ReleasesHandler.php

Indentation +84 added lines, -84 removed lines

@@ -31,96 +31,96 @@
  */
 class ReleasesHandler extends \XoopsPersistableObjectHandler
 {
-	/**
-	 * Constructor
-	 *
-	 * @param \XoopsDatabase $db
-	 */
-	public function __construct(\XoopsDatabase $db)
-	{
-		parent::__construct($db, 'wggithub_releases', Releases::class, 'rel_id', 'rel_name');
-	}
+    /**
+     * Constructor
+     *
+     * @param \XoopsDatabase $db
+     */
+    public function __construct(\XoopsDatabase $db)
+    {
+        parent::__construct($db, 'wggithub_releases', Releases::class, 'rel_id', 'rel_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 Releases in the database
-	 * @param int    $start
-	 * @param int    $limit
-	 * @param string $sort
-	 * @param string $order
-	 * @return int
-	 */
-	public function getCountReleases($start = 0, $limit = 0, $sort = 'rel_id ASC, rel_name', $order = 'ASC')
-	{
-		$crCountReleases = new \CriteriaCompo();
-		$crCountReleases = $this->getReleasesCriteria($crCountReleases, $start, $limit, $sort, $order);
-		return $this->getCount($crCountReleases);
-	}
+    /**
+     * Get Count Releases in the database
+     * @param int    $start
+     * @param int    $limit
+     * @param string $sort
+     * @param string $order
+     * @return int
+     */
+    public function getCountReleases($start = 0, $limit = 0, $sort = 'rel_id ASC, rel_name', $order = 'ASC')
+    {
+        $crCountReleases = new \CriteriaCompo();
+        $crCountReleases = $this->getReleasesCriteria($crCountReleases, $start, $limit, $sort, $order);
+        return $this->getCount($crCountReleases);
+    }
 
-	/**
-	 * Get All Releases in the database
-	 * @param int    $start
-	 * @param int    $limit
-	 * @param string $sort
-	 * @param string $order
-	 * @return array
-	 */
-	public function getAllReleases($start = 0, $limit = 0, $sort = 'rel_id ASC, rel_name', $order = 'ASC')
-	{
-		$crAllReleases = new \CriteriaCompo();
-		$crAllReleases = $this->getReleasesCriteria($crAllReleases, $start, $limit, $sort, $order);
-		return $this->getAll($crAllReleases);
-	}
+    /**
+     * Get All Releases in the database
+     * @param int    $start
+     * @param int    $limit
+     * @param string $sort
+     * @param string $order
+     * @return array
+     */
+    public function getAllReleases($start = 0, $limit = 0, $sort = 'rel_id ASC, rel_name', $order = 'ASC')
+    {
+        $crAllReleases = new \CriteriaCompo();
+        $crAllReleases = $this->getReleasesCriteria($crAllReleases, $start, $limit, $sort, $order);
+        return $this->getAll($crAllReleases);
+    }
 
-	/**
-	 * Get Criteria Releases
-	 * @param        $crReleases
-	 * @param int    $start
-	 * @param int    $limit
-	 * @param string $sort
-	 * @param string $order
-	 * @return int
-	 */
-	private function getReleasesCriteria($crReleases, $start, $limit, $sort, $order)
-	{
-		$crReleases->setStart($start);
-		$crReleases->setLimit($limit);
-		$crReleases->setSort($sort);
-		$crReleases->setOrder($order);
-		return $crReleases;
-	}
+    /**
+     * Get Criteria Releases
+     * @param        $crReleases
+     * @param int    $start
+     * @param int    $limit
+     * @param string $sort
+     * @param string $order
+     * @return int
+     */
+    private function getReleasesCriteria($crReleases, $start, $limit, $sort, $order)
+    {
+        $crReleases->setStart($start);
+        $crReleases->setLimit($limit);
+        $crReleases->setSort($sort);
+        $crReleases->setOrder($order);
+        return $crReleases;
+    }
 
     /**
      * Update table requests

Spacing +16 added lines, -16 removed lines

@@ -149,7 +149,7 @@ 
                 if (Constants::STATUS_NEW === $repoStatus || Constants::STATUS_UPDATED === $repoStatus || 0 == $releasesCount) {
                     $ghReleases = $libReleases->getReleases($repositoriesAll[$i]->getVar('repo_user'), $repositoriesAll[$i]->getVar('repo_name'));
                     if ($releasesCount > 0) {
-                        $sql = 'DELETE FROM `' . $GLOBALS['xoopsDB']->prefix('wggithub_releases') . '` WHERE `rel_repoid` = ' . $repoId;
+                        $sql = 'DELETE FROM `'.$GLOBALS['xoopsDB']->prefix('wggithub_releases').'` WHERE `rel_repoid` = '.$repoId;
                         if (!$result = $GLOBALS['xoopsDB']->queryF($sql)) {
                             return false;
                         }
@@ -159,13 +159,13 @@ 
                 $first = true;
                 $final = false;
                 foreach ($ghReleases as $ghRelease) {
-                    if ($first || (!$final && !(bool)$ghRelease['prerelease'])) {
+                    if ($first || (!$final && !(bool) $ghRelease['prerelease'])) {
                         // save first in any case and save first final version
                         $releasesObj = $releasesHandler->create();
                         $releasesObj->setVar('rel_repoid', $repoId);
                         $releasesObj->setVar('rel_type', $ghRelease['type']);
                         $releasesObj->setVar('rel_name', $ghRelease['name']);
-                        $releasesObj->setVar('rel_prerelease', (true == (bool)$ghRelease['prerelease']));
+                        $releasesObj->setVar('rel_prerelease', (true == (bool) $ghRelease['prerelease']));
                         $releasesObj->setVar('rel_publishedat', \strtotime($ghRelease['published_at']));
                         $releasesObj->setVar('rel_tarballurl', $ghRelease['tarball_url']);
                         $releasesObj->setVar('rel_zipballurl', $ghRelease['zipball_url']);
@@ -177,7 +177,7 @@ 
                         }
                     }
                     $first = false;
-                    if (false === (bool)$ghRelease['prerelease']) {
+                    if (false === (bool) $ghRelease['prerelease']) {
                         $final = true;
                     }
                 }
@@ -215,7 +215,7 @@ 
                 // must return true otherwise releases will not be loaded
                 return true;
             }
-            $sql = 'DELETE FROM `' . $GLOBALS['xoopsDB']->prefix('wggithub_releases') . '` WHERE `rel_repoid` = ' . $repoId;
+            $sql = 'DELETE FROM `'.$GLOBALS['xoopsDB']->prefix('wggithub_releases').'` WHERE `rel_repoid` = '.$repoId;
             if (!$result = $GLOBALS['xoopsDB']->queryF($sql)) {
                 return false;
             }
@@ -224,13 +224,13 @@ 
             $first = true;
             $final = false;
             foreach ($ghReleases as $ghRelease) {
-                if ($first || (!$final && !(bool)$ghRelease['prerelease'])) {
+                if ($first || (!$final && !(bool) $ghRelease['prerelease'])) {
                     // save first in any case and save first final version
                     $releasesObj = $releasesHandler->create();
                     $releasesObj->setVar('rel_repoid', $repoId);
                     $releasesObj->setVar('rel_type', $ghRelease['type']);
                     $releasesObj->setVar('rel_name', $ghRelease['name']);
-                    $releasesObj->setVar('rel_prerelease', (true == (bool)$ghRelease['prerelease']));
+                    $releasesObj->setVar('rel_prerelease', (true == (bool) $ghRelease['prerelease']));
                     $releasesObj->setVar('rel_publishedat', \strtotime($ghRelease['published_at']));
                     $releasesObj->setVar('rel_tarballurl', $ghRelease['tarball_url']);
                     $releasesObj->setVar('rel_zipballurl', $ghRelease['zipball_url']);
@@ -242,7 +242,7 @@ 
                     }
                 }
                 $first = false;
-                if (false === (bool)$ghRelease['prerelease']) {
+                if (false === (bool) $ghRelease['prerelease']) {
                     $final = true;
                 }
             }
@@ -259,17 +259,17 @@ 
     public function updateRepoReleases()
     {
         // update repo_prerelease
-        $sql = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('wggithub_repositories') . ' INNER JOIN ' . $GLOBALS['xoopsDB']->prefix('wggithub_releases');
-        $sql .= ' ON ' . $GLOBALS['xoopsDB']->prefix('wggithub_repositories') . '.repo_id = ' . $GLOBALS['xoopsDB']->prefix('wggithub_releases') . '.rel_repoid ';
-        $sql .= 'SET ' . $GLOBALS['xoopsDB']->prefix('wggithub_repositories') . '.repo_prerelease = 1 ';
-        $sql .= 'WHERE (((' . $GLOBALS['xoopsDB']->prefix('wggithub_releases') . '.rel_prerelease)=1));';
+        $sql = 'UPDATE '.$GLOBALS['xoopsDB']->prefix('wggithub_repositories').' INNER JOIN '.$GLOBALS['xoopsDB']->prefix('wggithub_releases');
+        $sql .= ' ON '.$GLOBALS['xoopsDB']->prefix('wggithub_repositories').'.repo_id = '.$GLOBALS['xoopsDB']->prefix('wggithub_releases').'.rel_repoid ';
+        $sql .= 'SET '.$GLOBALS['xoopsDB']->prefix('wggithub_repositories').'.repo_prerelease = 1 ';
+        $sql .= 'WHERE ((('.$GLOBALS['xoopsDB']->prefix('wggithub_releases').'.rel_prerelease)=1));';
         $GLOBALS['xoopsDB']->queryF($sql);
 
         // update repo_release
-        $sql = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('wggithub_repositories') . ' INNER JOIN ' . $GLOBALS['xoopsDB']->prefix('wggithub_releases');
-        $sql .= ' ON ' . $GLOBALS['xoopsDB']->prefix('wggithub_repositories') . '.repo_id = ' . $GLOBALS['xoopsDB']->prefix('wggithub_releases') . '.rel_repoid ';
-        $sql .= 'SET ' . $GLOBALS['xoopsDB']->prefix('wggithub_repositories') . '.repo_release = 1 ';
-        $sql .= 'WHERE (((' . $GLOBALS['xoopsDB']->prefix('wggithub_releases') . '.rel_prerelease)=0));';
+        $sql = 'UPDATE '.$GLOBALS['xoopsDB']->prefix('wggithub_repositories').' INNER JOIN '.$GLOBALS['xoopsDB']->prefix('wggithub_releases');
+        $sql .= ' ON '.$GLOBALS['xoopsDB']->prefix('wggithub_repositories').'.repo_id = '.$GLOBALS['xoopsDB']->prefix('wggithub_releases').'.rel_repoid ';
+        $sql .= 'SET '.$GLOBALS['xoopsDB']->prefix('wggithub_repositories').'.repo_release = 1 ';
+        $sql .= 'WHERE ((('.$GLOBALS['xoopsDB']->prefix('wggithub_releases').'.rel_prerelease)=0));';
         $GLOBALS['xoopsDB']->queryF($sql);
 
         return true;

class/RepositoriesHandler.php

Indentation +84 added lines, -84 removed lines

@@ -31,94 +31,94 @@
  */
 class RepositoriesHandler extends \XoopsPersistableObjectHandler
 {
-	/**
-	 * Constructor
-	 *
-	 * @param \XoopsDatabase $db
-	 */
-	public function __construct(\XoopsDatabase $db)
-	{
-		parent::__construct($db, 'wggithub_repositories', Repositories::class, 'repo_id', 'repo_name');
-	}
+    /**
+     * Constructor
+     *
+     * @param \XoopsDatabase $db
+     */
+    public function __construct(\XoopsDatabase $db)
+    {
+        parent::__construct($db, 'wggithub_repositories', Repositories::class, 'repo_id', 'repo_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 Repositories in the database
-	 * @param int    $start
-	 * @param int    $limit
-	 * @param string $sort
-	 * @param string $order
-	 * @return int
-	 */
-	public function getCountRepositories($start = 0, $limit = 0, $sort = 'repo_id ASC, repo_name', $order = 'ASC')
-	{
-		$crCountRepositories = new \CriteriaCompo();
-		$crCountRepositories = $this->getRepositoriesCriteria($crCountRepositories, $start, $limit, $sort, $order);
-		return $this->getCount($crCountRepositories);
-	}
+    /**
+     * Get Count Repositories in the database
+     * @param int    $start
+     * @param int    $limit
+     * @param string $sort
+     * @param string $order
+     * @return int
+     */
+    public function getCountRepositories($start = 0, $limit = 0, $sort = 'repo_id ASC, repo_name', $order = 'ASC')
+    {
+        $crCountRepositories = new \CriteriaCompo();
+        $crCountRepositories = $this->getRepositoriesCriteria($crCountRepositories, $start, $limit, $sort, $order);
+        return $this->getCount($crCountRepositories);
+    }
 
-	/**
-	 * Get All Repositories in the database
-	 * @param int    $start
-	 * @param int    $limit
-	 * @param string $sort
-	 * @param string $order
-	 * @return array
-	 */
-	public function getAllRepositories($start = 0, $limit = 0, $sort = 'repo_id ASC, repo_name', $order = 'ASC')
-	{
-		$crAllRepositories = new \CriteriaCompo();
-		$crAllRepositories = $this->getRepositoriesCriteria($crAllRepositories, $start, $limit, $sort, $order);
-		return $this->getAll($crAllRepositories);
-	}
+    /**
+     * Get All Repositories in the database
+     * @param int    $start
+     * @param int    $limit
+     * @param string $sort
+     * @param string $order
+     * @return array
+     */
+    public function getAllRepositories($start = 0, $limit = 0, $sort = 'repo_id ASC, repo_name', $order = 'ASC')
+    {
+        $crAllRepositories = new \CriteriaCompo();
+        $crAllRepositories = $this->getRepositoriesCriteria($crAllRepositories, $start, $limit, $sort, $order);
+        return $this->getAll($crAllRepositories);
+    }
 
-	/**
-	 * Get Criteria Repositories
-	 * @param        $crRepositories
-	 * @param int    $start
-	 * @param int    $limit
-	 * @param string $sort
-	 * @param string $order
-	 * @return int
-	 */
-	private function getRepositoriesCriteria($crRepositories, $start, $limit, $sort, $order)
-	{
-		$crRepositories->setStart($start);
-		$crRepositories->setLimit($limit);
-		$crRepositories->setSort($sort);
-		$crRepositories->setOrder($order);
-		return $crRepositories;
-	}
+    /**
+     * Get Criteria Repositories
+     * @param        $crRepositories
+     * @param int    $start
+     * @param int    $limit
+     * @param string $sort
+     * @param string $order
+     * @return int
+     */
+    private function getRepositoriesCriteria($crRepositories, $start, $limit, $sort, $order)
+    {
+        $crRepositories->setStart($start);
+        $crRepositories->setLimit($limit);
+        $crRepositories->setSort($sort);
+        $crRepositories->setOrder($order);
+        return $crRepositories;
+    }
 }

class/ReadmesHandler.php

Indentation +90 added lines, -90 removed lines

@@ -31,96 +31,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');
-	}
-
-	/**
-	 * @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 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 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;
-	}
+    /**
+     * 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);
+    }
+
+    /**
+     * 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 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 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 +2 added lines, -2 removed lines

@@ -166,7 +166,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)) {
@@ -227,7 +227,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)) {

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_SUBMIT  = 2;
-	public const PERM_GLOBAL_APPROVE = 3;
+    // Constants for permissions
+    public const PERM_GLOBAL_NONE    = 0;
+    public const PERM_GLOBAL_VIEW    = 1;
+    public const PERM_GLOBAL_SUBMIT  = 2;
+    public const PERM_GLOBAL_APPROVE = 3;
 
 }

class/Releases.php

Indentation +111 added lines, -111 removed lines

@@ -32,135 +32,135 @@
  */
 class Releases extends \XoopsObject
 {
-	/**
-	 * Constructor
-	 *
-	 * @param null
-	 */
-	public function __construct()
-	{
-		$this->initVar('rel_id', XOBJ_DTYPE_INT);
+    /**
+     * Constructor
+     *
+     * @param null
+     */
+    public function __construct()
+    {
+        $this->initVar('rel_id', XOBJ_DTYPE_INT);
         $this->initVar('rel_repoid', XOBJ_DTYPE_INT);
-		$this->initVar('rel_type', XOBJ_DTYPE_INT);
-		$this->initVar('rel_name', XOBJ_DTYPE_TXTBOX);
-		$this->initVar('rel_prerelease', XOBJ_DTYPE_INT);
+        $this->initVar('rel_type', XOBJ_DTYPE_INT);
+        $this->initVar('rel_name', XOBJ_DTYPE_TXTBOX);
+        $this->initVar('rel_prerelease', XOBJ_DTYPE_INT);
         $this->initVar('rel_publishedat', XOBJ_DTYPE_INT);
-		$this->initVar('rel_tarballurl', XOBJ_DTYPE_TXTBOX);
-		$this->initVar('rel_zipballurl', XOBJ_DTYPE_TXTBOX);
-		$this->initVar('rel_datecreated', XOBJ_DTYPE_INT);
-		$this->initVar('rel_submitter', XOBJ_DTYPE_INT);
-	}
+        $this->initVar('rel_tarballurl', XOBJ_DTYPE_TXTBOX);
+        $this->initVar('rel_zipballurl', XOBJ_DTYPE_TXTBOX);
+        $this->initVar('rel_datecreated', XOBJ_DTYPE_INT);
+        $this->initVar('rel_submitter', XOBJ_DTYPE_INT);
+    }
 
-	/**
-	 * @static function &getInstance
-	 *
-	 * @param null
-	 */
-	public static function getInstance()
-	{
-		static $instance = false;
-		if (!$instance) {
-			$instance = new self();
-		}
-	}
+    /**
+     * @static function &getInstance
+     *
+     * @param null
+     */
+    public static function getInstance()
+    {
+        static $instance = false;
+        if (!$instance) {
+            $instance = new self();
+        }
+    }
 
-	/**
-	 * The new inserted $Id
-	 * @return inserted id
-	 */
-	public function getNewInsertedIdReleases()
-	{
-		$newInsertedId = $GLOBALS['xoopsDB']->getInsertId();
-		return $newInsertedId;
-	}
+    /**
+     * The new inserted $Id
+     * @return inserted id
+     */
+    public function getNewInsertedIdReleases()
+    {
+        $newInsertedId = $GLOBALS['xoopsDB']->getInsertId();
+        return $newInsertedId;
+    }
 
-	/**
-	 * @public function getForm
-	 * @param bool $action
-	 * @return \XoopsThemeForm
-	 */
-	public function getFormReleases($action = false)
-	{
-		$helper = \XoopsModules\Wggithub\Helper::getInstance();
-		if (!$action) {
-			$action = $_SERVER['REQUEST_URI'];
-		}
-		// Title
-		$title = $this->isNew() ? \sprintf(_AM_WGGITHUB_RELEASE_ADD) : \sprintf(_AM_WGGITHUB_RELEASE_EDIT);
-		// Get Theme Form
-		\xoops_load('XoopsFormLoader');
-		$form = new \XoopsThemeForm($title, 'form', $action, 'post', true);
-		$form->setExtra('enctype="multipart/form-data"');
+    /**
+     * @public function getForm
+     * @param bool $action
+     * @return \XoopsThemeForm
+     */
+    public function getFormReleases($action = false)
+    {
+        $helper = \XoopsModules\Wggithub\Helper::getInstance();
+        if (!$action) {
+            $action = $_SERVER['REQUEST_URI'];
+        }
+        // Title
+        $title = $this->isNew() ? \sprintf(_AM_WGGITHUB_RELEASE_ADD) : \sprintf(_AM_WGGITHUB_RELEASE_EDIT);
+        // Get Theme Form
+        \xoops_load('XoopsFormLoader');
+        $form = new \XoopsThemeForm($title, 'form', $action, 'post', true);
+        $form->setExtra('enctype="multipart/form-data"');
         // Form Table repositories
         $repositoriesHandler = $helper->getHandler('Repositories');
         $rmRepoidSelect = new \XoopsFormSelect(_AM_WGGITHUB_README_REPOID, 'rel_repoid', $this->getVar('rel_repoid'));
         $rmRepoidSelect->addOptionArray($repositoriesHandler->getList());
         $form->addElement($rmRepoidSelect);
-		// Form Select relType
-		$relTypeSelect = new \XoopsFormSelect(_AM_WGGITHUB_RELEASE_TYPE, 'rel_type', $this->getVar('rel_type'));
-		$relTypeSelect->addOption('file');
-		$form->addElement($relTypeSelect);
-		// Form Text relName
-		$form->addElement(new \XoopsFormText(_AM_WGGITHUB_RELEASE_NAME, 'rel_name', 50, 255, $this->getVar('rel_name')), true);
+        // Form Select relType
+        $relTypeSelect = new \XoopsFormSelect(_AM_WGGITHUB_RELEASE_TYPE, 'rel_type', $this->getVar('rel_type'));
+        $relTypeSelect->addOption('file');
+        $form->addElement($relTypeSelect);
+        // Form Text relName
+        $form->addElement(new \XoopsFormText(_AM_WGGITHUB_RELEASE_NAME, 'rel_name', 50, 255, $this->getVar('rel_name')), true);
         // Form Radio Yes/No relPrerelease
         $relPrerelease = $this->isNew() ?: $this->getVar('rel_prerelease');
         $form->addElement(new \XoopsFormRadioYN(_AM_WGGITHUB_RELEASE_PRERELEASE, 'rel_prerelease', $relPrerelease));
-		// Form Text Date Select relPublishedat
-		$relPublishedat = $this->isNew() ?: $this->getVar('rel_publishedat');
-		$form->addElement(new \XoopsFormTextDateSelect(_AM_WGGITHUB_RELEASE_PUBLISHEDAT, 'rel_publishedat', '', $relPublishedat));
-		// Form Text relTarballurl
-		$form->addElement(new \XoopsFormText(_AM_WGGITHUB_RELEASE_TARBALLURL, 'rel_tarballurl', 50, 255, $this->getVar('rel_tarballurl')));
-		// Form Text relZipballurl
-		$form->addElement(new \XoopsFormText(_AM_WGGITHUB_RELEASE_ZIPBALLURL, 'rel_zipballurl', 50, 255, $this->getVar('rel_zipballurl')));
-		// Form Text Date Select relDatecreated
-		$relDatecreated = $this->isNew() ?: $this->getVar('rel_datecreated');
-		$form->addElement(new \XoopsFormTextDateSelect(_AM_WGGITHUB_RELEASE_DATECREATED, 'rel_datecreated', '', $relDatecreated));
-		// Form Select User relSubmitter
-		$form->addElement(new \XoopsFormSelectUser(_AM_WGGITHUB_RELEASE_SUBMITTER, 'rel_submitter', false, $this->getVar('rel_submitter')));
-		// To Save
-		$form->addElement(new \XoopsFormHidden('op', 'save'));
-		$form->addElement(new \XoopsFormButtonTray('', _SUBMIT, 'submit', '', false));
-		return $form;
-	}
+        // Form Text Date Select relPublishedat
+        $relPublishedat = $this->isNew() ?: $this->getVar('rel_publishedat');
+        $form->addElement(new \XoopsFormTextDateSelect(_AM_WGGITHUB_RELEASE_PUBLISHEDAT, 'rel_publishedat', '', $relPublishedat));
+        // Form Text relTarballurl
+        $form->addElement(new \XoopsFormText(_AM_WGGITHUB_RELEASE_TARBALLURL, 'rel_tarballurl', 50, 255, $this->getVar('rel_tarballurl')));
+        // Form Text relZipballurl
+        $form->addElement(new \XoopsFormText(_AM_WGGITHUB_RELEASE_ZIPBALLURL, 'rel_zipballurl', 50, 255, $this->getVar('rel_zipballurl')));
+        // Form Text Date Select relDatecreated
+        $relDatecreated = $this->isNew() ?: $this->getVar('rel_datecreated');
+        $form->addElement(new \XoopsFormTextDateSelect(_AM_WGGITHUB_RELEASE_DATECREATED, 'rel_datecreated', '', $relDatecreated));
+        // Form Select User relSubmitter
+        $form->addElement(new \XoopsFormSelectUser(_AM_WGGITHUB_RELEASE_SUBMITTER, 'rel_submitter', false, $this->getVar('rel_submitter')));
+        // To Save
+        $form->addElement(new \XoopsFormHidden('op', 'save'));
+        $form->addElement(new \XoopsFormButtonTray('', _SUBMIT, 'submit', '', false));
+        return $form;
+    }
 
-	/**
-	 * Get Values
-	 * @param null $keys
-	 * @param null $format
-	 * @param null $maxDepth
-	 * @return array
-	 */
-	public function getValuesReleases($keys = null, $format = null, $maxDepth = null)
-	{
+    /**
+     * Get Values
+     * @param null $keys
+     * @param null $format
+     * @param null $maxDepth
+     * @return array
+     */
+    public function getValuesReleases($keys = null, $format = null, $maxDepth = null)
+    {
         $helper  = \XoopsModules\Wggithub\Helper::getInstance();
-	    $ret = $this->getValues($keys, $format, $maxDepth);
-		$ret['id']          = $this->getVar('rel_id');
+        $ret = $this->getValues($keys, $format, $maxDepth);
+        $ret['id']          = $this->getVar('rel_id');
         $repositoriesHandler = $helper->getHandler('Repositories');
         $repositoriesObj = $repositoriesHandler->get($this->getVar('rel_repoid'));
         $ret['repoid']        = $repositoriesObj->getVar('repo_name');
-		$ret['type']        = $this->getVar('rel_type');
-		$ret['name']        = $this->getVar('rel_name');
+        $ret['type']        = $this->getVar('rel_type');
+        $ret['name']        = $this->getVar('rel_name');
         $ret['prerelease']  = (int)$this->getVar('rel_prerelease') > 0 ? _YES : _NO;
-		$ret['publishedat'] = \formatTimestamp($this->getVar('rel_publishedat'), 'm');
-		$ret['tarballurl']  = $this->getVar('rel_tarballurl');
-		$ret['zipballurl']  = $this->getVar('rel_zipballurl');
-		$ret['datecreated'] = \formatTimestamp($this->getVar('rel_datecreated'), 'm');
-		$ret['submitter']   = \XoopsUser::getUnameFromId($this->getVar('rel_submitter'));
-		return $ret;
-	}
+        $ret['publishedat'] = \formatTimestamp($this->getVar('rel_publishedat'), 'm');
+        $ret['tarballurl']  = $this->getVar('rel_tarballurl');
+        $ret['zipballurl']  = $this->getVar('rel_zipballurl');
+        $ret['datecreated'] = \formatTimestamp($this->getVar('rel_datecreated'), 'm');
+        $ret['submitter']   = \XoopsUser::getUnameFromId($this->getVar('rel_submitter'));
+        return $ret;
+    }
 
-	/**
-	 * Returns an array representation of the object
-	 *
-	 * @return array
-	 */
-	public function toArrayReleases()
-	{
-		$ret = [];
-		$vars = $this->getVars();
-		foreach (\array_keys($vars) as $var) {
-			$ret[$var] = $this->getVar('"{$var}"');
-		}
-		return $ret;
-	}
+    /**
+     * Returns an array representation of the object
+     *
+     * @return array
+     */
+    public function toArrayReleases()
+    {
+        $ret = [];
+        $vars = $this->getVars();
+        foreach (\array_keys($vars) as $var) {
+            $ret[$var] = $this->getVar('"{$var}"');
+        }
+        return $ret;
+    }
 }

Spacing +4 added lines, -4 removed lines

@@ -132,15 +132,15 @@
 	 */
 	public function getValuesReleases($keys = null, $format = null, $maxDepth = null)
 	{
-        $helper  = \XoopsModules\Wggithub\Helper::getInstance();
+        $helper = \XoopsModules\Wggithub\Helper::getInstance();
 	    $ret = $this->getValues($keys, $format, $maxDepth);
-		$ret['id']          = $this->getVar('rel_id');
+		$ret['id'] = $this->getVar('rel_id');
         $repositoriesHandler = $helper->getHandler('Repositories');
         $repositoriesObj = $repositoriesHandler->get($this->getVar('rel_repoid'));
-        $ret['repoid']        = $repositoriesObj->getVar('repo_name');
+        $ret['repoid'] = $repositoriesObj->getVar('repo_name');
 		$ret['type']        = $this->getVar('rel_type');
 		$ret['name']        = $this->getVar('rel_name');
-        $ret['prerelease']  = (int)$this->getVar('rel_prerelease') > 0 ? _YES : _NO;
+        $ret['prerelease'] = (int) $this->getVar('rel_prerelease') > 0 ? _YES : _NO;
 		$ret['publishedat'] = \formatTimestamp($this->getVar('rel_publishedat'), 'm');
 		$ret['tarballurl']  = $this->getVar('rel_tarballurl');
 		$ret['zipballurl']  = $this->getVar('rel_zipballurl');

class/Repositories.php

Indentation +157 added lines, -157 removed lines

@@ -32,84 +32,84 @@ 
  */
 class Repositories extends \XoopsObject
 {
-	/**
-	 * Constructor
-	 *
-	 * @param null
-	 */
-	public function __construct()
-	{
-		$this->initVar('repo_id', XOBJ_DTYPE_INT);
-		$this->initVar('repo_nodeid', XOBJ_DTYPE_TXTBOX);
-		$this->initVar('repo_user', XOBJ_DTYPE_TXTBOX);
+    /**
+     * Constructor
+     *
+     * @param null
+     */
+    public function __construct()
+    {
+        $this->initVar('repo_id', XOBJ_DTYPE_INT);
+        $this->initVar('repo_nodeid', XOBJ_DTYPE_TXTBOX);
+        $this->initVar('repo_user', XOBJ_DTYPE_TXTBOX);
         $this->initVar('repo_name', XOBJ_DTYPE_TXTBOX);
-		$this->initVar('repo_fullname', XOBJ_DTYPE_TXTBOX);
-		$this->initVar('repo_createdat', XOBJ_DTYPE_INT);
-		$this->initVar('repo_updatedat', XOBJ_DTYPE_INT);
-		$this->initVar('repo_htmlurl', XOBJ_DTYPE_TXTBOX);
+        $this->initVar('repo_fullname', XOBJ_DTYPE_TXTBOX);
+        $this->initVar('repo_createdat', XOBJ_DTYPE_INT);
+        $this->initVar('repo_updatedat', XOBJ_DTYPE_INT);
+        $this->initVar('repo_htmlurl', XOBJ_DTYPE_TXTBOX);
         $this->initVar('repo_prerelease', XOBJ_DTYPE_TXTBOX);
         $this->initVar('repo_release', XOBJ_DTYPE_TXTBOX);
-		$this->initVar('repo_status', XOBJ_DTYPE_INT);
-		$this->initVar('repo_datecreated', XOBJ_DTYPE_INT);
-		$this->initVar('repo_submitter', XOBJ_DTYPE_INT);
-	}
+        $this->initVar('repo_status', XOBJ_DTYPE_INT);
+        $this->initVar('repo_datecreated', XOBJ_DTYPE_INT);
+        $this->initVar('repo_submitter', XOBJ_DTYPE_INT);
+    }
 
-	/**
-	 * @static function &getInstance
-	 *
-	 * @param null
-	 */
-	public static function getInstance()
-	{
-		static $instance = false;
-		if (!$instance) {
-			$instance = new self();
-		}
-	}
+    /**
+     * @static function &getInstance
+     *
+     * @param null
+     */
+    public static function getInstance()
+    {
+        static $instance = false;
+        if (!$instance) {
+            $instance = new self();
+        }
+    }
 
-	/**
-	 * The new inserted $Id
-	 * @return inserted id
-	 */
-	public function getNewInsertedIdRepositories()
-	{
-		$newInsertedId = $GLOBALS['xoopsDB']->getInsertId();
-		return $newInsertedId;
-	}
+    /**
+     * The new inserted $Id
+     * @return inserted id
+     */
+    public function getNewInsertedIdRepositories()
+    {
+        $newInsertedId = $GLOBALS['xoopsDB']->getInsertId();
+        return $newInsertedId;
+    }
 
-	/**
-	 * @public function getForm
-	 * @param bool $action
-	 * @return \XoopsThemeForm
-	 */
-	public function getFormRepositories($action = false)
-	{
-		$helper = \XoopsModules\Wggithub\Helper::getInstance();
-		if (!$action) {
-			$action = $_SERVER['REQUEST_URI'];
-		}
-		// Title
-		$title = $this->isNew() ? \sprintf(_AM_WGGITHUB_REPOSITORY_ADD) : \sprintf(_AM_WGGITHUB_REPOSITORY_EDIT);
-		// Get Theme Form
-		\xoops_load('XoopsFormLoader');
-		$form = new \XoopsThemeForm($title, 'form', $action, 'post', true);
-		$form->setExtra('enctype="multipart/form-data"');
-		// Form Text repoNodeid
-		$form->addElement(new \XoopsFormText(_AM_WGGITHUB_REPOSITORY_NODEID, 'repo_nodeid', 50, 255, $this->getVar('repo_nodeid')));
+    /**
+     * @public function getForm
+     * @param bool $action
+     * @return \XoopsThemeForm
+     */
+    public function getFormRepositories($action = false)
+    {
+        $helper = \XoopsModules\Wggithub\Helper::getInstance();
+        if (!$action) {
+            $action = $_SERVER['REQUEST_URI'];
+        }
+        // Title
+        $title = $this->isNew() ? \sprintf(_AM_WGGITHUB_REPOSITORY_ADD) : \sprintf(_AM_WGGITHUB_REPOSITORY_EDIT);
+        // Get Theme Form
+        \xoops_load('XoopsFormLoader');
+        $form = new \XoopsThemeForm($title, 'form', $action, 'post', true);
+        $form->setExtra('enctype="multipart/form-data"');
+        // Form Text repoNodeid
+        $form->addElement(new \XoopsFormText(_AM_WGGITHUB_REPOSITORY_NODEID, 'repo_nodeid', 50, 255, $this->getVar('repo_nodeid')));
         // Form Text repoUser
         $form->addElement(new \XoopsFormText(_AM_WGGITHUB_REPOSITORY_USER, 'repo_user', 50, 255, $this->getVar('repo_user')), true);
-		// Form Text repoName
-		$form->addElement(new \XoopsFormText(_AM_WGGITHUB_REPOSITORY_NAME, 'repo_name', 50, 255, $this->getVar('repo_name')), true);
-		// Form Text repoFullname
-		$form->addElement(new \XoopsFormText(_AM_WGGITHUB_REPOSITORY_FULLNAME, 'repo_fullname', 50, 255, $this->getVar('repo_fullname')));
-		// Form Text Date Select repoCreatedat
-		$repoCreatedat = $this->isNew() ?: $this->getVar('repo_createdat');
-		$form->addElement(new \XoopsFormTextDateSelect(_AM_WGGITHUB_REPOSITORY_CREATEDAT, 'repo_createdat', '', $repoCreatedat));
-		// Form Text Date Select repoUpdatedat
-		$repoUpdatedat = $this->isNew() ?: $this->getVar('repo_updatedat');
-		$form->addElement(new \XoopsFormTextDateSelect(_AM_WGGITHUB_REPOSITORY_UPDATEDAT, 'repo_updatedat', '', $repoUpdatedat));
-		// Form Text repoHtmlurl
-		$form->addElement(new \XoopsFormText(_AM_WGGITHUB_REPOSITORY_HTMLURL, 'repo_htmlurl', 50, 255, $this->getVar('repo_htmlurl')));
+        // Form Text repoName
+        $form->addElement(new \XoopsFormText(_AM_WGGITHUB_REPOSITORY_NAME, 'repo_name', 50, 255, $this->getVar('repo_name')), true);
+        // Form Text repoFullname
+        $form->addElement(new \XoopsFormText(_AM_WGGITHUB_REPOSITORY_FULLNAME, 'repo_fullname', 50, 255, $this->getVar('repo_fullname')));
+        // Form Text Date Select repoCreatedat
+        $repoCreatedat = $this->isNew() ?: $this->getVar('repo_createdat');
+        $form->addElement(new \XoopsFormTextDateSelect(_AM_WGGITHUB_REPOSITORY_CREATEDAT, 'repo_createdat', '', $repoCreatedat));
+        // Form Text Date Select repoUpdatedat
+        $repoUpdatedat = $this->isNew() ?: $this->getVar('repo_updatedat');
+        $form->addElement(new \XoopsFormTextDateSelect(_AM_WGGITHUB_REPOSITORY_UPDATEDAT, 'repo_updatedat', '', $repoUpdatedat));
+        // Form Text repoHtmlurl
+        $form->addElement(new \XoopsFormText(_AM_WGGITHUB_REPOSITORY_HTMLURL, 'repo_htmlurl', 50, 255, $this->getVar('repo_htmlurl')));
         // Form Text repoPrerelease
         $form->addElement(new \XoopsFormText(_AM_WGGITHUB_REPOSITORY_PRERELEASE, 'repo_prerelease', 50, 255, $this->getVar('repo_prerelease')));
         // Form Text repoRelease
@@ -122,100 +122,100 @@ 
         $repoStatusSelect->addOption(Constants::STATUS_UPDATED, _AM_WGGITHUB_STATUS_UPDATED);
         $form->addElement($repoStatusSelect);
         // Form Text Date Select repoDatecreated
-		$repoDatecreated = $this->isNew() ?: $this->getVar('repo_datecreated');
-		$form->addElement(new \XoopsFormTextDateSelect(_AM_WGGITHUB_REPOSITORY_DATECREATED, 'repo_datecreated', '', $repoDatecreated));
-		// Form Select User repoSubmitter
-		$form->addElement(new \XoopsFormSelectUser(_AM_WGGITHUB_REPOSITORY_SUBMITTER, 'repo_submitter', false, $this->getVar('repo_submitter')));
-		// Permissions
-		$memberHandler = \xoops_getHandler('member');
-		$groupList = $memberHandler->getGroupList();
-		$grouppermHandler = \xoops_getHandler('groupperm');
-		$fullList[] = \array_keys($groupList);
-		if (!$this->isNew()) {
-			$groupsIdsApprove = $grouppermHandler->getGroupIds('wggithub_approve_repositories', $this->getVar('repo_id'), $GLOBALS['xoopsModule']->getVar('mid'));
-			$groupsIdsApprove[] = \array_values($groupsIdsApprove);
-			$groupsCanApproveCheckbox = new \XoopsFormCheckBox(_AM_WGGITHUB_PERMISSIONS_APPROVE, 'groups_approve_repositories[]', $groupsIdsApprove);
-			$groupsIdsSubmit = $grouppermHandler->getGroupIds('wggithub_submit_repositories', $this->getVar('repo_id'), $GLOBALS['xoopsModule']->getVar('mid'));
-			$groupsIdsSubmit[] = \array_values($groupsIdsSubmit);
-			$groupsCanSubmitCheckbox = new \XoopsFormCheckBox(_AM_WGGITHUB_PERMISSIONS_SUBMIT, 'groups_submit_repositories[]', $groupsIdsSubmit);
-			$groupsIdsView = $grouppermHandler->getGroupIds('wggithub_view_repositories', $this->getVar('repo_id'), $GLOBALS['xoopsModule']->getVar('mid'));
-			$groupsIdsView[] = \array_values($groupsIdsView);
-			$groupsCanViewCheckbox = new \XoopsFormCheckBox(_AM_WGGITHUB_PERMISSIONS_VIEW, 'groups_view_repositories[]', $groupsIdsView);
-		} else {
-			$groupsCanApproveCheckbox = new \XoopsFormCheckBox(_AM_WGGITHUB_PERMISSIONS_APPROVE, 'groups_approve_repositories[]', $fullList);
-			$groupsCanSubmitCheckbox = new \XoopsFormCheckBox(_AM_WGGITHUB_PERMISSIONS_SUBMIT, 'groups_submit_repositories[]', $fullList);
-			$groupsCanViewCheckbox = new \XoopsFormCheckBox(_AM_WGGITHUB_PERMISSIONS_VIEW, 'groups_view_repositories[]', $fullList);
-		}
-		// To Approve
-		$groupsCanApproveCheckbox->addOptionArray($groupList);
-		$form->addElement($groupsCanApproveCheckbox);
-		// To Submit
-		$groupsCanSubmitCheckbox->addOptionArray($groupList);
-		$form->addElement($groupsCanSubmitCheckbox);
-		// To View
-		$groupsCanViewCheckbox->addOptionArray($groupList);
-		$form->addElement($groupsCanViewCheckbox);
-		// To Save
-		$form->addElement(new \XoopsFormHidden('op', 'save'));
-		$form->addElement(new \XoopsFormButtonTray('', _SUBMIT, 'submit', '', false));
-		return $form;
-	}
+        $repoDatecreated = $this->isNew() ?: $this->getVar('repo_datecreated');
+        $form->addElement(new \XoopsFormTextDateSelect(_AM_WGGITHUB_REPOSITORY_DATECREATED, 'repo_datecreated', '', $repoDatecreated));
+        // Form Select User repoSubmitter
+        $form->addElement(new \XoopsFormSelectUser(_AM_WGGITHUB_REPOSITORY_SUBMITTER, 'repo_submitter', false, $this->getVar('repo_submitter')));
+        // Permissions
+        $memberHandler = \xoops_getHandler('member');
+        $groupList = $memberHandler->getGroupList();
+        $grouppermHandler = \xoops_getHandler('groupperm');
+        $fullList[] = \array_keys($groupList);
+        if (!$this->isNew()) {
+            $groupsIdsApprove = $grouppermHandler->getGroupIds('wggithub_approve_repositories', $this->getVar('repo_id'), $GLOBALS['xoopsModule']->getVar('mid'));
+            $groupsIdsApprove[] = \array_values($groupsIdsApprove);
+            $groupsCanApproveCheckbox = new \XoopsFormCheckBox(_AM_WGGITHUB_PERMISSIONS_APPROVE, 'groups_approve_repositories[]', $groupsIdsApprove);
+            $groupsIdsSubmit = $grouppermHandler->getGroupIds('wggithub_submit_repositories', $this->getVar('repo_id'), $GLOBALS['xoopsModule']->getVar('mid'));
+            $groupsIdsSubmit[] = \array_values($groupsIdsSubmit);
+            $groupsCanSubmitCheckbox = new \XoopsFormCheckBox(_AM_WGGITHUB_PERMISSIONS_SUBMIT, 'groups_submit_repositories[]', $groupsIdsSubmit);
+            $groupsIdsView = $grouppermHandler->getGroupIds('wggithub_view_repositories', $this->getVar('repo_id'), $GLOBALS['xoopsModule']->getVar('mid'));
+            $groupsIdsView[] = \array_values($groupsIdsView);
+            $groupsCanViewCheckbox = new \XoopsFormCheckBox(_AM_WGGITHUB_PERMISSIONS_VIEW, 'groups_view_repositories[]', $groupsIdsView);
+        } else {
+            $groupsCanApproveCheckbox = new \XoopsFormCheckBox(_AM_WGGITHUB_PERMISSIONS_APPROVE, 'groups_approve_repositories[]', $fullList);
+            $groupsCanSubmitCheckbox = new \XoopsFormCheckBox(_AM_WGGITHUB_PERMISSIONS_SUBMIT, 'groups_submit_repositories[]', $fullList);
+            $groupsCanViewCheckbox = new \XoopsFormCheckBox(_AM_WGGITHUB_PERMISSIONS_VIEW, 'groups_view_repositories[]', $fullList);
+        }
+        // To Approve
+        $groupsCanApproveCheckbox->addOptionArray($groupList);
+        $form->addElement($groupsCanApproveCheckbox);
+        // To Submit
+        $groupsCanSubmitCheckbox->addOptionArray($groupList);
+        $form->addElement($groupsCanSubmitCheckbox);
+        // To View
+        $groupsCanViewCheckbox->addOptionArray($groupList);
+        $form->addElement($groupsCanViewCheckbox);
+        // To Save
+        $form->addElement(new \XoopsFormHidden('op', 'save'));
+        $form->addElement(new \XoopsFormButtonTray('', _SUBMIT, 'submit', '', false));
+        return $form;
+    }
 
-	/**
-	 * Get Values
-	 * @param null $keys
-	 * @param null $format
-	 * @param null $maxDepth
-	 * @return array
-	 */
-	public function getValuesRepositories($keys = null, $format = null, $maxDepth = null)
-	{
-		$ret = $this->getValues($keys, $format, $maxDepth);
-		$ret['id']          = $this->getVar('repo_id');
-		$ret['nodeid']      = $this->getVar('repo_nodeid');
+    /**
+     * Get Values
+     * @param null $keys
+     * @param null $format
+     * @param null $maxDepth
+     * @return array
+     */
+    public function getValuesRepositories($keys = null, $format = null, $maxDepth = null)
+    {
+        $ret = $this->getValues($keys, $format, $maxDepth);
+        $ret['id']          = $this->getVar('repo_id');
+        $ret['nodeid']      = $this->getVar('repo_nodeid');
         $ret['user']        = $this->getVar('repo_user');
-		$ret['name']        = $this->getVar('repo_name');
-		$ret['fullname']    = $this->getVar('repo_fullname');
-		$ret['createdat']   = \formatTimestamp($this->getVar('repo_createdat'), 'm');
-		$ret['updatedat']   = \formatTimestamp($this->getVar('repo_updatedat'), 'm');
-		$ret['htmlurl']     = $this->getVar('repo_htmlurl');
+        $ret['name']        = $this->getVar('repo_name');
+        $ret['fullname']    = $this->getVar('repo_fullname');
+        $ret['createdat']   = \formatTimestamp($this->getVar('repo_createdat'), 'm');
+        $ret['updatedat']   = \formatTimestamp($this->getVar('repo_updatedat'), 'm');
+        $ret['htmlurl']     = $this->getVar('repo_htmlurl');
         $ret['prerelease']  = $this->getVar('repo_prerelease');
         $ret['release']     = $this->getVar('repo_release');
-		$status             = $this->getVar('repo_status');
-		$ret['status']      = $status;
-		switch ($status) {
-			case Constants::STATUS_NONE:
-			default:
-				$status_text = \_AM_WGGITHUB_STATUS_NONE;
-				break;
+        $status             = $this->getVar('repo_status');
+        $ret['status']      = $status;
+        switch ($status) {
+            case Constants::STATUS_NONE:
+            default:
+                $status_text = \_AM_WGGITHUB_STATUS_NONE;
+                break;
             case Constants::STATUS_NEW:
                 $status_text = \_AM_WGGITHUB_STATUS_NEW;
                 break;
             case Constants::STATUS_UPTODATE:
-				$status_text = \_AM_WGGITHUB_STATUS_UPTODATE;
-				break;
-			case Constants::STATUS_UPDATED:
-				$status_text = \_AM_WGGITHUB_STATUS_UPDATED;
-				break;
-		}
-		$ret['status_text'] = $status_text;
-		$ret['datecreated'] = \formatTimestamp($this->getVar('repo_datecreated'), 'm');
-		$ret['submitter']   = \XoopsUser::getUnameFromId($this->getVar('repo_submitter'));
-		return $ret;
-	}
+                $status_text = \_AM_WGGITHUB_STATUS_UPTODATE;
+                break;
+            case Constants::STATUS_UPDATED:
+                $status_text = \_AM_WGGITHUB_STATUS_UPDATED;
+                break;
+        }
+        $ret['status_text'] = $status_text;
+        $ret['datecreated'] = \formatTimestamp($this->getVar('repo_datecreated'), 'm');
+        $ret['submitter']   = \XoopsUser::getUnameFromId($this->getVar('repo_submitter'));
+        return $ret;
+    }
 
-	/**
-	 * Returns an array representation of the object
-	 *
-	 * @return array
-	 */
-	public function toArrayRepositories()
-	{
-		$ret = [];
-		$vars = $this->getVars();
-		foreach (\array_keys($vars) as $var) {
-			$ret[$var] = $this->getVar('"{$var}"');
-		}
-		return $ret;
-	}
+    /**
+     * Returns an array representation of the object
+     *
+     * @return array
+     */
+    public function toArrayRepositories()
+    {
+        $ret = [];
+        $vars = $this->getVars();
+        foreach (\array_keys($vars) as $var) {
+            $ret[$var] = $this->getVar('"{$var}"');
+        }
+        return $ret;
+    }
 }

Spacing +1 added lines, -1 removed lines

@@ -173,7 +173,7 @@
 		$ret = $this->getValues($keys, $format, $maxDepth);
 		$ret['id']          = $this->getVar('repo_id');
 		$ret['nodeid']      = $this->getVar('repo_nodeid');
-        $ret['user']        = $this->getVar('repo_user');
+        $ret['user'] = $this->getVar('repo_user');
 		$ret['name']        = $this->getVar('repo_name');
 		$ret['fullname']    = $this->getVar('repo_fullname');
 		$ret['createdat']   = \formatTimestamp($this->getVar('repo_createdat'), 'm');

class/Directories.php

Indentation +135 added lines, -135 removed lines

@@ -33,77 +33,77 @@ 
  */
 class Directories extends \XoopsObject
 {
-	/**
-	 * Constructor
-	 *
-	 * @param null
-	 */
-	public function __construct()
-	{
-		$this->initVar('dir_id', \XOBJ_DTYPE_INT);
-		$this->initVar('dir_name', \XOBJ_DTYPE_TXTBOX);
-		$this->initVar('dir_type', \XOBJ_DTYPE_INT);
+    /**
+     * Constructor
+     *
+     * @param null
+     */
+    public function __construct()
+    {
+        $this->initVar('dir_id', \XOBJ_DTYPE_INT);
+        $this->initVar('dir_name', \XOBJ_DTYPE_TXTBOX);
+        $this->initVar('dir_type', \XOBJ_DTYPE_INT);
         $this->initVar('dir_autoupdate', XOBJ_DTYPE_INT);
         $this->initVar('dir_online', XOBJ_DTYPE_INT);
         $this->initVar('dir_filterrelease', XOBJ_DTYPE_INT);
-		$this->initVar('dir_datecreated', \XOBJ_DTYPE_INT);
-		$this->initVar('dir_submitter', \XOBJ_DTYPE_INT);
-	}
+        $this->initVar('dir_datecreated', \XOBJ_DTYPE_INT);
+        $this->initVar('dir_submitter', \XOBJ_DTYPE_INT);
+    }
 
-	/**
-	 * @static function &getInstance
-	 *
-	 * @param null
-	 */
-	public static function getInstance()
-	{
-		static $instance = false;
-		if (!$instance) {
-			$instance = new self();
-		}
-	}
+    /**
+     * @static function &getInstance
+     *
+     * @param null
+     */
+    public static function getInstance()
+    {
+        static $instance = false;
+        if (!$instance) {
+            $instance = new self();
+        }
+    }
 
-	/**
-	 * The new inserted $Id
-	 * @return inserted id
-	 */
-	public function getNewInsertedIdDirectories()
-	{
-		$newInsertedId = $GLOBALS['xoopsDB']->getInsertId();
-		return $newInsertedId;
-	}
+    /**
+     * The new inserted $Id
+     * @return inserted id
+     */
+    public function getNewInsertedIdDirectories()
+    {
+        $newInsertedId = $GLOBALS['xoopsDB']->getInsertId();
+        return $newInsertedId;
+    }
 
-	/**
-	 * @public function getForm
-	 * @param bool $action
-	 * @return \XoopsThemeForm
-	 */
-	public function getFormDirectories($action = false)
-	{
-		$helper = \XoopsModules\Wggithub\Helper::getInstance();
-		if (!$action) {
-			$action = $_SERVER['REQUEST_URI'];
-		}
-		$isAdmin = $GLOBALS['xoopsUser']->isAdmin($GLOBALS['xoopsModule']->mid());
-		// Permissions for uploader
-		$grouppermHandler = \xoops_getHandler('groupperm');
-		$groups = \is_object($GLOBALS['xoopsUser']) ? $GLOBALS['xoopsUser']->getGroups() : \XOOPS_GROUP_ANONYMOUS;
-		$permissionUpload = $grouppermHandler->checkRight('upload_groups', 32, $groups, $GLOBALS['xoopsModule']->getVar('mid')) ? true : false;
-		// Title
-		$title = $this->isNew() ? \sprintf(\_AM_WGGITHUB_DIRECTORY_ADD) : \sprintf(\_AM_WGGITHUB_DIRECTORY_EDIT);
-		// Get Theme Form
-		\xoops_load('XoopsFormLoader');
-		$form = new \XoopsThemeForm($title, 'form', $action, 'post', true);
-		$form->setExtra('enctype="multipart/form-data"');
-		// Form Text dirName
-		$form->addElement(new \XoopsFormText(\_AM_WGGITHUB_DIRECTORY_NAME, 'dir_name', 50, 255, $this->getVar('dir_name')), true);
-		// Directories Handler
-		$directoriesHandler = $helper->getHandler('Directories');
-		// Form Select dirType
-		$dirTypeSelect = new \XoopsFormSelect(\_AM_WGGITHUB_DIRECTORY_TYPE, 'dir_type', $this->getVar('dir_type'), 5);
-		$dirTypeSelect->addOption(Constants::DIRECTORY_TYPE_USER, \_AM_WGGITHUB_DIRECTORY_TYPE_USER);
-		$dirTypeSelect->addOption(Constants::DIRECTORY_TYPE_ORG, \_AM_WGGITHUB_DIRECTORY_TYPE_ORG);
-		$form->addElement($dirTypeSelect, true);
+    /**
+     * @public function getForm
+     * @param bool $action
+     * @return \XoopsThemeForm
+     */
+    public function getFormDirectories($action = false)
+    {
+        $helper = \XoopsModules\Wggithub\Helper::getInstance();
+        if (!$action) {
+            $action = $_SERVER['REQUEST_URI'];
+        }
+        $isAdmin = $GLOBALS['xoopsUser']->isAdmin($GLOBALS['xoopsModule']->mid());
+        // Permissions for uploader
+        $grouppermHandler = \xoops_getHandler('groupperm');
+        $groups = \is_object($GLOBALS['xoopsUser']) ? $GLOBALS['xoopsUser']->getGroups() : \XOOPS_GROUP_ANONYMOUS;
+        $permissionUpload = $grouppermHandler->checkRight('upload_groups', 32, $groups, $GLOBALS['xoopsModule']->getVar('mid')) ? true : false;
+        // Title
+        $title = $this->isNew() ? \sprintf(\_AM_WGGITHUB_DIRECTORY_ADD) : \sprintf(\_AM_WGGITHUB_DIRECTORY_EDIT);
+        // Get Theme Form
+        \xoops_load('XoopsFormLoader');
+        $form = new \XoopsThemeForm($title, 'form', $action, 'post', true);
+        $form->setExtra('enctype="multipart/form-data"');
+        // Form Text dirName
+        $form->addElement(new \XoopsFormText(\_AM_WGGITHUB_DIRECTORY_NAME, 'dir_name', 50, 255, $this->getVar('dir_name')), true);
+        // Directories Handler
+        $directoriesHandler = $helper->getHandler('Directories');
+        // Form Select dirType
+        $dirTypeSelect = new \XoopsFormSelect(\_AM_WGGITHUB_DIRECTORY_TYPE, 'dir_type', $this->getVar('dir_type'), 5);
+        $dirTypeSelect->addOption(Constants::DIRECTORY_TYPE_USER, \_AM_WGGITHUB_DIRECTORY_TYPE_USER);
+        $dirTypeSelect->addOption(Constants::DIRECTORY_TYPE_ORG, \_AM_WGGITHUB_DIRECTORY_TYPE_ORG);
+        $form->addElement($dirTypeSelect, true);
         // Form Radio Yes/No dirAutoupdate
         $dirAutoupdate = $this->isNew() ?: $this->getVar('dir_autoupdate');
         $form->addElement(new \XoopsFormRadioYN(_AM_WGGITHUB_DIRECTORY_AUTOUPDATE, 'dir_autoupdate', $dirAutoupdate));
@@ -113,80 +113,80 @@ 
         // Form Radio Yes/No dirFilterrelease
         $dirFilterrelease = $this->isNew() ?: $this->getVar('dir_filterrelease');
         $form->addElement(new \XoopsFormRadioYN(_AM_WGGITHUB_DIRECTORY_FILTERRELEASE, 'dir_filterrelease', $dirFilterrelease));
-		// Form Text Date Select dirDatecreated
-		$dirDatecreated = $this->isNew() ?: $this->getVar('dir_datecreated');
-		$form->addElement(new \XoopsFormTextDateSelect(\_AM_WGGITHUB_DIRECTORY_DATECREATED, 'dir_datecreated', '', $dirDatecreated));
-		// Form Select User dirSubmitter
-		$form->addElement(new \XoopsFormSelectUser(\_AM_WGGITHUB_DIRECTORY_SUBMITTER, 'dir_submitter', false, $this->getVar('dir_submitter')));
-		// Permissions
-		$memberHandler = \xoops_getHandler('member');
-		$groupList = $memberHandler->getGroupList();
-		$grouppermHandler = \xoops_getHandler('groupperm');
-		$fullList[] = \array_keys($groupList);
-		if (!$this->isNew()) {
-			$groupsIdsApprove = $grouppermHandler->getGroupIds('wggithub_approve_directories', $this->getVar('dir_id'), $GLOBALS['xoopsModule']->getVar('mid'));
-			$groupsIdsApprove[] = \array_values($groupsIdsApprove);
-			$groupsCanApproveCheckbox = new \XoopsFormCheckBox(\_AM_WGGITHUB_PERMISSIONS_APPROVE, 'groups_approve_directories[]', $groupsIdsApprove);
-			$groupsIdsSubmit = $grouppermHandler->getGroupIds('wggithub_submit_directories', $this->getVar('dir_id'), $GLOBALS['xoopsModule']->getVar('mid'));
-			$groupsIdsSubmit[] = \array_values($groupsIdsSubmit);
-			$groupsCanSubmitCheckbox = new \XoopsFormCheckBox(\_AM_WGGITHUB_PERMISSIONS_SUBMIT, 'groups_submit_directories[]', $groupsIdsSubmit);
-			$groupsIdsView = $grouppermHandler->getGroupIds('wggithub_view_directories', $this->getVar('dir_id'), $GLOBALS['xoopsModule']->getVar('mid'));
-			$groupsIdsView[] = \array_values($groupsIdsView);
-			$groupsCanViewCheckbox = new \XoopsFormCheckBox(\_AM_WGGITHUB_PERMISSIONS_VIEW, 'groups_view_directories[]', $groupsIdsView);
-		} else {
-			$groupsCanApproveCheckbox = new \XoopsFormCheckBox(\_AM_WGGITHUB_PERMISSIONS_APPROVE, 'groups_approve_directories[]', $fullList);
-			$groupsCanSubmitCheckbox = new \XoopsFormCheckBox(\_AM_WGGITHUB_PERMISSIONS_SUBMIT, 'groups_submit_directories[]', $fullList);
-			$groupsCanViewCheckbox = new \XoopsFormCheckBox(\_AM_WGGITHUB_PERMISSIONS_VIEW, 'groups_view_directories[]', $fullList);
-		}
-		// To Approve
-		$groupsCanApproveCheckbox->addOptionArray($groupList);
-		$form->addElement($groupsCanApproveCheckbox);
-		// To Submit
-		$groupsCanSubmitCheckbox->addOptionArray($groupList);
-		$form->addElement($groupsCanSubmitCheckbox);
-		// To View
-		$groupsCanViewCheckbox->addOptionArray($groupList);
-		$form->addElement($groupsCanViewCheckbox);
-		// To Save
-		$form->addElement(new \XoopsFormHidden('op', 'save'));
-		$form->addElement(new \XoopsFormButtonTray('', _SUBMIT, 'submit', '', false));
-		return $form;
-	}
+        // Form Text Date Select dirDatecreated
+        $dirDatecreated = $this->isNew() ?: $this->getVar('dir_datecreated');
+        $form->addElement(new \XoopsFormTextDateSelect(\_AM_WGGITHUB_DIRECTORY_DATECREATED, 'dir_datecreated', '', $dirDatecreated));
+        // Form Select User dirSubmitter
+        $form->addElement(new \XoopsFormSelectUser(\_AM_WGGITHUB_DIRECTORY_SUBMITTER, 'dir_submitter', false, $this->getVar('dir_submitter')));
+        // Permissions
+        $memberHandler = \xoops_getHandler('member');
+        $groupList = $memberHandler->getGroupList();
+        $grouppermHandler = \xoops_getHandler('groupperm');
+        $fullList[] = \array_keys($groupList);
+        if (!$this->isNew()) {
+            $groupsIdsApprove = $grouppermHandler->getGroupIds('wggithub_approve_directories', $this->getVar('dir_id'), $GLOBALS['xoopsModule']->getVar('mid'));
+            $groupsIdsApprove[] = \array_values($groupsIdsApprove);
+            $groupsCanApproveCheckbox = new \XoopsFormCheckBox(\_AM_WGGITHUB_PERMISSIONS_APPROVE, 'groups_approve_directories[]', $groupsIdsApprove);
+            $groupsIdsSubmit = $grouppermHandler->getGroupIds('wggithub_submit_directories', $this->getVar('dir_id'), $GLOBALS['xoopsModule']->getVar('mid'));
+            $groupsIdsSubmit[] = \array_values($groupsIdsSubmit);
+            $groupsCanSubmitCheckbox = new \XoopsFormCheckBox(\_AM_WGGITHUB_PERMISSIONS_SUBMIT, 'groups_submit_directories[]', $groupsIdsSubmit);
+            $groupsIdsView = $grouppermHandler->getGroupIds('wggithub_view_directories', $this->getVar('dir_id'), $GLOBALS['xoopsModule']->getVar('mid'));
+            $groupsIdsView[] = \array_values($groupsIdsView);
+            $groupsCanViewCheckbox = new \XoopsFormCheckBox(\_AM_WGGITHUB_PERMISSIONS_VIEW, 'groups_view_directories[]', $groupsIdsView);
+        } else {
+            $groupsCanApproveCheckbox = new \XoopsFormCheckBox(\_AM_WGGITHUB_PERMISSIONS_APPROVE, 'groups_approve_directories[]', $fullList);
+            $groupsCanSubmitCheckbox = new \XoopsFormCheckBox(\_AM_WGGITHUB_PERMISSIONS_SUBMIT, 'groups_submit_directories[]', $fullList);
+            $groupsCanViewCheckbox = new \XoopsFormCheckBox(\_AM_WGGITHUB_PERMISSIONS_VIEW, 'groups_view_directories[]', $fullList);
+        }
+        // To Approve
+        $groupsCanApproveCheckbox->addOptionArray($groupList);
+        $form->addElement($groupsCanApproveCheckbox);
+        // To Submit
+        $groupsCanSubmitCheckbox->addOptionArray($groupList);
+        $form->addElement($groupsCanSubmitCheckbox);
+        // To View
+        $groupsCanViewCheckbox->addOptionArray($groupList);
+        $form->addElement($groupsCanViewCheckbox);
+        // To Save
+        $form->addElement(new \XoopsFormHidden('op', 'save'));
+        $form->addElement(new \XoopsFormButtonTray('', _SUBMIT, 'submit', '', false));
+        return $form;
+    }
 
-	/**
-	 * Get Values
-	 * @param null $keys
-	 * @param null $format
-	 * @param null $maxDepth
-	 * @return array
-	 */
-	public function getValuesDirectories($keys = null, $format = null, $maxDepth = null)
-	{
-		$ret = $this->getValues($keys, $format, $maxDepth);
-		$ret['id']          = $this->getVar('dir_id');
-		$ret['name']        = $this->getVar('dir_name');
-		$ret['type']        = $this->getVar('dir_type');
+    /**
+     * Get Values
+     * @param null $keys
+     * @param null $format
+     * @param null $maxDepth
+     * @return array
+     */
+    public function getValuesDirectories($keys = null, $format = null, $maxDepth = null)
+    {
+        $ret = $this->getValues($keys, $format, $maxDepth);
+        $ret['id']          = $this->getVar('dir_id');
+        $ret['name']        = $this->getVar('dir_name');
+        $ret['type']        = $this->getVar('dir_type');
         $ret['type_text']   = Constants::DIRECTORY_TYPE_USER == $this->getVar('dir_type') ? \_AM_WGGITHUB_DIRECTORY_TYPE_USER : \_AM_WGGITHUB_DIRECTORY_TYPE_ORG;
         $ret['autoupdate']  = (int)$this->getVar('dir_autoupdate') > 0 ? _YES : _NO;
         $ret['online']      = (int)$this->getVar('dir_online') > 0 ? _YES : _NO;
         $ret['filterrelease'] = (int)$this->getVar('dir_filterrelease') > 0 ? _YES : _NO;
-		$ret['datecreated'] = \formatTimestamp($this->getVar('dir_datecreated'), 's');
-		$ret['submitter']   = \XoopsUser::getUnameFromId($this->getVar('dir_submitter'));
-		return $ret;
-	}
+        $ret['datecreated'] = \formatTimestamp($this->getVar('dir_datecreated'), 's');
+        $ret['submitter']   = \XoopsUser::getUnameFromId($this->getVar('dir_submitter'));
+        return $ret;
+    }
 
-	/**
-	 * Returns an array representation of the object
-	 *
-	 * @return array
-	 */
-	public function toArrayDirectories()
-	{
-		$ret = [];
-		$vars = $this->getVars();
-		foreach (\array_keys($vars) as $var) {
-			$ret[$var] = $this->getVar('"{$var}"');
-		}
-		return $ret;
-	}
+    /**
+     * Returns an array representation of the object
+     *
+     * @return array
+     */
+    public function toArrayDirectories()
+    {
+        $ret = [];
+        $vars = $this->getVars();
+        foreach (\array_keys($vars) as $var) {
+            $ret[$var] = $this->getVar('"{$var}"');
+        }
+        return $ret;
+    }
 }

Spacing +3 added lines, -3 removed lines

@@ -167,9 +167,9 @@
 		$ret['name']        = $this->getVar('dir_name');
 		$ret['type']        = $this->getVar('dir_type');
         $ret['type_text']   = Constants::DIRECTORY_TYPE_USER == $this->getVar('dir_type') ? \_AM_WGGITHUB_DIRECTORY_TYPE_USER : \_AM_WGGITHUB_DIRECTORY_TYPE_ORG;
-        $ret['autoupdate']  = (int)$this->getVar('dir_autoupdate') > 0 ? _YES : _NO;
-        $ret['online']      = (int)$this->getVar('dir_online') > 0 ? _YES : _NO;
-        $ret['filterrelease'] = (int)$this->getVar('dir_filterrelease') > 0 ? _YES : _NO;
+        $ret['autoupdate']  = (int) $this->getVar('dir_autoupdate') > 0 ? _YES : _NO;
+        $ret['online']      = (int) $this->getVar('dir_online') > 0 ? _YES : _NO;
+        $ret['filterrelease'] = (int) $this->getVar('dir_filterrelease') > 0 ? _YES : _NO;
 		$ret['datecreated'] = \formatTimestamp($this->getVar('dir_datecreated'), 's');
 		$ret['submitter']   = \XoopsUser::getUnameFromId($this->getVar('dir_submitter'));
 		return $ret;

class/Utility.php

Indentation +8 added lines, -8 removed lines

@@ -191,14 +191,14 @@
         $donationform = [
             0   => '<form name="donation" id="donation" action="http://www.txmodxoops.org/modules/xdonations/" method="post" onsubmit="return xoopsFormValidate_donation();">',
             1   => '<table class="outer" cellspacing="1" width="100%"><tbody><tr><th colspan="2">'
-                   . _AM_WGGITHUB_ABOUT_MAKE_DONATION
-                   . '</th></tr><tr align="left" valign="top"><td class="head"><div class="xoops-form-element-caption-required"><span class="caption-text">'
-                   . _AM_WGGITHUB_DONATION_AMOUNT
-                   . '</span><span class="caption-marker">*</span></div></td><td class="even"><select size="1" name="item[A][amount]" id="item[A][amount]" title="Donation Amount"><option value="5">5.00 EUR</option><option value="10">10.00 EUR</option><option value="20">20.00 EUR</option><option value="40">40.00 EUR</option><option value="60">60.00 EUR</option><option value="80">80.00 EUR</option><option value="90">90.00 EUR</option><option value="100">100.00 EUR</option><option value="200">200.00 EUR</option></select></td></tr><tr align="left" valign="top"><td class="head"></td><td class="even"><input class="formButton" name="submit" id="submit" value="'
-                   . _SUBMIT
-                   . '" title="'
-                   . _SUBMIT
-                   . '" type="submit"></td></tr></tbody></table>',
+                    . _AM_WGGITHUB_ABOUT_MAKE_DONATION
+                    . '</th></tr><tr align="left" valign="top"><td class="head"><div class="xoops-form-element-caption-required"><span class="caption-text">'
+                    . _AM_WGGITHUB_DONATION_AMOUNT
+                    . '</span><span class="caption-marker">*</span></div></td><td class="even"><select size="1" name="item[A][amount]" id="item[A][amount]" title="Donation Amount"><option value="5">5.00 EUR</option><option value="10">10.00 EUR</option><option value="20">20.00 EUR</option><option value="40">40.00 EUR</option><option value="60">60.00 EUR</option><option value="80">80.00 EUR</option><option value="90">90.00 EUR</option><option value="100">100.00 EUR</option><option value="200">200.00 EUR</option></select></td></tr><tr align="left" valign="top"><td class="head"></td><td class="even"><input class="formButton" name="submit" id="submit" value="'
+                    . _SUBMIT
+                    . '" title="'
+                    . _SUBMIT
+                    . '" type="submit"></td></tr></tbody></table>',
             2   => '<input name="op" id="op" value="createinvoice" type="hidden"><input name="plugin" id="plugin" value="donations" type="hidden"><input name="donation" id="donation" value="1" type="hidden"><input name="drawfor" id="drawfor" value="Chronolabs Co-Operative" type="hidden"><input name="drawto" id="drawto" value="%s" type="hidden"><input name="drawto_email" id="drawto_email" value="%s" type="hidden"><input name="key" id="key" value="%s" type="hidden"><input name="currency" id="currency" value="EUR" type="hidden"><input name="weight_unit" id="weight_unit" value="kgs" type="hidden"><input name="item[A][cat]" id="item[A][cat]" value="XDN%s" type="hidden"><input name="item[A][name]" id="item[A][name]" value="Donation for %s" type="hidden"><input name="item[A][quantity]" id="item[A][quantity]" value="1" type="hidden"><input name="item[A][shipping]" id="item[A][shipping]" value="0" type="hidden"><input name="item[A][handling]" id="item[A][handling]" value="0" type="hidden"><input name="item[A][weight]" id="item[A][weight]" value="0" type="hidden"><input name="item[A][tax]" id="item[A][tax]" value="0" type="hidden"><input name="return" id="return" value="http://www.txmodxoops.org/modules/xdonations/success.php" type="hidden"><input name="cancel" id="cancel" value="http://www.txmodxoops.org/modules/xdonations/success.php" type="hidden"></form>',
             'D' => '',
             3   => '',

Spacing +7 added lines, -7 removed lines

@@ -56,11 +56,11 @@ 
     {
         if ($considerHtml) {
             // if the plain text is shorter than the maximum length, return the whole text
-            if (mb_strlen(\preg_replace('/<.*?' . '>/', '', $text)) <= $length) {
+            if (mb_strlen(\preg_replace('/<.*?'.'>/', '', $text)) <= $length) {
                 return $text;
             }
             // splits all html-tags to scanable lines
-            \preg_match_all('/(<.+?' . '>)?([^<>]*)/s', $text, $lines, PREG_SET_ORDER);
+            \preg_match_all('/(<.+?'.'>)?([^<>]*)/s', $text, $lines, PREG_SET_ORDER);
             $total_length = mb_strlen($ending);
             $open_tags    = [];
             $truncate     = '';
@@ -78,7 +78,7 @@ 
                             unset($open_tags[$pos]);
                         }
                         // if tag is an opening tag
-                    } elseif (\preg_match('/^<\s*([^\s>!]+).*?' . '>$/s', $line_matchings[1], $tag_matchings)) {
+                    } elseif (\preg_match('/^<\s*([^\s>!]+).*?'.'>$/s', $line_matchings[1], $tag_matchings)) {
                         // add tag to the beginning of $open_tags list
                         \array_unshift($open_tags, \mb_strtolower($tag_matchings[1]));
                     }
@@ -138,7 +138,7 @@ 
         if ($considerHtml) {
             // close all unclosed html-tags
             foreach ($open_tags as $tag) {
-                $truncate .= '</' . $tag . '>';
+                $truncate .= '</'.$tag.'>';
             }
         }
 
@@ -211,7 +211,7 @@ 
 //--></script>
 <!-- End Form Validation JavaScript //-->',
         ];
-        $paypalform   = [
+        $paypalform = [
             0 => '<form action="https://www.paypal.com/cgi-bin/webscr" method="post">',
             1 => '<input name="cmd" value="_s-xclick" type="hidden">',
             2 => '<input name="hosted_button_id" value="%s" type="hidden">',
@@ -224,11 +224,11 @@ 
                 case 2:
                     $donationform[$key] = \sprintf(
                         $donationform[$key],
-                        $GLOBALS['xoopsConfig']['sitename'] . ' - ' . ('' != $GLOBALS['xoopsUser']->getVar('name') ? $GLOBALS['xoopsUser']->getVar('name') . ' [' . $GLOBALS['xoopsUser']->getVar('uname') . ']' : $GLOBALS['xoopsUser']->getVar('uname')),
+                        $GLOBALS['xoopsConfig']['sitename'].' - '.('' != $GLOBALS['xoopsUser']->getVar('name') ? $GLOBALS['xoopsUser']->getVar('name').' ['.$GLOBALS['xoopsUser']->getVar('uname').']' : $GLOBALS['xoopsUser']->getVar('uname')),
                         $GLOBALS['xoopsUser']->getVar('email'),
                         XOOPS_LICENSE_KEY,
                         \mb_strtoupper($GLOBALS['xoopsModule']->getVar('dirname')),
-                        \mb_strtoupper($GLOBALS['xoopsModule']->getVar('dirname')) . ' ' . $GLOBALS['xoopsModule']->getVar('name')
+                        \mb_strtoupper($GLOBALS['xoopsModule']->getVar('dirname')).' '.$GLOBALS['xoopsModule']->getVar('name')
                     );
                     break;
             }

class/Readmes.php

Indentation +122 added lines, -122 removed lines

@@ -33,140 +33,140 @@
  */
 class Readmes extends \XoopsObject
 {
-	/**
-	 * Constructor
-	 *
-	 * @param null
-	 */
-	public function __construct()
-	{
-		$this->initVar('rm_id', XOBJ_DTYPE_INT);
-		$this->initVar('rm_repoid', XOBJ_DTYPE_INT);
-		$this->initVar('rm_name', XOBJ_DTYPE_TXTBOX);
-		$this->initVar('rm_type', XOBJ_DTYPE_TXTBOX);
-		$this->initVar('rm_content', XOBJ_DTYPE_TXTAREA);
-		$this->initVar('rm_encoding', XOBJ_DTYPE_TXTBOX);
-		$this->initVar('rm_downloadurl', XOBJ_DTYPE_TXTBOX);
-		$this->initVar('rm_datecreated', XOBJ_DTYPE_INT);
-		$this->initVar('rm_submitter', XOBJ_DTYPE_INT);
-	}
+    /**
+     * Constructor
+     *
+     * @param null
+     */
+    public function __construct()
+    {
+        $this->initVar('rm_id', XOBJ_DTYPE_INT);
+        $this->initVar('rm_repoid', XOBJ_DTYPE_INT);
+        $this->initVar('rm_name', XOBJ_DTYPE_TXTBOX);
+        $this->initVar('rm_type', XOBJ_DTYPE_TXTBOX);
+        $this->initVar('rm_content', XOBJ_DTYPE_TXTAREA);
+        $this->initVar('rm_encoding', XOBJ_DTYPE_TXTBOX);
+        $this->initVar('rm_downloadurl', XOBJ_DTYPE_TXTBOX);
+        $this->initVar('rm_datecreated', XOBJ_DTYPE_INT);
+        $this->initVar('rm_submitter', XOBJ_DTYPE_INT);
+    }
 
-	/**
-	 * @static function &getInstance
-	 *
-	 * @param null
-	 */
-	public static function getInstance()
-	{
-		static $instance = false;
-		if (!$instance) {
-			$instance = new self();
-		}
-	}
+    /**
+     * @static function &getInstance
+     *
+     * @param null
+     */
+    public static function getInstance()
+    {
+        static $instance = false;
+        if (!$instance) {
+            $instance = new self();
+        }
+    }
 
-	/**
-	 * The new inserted $Id
-	 * @return inserted id
-	 */
-	public function getNewInsertedIdReadmes()
-	{
-		$newInsertedId = $GLOBALS['xoopsDB']->getInsertId();
-		return $newInsertedId;
-	}
+    /**
+     * The new inserted $Id
+     * @return inserted id
+     */
+    public function getNewInsertedIdReadmes()
+    {
+        $newInsertedId = $GLOBALS['xoopsDB']->getInsertId();
+        return $newInsertedId;
+    }
 
-	/**
-	 * @public function getForm
-	 * @param bool $action
-	 * @return \XoopsThemeForm
-	 */
-	public function getFormReadmes($action = false)
-	{
-		$helper = \XoopsModules\Wggithub\Helper::getInstance();
-		if (!$action) {
-			$action = $_SERVER['REQUEST_URI'];
-		}
-		$isAdmin = $GLOBALS['xoopsUser']->isAdmin($GLOBALS['xoopsModule']->mid());
-		// Title
-		$title = $this->isNew() ? \sprintf(_AM_WGGITHUB_README_ADD) : \sprintf(_AM_WGGITHUB_README_EDIT);
-		// Get Theme Form
-		\xoops_load('XoopsFormLoader');
-		$form = new \XoopsThemeForm($title, 'form', $action, 'post', true);
-		$form->setExtra('enctype="multipart/form-data"');
-		// Form Table repositories
-		$repositoriesHandler = $helper->getHandler('Repositories');
-		$rmRepoidSelect = new \XoopsFormSelect(_AM_WGGITHUB_README_REPOID, 'rm_repoid', $this->getVar('rm_repoid'));
-		$rmRepoidSelect->addOptionArray($repositoriesHandler->getList());
-		$form->addElement($rmRepoidSelect);
-		// Form Text rmName
-		$form->addElement(new \XoopsFormText(_AM_WGGITHUB_README_NAME, 'rm_name', 50, 255, $this->getVar('rm_name')));
-		// Form Text rmType
+    /**
+     * @public function getForm
+     * @param bool $action
+     * @return \XoopsThemeForm
+     */
+    public function getFormReadmes($action = false)
+    {
+        $helper = \XoopsModules\Wggithub\Helper::getInstance();
+        if (!$action) {
+            $action = $_SERVER['REQUEST_URI'];
+        }
+        $isAdmin = $GLOBALS['xoopsUser']->isAdmin($GLOBALS['xoopsModule']->mid());
+        // Title
+        $title = $this->isNew() ? \sprintf(_AM_WGGITHUB_README_ADD) : \sprintf(_AM_WGGITHUB_README_EDIT);
+        // Get Theme Form
+        \xoops_load('XoopsFormLoader');
+        $form = new \XoopsThemeForm($title, 'form', $action, 'post', true);
+        $form->setExtra('enctype="multipart/form-data"');
+        // Form Table repositories
+        $repositoriesHandler = $helper->getHandler('Repositories');
+        $rmRepoidSelect = new \XoopsFormSelect(_AM_WGGITHUB_README_REPOID, 'rm_repoid', $this->getVar('rm_repoid'));
+        $rmRepoidSelect->addOptionArray($repositoriesHandler->getList());
+        $form->addElement($rmRepoidSelect);
+        // Form Text rmName
+        $form->addElement(new \XoopsFormText(_AM_WGGITHUB_README_NAME, 'rm_name', 50, 255, $this->getVar('rm_name')));
+        // Form Text rmType
         $form->addElement(new \XoopsFormText(_AM_WGGITHUB_README_TYPE, 'rm_type', 50, 255, $this->getVar('rm_type')));
-		// Form Editor TextArea rmContent
-		$form->addElement(new \XoopsFormTextArea(_AM_WGGITHUB_README_CONTENT, 'rm_content', $this->getVar('rm_content', 'e'), 4, 47));
-		// Form Text rmEncoding
-		$form->addElement(new \XoopsFormText(_AM_WGGITHUB_README_ENCODING, 'rm_encoding', 50, 255, $this->getVar('rm_encoding')));
-		// Form Text rmDownloadurl
-		$form->addElement(new \XoopsFormText(_AM_WGGITHUB_README_DOWNLOADURL, 'rm_downloadurl', 50, 255, $this->getVar('rm_downloadurl')));
-		// Form Text Date Select rmDatecreated
-		$rmDatecreated = $this->isNew() ?: $this->getVar('rm_datecreated');
-		$form->addElement(new \XoopsFormTextDateSelect(_AM_WGGITHUB_README_DATECREATED, 'rm_datecreated', '', $rmDatecreated));
-		// Form Select User rmSubmitter
-		$form->addElement(new \XoopsFormSelectUser(_AM_WGGITHUB_README_SUBMITTER, 'rm_submitter', false, $this->getVar('rm_submitter')));
-		// To Save
-		$form->addElement(new \XoopsFormHidden('op', 'save'));
-		$form->addElement(new \XoopsFormButtonTray('', _SUBMIT, 'submit', '', false));
-		return $form;
-	}
+        // Form Editor TextArea rmContent
+        $form->addElement(new \XoopsFormTextArea(_AM_WGGITHUB_README_CONTENT, 'rm_content', $this->getVar('rm_content', 'e'), 4, 47));
+        // Form Text rmEncoding
+        $form->addElement(new \XoopsFormText(_AM_WGGITHUB_README_ENCODING, 'rm_encoding', 50, 255, $this->getVar('rm_encoding')));
+        // Form Text rmDownloadurl
+        $form->addElement(new \XoopsFormText(_AM_WGGITHUB_README_DOWNLOADURL, 'rm_downloadurl', 50, 255, $this->getVar('rm_downloadurl')));
+        // Form Text Date Select rmDatecreated
+        $rmDatecreated = $this->isNew() ?: $this->getVar('rm_datecreated');
+        $form->addElement(new \XoopsFormTextDateSelect(_AM_WGGITHUB_README_DATECREATED, 'rm_datecreated', '', $rmDatecreated));
+        // Form Select User rmSubmitter
+        $form->addElement(new \XoopsFormSelectUser(_AM_WGGITHUB_README_SUBMITTER, 'rm_submitter', false, $this->getVar('rm_submitter')));
+        // To Save
+        $form->addElement(new \XoopsFormHidden('op', 'save'));
+        $form->addElement(new \XoopsFormButtonTray('', _SUBMIT, 'submit', '', false));
+        return $form;
+    }
 
-	/**
-	 * Get Values
-	 * @param null $keys
-	 * @param null $format
-	 * @param null $maxDepth
-	 * @return array
-	 */
-	public function getValuesReadmes($keys = null, $format = null, $maxDepth = null)
-	{
-		$helper  = \XoopsModules\Wggithub\Helper::getInstance();
-		$utility = new \XoopsModules\Wggithub\Utility();
-		$ret = $this->getValues($keys, $format, $maxDepth);
-		$ret['id']            = $this->getVar('rm_id');
-		$repositoriesHandler = $helper->getHandler('Repositories');
-		$repositoriesObj = $repositoriesHandler->get($this->getVar('rm_repoid'));
-		$ret['repoid']        = $repositoriesObj->getVar('repo_name');
+    /**
+     * Get Values
+     * @param null $keys
+     * @param null $format
+     * @param null $maxDepth
+     * @return array
+     */
+    public function getValuesReadmes($keys = null, $format = null, $maxDepth = null)
+    {
+        $helper  = \XoopsModules\Wggithub\Helper::getInstance();
+        $utility = new \XoopsModules\Wggithub\Utility();
+        $ret = $this->getValues($keys, $format, $maxDepth);
+        $ret['id']            = $this->getVar('rm_id');
+        $repositoriesHandler = $helper->getHandler('Repositories');
+        $repositoriesObj = $repositoriesHandler->get($this->getVar('rm_repoid'));
+        $ret['repoid']        = $repositoriesObj->getVar('repo_name');
         $rmName = $this->getVar('rm_name');
-		$ret['name']          = $rmName;
-		$ret['type']          = $this->getVar('rm_type');
-		$ret['content']       = $this->getVar('rm_content', 'e');
+        $ret['name']          = $rmName;
+        $ret['type']          = $this->getVar('rm_type');
+        $ret['content']       = $this->getVar('rm_content', 'e');
         $contentDecoded = base64_decode($this->getVar('rm_content', 'n'));
-		if ('.MD' == substr(strtoupper($rmName), -3)) {
+        if ('.MD' == substr(strtoupper($rmName), -3)) {
             $Parsedown = new MDParser\Parsedown();
             $contentClean = $Parsedown->text($contentDecoded);
         } else {
             $contentClean = $contentDecoded;
         }
         $ret['content_clean'] = $contentClean;
-		$editorMaxchar = $helper->getConfig('editor_maxchar');
-		$ret['content_short'] = $utility::truncateHtml($ret['content'], $editorMaxchar);
-		$ret['encoding']      = $this->getVar('rm_encoding');
-		$ret['downloadurl']   = $this->getVar('rm_downloadurl');
-		$ret['datecreated']   = \formatTimestamp($this->getVar('rm_datecreated'), 'm');
-		$ret['submitter']     = \XoopsUser::getUnameFromId($this->getVar('rm_submitter'));
-		return $ret;
-	}
+        $editorMaxchar = $helper->getConfig('editor_maxchar');
+        $ret['content_short'] = $utility::truncateHtml($ret['content'], $editorMaxchar);
+        $ret['encoding']      = $this->getVar('rm_encoding');
+        $ret['downloadurl']   = $this->getVar('rm_downloadurl');
+        $ret['datecreated']   = \formatTimestamp($this->getVar('rm_datecreated'), 'm');
+        $ret['submitter']     = \XoopsUser::getUnameFromId($this->getVar('rm_submitter'));
+        return $ret;
+    }
 
-	/**
-	 * Returns an array representation of the object
-	 *
-	 * @return array
-	 */
-	public function toArrayReadmes()
-	{
-		$ret = [];
-		$vars = $this->getVars();
-		foreach (\array_keys($vars) as $var) {
-			$ret[$var] = $this->getVar('"{$var}"');
-		}
-		return $ret;
-	}
+    /**
+     * Returns an array representation of the object
+     *
+     * @return array
+     */
+    public function toArrayReadmes()
+    {
+        $ret = [];
+        $vars = $this->getVars();
+        foreach (\array_keys($vars) as $var) {
+            $ret[$var] = $this->getVar('"{$var}"');
+        }
+        return $ret;
+    }
 }