mambax7 / planet

comment_edit.php

Spacing +2 added lines, -2 removed lines

@@ -24,5 +24,5 @@
 // URL: https://xoops.org                         //
 // Project: Article Project                                                 //
 // ------------------------------------------------------------------------ //
-include __DIR__ . '/../../mainfile.php';
-include XOOPS_ROOT_PATH . '/include/comment_edit.php';
+include __DIR__.'/../../mainfile.php';
+include XOOPS_ROOT_PATH.'/include/comment_edit.php';

comment_delete.php

Spacing +2 added lines, -2 removed lines

@@ -24,5 +24,5 @@
 // URL: https://xoops.org                         //
 // Project: Article Project                                                 //
 // ------------------------------------------------------------------------ //
-include __DIR__ . '/../../mainfile.php';
-include XOOPS_ROOT_PATH . '/include/comment_delete.php';
+include __DIR__.'/../../mainfile.php';
+include XOOPS_ROOT_PATH.'/include/comment_delete.php';

comment_post.php

Spacing +2 added lines, -2 removed lines

@@ -24,5 +24,5 @@
 // URL: https://xoops.org                         //
 // Project: Article Project                                                 //
 // ------------------------------------------------------------------------ //
-include __DIR__ . '/../../mainfile.php';
-include XOOPS_ROOT_PATH . '/include/comment_post.php';
+include __DIR__.'/../../mainfile.php';
+include XOOPS_ROOT_PATH.'/include/comment_post.php';

transfer.php

Spacing +18 added lines, -18 removed lines

@@ -14,43 +14,43 @@ 
 
 use Xmf\Request;
 
-include __DIR__ . '/header.php';
+include __DIR__.'/header.php';
 
 if (PlanetUtility::planetParseArguments($args_num, $args, $args_str)) {
     $args['article'] = @$args_num[0];
     $args['op']      = @$args_str[0];
 }
 
-$article_id = Request::getInt('article', Request::getInt('article', @$args['article'], 'POST'), 'GET');//(int)(empty($_GET['article']) ? (empty($_POST['article']) ? @$args['article'] : $_POST['article']) : $_GET['article']);
+$article_id = Request::getInt('article', Request::getInt('article', @$args['article'], 'POST'), 'GET'); //(int)(empty($_GET['article']) ? (empty($_POST['article']) ? @$args['article'] : $_POST['article']) : $_GET['article']);
 
