mambax7 / planet

action.blog.php

Spacing +30 added lines, -30 removed lines

@@ -26,11 +26,11 @@ 
 // ------------------------------------------------------------------------ //
 use Xmf\Request;
 
-include __DIR__ . '/header.php';
+include __DIR__.'/header.php';
 
-$op      = Request::getString('op', Request::getString('op', '', 'POST'), 'GET');//!empty($_POST['op']) ? $_POST['op'] : (!empty($_GET['op']) ? $_GET['op'] : '');
-$blog_id = Request::getArray('blog', Request::getArray('blog', [], 'POST'), 'GET');//!empty($_POST['blog']) ? $_POST['blog'] : (!empty($_GET['blog']) ? $_GET['blog'] : 0);
-$blog_id = is_array($blog_id) ? array_map('intval', $blog_id) : (int)$blog_id;
+$op      = Request::getString('op', Request::getString('op', '', 'POST'), 'GET'); //!empty($_POST['op']) ? $_POST['op'] : (!empty($_GET['op']) ? $_GET['op'] : '');
+$blog_id = Request::getArray('blog', Request::getArray('blog', [], 'POST'), 'GET'); //!empty($_POST['blog']) ? $_POST['blog'] : (!empty($_GET['blog']) ? $_GET['blog'] : 0);
+$blog_id = is_array($blog_id) ? array_map('intval', $blog_id) : (int) $blog_id;
 
 if (empty($xoopsModuleConfig['newblog_submit']) && (!is_object($xoopsUser) || !$xoopsUser->isAdmin())) {
     redirect_header('index.php', 2, _NOPERM);
@@ -41,10 +41,10 @@ 
 }
 
 if ($op === 'save' && !$GLOBALS['xoopsSecurity']->check()) {
-    redirect_header('javascript:history.go(-1);', 1, planet_constant('MD_INVALID') . ': security check failed');
+    redirect_header('javascript:history.go(-1);', 1, planet_constant('MD_INVALID').': security check failed');
 }
-include XOOPS_ROOT_PATH . '/header.php';
-include XOOPS_ROOT_PATH . '/modules/' . $xoopsModule->getVar('dirname') . '/include/vars.php';
+include XOOPS_ROOT_PATH.'/header.php';
+include XOOPS_ROOT_PATH.'/modules/'.$xoopsModule->getVar('dirname').'/include/vars.php';
 
 $blogHandler     = xoops_getModuleHandler('blog', $GLOBALS['moddirname']);
 $categoryHandler = xoops_getModuleHandler('category', $GLOBALS['moddirname']);
@@ -56,9 +56,9 @@ 
         if ($blog_id) {
             $blog_obj = $blogHandler->get($blog_id);
             if ($xoopsUser->isAdmin()) {
-                $blog_obj->setVar('blog_status', Request::getInt('blog_status', 0, 'POST'));// @$_POST['blog_status']);
+                $blog_obj->setVar('blog_status', Request::getInt('blog_status', 0, 'POST')); // @$_POST['blog_status']);
             }
-        } else {
+        }else {
             if ($blog_exists = $blogHandler->getCount(new Criteria('blog_feed', $myts->addSlashes(trim(Request::getText('blog_feed', '', 'POST')))))  //$_POST['blog_feed']))))
             ) {
                 redirect_header('index.php', 2, planet_constant('MD_BLOGEXISTS'));
@@ -71,7 +71,7 @@ 
                 case 2:
                     if (!is_object($xoopsUser)) {
                         $status = 0;
-                    } else {
+                    }else {
                         $status = 1;
                     }
                     break;
@@ -83,7 +83,7 @@ 
                 default:
                     if (!is_object($xoopsUser) || !$xoopsUser->isAdmin()) {
                         $status = 0;
-                    } else {
+                    }else {
                         $status = 1;
                     }
                     break;
@@ -92,14 +92,14 @@ 
             $blog_obj->setVar('blog_status', $status);
         }
 
-        $blog_obj->setVar('blog_title', Request::getString('blog_title', '', 'POST'));//$_POST['blog_title']);
-        $blog_obj->setVar('blog_desc', Request::getString('blog_desc', '', 'POST'));//$_POST['blog_desc']);
-        $blog_obj->setVar('blog_image', Request::getString('blog_image', '', 'POST'));//$_POST['blog_image']);
-        $blog_obj->setVar('blog_feed', Request::getText('blog_feed', '', 'POST'));//$_POST['blog_feed']);
-        $blog_obj->setVar('blog_link', Request::getString('blog_link', '', 'POST'));//$_POST['blog_link']);
-        $blog_obj->setVar('blog_language', Request::getString('blog_language', '', 'POST'));//$_POST['blog_language']);
-        $blog_obj->setVar('blog_charset', Request::getString('blog_charset', '', 'POST'));//$_POST['blog_charset']);
-        $blog_obj->setVar('blog_trackback', Request::getString('blog_trackback', '', 'POST'));//$_POST['blog_trackback']);
+        $blog_obj->setVar('blog_title', Request::getString('blog_title', '', 'POST')); //$_POST['blog_title']);
+        $blog_obj->setVar('blog_desc', Request::getString('blog_desc', '', 'POST')); //$_POST['blog_desc']);
+        $blog_obj->setVar('blog_image', Request::getString('blog_image', '', 'POST')); //$_POST['blog_image']);
+        $blog_obj->setVar('blog_feed', Request::getText('blog_feed', '', 'POST')); //$_POST['blog_feed']);
+        $blog_obj->setVar('blog_link', Request::getString('blog_link', '', 'POST')); //$_POST['blog_link']);
+        $blog_obj->setVar('blog_language', Request::getString('blog_language', '', 'POST')); //$_POST['blog_language']);
+        $blog_obj->setVar('blog_charset', Request::getString('blog_charset', '', 'POST')); //$_POST['blog_charset']);
+        $blog_obj->setVar('blog_trackback', Request::getString('blog_trackback', '', 'POST')); //$_POST['blog_trackback']);
         if ($blog_obj->isNew()) {
             $blog_obj->setVar('blog_submitter', is_object($xoopsUser) ? $xoopsUser->getVar('uid') : PlanetUtility::planetGetIP(true));
         }
@@ -110,10 +110,10 @@ 
             if (in_array(0, $_POST['categories'])) {
                 $_POST['categories'] = [];
             }
-            $blogHandler->setCategories($blog_id, Request::getString('andor', '', 'POST'));//$_POST['categories']);
+            $blogHandler->setCategories($blog_id, Request::getString('andor', '', 'POST')); //$_POST['categories']);
         }
         $message = planet_constant('MD_DBUPDATED');
