mambax7 / planet

view.archive.php

Indentation +148 added lines, -148 removed lines

@@ -29,9 +29,9 @@ 
 include __DIR__ . '/header.php';
 
 if (PlanetUtility::planetParseArguments($args_num, $args, $args_str)) {
-    $args['year']  = @$args_num[0];
-    $args['month'] = @$args_num[1];
-    $args['day']   = @$args_num[2];
+	$args['year']  = @$args_num[0];
+	$args['month'] = @$args_num[1];
+	$args['day']   = @$args_num[2];
 }
 
 $day     = Request::getInt('day', @$args['day'], 'GET'); //(int)(empty($_GET['day']) ? @$args['day'] : $_GET['day']);
@@ -52,31 +52,31 @@ 
 
 $year = empty($year) ? date('Y') : $year;
 if ($month < 1) {
-    $month        = $day = 0;
-    $page['time'] = sprintf(planet_constant('MD_TIME_Y'), $year);
+	$month        = $day = 0;
+	$page['time'] = sprintf(planet_constant('MD_TIME_Y'), $year);
 } elseif ($day < 1) {
-    $day          = 0;
-    $page['time'] = sprintf(planet_constant('MD_TIME_YM'), $year, $month);
+	$day          = 0;
+	$page['time'] = sprintf(planet_constant('MD_TIME_YM'), $year, $month);
 } else {
-    $page['time'] = sprintf(planet_constant('MD_TIME_YMD'), $year, $month, $day);
+	$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;
+	$timeoffset = ($xoopsUser->getVar('timezone_offset') - $xoopsConfig['server_TZ']) * 3600;
 } else {
-    $timeoffset = ($xoopsConfig['default_TZ'] - $xoopsConfig['server_TZ']) * 3600;
+	$timeoffset = ($xoopsConfig['default_TZ'] - $xoopsConfig['server_TZ']) * 3600;
 }
 
 $criteria = new CriteriaCompo();
 if ($blog_id) {
-    $criteria->add(new Criteria('blog_id', $blog_id));
+	$criteria->add(new Criteria('blog_id', $blog_id));
 }
 $criteria->add(new Criteria("YEAR(FROM_UNIXTIME(art_time_publish - $timeoffset))", $year));
 if ($month) {
-    $criteria->add(new Criteria("MONTH(FROM_UNIXTIME(art_time_publish - $timeoffset))", $month));
-    if ($day) {
-        $criteria->add(new Criteria("DAY(FROM_UNIXTIME(art_time_publish - $timeoffset))", $day));
-    }
+	$criteria->add(new Criteria("MONTH(FROM_UNIXTIME(art_time_publish - $timeoffset))", $month));
+	if ($day) {
+		$criteria->add(new Criteria("DAY(FROM_UNIXTIME(art_time_publish - $timeoffset))", $day));
+	}
 }
 $criteria->setStart($start);
 $criteria->setLimit($xoopsModuleConfig['articles_perpage']);
@@ -87,115 +87,115 @@ 
 $articles = [];
 $blogs_id = [];
 foreach ($articles_obj as $id => $article) {
-    $articles[]                            = [
-        'id'      => $id,
-        'blog'    => ['id' => $article->getVar('blog_id'), 'title' => ''],
-        'title'   => $article->getVar('art_title'),
-        'time'    => $article->getTime(),
-        'content' => $article->getVar('art_content')
-    ];
-    $articles[]                            = $_article;
-    $blogs_id[$article->getVar('blog_id')] = 1;
-    unset($_article);
+	$articles[]                            = [
+		'id'      => $id,
+		'blog'    => ['id' => $article->getVar('blog_id'), 'title' => ''],
+		'title'   => $article->getVar('art_title'),
+		'time'    => $article->getTime(),
+		'content' => $article->getVar('art_content')
+	];
+	$articles[]                            = $_article;
+	$blogs_id[$article->getVar('blog_id')] = 1;
+	unset($_article);
 }
 $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']];
+	$articles[$key]['blog']['title'] = $blogs[$articles[$key]['blog']['id']];
 }
 if ($blog_id > 0) {
-    $page['blog'] = $blogs[$blog_id];
+	$page['blog'] = $blogs[$blog_id];
 }
 
 if ($articles_count > $xoopsModuleConfig['articles_perpage']) {
-    include XOOPS_ROOT_PATH . '/class/pagenav.php';
-    $nav     = new XoopsPageNav($articles_count, $xoopsModuleConfig['articles_perpage'], $start, 'start', 'month=' . $month . '&amp;day=' . $day . '&amp;year=' . $year . '&amp;blog=' . (int)$blog_id);
-    $pagenav = $nav->renderNav(4);
+	include XOOPS_ROOT_PATH . '/class/pagenav.php';
+	$nav     = new XoopsPageNav($articles_count, $xoopsModuleConfig['articles_perpage'], $start, 'start', 'month=' . $month . '&amp;day=' . $day . '&amp;year=' . $year . '&amp;blog=' . (int)$blog_id);
+	$pagenav = $nav->renderNav(4);
 } else {
-    $pagenav = '';
+	$pagenav = '';
 }
 
 $timenav  = null;
 $calendar = null;
 $months   = null;
 if (empty($start)) {
-    if ($blog_id) {
-        $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
+	if ($blog_id) {
+		$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') . "
             WHERE YEAR(FROM_UNIXTIME(art_time - $timeoffset)) = $year
             " . $blog_criteria . '
             GROUP BY mon
             ';
-        $result = $xoopsDB->query($sql);
-        $months = [];
-        while ($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
-            ];
-        }
-        $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)
-        ];
-        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)
-            ];
-        }
-    } // Get daily list
-    elseif (empty($day)) {
-        $sql    = "SELECT DAY(FROM_UNIXTIME(art_time - $timeoffset)) AS day, COUNT(DISTINCT a.art_id) AS count
+		$result = $xoopsDB->query($sql);
+		$months = [];
+		while ($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
+			];
+		}
+		$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)
+		];
+		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)
+			];
+		}
+	} // 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') . "
             WHERE YEAR(FROM_UNIXTIME(art_time - $timeoffset)) = $year
             AND MONTH(FROM_UNIXTIME(art_time - $timeoffset)) = $month
             " . $blog_criteria . '
             GROUP BY day
             ';
-        $result = $xoopsDB->query($sql);
-        $days   = [];
-        while ($myrow = $xoopsDB->fetchArray($result)) {
-            $days[$myrow['day']]['count'] = $myrow['count'];
-        }
-        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
-            ];
-        }
-        $calendar   = planet_getCalendar($year, $month, $days);
-        $month_next = $month + 1;
-        $month_prev = $month - 1;
-        $_year      = $year;
-        if ($month == 12) {
-            $month_next = 1;
-            $_year      = $year + 1;
-        }
-        if ($month == 1) {
-            $month_pre = 12;
-            $_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)
-        ];
-        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)
-            ];
-        }
-    }
+		$result = $xoopsDB->query($sql);
+		$days   = [];
+		while ($myrow = $xoopsDB->fetchArray($result)) {
+			$days[$myrow['day']]['count'] = $myrow['count'];
+		}
+		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
+			];
+		}
+		$calendar   = planet_getCalendar($year, $month, $days);
+		$month_next = $month + 1;
+		$month_prev = $month - 1;
+		$_year      = $year;
+		if ($month == 12) {
+			$month_next = 1;
+			$_year      = $year + 1;
+		}
+		if ($month == 1) {
+			$month_pre = 12;
+			$_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)
+		];
+		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)
+			];
+		}
+	}
 }
 
 $xoopsTpl->assign('dirname', $GLOBALS['moddirname']);
@@ -222,66 +222,66 @@ 
  */
 function planet_getCalendar($year = null, $month = null, $days = null)
 {
-    $year      = empty($year) ? date('Y') : $year;
-    $month     = empty($month) ? date('n') : $month;
-    $unixmonth = mktime(0, 0, 0, $month, 1, $year);
+	$year      = empty($year) ? date('Y') : $year;
+	$month     = empty($month) ? date('n') : $month;
+	$unixmonth = mktime(0, 0, 0, $month, 1, $year);
 
-    ob_start();
-    echo '<table id="calendar">';
-    echo '<caption>';
-    printf(planet_constant('MD_TIME_YM'), $year, planet_constant('MD_MONTH_' . $month));
-    echo '</caption>';
+	ob_start();
+	echo '<table id="calendar">';
+	echo '<caption>';
+	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>';
+	echo '<tr>';
 
-    // See how much we should pad in the beginning
-    $week_begins = 1;
-    $pad         = planet_calendar_week_mod(date('w', $unixmonth) - $week_begins);
-    if (0 != $pad) {
-        echo "\n\t\t" . '<td colspan="' . $pad . '">&nbsp;</td>';
-    }
+	// See how much we should pad in the beginning
+	$week_begins = 1;
+	$pad         = planet_calendar_week_mod(date('w', $unixmonth) - $week_begins);
+	if (0 != $pad) {
+		echo "\n\t\t" . '<td colspan="' . $pad . '">&nbsp;</td>';
+	}
 
-    $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";
-        }
-        $newrow = false;
+	$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";
+		}
+		$newrow = false;
 
-        echo '<td>';
+		echo '<td>';
 
