ggoffy / wggithub

preloads/autoloader.php

Spacing +4 added lines, -4 removed lines

@@ -4,12 +4,12 @@ 
  * @see http://www.php-fig.org/psr/psr-4/examples/
  */
 spl_autoload_register(
-    static function ($class) {
+    static function($class) {
         // project-specific namespace prefix
-        $prefix = 'XoopsModules\\' . \ucfirst(\basename(\dirname(__DIR__)));
+        $prefix = 'XoopsModules\\'.\ucfirst(\basename(\dirname(__DIR__)));
 
         // base directory for the namespace prefix
-        $baseDir = __DIR__ . '/../class/';
+        $baseDir = __DIR__.'/../class/';
 
         // does the class use the namespace prefix?
         $len = mb_strlen($prefix);
@@ -24,7 +24,7 @@ 
         // replace the namespace prefix with the base directory, replace namespace
         // separators with directory separators in the relative class name, append
         // with .php
-        $file = $baseDir . \str_replace('\\', '/', $relativeClass) . '.php';
+        $file = $baseDir.\str_replace('\\', '/', $relativeClass).'.php';
 
         // if the file exists, require it
         if (\file_exists($file)) {

preloads/core.php

Spacing +1 added lines, -1 removed lines

@@ -28,6 +28,6 @@
      */
     public static function eventCoreIncludeCommonEnd($args)
     {
-        include __DIR__ . '/autoloader.php';
+        include __DIR__.'/autoloader.php';
     }
 }

admin/permissions.php

Spacing +3 added lines, -3 removed lines

@@ -24,7 +24,7 @@ 
 use XoopsModules\Wggithub;
 use XoopsModules\Wggithub\Constants;
 
-require __DIR__ . '/header.php';
+require __DIR__.'/header.php';
 
 // Template Index
 $templateMain = 'wggithub_admin_permissions.tpl';
@@ -33,7 +33,7 @@ 
 $op = Request::getCmd('op', 'global');
 
 // Get Form
-include_once XOOPS_ROOT_PATH . '/class/xoopsform/grouppermform.php';
+include_once XOOPS_ROOT_PATH.'/class/xoopsform/grouppermform.php';
 \xoops_load('XoopsFormLoader');
 
 $formTitle = \_AM_WGGITHUB_PERMISSIONS_GLOBAL;
@@ -61,4 +61,4 @@ 
     exit();
 }
 
-require __DIR__ . '/footer.php';
+require __DIR__.'/footer.php';

testdata/index.php

Spacing +19 added lines, -19 removed lines

@@ -24,8 +24,8 @@ 
     Utility
 };
 
-require_once dirname(__DIR__, 3) . '/include/cp_header.php';
-require \dirname(__DIR__) . '/preloads/autoloader.php';
+require_once dirname(__DIR__, 3).'/include/cp_header.php';
+require \dirname(__DIR__).'/preloads/autoloader.php';
 
 $op = \Xmf\Request::getCmd('op', '');
 
@@ -45,7 +45,7 @@ 
             loadSampleData();
         } else {
             xoops_cp_header();
-            xoops_confirm(['ok' => 1, 'op' => 'load'], 'index.php', \sprintf(\constant('CO_' . $moduleDirNameUpper . '_' . 'ADD_SAMPLEDATA_OK')), \constant('CO_' . $moduleDirNameUpper . '_' . 'CONFIRM'), true);
+            xoops_confirm(['ok' => 1, 'op' => 'load'], 'index.php', \sprintf(\constant('CO_'.$moduleDirNameUpper.'_'.'ADD_SAMPLEDATA_OK')), \constant('CO_'.$moduleDirNameUpper.'_'.'CONFIRM'), true);
             xoops_cp_footer();
         }
         break;
@@ -68,20 +68,20 @@ 
     $tables = Helper::getHelper($moduleDirName)->getModule()->getInfo('tables');
 
     $language = 'english/';
-    if (\is_dir(__DIR__ . '/' . $xoopsConfig['language'])) {
-        $language = $xoopsConfig['language'] . '/';
+    if (\is_dir(__DIR__.'/'.$xoopsConfig['language'])) {
+        $language = $xoopsConfig['language'].'/';
     }
 
     // load module tables
     foreach ($tables as $table) {
-        $tabledata = Yaml::readWrapped($language . $table . '.yml');
+        $tabledata = Yaml::readWrapped($language.$table.'.yml');
         TableLoad::truncateTable($table);
         TableLoad::loadTableFromArray($table, $tabledata);
     }
 
     // load permissions
     $table     = 'group_permission';
-    $tabledata = Yaml::readWrapped($language . $table . '.yml');
+    $tabledata = Yaml::readWrapped($language.$table.'.yml');
     $mid       = Helper::getHelper($moduleDirName)->getModule()->getVar('mid');
     loadTableFromArrayWithReplace($table, $tabledata, 'gperm_modid', $mid);
 
@@ -94,7 +94,7 @@ 
             $utility::rcopy($src, $dest);
         }
     }