-        redirect_header('index.php' . URL_DELIMITER . 'b' . $blog_id, 2, $message);
+        redirect_header('index.php'.URL_DELIMITER.'b'.$blog_id, 2, $message);
 
     /* edit a single blog */
     // no break
@@ -122,14 +122,14 @@ 
         if (!empty(Request::getString('fetch', '', 'POST'))) {
             $blog_obj = $blogHandler->fetch(Request::getText('blog_feed', '', 'POST'));
             $blog_obj->setVar('blog_id', $blog_id);
-        } else {
+        }else {
             $blog_obj = $blogHandler->get($blog_id);
         }
-        $categories = Request::getArray('categories', [], 'POST');//isset($_POST['categories']) ? $_POST['categories'] : array();
+        $categories = Request::getArray('categories', [], 'POST'); //isset($_POST['categories']) ? $_POST['categories'] : array();
         if (in_array('-1', $categories)) {
             $categories = [];
         }
-        if (empty($categories) && $blog_id > 0) {
+        if (empty($categories) && $blog_id>0) {
             $crit       = new Criteria('bc.blog_id', $blog_id);
             $categories = array_keys($categoryHandler->getByBlog($crit));
         }
@@ -137,23 +137,23 @@ 
             $categories = [0 => _NONE];
         }
 
-        echo "<fieldset><legend style='font-weight: bold; color: #900;'>" . _EDIT . '</legend>';
+        echo "<fieldset><legend style='font-weight: bold; color: #900;'>"._EDIT.'</legend>';
         echo '<br>';
         if (empty($blog_id) && $blog_obj->getVar('blog_feed')) {
             $criteria  = new Criteria('blog_feed', $blog_obj->getVar('blog_feed'));
             $blogs_obj = $blogHandler->getList($criteria);
-            if (count($blogs_obj) > 0) {
-                echo "<div class=\"errorMsg\">" . planet_constant('MD_BLOGEXISTS');
+            if (count($blogs_obj)>0) {
+                echo "<div class=\"errorMsg\">".planet_constant('MD_BLOGEXISTS');
                 foreach (array_keys($blogs_obj) as $bid) {
-                    echo "<br><a href=\"" . XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/index.php' . URL_DELIMITER . 'b' . $bid . "\" target=\"_blank\">" . $blogs_obj[$bid] . '</a>';
+                    echo "<br><a href=\"".XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/index.php'.URL_DELIMITER.'b'.$bid."\" target=\"_blank\">".$blogs_obj[$bid].'</a>';
                 }
                 echo '</div>';
                 unset($blogs_obj, $criteria);
             }
         }
-        include XOOPS_ROOT_PATH . '/modules/' . $GLOBALS['moddirname'] . '/include/form.blog.php';
+        include XOOPS_ROOT_PATH.'/modules/'.$GLOBALS['moddirname'].'/include/form.blog.php';
         echo '</fieldset>';
         break;
 }
 
-include XOOPS_ROOT_PATH . '/footer.php';
+include XOOPS_ROOT_PATH.'/footer.php';

xoops_version.php

Spacing +61 added lines, -61 removed lines

@@ -27,9 +27,9 @@ 
 
 // defined('XOOPS_ROOT_PATH') || exit('Restricted access.');
 
-include __DIR__ . '/include/vars.php';
+include __DIR__.'/include/vars.php';
 