-        if (!empty($days[$day]['url'])) {
-            echo '<a href="' . $days[$day]['url'] . "\"";
-            if (!empty($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 $day;
-        }
-        echo '</td>';
+		if (!empty($days[$day]['url'])) {
+			echo '<a href="' . $days[$day]['url'] . "\"";
+			if (!empty($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 $day;
+		}
+		echo '</td>';
 
-        if (6 == planet_calendar_week_mod(date('w', mktime(0, 0, 0, $month, $day, $year)) - $week_begins)) {
-            $newrow = true;
-        }
-    }
+		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);
-    if ($pad != 0 && $pad != 7) {
-        echo "\n\t\t" . '<td class="pad" colspan="' . $pad . '">&nbsp;</td>';
-    }
+	$pad = 7 - planet_calendar_week_mod(date('w', mktime(0, 0, 0, $month, $day, $year)) - $week_begins);
+	if ($pad != 0 && $pad != 7) {
+		echo "\n\t\t" . '<td class="pad" colspan="' . $pad . '">&nbsp;</td>';
+	}
 
-    echo "\n\t</tr>\n\t</tbody>\n\t</table>";
-    $calendar = ob_get_contents();
-    ob_end_clean();
+	echo "\n\t</tr>\n\t</tbody>\n\t</table>";
+	$calendar = ob_get_contents();
+	ob_end_clean();
 
-    return $calendar;
+	return $calendar;
 }
 
 // Used in get_calendar
@@ -291,7 +291,7 @@ 
  */
 function planet_calendar_week_mod($num)
 {
-    $base = 7;
+	$base = 7;
 
-    return ($num - $base * floor($num / $base));
+	return ($num - $base * floor($num / $base));
 }

Spacing +59 added lines, -59 removed lines

@@ -26,7 +26,7 @@ 
 // ------------------------------------------------------------------------ //
 use Xmf\Request;
 
-include __DIR__ . '/header.php';
+include __DIR__.'/header.php';
 
 if (PlanetUtility::planetParseArguments($args_num, $args, $args_str)) {
     $args['year']  = @$args_num[0];
@@ -45,26 +45,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();
@@ -87,7 +87,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'),
@@ -98,20 +98,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 > $xoopsModuleConfig['articles_perpage']) {
-    include XOOPS_ROOT_PATH . '/class/pagenav.php';
-    $nav     = new XoopsPageNav($articles_count, $xoopsModuleConfig['articles_perpage'], $start, 'start', 'month=' . $month . '&amp;day=' . $day . '&amp;year=' . $year . '&amp;blog=' . (int)$blog_id);
+if ($articles_count>$xoopsModuleConfig['articles_perpage']) {
+    include XOOPS_ROOT_PATH.'/class/pagenav.php';
+    $nav     = new XoopsPageNav($articles_count, $xoopsModuleConfig['articles_perpage'], $start, 'start', 'month='.$month.'&amp;day='.$day.'&amp;year='.$year.'&amp;blog='.(int) $blog_id);
     $pagenav = $nav->renderNav(4);
-} else {
+}else {
     $pagenav = '';
 }
 
@@ -120,43 +120,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 ($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);
@@ -164,35 +164,35 @@ 
         while ($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 ($month == 12) {
             $month_next = 1;
-            $_year      = $year + 1;
+            $_year      = $year+1;
         }
         if ($month == 1) {
             $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)
             ];
         }
     }
@@ -212,7 +212,7 @@ 
 $xoopsTpl->assign('timenav', $timenav);
 $xoopsTpl->assign('pagenav', $pagenav);
 
-require_once __DIR__ . '/footer.php';
+require_once __DIR__.'/footer.php';
 
 /**
  * @param null $year
@@ -229,24 +229,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";
         }
@@ -255,26 +255,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 ($pad != 0 && $pad != 7) {
-        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>";
@@ -293,5 +293,5 @@ 
 {
     $base = 7;
 
-    return ($num - $base * floor($num / $base));
+    return ($num-$base*floor($num/$base));
 }

transfer.php

Indentation +56 added lines, -56 removed lines

@@ -17,8 +17,8 @@ 
 include __DIR__ . '/header.php';
 
 if (PlanetUtility::planetParseArguments($args_num, $args, $args_str)) {
-    $args['article'] = @$args_num[0];
-    $args['op']      = @$args_str[0];
+	$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']);
@@ -27,33 +27,33 @@ 
 $op = strtolower(trim($op));
 
 if (empty($article_id)) {
-    if (empty(Request::getUrl('HTTP_REFERER', '', 'SERVER'))) {
-        //$_SERVER['HTTP_REFERER']))
+	if (empty(Request::getUrl('HTTP_REFERER', '', 'SERVER'))) {
+		//$_SERVER['HTTP_REFERER']))
 
-        include XOOPS_ROOT_PATH . '/header.php';
-        xoops_error(_NOPERM);
-        $xoopsOption['output_type'] = 'plain';
-        include XOOPS_ROOT_PATH . '/footer.php';
-        exit();
-    } 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']);
-        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';
-            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';
-            exit();
-        } else {
-            include XOOPS_ROOT_PATH . '/header.php';
-            xoops_error(_NOPERM);
-            $xoopsOption['output_type'] = 'plain';
-            include XOOPS_ROOT_PATH . '/footer.php';
-            exit();
-        }
-    }
+		include XOOPS_ROOT_PATH . '/header.php';
+		xoops_error(_NOPERM);
+		$xoopsOption['output_type'] = 'plain';
+		include XOOPS_ROOT_PATH . '/footer.php';
+		exit();
+	} 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']);
+		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';
+			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';
+			exit();
+		} else {
+			include XOOPS_ROOT_PATH . '/header.php';
+			xoops_error(_NOPERM);
+			$xoopsOption['output_type'] = 'plain';
+			include XOOPS_ROOT_PATH . '/footer.php';
+			exit();
+		}
+	}
 }
 
 $articleHandler = xoops_getModuleHandler('article', $GLOBALS['moddirname']);
@@ -61,37 +61,37 @@ 
 
 // 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';
-    exit();
+	$module_variables .= "<input type=\"hidden\" name=\"article\" id=\"article\" value=\"{$article_id}\">";
+	include XOOPS_ROOT_PATH . '/Frameworks/transfer/option.transfer.php';
+	exit();
 } 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['author'] = $article_obj->getVar('art_author');
+	$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['author'] = $article_obj->getVar('art_author');
 
-    switch ($op) {
+	switch ($op) {
 
-        // Use title
-        case 'bookmark':
-            break;
+		// Use title
+		case 'bookmark':
+			break;
 
-        case 'print':
-        case 'pdf':
-            ${"{$op}_data"}            =& $data;
-            ${"{$op}_data"}['date']    = $pdf_data['time'];
-            ${"{$op}_data"}['content'] = $article_obj->getVar('art_content');
-            break;
+		case 'print':
+		case 'pdf':
+			${"{$op}_data"}            =& $data;
+			${"{$op}_data"}['date']    = $pdf_data['time'];
+			${"{$op}_data"}['content'] = $article_obj->getVar('art_content');
+			break;
 
-        case 'newbb':
-        default:
-            $data['content'] = $article_obj->getSummary();
-            break;
-    }
-    include XOOPS_ROOT_PATH . '/Frameworks/transfer/action.transfer.php';
-    exit();
+		case 'newbb':
+		default:
+			$data['content'] = $article_obj->getSummary();
+			break;
+	}
+	include XOOPS_ROOT_PATH . '/Frameworks/transfer/action.transfer.php';
+	exit();
 }

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

action.blog.php

Indentation +107 added lines, -107 removed lines

@@ -33,15 +33,15 @@ 
 $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);
+	redirect_header('index.php', 2, _NOPERM);
 }
 
 if ($op === 'save' && !empty(Request::getString('fetch', '', 'POST'))) {//!empty($_POST['fetch'])) {
-    $op = 'edit';
+	$op = 'edit';
 }
 
 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';
@@ -50,110 +50,110 @@ 
 $categoryHandler = xoops_getModuleHandler('category', $GLOBALS['moddirname']);
 
 switch ($op) {
-    /* save a single blog */
-    case 'save':
-
-        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']);
-            }
-        } 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'));
-            }
-
-            $blog_obj = $blogHandler->create();
-            $blog_obj->setVar('blog_submitter', is_object($xoopsUser) ? $xoopsUser->getVar('uid') : PlanetUtility::planetGetIP(true));
-
-            switch ($xoopsModuleConfig['newblog_submit']) {
-                case 2:
-                    if (!is_object($xoopsUser)) {
-                        $status = 0;
-                    } else {
-                        $status = 1;
-                    }
-                    break;
-                case 0:
-                case 3:
-                    $status = 1;
-                    break;
-                case 1:
-                default:
-                    if (!is_object($xoopsUser) || !$xoopsUser->isAdmin()) {
-                        $status = 0;
-                    } else {
-                        $status = 1;
-                    }
-                    break;
-            }
-
-            $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']);
-        if ($blog_obj->isNew()) {
-            $blog_obj->setVar('blog_submitter', is_object($xoopsUser) ? $xoopsUser->getVar('uid') : PlanetUtility::planetGetIP(true));
-        }
-
-        if (!$blogHandler->insert($blog_obj)) {
-        } elseif (0 !== count(Request::getArray('categories', [], 'POST'))) {
-            $blog_id = $blog_obj->getVar('blog_id');
-            if (in_array(0, $_POST['categories'])) {
-                $_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);
-
-    /* edit a single blog */
-    // no break
-    case 'edit':
-    default:
-        if (!empty(Request::getString('fetch', '', 'POST'))) {
-            $blog_obj = $blogHandler->fetch(Request::getText('blog_feed', '', 'POST'));
-            $blog_obj->setVar('blog_id', $blog_id);
-        } else {
-            $blog_obj = $blogHandler->get($blog_id);
-        }
-        $categories = Request::getArray('categories', [], 'POST');//isset($_POST['categories']) ? $_POST['categories'] : array();
-        if (in_array('-1', $categories)) {
-            $categories = [];
-        }
-        if (empty($categories) && $blog_id > 0) {
-            $crit       = new Criteria('bc.blog_id', $blog_id);
-            $categories = array_keys($categoryHandler->getByBlog($crit));
-        }
-        if (empty($categories)) {
-            $categories = [0 => _NONE];
-        }
-
-        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');
-                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 '</div>';
-                unset($blogs_obj, $criteria);
-            }
-        }
-        include XOOPS_ROOT_PATH . '/modules/' . $GLOBALS['moddirname'] . '/include/form.blog.php';
-        echo '</fieldset>';
-        break;
+	/* save a single blog */
+	case 'save':
+
+		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']);
+			}
+		} 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'));
+			}
+
+			$blog_obj = $blogHandler->create();
+			$blog_obj->setVar('blog_submitter', is_object($xoopsUser) ? $xoopsUser->getVar('uid') : PlanetUtility::planetGetIP(true));
+
+			switch ($xoopsModuleConfig['newblog_submit']) {
+				case 2:
+					if (!is_object($xoopsUser)) {
+						$status = 0;
+					} else {
+						$status = 1;
+					}
+					break;
+				case 0:
+				case 3:
+					$status = 1;
+					break;
+				case 1:
+				default:
+					if (!is_object($xoopsUser) || !$xoopsUser->isAdmin()) {
+						$status = 0;
+					} else {
+						$status = 1;
+					}
+					break;
+			}
+
+			$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']);
+		if ($blog_obj->isNew()) {
+			$blog_obj->setVar('blog_submitter', is_object($xoopsUser) ? $xoopsUser->getVar('uid') : PlanetUtility::planetGetIP(true));
+		}
+
+		if (!$blogHandler->insert($blog_obj)) {
+		} elseif (0 !== count(Request::getArray('categories', [], 'POST'))) {
+			$blog_id = $blog_obj->getVar('blog_id');
+			if (in_array(0, $_POST['categories'])) {
+				$_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);
+
+	/* edit a single blog */
+	// no break
+	case 'edit':
+	default:
+		if (!empty(Request::getString('fetch', '', 'POST'))) {
+			$blog_obj = $blogHandler->fetch(Request::getText('blog_feed', '', 'POST'));
+			$blog_obj->setVar('blog_id', $blog_id);
+		} else {
+			$blog_obj = $blogHandler->get($blog_id);
+		}
+		$categories = Request::getArray('categories', [], 'POST');//isset($_POST['categories']) ? $_POST['categories'] : array();
+		if (in_array('-1', $categories)) {
+			$categories = [];
+		}
+		if (empty($categories) && $blog_id > 0) {
+			$crit       = new Criteria('bc.blog_id', $blog_id);
+			$categories = array_keys($categoryHandler->getByBlog($crit));
+		}
+		if (empty($categories)) {
+			$categories = [0 => _NONE];
+		}
+
+		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');
+				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 '</div>';
+				unset($blogs_obj, $criteria);
+			}
+		}
+		include XOOPS_ROOT_PATH . '/modules/' . $GLOBALS['moddirname'] . '/include/form.blog.php';
+		echo '</fieldset>';
+		break;
 }
 
 include XOOPS_ROOT_PATH . '/footer.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

