ggoffy / wggithub

lib/Github/Http/Client/Internal/AmpBody.php

Spacing +1 added lines, -1 removed lines

@@ -78,7 +78,7 @@
         ($this->onProgress)();
 
         $chunk = $this->doRead();
-        $chunk->onResolve(function ($e, $data) {
+        $chunk->onResolve(function($e, $data) {
             if (null !== $data) {
                 $this->uploaded = \strlen($data);
             } else {

lib/Github/Http/Client/CurlHttpClient.php

Indentation +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
 
     private $defaultOptions = self::OPTIONS_DEFAULTS + [
         'auth_ntlm' => null, // array|string - an array containing the username as first value, and optionally the
-                             //   password as the second one; or string like username:password - enabling NTLM auth
+                                //   password as the second one; or string like username:password - enabling NTLM auth
         'extra' => [
             'curl' => [],    // A list of extra curl options indexed by their corresponding CURLOPT_*
         ],

Spacing +7 added lines, -7 removed lines

@@ -41,7 +41,7 @@ 
         'auth_ntlm' => null, // array|string - an array containing the username as first value, and optionally the
                              //   password as the second one; or string like username:password - enabling NTLM auth
         'extra' => [
-            'curl' => [],    // A list of extra curl options indexed by their corresponding CURLOPT_*
+            'curl' => [], // A list of extra curl options indexed by their corresponding CURLOPT_*
         ],
     ];
 
@@ -97,7 +97,7 @@ 
             return;
         }
 
-        curl_multi_setopt($this->multi->handle, \CURLMOPT_PUSHFUNCTION, function ($parent, $pushed, array $requestHeaders) use ($maxPendingPushes) {
+        curl_multi_setopt($this->multi->handle, \CURLMOPT_PUSHFUNCTION, function($parent, $pushed, array $requestHeaders) use ($maxPendingPushes) {
             return $this->handlePush($parent, $pushed, $requestHeaders, $maxPendingPushes);
         });
     }
@@ -243,7 +243,7 @@ 
             } else {
                 $eof = false;
                 $buffer = '';
-                $curlopts[\CURLOPT_READFUNCTION] = static function ($ch, $fd, $length) use ($body, &$buffer, &$eof) {
+                $curlopts[\CURLOPT_READFUNCTION] = static function($ch, $fd, $length) use ($body, &$buffer, &$eof) {
                     return self::readRequestBody($length, $body, $buffer, $eof);
                 };
             }
@@ -473,18 +473,18 @@ 
         $redirectHeaders = [];
         if (0 < $options['max_redirects']) {
             $redirectHeaders['host'] = $host;
-            $redirectHeaders['with_auth'] = $redirectHeaders['no_auth'] = \array_filter($options['headers'], static function ($h) {
+            $redirectHeaders['with_auth'] = $redirectHeaders['no_auth'] = \array_filter($options['headers'], static function($h) {
                 return 0 !== stripos($h, 'Host:');
             });
 
             if (isset($options['normalized_headers']['authorization'][0]) || isset($options['normalized_headers']['cookie'][0])) {
-                $redirectHeaders['no_auth'] = \array_filter($options['headers'], static function ($h) {
+                $redirectHeaders['no_auth'] = \array_filter($options['headers'], static function($h) {
                     return 0 !== stripos($h, 'Authorization:') && 0 !== stripos($h, 'Cookie:');
                 });
             }
         }
 
-        return static function ($ch, string $location) use ($redirectHeaders) {
+        return static function($ch, string $location) use ($redirectHeaders) {
             try {
                 $location = self::parseUrl($location);
             } catch (InvalidArgumentException $e) {
@@ -504,7 +504,7 @@ 
 
     private function findConstantName($opt): ?string
     {
-        $constants = \array_filter(get_defined_constants(), static function ($v, $k) use ($opt) {
+        $constants = \array_filter(get_defined_constants(), static function($v, $k) use ($opt) {
             return $v === $opt && 'C' === $k[0] && (0 === \strpos($k, 'CURLOPT_') || 0 === \strpos($k, 'CURLINFO_'));
         }, \ARRAY_FILTER_USE_BOTH);
 

lib/Github/Http/Client/CachingHttpClient.php

Spacing +1 added lines, -1 removed lines

@@ -136,7 +136,7 @@
             return new ResponseStream(MockResponse::stream($mockResponses, $timeout));
         }
 
-        return new ResponseStream((function () use ($mockResponses, $clientResponses, $timeout) {
+        return new ResponseStream((function() use ($mockResponses, $clientResponses, $timeout) {
             yield from MockResponse::stream($mockResponses, $timeout);
             yield $this->client->stream($clientResponses, $timeout);
         })());

lib/Github/Http/Client/HttpClientTrait.php

Spacing +6 added lines, -6 removed lines

@@ -46,7 +46,7 @@ 
         $buffer = $options['buffer'] ?? true;
 
         if ($buffer instanceof \Closure) {
-            $options['buffer'] = static function (array $headers) use ($buffer) {
+            $options['buffer'] = static function(array $headers) use ($buffer) {
                 if (!\is_bool($buffer = $buffer($headers))) {
                     if (!\is_array($bufferInfo = @stream_get_meta_data($buffer))) {
                         throw new \LogicException(\sprintf('The closure passed as option "buffer" must return bool or stream resource, got "%s".', get_debug_type($buffer)));
@@ -287,8 +287,8 @@ 
             return $body;
         }
 
-        $generatorToCallable = static function (\Generator $body): \Closure {
-            return static function () use ($body) {
+        $generatorToCallable = static function(\Generator $body): \Closure {
+            return static function() use ($body) {
                 while ($body->valid()) {
                     $chunk = $body->current();
                     $body->next();
@@ -307,7 +307,7 @@ 
         }
 
         if ($body instanceof \Traversable) {
-            return $generatorToCallable((static function ($body) { yield from $body; })($body));
+            return $generatorToCallable((static function($body) { yield from $body; })($body));
         }
 
         if ($body instanceof \Closure) {
@@ -477,11 +477,11 @@ 
 
             if (false !== \strpos($parts[$part], '%')) {
                 // https://tools.ietf.org/html/rfc3986#section-2.3
-                $parts[$part] = preg_replace_callback('/%(?:2[DE]|3[0-9]|[46][1-9A-F]|5F|[57][0-9A]|7E)++/i', function ($m) { return rawurldecode($m[0]); }, $parts[$part]);
+                $parts[$part] = preg_replace_callback('/%(?:2[DE]|3[0-9]|[46][1-9A-F]|5F|[57][0-9A]|7E)++/i', function($m) { return rawurldecode($m[0]); }, $parts[$part]);
             }
 
             // https://tools.ietf.org/html/rfc3986#section-3.3
-            $parts[$part] = preg_replace_callback("#[^-A-Za-z0-9._~!$&/'()*+,;=:@%]++#", function ($m) { return rawurlencode($m[0]); }, $parts[$part]);
+            $parts[$part] = preg_replace_callback("#[^-A-Za-z0-9._~!$&/'()*+,;=:@%]++#", function($m) { return rawurlencode($m[0]); }, $parts[$part]);
         }
 
         return [

lib/Github/Http/Client/RetryableHttpClient.php

Spacing +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
         $content = '';
         $firstChunk = null;
 
-        return new AsyncResponse($this->client, $method, $url, $options, function (ChunkInterface $chunk, AsyncContext $context) use ($method, $url, $options, &$retryCount, &$content, &$firstChunk) {
+        return new AsyncResponse($this->client, $method, $url, $options, function(ChunkInterface $chunk, AsyncContext $context) use ($method, $url, $options, &$retryCount, &$content, &$firstChunk) {
             $exception = null;
             try {
                 if ($chunk->isTimeout() || null !== $chunk->getInformationalStatus()) {

directories.php

Indentation +32 added lines, -32 removed lines

@@ -47,38 +47,38 @@
 $GLOBALS['xoopsTpl']->assign('showItem', $dirId > 0);
 
 switch ($op) {
-	case 'show':
-	case 'list':
-	default:
-		$crDirectories = new \CriteriaCompo();
-		if ($dirId > 0) {
-			$crDirectories->add(new \Criteria('dir_id', $dirId));
-		}
-		$directoriesCount = $directoriesHandler->getCount($crDirectories);
-		$GLOBALS['xoopsTpl']->assign('directoriesCount', $directoriesCount);
-		$crDirectories->setStart($start);
-		$crDirectories->setLimit($limit);
-		$directoriesAll = $directoriesHandler->getAll($crDirectories);
-		if ($directoriesCount > 0) {
-			$directories = [];
-			// Get All Directories
-			foreach (\array_keys($directoriesAll) as $i) {
-				$directories[$i] = $directoriesAll[$i]->getValuesDirectories();
-				$keywords[$i] = $directoriesAll[$i]->getVar('dir_name');
-			}
-			$GLOBALS['xoopsTpl']->assign('directories', $directories);
-			unset($directories);
-			// Display Navigation
-			if ($directoriesCount > $limit) {
-				include_once \XOOPS_ROOT_PATH . '/class/pagenav.php';
-				$pagenav = new \XoopsPageNav($directoriesCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
-				$GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
-			}
-			$GLOBALS['xoopsTpl']->assign('type', $helper->getConfig('table_type'));
-			$GLOBALS['xoopsTpl']->assign('divideby', $helper->getConfig('divideby'));
-			$GLOBALS['xoopsTpl']->assign('numb_col', $helper->getConfig('numb_col'));
-		}
-		break;
+    case 'show':
+    case 'list':
+    default:
+        $crDirectories = new \CriteriaCompo();
+        if ($dirId > 0) {
+            $crDirectories->add(new \Criteria('dir_id', $dirId));
+        }
+        $directoriesCount = $directoriesHandler->getCount($crDirectories);
+        $GLOBALS['xoopsTpl']->assign('directoriesCount', $directoriesCount);
+        $crDirectories->setStart($start);
+        $crDirectories->setLimit($limit);
+        $directoriesAll = $directoriesHandler->getAll($crDirectories);
+        if ($directoriesCount > 0) {
+            $directories = [];
+            // Get All Directories
+            foreach (\array_keys($directoriesAll) as $i) {
+                $directories[$i] = $directoriesAll[$i]->getValuesDirectories();
+                $keywords[$i] = $directoriesAll[$i]->getVar('dir_name');
+            }
+            $GLOBALS['xoopsTpl']->assign('directories', $directories);
+            unset($directories);
+            // Display Navigation
+            if ($directoriesCount > $limit) {
+                include_once \XOOPS_ROOT_PATH . '/class/pagenav.php';
+                $pagenav = new \XoopsPageNav($directoriesCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
+                $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
+            }
+            $GLOBALS['xoopsTpl']->assign('type', $helper->getConfig('table_type'));
+            $GLOBALS['xoopsTpl']->assign('divideby', $helper->getConfig('divideby'));
+            $GLOBALS['xoopsTpl']->assign('numb_col', $helper->getConfig('numb_col'));
+        }
+        break;
 }
 
 // Breadcrumbs

Spacing +6 added lines, -6 removed lines

@@ -25,9 +25,9 @@ 
 use XoopsModules\Wggithub\Constants;
 use XoopsModules\Wggithub\Common;
 
-require __DIR__ . '/header.php';
+require __DIR__.'/header.php';
 $GLOBALS['xoopsOption']['template_main'] = 'wggithub_directories.tpl';
-include_once \XOOPS_ROOT_PATH . '/header.php';
+include_once \XOOPS_ROOT_PATH.'/header.php';
 
 $op    = Request::getCmd('op', 'list');
 $start = Request::getInt('start', 0);
@@ -70,8 +70,8 @@ 
 			unset($directories);
 			// Display Navigation
 			if ($directoriesCount > $limit) {
-				include_once \XOOPS_ROOT_PATH . '/class/pagenav.php';
-				$pagenav = new \XoopsPageNav($directoriesCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
+				include_once \XOOPS_ROOT_PATH.'/class/pagenav.php';
+				$pagenav = new \XoopsPageNav($directoriesCount, $limit, $start, 'start', 'op=list&limit='.$limit);
 				$GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
 			}
 			$GLOBALS['xoopsTpl']->assign('type', $helper->getConfig('table_type'));
@@ -85,7 +85,7 @@ 
 $xoBreadcrumbs[] = ['title' => \_MA_WGGITHUB_DIRECTORIES];
 
 // Keywords
-wggithubMetaKeywords($helper->getConfig('keywords') . ', ' . \implode(',', $keywords));
+wggithubMetaKeywords($helper->getConfig('keywords').', '.\implode(',', $keywords));
 unset($keywords);
 
 // Description
@@ -93,4 +93,4 @@ 
 $GLOBALS['xoopsTpl']->assign('xoops_mpageurl', WGGITHUB_URL.'/directories.php');
 $GLOBALS['xoopsTpl']->assign('wggithub_upload_url', WGGITHUB_UPLOAD_URL);
 
-require __DIR__ . '/footer.php';
+require __DIR__.'/footer.php';

footer.php

Indentation +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
  * @author         Goffy - XOOPS Development Team - Email:<goffy@wedega.com> - Website:<https://wedega.com>
  */
 if (\count($xoBreadcrumbs) > 1) {
-	$GLOBALS['xoopsTpl']->assign('xoBreadcrumbs', $xoBreadcrumbs);
+    $GLOBALS['xoopsTpl']->assign('xoBreadcrumbs', $xoBreadcrumbs);
 }
 $GLOBALS['xoopsTpl']->assign('adv', $helper->getConfig('advertise'));
 // 

Spacing +1 added lines, -1 removed lines

@@ -30,4 +30,4 @@
 $GLOBALS['xoopsTpl']->assign('admin', WGGITHUB_ADMIN);
 $GLOBALS['xoopsTpl']->assign('copyright', $copyright);
 // 
-include_once \XOOPS_ROOT_PATH . '/footer.php';
+include_once \XOOPS_ROOT_PATH.'/footer.php';

repositories.php

Indentation +32 added lines, -32 removed lines

@@ -47,38 +47,38 @@
 $GLOBALS['xoopsTpl']->assign('showItem', $repoId > 0);
 
 switch ($op) {
-	case 'show':
-	case 'list':
-	default:
-		$crRepositories = new \CriteriaCompo();
-		if ($repoId > 0) {
-			$crRepositories->add(new \Criteria('repo_id', $repoId));
-		}
-		$repositoriesCount = $repositoriesHandler->getCount($crRepositories);
-		$GLOBALS['xoopsTpl']->assign('repositoriesCount', $repositoriesCount);
-		$crRepositories->setStart($start);
-		$crRepositories->setLimit($limit);
-		$repositoriesAll = $repositoriesHandler->getAll($crRepositories);
-		if ($repositoriesCount > 0) {
-			$repositories = [];
-			// Get All Repositories
-			foreach (\array_keys($repositoriesAll) as $i) {
-				$repositories[$i] = $repositoriesAll[$i]->getValuesRepositories();
-				$keywords[$i] = $repositoriesAll[$i]->getVar('repo_name');
-			}
-			$GLOBALS['xoopsTpl']->assign('repositories', $repositories);
-			unset($repositories);
-			// Display Navigation
-			if ($repositoriesCount > $limit) {
-				include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
-				$pagenav = new \XoopsPageNav($repositoriesCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
-				$GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
-			}
-			$GLOBALS['xoopsTpl']->assign('type', $helper->getConfig('table_type'));
-			$GLOBALS['xoopsTpl']->assign('divideby', $helper->getConfig('divideby'));
-			$GLOBALS['xoopsTpl']->assign('numb_col', $helper->getConfig('numb_col'));
-		}
-		break;
+    case 'show':
+    case 'list':
+    default:
+        $crRepositories = new \CriteriaCompo();
+        if ($repoId > 0) {
+            $crRepositories->add(new \Criteria('repo_id', $repoId));
+        }
+        $repositoriesCount = $repositoriesHandler->getCount($crRepositories);
+        $GLOBALS['xoopsTpl']->assign('repositoriesCount', $repositoriesCount);
+        $crRepositories->setStart($start);
+        $crRepositories->setLimit($limit);
+        $repositoriesAll = $repositoriesHandler->getAll($crRepositories);
+        if ($repositoriesCount > 0) {
+            $repositories = [];
+            // Get All Repositories
+            foreach (\array_keys($repositoriesAll) as $i) {
+                $repositories[$i] = $repositoriesAll[$i]->getValuesRepositories();
+                $keywords[$i] = $repositoriesAll[$i]->getVar('repo_name');
+            }
+            $GLOBALS['xoopsTpl']->assign('repositories', $repositories);
+            unset($repositories);
+            // Display Navigation
+            if ($repositoriesCount > $limit) {
+                include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
+                $pagenav = new \XoopsPageNav($repositoriesCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
+                $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
+            }
+            $GLOBALS['xoopsTpl']->assign('type', $helper->getConfig('table_type'));
+            $GLOBALS['xoopsTpl']->assign('divideby', $helper->getConfig('divideby'));
+            $GLOBALS['xoopsTpl']->assign('numb_col', $helper->getConfig('numb_col'));
+        }
+        break;
 }
 
 // Breadcrumbs

Spacing +6 added lines, -6 removed lines

@@ -25,9 +25,9 @@ 
 use XoopsModules\Wggithub\Constants;
 use XoopsModules\Wggithub\Common;
 
-require __DIR__ . '/header.php';
+require __DIR__.'/header.php';
 $GLOBALS['xoopsOption']['template_main'] = 'wggithub_repositories.tpl';
-include_once XOOPS_ROOT_PATH . '/header.php';
+include_once XOOPS_ROOT_PATH.'/header.php';
 
 $op    = Request::getCmd('op', 'list');
 $start = Request::getInt('start', 0);
@@ -70,8 +70,8 @@ 
 			unset($repositories);
 			// Display Navigation
 			if ($repositoriesCount > $limit) {
-				include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
-				$pagenav = new \XoopsPageNav($repositoriesCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
+				include_once XOOPS_ROOT_PATH.'/class/pagenav.php';
+				$pagenav = new \XoopsPageNav($repositoriesCount, $limit, $start, 'start', 'op=list&limit='.$limit);
 				$GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
 			}
 			$GLOBALS['xoopsTpl']->assign('type', $helper->getConfig('table_type'));
@@ -85,7 +85,7 @@ 
 $xoBreadcrumbs[] = ['title' => _MA_WGGITHUB_REPOSITORIES];
 
 // Keywords
-wggithubMetaKeywords($helper->getConfig('keywords') . ', ' . \implode(',', $keywords));
+wggithubMetaKeywords($helper->getConfig('keywords').', '.\implode(',', $keywords));
 unset($keywords);
 
 // Description
@@ -93,4 +93,4 @@ 
 $GLOBALS['xoopsTpl']->assign('xoops_mpageurl', WGGITHUB_URL.'/repositories.php');
 $GLOBALS['xoopsTpl']->assign('wggithub_upload_url', WGGITHUB_UPLOAD_URL);
 
-require __DIR__ . '/footer.php';
+require __DIR__.'/footer.php';

class/Settings.php

Indentation +110 added lines, -110 removed lines

@@ -32,119 +32,119 @@
  */
 class Settings extends \XoopsObject
 {
-	/**
-	 * Constructor
-	 *
-	 * @param null
-	 */
-	public function __construct()
-	{
-		$this->initVar('set_id', XOBJ_DTYPE_INT);
-		$this->initVar('set_username', XOBJ_DTYPE_TXTBOX);
-		$this->initVar('set_token', XOBJ_DTYPE_TXTBOX);
-		$this->initVar('set_options', XOBJ_DTYPE_TXTAREA);
-		$this->initVar('set_primary', XOBJ_DTYPE_INT);
-		$this->initVar('set_date', XOBJ_DTYPE_INT);
-		$this->initVar('set_submitter', XOBJ_DTYPE_INT);
-	}
+    /**
+     * Constructor
+     *
+     * @param null
+     */
+    public function __construct()
+    {
+        $this->initVar('set_id', XOBJ_DTYPE_INT);
+        $this->initVar('set_username', XOBJ_DTYPE_TXTBOX);
+        $this->initVar('set_token', XOBJ_DTYPE_TXTBOX);
+        $this->initVar('set_options', XOBJ_DTYPE_TXTAREA);
+        $this->initVar('set_primary', XOBJ_DTYPE_INT);
+        $this->initVar('set_date', XOBJ_DTYPE_INT);
+        $this->initVar('set_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 getNewInsertedIdSettings()
-	{
-		$newInsertedId = $GLOBALS['xoopsDB']->getInsertId();
-		return $newInsertedId;
-	}
+    /**
+     * The new inserted $Id
+     * @return inserted id
+     */
+    public function getNewInsertedIdSettings()
+    {
+        $newInsertedId = $GLOBALS['xoopsDB']->getInsertId();
+        return $newInsertedId;
+    }
 
-	/**
-	 * @public function getForm
-	 * @param bool $action
-	 * @return \XoopsThemeForm
-	 */
-	public function getFormSettings($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_SETTING_ADD) : \sprintf(_AM_WGGITHUB_SETTING_EDIT);
-		// Get Theme Form
-		\xoops_load('XoopsFormLoader');
-		$form = new \XoopsThemeForm($title, 'form', $action, 'post', true);
-		$form->setExtra('enctype="multipart/form-data"');
-		// Form Text setUsername
-		$form->addElement(new \XoopsFormText(_AM_WGGITHUB_SETTING_USERNAME, 'set_username', 50, 255, $this->getVar('set_username')), true);
-		// Form Text setToken
-		$form->addElement(new \XoopsFormText(_AM_WGGITHUB_SETTING_TOKEN, 'set_token', 50, 255, $this->getVar('set_token')), true);
-		// Form Editor TextArea setOptions
-		$form->addElement(new \XoopsFormTextArea(_AM_WGGITHUB_SETTING_OPTIONS, 'set_options', $this->getVar('set_options', 'e'), 4, 47));
-		// Form Radio Yes/No setPrimary
-		$setPrimary = $this->isNew() ?: $this->getVar('set_primary');
-		$form->addElement(new \XoopsFormRadioYN(_AM_WGGITHUB_SETTING_PRIMARY, 'set_primary', $setPrimary));
-		// Form Text Date Select setDate
-		$setDate = $this->isNew() ?: $this->getVar('set_date');
-		$form->addElement(new \XoopsFormTextDateSelect(_AM_WGGITHUB_SETTING_DATE, 'set_date', '', $setDate));
-		// Form Select User setSubmitter
-		$form->addElement(new \XoopsFormSelectUser(_AM_WGGITHUB_SETTING_SUBMITTER, 'set_submitter', false, $this->getVar('set_submitter')));
-		// To Save
-		$form->addElement(new \XoopsFormHidden('op', 'save'));
-		$form->addElement(new \XoopsFormButtonTray('', _SUBMIT, 'submit', '', false));
-		return $form;
-	}
+    /**
+     * @public function getForm
+     * @param bool $action
+     * @return \XoopsThemeForm
+     */
+    public function getFormSettings($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_SETTING_ADD) : \sprintf(_AM_WGGITHUB_SETTING_EDIT);
+        // Get Theme Form
+        \xoops_load('XoopsFormLoader');
+        $form = new \XoopsThemeForm($title, 'form', $action, 'post', true);
+        $form->setExtra('enctype="multipart/form-data"');
+        // Form Text setUsername
+        $form->addElement(new \XoopsFormText(_AM_WGGITHUB_SETTING_USERNAME, 'set_username', 50, 255, $this->getVar('set_username')), true);
+        // Form Text setToken
+        $form->addElement(new \XoopsFormText(_AM_WGGITHUB_SETTING_TOKEN, 'set_token', 50, 255, $this->getVar('set_token')), true);
+        // Form Editor TextArea setOptions
+        $form->addElement(new \XoopsFormTextArea(_AM_WGGITHUB_SETTING_OPTIONS, 'set_options', $this->getVar('set_options', 'e'), 4, 47));
+        // Form Radio Yes/No setPrimary
+        $setPrimary = $this->isNew() ?: $this->getVar('set_primary');
+        $form->addElement(new \XoopsFormRadioYN(_AM_WGGITHUB_SETTING_PRIMARY, 'set_primary', $setPrimary));
+        // Form Text Date Select setDate
+        $setDate = $this->isNew() ?: $this->getVar('set_date');
+        $form->addElement(new \XoopsFormTextDateSelect(_AM_WGGITHUB_SETTING_DATE, 'set_date', '', $setDate));
+        // Form Select User setSubmitter
+        $form->addElement(new \XoopsFormSelectUser(_AM_WGGITHUB_SETTING_SUBMITTER, 'set_submitter', false, $this->getVar('set_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 getValuesSettings($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('set_id');
-		$ret['username']      = $this->getVar('set_username');
-		$ret['token']         = $this->getVar('set_token');
-		$ret['options']       = \strip_tags($this->getVar('set_options', 'e'));
-		$editorMaxchar = $helper->getConfig('editor_maxchar');
-		$ret['options_short'] = $utility::truncateHtml($ret['options'], $editorMaxchar);
-		$ret['primary']       = (int)$this->getVar('set_primary') > 0 ? _YES : _NO;
-		$ret['date']          = \formatTimestamp($this->getVar('set_date'), 's');
-		$ret['submitter']     = \XoopsUser::getUnameFromId($this->getVar('set_submitter'));
-		return $ret;
-	}
+    /**
+     * Get Values
+     * @param null $keys
+     * @param null $format
+     * @param null $maxDepth
+     * @return array
+     */
+    public function getValuesSettings($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('set_id');
+        $ret['username']      = $this->getVar('set_username');
+        $ret['token']         = $this->getVar('set_token');
+        $ret['options']       = \strip_tags($this->getVar('set_options', 'e'));
+        $editorMaxchar = $helper->getConfig('editor_maxchar');
+        $ret['options_short'] = $utility::truncateHtml($ret['options'], $editorMaxchar);
+        $ret['primary']       = (int)$this->getVar('set_primary') > 0 ? _YES : _NO;
+        $ret['date']          = \formatTimestamp($this->getVar('set_date'), 's');
+        $ret['submitter']     = \XoopsUser::getUnameFromId($this->getVar('set_submitter'));
+        return $ret;
+    }
 
-	/**
-	 * Returns an array representation of the object
-	 *
-	 * @return array
-	 */
-	public function toArraySettings()
-	{
-		$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 toArraySettings()
+    {
+        $ret = [];
+        $vars = $this->getVars();
+        foreach (\array_keys($vars) as $var) {
+            $ret[$var] = $this->getVar('"{$var}"');
+        }
+        return $ret;
+    }
 }

Spacing +1 added lines, -1 removed lines

@@ -127,7 +127,7 @@
 		$ret['options']       = \strip_tags($this->getVar('set_options', 'e'));
 		$editorMaxchar = $helper->getConfig('editor_maxchar');
 		$ret['options_short'] = $utility::truncateHtml($ret['options'], $editorMaxchar);
-		$ret['primary']       = (int)$this->getVar('set_primary') > 0 ? _YES : _NO;
+		$ret['primary']       = (int) $this->getVar('set_primary') > 0 ? _YES : _NO;
 		$ret['date']          = \formatTimestamp($this->getVar('set_date'), 's');
 		$ret['submitter']     = \XoopsUser::getUnameFromId($this->getVar('set_submitter'));
 		return $ret;