ggoffy / wggithub

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;

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';

admin/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'];
 
 

include/functions.php

Indentation +2 added lines, -2 removed lines

@@ -133,7 +133,7 @@ 
 
         case 'none':
             if($topic_name) {
-                 $topic_name = 'topic=' . $topic_name . '&';
+                    $topic_name = 'topic=' . $topic_name . '&';
             }
             $rewrite_base = '/modules/';
             $page = 'page=' . $array['content_alias'];
@@ -163,7 +163,7 @@ 
             return XOOPS_URL . $rewrite_base . $module_name . $type . $topic_name  . $id . $page . $rewrite_ext;
             break;
 
-         case 'short':
+            case 'short':
             if($topic_name) {
                 $topic_name .= '/';
             }

Spacing +22 added lines, -22 removed lines

@@ -33,7 +33,7 @@ 
         $cat_sql .= current($cats);
         array_shift($cats);
         foreach ($cats as $cat) {
-            $cat_sql .= ',' . $cat;
+            $cat_sql .= ','.$cat;
         }
     }
     $cat_sql .= ')';
@@ -72,9 +72,9 @@ 
 {
     global $xoopsTpl, $xoTheme;
     $myts = MyTextSanitizer::getInstance();
-    $content= $myts->undoHtmlSpecialChars($myts->displayTarea($content));
-    if(isset($xoTheme) && \is_object($xoTheme)) {
-        $xoTheme->addMeta( 'meta', 'keywords', \strip_tags($content));
+    $content = $myts->undoHtmlSpecialChars($myts->displayTarea($content));
+    if (isset($xoTheme) && \is_object($xoTheme)) {
+        $xoTheme->addMeta('meta', 'keywords', \strip_tags($content));
     } else {    // Compatibility for old Xoops versions
         $xoopsTpl->assign('xoops_meta_keywords', \strip_tags($content));
     }
@@ -91,8 +91,8 @@ 
     global $xoopsTpl, $xoTheme;
     $myts = MyTextSanitizer::getInstance();
     $content = $myts->undoHtmlSpecialChars($myts->displayTarea($content));
-    if(isset($xoTheme) && \is_object($xoTheme)) {
-        $xoTheme->addMeta( 'meta', 'description', \strip_tags($content));
+    if (isset($xoTheme) && \is_object($xoTheme)) {
+        $xoTheme->addMeta('meta', 'description', \strip_tags($content));
     } else {    // Compatibility for old Xoops versions
         $xoopsTpl->assign('xoops_meta_description', \strip_tags($content));
     }
@@ -117,7 +117,7 @@ 
     if (0 != $lenght_id) {
         $id = $array['content_id'];
         while (\strlen($id) < $lenght_id) {
-            $id = '0' . $id;
+            $id = '0'.$id;
         }
     } else {
         $id = $array['content_id'];
@@ -132,23 +132,23 @@ 
     switch ($rewrite_url) {
 
         case 'none':
-            if($topic_name) {
-                 $topic_name = 'topic=' . $topic_name . '&amp;';
+            if ($topic_name) {
+                 $topic_name = 'topic='.$topic_name.'&amp;';
             }
             $rewrite_base = '/modules/';
-            $page = 'page=' . $array['content_alias'];
-            return XOOPS_URL . $rewrite_base . $module . '/' . $type . '.php?' . $topic_name . 'id=' . $id . '&amp;' . $page . $comment;
+            $page = 'page='.$array['content_alias'];
+            return XOOPS_URL.$rewrite_base.$module.'/'.$type.'.php?'.$topic_name.'id='.$id.'&amp;'.$page.$comment;
             break;
 
         case 'rewrite':
-            if($topic_name) {
+            if ($topic_name) {
                 $topic_name .= '/';
             }
             $rewrite_base = xoops_getModuleOption('rewrite_mode', $module);
             $rewrite_ext = xoops_getModuleOption('rewrite_ext', $module);
             $module_name = '';
-            if(xoops_getModuleOption('rewrite_name', $module)) {
-                $module_name = xoops_getModuleOption('rewrite_name', $module) . '/';
+            if (xoops_getModuleOption('rewrite_name', $module)) {
+                $module_name = xoops_getModuleOption('rewrite_name', $module).'/';
             }
             $page = $array['content_alias'];
             $type .= '/';
@@ -157,21 +157,21 @@ 
                 $type = '';
             }
             if ('comment-edit/' === $type || 'comment-reply/' === $type || 'comment-delete/' === $type) {
-                return XOOPS_URL . $rewrite_base . $module_name . $type . $id . '/';
+                return XOOPS_URL.$rewrite_base.$module_name.$type.$id.'/';
             }
 
-            return XOOPS_URL . $rewrite_base . $module_name . $type . $topic_name  . $id . $page . $rewrite_ext;
+            return XOOPS_URL.$rewrite_base.$module_name.$type.$topic_name.$id.$page.$rewrite_ext;
             break;
 
          case 'short':
-            if($topic_name) {
+            if ($topic_name) {
                 $topic_name .= '/';
             }
             $rewrite_base = xoops_getModuleOption('rewrite_mode', $module);
             $rewrite_ext = xoops_getModuleOption('rewrite_ext', $module);
             $module_name = '';
-            if(xoops_getModuleOption('rewrite_name', $module)) {
-                $module_name = xoops_getModuleOption('rewrite_name', $module) . '/';
+            if (xoops_getModuleOption('rewrite_name', $module)) {
+                $module_name = xoops_getModuleOption('rewrite_name', $module).'/';
             }
             $page = $array['content_alias'];
             $type .= '/';
@@ -179,10 +179,10 @@ 
                 $type = '';
             }
             if ('comment-edit/' === $type || 'comment-reply/' === $type || 'comment-delete/' === $type) {
-                return XOOPS_URL . $rewrite_base . $module_name . $type . $id . '/';
+                return XOOPS_URL.$rewrite_base.$module_name.$type.$id.'/';
             }
 
-            return XOOPS_URL . $rewrite_base . $module_name . $type . $topic_name . $page . $rewrite_ext;
+            return XOOPS_URL.$rewrite_base.$module_name.$type.$topic_name.$page.$rewrite_ext;
             break;
     }
     return null;
@@ -207,6 +207,6 @@ 
     $url .= htmlentities($url, ENT_COMPAT, 'utf-8');
     $url .= \preg_replace('`&([a-z])(acute|uml|circ|grave|ring|cedil|slash|tilde|caron|lig);`i', "\1", $url);
     $url .= \preg_replace(array($regular_expression, '`[-]+`'), '-', $url);
-    $url = ($url == '') ? $type : strtolower(    rim($url, '-'));
+    $url = ($url == '') ? $type : strtolower(rim($url, '-'));
     return $url;
 }
\ No newline at end of file

include/common.php

Spacing +21 added lines, -21 removed lines

@@ -20,29 +20,29 @@
  * @author         Goffy - XOOPS Development Team - Email:<goffy@wedega.com> - Website:<https://wedega.com>
  */
 if (!\defined('XOOPS_ICONS32_PATH')) {
-    \define('XOOPS_ICONS32_PATH', XOOPS_ROOT_PATH . '/Frameworks/moduleclasses/icons/32');
+    \define('XOOPS_ICONS32_PATH', XOOPS_ROOT_PATH.'/Frameworks/moduleclasses/icons/32');
 }
 if (!\defined('XOOPS_ICONS32_URL')) {
-    \define('XOOPS_ICONS32_URL', XOOPS_URL . '/Frameworks/moduleclasses/icons/32');
+    \define('XOOPS_ICONS32_URL', XOOPS_URL.'/Frameworks/moduleclasses/icons/32');
 }
 \define('WGGITHUB_DIRNAME', 'wggithub');
-\define('WGGITHUB_PATH', XOOPS_ROOT_PATH . '/modules/' . WGGITHUB_DIRNAME);
-\define('WGGITHUB_URL', XOOPS_URL . '/modules/' . WGGITHUB_DIRNAME);
-\define('WGGITHUB_ICONS_PATH', WGGITHUB_PATH . '/assets/icons');
-\define('WGGITHUB_ICONS_URL', WGGITHUB_URL . '/assets/icons');
-\define('WGGITHUB_IMAGE_PATH', WGGITHUB_PATH . '/assets/images');
-\define('WGGITHUB_IMAGE_URL', WGGITHUB_URL . '/assets/images');
-\define('WGGITHUB_UPLOAD_PATH', XOOPS_UPLOAD_PATH . '/' . WGGITHUB_DIRNAME);
-\define('WGGITHUB_UPLOAD_URL', XOOPS_UPLOAD_URL . '/' . WGGITHUB_DIRNAME);
-\define('WGGITHUB_UPLOAD_FILES_PATH', WGGITHUB_UPLOAD_PATH . '/files');
-\define('WGGITHUB_UPLOAD_FILES_URL', WGGITHUB_UPLOAD_URL . '/files');
-\define('WGGITHUB_UPLOAD_IMAGE_PATH', WGGITHUB_UPLOAD_PATH . '/images');
-\define('WGGITHUB_UPLOAD_IMAGE_URL', WGGITHUB_UPLOAD_URL . '/images');
-\define('WGGITHUB_UPLOAD_SHOTS_PATH', WGGITHUB_UPLOAD_PATH . '/images/shots');
-\define('WGGITHUB_UPLOAD_SHOTS_URL', WGGITHUB_UPLOAD_URL . '/images/shots');
-\define('WGGITHUB_ADMIN', WGGITHUB_URL . '/admin/index.php');
-$localLogo = WGGITHUB_IMAGE_URL . '/tdmxoops_logo.png';
+\define('WGGITHUB_PATH', XOOPS_ROOT_PATH.'/modules/'.WGGITHUB_DIRNAME);
+\define('WGGITHUB_URL', XOOPS_URL.'/modules/'.WGGITHUB_DIRNAME);
+\define('WGGITHUB_ICONS_PATH', WGGITHUB_PATH.'/assets/icons');
+\define('WGGITHUB_ICONS_URL', WGGITHUB_URL.'/assets/icons');
+\define('WGGITHUB_IMAGE_PATH', WGGITHUB_PATH.'/assets/images');
+\define('WGGITHUB_IMAGE_URL', WGGITHUB_URL.'/assets/images');
+\define('WGGITHUB_UPLOAD_PATH', XOOPS_UPLOAD_PATH.'/'.WGGITHUB_DIRNAME);
+\define('WGGITHUB_UPLOAD_URL', XOOPS_UPLOAD_URL.'/'.WGGITHUB_DIRNAME);
+\define('WGGITHUB_UPLOAD_FILES_PATH', WGGITHUB_UPLOAD_PATH.'/files');
+\define('WGGITHUB_UPLOAD_FILES_URL', WGGITHUB_UPLOAD_URL.'/files');
+\define('WGGITHUB_UPLOAD_IMAGE_PATH', WGGITHUB_UPLOAD_PATH.'/images');
+\define('WGGITHUB_UPLOAD_IMAGE_URL', WGGITHUB_UPLOAD_URL.'/images');
+\define('WGGITHUB_UPLOAD_SHOTS_PATH', WGGITHUB_UPLOAD_PATH.'/images/shots');
+\define('WGGITHUB_UPLOAD_SHOTS_URL', WGGITHUB_UPLOAD_URL.'/images/shots');
+\define('WGGITHUB_ADMIN', WGGITHUB_URL.'/admin/index.php');
+$localLogo = WGGITHUB_IMAGE_URL.'/tdmxoops_logo.png';
 // Module Information
-$copyright = "<a href='https://wedega.com' title='XOOPS on Wedega' target='_blank'><img src='" . $localLogo . "' alt='XOOPS on Wedega' /></a>";
-include_once XOOPS_ROOT_PATH . '/class/xoopsrequest.php';
-include_once WGGITHUB_PATH . '/include/functions.php';
+$copyright = "<a href='https://wedega.com' title='XOOPS on Wedega' target='_blank'><img src='".$localLogo."' alt='XOOPS on Wedega' /></a>";
+include_once XOOPS_ROOT_PATH.'/class/xoopsrequest.php';
+include_once WGGITHUB_PATH.'/include/functions.php';

include/update.php

Spacing +16 added lines, -16 removed lines

@@ -39,7 +39,7 @@ 
     $ret = wggithub_check_db($module);
 
     //check upload directory
-    include_once __DIR__ . '/install.php';
+    include_once __DIR__.'/install.php';
     $ret = xoops_module_install_wggithub($module);
 
     $errors = $module->getErrors();
@@ -61,7 +61,7 @@ 
 {
     global $xoopsDB;
     $result = $xoopsDB->query(
-        'SELECT t1.tpl_id FROM ' . $xoopsDB->prefix('tplfile') . ' t1, ' . $xoopsDB->prefix('tplfile') . ' t2 WHERE t1.tpl_refid = t2.tpl_refid AND t1.tpl_module = t2.tpl_module AND t1.tpl_tplset=t2.tpl_tplset AND t1.tpl_file = t2.tpl_file AND t1.tpl_type = t2.tpl_type AND t1.tpl_id > t2.tpl_id'
+        'SELECT t1.tpl_id FROM '.$xoopsDB->prefix('tplfile').' t1, '.$xoopsDB->prefix('tplfile').' t2 WHERE t1.tpl_refid = t2.tpl_refid AND t1.tpl_module = t2.tpl_module AND t1.tpl_tplset=t2.tpl_tplset AND t1.tpl_file = t2.tpl_file AND t1.tpl_type = t2.tpl_type AND t1.tpl_id > t2.tpl_id'
     );
     $tplids = [];
     while (false !== (list($tplid) = $xoopsDB->fetchRow($result))) {
@@ -69,7 +69,7 @@ 
     }
     if (\count($tplids) > 0) {
         $tplfileHandler  = \xoops_getHandler('tplfile');
-        $duplicate_files = $tplfileHandler->getObjects(new \Criteria('tpl_id', '(' . \implode(',', $tplids) . ')', 'IN'));
+        $duplicate_files = $tplfileHandler->getObjects(new \Criteria('tpl_id', '('.\implode(',', $tplids).')', 'IN'));
 
         if (\count($duplicate_files) > 0) {
             foreach (\array_keys($duplicate_files) as $i) {
@@ -77,9 +77,9 @@ 
             }
         }
     }
-    $sql = 'SHOW INDEX FROM ' . $xoopsDB->prefix('tplfile') . " WHERE KEY_NAME = 'tpl_refid_module_set_file_type'";
+    $sql = 'SHOW INDEX FROM '.$xoopsDB->prefix('tplfile')." WHERE KEY_NAME = 'tpl_refid_module_set_file_type'";
     if (!$result = $xoopsDB->queryF($sql)) {
-        xoops_error($xoopsDB->error() . '<br>' . $sql);
+        xoops_error($xoopsDB->error().'<br>'.$sql);
 
         return false;
     }
@@ -92,9 +92,9 @@ 
 
         return true;
     }
-    $sql = 'ALTER TABLE ' . $xoopsDB->prefix('tplfile') . ' ADD UNIQUE tpl_refid_module_set_file_type ( tpl_refid, tpl_module, tpl_tplset, tpl_file, tpl_type )';
+    $sql = 'ALTER TABLE '.$xoopsDB->prefix('tplfile').' ADD UNIQUE tpl_refid_module_set_file_type ( tpl_refid, tpl_module, tpl_tplset, tpl_file, tpl_type )';
     if (!$result = $xoopsDB->queryF($sql)) {
-        xoops_error($xoopsDB->error() . '<br>' . $sql);
+        xoops_error($xoopsDB->error().'<br>'.$sql);
         $module->setErrors("'tpl_refid_module_set_file_type' unique index is not added to 'tplfile' table. Warning: do not use XOOPS until you add this unique index.");
 
         return false;
@@ -118,12 +118,12 @@ 
     // Example: update table (add new field)
     $table   = $GLOBALS['xoopsDB']->prefix('wggithub_repositories');
     $field   = 'repo_release';
-    $check   = $GLOBALS['xoopsDB']->queryF('SHOW COLUMNS FROM `' . $table . "` LIKE '" . $field . "'");
+    $check   = $GLOBALS['xoopsDB']->queryF('SHOW COLUMNS FROM `'.$table."` LIKE '".$field."'");
     $numRows = $GLOBALS['xoopsDB']->getRowsNum($check);
     if (!$numRows) {
         $sql = "ALTER TABLE `$table` ADD `$field` INT(1) NOT NULL DEFAULT '0' AFTER `repo_htmlurl`;";
         if (!$result = $GLOBALS['xoopsDB']->queryF($sql)) {
-            xoops_error($GLOBALS['xoopsDB']->error() . '<br>' . $sql);
+            xoops_error($GLOBALS['xoopsDB']->error().'<br>'.$sql);
             $module->setErrors("Error when adding '$field' to table '$table'.");
             $ret = false;
         }
@@ -132,12 +132,12 @@ 
     // Example: update table (add new field)
     $table   = $GLOBALS['xoopsDB']->prefix('wggithub_repositories');
     $field   = 'repo_prerelease';
-    $check   = $GLOBALS['xoopsDB']->queryF('SHOW COLUMNS FROM `' . $table . "` LIKE '" . $field . "'");
+    $check   = $GLOBALS['xoopsDB']->queryF('SHOW COLUMNS FROM `'.$table."` LIKE '".$field."'");
     $numRows = $GLOBALS['xoopsDB']->getRowsNum($check);
     if (!$numRows) {
         $sql = "ALTER TABLE `$table` ADD `$field` INT(1) NOT NULL DEFAULT '0' AFTER `repo_htmlurl`;";
         if (!$result = $GLOBALS['xoopsDB']->queryF($sql)) {
-            xoops_error($GLOBALS['xoopsDB']->error() . '<br>' . $sql);
+            xoops_error($GLOBALS['xoopsDB']->error().'<br>'.$sql);
             $module->setErrors("Error when adding '$field' to table '$table'.");
             $ret = false;
         }
@@ -145,12 +145,12 @@ 
     // Example: update table (add new field)
     $table   = $GLOBALS['xoopsDB']->prefix('wggithub_repositories');
     $field   = 'repo_readme';
-    $check   = $GLOBALS['xoopsDB']->queryF('SHOW COLUMNS FROM `' . $table . "` LIKE '" . $field . "'");
+    $check   = $GLOBALS['xoopsDB']->queryF('SHOW COLUMNS FROM `'.$table."` LIKE '".$field."'");
     $numRows = $GLOBALS['xoopsDB']->getRowsNum($check);
     if (!$numRows) {
         $sql = "ALTER TABLE `$table` ADD `$field` INT(1) NOT NULL DEFAULT '0' AFTER `repo_htmlurl`;";
         if (!$result = $GLOBALS['xoopsDB']->queryF($sql)) {
-            xoops_error($GLOBALS['xoopsDB']->error() . '<br>' . $sql);
+            xoops_error($GLOBALS['xoopsDB']->error().'<br>'.$sql);
             $module->setErrors("Error when adding '$field' to table '$table'.");
             $ret = false;
         }
@@ -159,12 +159,12 @@ 
 // Example: update table (add new field)
     $table   = $GLOBALS['xoopsDB']->prefix('wggithub_directories');
     $field   = 'dir_filterrelease';
-    $check   = $GLOBALS['xoopsDB']->queryF('SHOW COLUMNS FROM `' . $table . "` LIKE '" . $field . "'");
+    $check   = $GLOBALS['xoopsDB']->queryF('SHOW COLUMNS FROM `'.$table."` LIKE '".$field."'");
     $numRows = $GLOBALS['xoopsDB']->getRowsNum($check);
     if (!$numRows) {
         $sql = "ALTER TABLE `$table` ADD `$field` INT(1) NOT NULL DEFAULT '0' AFTER `dir_online`;";
         if (!$result = $GLOBALS['xoopsDB']->queryF($sql)) {
-            xoops_error($GLOBALS['xoopsDB']->error() . '<br>' . $sql);
+            xoops_error($GLOBALS['xoopsDB']->error().'<br>'.$sql);
             $module->setErrors("Error when adding '$field' to table '$table'.");
             $ret = false;
         }
@@ -186,7 +186,7 @@ 
               PRIMARY KEY (`log_id`)
                 ) ENGINE=InnoDB;";
         if (!$result = $GLOBALS['xoopsDB']->queryF($sql)) {
-            xoops_error($GLOBALS['xoopsDB']->error() . '<br>' . $sql);
+            xoops_error($GLOBALS['xoopsDB']->error().'<br>'.$sql);
             $module->setErrors("Error when creating table '$table'.");
             $ret = false;
         }

include/install.php

Spacing +7 added lines, -7 removed lines

@@ -29,7 +29,7 @@ 
  */
 function xoops_module_pre_install_wggithub(\XoopsModule $module)
 {
-    require \dirname(__DIR__) . '/preloads/autoloader.php';
+    require \dirname(__DIR__).'/preloads/autoloader.php';
 
     $utility = new Wggithub\Utility();
 
@@ -42,7 +42,7 @@ 
     if ($xoopsSuccess && $phpSuccess) {
         $moduleTables = &$module->getInfo('tables');
         foreach ($moduleTables as $table) {
-            $GLOBALS['xoopsDB']->queryF('DROP TABLE IF EXISTS ' . $GLOBALS['xoopsDB']->prefix($table) . ';');
+            $GLOBALS['xoopsDB']->queryF('DROP TABLE IF EXISTS '.$GLOBALS['xoopsDB']->prefix($table).';');
         }
     }
 
@@ -55,7 +55,7 @@ 
  */
 function xoops_module_install_wggithub(\XoopsModule $module)
 {
-    require \dirname(__DIR__) . '/preloads/autoloader.php';
+    require \dirname(__DIR__).'/preloads/autoloader.php';
 
     /** @var Wggithub\Helper $helper */ 
     /** @var Wggithub\Utility $utility */
@@ -79,14 +79,14 @@ 
 
     //  ---  COPY blank.gif FILES ---------------
     if ($configurator->copyBlankFiles && \is_array($configurator->copyBlankFiles)) {
-        $file = \dirname(__DIR__) . '/assets/images/blank.gif';
+        $file = \dirname(__DIR__).'/assets/images/blank.gif';
         foreach (\array_keys($configurator->copyBlankFiles) as $i) {
-            $dest = $configurator->copyBlankFiles[$i] . '/blank.gif';
+            $dest = $configurator->copyBlankFiles[$i].'/blank.gif';
             $utility::copyFile($file, $dest);
         }
-        $file = \dirname(__DIR__) . '/assets/images/blank.png';
+        $file = \dirname(__DIR__).'/assets/images/blank.png';
         foreach (\array_keys($configurator->copyBlankFiles) as $i) {
-            $dest = $configurator->copyBlankFiles[$i] . '/blank.png';
+            $dest = $configurator->copyBlankFiles[$i].'/blank.png';
             $utility::copyFile($file, $dest);
         }
     }

include/search.inc.php

Spacing +3 added lines, -3 removed lines

@@ -47,7 +47,7 @@ 
     }
     if ($elementCount > 0) {
         $crKeywords = new \CriteriaCompo();
-        for ($i = 0; $i  <  $elementCount; $i++) {
+        for ($i = 0; $i < $elementCount; $i++) {
             $crKeyword = new \CriteriaCompo();
             unset($crKeyword);
         }
@@ -56,7 +56,7 @@ 
     if ($userid && \is_array($userid)) {
         $userid = array_map('intval', $userid);
         $crUser = new \CriteriaCompo();
-        $crUser->add(new \Criteria('repo_submitter', '(' . \implode(',', $userid) . ')', 'IN'), 'OR');
+        $crUser->add(new \Criteria('repo_submitter', '('.\implode(',', $userid).')', 'IN'), 'OR');
     } elseif (is_numeric($userid) && $userid > 0) {
         $crUser = new \CriteriaCompo();
         $crUser->add(new \Criteria('repo_submitter', $userid), 'OR');
@@ -76,7 +76,7 @@ 
     foreach (\array_keys($repositoriesAll) as $i) {
         $ret[] = [
             'image'  => 'assets/icons/16/repositories.png',
-            'link'   => 'repositories.php?op=show&amp;repo_id=' . $repositoriesAll[$i]->getVar('repo_id'),
+            'link'   => 'repositories.php?op=show&amp;repo_id='.$repositoriesAll[$i]->getVar('repo_id'),
             'title'  => $repositoriesAll[$i]->getVar('repo_name'),
             'time'   => $repositoriesAll[$i]->getVar('repo_datecreated')
         ];

include/notification.inc.php

Spacing +3 added lines, -3 removed lines

@@ -32,7 +32,7 @@ 
     global $xoopsDB;
 
     if (!\defined('WGGITHUB_URL')) {
-        \define('WGGITHUB_URL', \XOOPS_URL . '/modules/wggithub');
+        \define('WGGITHUB_URL', \XOOPS_URL.'/modules/wggithub');
     }
 
     switch ($category) {
@@ -42,11 +42,11 @@ 
             return $item;
             break;
         case 'repositories':
-            $sql          = 'SELECT repo_name FROM ' . $xoopsDB->prefix('wggithub_repositories') . ' WHERE repo_id = '. $item_id;
+            $sql          = 'SELECT repo_name FROM '.$xoopsDB->prefix('wggithub_repositories').' WHERE repo_id = '.$item_id;
             $result       = $xoopsDB->query($sql);
             $result_array = $xoopsDB->fetchArray($result);
             $item['name'] = $result_array['repo_name'];
-            $item['url']  = WGGITHUB_URL . '/repositories.php?repo_id=' . $item_id;
+            $item['url']  = WGGITHUB_URL.'/repositories.php?repo_id='.$item_id;
             return $item;
             break;
     }