-    \redirect_header('../admin/index.php', 1, \constant('CO_' . $moduleDirNameUpper . '_' . 'SAMPLEDATA_SUCCESS'));
+    \redirect_header('../admin/index.php', 1, \constant('CO_'.$moduleDirNameUpper.'_'.'SAMPLEDATA_SUCCESS'));
 }
 
 function saveSampleData()
@@ -108,23 +108,23 @@ 
 
     $tables = Helper::getHelper($moduleDirName)->getModule()->getInfo('tables');
 
-    $languageFolder = __DIR__ . '/' . $xoopsConfig['language'];
-    if (!\file_exists($languageFolder . '/')) {
-        Utility::createFolder($languageFolder . '/');
+    $languageFolder = __DIR__.'/'.$xoopsConfig['language'];
+    if (!\file_exists($languageFolder.'/')) {
+        Utility::createFolder($languageFolder.'/');
     }
-    $exportFolder = $languageFolder . '/Exports-' . date('Y-m-d-H-i-s') . '/';
+    $exportFolder = $languageFolder.'/Exports-'.date('Y-m-d-H-i-s').'/';
     Utility::createFolder($exportFolder);
 
     // save module tables
     foreach ($tables as $table) {
-        TableLoad::saveTableToYamlFile($table, $exportFolder . $table . '.yml');
+        TableLoad::saveTableToYamlFile($table, $exportFolder.$table.'.yml');
     }
 
     // save permissions
     $criteria = new \CriteriaCompo();
     $criteria->add(new \Criteria('gperm_modid', Helper::getHelper($moduleDirName)->getModule()->getVar('mid')));
     $skipColumns[] = 'gperm_id';
-    TableLoad::saveTableToYamlFile('group_permission', $exportFolder . 'group_permission.yml', $criteria, $skipColumns);
+    TableLoad::saveTableToYamlFile('group_permission', $exportFolder.'group_permission.yml', $criteria, $skipColumns);
     unset($criteria);
 
     //  ---  COPY test folder files ---------------
@@ -135,7 +135,7 @@ 
             Utility::rcopy($src, $dest);
         }
     }
-    \redirect_header('../admin/index.php', 1, \constant('CO_' . $moduleDirNameUpper . '_' . 'SAVE_SAMPLEDATA_SUCCESS'));
+    \redirect_header('../admin/index.php', 1, \constant('CO_'.$moduleDirNameUpper.'_'.'SAVE_SAMPLEDATA_SUCCESS'));
 }
 
 function exportSchema()
@@ -150,7 +150,7 @@ 
         //
         //        \redirect_header('../admin/index.php', 1, \constant('CO_' . $moduleDirNameUpper . '_' . 'EXPORT_SCHEMA_SUCCESS'));
     } catch (\Exception $e) {
-        exit(\constant('CO_' . $moduleDirNameUpper . '_' . 'EXPORT_SCHEMA_ERROR'));
+        exit(\constant('CO_'.$moduleDirNameUpper.'_'.'EXPORT_SCHEMA_ERROR'));
     }
 
 }
@@ -173,10 +173,10 @@ 
     $db = \XoopsDatabaseFactory::getDatabaseConnection();
     $prefixedTable = $db->prefix($table);
     $count = 0;
-    $sql = 'DELETE FROM ' . $prefixedTable . ' WHERE `' . $search . '`=' . $db->quote($replace);
+    $sql = 'DELETE FROM '.$prefixedTable.' WHERE `'.$search.'`='.$db->quote($replace);
     $db->queryF($sql);
     foreach ($data as $row) {
-        $insertInto = 'INSERT INTO ' . $prefixedTable . ' (';
+        $insertInto = 'INSERT INTO '.$prefixedTable.' (';
         $valueClause = ' VALUES (';
         $first = true;
         foreach ($row as $column => $value) {
@@ -193,7 +193,7 @@ 
                 $valueClause .= $db->quote($value);
             }
         }
-        $sql = $insertInto . ') ' . $valueClause . ')';
+        $sql = $insertInto.') '.$valueClause.')';
         $result = $db->queryF($sql);
         if (false !== $result) {
             ++$count;

class/SettingsHandler.php

Spacing +2 added lines, -2 removed lines

@@ -148,7 +148,7 @@ 
      */
     public function setPrimarySetting($setId)
     {
-        $helper  = \XoopsModules\Wggithub\Helper::getInstance();
+        $helper = \XoopsModules\Wggithub\Helper::getInstance();
         $settingsObj = null;
         $settingsHandler = $helper->getHandler('Settings');
         if (isset($setId)) {
@@ -158,7 +158,7 @@ 
         }
 
         // reset all
-        $strSQL = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('wggithub_settings') . ' SET ' . $GLOBALS['xoopsDB']->prefix('wggithub_settings') . '.set_primary = 0';
+        $strSQL = 'UPDATE '.$GLOBALS['xoopsDB']->prefix('wggithub_settings').' SET '.$GLOBALS['xoopsDB']->prefix('wggithub_settings').'.set_primary = 0';
         $GLOBALS['xoopsDB']->queryF($strSQL);
         // Set Vars
         $settingsObj->setVar('set_primary', 1);

class/LogsHandler.php

Spacing +1 added lines, -1 removed lines

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

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