ggoffy / wggithub

class/Common/DirectoryChecker.php

Spacing +16 added lines, -16 removed lines

@@ -24,7 +24,7 @@ 
 use XoopsModules\Wggithub;
 
 
-require_once \dirname(__DIR__, 4) . '/mainfile.php';
+require_once \dirname(__DIR__, 4).'/mainfile.php';
 $moduleDirName      = \basename(\dirname(__DIR__, 2));
 $moduleDirNameUpper = \mb_strtoupper($moduleDirName);
 \xoops_loadLanguage('directorychecker', $moduleDirName);
@@ -56,38 +56,38 @@ 
         $moduleDirNameUpper = \mb_strtoupper($moduleDirName);
         if (!@\is_dir($path)) {
             $path_status = "<img src='$pathIcon16/0.png' >";
-            $path_status .= "$path (" . \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_NOTAVAILABLE') . ') ';
-            $path_status .= "<form action='" . $_SERVER['SCRIPT_NAME'] . "' method='post'>";
+            $path_status .= "$path (".\constant('CO_'.$moduleDirNameUpper.'_'.'DC_NOTAVAILABLE').') ';
+            $path_status .= "<form action='".$_SERVER['SCRIPT_NAME']."' method='post'>";
             $path_status .= "<input type='hidden' name='op' value='createdir'>";
             $path_status .= "<input type='hidden' name='path' value='$path'>";
             $path_status .= "<input type='hidden' name='redirect' value='$redirectFile'>";
-            $path_status .= "<button class='submit' onClick='this.form.submit();'>" . \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_CREATETHEDIR') . '</button>';
+            $path_status .= "<button class='submit' onClick='this.form.submit();'>".\constant('CO_'.$moduleDirNameUpper.'_'.'DC_CREATETHEDIR').'</button>';
             $path_status .= '</form>';
         } elseif (@\is_writable($path)) {
             $path_status = "<img src='$pathIcon16/1.png' >";
-            $path_status .= "$path (" . \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_AVAILABLE') . ') ';
+            $path_status .= "$path (".\constant('CO_'.$moduleDirNameUpper.'_'.'DC_AVAILABLE').') ';
             $currentMode = \mb_substr(\decoct(\fileperms($path)), 2);
             if ($currentMode != \decoct($mode)) {
                 $path_status = "<img src='$pathIcon16/0.png' >";
-                $path_status .= $path . \sprintf(\constant('CO_' . $moduleDirNameUpper . '_' . 'DC_NOTWRITABLE'), \decoct($mode), $currentMode);
-                $path_status .= "<form action='" . $_SERVER['SCRIPT_NAME'] . "' method='post'>";
+                $path_status .= $path.\sprintf(\constant('CO_'.$moduleDirNameUpper.'_'.'DC_NOTWRITABLE'), \decoct($mode), $currentMode);
+                $path_status .= "<form action='".$_SERVER['SCRIPT_NAME']."' method='post'>";
                 $path_status .= "<input type='hidden' name='op' value='setdirperm'>";
                 $path_status .= "<input type='hidden' name='mode' value='$mode'>";
                 $path_status .= "<input type='hidden' name='path' value='$path'>";
                 $path_status .= "<input type='hidden' name='redirect' value='$redirectFile'>";
-                $path_status .= "<button class='submit' onClick='this.form.submit();'>" . \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_SETMPERM') . '</button>';
+                $path_status .= "<button class='submit' onClick='this.form.submit();'>".\constant('CO_'.$moduleDirNameUpper.'_'.'DC_SETMPERM').'</button>';
                 $path_status .= '</form>';
             }
         } else {
             $currentMode = \mb_substr(\decoct(\fileperms($path)), 2);
             $path_status = "<img src='$pathIcon16/0.png' >";
-            $path_status .= $path . \sprintf(\constant('CO_' . $moduleDirNameUpper . '_' . 'DC_NOTWRITABLE'), \decoct($mode), $currentMode);
-            $path_status .= "<form action='" . $_SERVER['SCRIPT_NAME'] . "' method='post'>";
+            $path_status .= $path.\sprintf(\constant('CO_'.$moduleDirNameUpper.'_'.'DC_NOTWRITABLE'), \decoct($mode), $currentMode);
+            $path_status .= "<form action='".$_SERVER['SCRIPT_NAME']."' method='post'>";
             $path_status .= "<input type='hidden' name='op' value='setdirperm'>";
             $path_status .= "<input type='hidden' name='mode' value='$mode'>";
             $path_status .= "<input type='hidden' name='path' value='$path'>";
             $path_status .= "<input type='hidden' name='redirect' value='$redirectFile'>";
-            $path_status .= "<button class='submit' onClick='this.form.submit();'>" . \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_SETMPERM') . '</button>';
+            $path_status .= "<button class='submit' onClick='this.form.submit();'>".\constant('CO_'.$moduleDirNameUpper.'_'.'DC_SETMPERM').'</button>';
             $path_status .= '</form>';
         }
 
