ggoffy / wggithub

admin/readmes.php

Spacing +6 added lines, -6 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');

@@ -64,7 +64,7 @@ 
                     $in[] = $i;

                 }

             }

-            $crReadmes->add(new Criteria('rm_repoid', '(' . implode(',', $in) . ')', 'IN'));

+            $crReadmes->add(new Criteria('rm_repoid', '('.implode(',', $in).')', 'IN'));

         }

         $crReadmes->setStart($start);

         $crReadmes->setLimit($limit);

@@ -82,8 +82,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));

             }

             $form = $readmesHandler->getFormFilterReadmes(false, $start, $limit);

@@ -125,7 +125,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());

@@ -167,4 +167,4 @@ 
         }

         break;

 }

-require __DIR__ . '/footer.php';

+require __DIR__.'/footer.php';

admin/releases.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');

 $relId = Request::getInt('rel_id');

@@ -71,8 +71,8 @@ 
             }

             // Display Navigation

             if ($releasesCount > $limit) {

-                include_once XOOPS_ROOT_PATH . '/class/pagenav.php';

-                $pagenav = new \XoopsPageNav($releasesCount, $limit, $start, 'start', 'op=list&limit=' . $limit);

+                include_once XOOPS_ROOT_PATH.'/class/pagenav.php';

+                $pagenav = new \XoopsPageNav($releasesCount, $limit, $start, 'start', 'op=list&limit='.$limit);

                 $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));

             }

             $form = $releasesHandler->getFormFilterReleases(false, $start, $limit);

@@ -114,7 +114,7 @@ 
         $releasesObj->setVar('rel_submitter', Request::getInt('rel_submitter', 0));

         // Insert Data

         if ($releasesHandler->insert($releasesObj)) {

-            \redirect_header('releases.php?op=list&start=' . $start . '&limit=' . $limit, 2, _AM_WGGITHUB_FORM_OK);

+            \redirect_header('releases.php?op=list&start='.$start.'&limit='.$limit, 2, _AM_WGGITHUB_FORM_OK);

         }

         // Get Form

         $GLOBALS['xoopsTpl']->assign('error', $releasesObj->getHtmlErrors());

@@ -156,4 +156,4 @@ 
         }

         break;

 }

-require __DIR__ . '/footer.php';

+require __DIR__.'/footer.php';

admin/directories.php

Spacing +14 added lines, -14 removed lines

@@ -27,7 +27,7 @@ 
     Github\GithubClient

 };

 

-require __DIR__ . '/header.php';

+require __DIR__.'/header.php';

 // It recovered the value of argument op in URL$

 $op    = Request::getCmd('op', 'list');

 $dirId = Request::getInt('dir_id');

@@ -41,8 +41,8 @@ 
     default:

         // Define Stylesheet

         $GLOBALS['xoTheme']->addStylesheet($style, null);

-        $GLOBALS['xoTheme']->addScript(WGGITHUB_URL . '/assets/js/jquery-ui.js');

-        $GLOBALS['xoTheme']->addScript(WGGITHUB_URL . '/assets/js/sortable.js');

+        $GLOBALS['xoTheme']->addScript(WGGITHUB_URL.'/assets/js/jquery-ui.js');

+        $GLOBALS['xoTheme']->addScript(WGGITHUB_URL.'/assets/js/sortable.js');

         $templateMain = 'wggithub_admin_directories.tpl';

         $GLOBALS['xoopsTpl']->assign('navigation', $adminObject->displayNavigation('directories.php'));

         $adminObject->addItemButton(_AM_WGGITHUB_ADD_DIRECTORY, 'directories.php?op=new', 'add');

@@ -52,7 +52,7 @@ 
         $GLOBALS['xoopsTpl']->assign('directories_count', $directoriesCount);

         $GLOBALS['xoopsTpl']->assign('wggithub_url', WGGITHUB_URL);

         $GLOBALS['xoopsTpl']->assign('wggithub_upload_url', WGGITHUB_UPLOAD_URL);

-        $GLOBALS['xoopsTpl']->assign('wggithub_icons_url_16', WGGITHUB_ICONS_URL . '/16');

