ggoffy / wggithub

class/Github/Storages/ISessionStorage.php

Indentation +20 added lines, -20 removed lines

@@ -8,25 +8,25 @@
  */
 interface ISessionStorage
 {
-	/**
-	 * @param  string
-	 * @param  mixed
-	 * @return self
-	 */
-	function set($name, $value);
-
-
-	/**
-	 * @param  string
-	 * @return mixed
-	 */
-	function get($name);
-
-
-	/**
-	 * @param  string
-	 * @return self
-	 */
-	function remove($name);
+    /**
+     * @param  string
+     * @param  mixed
+     * @return self
+     */
+    function set($name, $value);
+
+
+    /**
+     * @param  string
+     * @return mixed
+     */
+    function get($name);
+
+
+    /**
+     * @param  string
+     * @return self
+     */
+    function remove($name);
 
 }

class/Github/Storages/SessionStorage.php

Indentation +70 added lines, -70 removed lines

@@ -12,75 +12,75 @@
  */
 class SessionStorage extends Github\Sanity implements ISessionStorage
 {
-	const SESSION_KEY = 'milo.github-api';
-
-	/** @var string */
-	private $sessionKey;
-
-
-	/**
-	 * @param  string
-	 */
-	public function __construct($sessionKey = self::SESSION_KEY)
-	{
-		$this->sessionKey = $sessionKey;
-	}
-
-
-	/**
-	 * @param  string
-	 * @param  mixed
-	 * @return self
-	 */
-	public function set($name, $value)
-	{
-		if ($value === NULL) {
-			return $this->remove($name);
-		}
-
-		$this->check(__METHOD__);
-		$_SESSION[$this->sessionKey][$name] = $value;
-
-		return $this;
-	}
-
-
-	/**
-	 * @param  string
-	 * @return mixed
-	 */
-	public function get($name)
-	{
-		$this->check(__METHOD__);
-
-		return isset($_SESSION[$this->sessionKey][$name])
-			? $_SESSION[$this->sessionKey][$name]
-			: NULL;
-	}
-
-
-	/**
-	 * @param  string
-	 * @return self
-	 */
-	public function remove($name)
-	{
-		$this->check(__METHOD__);
-
-		unset($_SESSION[$this->sessionKey][$name]);
-
-		return $this;
-	}
-
-
-	/**
-	 * @param  string
-	 */
-	private function check($method)
-	{
-		if (!isset($_SESSION)) {
-			trigger_error("Start session before using $method().", E_USER_WARNING);
-		}
-	}
+    const SESSION_KEY = 'milo.github-api';
+
+    /** @var string */
+    private $sessionKey;
+
+
+    /**
+     * @param  string
+     */
+    public function __construct($sessionKey = self::SESSION_KEY)
+    {
+        $this->sessionKey = $sessionKey;
+    }
+
+
+    /**
+     * @param  string
+     * @param  mixed
+     * @return self
+     */
+    public function set($name, $value)
+    {
+        if ($value === NULL) {
+            return $this->remove($name);
+        }
+
+        $this->check(__METHOD__);
+        $_SESSION[$this->sessionKey][$name] = $value;
+
+        return $this;
+    }
+
+
+    /**
+     * @param  string
+     * @return mixed
+     */
+    public function get($name)
+    {
+        $this->check(__METHOD__);
+
+        return isset($_SESSION[$this->sessionKey][$name])
+            ? $_SESSION[$this->sessionKey][$name]
+            : NULL;
+    }
+
+
+    /**
+     * @param  string
+     * @return self
+     */
+    public function remove($name)
+    {
+        $this->check(__METHOD__);
+
+        unset($_SESSION[$this->sessionKey][$name]);
+
+        return $this;
+    }
+
+
+    /**
+     * @param  string
+     */
+    private function check($method)
+    {
+        if (!isset($_SESSION)) {
+            trigger_error("Start session before using $method().", E_USER_WARNING);
+        }
+    }
 
 }

