ggoffy / wggithub

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

class/MDParser/Parsedown.php

Spacing +59 added lines, -59 removed lines

@@ -209,7 +209,7 @@ 
 
             if (isset($CurrentBlock['continuable']))
             {
-                $methodName = 'block' . $CurrentBlock['type'] . 'Continue';
+                $methodName = 'block'.$CurrentBlock['type'].'Continue';
                 $Block = $this->$methodName($Line, $CurrentBlock);
 
                 if (isset($Block))
@@ -220,7 +220,7 @@ 
                 } else {
                     if ($this->isBlockCompletable($CurrentBlock['type']))
                     {
-                        $methodName = 'block' . $CurrentBlock['type'] . 'Complete';
+                        $methodName = 'block'.$CurrentBlock['type'].'Complete';
                         $CurrentBlock = $this->$methodName($CurrentBlock);
                     }
                 }
@@ -238,7 +238,7 @@ 
             {
                 foreach ($this->BlockTypes[$marker] as $blockType)
                 {
-                    $blockTypes []= $blockType;
+                    $blockTypes [] = $blockType;
                 }
             }
 
@@ -253,7 +253,7 @@ 
                 {
                     $Block['type'] = $blockType;
 
-                    if ( ! isset($Block['identified']))
+                    if (!isset($Block['identified']))
                     {
                         if (isset($CurrentBlock))
                         {
@@ -300,7 +300,7 @@ 
 
         if (isset($CurrentBlock['continuable']) and $this->isBlockCompletable($CurrentBlock['type']))
         {
-            $methodName = 'block' . $CurrentBlock['type'] . 'Complete';
+            $methodName = 'block'.$CurrentBlock['type'].'Complete';
             $CurrentBlock = $this->$methodName($CurrentBlock);
         }
 
@@ -318,7 +318,7 @@ 
 
     protected function extractElement(array $Component)
     {
-        if ( ! isset($Component['element']))
+        if (!isset($Component['element']))
         {
             if (isset($Component['markup']))
             {
@@ -333,12 +333,12 @@ 
 
     protected function isBlockContinuable($Type)
     {
-        return method_exists($this, 'block' . $Type . 'Continue');
+        return method_exists($this, 'block'.$Type.'Continue');
     }
 
     protected function isBlockCompletable($Type)
     {
-        return method_exists($this, 'block' . $Type . 'Complete');
+        return method_exists($this, 'block'.$Type.'Complete');
     }
 
     #
@@ -346,7 +346,7 @@ 
 
     protected function blockCode($Line, $Block = null)
     {
-        if (isset($Block) and $Block['type'] === 'Paragraph' and ! isset($Block['interrupted']))
+        if (isset($Block) and $Block['type'] === 'Paragraph' and !isset($Block['interrupted']))
         {
             return;
         }
@@ -430,7 +430,7 @@ 
             return;
         }
 
-        $Block['element']['rawHtml'] .= "\n" . $Line['body'];
+        $Block['element']['rawHtml'] .= "\n".$Line['body'];
 
         if (strpos($Line['text'], '-->') !== false)
         {
@@ -521,7 +521,7 @@ 
             return $Block;
         }
 
-        $Block['element']['element']['text'] .= "\n" . $Line['body'];
+        $Block['element']['element']['text'] .= "\n".$Line['body'];
 
         return $Block;
     }
@@ -554,7 +554,7 @@ 
 
         $Block = array(
             'element' => array(
-                'name' => 'h' . $level,
+                'name' => 'h'.$level,
                 'handler' => array(
                     'function' => 'lineElements',
                     'argument' => $text,
@@ -581,7 +581,7 @@ 
             {
                 $contentIndent -= 1;
                 $matches[1] = substr($matches[1], 0, -$contentIndent);
-                $matches[3] = str_repeat(' ', $contentIndent) . $matches[3];
+                $matches[3] = str_repeat(' ', $contentIndent).$matches[3];
             } elseif ($contentIndent === 0) {
                 $matches[1] .= ' ';
             }
@@ -612,7 +612,7 @@ 
                     if (
                         isset($CurrentBlock)
                         and $CurrentBlock['type'] === 'Paragraph'
-                        and ! isset($CurrentBlock['interrupted'])
+                        and !isset($CurrentBlock['interrupted'])
                     ) {
                         return;
                     }
@@ -630,7 +630,7 @@ 
                 )
             );
 
-            $Block['element']['elements'] []= & $Block['li'];
+            $Block['element']['elements'] [] = & $Block['li'];
 
             return $Block;
         }
@@ -657,7 +657,7 @@ 
             )
         ) {
             if (isset($Block['interrupted'])) {
-                $Block['li']['handler']['argument'] []= '';
+                $Block['li']['handler']['argument'] [] = '';
 
                 $Block['loose'] = true;
 
@@ -679,7 +679,7 @@ 
                 )
             );
 
-            $Block['element']['elements'] []= & $Block['li'];
+            $Block['element']['elements'] [] = & $Block['li'];
 
             return $Block;
         } elseif ($Line['indent'] < $requiredIndent and $this->blockList($Line)) {
@@ -693,7 +693,7 @@ 
         if ($Line['indent'] >= $requiredIndent) {
             if (isset($Block['interrupted']))
             {
-                $Block['li']['handler']['argument'] []= '';
+                $Block['li']['handler']['argument'] [] = '';
 
                 $Block['loose'] = true;
 
@@ -702,15 +702,15 @@ 
 
             $text = substr($Line['body'], $requiredIndent);
 
-            $Block['li']['handler']['argument'] []= $text;
+            $Block['li']['handler']['argument'] [] = $text;
 
             return $Block;
         }
 
-        if ( ! isset($Block['interrupted'])) {
+        if (!isset($Block['interrupted'])) {
             $text = preg_replace('/^[ ]{0,'.$requiredIndent.'}+/', '', $Line['body']);
 
-            $Block['li']['handler']['argument'] []= $text;
+            $Block['li']['handler']['argument'] [] = $text;
 
             return $Block;
         }
@@ -723,7 +723,7 @@ 
             {
                 if (end($li['handler']['argument']) !== '')
                 {
-                    $li['handler']['argument'] []= '';
+                    $li['handler']['argument'] [] = '';
                 }
             }
         }
@@ -759,13 +759,13 @@ 
         }
 
         if ($Line['text'][0] === '>' and preg_match('/^>[ ]?+(.*+)/', $Line['text'], $matches)) {
-            $Block['element']['handler']['argument'] []= $matches[1];
+            $Block['element']['handler']['argument'] [] = $matches[1];
 
             return $Block;
         }
 
-        if ( ! isset($Block['interrupted'])) {
-            $Block['element']['handler']['argument'] []= $Line['text'];
+        if (!isset($Block['interrupted'])) {
+            $Block['element']['handler']['argument'] [] = $Line['text'];
 
             return $Block;
         }
@@ -794,7 +794,7 @@ 
 
     protected function blockSetextHeader($Line, array $Block = null)
     {
-        if ( ! isset($Block) or $Block['type'] !== 'Paragraph' or isset($Block['interrupted'])) {
+        if (!isset($Block) or $Block['type'] !== 'Paragraph' or isset($Block['interrupted'])) {
             return;
         }
 
@@ -839,7 +839,7 @@ 
             return;
         }
 
-        $Block['element']['rawHtml'] .= "\n" . $Line['body'];
+        $Block['element']['rawHtml'] .= "\n".$Line['body'];
 
         return $Block;
     }
@@ -874,7 +874,7 @@ 
 
     protected function blockTable($Line, array $Block = null)
     {
-        if ( ! isset($Block) or $Block['type'] !== 'Paragraph' or isset($Block['interrupted'])) {
+        if (!isset($Block) or $Block['type'] !== 'Paragraph' or isset($Block['interrupted'])) {
             return;
         }
 
@@ -917,7 +917,7 @@ 
                 $alignment = $alignment === 'left' ? 'center' : 'right';
             }
 
-            $alignments []= $alignment;
+            $alignments [] = $alignment;
         }
 
         # ~
@@ -955,7 +955,7 @@ 
                 );
             }
 
-            $HeaderElements []= $HeaderElement;
+            $HeaderElements [] = $HeaderElement;
         }
 
         # ~
@@ -969,16 +969,16 @@ 
             ),
         );
 