-$modversion                 = [
+$modversion = [
     'name'          => planet_constant('MI_NAME'),
     'version'       => 2.10,
     'module_status' => 'Final',
@@ -39,11 +39,11 @@ 
     'credits'       => 'XOOPS Project',
     'image'         => 'assets/images/logoModule.png',
     'dirname'       => $GLOBALS['moddirname'],
-    'help'          => XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/readme.html'
+    'help'          => XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/readme.html'
 ];
 $modversion['help']         = 'page=help';
 $modversion['license']      = 'GNU see LICENSE';
-$modversion['license_file'] = XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/gpl.txt';
+$modversion['license_file'] = XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/gpl.txt';
 $modversion['author_word']  = '';
 $modversion['module_team']  = '';
 
@@ -62,12 +62,12 @@ 
 // database tables
 $modversion['sqlfile']['mysql'] = 'sql/mysql.sql';
 $modversion['tables']           = [
-    $GLOBALS['MOD_DB_PREFIX'] . '_category',
-    $GLOBALS['MOD_DB_PREFIX'] . '_article',
-    $GLOBALS['MOD_DB_PREFIX'] . '_blog',
-    $GLOBALS['MOD_DB_PREFIX'] . '_blogcat',
-    $GLOBALS['MOD_DB_PREFIX'] . '_bookmark',
-    $GLOBALS['MOD_DB_PREFIX'] . '_rate'
+    $GLOBALS['MOD_DB_PREFIX'].'_category',
+    $GLOBALS['MOD_DB_PREFIX'].'_article',
+    $GLOBALS['MOD_DB_PREFIX'].'_blog',
+    $GLOBALS['MOD_DB_PREFIX'].'_blogcat',
+    $GLOBALS['MOD_DB_PREFIX'].'_bookmark',
+    $GLOBALS['MOD_DB_PREFIX'].'_rate'
 ];
 
 // Admin things
@@ -93,11 +93,11 @@ 
 /**
  * Templates
  */
-$modversion['templates'][] = ['file' => $GLOBALS['VAR_PREFIX'] . '_index.tpl', 'description' => ''];
-$modversion['templates'][] = ['file' => $GLOBALS['VAR_PREFIX'] . '_article.tpl', 'description' => ''];
-$modversion['templates'][] = ['file' => $GLOBALS['VAR_PREFIX'] . '_archive.tpl', 'description' => ''];
-$modversion['templates'][] = ['file' => $GLOBALS['VAR_PREFIX'] . '_blogs.tpl', 'description' => ''];
-$modversion['templates'][] = ['file' => $GLOBALS['VAR_PREFIX'] . '_search.tpl', 'description' => ''];
+$modversion['templates'][] = ['file' => $GLOBALS['VAR_PREFIX'].'_index.tpl', 'description' => ''];
+$modversion['templates'][] = ['file' => $GLOBALS['VAR_PREFIX'].'_article.tpl', 'description' => ''];
+$modversion['templates'][] = ['file' => $GLOBALS['VAR_PREFIX'].'_archive.tpl', 'description' => ''];
+$modversion['templates'][] = ['file' => $GLOBALS['VAR_PREFIX'].'_blogs.tpl', 'description' => ''];
+$modversion['templates'][] = ['file' => $GLOBALS['VAR_PREFIX'].'_search.tpl', 'description' => ''];
 
 //module css
 $modversion['css'] = 'templates/style.css';
@@ -109,31 +109,31 @@ 
 $modversion['blocks'][$i]['file']        = 'block.php';
 $modversion['blocks'][$i]['name']        = planet_constant('MI_ARTICLE');
 $modversion['blocks'][$i]['description'] = planet_constant('MI_ARTICLE_DESC');
-$modversion['blocks'][$i]['show_func']   = $GLOBALS['VAR_PREFIX'] . '_article_show';
+$modversion['blocks'][$i]['show_func']   = $GLOBALS['VAR_PREFIX'].'_article_show';
 $modversion['blocks'][$i]['options']     = 'time|10|0|0'; // type|MaxItems|TitleLength|SummaryLength
-$modversion['blocks'][$i]['edit_func']   = $GLOBALS['VAR_PREFIX'] . '_article_edit';
-$modversion['blocks'][$i]['template']    = $GLOBALS['VAR_PREFIX'] . '_block_article.tpl';
+$modversion['blocks'][$i]['edit_func']   = $GLOBALS['VAR_PREFIX'].'_article_edit';
+$modversion['blocks'][$i]['template']    = $GLOBALS['VAR_PREFIX'].'_block_article.tpl';
 
 ++$i;
 $modversion['blocks'][$i]['file']        = 'block.php';
 $modversion['blocks'][$i]['name']        = planet_constant('MI_CATEGORY');
 $modversion['blocks'][$i]['description'] = planet_constant('MI_CATEGORY_DESC');
-$modversion['blocks'][$i]['show_func']   = $GLOBALS['VAR_PREFIX'] . '_category_show';
-$modversion['blocks'][$i]['template']    = $GLOBALS['VAR_PREFIX'] . '_block_category.tpl';
+$modversion['blocks'][$i]['show_func']   = $GLOBALS['VAR_PREFIX'].'_category_show';
+$modversion['blocks'][$i]['template']    = $GLOBALS['VAR_PREFIX'].'_block_category.tpl';
 
 ++$i;
 $modversion['blocks'][$i]['file']        = 'block.php';
 $modversion['blocks'][$i]['name']        = planet_constant('MI_BLOG');
 $modversion['blocks'][$i]['description'] = planet_constant('MI_BLOG_DESC');
-$modversion['blocks'][$i]['show_func']   = $GLOBALS['VAR_PREFIX'] . '_blog_show';
+$modversion['blocks'][$i]['show_func']   = $GLOBALS['VAR_PREFIX'].'_blog_show';
 $modversion['blocks'][$i]['options']     = 'feature|10|0|1'; // type|MaxItems|TitleLength|ShowDesc
-$modversion['blocks'][$i]['edit_func']   = $GLOBALS['VAR_PREFIX'] . '_blog_edit';
-$modversion['blocks'][$i]['template']    = $GLOBALS['VAR_PREFIX'] . '_block_blog.tpl';
+$modversion['blocks'][$i]['edit_func']   = $GLOBALS['VAR_PREFIX'].'_blog_edit';
+$modversion['blocks'][$i]['template']    = $GLOBALS['VAR_PREFIX'].'_block_blog.tpl';
 
 // Search
 $modversion['hasSearch']      = 1;
 $modversion['search']['file'] = 'include/search.inc.php';
-$modversion['search']['func'] = $GLOBALS['VAR_PREFIX'] . '_search';
+$modversion['search']['func'] = $GLOBALS['VAR_PREFIX'].'_search';
 
 // Comments
 $modversion['hasComments'] = 1;
@@ -143,15 +143,15 @@ 
 
 // Comment callback functions
 $modversion['comments']['callbackFile']        = 'include/comment.inc.php';
-$modversion['comments']['callback']['approve'] = $GLOBALS['VAR_PREFIX'] . '_com_approve';
-$modversion['comments']['callback']['update']  = $GLOBALS['VAR_PREFIX'] . '_com_update';
+$modversion['comments']['callback']['approve'] = $GLOBALS['VAR_PREFIX'].'_com_approve';
+$modversion['comments']['callback']['update']  = $GLOBALS['VAR_PREFIX'].'_com_update';
 
 // Configs
 // Config items
 $modversion['config'][] = [
     'name'        => 'do_debug',
-    'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_DODEBUG',
-    'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_DODEBUG_DESC',
+    'title'       => $GLOBALS['VAR_PREFIXU'].'_MI_DODEBUG',
+    'description' => $GLOBALS['VAR_PREFIXU'].'_MI_DODEBUG_DESC',
     'formtype'    => 'yesno',
     'valuetype'   => 'int',
     'default'     => 1
@@ -159,8 +159,8 @@ 
 
 $modversion['config'][] = [
     'name'        => 'do_urw',
-    'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_DOURLREWRITE',
-    'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_DOURLREWRITE_DESC',
+    'title'       => $GLOBALS['VAR_PREFIXU'].'_MI_DOURLREWRITE',
+    'description' => $GLOBALS['VAR_PREFIXU'].'_MI_DOURLREWRITE_DESC',
     'formtype'    => 'yesno',
     'valuetype'   => 'int',
     'default'     => in_array(php_sapi_name(), ['apache', 'apache2handler'])
@@ -168,8 +168,8 @@ 
 
 $modversion['config'][] = [
     'name'        => 'theme_set',
-    'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_THEMESET',
-    'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_THEMESET_DESC',
+    'title'       => $GLOBALS['VAR_PREFIXU'].'_MI_THEMESET',
+    'description' => $GLOBALS['VAR_PREFIXU'].'_MI_THEMESET_DESC',
     'formtype'    => 'select',
     'valuetype'   => 'text',
     'options'     => [_NONE => '0'],
@@ -178,23 +178,23 @@ 
 
 $modversion['config'][] = [
     'name'        => 'timeformat',
-    'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_TIMEFORMAT',
-    'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_TIMEFORMAT_DESC',
+    'title'       => $GLOBALS['VAR_PREFIXU'].'_MI_TIMEFORMAT',
+    'description' => $GLOBALS['VAR_PREFIXU'].'_MI_TIMEFORMAT_DESC',
     'formtype'    => 'select',
     'valuetype'   => 'text',
     'options'     => [
         _DATESTRING                                       => 'l',
         _MEDIUMDATESTRING                                 => 'm',
         _SHORTDATESTRING                                  => 's',
-        $GLOBALS['VAR_PREFIXU'] . '_MI_TIMEFORMAT_CUSTOM' => 'c'
+        $GLOBALS['VAR_PREFIXU'].'_MI_TIMEFORMAT_CUSTOM' => 'c'
     ],
     'default'     => 'c'
 ];
 
 $modversion['config'][] = [
     'name'        => 'articles_perpage',
-    'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_ARTICLESPERPAGE',
-    'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_ARTICLESPERPAGE_DESC',
+    'title'       => $GLOBALS['VAR_PREFIXU'].'_MI_ARTICLESPERPAGE',
+    'description' => $GLOBALS['VAR_PREFIXU'].'_MI_ARTICLESPERPAGE_DESC',
     'formtype'    => 'textbox',
     'valuetype'   => 'int',
     'default'     => 10
@@ -202,8 +202,8 @@ 
 
 $modversion['config'][] = [
     'name'        => 'list_perpage',
-    'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_LISTPERPAGE',
-    'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_LISTPERPAGE_DESC',
+    'title'       => $GLOBALS['VAR_PREFIXU'].'_MI_LISTPERPAGE',
+    'description' => $GLOBALS['VAR_PREFIXU'].'_MI_LISTPERPAGE_DESC',
     'formtype'    => 'textbox',
     'valuetype'   => 'int',
     'default'     => 20
@@ -211,8 +211,8 @@ 
 
 $modversion['config'][] = [
     'name'        => 'blogs_perupdate',
-    'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_BLOGSPERUPDATE',
-    'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_BLOGSPERUPDATE_DESC',
+    'title'       => $GLOBALS['VAR_PREFIXU'].'_MI_BLOGSPERUPDATE',
+    'description' => $GLOBALS['VAR_PREFIXU'].'_MI_BLOGSPERUPDATE_DESC',
     'formtype'    => 'textbox',
     'valuetype'   => 'int',
     'default'     => 10
@@ -220,8 +220,8 @@ 
 
 $modversion['config'][] = [
     'name'        => 'article_expire',
-    'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_ARTICLE_EXPIRE',
-    'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_ARTICLE_EXPIRE_DESC',
+    'title'       => $GLOBALS['VAR_PREFIXU'].'_MI_ARTICLE_EXPIRE',
+    'description' => $GLOBALS['VAR_PREFIXU'].'_MI_ARTICLE_EXPIRE_DESC',
     'formtype'    => 'textbox',
     'valuetype'   => 'int',
     'default'     => 30
@@ -229,8 +229,8 @@ 
 
 $modversion['config'][] = [
     'name'        => 'display_summary',
-    'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_DISPLAY_SUMMARY',
-    'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_DISPLAY_SUMMARY_DESC',
+    'title'       => $GLOBALS['VAR_PREFIXU'].'_MI_DISPLAY_SUMMARY',
+    'description' => $GLOBALS['VAR_PREFIXU'].'_MI_DISPLAY_SUMMARY_DESC',
     'formtype'    => 'textbox',
     'valuetype'   => 'int',
     'default'     => 0
@@ -238,8 +238,8 @@ 
 
 $modversion['config'][] = [
     'name'        => 'do_sibling',
-    'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_DOSIBLING',
-    'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_DOSIBLING_DESC',
+    'title'       => $GLOBALS['VAR_PREFIXU'].'_MI_DOSIBLING',
+    'description' => $GLOBALS['VAR_PREFIXU'].'_MI_DOSIBLING_DESC',
     'formtype'    => 'yesno',
     'valuetype'   => 'int',
     'default'     => 1
@@ -247,8 +247,8 @@ 
 
 $modversion['config'][] = [
     'name'        => 'pings',
-    'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_PING',
-    'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_PING_DESC',
+    'title'       => $GLOBALS['VAR_PREFIXU'].'_MI_PING',
+    'description' => $GLOBALS['VAR_PREFIXU'].'_MI_PING_DESC',
     'formtype'    => 'textarea',
     'valuetype'   => 'text',
     'default'     => ''
@@ -256,8 +256,8 @@ 
 
 $modversion['config'][] = [
     'name'        => 'trackback_option',
-    'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_TRACKBACK_OPTION',
-    'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_TRACKBACK_OPTION_DESC',
+    'title'       => $GLOBALS['VAR_PREFIXU'].'_MI_TRACKBACK_OPTION',
+    'description' => $GLOBALS['VAR_PREFIXU'].'_MI_TRACKBACK_OPTION_DESC',
     'formtype'    => 'select',
     'valuetype'   => 'int',
     'default'     => 0,
@@ -266,17 +266,17 @@ 
 
 $modversion['config'][] = [
     'name'        => 'copyright',
-    'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_COPYRIGHT',
-    'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_COPYRIGHT_DESC',
+    'title'       => $GLOBALS['VAR_PREFIXU'].'_MI_COPYRIGHT',
+    'description' => $GLOBALS['VAR_PREFIXU'].'_MI_COPYRIGHT_DESC',
     'formtype'    => 'textbox',
     'valuetype'   => 'text',
-    'default'     => 'Copyright© %s & ' . $xoopsConfig['sitename']
+    'default'     => 'Copyright© %s & '.$xoopsConfig['sitename']
 ];
 
 $modversion['config'][] = [
     'name'        => 'newblog_submit',
-    'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_NEWBLOG_SUBMIT',
-    'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_NEWBLOG_SUBMIT_DESC',
+    'title'       => $GLOBALS['VAR_PREFIXU'].'_MI_NEWBLOG_SUBMIT',
+    'description' => $GLOBALS['VAR_PREFIXU'].'_MI_NEWBLOG_SUBMIT_DESC',
     'formtype'    => 'select',
     'valuetype'   => 'int',
     'default'     => 2,
@@ -291,8 +291,8 @@ 
 
 $modversion['config'][] = [
     'name'        => 'anonymous_rate',
-    'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_ANONYMOUSRATE',
-    'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_ANONYMOUSRATE_DESC',
+    'title'       => $GLOBALS['VAR_PREFIXU'].'_MI_ANONYMOUSRATE',
+    'description' => $GLOBALS['VAR_PREFIXU'].'_MI_ANONYMOUSRATE_DESC',
     'formtype'    => 'yesno',
     'valuetype'   => 'int',
     'default'     => 0
@@ -300,8 +300,8 @@ 
 
 $modversion['config'][] = [
     'name'        => 'do_pseudocron',
-    'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_PSEUDOCRON',
-    'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_PSEUDOCRON_DESC',
+    'title'       => $GLOBALS['VAR_PREFIXU'].'_MI_PSEUDOCRON',
+    'description' => $GLOBALS['VAR_PREFIXU'].'_MI_PSEUDOCRON_DESC',
     'formtype'    => 'yesno',
     'valuetype'   => 'int',
     'default'     => 1
@@ -311,7 +311,7 @@ 
 
 $modversion['hasNotification']             = 1;
 $modversion['notification']['lookup_file'] = 'include/notification.inc.php';
-$modversion['notification']['lookup_func'] = $GLOBALS['VAR_PREFIX'] . '_notify_iteminfo';
+$modversion['notification']['lookup_func'] = $GLOBALS['VAR_PREFIX'].'_notify_iteminfo';
 
 $i = 0;
 ++$i;

include/config.php

Spacing +2 added lines, -2 removed lines

@@ -20,12 +20,12 @@
  */
 
 // defined('XOOPS_ROOT_PATH') || exit('Restricted access.');
-require_once __DIR__ . '/common.php';
+require_once __DIR__.'/common.php';
 
 $moduleDirName = basename(dirname(__DIR__));
 $uploadFolders = [
     NEWBB_UPLOAD_PATH,
-    NEWBB_UPLOAD_PATH . '/thumbs'
+    NEWBB_UPLOAD_PATH.'/thumbs'
 ];
 
 //$copyFiles = array(

include/functions.ini.php

Spacing +4 added lines, -4 removed lines

@@ -80,7 +80,7 @@ 
             if (isset($GLOBALS['xoopsModuleConfig'])) {
                 $moduleConfig = $GLOBALS['xoopsModuleConfig'];
             }
-        } else {
+        }else {
             /** @var XoopsModuleHandler $moduleHandler */
             $moduleHandler = xoops_getHandler('module');
             $module        = $moduleHandler->getByDirname($GLOBALS['moddirname']);
@@ -93,7 +93,7 @@ 
             }
             unset($configs);
         }
-        if ($customConfig = @include XOOPS_ROOT_PATH . '/modules/' . $GLOBALS['moddirname'] . '/include/plugin.php') {
+        if ($customConfig = @include XOOPS_ROOT_PATH.'/modules/'.$GLOBALS['moddirname'].'/include/plugin.php') {
             $moduleConfig = array_merge($moduleConfig, $customConfig);
         }
 
@@ -106,11 +106,11 @@ 
             if (!in_array(URL_DELIMITER, ['?', '/'])) {
                 die('Exit on security');
             }
-        } else {
+        }else {
             $moduleConfig = planet_load_config();
             if (empty($moduleConfig['do_urw'])) {
                 define('URL_DELIMITER', '?');
-            } else {
+            }else {
                 define('URL_DELIMITER', '/');
             }
         }

include/functions.php

Spacing +42 added lines, -42 removed lines

@@ -38,9 +38,9 @@ 
 if (!defined('planet_FUNCTIONS')):
     define('planet_FUNCTIONS', 1);
 
-    require XOOPS_ROOT_PATH . '/modules/' . $GLOBALS['moddirname'] . '/include/vars.php';
-    require_once XOOPS_ROOT_PATH . '/class/xoopslists.php';
-    require_once XOOPS_ROOT_PATH . '/Frameworks/art/functions.php';
+    require XOOPS_ROOT_PATH.'/modules/'.$GLOBALS['moddirname'].'/include/vars.php';
+    require_once XOOPS_ROOT_PATH.'/class/xoopslists.php';
+    require_once XOOPS_ROOT_PATH.'/Frameworks/art/functions.php';
 
     /**
      * Function to display messages
@@ -69,7 +69,7 @@ 
     */
     function planetParseArguments(&$args_numeric, &$args, &$args_string)
     {
-        $args_abb     = [
+        $args_abb = [
             'a' => 'article',
             'b' => 'blog',
             'c' => 'category',
@@ -84,24 +84,24 @@ 
         if (preg_match("/[^\?]*\.php[\/|\?]([^\?]*)/i", Request::getUrl('REQUEST_URI', '', 'SERVER'), $matches)) {
             $vars = preg_split("/[\/|&]/", $matches[1]);
             $vars = array_map('trim', $vars);
-            if (count($vars) > 0) {
+            if (count($vars)>0) {
                 foreach ($vars as $var) {
                     if (is_numeric($var)) {
                         $args_numeric[] = $var;
                     } elseif (false === strpos($var, '=')) {
                         if (is_numeric(substr($var, 1))) {
-                            $args[$args_abb[strtolower($var{0})]] = (int)substr($var, 1);
-                        } else {
+                            $args[$args_abb[strtolower($var{0})]] = (int) substr($var, 1);
+                        }else {
                             $args_string[] = urldecode($var);
                         }
-                    } else {
+                    }else {
                         parse_str($var, $args);
                     }
                 }
             }
         }
 
-        return (count($args) + count($args_numeric) + count($args_string) == 0) ? null : true;
+        return (count($args)+count($args_numeric)+count($args_string) == 0) ? null : true;
     }
 
     /**
@@ -124,7 +124,7 @@ 
             $pattern     = [$pattern];
             $replacement = [$replacement];
         }
-        if (is_array($pattern) && count($pattern) > 0) {
+        if (is_array($pattern) && count($pattern)>0) {
             $ii = 0;
             foreach ($pattern as $pat) {
                 if (!in_array($pat, $patterns)) {
@@ -160,7 +160,7 @@ 
             $pattern     = [$pattern];
             $replacement = [$replacement];
         }
-        if (is_array($pattern) && count($pattern) > 0) {
+        if (is_array($pattern) && count($pattern)>0) {
             $ii = 0;
             foreach ($pattern as $pat) {
                 if (!in_array($pat, $patterns)) {
@@ -203,7 +203,7 @@ 
         if (!is_array($userid)) {
             $userid = [$userid];
         }
-        $users =& mod_getUnameFromIds($userid, $usereal, $linked);
+        $users = & mod_getUnameFromIds($userid, $usereal, $linked);
 
         return $users;
     }
@@ -220,7 +220,7 @@ 
         $myts       = MyTextSanitizer::getInstance();
         $link_array = preg_split("/(\r\n|\r|\n)( *)/", $text);
         $links      = [];
-        if (count($link_array) > 0) {
+        if (count($link_array)>0) {
             foreach ($link_array as $link) {
                 @list($url, $title) = array_map('trim', preg_split('/ /', $link, 2));
                 if (empty($url)) {
@@ -240,7 +240,7 @@ 
      */
     function planetGetTemplate($pagename)
     {
-        return $GLOBALS['VAR_PREFIX'] . '_' . $pagename . '.tpl';
+        return $GLOBALS['VAR_PREFIX'].'_'.$pagename.'.tpl';
     }
 
     /**
@@ -268,7 +268,7 @@ 
             return false;
         }
         @list($pat, $rep) = array_map('trim', preg_split("#[\s]+#", $pattern));
-        $trackback_url = preg_replace('#' . $pat . '#', $rep, $article_obj->getVar('art_link'));
+        $trackback_url = preg_replace('#'.$pat.'#', $rep, $article_obj->getVar('art_link'));
 
         return planetTrackback($trackback_url, $article);
     }
@@ -284,7 +284,7 @@ 
 
         $title         = $article->getVar('art_title');
         $excerpt       = $article->getVar('art_content');
-        $blog_name     = $xoopsConfig['sitename'] . '-' . $xoopsModule->getVar('name');
+        $blog_name     = $xoopsConfig['sitename'].'-'.$xoopsModule->getVar('name');
         $title         = xoops_utf8_encode($title);
         $excerpt       = xoops_utf8_encode($excerpt);
         $blog_name     = xoops_utf8_encode($blog_name);
@@ -292,15 +292,15 @@ 
         $title1        = urlencode($title);
         $excerpt1      = urlencode($excerpt);
         $name1         = urlencode($blog_name);
-        $url           = urlencode(XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/view.article.php' . URL_DELIMITER . '' . $article->getVar('art_id'));
+        $url           = urlencode(XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/view.article.php'.URL_DELIMITER.''.$article->getVar('art_id'));
         $query_string  = "title=$title1&url=$url&blog_name=$name1&excerpt=$excerpt1&charset=$charset";
         $trackback_url = parse_url($trackback_url);
 
-        $http_request = 'POST ' . $trackback_url['path'] . ($trackback_url['query'] ? '?' . $trackback_url['query'] : '') . " HTTP/1.0\r\n";
-        $http_request .= 'Host: ' . $trackback_url['host'] . "\r\n";
-        $http_request .= 'Content-Type: application/x-www-form-urlencoded; charset=' . $charset . "\r\n";
-        $http_request .= 'Content-Length: ' . strlen($query_string) . "\r\n";
-        $http_request .= 'User-Agent: XOOPS Blogs/' . XOOPS_VERSION;
+        $http_request = 'POST '.$trackback_url['path'].($trackback_url['query'] ? '?'.$trackback_url['query'] : '')." HTTP/1.0\r\n";
+        $http_request .= 'Host: '.$trackback_url['host']."\r\n";
+        $http_request .= 'Content-Type: application/x-www-form-urlencoded; charset='.$charset."\r\n";
+        $http_request .= 'Content-Length: '.strlen($query_string)."\r\n";
+        $http_request .= 'User-Agent: XOOPS Blogs/'.XOOPS_VERSION;
         $http_request .= "\r\n\r\n";
         $http_request .= $query_string;
         if ('' == $trackback_url['port']) {
@@ -309,11 +309,11 @@ 
         $fs = @fsockopen($trackback_url['host'], $trackback_url['port'], $errno, $errstr, 4);
         @fwrite($fs, $http_request);
         if ($xoopsModuleConfig['do_debug']) {
-            $debug_file = XOOPS_CACHE_PATH . '/' . $GLOBALS['moddirname'] . '_trackback.log';
+            $debug_file = XOOPS_CACHE_PATH.'/'.$GLOBALS['moddirname'].'_trackback.log';
             $fr         = "\n*****\nRequest:\n\n$http_request\n\nResponse:\n\n";
             $fr         .= "CHARSET:$charset\n";
             $fr         .= "NAME:$blog_name\n";
-            $fr         .= 'TITLE:' . $title . "\n";
+            $fr         .= 'TITLE:'.$title."\n";
             $fr         .= "EXCERPT:$excerpt\n\n";
             while (!@feof($fs)) {
                 $fr .= @fgets($fs, 4096);
@@ -323,7 +323,7 @@ 
             if ($fp = fopen($debug_file, 'a')) {
                 fwrite($fp, $fr);
                 fclose($fp);
-            } else {
+            }else {
             }
         }
         @fclose($fs);
@@ -343,19 +343,19 @@ 
                 planetGetPing($serv, $id);
             }
         }
-        require_once XOOPS_ROOT_PATH . '/modules/' . $GLOBALS['moddirname'] . '/class-IXR.php';
+        require_once XOOPS_ROOT_PATH.'/modules/'.$GLOBALS['moddirname'].'/class-IXR.php';
 
         // using a timeout of 3 seconds should be enough to cover slow servers
         $client            = new IXR_Client($server, false);
         $client->timeout   = 3;
-        $client->useragent .= ' -- XOOPS Article/' . XOOPS_VERSION;
+        $client->useragent .= ' -- XOOPS Article/'.XOOPS_VERSION;
 
         // when set to true, this outputs debug messages by itself
         $client->debug = false;
 
         $blogname = xoops_utf8_encode($GLOBALS['xoopsModule']->getVar('name'));
-        $home     = XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/';
-        $rss2_url = XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/xml.php' . URL_DELIMITER . 'rss2.0/' . $id;
+        $home     = XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/';
+        $rss2_url = XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/xml.php'.URL_DELIMITER.'rss2.0/'.$id;
 
         if (!$client->query('weblogUpdates.extendedPing', $blogname, $home, $rss2_url)) { // then try a normal ping
             $client->query('weblogUpdates.ping', $blogname, $home);
@@ -371,16 +371,16 @@ 
     {
         $charset       = 'utf-8';
         $error_message = xoops_utf8_encode($error_message);
-        header('Content-Type: text/xml; charset="' . $charset . '"');
+        header('Content-Type: text/xml; charset="'.$charset.'"');
         if ($error) {
-            echo '<?xml version="1.0" encoding="' . $charset . '"?' . ">\n";
+            echo '<?xml version="1.0" encoding="'.$charset.'"?'.">\n";
             echo "<response>\n";
             echo "<error>1</error>\n";
             echo "<message>$error_message</message>\n";
             echo '</response>';
             die();
-        } else {
-            echo '<?xml version="1.0" encoding="' . $charset . '"?' . ">\n";
+        }else {
+            echo '<?xml version="1.0" encoding="'.$charset.'"?'.">\n";
             echo "<response>\n";
             echo "<error>0</error>\n";
             echo '</response>';
@@ -400,11 +400,11 @@ 
         if (is_array($string)) {
             $value = [];
             foreach ($string as $key => $val) {
-                $value[] = $key . '|' . $val;
+                $value[] = $key.'|'.$val;
             }
             $string = implode(',', $value);
         }
-        setcookie($GLOBALS['VAR_PREFIX'] . $name, $string, (int)$expire, '/');
+        setcookie($GLOBALS['VAR_PREFIX'].$name, $string, (int) $expire, '/');
     }
 
     /**
@@ -414,14 +414,14 @@ 
      */
     function planetGetCookie($name, $isArray = false)
     {
-        $value = isset($_COOKIE[$GLOBALS['VAR_PREFIX'] . $name]) ? $_COOKIE[$GLOBALS['VAR_PREFIX'] . $name] : null;
+        $value = isset($_COOKIE[$GLOBALS['VAR_PREFIX'].$name]) ? $_COOKIE[$GLOBALS['VAR_PREFIX'].$name] : null;
         if ($isArray) {
             $_value = $value ? explode(',', $value) : [];
             $value  = [];
-            if (count($_value) > 0) {
+            if (count($_value)>0) {
                 foreach ($_value as $string) {
                     $key         = substr($string, 0, strpos($string, '|'));
-                    $val         = substr($string, strpos($string, '|') + 1);
+                    $val         = substr($string, strpos($string, '|')+1);
                     $value[$key] = $val;
                 }
             }
@@ -479,7 +479,7 @@ 
      */
     function planet_fetch_snoopy($url)
     {
-        require_once XOOPS_ROOT_PATH . '/class/snoopy.php';
+        require_once XOOPS_ROOT_PATH.'/class/snoopy.php';
         $snoopy = new Snoopy;
         $data   = '';
         if (@$snoopy->fetch($url)) {
@@ -498,10 +498,10 @@ 
         if (!function_exists('curl_init')) {
             return false;
         }
-        $ch = curl_init();    // initialize curl handle
+        $ch = curl_init(); // initialize curl handle
         curl_setopt($ch, CURLOPT_URL, $url); // set url to post to
         curl_setopt($ch, CURLOPT_FAILONERROR, 1);
-        curl_setopt($ch, CURLOPT_FOLLOWLOCATION, 1);// allow redirects
+        curl_setopt($ch, CURLOPT_FOLLOWLOCATION, 1); // allow redirects
         curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1); // return into a variable
         curl_setopt($ch, CURLOPT_TIMEOUT, 30); // times out after 31s
         $data = curl_exec($ch); // run the whole process
@@ -543,7 +543,7 @@ 
         if ($index === false) {
             return false;
         }
-        $index = strlen($haystack) - strlen($needle) - $index;
+        $index = strlen($haystack)-strlen($needle)-$index;
 
         return $index;
     }

include/common.php

Spacing +10 added lines, -10 removed lines

@@ -22,19 +22,19 @@
 // defined('XOOPS_ROOT_PATH') || exit('Restricted access.');
 
 define('NEWBB_DIRNAME', basename(dirname(__DIR__)));
-define('NEWBB_URL', XOOPS_URL . '/modules/' . NEWBB_DIRNAME);
-define('NEWBB_PATH', XOOPS_ROOT_PATH . '/modules/' . NEWBB_DIRNAME);
-define('NEWBB_IMAGES_URL', NEWBB_URL . '/assets/images');
-define('NEWBB_ADMIN_URL', NEWBB_URL . '/admin');
-define('NEWBB_ADMIN_PATH', NEWBB_PATH . '/admin/index.php');
-define('NEWBB_ROOT_PATH', $GLOBALS['xoops']->path('modules/' . NEWBB_DIRNAME));
-define('NEWBB_AUTHOR_LOGOIMG', NEWBB_URL . '/assets/images/logo_module.png');
-define('NEWBB_UPLOAD_URL', XOOPS_UPLOAD_URL . '/' . NEWBB_DIRNAME); // WITHOUT Trailing slash
-define('NEWBB_UPLOAD_PATH', XOOPS_UPLOAD_PATH . '/' . NEWBB_DIRNAME); // WITHOUT Trailing slash
+define('NEWBB_URL', XOOPS_URL.'/modules/'.NEWBB_DIRNAME);
+define('NEWBB_PATH', XOOPS_ROOT_PATH.'/modules/'.NEWBB_DIRNAME);
+define('NEWBB_IMAGES_URL', NEWBB_URL.'/assets/images');
+define('NEWBB_ADMIN_URL', NEWBB_URL.'/admin');
+define('NEWBB_ADMIN_PATH', NEWBB_PATH.'/admin/index.php');
+define('NEWBB_ROOT_PATH', $GLOBALS['xoops']->path('modules/'.NEWBB_DIRNAME));
+define('NEWBB_AUTHOR_LOGOIMG', NEWBB_URL.'/assets/images/logo_module.png');
+define('NEWBB_UPLOAD_URL', XOOPS_UPLOAD_URL.'/'.NEWBB_DIRNAME); // WITHOUT Trailing slash
+define('NEWBB_UPLOAD_PATH', XOOPS_UPLOAD_PATH.'/'.NEWBB_DIRNAME); // WITHOUT Trailing slash
 
 // module information
 $mod_copyright = "<a href='https://xoops.org' title='XOOPS Project' target='_blank'>
-                     <img src='" . NEWBB_AUTHOR_LOGOIMG . "' alt='XOOPS Project'></a>";
+                     <img src='" . NEWBB_AUTHOR_LOGOIMG."' alt='XOOPS Project'></a>";
 
 xoops_loadLanguage('common', NEWBB_DIRNAME);
 

include/form.blog.php

Spacing +2 added lines, -2 removed lines

@@ -26,7 +26,7 @@ 
 // ------------------------------------------------------------------------ //
 // defined('XOOPS_ROOT_PATH') || exit('Restricted access.');
 
-require_once XOOPS_ROOT_PATH . '/class/xoopsformloader.php';
+require_once XOOPS_ROOT_PATH.'/class/xoopsformloader.php';
 $categoryHandler = xoops_getModuleHandler('category', $GLOBALS['moddirname']);
 
 $form = new XoopsThemeForm(_EDIT, 'formblog', xoops_getenv('PHP_SELF'), 'POST', true);
@@ -76,7 +76,7 @@ 
 $butt_save = new XoopsFormButton('', 'submit', _SUBMIT, 'submit');
 $button_tray->addElement($butt_save);
 $butt_cancel = new XoopsFormButton('', 'cancel', _CANCEL, 'button');
-$butt_cancel->setExtra("onclick='window.document.location=\"" . XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/index.php' . URL_DELIMITER . 'b' . (int)$blog_id . "\"'");
+$butt_cancel->setExtra("onclick='window.document.location=\"".XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/index.php'.URL_DELIMITER.'b'.(int) $blog_id."\"'");
 $button_tray->addElement($butt_cancel);
 $form->addElement($button_tray);
 $form->display();

include/plugin.transfer.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 
 // defined('XOOPS_ROOT_PATH') || exit('Restricted access.');
 
-if (!@require_once XOOPS_ROOT_PATH . '/Frameworks/transfer/transfer.php') {
+if (!@require_once XOOPS_ROOT_PATH.'/Frameworks/transfer/transfer.php') {
     return null;
 }
 

include/action.module.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
 
 // defined('XOOPS_ROOT_PATH') || exit('Restricted access.');
 
-include __DIR__ . '/vars.php';
+include __DIR__.'/vars.php';
 //mod_loadFunctions('', $GLOBALS['moddirname']);
 
 PlanetUtility::planetParseFunction('