ggoffy / wggithub

admin/logs.php

Spacing +5 added lines, -5 removed lines

@@ -25,7 +25,7 @@ 
 use XoopsModules\Wggithub\Constants;
 use XoopsModules\Wggithub\Common;
 
-require __DIR__ . '/header.php';
+require __DIR__.'/header.php';
 // It recovered the value of argument op in URL$
 $op    = Request::getCmd('op', 'list');
 $logId = Request::getInt('log_id');
@@ -58,8 +58,8 @@ 
             }
             // Display Navigation
             if ($logsCount > $limit) {
-                include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
-                $pagenav = new \XoopsPageNav($logsCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
+                include_once XOOPS_ROOT_PATH.'/class/pagenav.php';
+                $pagenav = new \XoopsPageNav($logsCount, $limit, $start, 'start', 'op=list&limit='.$limit);
                 $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
             }
         } else {
@@ -95,7 +95,7 @@ 
         $logsObj->setVar('log_submitter', Request::getInt('log_submitter', 0));
         // Insert Data
         if ($logsHandler->insert($logsObj)) {
-            \redirect_header('logs.php?op=list&start=' . $start . '&limit=' . $limit, 2, _AM_WGGITHUB_FORM_OK);
+            \redirect_header('logs.php?op=list&start='.$start.'&limit='.$limit, 2, _AM_WGGITHUB_FORM_OK);
         }
         // Get Form
         $GLOBALS['xoopsTpl']->assign('error', $logsObj->getHtmlErrors());
@@ -156,4 +156,4 @@ 
         }
         break;
 }
-require __DIR__ . '/footer.php';
+require __DIR__.'/footer.php';

admin/settings.php

Spacing +5 added lines, -5 removed lines

@@ -25,7 +25,7 @@ 
 use XoopsModules\Wggithub\Constants;
 use XoopsModules\Wggithub\Common;
 
-require __DIR__ . '/header.php';
+require __DIR__.'/header.php';
 // It recovered the value of argument op in URL$
 $op    = Request::getCmd('op', 'list');
 $setId = Request::getInt('set_id');
@@ -59,8 +59,8 @@ 
             }
             // Display Navigation
             if ($settingsCount > $limit) {
-                include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
-                $pagenav = new \XoopsPageNav($settingsCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
+                include_once XOOPS_ROOT_PATH.'/class/pagenav.php';
+                $pagenav = new \XoopsPageNav($settingsCount, $limit, $start, 'start', 'op=list&limit='.$limit);
                 $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
             }
         } else {
@@ -102,7 +102,7 @@ 
                 $setId = $setId > 0 ? $setId : $newSetId;
                 $settingsHandler->setPrimarySetting($setId);
             }
-            \redirect_header('settings.php?op=list&start=' . $start . '&limit=' . $limit, 2, _AM_WGGITHUB_FORM_OK);
+            \redirect_header('settings.php?op=list&start='.$start.'&limit='.$limit, 2, _AM_WGGITHUB_FORM_OK);
         }
         // Get Form
         $GLOBALS['xoopsTpl']->assign('error', $settingsObj->getHtmlErrors());
@@ -144,4 +144,4 @@ 
         }
         break;
 }
-require __DIR__ . '/footer.php';
+require __DIR__.'/footer.php';

admin/readmes.php

Spacing +5 added lines, -5 removed lines

@@ -25,7 +25,7 @@ 
 use XoopsModules\Wggithub\Constants;
 use XoopsModules\Wggithub\Common;
 
-require __DIR__ . '/header.php';
+require __DIR__.'/header.php';
 // It recovered the value of argument op in URL$
 $op    = Request::getCmd('op', 'list');
 $rmId  = Request::getInt('rm_id');
@@ -57,8 +57,8 @@ 
             }
             // Display Navigation
             if ($readmesCount > $limit) {
-                include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
-                $pagenav = new \XoopsPageNav($readmesCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
+                include_once XOOPS_ROOT_PATH.'/class/pagenav.php';
+                $pagenav = new \XoopsPageNav($readmesCount, $limit, $start, 'start', 'op=list&limit='.$limit);
                 $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
             }
         } else {
@@ -97,7 +97,7 @@ 
         $readmesObj->setVar('rm_submitter', Request::getInt('rm_submitter', 0));
         // Insert Data
         if ($readmesHandler->insert($readmesObj)) {
-            \redirect_header('readmes.php?op=list&start=' . $start . '&limit=' . $limit, 2, _AM_WGGITHUB_FORM_OK);
+            \redirect_header('readmes.php?op=list&start='.$start.'&limit='.$limit, 2, _AM_WGGITHUB_FORM_OK);
         }
         // Get Form
         $GLOBALS['xoopsTpl']->assign('error', $readmesObj->getHtmlErrors());
@@ -139,4 +139,4 @@ 
         }
         break;
 }