-        $Block['element']['elements'] []= array(
+        $Block['element']['elements'] [] = array(
             'name' => 'thead',
         );
 
-        $Block['element']['elements'] []= array(
+        $Block['element']['elements'] [] = array(
             'name' => 'tbody',
             'elements' => array(),
         );
 
-        $Block['element']['elements'][0]['elements'] []= array(
+        $Block['element']['elements'][0]['elements'] [] = array(
             'name' => 'tr',
             'elements' => $HeaderElements,
         );
@@ -1021,11 +1021,11 @@ 
                 if (isset($Block['alignments'][$index]))
                 {
                     $Element['attributes'] = array(
-                        'style' => 'text-align: ' . $Block['alignments'][$index] . ';',
+                        'style' => 'text-align: '.$Block['alignments'][$index].';',
                     );
                 }
 
-                $Elements []= $Element;
+                $Elements [] = $Element;
             }
 
             $Element = array(
@@ -1033,7 +1033,7 @@ 
                 'elements' => $Elements,
             );
 
-            $Block['element']['elements'][1]['elements'] []= $Element;
+            $Block['element']['elements'][1]['elements'] [] = $Element;
 
             return $Block;
         }
@@ -1133,7 +1133,7 @@ 
 
                 $Inline = $this->{"inline$inlineType"}($Excerpt);
 
-                if ( ! isset($Inline))
+                if (!isset($Inline))
                 {
                     continue;
                 }
@@ -1147,7 +1147,7 @@ 
 
                 # sets a default inline position
 
-                if ( ! isset($Inline['position']))
+                if (!isset($Inline['position']))
                 {
                     $Inline['position'] = $markerPosition;
                 }
@@ -1191,7 +1191,7 @@ 
 
         foreach ($Elements as &$Element)
         {
-            if ( ! isset($Element['autobreak']))
+            if (!isset($Element['autobreak']))
             {
                 $Element['autobreak'] = false;
             }
@@ -1247,14 +1247,14 @@ 
         $hostnameLabel = '[a-zA-Z0-9](?:[a-zA-Z0-9-]{0,61}[a-zA-Z0-9])?';
 
         $commonMarkEmail = '[a-zA-Z0-9.!#$%&\'*+\/=?^_`{|}~-]++@'
-            . $hostnameLabel . '(?:\.' . $hostnameLabel . ')*';
+            . $hostnameLabel.'(?:\.'.$hostnameLabel.')*';
 
         if (strpos($Excerpt['text'], '>') !== false
             and preg_match("/^<((mailto:)?$commonMarkEmail)>/i", $Excerpt['text'], $matches)
-        ){
+        ) {
             $url = $matches[1];
 
-            if ( ! isset($matches[2]))
+            if (!isset($matches[2]))
             {
                 $url = "mailto:$url";
             }
@@ -1274,7 +1274,7 @@ 
 
     protected function inlineEmphasis($Excerpt)
     {
-        if ( ! isset($Excerpt['text'][1]))
+        if (!isset($Excerpt['text'][1]))
         {
             return;
         }
@@ -1320,12 +1320,12 @@ 
 
     protected function inlineImage($Excerpt)
     {
-        if ( ! isset($Excerpt['text'][1]) or $Excerpt['text'][1] !== '[')
+        if (!isset($Excerpt['text'][1]) or $Excerpt['text'][1] !== '[')
         {
             return;
         }
 
-        $Excerpt['text']= substr($Excerpt['text'], 1);
+        $Excerpt['text'] = substr($Excerpt['text'], 1);
 
         $Link = $this->inlineLink($Excerpt);
 
@@ -1411,7 +1411,7 @@ 
                 $definition = strtolower($Element['handler']['argument']);
             }
 
-            if ( ! isset($this->DefinitionData['Reference'][$definition]))
+            if (!isset($this->DefinitionData['Reference'][$definition]))
             {
                 return;
             }
@@ -1466,7 +1466,7 @@ 
             and preg_match('/^&(#?+[0-9a-zA-Z]++);/', $Excerpt['text'], $matches)
         ) {
             return array(
-                'element' => array('rawHtml' => '&' . $matches[1] . ';'),
+                'element' => array('rawHtml' => '&'.$matches[1].';'),
                 'extent' => strlen($matches[0]),
             );
         }
@@ -1476,7 +1476,7 @@ 
 
     protected function inlineStrikethrough($Excerpt)
     {
-        if ( ! isset($Excerpt['text'][1]))
+        if (!isset($Excerpt['text'][1]))
         {
             return;
         }
@@ -1499,7 +1499,7 @@ 
 
     protected function inlineUrl($Excerpt)
     {
-        if ($this->urlsLinked !== true or ! isset($Excerpt['text'][2]) or $Excerpt['text'][2] !== '/')
+        if ($this->urlsLinked !== true or !isset($Excerpt['text'][2]) or $Excerpt['text'][2] !== '/')
         {
             return;
         }
@@ -1670,7 +1670,7 @@ 
 
         if ($hasName)
         {
-            $markup .= '<' . $Element['name'];
+            $markup .= '<'.$Element['name'];
 
             if (isset($Element['attributes']))
             {
@@ -1728,7 +1728,7 @@ 
                 }
             }
 
-            $markup .= $hasName ? '</' . $Element['name'] . '>' : '';
+            $markup .= $hasName ? '</'.$Element['name'].'>' : '';
         }
         elseif ($hasName)
         {
@@ -1757,7 +1757,7 @@ 
             // (autobreak === false) covers both sides of an element
             $autoBreak = !$autoBreak ? $autoBreak : $autoBreakNext;
 
-            $markup .= ($autoBreak ? "\n" : '') . $this->element($Element);
+            $markup .= ($autoBreak ? "\n" : '').$this->element($Element);
             $autoBreak = $autoBreakNext;
         }
 
@@ -1772,7 +1772,7 @@ 
     {
         $Elements = $this->linesElements($lines);
 
-        if ( ! in_array('', $lines)
+        if (!in_array('', $lines)
             and isset($Elements[0]) and isset($Elements[0]['name'])
             and $Elements[0]['name'] === 'p'
         ) {
@@ -1829,12 +1829,12 @@ 
     protected function sanitiseElement(array $Element)
     {
         static $goodAttribute = '/^[a-zA-Z0-9][a-zA-Z0-9-_]*+$/';
-        static $safeUrlNameToAtt  = array(
+        static $safeUrlNameToAtt = array(
             'a'   => 'href',
             'img' => 'src',
         );
 
-        if ( ! isset($Element['name']))
+        if (!isset($Element['name']))
         {
             unset($Element['attributes']);
             return $Element;
@@ -1845,12 +1845,12 @@ 
             $Element = $this->filterUnsafeUrlInAttribute($Element, $safeUrlNameToAtt[$Element['name']]);
         }
 
-        if ( ! empty($Element['attributes']))
+        if (!empty($Element['attributes']))
         {
             foreach ($Element['attributes'] as $att => $val)
             {
                 # filter out badly parsed attribute
-                if ( ! preg_match($goodAttribute, $att))
+                if (!preg_match($goodAttribute, $att))
                 {
                     unset($Element['attributes'][$att]);
                 }
@@ -1951,8 +1951,8 @@ 
     protected $textLevelElements = array(
         'a', 'br', 'bdo', 'abbr', 'blink', 'nextid', 'acronym', 'basefont',
         'b', 'em', 'big', 'cite', 'small', 'spacer', 'listing',
-        'i', 'rp', 'del', 'code',          'strike', 'marquee',
-        'q', 'rt', 'ins', 'font',          'strong',
+        'i', 'rp', 'del', 'code', 'strike', 'marquee',
+        'q', 'rt', 'ins', 'font', 'strong',
         's', 'tt', 'kbd', 'mark',
         'u', 'xm', 'sub', 'nobr',
                    'sup', 'ruby',

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)) {

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;

include/functions.php

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