Indentation +127 added lines, -127 removed lines

@@ -30,16 +30,16 @@ 
 include __DIR__ . '/include/vars.php';
 
 $modversion                 = [
-    'name'          => planet_constant('MI_NAME'),
-    'version'       => 2.10,
-    'module_status' => 'Final',
-    'release_date'  => '2016/08/01',
-    'description'   => planet_constant('MI_DESC'),
-    'author'        => 'D.J. (phppp)',
-    'credits'       => 'XOOPS Project',
-    'image'         => 'assets/images/logoModule.png',
-    'dirname'       => $GLOBALS['moddirname'],
-    'help'          => XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/readme.html'
+	'name'          => planet_constant('MI_NAME'),
+	'version'       => 2.10,
+	'module_status' => 'Final',
+	'release_date'  => '2016/08/01',
+	'description'   => planet_constant('MI_DESC'),
+	'author'        => 'D.J. (phppp)',
+	'credits'       => 'XOOPS Project',
+	'image'         => 'assets/images/logoModule.png',
+	'dirname'       => $GLOBALS['moddirname'],
+	'help'          => XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/readme.html'
 ];
 $modversion['help']         = 'page=help';
 $modversion['license']      = 'GNU see LICENSE';
@@ -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
@@ -149,162 +149,162 @@ 
 // Configs
 // Config items
 $modversion['config'][] = [
-    'name'        => 'do_debug',
-    'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_DODEBUG',
-    'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_DODEBUG_DESC',
-    'formtype'    => 'yesno',
-    'valuetype'   => 'int',
-    'default'     => 1
+	'name'        => 'do_debug',
+	'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_DODEBUG',
+	'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_DODEBUG_DESC',
+	'formtype'    => 'yesno',
+	'valuetype'   => 'int',
+	'default'     => 1
 ];
 
 $modversion['config'][] = [
-    'name'        => 'do_urw',
-    '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'])
+	'name'        => 'do_urw',
+	'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'])
 ];
 
 $modversion['config'][] = [
-    'name'        => 'theme_set',
-    'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_THEMESET',
-    'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_THEMESET_DESC',
-    'formtype'    => 'select',
-    'valuetype'   => 'text',
-    'options'     => [_NONE => '0'],
-    'default'     => ''
+	'name'        => 'theme_set',
+	'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_THEMESET',
+	'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_THEMESET_DESC',
+	'formtype'    => 'select',
+	'valuetype'   => 'text',
+	'options'     => [_NONE => '0'],
+	'default'     => ''
 ];
 
 $modversion['config'][] = [
-    'name'        => 'timeformat',
-    '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'
-    ],
-    'default'     => 'c'
+	'name'        => 'timeformat',
+	'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'
+	],
+	'default'     => 'c'
 ];
 
 $modversion['config'][] = [
-    'name'        => 'articles_perpage',
-    'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_ARTICLESPERPAGE',
-    'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_ARTICLESPERPAGE_DESC',
-    'formtype'    => 'textbox',
-    'valuetype'   => 'int',
-    'default'     => 10
+	'name'        => 'articles_perpage',
+	'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_ARTICLESPERPAGE',
+	'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_ARTICLESPERPAGE_DESC',
+	'formtype'    => 'textbox',
+	'valuetype'   => 'int',
+	'default'     => 10
 ];
 
 $modversion['config'][] = [
-    'name'        => 'list_perpage',
-    'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_LISTPERPAGE',
-    'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_LISTPERPAGE_DESC',
-    'formtype'    => 'textbox',
-    'valuetype'   => 'int',
-    'default'     => 20
+	'name'        => 'list_perpage',
+	'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_LISTPERPAGE',
+	'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_LISTPERPAGE_DESC',
+	'formtype'    => 'textbox',
+	'valuetype'   => 'int',
+	'default'     => 20
 ];
 
 $modversion['config'][] = [
-    'name'        => 'blogs_perupdate',
-    'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_BLOGSPERUPDATE',
-    'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_BLOGSPERUPDATE_DESC',
-    'formtype'    => 'textbox',
-    'valuetype'   => 'int',
-    'default'     => 10
+	'name'        => 'blogs_perupdate',
+	'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_BLOGSPERUPDATE',
+	'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_BLOGSPERUPDATE_DESC',
+	'formtype'    => 'textbox',
+	'valuetype'   => 'int',
+	'default'     => 10
 ];
 
 $modversion['config'][] = [
-    'name'        => 'article_expire',
-    'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_ARTICLE_EXPIRE',
-    'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_ARTICLE_EXPIRE_DESC',
-    'formtype'    => 'textbox',
-    'valuetype'   => 'int',
-    'default'     => 30
+	'name'        => 'article_expire',
+	'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_ARTICLE_EXPIRE',
+	'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_ARTICLE_EXPIRE_DESC',
+	'formtype'    => 'textbox',
+	'valuetype'   => 'int',
+	'default'     => 30
 ];
 
 $modversion['config'][] = [
-    'name'        => 'display_summary',
-    'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_DISPLAY_SUMMARY',
-    'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_DISPLAY_SUMMARY_DESC',
-    'formtype'    => 'textbox',
-    'valuetype'   => 'int',
-    'default'     => 0
+	'name'        => 'display_summary',
+	'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_DISPLAY_SUMMARY',
+	'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_DISPLAY_SUMMARY_DESC',
+	'formtype'    => 'textbox',
+	'valuetype'   => 'int',
+	'default'     => 0
 ];
 
 $modversion['config'][] = [
-    'name'        => 'do_sibling',
-    'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_DOSIBLING',
-    'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_DOSIBLING_DESC',
-    'formtype'    => 'yesno',
-    'valuetype'   => 'int',
-    'default'     => 1
+	'name'        => 'do_sibling',
+	'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_DOSIBLING',
+	'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_DOSIBLING_DESC',
+	'formtype'    => 'yesno',
+	'valuetype'   => 'int',
+	'default'     => 1
 ];
 
 $modversion['config'][] = [
-    'name'        => 'pings',
-    'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_PING',
-    'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_PING_DESC',
-    'formtype'    => 'textarea',
-    'valuetype'   => 'text',
-    'default'     => ''
+	'name'        => 'pings',
+	'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_PING',
+	'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_PING_DESC',
+	'formtype'    => 'textarea',
+	'valuetype'   => 'text',
+	'default'     => ''
 ];
 
 $modversion['config'][] = [
-    'name'        => 'trackback_option',
-    'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_TRACKBACK_OPTION',
-    'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_TRACKBACK_OPTION_DESC',
-    'formtype'    => 'select',
-    'valuetype'   => 'int',
-    'default'     => 0,
-    'options'     => [planet_constant('MI_MODERATION') => 0, _ALL => 1, _NONE => 2]
+	'name'        => 'trackback_option',
+	'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_TRACKBACK_OPTION',
+	'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_TRACKBACK_OPTION_DESC',
+	'formtype'    => 'select',
+	'valuetype'   => 'int',
+	'default'     => 0,
+	'options'     => [planet_constant('MI_MODERATION') => 0, _ALL => 1, _NONE => 2]
 ];
 
 $modversion['config'][] = [
-    'name'        => 'copyright',
-    'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_COPYRIGHT',
-    'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_COPYRIGHT_DESC',
-    'formtype'    => 'textbox',
-    'valuetype'   => 'text',
-    'default'     => 'Copyright© %s & ' . $xoopsConfig['sitename']
+	'name'        => 'copyright',
+	'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_COPYRIGHT',
+	'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_COPYRIGHT_DESC',
+	'formtype'    => 'textbox',
+	'valuetype'   => 'text',
+	'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',
-    'formtype'    => 'select',
-    'valuetype'   => 'int',
-    'default'     => 2,
-    'options'     => [
-        _NONE                            => 0,
-        planet_constant('MI_MODERATION') => 1,
-        planet_constant('MI_MEMBER')     => 2,
-        _ALL                             => 3
-    ]
-    // 0 - Only admin; 1 - all but need approval; 2 - members auto approved; 3 - all
+	'name'        => 'newblog_submit',
+	'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_NEWBLOG_SUBMIT',
+	'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_NEWBLOG_SUBMIT_DESC',
+	'formtype'    => 'select',
+	'valuetype'   => 'int',
+	'default'     => 2,
+	'options'     => [
+		_NONE                            => 0,
+		planet_constant('MI_MODERATION') => 1,
+		planet_constant('MI_MEMBER')     => 2,
+		_ALL                             => 3
+	]
+	// 0 - Only admin; 1 - all but need approval; 2 - members auto approved; 3 - all
 ];
 
 $modversion['config'][] = [
-    'name'        => 'anonymous_rate',
-    'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_ANONYMOUSRATE',
-    'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_ANONYMOUSRATE_DESC',
-    'formtype'    => 'yesno',
-    'valuetype'   => 'int',
-    'default'     => 0
+	'name'        => 'anonymous_rate',
+	'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_ANONYMOUSRATE',
+	'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_ANONYMOUSRATE_DESC',
+	'formtype'    => 'yesno',
+	'valuetype'   => 'int',
+	'default'     => 0
 ];
 
 $modversion['config'][] = [
-    'name'        => 'do_pseudocron',
-    'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_PSEUDOCRON',
-    'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_PSEUDOCRON_DESC',
-    'formtype'    => 'yesno',
-    'valuetype'   => 'int',
-    'default'     => 1
+	'name'        => 'do_pseudocron',
+	'title'       => $GLOBALS['VAR_PREFIXU'] . '_MI_PSEUDOCRON',
+	'description' => $GLOBALS['VAR_PREFIXU'] . '_MI_PSEUDOCRON_DESC',
+	'formtype'    => 'yesno',
+	'valuetype'   => 'int',
+	'default'     => 1
 ];
 
 // Notification

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;