@@ -118,7 +118,7 @@ 
     {
         $target = \str_replace('..', '', $target);
 
-        return @\chmod($target, (int)$mode);
+        return @\chmod($target, (int) $mode);
     }
 
     /**
@@ -141,8 +141,8 @@ 
         if (\Xmf\Request::hasVar('redirect', 'POST')) {
             $redirect = $_POST['redirect'];
         }
-        $msg = DirectoryChecker::createDirectory($path) ? \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_DIRCREATED') : \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_DIRNOTCREATED');
-        \redirect_header($redirect, 2, $msg . ': ' . $path);
+        $msg = DirectoryChecker::createDirectory($path) ? \constant('CO_'.$moduleDirNameUpper.'_'.'DC_DIRCREATED') : \constant('CO_'.$moduleDirNameUpper.'_'.'DC_DIRNOTCREATED');
+        \redirect_header($redirect, 2, $msg.': '.$path);
         break;
     case 'setdirperm':
         if (\Xmf\Request::hasVar('path', 'POST')) {
@@ -154,7 +154,7 @@ 
         if (\Xmf\Request::hasVar('mode', 'POST')) {
             $mode = $_POST['mode'];
         }
-        $msg = DirectoryChecker::setDirectoryPermissions($path, $mode) ? \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_PERMSET') : \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_PERMNOTSET');
-        \redirect_header($redirect, 2, $msg . ': ' . $path);
+        $msg = DirectoryChecker::setDirectoryPermissions($path, $mode) ? \constant('CO_'.$moduleDirNameUpper.'_'.'DC_PERMSET') : \constant('CO_'.$moduleDirNameUpper.'_'.'DC_PERMNOTSET');
+        \redirect_header($redirect, 2, $msg.': '.$path);
         break;
 }

class/Common/MigrateHelper.php

Spacing +23 added lines, -23 removed lines

@@ -84,7 +84,7 @@ 
             $line = \trim($value);
             if ('CREATE TABLE' === \mb_strtoupper(\substr($line, 0, 12))) {
                 // start table definition
-                $tableName = $this->getTableName ($line);
+                $tableName = $this->getTableName($line);
                 $tables[$tableName] = [];
                 $tables[$tableName]['options'] = '';
                 $tables[$tableName]['columns'] = [];
@@ -121,22 +121,22 @@ 
             $schema[] = "{$tkey}:\n";
             foreach ($table as $lkey => $line) {
                 if ('keys' == $lkey) {
-                    $schema[] = $level1 . "keys:\n";
+                    $schema[] = $level1."keys:\n";
                     foreach ($line as $kkey => $kvalue) {
                         foreach ($kvalue as $kkey2 => $kvalue2) {
-                            $schema[] = $level2 . $kkey2 . ":\n";
-                            $schema[] = $level3 . 'columns: ' . $kvalue2['columns'] . "\n";
-                            $schema[] = $level3 . 'unique: ' . $kvalue2['unique'] . "\n";
+                            $schema[] = $level2.$kkey2.":\n";
+                            $schema[] = $level3.'columns: '.$kvalue2['columns']."\n";
+                            $schema[] = $level3.'unique: '.$kvalue2['unique']."\n";
                         }
                     }
                 } elseif ('options' == $lkey) {
-                    $schema[] = $level1 . 'options: ' . $line . "\n";
+                    $schema[] = $level1.'options: '.$line."\n";
                 } else {
-                    $schema[] = $level1 . 'columns: ' . "\n";
+                    $schema[] = $level1.'columns: '."\n";
                     foreach ($line as $kkey => $kvalue) {
-                        $schema[] = $level2 . '-' . "\n";
+                        $schema[] = $level2.'-'."\n";
                         foreach ($kvalue as $kkey2 => $kvalue2) {
-                            $schema[] = $level3 . $kkey2 . ": " . $kvalue2 . "\n";
+                            $schema[] = $level3.$kkey2.": ".$kvalue2."\n";
                         }
                     }
                 }
@@ -164,10 +164,10 @@ 
      * @param  string $line
      * @return string|bool
      */