-$op = Request::getString('op', Request::getString('op', @$args['op'], 'POST'), 'GET');//empty($_GET['op']) ? (empty($_POST['op']) ? @$args['op'] : $_POST['op']) : $_GET['op'];
+$op = Request::getString('op', Request::getString('op', @$args['op'], 'POST'), 'GET'); //empty($_GET['op']) ? (empty($_POST['op']) ? @$args['op'] : $_POST['op']) : $_GET['op'];
 $op = strtolower(trim($op));
 
 if (empty($article_id)) {
     if (empty(Request::getUrl('HTTP_REFERER', '', 'SERVER'))) {
         //$_SERVER['HTTP_REFERER']))
 
-        include XOOPS_ROOT_PATH . '/header.php';
+        include XOOPS_ROOT_PATH.'/header.php';
         xoops_error(_NOPERM);
         $xoopsOption['output_type'] = 'plain';
-        include XOOPS_ROOT_PATH . '/footer.php';
+        include XOOPS_ROOT_PATH.'/footer.php';
         exit();
-    } else {
+    }else {
         $ref_parser = parse_url(Request::getUrl('HTTP_REFERER', '', 'SERVER')); //$_SERVER['HTTP_REFERER']);
-        $uri_parser = parse_url(Request::getUrl('REQUEST_URI', '', 'SERVER'));// $_SERVER['REQUEST_URI']);
+        $uri_parser = parse_url(Request::getUrl('REQUEST_URI', '', 'SERVER')); // $_SERVER['REQUEST_URI']);
         if ((!empty($ref_parser['host']) && !empty($uri_parser['host']) && $uri_parser['host'] != $ref_parser['host'])
             || ($ref_parser['path'] != $uri_parser['path'])) {
-            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';
             xoops_confirm([], 'javascript: window.close();', sprintf(planet_constant('MD_TRANSFER_DONE'), ''), _CLOSE, Request::getUrl('HTTP_REFERER', '', 'SERVER'));
             $xoopsOption['output_type'] = 'plain';
-            include XOOPS_ROOT_PATH . '/footer.php';
+            include XOOPS_ROOT_PATH.'/footer.php';
             exit();
-        } else {
-            include XOOPS_ROOT_PATH . '/header.php';
+        }else {
+            include XOOPS_ROOT_PATH.'/header.php';
             xoops_error(_NOPERM);
             $xoopsOption['output_type'] = 'plain';
-            include XOOPS_ROOT_PATH . '/footer.php';
+            include XOOPS_ROOT_PATH.'/footer.php';
             exit();
         }
     }
@@ -62,16 +62,16 @@ 
 // Display option form
 if (empty($op)) {
     $module_variables .= "<input type=\"hidden\" name=\"article\" id=\"article\" value=\"{$article_id}\">";
-    include XOOPS_ROOT_PATH . '/Frameworks/transfer/option.transfer.php';
+    include XOOPS_ROOT_PATH.'/Frameworks/transfer/option.transfer.php';
     exit();
-} else {
+}else {
     $data           = [];
     $data['id']     = $article_id;
     $data['title']  = $article_obj->getVar('art_title');
     $data['time']   = $article_obj->getTime('l');
     $data['image']  = '';
     $data['source'] = $article_obj->getVar('art_link');
-    $data['url']    = XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/view.article.php' . URL_DELIMITER . '' . $article_obj->getVar('art_id');
+    $data['url']    = XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/view.article.php'.URL_DELIMITER.''.$article_obj->getVar('art_id');
     $data['author'] = $article_obj->getVar('art_author');
 
     switch ($op) {
@@ -82,7 +82,7 @@ 
 
         case 'print':
         case 'pdf':
-            ${"{$op}_data"}            =& $data;
+            ${"{$op}_data"}            = & $data;
             ${"{$op}_data"}['date']    = $pdf_data['time'];
             ${"{$op}_data"}['content'] = $article_obj->getVar('art_content');
             break;
@@ -92,6 +92,6 @@ 
             $data['content'] = $article_obj->getSummary();
             break;
     }
-    include XOOPS_ROOT_PATH . '/Frameworks/transfer/action.transfer.php';
+    include XOOPS_ROOT_PATH.'/Frameworks/transfer/action.transfer.php';
     exit();
 }

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

class/magpie.inc.php

Spacing +32 added lines, -32 removed lines

@@ -35,16 +35,16 @@ 
 {
     public $parser;
 
-    public $current_item = [];  // item currently being parsed
-    public $items        = [];  // collection of parsed items
-    public $channel      = [];  // hash of channel fields
+    public $current_item = []; // item currently being parsed
+    public $items        = []; // collection of parsed items
+    public $channel      = []; // hash of channel fields
     public $textinput    = [];
     public $image        = [];
     public $feed_type;
     public $feed_version;
-    public $encoding     = '';       // output encoding of parsed rss
+    public $encoding     = ''; // output encoding of parsed rss
 
-    public $_source_encoding = '';     // only set if we have to parse xml prolog
+    public $_source_encoding = ''; // only set if we have to parse xml prolog
 
     public $ERROR   = '';
     public $WARNING = '';
@@ -101,13 +101,13 @@ 
         # if PHP xml isn't compiled in, die
         #
         if (!function_exists('xml_parser_create')) {
-            $this->error("Failed to load PHP's XML Extension. " . 'http://www.php.net/manual/en/ref.xml.php', E_USER_ERROR);
+            $this->error("Failed to load PHP's XML Extension. ".'http://www.php.net/manual/en/ref.xml.php', E_USER_ERROR);
         }
 
         list($parser, $source) = $this->create_parser($source, $output_encoding, $input_encoding, $detect_encoding);
 
         if (!is_resource($parser)) {
-            $this->error("Failed to create an instance of PHP's XML parser. " . 'http://www.php.net/manual/en/ref.xml.php', E_USER_ERROR);
+            $this->error("Failed to create an instance of PHP's XML parser. ".'http://www.php.net/manual/en/ref.xml.php', E_USER_ERROR);
         }
 
         $this->parser = $parser;
@@ -217,8 +217,8 @@ 
         elseif (ATOM == $this->feed_type && 'link' === $el) {
             if (isset($attrs['rel']) && 'alternate' === $attrs['rel']) {
                 $link_el = 'link';
-            } else {
-                $link_el = 'link_' . $attrs['rel'];
+            }else {
+                $link_el = 'link_'.$attrs['rel'];
             }
 
             $this->append($link_el, $attrs['href']);
@@ -236,7 +236,7 @@ 
     {
         if (ATOM == $this->feed_type && $this->incontent) {
             $this->append_content($text);
-        } else {
+        }else {
             $current_el = implode('_', array_reverse($this->stack));
             $this->append($current_el, $text);
         }
@@ -267,12 +267,12 @@ 
             // note:  i don't think this is actually neccessary
             if ($this->stack[0] == $el) {
                 $this->append_content("</$el>");
-            } else {
+            }else {
                 $this->append_content("<$el>");
             }
 
             array_shift($this->stack);
-        } else {
+        }else {
             array_shift($this->stack);
         }
 
@@ -324,7 +324,7 @@ 
             } elseif ($this->inimage) {
                 $this->concat($this->image[$this->current_namespace][$el], $text);
             }
-        } else {
+        }else {
             if ($this->initem) {
                 $this->concat($this->current_item[$el], $text);
             } elseif ($this->intextinput) {
@@ -342,7 +342,7 @@ 
         // if atom populate rss fields
         if ($this->is_atom()) {
             $this->channel['description'] = $this->channel['tagline'];
-            for ($i = 0, $iMax = count($this->items); $i < $iMax; ++$i) {
+            for ($i = 0, $iMax = count($this->items); $i<$iMax; ++$i) {
                 $item = $this->items[$i];
                 if (isset($item['summary'])) {
                     $item['description'] = $item['summary'];
@@ -354,7 +354,7 @@ 
                 $atom_date = isset($item['issued']) ? $item['issued'] : @$item['modified'];
                 if ($atom_date) {
                     $epoch = @parse_w3cdtf($atom_date);
-                    if ($epoch && $epoch > 0) {
+                    if ($epoch && $epoch>0) {
                         $item['date_timestamp'] = $epoch;
                     }
                 }
@@ -363,7 +363,7 @@ 
             }
         } elseif ($this->is_rss()) {
             $this->channel['tagline'] = $this->channel['description'];
-            for ($i = 0, $iMax = count($this->items); $i < $iMax; ++$i) {
+            for ($i = 0, $iMax = count($this->items); $i<$iMax; ++$i) {
                 $item = $this->items[$i];
                 if (isset($item['description'])) {
                     $item['summary'] = $item['description'];
@@ -374,12 +374,12 @@ 
 
                 if ('1.0' === $this->is_rss() && isset($item['dc']['date'])) {
                     $epoch = @parse_w3cdtf($item['dc']['date']);
-                    if ($epoch && $epoch > 0) {
+                    if ($epoch && $epoch>0) {
                         $item['date_timestamp'] = $epoch;
                     }
                 } elseif (isset($item['pubdate'])) {
                     $epoch = @strtotime($item['pubdate']);
-                    if ($epoch > 0) {
+                    if ($epoch>0) {
                         $item['date_timestamp'] = $epoch;
                     }
                 }
@@ -396,7 +396,7 @@ 
     {
         if (RSS == $this->feed_type) {
             return $this->feed_version;
-        } else {
+        }else {
             return false;
         }
     }
@@ -408,7 +408,7 @@ 
     {
         if (ATOM == $this->feed_type) {
             return $this->feed_version;
-        } else {
+        }else {
             return false;
         }
     }
@@ -425,7 +425,7 @@ 
     {
         if (5 == substr(PHP_VERSION, 0, 1)) {
             $parser = $this->php5_create_parser($in_enc, $detect);
-        } else {
+        }else {
             list($parser, $source) = $this->php4_create_parser($source, $in_enc, $detect);
         }
         if ($out_enc) {
@@ -452,7 +452,7 @@ 
         // by default php5 does a fine job of detecting input encodings
         if (!$detect && $in_enc) {
             return xml_parser_create($in_enc);
-        } else {
+        }else {
             return xml_parser_create('');
         }
     }
@@ -485,7 +485,7 @@ 
             if (preg_match('/<?xml.*encoding=[\'"](.*?)[\'"].*?>/m', $source, $m)) {
                 $in_enc                = strtoupper($m[1]);
                 $this->source_encoding = $in_enc;
-            } else {
+            }else {
                 $in_enc = 'UTF-8';
             }
         }
@@ -535,7 +535,7 @@ 
         $enc = strtoupper($enc);
         if (in_array($enc, $this->_KNOWN_ENCODINGS)) {
             return $enc;
-        } else {
+        }else {
             return false;
         }
     }
@@ -552,14 +552,14 @@ 
         }
         if (MAGPIE_DEBUG) {
             trigger_error($errormsg, $lvl);
-        } else {
+        }else {
             error_log($errormsg, 0);
         }
 
         $notices = E_USER_NOTICE | E_NOTICE;
         if ($lvl & $notices) {
             $this->WARNING = $errormsg;
-        } else {
+        }else {
             $this->ERROR = $errormsg;
         }
     }
@@ -595,12 +595,12 @@ 
         ];
 
         # calc epoch for current date assuming GMT
-        $epoch = gmmktime((int)$hours, (int)$minutes, (int)$seconds, (int)$month, (int)$day, (int)$year);
+        $epoch = gmmktime((int) $hours, (int) $minutes, (int) $seconds, (int) $month, (int) $day, (int) $year);
 
         $offset = 0;
         if ('Z' === $match[10]) {
             # zulu time, aka GMT
-        } else {
+        }else {
             list($tz_mod, $tz_hour, $tz_min) = [$match[8], $match[9], $match[10]];
 
             # zero out the variables
@@ -611,20 +611,20 @@ 
                 $tz_min = 0;
             }
 
-            $offset_secs = (($tz_hour * 60) + $tz_min) * 60;
+            $offset_secs = (($tz_hour*60)+$tz_min)*60;
 
             # is timezone ahead of GMT?  then subtract offset
             #
             if ('+' == $tz_mod) {
-                $offset_secs = $offset_secs * -1;
+                $offset_secs = $offset_secs*-1;
             }
 
             $offset = $offset_secs;
         }
-        $epoch = $epoch + $offset;
+        $epoch = $epoch+$offset;
 
         return $epoch;
-    } else {
+    }else {
         return -1;
     }
 }

class/xmlparser.php

Spacing +7 added lines, -7 removed lines

@@ -30,7 +30,7 @@ 
  */
 global $msg;
 
-require_once XOOPS_ROOT_PATH . '/modules/' . $GLOBALS['moddirname'] . '/class/magpie.inc.php';
+require_once XOOPS_ROOT_PATH.'/modules/'.$GLOBALS['moddirname'].'/class/magpie.inc.php';
 
 /**
  * XmlParser
@@ -92,7 +92,7 @@ 
             $this->feed_version = empty($this->feed_version) ? '0.3' : $this->feed_version;
 
             return $this->feed_version;
-        } else {
+        }else {
             return false;
         }
     }
@@ -105,7 +105,7 @@ 
                 $this->channel['tagline'] = @$this->channel['subtitle'];
                 unset($this->channel['subtitle']);
             }
-        for ($i = 0, $iMax = count($this->items); $i < $iMax; ++$i) {
+        for ($i = 0, $iMax = count($this->items); $i<$iMax; ++$i) {
             // ATOM time
             if ($date = @$this->items[$i]['modified']) {
                 continue;
@@ -124,7 +124,7 @@ 
             }
             $this->items[$i]['modified'] = $date;
         } elseif ('1.0' !== $this->is_rss()):
-            for ($i = 0, $iMax = count($this->items); $i < $iMax; ++$i) {
+            for ($i = 0, $iMax = count($this->items); $i<$iMax; ++$i) {
                 if ($date = @$this->items[$i]['pubdate']) {
                     continue;
                 }
@@ -150,7 +150,7 @@ 
             unset($this->channel['tagline']);
         }
 
-        for ($i = 0, $iMax = count($this->items); $i < $iMax; ++$i) {
+        for ($i = 0, $iMax = count($this->items); $i<$iMax; ++$i) {
             if ($date_timestamp = @$this->items[$i]['date_timestamp']) {
                 continue;
             }
@@ -158,7 +158,7 @@ 
                 $this->items[$i]['date_timestamp'] = $date_timestamp;
             } elseif ($date_timestamp = @$this->items[$i]['dc']['date']) {
                 $this->items[$i]['date_timestamp'] = $date_timestamp;
-            } else {
+            }else {
                 $this->items[$i]['date_timestamp'] = time();
             }
             if (!is_numeric($this->items[$i]['date_timestamp'])) {
@@ -204,7 +204,7 @@ 
             foreach (array_keys($val) as $key) {
                 $val[$key] = $this->_encoding($val[$key]);
             }
-        } else {
+        }else {
             $val = XoopsLocal::convert_encoding($val, $this->charset_out, $this->charset_in);
         }
 

view.archive.php

Spacing +59 added lines, -59 removed lines

@@ -29,7 +29,7 @@ 
 /** @var Planet\Helper $helper */
 $helper = Planet\Helper::getInstance();
 
-include __DIR__ . '/header.php';
+include __DIR__.'/header.php';
 
 if (PlanetUtility::planetParseArguments($args_num, $args, $args_str)) {
     $args['year']  = @$args_num[0];
@@ -48,26 +48,26 @@ 
 $articleHandler = xoops_getModuleHandler('article', $GLOBALS['moddirname']);
 $blogHandler    = xoops_getModuleHandler('blog', $GLOBALS['moddirname']);
 
-$xoopsOption['xoops_pagetitle'] = $xoopsModule->getVar('name') . ' - ' . planet_constant('MD_ACHIVE');
+$xoopsOption['xoops_pagetitle'] = $xoopsModule->getVar('name').' - '.planet_constant('MD_ACHIVE');
 $xoopsOption['template_main']   = PlanetUtility::planetGetTemplate('archive');
-require_once XOOPS_ROOT_PATH . '/header.php';
-include XOOPS_ROOT_PATH . '/modules/' . $xoopsModule->getVar('dirname') . '/include/vars.php';
+require_once XOOPS_ROOT_PATH.'/header.php';
+include XOOPS_ROOT_PATH.'/modules/'.$xoopsModule->getVar('dirname').'/include/vars.php';
 
 $year = empty($year) ? date('Y') : $year;
-if ($month < 1) {
+if ($month<1) {
     $month        = $day = 0;
     $page['time'] = sprintf(planet_constant('MD_TIME_Y'), $year);
-} elseif ($day < 1) {
+} elseif ($day<1) {
     $day          = 0;
     $page['time'] = sprintf(planet_constant('MD_TIME_YM'), $year, $month);
-} else {
+}else {
     $page['time'] = sprintf(planet_constant('MD_TIME_YMD'), $year, $month, $day);
 }
 $time = ['year' => $year, 'month' => $month, 'day' => $day];
 if ($xoopsUser) {
-    $timeoffset = ($xoopsUser->getVar('timezone_offset') - $xoopsConfig['server_TZ']) * 3600;
-} else {
-    $timeoffset = ($xoopsConfig['default_TZ'] - $xoopsConfig['server_TZ']) * 3600;
+    $timeoffset = ($xoopsUser->getVar('timezone_offset')-$xoopsConfig['server_TZ'])*3600;
+}else {
+    $timeoffset = ($xoopsConfig['default_TZ']-$xoopsConfig['server_TZ'])*3600;
 }
 
 $criteria = new \CriteriaCompo();
@@ -90,7 +90,7 @@ 
 $articles = [];
 $blogs_id = [];
 foreach ($articles_obj as $id => $article) {
-    $articles[]                            = [
+    $articles[] = [
         'id'      => $id,
         'blog'    => ['id' => $article->getVar('blog_id'), 'title' => ''],
         'title'   => $article->getVar('art_title'),
@@ -101,20 +101,20 @@ 
     $blogs_id[$article->getVar('blog_id')] = 1;
     unset($_article);
 }
-$criteria_blog = new \Criteria('blog_id', '(' . implode(',', array_keys($blog_array)) . ')', 'IN');
+$criteria_blog = new \Criteria('blog_id', '('.implode(',', array_keys($blog_array)).')', 'IN');
 $blogs         = $blogHandler->getList($criteria_blog);
 foreach (array_keys($articles) as $key) {
     $articles[$key]['blog']['title'] = $blogs[$articles[$key]['blog']['id']];
 }
-if ($blog_id > 0) {
+if ($blog_id>0) {
     $page['blog'] = $blogs[$blog_id];
 }
 
-if ($articles_count > $helper->getConfig('articles_perpage')) {
-    include XOOPS_ROOT_PATH . '/class/pagenav.php';
-    $nav     = new \XoopsPageNav($articles_count, $helper->getConfig('articles_perpage'), $start, 'start', 'month=' . $month . '&amp;day=' . $day . '&amp;year=' . $year . '&amp;blog=' . $blog_id);
+if ($articles_count>$helper->getConfig('articles_perpage')) {
+    include XOOPS_ROOT_PATH.'/class/pagenav.php';
+    $nav     = new \XoopsPageNav($articles_count, $helper->getConfig('articles_perpage'), $start, 'start', 'month='.$month.'&amp;day='.$day.'&amp;year='.$year.'&amp;blog='.$blog_id);
     $pagenav = $nav->renderNav(4);
-} else {
+}else {
     $pagenav = '';
 }
 
@@ -123,43 +123,43 @@ 
 $months   = null;
 if (empty($start)) {
     if ($blog_id) {
-        $blog_criteria = ' AND blog_id=' . $blog_id;
-    } else {
+        $blog_criteria = ' AND blog_id='.$blog_id;
+    }else {
         $blog_criteria = '';
     }
     // Get monthly list
     if (empty($month)) {
-        $sql    = "SELECT MONTH(FROM_UNIXTIME(art_time - $timeoffset)) AS mon, COUNT(DISTINCT art_id) AS count
-            FROM " . planet_DB_prefix('article') . "
+        $sql = "SELECT MONTH(FROM_UNIXTIME(art_time - $timeoffset)) AS mon, COUNT(DISTINCT art_id) AS count
+            FROM ".planet_DB_prefix('article')."
             WHERE YEAR(FROM_UNIXTIME(art_time - $timeoffset)) = $year
-            " . $blog_criteria . '
+            ".$blog_criteria.'
             GROUP BY mon
             ';
         $result = $xoopsDB->query($sql);
         $months = [];
         while (false !== ($myrow = $xoopsDB->fetchArray($result))) {
             $months[] = [
-                'title' => planet_constant('MD_MONTH_' . (int)$myrow['mon']) . ' (' . (int)$myrow['count'] . ')',
-                'url'   => XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/view.archive.php' . URL_DELIMITER . '' . $year . '/' . $myrow['mon'] . '/b' . $blog_id
+                'title' => planet_constant('MD_MONTH_'.(int) $myrow['mon']).' ('.(int) $myrow['count'].')',
+                'url'   => XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/view.archive.php'.URL_DELIMITER.''.$year.'/'.$myrow['mon'].'/b'.$blog_id
             ];
         }
         $timenav['prev'] = [
-            'url'   => XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/view.archive.php' . URL_DELIMITER . '' . ($year - 1) . '/b' . $blog_id,
-            'title' => sprintf(planet_constant('MD_TIME_Y'), $year - 1)
+            'url'   => XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/view.archive.php'.URL_DELIMITER.''.($year-1).'/b'.$blog_id,
+            'title' => sprintf(planet_constant('MD_TIME_Y'), $year-1)
         ];
-        if ($year < date('Y')) {
+        if ($year<date('Y')) {
             $timenav['next'] = [
-                'url'   => XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/view.archive.php' . URL_DELIMITER . '' . ($year + 1) . '/b' . $blog_id,
-                'title' => sprintf(planet_constant('MD_TIME_Y'), $year + 1)
+                'url'   => XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/view.archive.php'.URL_DELIMITER.''.($year+1).'/b'.$blog_id,
+                'title' => sprintf(planet_constant('MD_TIME_Y'), $year+1)
             ];
         }
     } // Get daily list
     elseif (empty($day)) {
-        $sql    = "SELECT DAY(FROM_UNIXTIME(art_time - $timeoffset)) AS day, COUNT(DISTINCT a.art_id) AS count
-            FROM " . planet_DB_prefix('article') . "
+        $sql = "SELECT DAY(FROM_UNIXTIME(art_time - $timeoffset)) AS day, COUNT(DISTINCT a.art_id) AS count
+            FROM ".planet_DB_prefix('article')."
             WHERE YEAR(FROM_UNIXTIME(art_time - $timeoffset)) = $year
             AND MONTH(FROM_UNIXTIME(art_time - $timeoffset)) = $month
-            " . $blog_criteria . '
+            ".$blog_criteria.'
             GROUP BY day
             ';
         $result = $xoopsDB->query($sql);
@@ -167,35 +167,35 @@ 
         while (false !== ($myrow = $xoopsDB->fetchArray($result))) {
             $days[$myrow['day']]['count'] = $myrow['count'];
         }
-        for ($i = 1; $i <= 31; ++$i) {
+        for ($i = 1; $i<=31; ++$i) {
             if (!isset($days[$i])) {
                 continue;
             }
             $days[$i] = [
                 'title' => $days[$i]['count'],
-                'url'   => XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/view.archive.php' . URL_DELIMITER . '' . $year . '/' . $month . '/' . $i . '/b' . $blog_id
+                'url'   => XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/view.archive.php'.URL_DELIMITER.''.$year.'/'.$month.'/'.$i.'/b'.$blog_id
             ];
         }
         $calendar   = planet_getCalendar($year, $month, $days);
-        $month_next = $month + 1;
-        $month_prev = $month - 1;
+        $month_next = $month+1;
+        $month_prev = $month-1;
         $_year      = $year;
         if (12 == $month) {
             $month_next = 1;
-            $_year      = $year + 1;
+            $_year      = $year+1;
         }
         if (1 == $month) {
             $month_pre = 12;
-            $_year     = $year - 1;
+            $_year     = $year-1;
         }
         $timenav['prev'] = [
-            'url'   => XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/view.archive.php' . URL_DELIMITER . '' . $_year . '/' . $month_prev . '/b' . $blog_id,
-            'title' => planet_constant('MD_MONTH_' . $month_prev)
+            'url'   => XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/view.archive.php'.URL_DELIMITER.''.$_year.'/'.$month_prev.'/b'.$blog_id,
+            'title' => planet_constant('MD_MONTH_'.$month_prev)
         ];
-        if ($year < date('Y') || $month < date('n')) {
+        if ($year<date('Y') || $month<date('n')) {
             $timenav['next'] = [
-                'url'   => XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/view.archive.php' . URL_DELIMITER . '' . $_year . '/' . $month_next . '/b' . $blog_id,
-                'title' => planet_constant('MD_MONTH_' . $month_next)
+                'url'   => XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/view.archive.php'.URL_DELIMITER.''.$_year.'/'.$month_next.'/b'.$blog_id,
+                'title' => planet_constant('MD_MONTH_'.$month_next)
             ];
         }
     }
@@ -215,7 +215,7 @@ 
 $xoopsTpl->assign('timenav', $timenav);
 $xoopsTpl->assign('pagenav', $pagenav);
 
-require_once __DIR__ . '/footer.php';
+require_once __DIR__.'/footer.php';
 
 /**
  * @param null $year
@@ -232,24 +232,24 @@ 
     ob_start();
     echo '<table id="calendar">';
     echo '<caption>';
-    printf(planet_constant('MD_TIME_YM'), $year, planet_constant('MD_MONTH_' . $month));
+    printf(planet_constant('MD_TIME_YM'), $year, planet_constant('MD_MONTH_'.$month));
     echo '</caption>';
 
-    for ($i = 1; $i <= 7; ++$i) {
-        echo "\n\t\t<th abbr=\"" . planet_constant('MD_WEEK_' . $i) . '" scope="col" title="' . planet_constant('MD_WEEK_' . $i) . '">' . planet_constant('MD_WEEK_' . $i) . '</th>';
+    for ($i = 1; $i<=7; ++$i) {
+        echo "\n\t\t<th abbr=\"".planet_constant('MD_WEEK_'.$i).'" scope="col" title="'.planet_constant('MD_WEEK_'.$i).'">'.planet_constant('MD_WEEK_'.$i).'</th>';
     }
 
     echo '<tr>';
 
     // See how much we should pad in the beginning
     $week_begins = 1;
-    $pad         = planet_calendar_week_mod(date('w', $unixmonth) - $week_begins);
+    $pad         = planet_calendar_week_mod(date('w', $unixmonth)-$week_begins);
     if (0 != $pad) {
-        echo "\n\t\t" . '<td colspan="' . $pad . '">&nbsp;</td>';
+        echo "\n\t\t".'<td colspan="'.$pad.'">&nbsp;</td>';
     }
 
-    $daysinmonth = (int)date('t', $unixmonth);
-    for ($day = 1; $day <= $daysinmonth; ++$day) {
+    $daysinmonth = (int) date('t', $unixmonth);
+    for ($day = 1; $day<=$daysinmonth; ++$day) {
         if (isset($newrow) && $newrow) {
             echo "\n\t</tr>\n\t<tr>\n\t\t";
         }
@@ -258,26 +258,26 @@ 
         echo '<td>';
 
         if (!empty($days[$day]['url'])) {
-            echo '<a href="' . $days[$day]['url'] . '"';
+            echo '<a href="'.$days[$day]['url'].'"';
             if (!empty($days[$day]['title'])) {
-                echo 'title="' . $days[$day]['title'] . '"';
+                echo 'title="'.$days[$day]['title'].'"';
             }
             echo ">$day</a>";
         } elseif (!empty($days[$day]['title'])) {
-            echo '<acronym title="' . $days[$day]['title'] . "\">$day</acronym>";
-        } else {
+            echo '<acronym title="'.$days[$day]['title']."\">$day</acronym>";
+        }else {
             echo $day;
         }
         echo '</td>';
 
-        if (6 == planet_calendar_week_mod(date('w', mktime(0, 0, 0, $month, $day, $year)) - $week_begins)) {
+        if (6 == planet_calendar_week_mod(date('w', mktime(0, 0, 0, $month, $day, $year))-$week_begins)) {
             $newrow = true;
         }
     }
 
-    $pad = 7 - planet_calendar_week_mod(date('w', mktime(0, 0, 0, $month, $day, $year)) - $week_begins);
+    $pad = 7-planet_calendar_week_mod(date('w', mktime(0, 0, 0, $month, $day, $year))-$week_begins);
     if (0 != $pad && 7 != $pad) {
-        echo "\n\t\t" . '<td class="pad" colspan="' . $pad . '">&nbsp;</td>';
+        echo "\n\t\t".'<td class="pad" colspan="'.$pad.'">&nbsp;</td>';
     }
 
     echo "\n\t</tr>\n\t</tbody>\n\t</table>";
@@ -296,5 +296,5 @@ 
 {
     $base = 7;
 
-    return ($num - $base * floor($num / $base));
+    return ($num-$base*floor($num/$base));
 }

action.article.php

Spacing +13 added lines, -13 removed lines

@@ -26,26 +26,26 @@ 
 // ------------------------------------------------------------------------ //
 use Xmf\Request;
 
-include __DIR__ . '/header.php';
+include __DIR__.'/header.php';
 
-$art_id = Request::getInt('article', Request::getInt('article', 0, 'POST'), 'GET');//(int)(isset($_GET['article']) ? $_GET['article'] : (isset($_POST['article']) ? $_POST['article'] : 0));
+$art_id = Request::getInt('article', Request::getInt('article', 0, 'POST'), 'GET'); //(int)(isset($_GET['article']) ? $_GET['article'] : (isset($_POST['article']) ? $_POST['article'] : 0));
 if (empty($art_id)) {
     redirect_header('javascript:history.go(-1);', 1, planet_constant('MD_INVALID'));
 }
 if (!$xoopsUser->isAdmin()) {
     redirect_header('javascript:history.go(-1);', 2, _NOPERM);
 }
-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';
 
 $articleHandler = xoops_getModuleHandler('article', $GLOBALS['moddirname']);
 $article_obj    = $articleHandler->get($art_id);
 
-$op = Request::getCmd('op', 'check', 'POST');//isset($_POST['op']) ? $_POST['op'] : '';
+$op = Request::getCmd('op', 'check', 'POST'); //isset($_POST['op']) ? $_POST['op'] : '';
 
 if ('del' === $op || !empty(Request::getString('del', '', 'POST'))) {
     $articleHandler->delete($article_obj);
-    $redirect = XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/index.php';
+    $redirect = XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/index.php';
     $message  = planet_constant('MD_SAVED');
     redirect_header($redirect, 2, $message);
 } elseif ('save' === $op) {
@@ -66,15 +66,15 @@ 
 
     $art_id_new = $articleHandler->insert($article_obj);
     if (!$article_obj->getVar('art_id')) {
-        $redirect = XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/index.php';
+        $redirect = XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/index.php';
         $message  = planet_constant('MD_INSERTERROR');
-    } else {
-        $redirect = XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/view.article.php' . URL_DELIMITER . '' . $article_obj->getVar('art_id');
+    }else {
+        $redirect = XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/view.article.php'.URL_DELIMITER.''.$article_obj->getVar('art_id');
         $message  = planet_constant('MD_SAVED');
     }
     redirect_header($redirect, 2, $message);
-} else {
-    require_once XOOPS_ROOT_PATH . '/class/xoopsformloader.php';
+}else {
+    require_once XOOPS_ROOT_PATH.'/class/xoopsformloader.php';
 
     $form = new \XoopsThemeForm(_EDIT, 'formarticle', xoops_getenv('PHP_SELF'), 'post', true);
 
@@ -93,9 +93,9 @@ 
     $butt_del->setExtra("onClick='document.forms.formarticle.op.value=del'");
     $button_tray->addElement($butt_del);
     $butt_cancel = new \XoopsFormButton('', 'cancel', _CANCEL, 'button');
-    $butt_cancel->setExtra("onclick='window.document.location=\"" . XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/view.article.php' . URL_DELIMITER . '' . $art_id . "\"'");
+    $butt_cancel->setExtra("onclick='window.document.location=\"".XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/view.article.php'.URL_DELIMITER.''.$art_id."\"'");
     $button_tray->addElement($butt_cancel);
     $form->addElement($button_tray);
     $form->display();
 }
-include XOOPS_ROOT_PATH . '/footer.php';
+include XOOPS_ROOT_PATH.'/footer.php';