trackback.php

Indentation +142 added lines, -142 removed lines

@@ -38,159 +38,159 @@
 $charset    = trim(Request::getString('charset', '', 'POST'));//trim($_POST['charset']);
 
 if (empty($xoopsModuleConfig['trackback_option'])) {
-    PlanetUtility::planetRespondToTrackback(1, 'Trackback is closed');
+	PlanetUtility::planetRespondToTrackback(1, 'Trackback is closed');
 }
 if (!strlen($title . $url . $blog_name)) {
-    PlanetUtility::planetRespondToTrackback(1, planet_constant('MD_INVALID'));
+	PlanetUtility::planetRespondToTrackback(1, planet_constant('MD_INVALID'));
 }
 
 if (!empty($article_id) && !empty($url)) {
-    $trackbackHandler = xoops_getModuleHandler('trackback', $GLOBALS['moddirname']);
-    $criteria         = new CriteriaCompo(new Criteria('art_id', $article_id));
-    $criteria->add(new Criteria('tb_url', $url));
-    if ($trackbackHandler->getCount($criteria) > 0) {
-        PlanetUtility::planetRespondToTrackback(1, 'We already have a ping from that URI for this article.');
-    }
+	$trackbackHandler = xoops_getModuleHandler('trackback', $GLOBALS['moddirname']);
+	$criteria         = new CriteriaCompo(new Criteria('art_id', $article_id));
+	$criteria->add(new Criteria('tb_url', $url));
+	if ($trackbackHandler->getCount($criteria) > 0) {
+		PlanetUtility::planetRespondToTrackback(1, 'We already have a ping from that URI for this article.');
+	}
 
-    $charset   = empty($charset) ? 'utf-8' : $charset;
-    $title     = XoopsLocal::convert_encoding($title, _CHARSET, $charset);
-    $excerpt   = XoopsLocal::convert_encoding($excerpt, _CHARSET, $charset);
-    $blog_name = XoopsLocal::convert_encoding($blog_name, _CHARSET, $charset);
-    $tb_status = (int)$xoopsModuleConfig['trackback_option'];
+	$charset   = empty($charset) ? 'utf-8' : $charset;
+	$title     = XoopsLocal::convert_encoding($title, _CHARSET, $charset);
+	$excerpt   = XoopsLocal::convert_encoding($excerpt, _CHARSET, $charset);
+	$blog_name = XoopsLocal::convert_encoding($blog_name, _CHARSET, $charset);
+	$tb_status = (int)$xoopsModuleConfig['trackback_option'];
 
-    $com_pid    = 0;
-    $com_itemid = $article_id;
-    $com_rootid = 0;
-    $com_title  = $title;
-    $com_text   = $excerpt;
-    $com_text   .= "\n\n[TRACKBACK]" . _POSTEDBY . ': ';
-    if (!empty($url)) {
-        $com_text .= '[url=' . $url . ']' . $blog_name . '[/url]';
-    } else {
-        $com_text .= $blog_name;
-    }
-    $com_modid = $xoopsModule->getVar('mid');
+	$com_pid    = 0;
+	$com_itemid = $article_id;
+	$com_rootid = 0;
+	$com_title  = $title;
+	$com_text   = $excerpt;
+	$com_text   .= "\n\n[TRACKBACK]" . _POSTEDBY . ': ';
+	if (!empty($url)) {
+		$com_text .= '[url=' . $url . ']' . $blog_name . '[/url]';
+	} else {
+		$com_text .= $blog_name;
+	}
+	$com_modid = $xoopsModule->getVar('mid');
 
-    $commentHandler = xoops_getHandler('comment');
-    $comment        = $commentHandler->create();
-    $comment->setVar('com_created', time());
-    $comment->setVar('com_pid', $com_pid);
-    $comment->setVar('com_itemid', $com_itemid);
-    $comment->setVar('com_rootid', $com_rootid);
-    $comment->setVar('com_ip', xoops_getenv('REMOTE_ADDR'));
-    switch ($tb_status) {
-        case 2:
-            $comment->setVar('com_status', 2);
-            $call_approvefunc = true;
-            $call_updatefunc  = true;
-            $notify_event     = 'comment';
-            break;
-        case 1:
-        default:
-            $comment->setVar('com_status', 1);
-            $notify_event = 'comment_submit';
-            break;
-    }
-    $comment->setVar('com_uid', 0);
-    $com_title = xoops_trim($com_title);
-    $com_title = empty($com_title) ? _NOTITLE : $com_title;
-    $comment->setVar('com_title', $com_title);
-    $comment->setVar('com_text', $com_text);
-    $comment->setVar('dohtml', 0);
-    $comment->setVar('dosmiley', 0);
-    $comment->setVar('doxcode', 1);
-    $comment->setVar('doimage', 0);
-    $comment->setVar('dobr', 1);
-    $comment->setVar('com_icon', '');
-    $comment->setVar('com_modified', time());
-    $comment->setVar('com_modid', $com_modid);
-    if (false != $commentHandler->insert($comment)) {
-        $newcid = $comment->getVar('com_id');
+	$commentHandler = xoops_getHandler('comment');
+	$comment        = $commentHandler->create();
+	$comment->setVar('com_created', time());
+	$comment->setVar('com_pid', $com_pid);
+	$comment->setVar('com_itemid', $com_itemid);
+	$comment->setVar('com_rootid', $com_rootid);
+	$comment->setVar('com_ip', xoops_getenv('REMOTE_ADDR'));
+	switch ($tb_status) {
+		case 2:
+			$comment->setVar('com_status', 2);
+			$call_approvefunc = true;
+			$call_updatefunc  = true;
+			$notify_event     = 'comment';
+			break;
+		case 1:
+		default:
+			$comment->setVar('com_status', 1);
+			$notify_event = 'comment_submit';
+			break;
+	}
+	$comment->setVar('com_uid', 0);
+	$com_title = xoops_trim($com_title);
+	$com_title = empty($com_title) ? _NOTITLE : $com_title;
+	$comment->setVar('com_title', $com_title);
+	$comment->setVar('com_text', $com_text);
+	$comment->setVar('dohtml', 0);
+	$comment->setVar('dosmiley', 0);
+	$comment->setVar('doxcode', 1);
+	$comment->setVar('doimage', 0);
+	$comment->setVar('dobr', 1);
+	$comment->setVar('com_icon', '');
+	$comment->setVar('com_modified', time());
+	$comment->setVar('com_modid', $com_modid);
+	if (false != $commentHandler->insert($comment)) {
+		$newcid = $comment->getVar('com_id');
 
-        // set own id as root id
-        $com_rootid = $newcid;
-        if (!$commentHandler->updateByField($comment, 'com_rootid', $com_rootid)) {
-            $commentHandler->delete($comment);
-            PlanetUtility::planetRespondToTrackback(1, xoops_error());
-        }
+		// set own id as root id
+		$com_rootid = $newcid;
+		if (!$commentHandler->updateByField($comment, 'com_rootid', $com_rootid)) {
+			$commentHandler->delete($comment);
+			PlanetUtility::planetRespondToTrackback(1, xoops_error());
+		}
 
-        // call custom approve function if any
-        if (false != $call_approvefunc && isset($comment_config['callback']['approve'])
-            && trim($comment_config['callback']['approve']) != '') {
-            $skip = false;
-            if (!function_exists($comment_config['callback']['approve'])) {
-                if (isset($comment_config['callbackFile'])) {
-                    $callbackfile = trim($comment_config['callbackFile']);
-                    if ($callbackfile != ''
-                        && file_exists(XOOPS_ROOT_PATH . '/modules/' . $moddir . '/' . $callbackfile)) {
-                        require_once XOOPS_ROOT_PATH . '/modules/' . $moddir . '/' . $callbackfile;
-                    }
-                    if (!function_exists($comment_config['callback']['approve'])) {
-                        $skip = true;
-                    }
-                } else {
-                    $skip = true;
-                }
-            }
-            if (!$skip) {
-                $comment_config['callback']['approve']($comment);
-            }
-        }
+		// call custom approve function if any
+		if (false != $call_approvefunc && isset($comment_config['callback']['approve'])
+			&& trim($comment_config['callback']['approve']) != '') {
+			$skip = false;
+			if (!function_exists($comment_config['callback']['approve'])) {
+				if (isset($comment_config['callbackFile'])) {
+					$callbackfile = trim($comment_config['callbackFile']);
+					if ($callbackfile != ''
+						&& file_exists(XOOPS_ROOT_PATH . '/modules/' . $moddir . '/' . $callbackfile)) {
+						require_once XOOPS_ROOT_PATH . '/modules/' . $moddir . '/' . $callbackfile;
+					}
+					if (!function_exists($comment_config['callback']['approve'])) {
+						$skip = true;
+					}
+				} else {
+					$skip = true;
+				}
+			}
+			if (!$skip) {
+				$comment_config['callback']['approve']($comment);
+			}
+		}
 
-        // call custom update function if any
-        if (false != $call_updatefunc && isset($comment_config['callback']['update'])
-            && trim($comment_config['callback']['update']) != '') {
-            $skip = false;
-            if (!function_exists($comment_config['callback']['update'])) {
-                if (isset($comment_config['callbackFile'])) {
-                    $callbackfile = trim($comment_config['callbackFile']);
-                    if ($callbackfile != ''
-                        && file_exists(XOOPS_ROOT_PATH . '/modules/' . $moddir . '/' . $callbackfile)) {
-                        require_once XOOPS_ROOT_PATH . '/modules/' . $moddir . '/' . $callbackfile;
-                    }
-                    if (!function_exists($comment_config['callback']['update'])) {
-                        $skip = true;
-                    }
-                } else {
-                    $skip = true;
-                }
-            }
-            if (!$skip) {
-                $criteria = new CriteriaCompo(new Criteria('com_modid', $com_modid));
-                $criteria->add(new Criteria('com_itemid', $com_itemid));
-                $criteria->add(new Criteria('com_status', XOOPS_COMMENT_ACTIVE));
-                $comment_count = $commentHandler->getCount($criteria);
-                $func          = $comment_config['callback']['update'];
-                call_user_func_array($func, [$com_itemid, $comment_count, $comment->getVar('com_id')]);
-            }
-        }
+		// call custom update function if any
+		if (false != $call_updatefunc && isset($comment_config['callback']['update'])
+			&& trim($comment_config['callback']['update']) != '') {
+			$skip = false;
+			if (!function_exists($comment_config['callback']['update'])) {
+				if (isset($comment_config['callbackFile'])) {
+					$callbackfile = trim($comment_config['callbackFile']);
+					if ($callbackfile != ''
+						&& file_exists(XOOPS_ROOT_PATH . '/modules/' . $moddir . '/' . $callbackfile)) {
+						require_once XOOPS_ROOT_PATH . '/modules/' . $moddir . '/' . $callbackfile;
+					}
+					if (!function_exists($comment_config['callback']['update'])) {
+						$skip = true;
+					}
+				} else {
+					$skip = true;
+				}
+			}
+			if (!$skip) {
+				$criteria = new CriteriaCompo(new Criteria('com_modid', $com_modid));
+				$criteria->add(new Criteria('com_itemid', $com_itemid));
+				$criteria->add(new Criteria('com_status', XOOPS_COMMENT_ACTIVE));
+				$comment_count = $commentHandler->getCount($criteria);
+				$func          = $comment_config['callback']['update'];
+				call_user_func_array($func, [$com_itemid, $comment_count, $comment->getVar('com_id')]);
+			}
+		}
 
-        // RMV-NOTIFY
-        // trigger notification event if necessary
-        if ($notify_event) {
-            $not_modid = $com_modid;
-            //            require_once XOOPS_ROOT_PATH . '/include/notification_functions.php';
-            $not_catinfo  = notificationCommentCategoryInfo($not_modid);
-            $not_category = $not_catinfo['name'];
-            $not_itemid   = $com_itemid;
-            $not_event    = $notify_event;
-            // Build an ABSOLUTE URL to view the comment.  Make sure we
-            // point to a viewable page (i.e. not the system administration
-            // module).
-            $comment_tags = [];
-            $not_module   = $xoopsModule;
-            if (!isset($comment_url)) {
-                $com_config  = $not_module->getInfo('comments');
-                $comment_url = $com_config['pageName'] . '?';
-                $comment_url .= $com_config['itemName'];
-            }
-            $comment_tags['X_COMMENT_URL'] = XOOPS_URL . '/modules/' . $not_module->getVar('dirname') . '/' . $comment_url . '=' . $com_itemid . '&com_id=' . $newcid . '&com_rootid=' . $com_rootid . '&com_mode=' . $com_mode . '&com_order=' . $com_order . '#comment' . $newcid;
-            $notificationHandler           = xoops_getHandler('notification');
-            $notificationHandler->triggerEvent($not_category, $not_itemid, $not_event, $comment_tags, false, $not_modid);
-        }
+		// RMV-NOTIFY
+		// trigger notification event if necessary
+		if ($notify_event) {
+			$not_modid = $com_modid;
+			//            require_once XOOPS_ROOT_PATH . '/include/notification_functions.php';
+			$not_catinfo  = notificationCommentCategoryInfo($not_modid);
+			$not_category = $not_catinfo['name'];
+			$not_itemid   = $com_itemid;
+			$not_event    = $notify_event;
+			// Build an ABSOLUTE URL to view the comment.  Make sure we
+			// point to a viewable page (i.e. not the system administration
+			// module).
+			$comment_tags = [];
+			$not_module   = $xoopsModule;
+			if (!isset($comment_url)) {
+				$com_config  = $not_module->getInfo('comments');
+				$comment_url = $com_config['pageName'] . '?';
+				$comment_url .= $com_config['itemName'];
+			}
+			$comment_tags['X_COMMENT_URL'] = XOOPS_URL . '/modules/' . $not_module->getVar('dirname') . '/' . $comment_url . '=' . $com_itemid . '&com_id=' . $newcid . '&com_rootid=' . $com_rootid . '&com_mode=' . $com_mode . '&com_order=' . $com_order . '#comment' . $newcid;
+			$notificationHandler           = xoops_getHandler('notification');
+			$notificationHandler->triggerEvent($not_category, $not_itemid, $not_event, $comment_tags, false, $not_modid);
+		}
 
-        PlanetUtility::planetRespondToTrackback(0);
-    } else {
-        PlanetUtility::planetRespondToTrackback(1, xoops_error($comment->getHtmlErrors()));
-    }
+		PlanetUtility::planetRespondToTrackback(0);
+	} else {
+		PlanetUtility::planetRespondToTrackback(1, xoops_error($comment->getHtmlErrors()));
+	}
 }