-    private function getTableName (string $line)
+    private function getTableName(string $line)
     {
 
-        $arrLine = \explode( '`', $line);
+        $arrLine = \explode('`', $line);
         if (\count($arrLine) > 0) {
             return $arrLine[1];
         }
@@ -182,12 +182,12 @@ 
      * @param string $line
      * @return array|bool
      */
-    private function getColumns (string $line)
+    private function getColumns(string $line)
     {
 
         $columns = [];
 
-        $arrCol = \explode( ' ', \trim($line));
+        $arrCol = \explode(' ', \trim($line));
         if (\count($arrCol) > 0) {
             $name = \str_replace(['`'], '', $arrCol[0]);
         } else {
@@ -198,11 +198,11 @@ 
         $columns['name'] = $name;
         // update quotes
         if (\strpos($attributes, "''") > 0) {
-            $attributes = \trim(\str_replace("''", "''''''''" , $attributes));
+            $attributes = \trim(\str_replace("''", "''''''''", $attributes));
         } elseif (\strpos($attributes, "'") > 0) {
-            $attributes = \trim(\str_replace("'", "''" , $attributes));
+            $attributes = \trim(\str_replace("'", "''", $attributes));
         }
-        $columns['attributes'] = "' " . $attributes . " '";
+        $columns['attributes'] = "' ".$attributes." '";
 
         return $columns;
 
@@ -214,12 +214,12 @@ 
      * @param string $line
      * @return string
      */
-    private function getOptions (string $line): string
+    private function getOptions(string $line): string
     {
 
         $options = \str_replace([')', ';'], '', $line);
         $options = \trim($options);
-        $options = "'" . $options . "'";
+        $options = "'".$options."'";
 
         return $options;
 
@@ -231,7 +231,7 @@ 
      * @param string $line
      * @return array|bool
      */
-    private function getKey (string $line)
+    private function getKey(string $line)
     {
 
         $key = [];
@@ -254,12 +254,12 @@ 
                 if ('' == $name) {
                     $name = $columns;
                 }
-                if (\strpos($name,' ') > 0) {
-                    $name = "'" . $name . "'";
+                if (\strpos($name, ' ') > 0) {
+                    $name = "'".$name."'";
                 }
                 $key[$name] = [];
-                if (\strpos($columns,' ') > 0) {
-                    $columns = "'" . $columns . "'";
+                if (\strpos($columns, ' ') > 0) {
+                    $columns = "'".$columns."'";
                 }
                 $key[$name]['columns'] = $columns;
                 $key[$name]['unique'] = $unique;

xoops_version.php

Spacing +4 added lines, -4 removed lines

@@ -39,10 +39,10 @@ 
     'license_url'         => 'http://www.gnu.org/licenses/gpl-3.0.en.html',

     'help'                => 'page=help',

     'release_info'        => 'release_info',

-    'release_file'        => \XOOPS_URL . '/modules/wggithub/docs/release_info file',

+    'release_file'        => \XOOPS_URL.'/modules/wggithub/docs/release_info file',

     'release_date'        => '2021/03/31',

     'manual'              => 'link to manual file',

-    'manual_file'         => \XOOPS_URL . '/modules/wggithub/docs/install.txt',

+    'manual_file'         => \XOOPS_URL.'/modules/wggithub/docs/install.txt',

     'min_php'             => '7.4',

     'min_xoops'           => '2.5.11 Beta1',

     'min_admin'           => '1.2',

@@ -299,8 +299,8 @@ 
 // Make Sample button visible?

 $modversion['config'][] = [

     'name'        => 'displaySampleButton',

-    'title'       => 'CO_' . $moduleDirNameUpper . '_' . 'SHOW_SAMPLE_BUTTON',

-    'description' => 'CO_' . $moduleDirNameUpper . '_' . 'SHOW_SAMPLE_BUTTON_DESC',

+    'title'       => 'CO_'.$moduleDirNameUpper.'_'.'SHOW_SAMPLE_BUTTON',

+    'description' => 'CO_'.$moduleDirNameUpper.'_'.'SHOW_SAMPLE_BUTTON_DESC',

     'formtype'    => 'yesno',

     'valuetype'   => 'int',

     'default'     => 1,

footer.php

Spacing +1 added lines, -1 removed lines

@@ -32,4 +32,4 @@
 $GLOBALS['xoopsTpl']->assign('admin', \WGGITHUB_ADMIN);

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

 // 

-include_once \XOOPS_ROOT_PATH . '/footer.php';

+include_once \XOOPS_ROOT_PATH.'/footer.php';

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;

     }

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 . '/wedega_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.'/wedega_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/search.inc.php

Spacing +6 added lines, -6 removed lines

@@ -59,15 +59,15 @@ 
     }

     if ($elementCount > 0) {

         $crKeywords = new \CriteriaCompo();

-        for ($i = 0; $i  <  $elementCount; $i++) {

-            $crKeywords->add(new \Criteria('repo_name', '%' . $queryarray[$i] . '%', 'LIKE'), 'OR');

+        for ($i = 0; $i < $elementCount; $i++) {

+            $crKeywords->add(new \Criteria('repo_name', '%'.$queryarray[$i].'%', 'LIKE'), 'OR');

         }

     }

     // search user(s)

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

@@ -79,7 +79,7 @@ 
     if (isset($crUser)) {

         $crSearch->add($crUser, 'AND');

     }

-    if (0 === (int)$helper->getConfig('autoapproved')) {

+    if (0 === (int) $helper->getConfig('autoapproved')) {

         $crSearch->add(new \Criteria('repo_approved', 1));

     }

     $crSearch->add(new \Criteria('repo_status', Constants::STATUS_UPTODATE));

@@ -91,8 +91,8 @@ 
     foreach (\array_keys($repositoriesAll) as $i) {

         $ret[] = [

             'image'  => 'assets/icons/16/github.png',

-            'link'   => 'index.php?op=show&amp;dir_id=' . $directories[$repositoriesAll[$i]->getVar('repo_user')] . '&amp;repo_id=' . $i,

-            'title'  => $repositoriesAll[$i]->getVar('repo_name') . ' (' . $repositoriesAll[$i]->getVar('repo_user') . ')',

+            'link'   => 'index.php?op=show&amp;dir_id='.$directories[$repositoriesAll[$i]->getVar('repo_user')].'&amp;repo_id='.$i,

+            'title'  => $repositoriesAll[$i]->getVar('repo_name').' ('.$repositoriesAll[$i]->getVar('repo_user').')',

             'time'   => $repositoriesAll[$i]->getVar('repo_datecreated')

         ];

     }

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;

@@ -206,6 +206,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([$regular_expression, '`[-]+`'], '-', $url);

-    $url = ($url == '') ? $type : strtolower(    rim($url, '-'));

+    $url = ($url == '') ? $type : strtolower(rim($url, '-'));

     return $url;

 }
\ No newline at end of file

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 = [

             'element' => [

-                'name' => 'h' . $level,

+                'name' => 'h'.$level,

                 'handler' => [

                     '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'] []= [

+        $Block['element']['elements'] [] = [

             'name' => 'thead',

         ];

 

-        $Block['element']['elements'] []= [

+        $Block['element']['elements'] [] = [

             'name' => 'tbody',

             'elements' => [],

         ];

 

-        $Block['element']['elements'][0]['elements'] []= [

+        $Block['element']['elements'][0]['elements'] [] = [

             'name' => 'tr',

             'elements' => $HeaderElements,

         ];

@@ -1021,11 +1021,11 @@ 
                 if (isset($Block['alignments'][$index]))

                 {

                     $Element['attributes'] = [

-                        'style' => 'text-align: ' . $Block['alignments'][$index] . ';',

+                        'style' => 'text-align: '.$Block['alignments'][$index].';',

                     ];

                 }

 

-                $Elements []= $Element;

+                $Elements [] = $Element;

             }

 

             $Element = [

@@ -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 [

-                'element' => ['rawHtml' => '&' . $matches[1] . ';'],

+                'element' => ['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  = [

+        static $safeUrlNameToAtt = [

             '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 = [

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