class/Github/Helpers.php

Indentation +84 added lines, -84 removed lines

@@ -13,89 +13,89 @@
  */
 class Helpers
 {
-	private static $jsonMessages = [
-		JSON_ERROR_DEPTH => 'The maximum stack depth has been exceeded',
-		JSON_ERROR_STATE_MISMATCH => 'Syntax error, malformed JSON',
-		JSON_ERROR_CTRL_CHAR => 'Unexpected control character found',
-		JSON_ERROR_SYNTAX => 'Syntax error, malformed JSON',
-		JSON_ERROR_UTF8 => 'Invalid UTF-8 sequence',
-	];
-
-
-	/** @var Http\IClient */
-	private static $client;
-
-
-	/**
-	 * @param  mixed
-	 * @return string
-	 *
-	 * @throws JsonException
-	 */
-	public static function jsonEncode($value)
-	{
-		if (PHP_VERSION_ID < 50500) {
-			set_error_handler(function($severity, $message) { // needed to receive 'recursion detected' error
-				restore_error_handler();
-				throw new JsonException($message);
-			});
-		}
-
-		$json = json_encode($value, JSON_UNESCAPED_UNICODE);
-
-		if (PHP_VERSION_ID < 50500) {
-			restore_error_handler();
-		}
-
-		if ($error = json_last_error()) {
-			$message = isset(static::$jsonMessages[$error])
-				? static::$jsonMessages[$error]
-				: (PHP_VERSION_ID >= 50500 ? json_last_error_msg() : 'Unknown error');
-
-			throw new JsonException($message, $error);
-		}
-
-		$json = str_replace(array("\xe2\x80\xa8", "\xe2\x80\xa9"), array('\u2028', '\u2029'), $json);
-		return $json;
-	}
-
-
-	/**
-	 * @param  mixed
-	 * @return string
-	 *
-	 * @throws JsonException
-	 */
-	public static function jsonDecode($json, $assoc = false)
-	{
-		$json = (string) $json;
-		if (!preg_match('##u', $json)) {
-			throw new JsonException('Invalid UTF-8 sequence', 5); // PECL JSON-C
-		}
-
-		$value = json_decode($json, $assoc, 512, (defined('JSON_C_VERSION') && PHP_INT_SIZE > 4) ? 0 : JSON_BIGINT_AS_STRING);
-
-		if ($value === NULL && $json !== '' && strcasecmp($json, 'null')) { // '' does not clear json_last_error()
-			$error = json_last_error();
-			throw new JsonException(isset(static::$jsonMessages[$error]) ? static::$jsonMessages[$error] : 'Unknown error', $error);
-		}
-		return $value;
-	}
-
-
-	/**
-	 * @param  bool
-	 * @return Http\IClient
-	 */
-	public static function createDefaultClient($newInstance = FALSE)
-	{
-		if (self::$client === NULL || $newInstance) {
-			self::$client = extension_loaded('curl')
-				? new Http\CurlClient
-				: new Http\StreamClient;
-		}
-
-		return self::$client;
-	}
+    private static $jsonMessages = [
+        JSON_ERROR_DEPTH => 'The maximum stack depth has been exceeded',
+        JSON_ERROR_STATE_MISMATCH => 'Syntax error, malformed JSON',
+        JSON_ERROR_CTRL_CHAR => 'Unexpected control character found',
+        JSON_ERROR_SYNTAX => 'Syntax error, malformed JSON',
+        JSON_ERROR_UTF8 => 'Invalid UTF-8 sequence',
+    ];
+
+
+    /** @var Http\IClient */
+    private static $client;
+
+
+    /**
+     * @param  mixed
+     * @return string
+     *
+     * @throws JsonException
+     */
+    public static function jsonEncode($value)
+    {
+        if (PHP_VERSION_ID < 50500) {
+            set_error_handler(function($severity, $message) { // needed to receive 'recursion detected' error
+                restore_error_handler();
+                throw new JsonException($message);
+            });
+        }
+
+        $json = json_encode($value, JSON_UNESCAPED_UNICODE);
+
+        if (PHP_VERSION_ID < 50500) {
+            restore_error_handler();
+        }
+
+        if ($error = json_last_error()) {
+            $message = isset(static::$jsonMessages[$error])
+                ? static::$jsonMessages[$error]
+                : (PHP_VERSION_ID >= 50500 ? json_last_error_msg() : 'Unknown error');
+
+            throw new JsonException($message, $error);
+        }
+
+        $json = str_replace(array("\xe2\x80\xa8", "\xe2\x80\xa9"), array('\u2028', '\u2029'), $json);
+        return $json;
+    }
+
+
+    /**
+     * @param  mixed
+     * @return string
+     *
+     * @throws JsonException
+     */
+    public static function jsonDecode($json, $assoc = false)
+    {
+        $json = (string) $json;
+        if (!preg_match('##u', $json)) {
+            throw new JsonException('Invalid UTF-8 sequence', 5); // PECL JSON-C
+        }
+
+        $value = json_decode($json, $assoc, 512, (defined('JSON_C_VERSION') && PHP_INT_SIZE > 4) ? 0 : JSON_BIGINT_AS_STRING);
+
+        if ($value === NULL && $json !== '' && strcasecmp($json, 'null')) { // '' does not clear json_last_error()
+            $error = json_last_error();
+            throw new JsonException(isset(static::$jsonMessages[$error]) ? static::$jsonMessages[$error] : 'Unknown error', $error);
+        }
+        return $value;
+    }
+
+
+    /**
+     * @param  bool
+     * @return Http\IClient
+     */
+    public static function createDefaultClient($newInstance = FALSE)
+    {
+        if (self::$client === NULL || $newInstance) {
+            self::$client = extension_loaded('curl')
+                ? new Http\CurlClient
+                : new Http\StreamClient;
+        }
+
+        return self::$client;
+    }
 
 }