pdf.php

Indentation +32 added lines, -32 removed lines

@@ -32,51 +32,51 @@
 include __DIR__ . '/header.php';
 global $pdf_data;
 if (!empty($_POST['pdf_data'])) {
-    $pdf_data = unserialize(base64_decode(Request::getText('pdf_data', '', 'POST')));
+	$pdf_data = unserialize(base64_decode(Request::getText('pdf_data', '', 'POST')));
 } elseif (!empty($pdf_data)) {
 } else {
-    error_reporting(0);
-    include __DIR__ . '/header.php';
-    error_reporting(0);
+	error_reporting(0);
+	include __DIR__ . '/header.php';
+	error_reporting(0);
 
-    if (PlanetUtility::planetParseArguments($args_num, $args, $args_str)) {
-        $args['article'] = @$args_num[0];
-    }
+	if (PlanetUtility::planetParseArguments($args_num, $args, $args_str)) {
+		$args['article'] = @$args_num[0];
+	}
 
-    $article_id = (int)(empty($_GET['article']) ? @$args['article'] : $_GET['article']);
+	$article_id = (int)(empty($_GET['article']) ? @$args['article'] : $_GET['article']);
 
-    $articleHandler = xoops_getModuleHandler('article', $GLOBALS['moddirname']);
-    $article_obj    = $articleHandler->get($article_id);
+	$articleHandler = xoops_getModuleHandler('article', $GLOBALS['moddirname']);
+	$article_obj    = $articleHandler->get($article_id);
 
-    $article_data = [];
+	$article_data = [];
 
-    // title
-    $article_data['title'] = $article_obj->getVar('art_title');
+	// title
+	$article_data['title'] = $article_obj->getVar('art_title');
 
-    $article_data['author'] = $article_obj->getVar('art_author');
+	$article_data['author'] = $article_obj->getVar('art_author');
 
-    // source
-    $article_data['source'] = $article_obj->getVar('art_link');
+	// source
+	$article_data['source'] = $article_obj->getVar('art_link');
 
-    // publish time
-    $article_data['time'] = $article_obj->getTime();
+	// publish time
+	$article_data['time'] = $article_obj->getTime();
 
-    // summary
-    $article_data['summary'] = $article_obj->getSummary();
+	// summary
+	$article_data['summary'] = $article_obj->getSummary();
 
-    // text of page
-    $article_data['text'] = $article_obj->getVar('art_content');
+	// text of page
+	$article_data['text'] = $article_obj->getVar('art_content');
 
-    // Build the pdf_data array
-    $pdf_data['title']   = $article_data['title'];
-    $pdf_data['author']  = $article_data['author'];
-    $pdf_data['date']    = $article_data['time'];
-    $pdf_data['content'] = '';
-    if ($article_data['summary']) {
-        $pdf_data['content'] .= planet_constant('MD_SUMMARY') . ': ' . $article_data['summary'] . '<br><br>';
-    }
-    $pdf_data['content'] .= $article_data['text'] . '<br>';
-    $pdf_data['url']     = XOOPS_URL . '/modules/' . $GLOBALS['artdirname'] . '/view.article.php' . URL_DELIMITER . $article_obj->getVar('art_id');
+	// Build the pdf_data array
+	$pdf_data['title']   = $article_data['title'];
+	$pdf_data['author']  = $article_data['author'];
+	$pdf_data['date']    = $article_data['time'];
+	$pdf_data['content'] = '';
+	if ($article_data['summary']) {
+		$pdf_data['content'] .= planet_constant('MD_SUMMARY') . ': ' . $article_data['summary'] . '<br><br>';
+	}
+	$pdf_data['content'] .= $article_data['text'] . '<br>';
+	$pdf_data['url']     = XOOPS_URL . '/modules/' . $GLOBALS['artdirname'] . '/view.article.php' . URL_DELIMITER . $article_obj->getVar('art_id');
 }
 $pdf_data['filename'] = preg_replace("/[^0-9a-z\-_\.]/i", '', $pdf_data['title']);
 

include/config.php

Indentation +2 added lines, -2 removed lines

@@ -24,8 +24,8 @@
 
 $moduleDirName = basename(dirname(__DIR__));
 $uploadFolders = [
-    NEWBB_UPLOAD_PATH,
-    NEWBB_UPLOAD_PATH . '/thumbs'
+	NEWBB_UPLOAD_PATH,
+	NEWBB_UPLOAD_PATH . '/thumbs'
 ];
 
 //$copyFiles = array(

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

Indentation +72 added lines, -72 removed lines

@@ -32,88 +32,88 @@
 
 // defined('XOOPS_ROOT_PATH') || exit('Restricted access.');
 if (!defined('PLANET_INI')) {
-    exit();
+	exit();
 }
 
 if (!defined('PLANET_FUNCTIONS_INI')):
-    define('PLANET_FUNCTIONS_INI', 1);
+	define('PLANET_FUNCTIONS_INI', 1);
 