+        $GLOBALS['xoopsTpl']->assign('wggithub_icons_url_16', WGGITHUB_ICONS_URL.'/16');

         // Table view directories

         if ($directoriesCount > 0) {

             foreach (\array_keys($directoriesAll) as $i) {

@@ -62,8 +62,8 @@ 
             }

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

             }

         } else {

@@ -103,7 +103,7 @@ 
         $directoriesObj->setVar('dir_submitter', Request::getInt('dir_submitter', 0));

         // Insert Data

         if ($directoriesHandler->insert($directoriesObj)) {

-            \redirect_header('directories.php?op=list&start=' . $start . '&limit=' . $limit, 2, _AM_WGGITHUB_FORM_OK);

+            \redirect_header('directories.php?op=list&start='.$start.'&limit='.$limit, 2, _AM_WGGITHUB_FORM_OK);

         }

         // Get Form

         $GLOBALS['xoopsTpl']->assign('error', $directoriesObj->getHtmlErrors());

@@ -142,7 +142,7 @@ 
                     $crReadmes = new \CriteriaCompo();

                     $crReadmes->add(new \Criteria('rm_repoid', $repoId));

                     if (!$readmesHandler->deleteAll($crReadmes)) {

-                        $errors[] = \_AM_WGGITHUB_ERROR_DELETE_DATA . ' READMES';

+                        $errors[] = \_AM_WGGITHUB_ERROR_DELETE_DATA.' READMES';

                     }

                 }

                 if ($repositoriesAll[$i]->getVar('repo_release') > 0 || $repositoriesAll[$i]->getVar('repo_prerelease') > 0) {

@@ -150,17 +150,17 @@ 
                     $crReleases = new \CriteriaCompo();

                     $crReleases->add(new \Criteria('rel_repoid', $repoId));

                     if (!$releasesHandler->deleteAll($crReleases)) {

-                        $errors[] = \_AM_WGGITHUB_ERROR_DELETE_DATA . ' RELEASES';

+                        $errors[] = \_AM_WGGITHUB_ERROR_DELETE_DATA.' RELEASES';

                     }

                 }

             }

             if (!$repositoriesHandler->deleteAll($crRepositories)) {

-                $errors[] = \_AM_WGGITHUB_ERROR_DELETE_DATA . ' REPOSITORIES';

+                $errors[] = \_AM_WGGITHUB_ERROR_DELETE_DATA.' REPOSITORIES';

             }

             unset($crReadmes, $crReleases, $repositoriesAll);

             //delete directory

             if (!$directoriesHandler->delete($directoriesObj)) {

-                $errors[] = \_AM_WGGITHUB_ERROR_DELETE_DATA . ' DIRECTORIES - ' . $directoriesObj->getHtmlErrors();

+                $errors[] = \_AM_WGGITHUB_ERROR_DELETE_DATA.' DIRECTORIES - '.$directoriesObj->getHtmlErrors();

             }

             if (0 == \count($errors)) {

                 \redirect_header('directories.php', 3, _AM_WGGITHUB_FORM_DELETE_OK);

@@ -181,7 +181,7 @@ 
         $dirName = $directoriesObj->getVar('dir_name');

         $githubClient = GithubClient::getInstance();

         $result = $githubClient->executeUpdate($dirName);

-        $redir = 'directories.php?op=list&start=' . $start . '&limit=' . $limit;

+        $redir = 'directories.php?op=list&start='.$start.'&limit='.$limit;

         if ($result) {

             \redirect_header($redir, 2, \_MA_WGGITHUB_READGH_SUCCESS);

         } else {

@@ -194,7 +194,7 @@ 
             $directoriesObj->setVar(Request::getString('field'), Request::getInt('value', 0));

             // Insert Data

             if ($directoriesHandler->insert($directoriesObj, true)) {

-                \redirect_header('directories.php?op=list&start=' . $start . '&limit=' . $limit, 2, _AM_WGGITHUB_FORM_OK);

+                \redirect_header('directories.php?op=list&start='.$start.'&limit='.$limit, 2, _AM_WGGITHUB_FORM_OK);

             }

         }

         break;

@@ -207,4 +207,4 @@ 
         }

         break;

 }