class/Github/GithubClient.php

Spacing +8 added lines, -8 removed lines

@@ -110,7 +110,7 @@ 
      */
     public function getUserRepositories($username, $per_page = 100, $page = 1)
     {
-        $url = static::BASE_URL . 'users/' . \rawurlencode($username) . '/repos?per_page=' . $per_page . '&page=' . $page;
+        $url = static::BASE_URL.'users/'.\rawurlencode($username).'/repos?per_page='.$per_page.'&page='.$page;
 
         return $this->_get($url);
     }
@@ -125,7 +125,7 @@ 
      */
     public function getOrgRepositories($org, $per_page = 100, $page = 1)
     {
-        $url = static::BASE_URL . 'orgs/' . \rawurlencode($org) . '/repos?per_page=' . $per_page . '&page=' . $page;
+        $url = static::BASE_URL.'orgs/'.\rawurlencode($org).'/repos?per_page='.$per_page.'&page='.$page;
 
         return $this->_get($url);
     }
@@ -142,7 +142,7 @@ 
      */
     public function getReadme($username, $repository)
     {
-        $url = static::BASE_URL . 'repos/' . \rawurlencode($username) . '/' . \rawurlencode($repository) . '/readme';
+        $url = static::BASE_URL.'repos/'.\rawurlencode($username).'/'.\rawurlencode($repository).'/readme';
 
         return $this->_get($url, false, false);
     }
@@ -156,7 +156,7 @@ 
      */
     public function getReleases($user, $repository)
     {
-        $url = static::BASE_URL . 'repos/' . $user . '/' . $repository . '/releases';
+        $url = static::BASE_URL.'repos/'.$user.'/'.$repository.'/releases';
 
         return $this->_get($url);
     }