-    /**
-     * @param $name
-     * @return mixed
-     */
-    function planet_constant($name)
-    {
-        return mod_constant($name);
-    }
+	/**
+	 * @param $name
+	 * @return mixed
+	 */
+	function planet_constant($name)
+	{
+		return mod_constant($name);
+	}
 
-    /**
-     * @param      $name
-     * @param bool $isRel
-     * @return string
-     */
-    function planet_DB_prefix($name, $isRel = false)
-    {
-        return mod_DB_prefix($name, $isRel);
-    }
+	/**
+	 * @param      $name
+	 * @param bool $isRel
+	 * @return string
+	 */
+	function planet_DB_prefix($name, $isRel = false)
+	{
+		return mod_DB_prefix($name, $isRel);
+	}
 
-    /**
-     * @return bool
-     */
-    function planet_load_object()
-    {
-        return load_object();
-    }
+	/**
+	 * @return bool
+	 */
+	function planet_load_object()
+	{
+		return load_object();
+	}
 
-    /**
-     * @return array|mixed
-     */
-    function planet_load_config()
-    {
-        static $moduleConfig;
-        if (isset($moduleConfig)) {
-            return $moduleConfig;
-        }
+	/**
+	 * @return array|mixed
+	 */
+	function planet_load_config()
+	{
+		static $moduleConfig;
+		if (isset($moduleConfig)) {
+			return $moduleConfig;
+		}
 
-        if (is_object($GLOBALS['xoopsModule'])
-            && $GLOBALS['xoopsModule']->getVar('dirname') == $GLOBALS['moddirname']) {
-            if (isset($GLOBALS['xoopsModuleConfig'])) {
-                $moduleConfig = $GLOBALS['xoopsModuleConfig'];
-            }
-        } else {
-            /** @var XoopsModuleHandler $moduleHandler */
-            $moduleHandler = xoops_getHandler('module');
-            $module        = $moduleHandler->getByDirname($GLOBALS['moddirname']);
+		if (is_object($GLOBALS['xoopsModule'])
+			&& $GLOBALS['xoopsModule']->getVar('dirname') == $GLOBALS['moddirname']) {
+			if (isset($GLOBALS['xoopsModuleConfig'])) {
+				$moduleConfig = $GLOBALS['xoopsModuleConfig'];
+			}
+		} else {
+			/** @var XoopsModuleHandler $moduleHandler */
+			$moduleHandler = xoops_getHandler('module');
+			$module        = $moduleHandler->getByDirname($GLOBALS['moddirname']);
 
-            $configHandler = xoops_getHandler('config');
-            $criteria      = new CriteriaCompo(new Criteria('conf_modid', $module->getVar('mid')));
-            $configs       = $configHandler->getConfigs($criteria);
-            foreach (array_keys($configs) as $i) {
-                $moduleConfig[$configs[$i]->getVar('conf_name')] = $configs[$i]->getConfValueForOutput();
-            }
-            unset($configs);
-        }
-        if ($customConfig = @include XOOPS_ROOT_PATH . '/modules/' . $GLOBALS['moddirname'] . '/include/plugin.php') {
-            $moduleConfig = array_merge($moduleConfig, $customConfig);
-        }
+			$configHandler = xoops_getHandler('config');
+			$criteria      = new CriteriaCompo(new Criteria('conf_modid', $module->getVar('mid')));
+			$configs       = $configHandler->getConfigs($criteria);
+			foreach (array_keys($configs) as $i) {
+				$moduleConfig[$configs[$i]->getVar('conf_name')] = $configs[$i]->getConfValueForOutput();
+			}
+			unset($configs);
+		}
+		if ($customConfig = @include XOOPS_ROOT_PATH . '/modules/' . $GLOBALS['moddirname'] . '/include/plugin.php') {
+			$moduleConfig = array_merge($moduleConfig, $customConfig);
+		}
 
-        return $moduleConfig;
-    }
+		return $moduleConfig;
+	}
 
-    function planet_define_url_delimiter()
-    {
-        if (defined('URL_DELIMITER')) {
-            if (!in_array(URL_DELIMITER, ['?', '/'])) {
-                die('Exit on security');
-            }
-        } else {
-            $moduleConfig = planet_load_config();
-            if (empty($moduleConfig['do_urw'])) {
-                define('URL_DELIMITER', '?');
-            } else {
-                define('URL_DELIMITER', '/');
-            }
-        }
-    }
+	function planet_define_url_delimiter()
+	{
+		if (defined('URL_DELIMITER')) {
+			if (!in_array(URL_DELIMITER, ['?', '/'])) {
+				die('Exit on security');
+			}
+		} else {
+			$moduleConfig = planet_load_config();
+			if (empty($moduleConfig['do_urw'])) {
+				define('URL_DELIMITER', '?');
+			} else {
+				define('URL_DELIMITER', '/');
+			}
+		}
+	}
 
 endif;

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

Indentation +509 added lines, -509 removed lines

@@ -30,521 +30,521 @@
 
 $current_path = __FILE__;
 if (DIRECTORY_SEPARATOR !== '/') {
-    $current_path = str_replace(strpos($current_path, '\\\\', 2) ? '\\\\' : DIRECTORY_SEPARATOR, '/', $current_path);
+	$current_path = str_replace(strpos($current_path, '\\\\', 2) ? '\\\\' : DIRECTORY_SEPARATOR, '/', $current_path);
 }
 $url_arr               = explode('/', strstr($current_path, '/modules/'));
 $GLOBALS['moddirname'] = $url_arr[2];
 
 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';
-
-    /**
-     * Function to display messages
-     *
-     * @var mixed $messages
-     * @return bool
-     */
-    function planetDisplayMessage($message)
-    {
-        return mod_message($message);
-    }
-
-    /**
-     * Function to parse arguments for a page according to $_SERVER['REQUEST_URI']
-     *
-     * @var array $args_numeric array of numeric variable values
-     * @var array $args         array of indexed variables: name and value
-     * @var array $args_string  array of string variable values
-     *
-     * @return bool true on args parsed
-     */
-
-    /* known issues:
+	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';
+
+	/**
+	 * Function to display messages
+	 *
+	 * @var mixed $messages
+	 * @return bool
+	 */
+	function planetDisplayMessage($message)
+	{
+		return mod_message($message);
+	}
+
+	/**
+	 * Function to parse arguments for a page according to $_SERVER['REQUEST_URI']
+	 *
+	 * @var array $args_numeric array of numeric variable values
+	 * @var array $args         array of indexed variables: name and value
+	 * @var array $args_string  array of string variable values
+	 *
+	 * @return bool true on args parsed
+	 */
+
+	/* known issues:
      * - "/" in a string
      * - "&" in a string
     */