-require __DIR__ . '/footer.php';
+require __DIR__.'/footer.php';

class/RepositoriesHandler.php

Spacing +1 added lines, -1 removed lines

@@ -140,7 +140,7 @@
         $submitter = isset($GLOBALS['xoopsUser']) && \is_object($GLOBALS['xoopsUser']) ? $GLOBALS['xoopsUser']->getVar('uid') : 0;
         // add/update all items from table repositories
         foreach ($repos as $key => $repo) {
-            $fork = (bool)$repo['fork'];
+            $fork = (bool) $repo['fork'];
             if (Constants::DIRECTORY_CONTENT_ALL == $dirContent || false === $fork) {
                 $crRepositories = new \CriteriaCompo();
                 $crRepositories->add(new \Criteria('repo_nodeid', $repo['node_id']));

class/Constants.php

Spacing +2 added lines, -2 removed lines

@@ -40,7 +40,7 @@ 
     public const DIRECTORY_TYPE_USER = 1;
     public const DIRECTORY_TYPE_ORG  = 2;
 
-    public const DIRECTORY_CONTENT_ALL  = 0;
+    public const DIRECTORY_CONTENT_ALL = 0;
     public const DIRECTORY_CONTENT_OWN = 1;
 
     // Constants for status
@@ -57,7 +57,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/Releases.php

Spacing +3 added lines, -3 removed lines

@@ -136,9 +136,9 @@ 
      */
     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'));
         if (\is_object($repositoriesObj)) {
@@ -148,7 +148,7 @@ 
         }
         $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');

tests/github-api.php

Spacing +25 added lines, -25 removed lines

@@ -30,7 +30,7 @@ 
 use XoopsModules\Wggithub;
 use XoopsModules\Wggithub\Github;
 
-require __DIR__ . '/header.php';
+require __DIR__.'/header.php';
 // It recovered the value of argument op in URL$
 
 /*
@@ -45,46 +45,46 @@ 
 
 $data = [];
 $url = '/orgs/XoopsModules25x/repos?per_page=100&page=1';
-echo "<br>Test read org repo:" . $url;
+echo "<br>Test read org repo:".$url;
 $api = new XoopsModules\Wggithub\Github\GithubClient();
 $data = $api->testMilo($url);
 $count = 0;
 foreach ($data as $key => $repo) {
-    echo "<br>key:" . $key . ' repo:' . $repo['name'];
+    echo "<br>key:".$key.' repo:'.$repo['name'];
     $count++;
-    if ($count>5) {break;}
+    if ($count > 5) {break; }
 }
 
 
 $data = [];
 $url = '/users/ggoffy/repos?per_page=100&page=1';
-echo "<br><br>Test read user repo:" . $url;
+echo "<br><br>Test read user repo:".$url;
 $api = new XoopsModules\Wggithub\Github\GithubClient();
 $data = $api->testMilo($url);
 $count = 0;
 foreach ($data as $key => $repo) {
-    echo "<br>key:" . $key . ' repo:' . $repo['name'];
+    echo "<br>key:".$key.' repo:'.$repo['name'];
     $count++;
-    if ($count>5) {break;}
+    if ($count > 5) {break; }
 }
 
 
 $data = [];
 $url = '/repos/XoopsModules25x/smallworld/readme';
-echo "<br><br>test read readme orgs:" . $url;
+echo "<br><br>test read readme orgs:".$url;
 $api = new XoopsModules\Wggithub\Github\GithubClient();
 $data = $api->testMilo($url);
-echo '<br>name:' . $data['name'];
-echo '<br>download_url:' . $data['download_url'];
+echo '<br>name:'.$data['name'];
+echo '<br>download_url:'.$data['download_url'];
 
 
 $data = [];
 $url = '/repos/ggoffy/wggithub/readme';
-echo "<br><br>test read readme user:" . $url;
+echo "<br><br>test read readme user:".$url;
 $api = new XoopsModules\Wggithub\Github\GithubClient();
 $data = $api->testMilo($url);
-echo '<br>name:' . $data['name'];
-echo '<br>download_url:' . $data['download_url'];
+echo '<br>name:'.$data['name'];
+echo '<br>download_url:'.$data['download_url'];
 
 
 echo "<br><br><br>-----------------------------------";
@@ -93,45 +93,45 @@ 
 
 $data = [];
 $url = '/orgs/XoopsModules25x/repos?per_page=100&page=1';
-echo "<br>Test read org repo:" . $url;
+echo "<br>Test read org repo:".$url;
 $api = new XoopsModules\Wggithub\Github\GithubClient();
 $data = $api->testMilo2($url);
 $count = 0;
 foreach ($data as $key => $repo) {
-    echo "<br>key:" . $key . ' repo:' . $repo['name'];
+    echo "<br>key:".$key.' repo:'.$repo['name'];
     $count++;
-    if ($count>5) {break;}
+    if ($count > 5) {break; }
 }
 
 
 $data = [];
 $url = '/users/ggoffy/repos?per_page=100&page=1';
-echo "<br><br>Test read user repo:" . $url;
+echo "<br><br>Test read user repo:".$url;
 $api = new XoopsModules\Wggithub\Github\GithubClient();
 $data = $api->testMilo2($url);
 $count = 0;
 foreach ($data as $key => $repo) {
-    echo "<br>key:" . $key . ' repo:' . $repo['name'];
+    echo "<br>key:".$key.' repo:'.$repo['name'];
     $count++;
-    if ($count>5) {break;}
+    if ($count > 5) {break; }
 }
 
 
 $data = [];
 $url = '/repos/XoopsModules25x/smallworld/readme';
-echo "<br><br>test read readme orgs:" . $url;
+echo "<br><br>test read readme orgs:".$url;
 $api = new XoopsModules\Wggithub\Github\GithubClient();
 $data = $api->testMilo2($url);
-echo '<br>name:' . $data['name'];
-echo '<br>download_url:' . $data['download_url'];
+echo '<br>name:'.$data['name'];
+echo '<br>download_url:'.$data['download_url'];
 
 
 $data = [];
 $url = '/repos/ggoffy/wggithub/readme';
-echo "<br><br>test read readme user:" . $url;
+echo "<br><br>test read readme user:".$url;
 $api = new XoopsModules\Wggithub\Github\GithubClient();
 $data = $api->testMilo2($url);
-echo '<br>name:' . $data['name'];
-echo '<br>download_url:' . $data['download_url'];
+echo '<br>name:'.$data['name'];
+echo '<br>download_url:'.$data['download_url'];
 
 

tests/test.php

Spacing +3 added lines, -3 removed lines

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

index.php

Spacing +15 added lines, -15 removed lines

@@ -28,15 +28,15 @@ 
     Github\GithubClient
 };
 
-require __DIR__ . '/header.php';
+require __DIR__.'/header.php';
 $GLOBALS['xoopsOption']['template_main'] = 'wggithub_index.tpl';
-include_once \XOOPS_ROOT_PATH . '/header.php';
+include_once \XOOPS_ROOT_PATH.'/header.php';
 
 // Permissions
 $permGlobalView = $permissionsHandler->getPermGlobalView();
 if (!$permGlobalView) {
     $GLOBALS['xoopsTpl']->assign('error', _NOPERM);
-    require __DIR__ . '/footer.php';
+    require __DIR__.'/footer.php';
 }
 $permGlobalRead   = $permissionsHandler->getPermGlobalRead();
 $permReadmeUpdate = $permissionsHandler->getPermReadmeUpdate();
@@ -50,7 +50,7 @@ 
 
 // Define Stylesheet
 $GLOBALS['xoTheme']->addStylesheet($style, null);
-$GLOBALS['xoTheme']->addStylesheet(WGGITHUB_URL . '/assets/css/tabs.css', null);
+$GLOBALS['xoTheme']->addStylesheet(WGGITHUB_URL.'/assets/css/tabs.css', null);
 $keywords = [];
 // 
 $GLOBALS['xoopsTpl']->assign('xoops_icons32_url', XOOPS_ICONS32_URL);
@@ -127,7 +127,7 @@ 
             foreach (\array_keys($directoriesAll) as $i) {
                 $directories[$i] = $directoriesAll[$i]->getValuesDirectories();
                 $dirName = $directoriesAll[$i]->getVar('dir_name');
-                $dirFilterRelease = (bool)$directoriesAll[$i]->getVar('dir_filterrelease');
+                $dirFilterRelease = (bool) $directoriesAll[$i]->getVar('dir_filterrelease');
                 $repos = [];
                 $crRepositories = new \CriteriaCompo();
                 //first block/parentheses
@@ -202,9 +202,9 @@ 
                 }
                 $directories[$i]['repos'] = $repos;
                 $directories[$i]['previousRepos'] = $start > 0;
-                $directories[$i]['previousOp'] = '&start=' . ($start - $limit) . '&limit=' . $limit . '&release=' . $filterRelease . '&sortby=' . $filterSortby;
+                $directories[$i]['previousOp'] = '&start='.($start - $limit).'&limit='.$limit.'&release='.$filterRelease.'&sortby='.$filterSortby;
                 $directories[$i]['nextRepos'] = ($repositoriesCount - $start) > $limit;
-                $directories[$i]['nextOp'] = '&start=' . ($start + $limit) . '&limit=' . $limit . '&release=' . $filterRelease . '&sortby=' . $filterSortby;
+                $directories[$i]['nextOp'] = '&start='.($start + $limit).'&limit='.$limit.'&release='.$filterRelease.'&sortby='.$filterSortby;
                 $GLOBALS['xoopsTpl']->assign('start', $start);
                 $GLOBALS['xoopsTpl']->assign('limit', $limit);
                 $GLOBALS['xoopsTpl']->assign('menu', $menu);
@@ -214,8 +214,8 @@ 
             unset($crDirectories, $directories);
             // Display Navigation
             if ($directoriesCount > $limit) {
-                require_once \XOOPS_ROOT_PATH . '/class/pagenav.php';
-                $pagenav = new \XoopsPageNav($directoriesCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
+                require_once \XOOPS_ROOT_PATH.'/class/pagenav.php';
+                $pagenav = new \XoopsPageNav($directoriesCount, $limit, $start, 'start', 'op=list&limit='.$limit);
                 $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
             }
             $GLOBALS['xoopsTpl']->assign('lang_thereare', \sprintf(\_MA_WGGITHUB_INDEX_THEREARE, $directoriesCount));
@@ -228,12 +228,12 @@ 
         // Permissions
         if (!$permGlobalRead) {
             $GLOBALS['xoopsTpl']->assign('error', \_NOPERM);
-            require __DIR__ . '/footer.php';
+            require __DIR__.'/footer.php';
         }
         $dirName = Request::getString('dir_name', '');
         $start   = 0; //reset to default
         $limit   = Request::getInt('limit', $helper->getConfig('userpager'));
-        $redir   = 'index.php?op=list_afterupdate&start=' . $start . '&limit=' . $limit . '&release=' . $filterRelease . '&sortby=' . $filterSortby;
+        $redir   = 'index.php?op=list_afterupdate&start='.$start.'&limit='.$limit.'&release='.$filterRelease.'&sortby='.$filterSortby;
         $githubClient = GithubClient::getInstance();
         $result = $githubClient->executeUpdate($dirName);
         if ($result) {
@@ -247,14 +247,14 @@ 
         // Permissions
         if (!$permReadmeUpdate) {
             $GLOBALS['xoopsTpl']->assign('error', \_NOPERM);
-            require __DIR__ . '/footer.php';
+            require __DIR__.'/footer.php';
         }
         $start    = 0; //reset to default
         $limit    = Request::getInt('limit', $helper->getConfig('userpager'));
         $repoId   = Request::getInt('repo_id', 0);
         $repoUser = Request::getString('repo_user', 'none');
         $repoName = Request::getString('repo_name', 'none');
-        $redir    = 'index.php?op=list_afterupdate&start=' . $start . '&limit=' . $limit . '&release=' . $filterRelease . '&sortby=' . $filterSortby;
+        $redir    = 'index.php?op=list_afterupdate&start='.$start.'&limit='.$limit.'&release='.$filterRelease.'&sortby='.$filterSortby;
         $result = $helper->getHandler('Readmes')->updateReadmes($repoId, $repoUser, $repoName);
         if ($result) {
             \redirect_header($redir, 2, \_MA_WGGITHUB_READGH_SUCCESS);
@@ -268,11 +268,11 @@ 
 // Breadcrumbs
 $xoBreadcrumbs[] = ['title' => \_MA_WGGITHUB_INDEX];
 // Keywords
-wggithubMetaKeywords($helper->getConfig('keywords') . ', ' . \implode(',', $keywords));
+wggithubMetaKeywords($helper->getConfig('keywords').', '.\implode(',', $keywords));
 unset($keywords);
 // Description
 wggithubMetaDescription(\_MA_WGGITHUB_INDEX_DESC);
 $GLOBALS['xoopsTpl']->assign('xoops_mpageurl', WGGITHUB_URL.'/index.php');
 $GLOBALS['xoopsTpl']->assign('xoops_icons32_url', XOOPS_ICONS32_URL);
 $GLOBALS['xoopsTpl']->assign('wggithub_upload_url', WGGITHUB_UPLOAD_URL);
-require __DIR__ . '/footer.php';
+require __DIR__.'/footer.php';