@@ -172,9 +172,9 @@ 
     public function getLatestRelease($user, $repository, $prerelease = false)
     {
         if ($prerelease) {
-            $url = static::BASE_URL . 'repos/' . $user . '/' . $repository . '/releases';
+            $url = static::BASE_URL.'repos/'.$user.'/'.$repository.'/releases';
         } else {
-            $url = static::BASE_URL . 'repos/' . $user . '/' . $repository . '/releases/latest';
+            $url = static::BASE_URL.'repos/'.$user.'/'.$repository.'/releases/latest';
         }
         $result = $this->_get($url);
 
@@ -199,7 +199,7 @@ 
         $api->setToken($token);
         $request = $api->createRequest('GET', $url, [], [], '');
         $response = $api->request($request);
-        $data = (array)$api->decode($response);
+        $data = (array) $api->decode($response);
 
         return $data;
     }
@@ -216,7 +216,7 @@ 
         $setting = $settingsHandler->getPrimarySetting();
 
         if (0 == \count($setting)) {
-            \redirect_header(\XOOPS_URL . '/index.php', 3, \_AM_WGGITHUB_THEREARENT_SETTINGS);
+            \redirect_header(\XOOPS_URL.'/index.php', 3, \_AM_WGGITHUB_THEREARENT_SETTINGS);
         }
         $this->userAuth = $setting['user'];
         $this->tokenAuth = $setting['token'];

class/RepositoriesHandler.php

Indentation +90 added lines, -90 removed lines