-    function planetParseArguments(&$args_numeric, &$args, &$args_string)
-    {
-        $args_abb     = [
-            'a' => 'article',
-            'b' => 'blog',
-            'c' => 'category',
-            'l' => 'list',
-            'o' => 'sort',
-            's' => 'start',
-            'u' => 'uid'
-        ];
-        $args         = [];
-        $args_numeric = [];
-        $args_string  = [];
-        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) {
-                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_string[] = urldecode($var);
-                        }
-                    } else {
-                        parse_str($var, $args);
-                    }
-                }
-            }
-        }
-
-        return (count($args) + count($args_numeric) + count($args_string) == 0) ? null : true;
-    }
-
-    /**
-     * Function to parse class prefix
-     *
-     * @var string $class_string string to be parsed
-     * @var mixed  $pattern
-     * @var mixed  $replacement
-     *
-     * @return bool true on success
-     */
-    function planetParseClass($class_string, $pattern = '', $replacement = '')
-    {
-        if (empty($class_string)) {
-            return;
-        }
-        $patterns     = ["/\[CLASS_PREFIX\]/"];
-        $replacements = [ucfirst(strtolower($GLOBALS['moddirname']))];
-        if (!empty($pattern) && !is_array($pattern) && !is_array($replacement)) {
-            $pattern     = [$pattern];
-            $replacement = [$replacement];
-        }
-        if (is_array($pattern) && count($pattern) > 0) {
-            $ii = 0;
-            foreach ($pattern as $pat) {
-                if (!in_array($pat, $patterns)) {
-                    $patterns[]     = $pat;
-                    $replacements[] = isset($replacement[$ii]) ? $replacement[$ii] : '';
-                }
-                ++$ii;
-            }
-        }
-        $class_string = preg_replace($patterns, $replacements, $class_string);
-        eval($class_string);
-
-        return true;
-    }
-
-    /**
-     * Function to parse function prefix
-     *
-     * @var string $function_string string to be parsed
-     * @var mixed  $pattern
-     * @var mixed  $replacement
-     *
-     * @return bool true on success
-     */
-    function planetParseFunction($function_string, $pattern = '', $replacement = '')
-    {
-        if (empty($function_string)) {
-            return;
-        }
-        $patterns     = ["/\[DIRNAME\]/", "/\[VAR_PREFIX\]/"];
-        $replacements = [$GLOBALS['moddirname'], $GLOBALS['VAR_PREFIX']];
-        if (!empty($pattern) && !is_array($pattern) && !is_array($replacement)) {
-            $pattern     = [$pattern];
-            $replacement = [$replacement];
-        }
-        if (is_array($pattern) && count($pattern) > 0) {
-            $ii = 0;
-            foreach ($pattern as $pat) {
-                if (!in_array($pat, $patterns)) {
-                    $patterns[]     = $pat;
-                    $replacements[] = isset($replacement[$ii]) ? $replacement[$ii] : '';
-                }
-                ++$ii;
-            }
-        }
-        $function_string = preg_replace($patterns, $replacements, $function_string);
-        eval($function_string);
-
-        return true;
-    }
-
-    /**
-     * Function to convert UNIX time to formatted time string
-     * @param        $time
-     * @param string $format
-     * @return string
-     */
-    function planet_formatTimestamp($time, $format = '')
-    {
-        if (empty($time)) {
-            return '';
-        }
-
-        return formatTimestamp($time, $format);
-    }
-
-    /**
-     * Function to a list of user names associated with their user IDs
-     * @param int  $userid
-     * @param int  $usereal
-     * @param bool $linked
-     * @return array
-     */
-    function &planetGetUnameFromId($userid, $usereal = 0, $linked = false)
-    {
-        if (!is_array($userid)) {
-            $userid = [$userid];
-        }
-        $users =& mod_getUnameFromIds($userid, $usereal, $linked);
-
-        return $users;
-    }
-
-    /**
-     * Function to parse links, links are delimited by link break, URL and title of a link are delimited by space
-     *
-     * @var string $text raw content
-     *
-     * @return array associative array of link url and title
-     */
-    function &planetParseLinks($text)
-    {
-        $myts       = MyTextSanitizer::getInstance();
-        $link_array = preg_split("/(\r\n|\r|\n)( *)/", $text);
-        $links      = [];
-        if (count($link_array) > 0) {
-            foreach ($link_array as $link) {
-                @list($url, $title) = array_map('trim', preg_split('/ /', $link, 2));
-                if (empty($url)) {
-                    continue;
-                }
-                //if(empty($title)) $title = $url;
-                $links[] = ['url' => $url, 'title' => $myts->htmlSpecialChars($title)];
-            }
-        }
-
-        return $links;
-    }
-
-    /**
-     * @param $pagename
-     * @return string
-     */
-    function planetGetTemplate($pagename)
-    {
-        return $GLOBALS['VAR_PREFIX'] . '_' . $pagename . '.tpl';
-    }
-
-    /**
-     * @param int $currentoption
-     */
-    function planet_adminmenu($currentoption = -1)
-    {
-        loadModuleAdminMenu($currentoption, '');
-
-        return;
-    }
-
-    /**
-     * Function to send a trackback
-     *
-     * @param $article
-     * @param $comment
-     * @return bool
-     */
-    function planet_com_trackback(&$article, &$comment)
-    {
-        $blogHandler = xoops_getModuleHandler('blog', $GLOBALS['moddirname']);
-        $blog_obj    = $blogHandler->get($article->getVar('blog_id'));
-        if (!$pattern = $blog_obj->getVar('blog_trackback')) {
-            return false;
-        }
-        @list($pat, $rep) = array_map('trim', preg_split("#[\s]+#", $pattern));
-        $trackback_url = preg_replace('#' . $pat . '#', $rep, $article_obj->getVar('art_link'));
-
-        return planetTrackback($trackback_url, $article);
-    }
-
-    /**
-     * @param $trackback_url
-     * @param $article
-     * @return bool
-     */
-    function planetTrackback($trackback_url, $article)
-    {
-        global $myts, $xoopsConfig, $xoopsModule, $xoopsModuleConfig;
-
-        $title         = $article->getVar('art_title');
-        $excerpt       = $article->getVar('art_content');
-        $blog_name     = $xoopsConfig['sitename'] . '-' . $xoopsModule->getVar('name');
-        $title         = xoops_utf8_encode($title);
-        $excerpt       = xoops_utf8_encode($excerpt);
-        $blog_name     = xoops_utf8_encode($blog_name);
-        $charset       = 'utf-8';
-        $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'));
-        $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 .= "\r\n\r\n";
-        $http_request .= $query_string;
-        if ('' == $trackback_url['port']) {
-            $trackback_url['port'] = 80;
-        }
-        $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';
-            $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         .= "EXCERPT:$excerpt\n\n";
-            while (!@feof($fs)) {
-                $fr .= @fgets($fs, 4096);
-            }
-            $fr .= "\n\n";
-
-            if ($fp = fopen($debug_file, 'a')) {
-                fwrite($fp, $fr);
-                fclose($fp);
-            } else {
-            }
-        }
-        @fclose($fs);
-
-        return true;
-    }
-
-    /**
-     * Function to ping servers
-     * @param $server
-     * @param $id
-     */
-    function planetGetPing($server, $id)
-    {
-        if (is_array($server)) {
-            foreach ($server as $serv) {
-                planetGetPing($serv, $id);
-            }
-        }
-        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;
-
-        // 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;
-
-        if (!$client->query('weblogUpdates.extendedPing', $blogname, $home, $rss2_url)) { // then try a normal ping
-            $client->query('weblogUpdates.ping', $blogname, $home);
-        }
-    }
-
-    /**
-     * Function to respond to a trackback
-     * @param int    $error
-     * @param string $error_message
-     */
-    function planetRespondToTrackback($error = 0, $error_message = '')
-    {
-        $charset       = 'utf-8';
-        $error_message = xoops_utf8_encode($error_message);
-        header('Content-Type: text/xml; charset="' . $charset . '"');
-        if ($error) {
-            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";
-            echo "<response>\n";
-            echo "<error>0</error>\n";
-            echo '</response>';
-        }
-    }
-
-    /**
-     * Function to set a cookie with module-specified name
-     *
-     * using customized serialization method
-     * @param        $name
-     * @param string $string
-     * @param int    $expire
-     */
-    function planetSetCookie($name, $string = '', $expire = 0)
-    {
-        if (is_array($string)) {
-            $value = [];
-            foreach ($string as $key => $val) {
-                $value[] = $key . '|' . $val;
-            }
-            $string = implode(',', $value);
-        }
-        setcookie($GLOBALS['VAR_PREFIX'] . $name, $string, (int)$expire, '/');
-    }
-
-    /**
-     * @param      $name
-     * @param bool $isArray
-     * @return array|null
-     */
-    function planetGetCookie($name, $isArray = false)
-    {
-        $value = isset($_COOKIE[$GLOBALS['VAR_PREFIX'] . $name]) ? $_COOKIE[$GLOBALS['VAR_PREFIX'] . $name] : null;
-        if ($isArray) {
-            $_value = $value ? explode(',', $value) : [];
-            $value  = [];
-            if (count($_value) > 0) {
-                foreach ($_value as $string) {
-                    $key         = substr($string, 0, strpos($string, '|'));
-                    $val         = substr($string, strpos($string, '|') + 1);
-                    $value[$key] = $val;
-                }
-            }
-            unset($_value);
-        }
-
-        return $value;
-    }
-
-    /**
-     * Function to filter text
-     *
-     * @param $document
-     * @return string filtered text
-     */
-    function &planetHtml2text(&$document)
-    {
-        $document = strip_tags($document);
-
-        return $document;
-    }
-
-    // Adapted from PMA_getIp() [phpmyadmin project]
-    /**
-     * @param bool $asString
-     * @return mixed
-     */
-    function planetGetIP($asString = false)
-    {
-        return mod_getIP($asString);
-    }
-
-    /**
-     * @param $url
-     * @return bool|mixed|string
-     */
-    function planetGetRemoteContent($url)
-    {
-        if ($data = planet_fetch_snoopy($url)) {
-            return $data;
-        }
-        if ($data = planet_fetch_CURL($url)) {
-            return $data;
-        }
-        if ($data = planet_fetch_fopen($url)) {
-            return $data;
-        }
-
-        return false;
-    }
-
-    /**
-     * @param $url
-     * @return string
-     */
-    function planet_fetch_snoopy($url)
-    {
-        require_once XOOPS_ROOT_PATH . '/class/snoopy.php';
-        $snoopy = new Snoopy;
-        $data   = '';
-        if (@$snoopy->fetch($url)) {
-            $data = is_array($snoopy->results) ? implode("\n", $snoopy->results) : $snoopy->results;
-        }
-
-        return $data;
-    }
-
-    /**
-     * @param $url
-     * @return bool|mixed
-     */
-    function planet_fetch_CURL($url)
-    {
-        if (!function_exists('curl_init')) {
-            return false;
-        }
-        $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_RETURNTRANSFER, 1); // return into a variable
-        curl_setopt($ch, CURLOPT_TIMEOUT, 30); // times out after 31s
-        $data = curl_exec($ch); // run the whole process
-        curl_close($ch);
-
-        return $data;
-    }
-
-    /**
-     * @param $url
-     * @return bool|string
-     */
-    function planet_fetch_fopen($url)
-    {
-        if (!$fp = @fopen($url, 'r')) {
-            return false;
-        }
-        $data = '';
-        while (!feof($fp)) {
-            $data .= fgets($fp, 1024);
-        }
-        fclose($fp);
-
-        return $data;
-    }
-
-    /**
-     * @param     $haystack
-     * @param     $needle
-     * @param int $offset
-     * @return bool|int
-     */
-    function planetStrrPos($haystack, $needle, $offset = 0)
-    {
-        if (substr(PHP_VERSION, 0, 1) == 5) {
-            return strrpos($haystack, $needle, $offset);
-        }
-        $index = strpos(strrev($haystack), strrev($needle));
-        if ($index === false) {
-            return false;
-        }
-        $index = strlen($haystack) - strlen($needle) - $index;
-
-        return $index;
-    }
+	function planetParseArguments(&$args_numeric, &$args, &$args_string)
+	{
+		$args_abb     = [
+			'a' => 'article',
+			'b' => 'blog',
+			'c' => 'category',
+			'l' => 'list',
+			'o' => 'sort',
+			's' => 'start',
+			'u' => 'uid'
+		];
+		$args         = [];
+		$args_numeric = [];
+		$args_string  = [];
+		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) {
+				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_string[] = urldecode($var);
+						}
+					} else {
+						parse_str($var, $args);
+					}
+				}
+			}
+		}
+
+		return (count($args) + count($args_numeric) + count($args_string) == 0) ? null : true;
+	}
+
+	/**
+	 * Function to parse class prefix
+	 *
+	 * @var string $class_string string to be parsed
+	 * @var mixed  $pattern
+	 * @var mixed  $replacement
+	 *
+	 * @return bool true on success
+	 */
+	function planetParseClass($class_string, $pattern = '', $replacement = '')
+	{
+		if (empty($class_string)) {
+			return;
+		}
+		$patterns     = ["/\[CLASS_PREFIX\]/"];
+		$replacements = [ucfirst(strtolower($GLOBALS['moddirname']))];
+		if (!empty($pattern) && !is_array($pattern) && !is_array($replacement)) {
+			$pattern     = [$pattern];
+			$replacement = [$replacement];
+		}
+		if (is_array($pattern) && count($pattern) > 0) {
+			$ii = 0;
+			foreach ($pattern as $pat) {
+				if (!in_array($pat, $patterns)) {
+					$patterns[]     = $pat;
+					$replacements[] = isset($replacement[$ii]) ? $replacement[$ii] : '';
+				}
+				++$ii;
+			}
+		}
+		$class_string = preg_replace($patterns, $replacements, $class_string);
+		eval($class_string);
+
+		return true;
+	}
+
+	/**
+	 * Function to parse function prefix
+	 *
+	 * @var string $function_string string to be parsed
+	 * @var mixed  $pattern
+	 * @var mixed  $replacement
+	 *
+	 * @return bool true on success
+	 */
+	function planetParseFunction($function_string, $pattern = '', $replacement = '')
+	{
+		if (empty($function_string)) {
+			return;
+		}
+		$patterns     = ["/\[DIRNAME\]/", "/\[VAR_PREFIX\]/"];
+		$replacements = [$GLOBALS['moddirname'], $GLOBALS['VAR_PREFIX']];
+		if (!empty($pattern) && !is_array($pattern) && !is_array($replacement)) {
+			$pattern     = [$pattern];
+			$replacement = [$replacement];
+		}
+		if (is_array($pattern) && count($pattern) > 0) {
+			$ii = 0;
+			foreach ($pattern as $pat) {
+				if (!in_array($pat, $patterns)) {
+					$patterns[]     = $pat;
+					$replacements[] = isset($replacement[$ii]) ? $replacement[$ii] : '';
+				}
+				++$ii;
+			}
+		}
+		$function_string = preg_replace($patterns, $replacements, $function_string);
+		eval($function_string);
+
+		return true;
+	}
+
+	/**
+	 * Function to convert UNIX time to formatted time string
+	 * @param        $time
+	 * @param string $format
+	 * @return string
+	 */
+	function planet_formatTimestamp($time, $format = '')
+	{
+		if (empty($time)) {
+			return '';
+		}
+
+		return formatTimestamp($time, $format);
+	}
+
+	/**
+	 * Function to a list of user names associated with their user IDs
+	 * @param int  $userid
+	 * @param int  $usereal
+	 * @param bool $linked
+	 * @return array
+	 */
+	function &planetGetUnameFromId($userid, $usereal = 0, $linked = false)
+	{
+		if (!is_array($userid)) {
+			$userid = [$userid];
+		}
+		$users =& mod_getUnameFromIds($userid, $usereal, $linked);
+
+		return $users;
+	}
+
+	/**
+	 * Function to parse links, links are delimited by link break, URL and title of a link are delimited by space
+	 *
+	 * @var string $text raw content
+	 *
+	 * @return array associative array of link url and title
+	 */
+	function &planetParseLinks($text)
+	{
+		$myts       = MyTextSanitizer::getInstance();
+		$link_array = preg_split("/(\r\n|\r|\n)( *)/", $text);
+		$links      = [];
+		if (count($link_array) > 0) {
+			foreach ($link_array as $link) {
+				@list($url, $title) = array_map('trim', preg_split('/ /', $link, 2));
+				if (empty($url)) {
+					continue;
+				}
+				//if(empty($title)) $title = $url;
+				$links[] = ['url' => $url, 'title' => $myts->htmlSpecialChars($title)];
+			}
+		}
+
+		return $links;
+	}
+
+	/**
+	 * @param $pagename
+	 * @return string
+	 */
+	function planetGetTemplate($pagename)
+	{
+		return $GLOBALS['VAR_PREFIX'] . '_' . $pagename . '.tpl';
+	}
+
+	/**
+	 * @param int $currentoption
+	 */
+	function planet_adminmenu($currentoption = -1)
+	{
+		loadModuleAdminMenu($currentoption, '');
+
+		return;
+	}
+
+	/**
+	 * Function to send a trackback
+	 *
+	 * @param $article
+	 * @param $comment
+	 * @return bool
+	 */
+	function planet_com_trackback(&$article, &$comment)
+	{
+		$blogHandler = xoops_getModuleHandler('blog', $GLOBALS['moddirname']);
+		$blog_obj    = $blogHandler->get($article->getVar('blog_id'));
+		if (!$pattern = $blog_obj->getVar('blog_trackback')) {
+			return false;
+		}
+		@list($pat, $rep) = array_map('trim', preg_split("#[\s]+#", $pattern));
+		$trackback_url = preg_replace('#' . $pat . '#', $rep, $article_obj->getVar('art_link'));
+
+		return planetTrackback($trackback_url, $article);
+	}
+
+	/**
+	 * @param $trackback_url
+	 * @param $article
+	 * @return bool
+	 */
+	function planetTrackback($trackback_url, $article)
+	{
+		global $myts, $xoopsConfig, $xoopsModule, $xoopsModuleConfig;
+
+		$title         = $article->getVar('art_title');
+		$excerpt       = $article->getVar('art_content');
+		$blog_name     = $xoopsConfig['sitename'] . '-' . $xoopsModule->getVar('name');
+		$title         = xoops_utf8_encode($title);
+		$excerpt       = xoops_utf8_encode($excerpt);
+		$blog_name     = xoops_utf8_encode($blog_name);
+		$charset       = 'utf-8';
+		$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'));
+		$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 .= "\r\n\r\n";
+		$http_request .= $query_string;
+		if ('' == $trackback_url['port']) {
+			$trackback_url['port'] = 80;
+		}
+		$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';
+			$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         .= "EXCERPT:$excerpt\n\n";
+			while (!@feof($fs)) {
+				$fr .= @fgets($fs, 4096);
+			}
+			$fr .= "\n\n";
+
+			if ($fp = fopen($debug_file, 'a')) {
+				fwrite($fp, $fr);
+				fclose($fp);
+			} else {
+			}
+		}
+		@fclose($fs);
+
+		return true;
+	}
+
+	/**
+	 * Function to ping servers
+	 * @param $server
+	 * @param $id
+	 */
+	function planetGetPing($server, $id)
+	{
+		if (is_array($server)) {
+			foreach ($server as $serv) {
+				planetGetPing($serv, $id);
+			}
+		}
+		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;
+
+		// 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;
+
+		if (!$client->query('weblogUpdates.extendedPing', $blogname, $home, $rss2_url)) { // then try a normal ping
+			$client->query('weblogUpdates.ping', $blogname, $home);
+		}
+	}
+
+	/**
+	 * Function to respond to a trackback
+	 * @param int    $error
+	 * @param string $error_message
+	 */
+	function planetRespondToTrackback($error = 0, $error_message = '')
+	{
+		$charset       = 'utf-8';
+		$error_message = xoops_utf8_encode($error_message);
+		header('Content-Type: text/xml; charset="' . $charset . '"');
+		if ($error) {
+			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";
+			echo "<response>\n";
+			echo "<error>0</error>\n";
+			echo '</response>';
+		}
+	}
+
+	/**
+	 * Function to set a cookie with module-specified name
+	 *
+	 * using customized serialization method
+	 * @param        $name
+	 * @param string $string
+	 * @param int    $expire
+	 */
+	function planetSetCookie($name, $string = '', $expire = 0)
+	{
+		if (is_array($string)) {
+			$value = [];
+			foreach ($string as $key => $val) {
+				$value[] = $key . '|' . $val;
+			}
+			$string = implode(',', $value);
+		}
+		setcookie($GLOBALS['VAR_PREFIX'] . $name, $string, (int)$expire, '/');
+	}
+
+	/**
+	 * @param      $name
+	 * @param bool $isArray
+	 * @return array|null
+	 */
+	function planetGetCookie($name, $isArray = false)
+	{
+		$value = isset($_COOKIE[$GLOBALS['VAR_PREFIX'] . $name]) ? $_COOKIE[$GLOBALS['VAR_PREFIX'] . $name] : null;
+		if ($isArray) {
+			$_value = $value ? explode(',', $value) : [];
+			$value  = [];
+			if (count($_value) > 0) {
+				foreach ($_value as $string) {
+					$key         = substr($string, 0, strpos($string, '|'));
+					$val         = substr($string, strpos($string, '|') + 1);
+					$value[$key] = $val;
+				}
+			}
+			unset($_value);
+		}
+
+		return $value;
+	}
+
+	/**
+	 * Function to filter text
+	 *
+	 * @param $document
+	 * @return string filtered text
+	 */
+	function &planetHtml2text(&$document)
+	{
+		$document = strip_tags($document);
+
+		return $document;
+	}
+
+	// Adapted from PMA_getIp() [phpmyadmin project]
+	/**
+	 * @param bool $asString
+	 * @return mixed
+	 */
+	function planetGetIP($asString = false)
+	{
+		return mod_getIP($asString);
+	}
+
+	/**
+	 * @param $url
+	 * @return bool|mixed|string
+	 */
+	function planetGetRemoteContent($url)
+	{
+		if ($data = planet_fetch_snoopy($url)) {
+			return $data;
+		}
+		if ($data = planet_fetch_CURL($url)) {
+			return $data;
+		}
+		if ($data = planet_fetch_fopen($url)) {
+			return $data;
+		}
+
+		return false;
+	}
+
+	/**
+	 * @param $url
+	 * @return string
+	 */
+	function planet_fetch_snoopy($url)
+	{
+		require_once XOOPS_ROOT_PATH . '/class/snoopy.php';
+		$snoopy = new Snoopy;
+		$data   = '';
+		if (@$snoopy->fetch($url)) {
+			$data = is_array($snoopy->results) ? implode("\n", $snoopy->results) : $snoopy->results;
+		}
+
+		return $data;
+	}
+
+	/**
+	 * @param $url
+	 * @return bool|mixed
+	 */
+	function planet_fetch_CURL($url)
+	{
+		if (!function_exists('curl_init')) {
+			return false;
+		}
+		$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_RETURNTRANSFER, 1); // return into a variable
+		curl_setopt($ch, CURLOPT_TIMEOUT, 30); // times out after 31s
+		$data = curl_exec($ch); // run the whole process
+		curl_close($ch);
+
+		return $data;
+	}
+
+	/**
+	 * @param $url
+	 * @return bool|string
+	 */
+	function planet_fetch_fopen($url)
+	{
+		if (!$fp = @fopen($url, 'r')) {
+			return false;
+		}
+		$data = '';
+		while (!feof($fp)) {
+			$data .= fgets($fp, 1024);
+		}
+		fclose($fp);
+
+		return $data;
+	}
+
+	/**
+	 * @param     $haystack
+	 * @param     $needle
+	 * @param int $offset
+	 * @return bool|int
+	 */
+	function planetStrrPos($haystack, $needle, $offset = 0)
+	{
+		if (substr(PHP_VERSION, 0, 1) == 5) {
+			return strrpos($haystack, $needle, $offset);
+		}
+		$index = strpos(strrev($haystack), strrev($needle));
+		if ($index === false) {
+			return false;
+		}
+		$index = strlen($haystack) - strlen($needle) - $index;
+
+		return $index;
+	}
 endif;

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