-require __DIR__ . '/footer.php';

+require __DIR__.'/footer.php';

admin/repositories.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');

 $repoId = Request::getInt('repo_id');

@@ -72,8 +72,8 @@ 
             }

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

             }

             $form = $repositoriesHandler->getFormFilterRepos(false, $start, $limit);

@@ -120,7 +120,7 @@ 
         $repositoriesObj->setVar('repo_submitter', Request::getInt('repo_submitter', 0));

         // Insert Data

         if ($repositoriesHandler->insert($repositoriesObj)) {

-            \redirect_header('repositories.php?op=list&start=' . $start . '&limit=' . $limit, 2, _AM_WGGITHUB_FORM_OK);

+            \redirect_header('repositories.php?op=list&start='.$start.'&limit='.$limit, 2, _AM_WGGITHUB_FORM_OK);

         }

         // Get Form

         $GLOBALS['xoopsTpl']->assign('error', $repositoriesObj->getHtmlErrors());

@@ -162,4 +162,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/ReadmesHandler.php

Spacing +6 added lines, -6 removed lines

@@ -170,7 +170,7 @@ 
                     $readmesObj->setVar('rm_content', $readme['content']);

                     $readmesObj->setVar('rm_encoding', $readme['encoding']);

                     $readmesObj->setVar('rm_downloadurl', $readme['download_url']);

-                    $readmesObj->setVar('rm_datecreated',time());

+                    $readmesObj->setVar('rm_datecreated', time());

                     $readmesObj->setVar('rm_submitter', $submitter);

                     // Insert Data

                     if (!$readmesHandler->insert($readmesObj)) {

@@ -232,7 +232,7 @@ 
             $readmesObj->setVar('rm_downloadurl', $readme['download_url']);

             $baseurl = \substr($readme['html_url'], 0, \strrpos($readme['html_url'], '/') + 1);

             $readmesObj->setVar('rm_baseurl', $baseurl);

-            $readmesObj->setVar('rm_datecreated',time());

+            $readmesObj->setVar('rm_datecreated', time());

             $readmesObj->setVar('rm_submitter', $submitter);

             // Insert Data

             if (!$readmesHandler->insert($readmesObj)) {

@@ -266,10 +266,10 @@ 
     public function updateRepoReadme()

     {

         // update repo_readme

-        $sql = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('wggithub_repositories') . ' INNER JOIN ' . $GLOBALS['xoopsDB']->prefix('wggithub_readmes');

-        $sql .= ' ON ' . $GLOBALS['xoopsDB']->prefix('wggithub_repositories') . '.repo_id = ' . $GLOBALS['xoopsDB']->prefix('wggithub_readmes') . '.rm_repoid ';

-        $sql .= 'SET ' . $GLOBALS['xoopsDB']->prefix('wggithub_repositories') . '.repo_readme = 1 ';

-        $sql .= 'WHERE (((' . $GLOBALS['xoopsDB']->prefix('wggithub_readmes') . '.rm_id)>0));';

+        $sql = 'UPDATE '.$GLOBALS['xoopsDB']->prefix('wggithub_repositories').' INNER JOIN '.$GLOBALS['xoopsDB']->prefix('wggithub_readmes');

+        $sql .= ' ON '.$GLOBALS['xoopsDB']->prefix('wggithub_repositories').'.repo_id = '.$GLOBALS['xoopsDB']->prefix('wggithub_readmes').'.rm_repoid ';

+        $sql .= 'SET '.$GLOBALS['xoopsDB']->prefix('wggithub_repositories').'.repo_readme = 1 ';

+        $sql .= 'WHERE ((('.$GLOBALS['xoopsDB']->prefix('wggithub_readmes').'.rm_id)>0));';

         $GLOBALS['xoopsDB']->queryF($sql);

 

         return true;

class/ReleasesHandler.php

Spacing +16 added lines, -16 removed lines

@@ -149,7 +149,7 @@ 
                 if (Constants::STATUS_NEW === $repoStatus || Constants::STATUS_UPDATED === $repoStatus || 0 == $releasesCount) {

                     $ghReleases = $githubClient->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;