@@ -31,96 +31,96 @@
  */
 class RepositoriesHandler extends \XoopsPersistableObjectHandler
 {
-	/**
-	 * 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);
-	}
-
-	/**
-	 * 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 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 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;
-	}
+    /**
+     * 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);
+    }
+
+    /**
+     * 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 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 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;
+    }
 
     /**
      * Update table repositories

class/Logs.php

Indentation +94 added lines, -94 removed lines

@@ -32,96 +32,96 @@ 
  */
 class Logs extends \XoopsObject
 {
-	/**
-	 * Constructor
-	 *
-	 * @param null
-	 */
-	public function __construct()
-	{
-		$this->initVar('log_id', XOBJ_DTYPE_INT);
+    /**
+     * Constructor
+     *
+     * @param null
+     */
+    public function __construct()
+    {
+        $this->initVar('log_id', XOBJ_DTYPE_INT);
         $this->initVar('log_type', XOBJ_DTYPE_INT);
-		$this->initVar('log_details', XOBJ_DTYPE_TXTBOX);
-		$this->initVar('log_result', XOBJ_DTYPE_TXTAREA);
-		$this->initVar('log_datecreated', XOBJ_DTYPE_INT);
-		$this->initVar('log_submitter', XOBJ_DTYPE_INT);
-	}
+        $this->initVar('log_details', XOBJ_DTYPE_TXTBOX);
+        $this->initVar('log_result', XOBJ_DTYPE_TXTAREA);
+        $this->initVar('log_datecreated', XOBJ_DTYPE_INT);
+        $this->initVar('log_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 getNewInsertedIdLogs()
-	{
-		$newInsertedId = $GLOBALS['xoopsDB']->getInsertId();
-		return $newInsertedId;
-	}
+    /**
+     * The new inserted $Id
+     * @return inserted id
+     */
+    public function getNewInsertedIdLogs()
+    {
+        $newInsertedId = $GLOBALS['xoopsDB']->getInsertId();
+        return $newInsertedId;
+    }
 
-	/**
-	 * @public function getForm
-	 * @param bool $action
-	 * @return \XoopsThemeForm
-	 */
-	public function getFormLogs($action = false)
-	{
-		$helper = \XoopsModules\Wggithub\Helper::getInstance();
-		if (!$action) {
-			$action = $_SERVER['REQUEST_URI'];
-		}
-		// Title
-		$title = $this->isNew() ? \sprintf(_AM_WGGITHUB_LOG_ADD) : \sprintf(_AM_WGGITHUB_LOG_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 getFormLogs($action = false)
+    {
+        $helper = \XoopsModules\Wggithub\Helper::getInstance();
+        if (!$action) {
+            $action = $_SERVER['REQUEST_URI'];
+        }
+        // Title
+        $title = $this->isNew() ? \sprintf(_AM_WGGITHUB_LOG_ADD) : \sprintf(_AM_WGGITHUB_LOG_EDIT);
+        // Get Theme Form
+        \xoops_load('XoopsFormLoader');
+        $form = new \XoopsThemeForm($title, 'form', $action, 'post', true);
+        $form->setExtra('enctype="multipart/form-data"');
 
         $logTypeSelect = new \XoopsFormSelect(_AM_WGGITHUB_LOG_TYPE, 'log_type', $this->getVar('log_type'));
         $logTypeSelect->addOption(Constants::LOG_TYPE_NONE, _AM_WGGITHUB_LOG_TYPE_NONE);
         $logTypeSelect->addOption(Constants::LOG_TYPE_UPDATE_START, _AM_WGGITHUB_LOG_TYPE_UPDATE_START);
         $logTypeSelect->addOption(Constants::LOG_TYPE_UPDATE_END, _AM_WGGITHUB_LOG_TYPE_UPDATE_END);
         $form->addElement($logTypeSelect);
-		// Form Text logDetails
-		$form->addElement(new \XoopsFormText(_AM_WGGITHUB_LOG_DETAILS, 'log_details', 50, 255, $this->getVar('log_details')), true);
-		// Form Text logResult
+        // Form Text logDetails
+        $form->addElement(new \XoopsFormText(_AM_WGGITHUB_LOG_DETAILS, 'log_details', 50, 255, $this->getVar('log_details')), true);
+        // Form Text logResult
         $form->addElement(new \XoopsFormTextArea(_AM_WGGITHUB_LOG_RESULT, 'log_result', $this->getVar('log_result', 'e'), 4, 47));
-		// Form Text Date Select logDatecreated
-		$logDatecreated = $this->isNew() ?: $this->getVar('log_datecreated');
-		$form->addElement(new \XoopsFormTextDateSelect(_AM_WGGITHUB_LOG_DATECREATED, 'log_datecreated', '', $logDatecreated));
-		// Form Select User reqSubmitter
-		$form->addElement(new \XoopsFormSelectUser(_AM_WGGITHUB_LOG_SUBMITTER, 'log_submitter', false, $this->getVar('log_submitter')));
-		// To Save
-		$form->addElement(new \XoopsFormHidden('op', 'save'));
-		$form->addElement(new \XoopsFormButtonTray('', _SUBMIT, 'submit', '', false));
-		return $form;
-	}
+        // Form Text Date Select logDatecreated
+        $logDatecreated = $this->isNew() ?: $this->getVar('log_datecreated');
+        $form->addElement(new \XoopsFormTextDateSelect(_AM_WGGITHUB_LOG_DATECREATED, 'log_datecreated', '', $logDatecreated));
+        // Form Select User reqSubmitter
+        $form->addElement(new \XoopsFormSelectUser(_AM_WGGITHUB_LOG_SUBMITTER, 'log_submitter', false, $this->getVar('log_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 getValuesLogs($keys = null, $format = null, $maxDepth = null)
-	{
+    /**
+     * Get Values
+     * @param null $keys
+     * @param null $format
+     * @param null $maxDepth
+     * @return array
+     */
+    public function getValuesLogs($keys = null, $format = null, $maxDepth = null)
+    {
         $helper  = \XoopsModules\Wggithub\Helper::getInstance();
         $utility = new \XoopsModules\Wggithub\Utility();
         $editorMaxchar = $helper->getConfig('editor_maxchar');
-	    $ret = $this->getValues($keys, $format, $maxDepth);
-		$ret['id']           = $this->getVar('log_id');
+        $ret = $this->getValues($keys, $format, $maxDepth);
+        $ret['id']           = $this->getVar('log_id');
         $ret['type']         = $this->getVar('log_type');
         switch ($ret['type']) {
             case Constants::LOG_TYPE_NONE:
@@ -139,26 +139,26 @@ 
                 break;
         }
         $ret['type_text']    = $type_text;
-		$ret['details']      = $this->getVar('log_details');
+        $ret['details']      = $this->getVar('log_details');
         $ret['result']       = \strip_tags($this->getVar('log_result', 'e'));
         $ret['result_short'] = $utility::truncateHtml($ret['result'], $editorMaxchar);
-		$ret['datecreated']  = \formatTimestamp($this->getVar('log_datecreated'), 'm');
-		$ret['submitter']    = \XoopsUser::getUnameFromId($this->getVar('log_submitter'));
-		return $ret;
-	}
+        $ret['datecreated']  = \formatTimestamp($this->getVar('log_datecreated'), 'm');
+        $ret['submitter']    = \XoopsUser::getUnameFromId($this->getVar('log_submitter'));
+        return $ret;
+    }
 
-	/**
-	 * Returns an array representation of the object
-	 *
-	 * @return array
-	 */
-	public function toArrayRequests()
-	{
-		$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 toArrayRequests()
+    {
+        $ret = [];
+        $vars = $this->getVars();
+        foreach (\array_keys($vars) as $var) {
+            $ret[$var] = $this->getVar('"{$var}"');
+        }
+        return $ret;
+    }
 }

Spacing +1 added lines, -1 removed lines

@@ -122,7 +122,7 @@
         $editorMaxchar = $helper->getConfig('editor_maxchar');
 	    $ret = $this->getValues($keys, $format, $maxDepth);
 		$ret['id']           = $this->getVar('log_id');
-        $ret['type']         = $this->getVar('log_type');
+        $ret['type'] = $this->getVar('log_type');
         switch ($ret['type']) {
             case Constants::LOG_TYPE_NONE:
             default:

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,17 +39,17 @@ 
     public const DIRECTORY_TYPE_USER = 1;
     public const DIRECTORY_TYPE_ORG  = 2;
 
-	// Constants for status
-	public const STATUS_NONE     = 0;
+    // Constants for status
+    public const STATUS_NONE     = 0;
     public const STATUS_NEW      = 1;
     public const STATUS_UPDATED  = 2;
-	public const STATUS_UPTODATE = 3;
+    public const STATUS_UPTODATE = 3;
 
-	// Constants for permissions
-	public const PERM_GLOBAL_NONE   = 0;
-	public const PERM_GLOBAL_VIEW   = 1;
-	public const PERM_GLOBAL_READ   = 2;
-	public const PERM_README_UPDATE = 3;
+    // Constants for permissions
+    public const PERM_GLOBAL_NONE   = 0;
+    public const PERM_GLOBAL_VIEW   = 1;
+    public const PERM_GLOBAL_READ   = 2;
+    public const PERM_README_UPDATE = 3;
 
     // Constants for log type
     public const LOG_TYPE_NONE  = 0;

Spacing +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
 	public const PERM_README_UPDATE = 3;
 
     // Constants for log type
-    public const LOG_TYPE_NONE  = 0;
+    public const LOG_TYPE_NONE = 0;
     public const LOG_TYPE_UPDATE_START = 1;
     public const LOG_TYPE_UPDATE_END   = 2;
     public const LOG_TYPE_REQUEST      = 3;

class/LogsHandler.php

Indentation +90 added lines, -90 removed lines

@@ -31,96 +31,96 @@
  */
 class LogsHandler extends \XoopsPersistableObjectHandler
 {
-	/**
-	 * Constructor
-	 *
-	 * @param \XoopsDatabase $db
-	 */
-	public function __construct(\XoopsDatabase $db)
-	{
-		parent::__construct($db, 'wggithub_logs', Logs::class, 'log_id', 'log_detail');
-	}
-
-	/**
-	 * @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 Logs in the database
-	 * @param int    $start
-	 * @param int    $limit
-	 * @param string $sort
-	 * @param string $order
-	 * @return int
-	 */
-	public function getCountLogs($start = 0, $limit = 0, $sort = 'log_id', $order = 'ASC')
-	{
-		$crCountLogs = new \CriteriaCompo();
-		$crCountLogs = $this->getLogsCriteria($crCountLogs, $start, $limit, $sort, $order);
-		return $this->getCount($crCountLogs);
-	}
-
-	/**
-	 * Get All Logs in the database
-	 * @param int    $start
-	 * @param int    $limit
-	 * @param string $sort
-	 * @param string $order
-	 * @return array
-	 */
-	public function getAllLogs($start = 0, $limit = 0, $sort = 'log_id', $order = 'ASC')
-	{
-		$crAllLogs = new \CriteriaCompo();
-		$crAllLogs = $this->getLogsCriteria($crAllLogs, $start, $limit, $sort, $order);
-		return $this->getAll($crAllLogs);
-	}
-
-	/**
-	 * Get Criteria Logs
-	 * @param        $crLogs
-	 * @param int    $start
-	 * @param int    $limit
-	 * @param string $sort
-	 * @param string $order
-	 * @return int
-	 */
-	private function getLogsCriteria($crLogs, $start, $limit, $sort, $order)
-	{
-		$crLogs->setStart($start);
-		$crLogs->setLimit($limit);
-		$crLogs->setSort($sort);
-		$crLogs->setOrder($order);
-		return $crLogs;
-	}
+    /**
+     * Constructor
+     *
+     * @param \XoopsDatabase $db
+     */
+    public function __construct(\XoopsDatabase $db)
+    {
+        parent::__construct($db, 'wggithub_logs', Logs::class, 'log_id', 'log_detail');
+    }
+
+    /**
+     * @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 Logs in the database
+     * @param int    $start
+     * @param int    $limit
+     * @param string $sort
+     * @param string $order
+     * @return int
+     */
+    public function getCountLogs($start = 0, $limit = 0, $sort = 'log_id', $order = 'ASC')
+    {
+        $crCountLogs = new \CriteriaCompo();
+        $crCountLogs = $this->getLogsCriteria($crCountLogs, $start, $limit, $sort, $order);
+        return $this->getCount($crCountLogs);
+    }
+
+    /**
+     * Get All Logs in the database
+     * @param int    $start
+     * @param int    $limit
+     * @param string $sort
+     * @param string $order
+     * @return array
+     */
+    public function getAllLogs($start = 0, $limit = 0, $sort = 'log_id', $order = 'ASC')
+    {
+        $crAllLogs = new \CriteriaCompo();
+        $crAllLogs = $this->getLogsCriteria($crAllLogs, $start, $limit, $sort, $order);
+        return $this->getAll($crAllLogs);
+    }
+
+    /**
+     * Get Criteria Logs
+     * @param        $crLogs
+     * @param int    $start
+     * @param int    $limit
+     * @param string $sort
+     * @param string $order
+     * @return int
+     */
+    private function getLogsCriteria($crLogs, $start, $limit, $sort, $order)
+    {
+        $crLogs->setStart($start);
+        $crLogs->setLimit($limit);
+        $crLogs->setSort($sort);
+        $crLogs->setOrder($order);
+        return $crLogs;
+    }
 
     /**
      * Update table requests

Spacing +1 added lines, -1 removed lines

@@ -142,7 +142,7 @@
         $logsObj->setVar('log_type', $type);
         $logsObj->setVar('log_details', $detail);
         $logsObj->setVar('log_result', $result);
-        $logsObj->setVar('log_datecreated',time());
+        $logsObj->setVar('log_datecreated', time());
         $logsObj->setVar('log_submitter', $submitter);
         // Insert Data
         if ($logsHandler->insert($logsObj)) {