mambax7 / planet

include/plugin.transfer.php

Indentation +34 added lines, -34 removed lines

@@ -15,7 +15,7 @@ 
 // defined('XOOPS_ROOT_PATH') || die('Restricted access');
 
 if (!@require_once XOOPS_ROOT_PATH . '/Frameworks/transfer/transfer.php') {
-    return null;
+	return null;
 }
 
 // Specify the addons to skip for the module
@@ -28,41 +28,41 @@ 
  */
 class ModuleTransferHandler extends TransferHandler
 {
-    /**
-     * ModuleTransferHandler constructor.
-     */
-    public function __construct()
-    {
-        parent::__construct();
-    }
+	/**
+	 * ModuleTransferHandler constructor.
+	 */
+	public function __construct()
+	{
+		parent::__construct();
+	}
 
-    /**
-     * Get valid addon list
-     *
-     * @param array   $skip Addons to skip
-     * @param boolean $sort To sort the list upon 'level'
-     *                      return  array   $list
-     */
-    public function &getList($skip = [], $sort = true)
-    {
-        $list = parent::getList($skip, $sort);
+	/**
+	 * Get valid addon list
+	 *
+	 * @param array   $skip Addons to skip
+	 * @param boolean $sort To sort the list upon 'level'
+	 *                      return  array   $list
+	 */
+	public function &getList($skip = [], $sort = true)
+	{
+		$list = parent::getList($skip, $sort);
 
-        return $list;
-    }
+		return $list;
+	}
 
-    /**
-     * If need change config of an item
-     * 1 parent::load_item
-     * 2 $this->config
-     * 3 $this->do_transfer
-     * @param $item
-     * @param $data
-     * @return
-     */
-    public function do_transfer($item, &$data)
-    {
-        $ret = parent::do_transfer($item, $data);
+	/**
+	 * If need change config of an item
+	 * 1 parent::load_item
+	 * 2 $this->config
+	 * 3 $this->do_transfer
+	 * @param $item
+	 * @param $data
+	 * @return
+	 */
+	public function do_transfer($item, &$data)
+	{
+		$ret = parent::do_transfer($item, $data);
 
-        return $ret;
-    }
+		return $ret;
+	}
 }

Spacing +1 added lines, -1 removed lines

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

include/oninstall.php

Indentation +61 added lines, -61 removed lines

@@ -28,20 +28,20 @@ 
  */
 function xoops_module_pre_install_planet(\XoopsModule $module)
 {
-    include __DIR__ . '/../preloads/autoloader.php';
-    /** @var \Utility $utility */
-    $utility = new \XoopsModules\Planet\Utility();
-    $xoopsSuccess = $utility::checkVerXoops($module);
-    $phpSuccess   = $utility::checkVerPhp($module);
-
-    if (false !== $xoopsSuccess && false !==  $phpSuccess) {
-        $moduleTables =& $module->getInfo('tables');
-        foreach ($moduleTables as $table) {
-            $GLOBALS['xoopsDB']->queryF('DROP TABLE IF EXISTS ' . $GLOBALS['xoopsDB']->prefix($table) . ';');
-        }
-    }
-
-    return $xoopsSuccess && $phpSuccess;
+	include __DIR__ . '/../preloads/autoloader.php';
+	/** @var \Utility $utility */
+	$utility = new \XoopsModules\Planet\Utility();
+	$xoopsSuccess = $utility::checkVerXoops($module);
+	$phpSuccess   = $utility::checkVerPhp($module);
+
+	if (false !== $xoopsSuccess && false !==  $phpSuccess) {
+		$moduleTables =& $module->getInfo('tables');
+		foreach ($moduleTables as $table) {
+			$GLOBALS['xoopsDB']->queryF('DROP TABLE IF EXISTS ' . $GLOBALS['xoopsDB']->prefix($table) . ';');
+		}
+	}
+
+	return $xoopsSuccess && $phpSuccess;
 }
 
 /**
@@ -53,51 +53,51 @@ 
  */
 function xoops_module_install_planet(\XoopsModule $module)
 {
-    require_once  __DIR__ . '/../../../mainfile.php';
-    require_once  __DIR__ . '/../include/config.php';
-
-    $moduleDirName = basename(dirname(__DIR__));
-
-    $helper       = Planet\Helper::getInstance();
-    $utility      = new Planet\Utility();
-    $configurator = new Planet\Configurator();
-    // Load language files
-    $helper->loadLanguage('admin');
-    $helper->loadLanguage('modinfo');
-
-    // default Permission Settings ----------------------
-    global $xoopsModule;
-    $moduleId     = $xoopsModule->getVar('mid');
-    $moduleId2    = $helper->getModule()->mid();
-    $gpermHandler = xoops_getHandler('groupperm');
-    // access rights ------------------------------------------
-    $gpermHandler->addRight($moduleDirName . '_approve', 1, XOOPS_GROUP_ADMIN, $moduleId);
-    $gpermHandler->addRight($moduleDirName . '_submit', 1, XOOPS_GROUP_ADMIN, $moduleId);
-    $gpermHandler->addRight($moduleDirName . '_view', 1, XOOPS_GROUP_ADMIN, $moduleId);
-    $gpermHandler->addRight($moduleDirName . '_view', 1, XOOPS_GROUP_USERS, $moduleId);
-    $gpermHandler->addRight($moduleDirName . '_view', 1, XOOPS_GROUP_ANONYMOUS, $moduleId);
-
-    //  ---  CREATE FOLDERS ---------------
-    if (count($configurator->uploadFolders) > 0) {
-        //    foreach (array_keys($GLOBALS['uploadFolders']) as $i) {
-        foreach (array_keys($configurator->uploadFolders) as $i) {
-            $utilityClass::createFolder($configurator->uploadFolders[$i]);
-        }
-    }
-
-    //  ---  COPY blank.png FILES ---------------
-    if (count($configurator->copyBlankFiles) > 0) {
-        $file = __DIR__ . '/../assets/images/blank.png';
-        foreach (array_keys($configurator->copyBlankFiles) as $i) {
-            $dest = $configurator->copyBlankFiles[$i] . '/blank.png';
-            $utilityClass::copyFile($file, $dest);
-        }
-    }
-    //delete .html entries from the tpl table
-    $sql = 'DELETE FROM ' . $xoopsDB->prefix('tplfile') . " WHERE `tpl_module` = '" . $xoopsModule->getVar('dirname', 'n') . "' AND `tpl_file` LIKE '%.html%'";
-    $xoopsDB->queryF($sql);
-
-    return true;
+	require_once  __DIR__ . '/../../../mainfile.php';
+	require_once  __DIR__ . '/../include/config.php';
+
+	$moduleDirName = basename(dirname(__DIR__));
+
+	$helper       = Planet\Helper::getInstance();
+	$utility      = new Planet\Utility();
+	$configurator = new Planet\Configurator();
+	// Load language files
+	$helper->loadLanguage('admin');
+	$helper->loadLanguage('modinfo');
+
+	// default Permission Settings ----------------------
+	global $xoopsModule;
+	$moduleId     = $xoopsModule->getVar('mid');
+	$moduleId2    = $helper->getModule()->mid();
+	$gpermHandler = xoops_getHandler('groupperm');
+	// access rights ------------------------------------------
+	$gpermHandler->addRight($moduleDirName . '_approve', 1, XOOPS_GROUP_ADMIN, $moduleId);
+	$gpermHandler->addRight($moduleDirName . '_submit', 1, XOOPS_GROUP_ADMIN, $moduleId);
+	$gpermHandler->addRight($moduleDirName . '_view', 1, XOOPS_GROUP_ADMIN, $moduleId);
+	$gpermHandler->addRight($moduleDirName . '_view', 1, XOOPS_GROUP_USERS, $moduleId);
+	$gpermHandler->addRight($moduleDirName . '_view', 1, XOOPS_GROUP_ANONYMOUS, $moduleId);
+
+	//  ---  CREATE FOLDERS ---------------
+	if (count($configurator->uploadFolders) > 0) {
+		//    foreach (array_keys($GLOBALS['uploadFolders']) as $i) {
+		foreach (array_keys($configurator->uploadFolders) as $i) {
+			$utilityClass::createFolder($configurator->uploadFolders[$i]);
+		}
+	}
+
+	//  ---  COPY blank.png FILES ---------------
+	if (count($configurator->copyBlankFiles) > 0) {
+		$file = __DIR__ . '/../assets/images/blank.png';
+		foreach (array_keys($configurator->copyBlankFiles) as $i) {
+			$dest = $configurator->copyBlankFiles[$i] . '/blank.png';
+			$utilityClass::copyFile($file, $dest);
+		}
+	}
+	//delete .html entries from the tpl table
+	$sql = 'DELETE FROM ' . $xoopsDB->prefix('tplfile') . " WHERE `tpl_module` = '" . $xoopsModule->getVar('dirname', 'n') . "' AND `tpl_file` LIKE '%.html%'";
+	$xoopsDB->queryF($sql);
+
+	return true;
 }
 
 //======================================================
@@ -108,7 +108,7 @@ 
 //Creation du dossier "uploads" pour le module à la racine du site
 $module_uploads = $GLOBALS['xoops']->path('uploads/randomquote');
 if (!is_dir($module_uploads)) {
-    mkdir($module_uploads, 0777);
+	mkdir($module_uploads, 0777);
 }
 chmod($module_uploads, 0777);
 copy($indexFile, $GLOBALS['xoops']->path('uploads/randomquote/index.html'));
@@ -116,7 +116,7 @@ 
 //Creation du fichier citas dans uploads
 $module_uploads = $GLOBALS['xoops']->path('uploads/randomquote/citas');
 if (!is_dir($module_uploads)) {
-    mkdir($module_uploads, 0777);
+	mkdir($module_uploads, 0777);
 }
 chmod($module_uploads, 0777);
 copy($indexFile, $GLOBALS['xoops']->path('uploads/randomquote/citas/index.html'));

Spacing +16 added lines, -16 removed lines

@@ -28,16 +28,16 @@ 
  */
 function xoops_module_pre_install_planet(\XoopsModule $module)
 {
-    include __DIR__ . '/../preloads/autoloader.php';
+    include __DIR__.'/../preloads/autoloader.php';
     /** @var \Utility $utility */
     $utility = new \XoopsModules\Planet\Utility();
     $xoopsSuccess = $utility::checkVerXoops($module);
     $phpSuccess   = $utility::checkVerPhp($module);
 
-    if (false !== $xoopsSuccess && false !==  $phpSuccess) {
-        $moduleTables =& $module->getInfo('tables');
+    if (false !== $xoopsSuccess && false !== $phpSuccess) {
+        $moduleTables = & $module->getInfo('tables');
         foreach ($moduleTables as $table) {
-            $GLOBALS['xoopsDB']->queryF('DROP TABLE IF EXISTS ' . $GLOBALS['xoopsDB']->prefix($table) . ';');
+            $GLOBALS['xoopsDB']->queryF('DROP TABLE IF EXISTS '.$GLOBALS['xoopsDB']->prefix($table).';');
         }
     }
 
@@ -53,8 +53,8 @@ 
  */
 function xoops_module_install_planet(\XoopsModule $module)
 {
-    require_once  __DIR__ . '/../../../mainfile.php';
-    require_once  __DIR__ . '/../include/config.php';
+    require_once  __DIR__.'/../../../mainfile.php';
+    require_once  __DIR__.'/../include/config.php';
 
     $moduleDirName = basename(dirname(__DIR__));
 
@@ -71,14 +71,14 @@ 
     $moduleId2    = $helper->getModule()->mid();
     $gpermHandler = xoops_getHandler('groupperm');
     // access rights ------------------------------------------
-    $gpermHandler->addRight($moduleDirName . '_approve', 1, XOOPS_GROUP_ADMIN, $moduleId);
-    $gpermHandler->addRight($moduleDirName . '_submit', 1, XOOPS_GROUP_ADMIN, $moduleId);
-    $gpermHandler->addRight($moduleDirName . '_view', 1, XOOPS_GROUP_ADMIN, $moduleId);
-    $gpermHandler->addRight($moduleDirName . '_view', 1, XOOPS_GROUP_USERS, $moduleId);
-    $gpermHandler->addRight($moduleDirName . '_view', 1, XOOPS_GROUP_ANONYMOUS, $moduleId);
+    $gpermHandler->addRight($moduleDirName.'_approve', 1, XOOPS_GROUP_ADMIN, $moduleId);
+    $gpermHandler->addRight($moduleDirName.'_submit', 1, XOOPS_GROUP_ADMIN, $moduleId);
+    $gpermHandler->addRight($moduleDirName.'_view', 1, XOOPS_GROUP_ADMIN, $moduleId);
+    $gpermHandler->addRight($moduleDirName.'_view', 1, XOOPS_GROUP_USERS, $moduleId);
+    $gpermHandler->addRight($moduleDirName.'_view', 1, XOOPS_GROUP_ANONYMOUS, $moduleId);
 
     //  ---  CREATE FOLDERS ---------------
-    if (count($configurator->uploadFolders) > 0) {
+    if (count($configurator->uploadFolders)>0) {
         //    foreach (array_keys($GLOBALS['uploadFolders']) as $i) {
         foreach (array_keys($configurator->uploadFolders) as $i) {
             $utilityClass::createFolder($configurator->uploadFolders[$i]);
@@ -86,15 +86,15 @@ 
     }
 
     //  ---  COPY blank.png FILES ---------------
-    if (count($configurator->copyBlankFiles) > 0) {
-        $file = __DIR__ . '/../assets/images/blank.png';
+    if (count($configurator->copyBlankFiles)>0) {
+        $file = __DIR__.'/../assets/images/blank.png';
         foreach (array_keys($configurator->copyBlankFiles) as $i) {
-            $dest = $configurator->copyBlankFiles[$i] . '/blank.png';
+            $dest = $configurator->copyBlankFiles[$i].'/blank.png';
             $utilityClass::copyFile($file, $dest);
         }
     }
     //delete .html entries from the tpl table
-    $sql = 'DELETE FROM ' . $xoopsDB->prefix('tplfile') . " WHERE `tpl_module` = '" . $xoopsModule->getVar('dirname', 'n') . "' AND `tpl_file` LIKE '%.html%'";
+    $sql = 'DELETE FROM '.$xoopsDB->prefix('tplfile')." WHERE `tpl_module` = '".$xoopsModule->getVar('dirname', 'n')."' AND `tpl_file` LIKE '%.html%'";
     $xoopsDB->queryF($sql);
 
     return true;

include/functions.php

Indentation +511 added lines, -511 removed lines

@@ -33,523 +33,523 @@
 
 $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 (0 == count($args) + count($args_numeric) + count($args_string)) ? 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;
-        /** @var Planet\Helper $helper */
-        $helper = Planet\Helper::getInstance();
-
-        $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 ($helper->getConfig('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 (5 == substr(PHP_VERSION, 0, 1)) {
-            return strrpos($haystack, $needle, $offset);
-        }
-        $index = strpos(strrev($haystack), strrev($needle));
-        if (false === $index) {
-            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 (0 == count($args) + count($args_numeric) + count($args_string)) ? 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;
+		/** @var Planet\Helper $helper */
+		$helper = Planet\Helper::getInstance();
+
+		$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 ($helper->getConfig('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 (5 == substr(PHP_VERSION, 0, 1)) {
+			return strrpos($haystack, $needle, $offset);
+		}
+		$index = strpos(strrev($haystack), strrev($needle));
+		if (false === $index) {
+			return false;
+		}
+		$index = strlen($haystack) - strlen($needle) - $index;
+
+		return $index;
+	}
 endif;

Spacing +42 added lines, -42 removed lines

@@ -41,9 +41,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
@@ -72,7 +72,7 @@ 
     */
     function planetParseArguments(&$args_numeric, &$args, &$args_string)
     {
-        $args_abb     = [
+        $args_abb = [
             'a' => 'article',
             'b' => 'blog',
             'c' => 'category',
@@ -87,24 +87,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 (0 == count($args) + count($args_numeric) + count($args_string)) ? null : true;
+        return (0 == count($args)+count($args_numeric)+count($args_string)) ? null : true;
     }
 
     /**
@@ -127,7 +127,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)) {
@@ -163,7 +163,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)) {
@@ -206,7 +206,7 @@ 
         if (!is_array($userid)) {
             $userid = [$userid];
         }
-        $users =& mod_getUnameFromIds($userid, $usereal, $linked);
+        $users = & mod_getUnameFromIds($userid, $usereal, $linked);
 
         return $users;
     }
@@ -223,7 +223,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)) {
@@ -243,7 +243,7 @@ 
      */
     function planetGetTemplate($pagename)
     {
-        return $GLOBALS['VAR_PREFIX'] . '_' . $pagename . '.tpl';
+        return $GLOBALS['VAR_PREFIX'].'_'.$pagename.'.tpl';
     }
 
     /**
@@ -271,7 +271,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);
     }
@@ -289,7 +289,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);
@@ -297,15 +297,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']) {
@@ -314,11 +314,11 @@ 
         $fs = @fsockopen($trackback_url['host'], $trackback_url['port'], $errno, $errstr, 4);
         @fwrite($fs, $http_request);
         if ($helper->getConfig('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);
@@ -328,7 +328,7 @@ 
             if ($fp = fopen($debug_file, 'a')) {
                 fwrite($fp, $fr);
                 fclose($fp);
-            } else {
+            }else {
             }
         }
         @fclose($fs);
@@ -348,19 +348,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);
@@ -376,16 +376,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>';
@@ -405,11 +405,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, '/');
     }
 
     /**
@@ -419,14 +419,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;
                 }
             }
@@ -484,7 +484,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)) {
@@ -503,10 +503,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
@@ -548,7 +548,7 @@ 
         if (false === $index) {
             return false;
         }
-        $index = strlen($haystack) - strlen($needle) - $index;
+        $index = strlen($haystack)-strlen($needle)-$index;
 
         return $index;
     }

include/search.inc.php

Spacing +1 added lines, -1 removed lines

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

include/action.module.php

Spacing +1 added lines, -1 removed lines

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

include/onuninstall.php

Indentation +29 added lines, -29 removed lines

@@ -18,8 +18,8 @@ 
 
 function xoops_module_pre_uninstall_xxxx(\XoopsModule $module)
 {
-    // Do some synchronization
-    return true;
+	// Do some synchronization
+	return true;
 }
 
 /**
@@ -33,36 +33,36 @@ 
 {
 //    return true;
 
-    $moduleDirName = basename(dirname(__DIR__));
-    $xsitemapHelper      = \Xmf\Module\Helper::getHelper($moduleDirName);
+	$moduleDirName = basename(dirname(__DIR__));
+	$xsitemapHelper      = \Xmf\Module\Helper::getHelper($moduleDirName);
 
-    /** @var XXXXXXUtility $utilityClass */
-    $utilityClass     = ucfirst($moduleDirName) . 'Utility';
-    if (!class_exists($utilityClass)) {
-        xoops_load('utility', $moduleDirName);
-    }
+	/** @var XXXXXXUtility $utilityClass */
+	$utilityClass     = ucfirst($moduleDirName) . 'Utility';
+	if (!class_exists($utilityClass)) {
+		xoops_load('utility', $moduleDirName);
+	}
 
-    $success = true;
-    $xsitemapHelper->loadLanguage('admin');
+	$success = true;
+	$xsitemapHelper->loadLanguage('admin');
 
 
-    //------------------------------------------------------------------
-    // Remove uploads folder (and all subfolders) if they exist
-    //------------------------------------------------------------------
+	//------------------------------------------------------------------
+	// Remove uploads folder (and all subfolders) if they exist
+	//------------------------------------------------------------------
 
-    $old_directories = [$GLOBALS['xoops']->path("uploads/{$moduleDirName}")];
-    foreach ($old_directories as $old_dir) {
-        $dirInfo = new \SplFileInfo($old_dir);
-        if ($dirInfo->isDir()) {
-            // The directory exists so delete it
-            if (false === $utilityClass::rrmdir($old_dir)) {
-                $module->setErrors(sprintf(constant('CO_' . $moduleDirNameUpper . '_ERROR_BAD_DEL_PATH'), $old_dir));
-                $success = false;
-            }
-        }
-        unset($dirInfo);
-    }
-    /*
+	$old_directories = [$GLOBALS['xoops']->path("uploads/{$moduleDirName}")];
+	foreach ($old_directories as $old_dir) {
+		$dirInfo = new \SplFileInfo($old_dir);
+		if ($dirInfo->isDir()) {
+			// The directory exists so delete it
+			if (false === $utilityClass::rrmdir($old_dir)) {
+				$module->setErrors(sprintf(constant('CO_' . $moduleDirNameUpper . '_ERROR_BAD_DEL_PATH'), $old_dir));
+				$success = false;
+			}
+		}
+		unset($dirInfo);
+	}
+	/*
     //------------ START ----------------
     //------------------------------------------------------------------
     // Remove xsitemap.xml from XOOPS root folder if it exists
@@ -76,6 +76,6 @@ 
 //    return $success && $delOk; // use this if you're using this routine
 */
 
-    return $success;
-    //------------ END  ----------------
+	return $success;
+	//------------ END  ----------------
 }

Spacing +3 added lines, -3 removed lines

@@ -34,10 +34,10 @@ 
 //    return true;
 
     $moduleDirName = basename(dirname(__DIR__));
-    $xsitemapHelper      = \Xmf\Module\Helper::getHelper($moduleDirName);
+    $xsitemapHelper = \Xmf\Module\Helper::getHelper($moduleDirName);
 
     /** @var XXXXXXUtility $utilityClass */
-    $utilityClass     = ucfirst($moduleDirName) . 'Utility';
+    $utilityClass = ucfirst($moduleDirName).'Utility';
     if (!class_exists($utilityClass)) {
         xoops_load('utility', $moduleDirName);
     }
@@ -56,7 +56,7 @@ 
         if ($dirInfo->isDir()) {
             // The directory exists so delete it
             if (false === $utilityClass::rrmdir($old_dir)) {
-                $module->setErrors(sprintf(constant('CO_' . $moduleDirNameUpper . '_ERROR_BAD_DEL_PATH'), $old_dir));
+                $module->setErrors(sprintf(constant('CO_'.$moduleDirNameUpper.'_ERROR_BAD_DEL_PATH'), $old_dir));
                 $success = false;
             }
         }

include/notification.inc.php

Spacing +1 added lines, -1 removed lines

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

include/form.blog.php

Indentation +18 added lines, -18 removed lines

@@ -43,28 +43,28 @@
 $categories_option = $categoryHandler->getList();
 natsort($categories_option);
 if (count($categories_option)) {
-    $cat_option_tray = new \XoopsFormElementTray(planet_constant('MD_CATEGORY'), '<br>');
-    $options         = [0 => _NONE];
-    foreach ($categories_option as $id => $title) {
-        $options[$id] = $title;
-    }
-    $cat_select = new \XoopsFormSelect('', 'categories', $categories, 3, true);
-    $cat_select->addOptionArray($options);
-    $cat_option_tray->addElement($cat_select);
-    $form->addElement($cat_option_tray);
+	$cat_option_tray = new \XoopsFormElementTray(planet_constant('MD_CATEGORY'), '<br>');
+	$options         = [0 => _NONE];
+	foreach ($categories_option as $id => $title) {
+		$options[$id] = $title;
+	}
+	$cat_select = new \XoopsFormSelect('', 'categories', $categories, 3, true);
+	$cat_select->addOptionArray($options);
+	$cat_option_tray->addElement($cat_select);
+	$form->addElement($cat_option_tray);
 }
 
 /* For admin only */
 if (is_object($xoopsUser) && $xoopsUser->isAdmin()) {
-    $status_option_tray = new \XoopsFormElementTray(planet_constant('MD_STATUS'), '<br>');
-    $status_select      = new \XoopsFormSelect('', 'blog_status', $blog_obj->getVar('blog_status'));
-    $status_select->addOptionArray([
-                                       '0' => planet_constant('MD_PENDING'),
-                                       '1' => planet_constant('MD_ACTIVE'),
-                                       '2' => planet_constant('MD_FEATURED')
-                                   ]);
-    $status_option_tray->addElement($status_select);
-    $form->addElement($status_option_tray);
+	$status_option_tray = new \XoopsFormElementTray(planet_constant('MD_STATUS'), '<br>');
+	$status_select      = new \XoopsFormSelect('', 'blog_status', $blog_obj->getVar('blog_status'));
+	$status_select->addOptionArray([
+									   '0' => planet_constant('MD_PENDING'),
+									   '1' => planet_constant('MD_ACTIVE'),
+									   '2' => planet_constant('MD_FEATURED')
+								   ]);
+	$status_option_tray->addElement($status_select);
+	$form->addElement($status_option_tray);
 }
 
 $form->addElement(new \XoopsFormHidden('blog', $blog_id));

Spacing +2 added lines, -2 removed lines

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

include/onupdate.php

Indentation +105 added lines, -105 removed lines

@@ -18,9 +18,9 @@ 
  */
 
 if ((!defined('XOOPS_ROOT_PATH')) || !($GLOBALS['xoopsUser'] instanceof \XoopsUser)
-    || !$GLOBALS['xoopsUser']->IsAdmin()
+	|| !$GLOBALS['xoopsUser']->IsAdmin()
 ) {
-    exit('Restricted access' . PHP_EOL);
+	exit('Restricted access' . PHP_EOL);
 }
 
 /**
@@ -30,9 +30,9 @@ 
  */
 function tableExists($tablename)
 {
-    $result = $GLOBALS['xoopsDB']->queryF("SHOW TABLES LIKE '$tablename'");
+	$result = $GLOBALS['xoopsDB']->queryF("SHOW TABLES LIKE '$tablename'");
 
-    return ($GLOBALS['xoopsDB']->getRowsNum($result) > 0) ? true : false;
+	return ($GLOBALS['xoopsDB']->getRowsNum($result) > 0) ? true : false;
 }
 
 /**
@@ -44,15 +44,15 @@ 
  */
 function xoops_module_pre_update_planet(\XoopsModule $module)
 {
-    $moduleDirName = basename(dirname(__DIR__));
-    /** @var Planet\Helper $helper */
-    /** @var Planet\Utility $utility */
-    $helper       = Planet\Helper::getInstance();
-    $utility      = new Planet\Utility();
-
-    $xoopsSuccess = $utility::checkVerXoops($module);
-    $phpSuccess   = $utility::checkVerPhp($module);
-    return $xoopsSuccess && $phpSuccess;
+	$moduleDirName = basename(dirname(__DIR__));
+	/** @var Planet\Helper $helper */
+	/** @var Planet\Utility $utility */
+	$helper       = Planet\Helper::getInstance();
+	$utility      = new Planet\Utility();
+
+	$xoopsSuccess = $utility::checkVerXoops($module);
+	$phpSuccess   = $utility::checkVerPhp($module);
+	return $xoopsSuccess && $phpSuccess;
 }
 
 /**
@@ -66,96 +66,96 @@ 
 
 function xoops_module_update_planet(\XoopsModule $module, $previousVersion = null)
 {
-    $moduleDirName = basename(dirname(__DIR__));
-    $capsDirName   = strtoupper($moduleDirName);
-
-    /** @var Planet\Helper $helper */
-    /** @var Planet\Utility $utility */
-    /** @var Planet\Configurator $configurator */
-    $helper  = Planet\Helper::getInstance();
-    $utility = new Planet\Utility();
-    $configurator = new Planet\Configurator();
-
-    if ($previousVersion < 240) {
-
-        //rename column EXAMPLE
-        $tables     = new Tables();
-        $table      = 'planetx_categories';
-        $column     = 'ordre';
-        $newName    = 'order';
-        $attributes = "INT(5) NOT NULL DEFAULT '0'";
-        if ($tables->useTable($table)) {
-            $tables->alterColumn($table, $column, $attributes, $newName);
-            if (!$tables->executeQueue()) {
-                echo '<br>' . _AM_XXXXX_UPGRADEFAILED0 . ' ' . $migrate->getLastError();
-            }
-        }
-
-        //delete old HTML templates
-        if (count($configurator->templateFolders) > 0) {
-            foreach ($configurator->templateFolders as $folder) {
-                $templateFolder = $GLOBALS['xoops']->path('modules/' . $moduleDirName . $folder);
-                if (is_dir($templateFolder)) {
-                    $templateList = array_diff(scandir($templateFolder, SCANDIR_SORT_NONE), ['..', '.']);
-                    foreach ($templateList as $k => $v) {
-                        $fileInfo = new \SplFileInfo($templateFolder . $v);
-                        if ('html' === $fileInfo->getExtension() && 'index.html' !== $fileInfo->getFilename()) {
-                            if (file_exists($templateFolder . $v)) {
-                                unlink($templateFolder . $v);
-                            }
-                        }
-                    }
-                }
-            }
-        }
-
-        //  ---  DELETE OLD FILES ---------------
-        if (count($configurator->oldFiles) > 0) {
-            //    foreach (array_keys($GLOBALS['uploadFolders']) as $i) {
-            foreach (array_keys($configurator->oldFiles) as $i) {
-                $tempFile = $GLOBALS['xoops']->path('modules/' . $moduleDirName . $configurator->oldFiles[$i]);
-                if (is_file($tempFile)) {
-                    unlink($tempFile);
-                }
-            }
-        }
-
-        //  ---  DELETE OLD FOLDERS ---------------
-        xoops_load('XoopsFile');
-        if (count($configurator->oldFolders) > 0) {
-            //    foreach (array_keys($GLOBALS['uploadFolders']) as $i) {
-            foreach (array_keys($configurator->oldFolders) as $i) {
-                $tempFolder = $GLOBALS['xoops']->path('modules/' . $moduleDirName . $configurator->oldFolders[$i]);
-                /* @var $folderHandler XoopsObjectHandler */
-                $folderHandler = XoopsFile::getHandler('folder', $tempFolder);
-                $folderHandler->delete($tempFolder);
-            }
-        }
-
-        //  ---  CREATE FOLDERS ---------------
-        if (count($configurator->uploadFolders) > 0) {
-            //    foreach (array_keys($GLOBALS['uploadFolders']) as $i) {
-            foreach (array_keys($configurator->uploadFolders) as $i) {
-                $utilityClass::createFolder($configurator->uploadFolders[$i]);
-            }
-        }
-
-        //  ---  COPY blank.png FILES ---------------
-        if (count($configurator->copyBlankFiles) > 0) {
-            $file = __DIR__ . '/../assets/images/blank.png';
-            foreach (array_keys($configurator->copyBlankFiles) as $i) {
-                $dest = $configurator->copyBlankFiles[$i] . '/blank.png';
-                $utilityClass::copyFile($file, $dest);
-            }
-        }
-
-        //delete .html entries from the tpl table
-        $sql = 'DELETE FROM ' . $GLOBALS['xoopsDB']->prefix('tplfile') . " WHERE `tpl_module` = '" . $module->getVar('dirname', 'n') . '\' AND `tpl_file` LIKE \'%.html%\'';
-        $GLOBALS['xoopsDB']->queryF($sql);
-
-        /** @var XoopsGroupPermHandler $gpermHandler */
-        $gpermHandler = xoops_getHandler('groupperm');
-        return $gpermHandler->deleteByModule($module->getVar('mid'), 'item_read');
-    }
-    return true;
+	$moduleDirName = basename(dirname(__DIR__));
+	$capsDirName   = strtoupper($moduleDirName);
+
+	/** @var Planet\Helper $helper */
+	/** @var Planet\Utility $utility */
+	/** @var Planet\Configurator $configurator */
+	$helper  = Planet\Helper::getInstance();
+	$utility = new Planet\Utility();
+	$configurator = new Planet\Configurator();
+
+	if ($previousVersion < 240) {
+
+		//rename column EXAMPLE
+		$tables     = new Tables();
+		$table      = 'planetx_categories';
+		$column     = 'ordre';
+		$newName    = 'order';
+		$attributes = "INT(5) NOT NULL DEFAULT '0'";
+		if ($tables->useTable($table)) {
+			$tables->alterColumn($table, $column, $attributes, $newName);
+			if (!$tables->executeQueue()) {
+				echo '<br>' . _AM_XXXXX_UPGRADEFAILED0 . ' ' . $migrate->getLastError();
+			}
+		}
+
+		//delete old HTML templates
+		if (count($configurator->templateFolders) > 0) {
+			foreach ($configurator->templateFolders as $folder) {
+				$templateFolder = $GLOBALS['xoops']->path('modules/' . $moduleDirName . $folder);
+				if (is_dir($templateFolder)) {
+					$templateList = array_diff(scandir($templateFolder, SCANDIR_SORT_NONE), ['..', '.']);
+					foreach ($templateList as $k => $v) {
+						$fileInfo = new \SplFileInfo($templateFolder . $v);
+						if ('html' === $fileInfo->getExtension() && 'index.html' !== $fileInfo->getFilename()) {
+							if (file_exists($templateFolder . $v)) {
+								unlink($templateFolder . $v);
+							}
+						}
+					}
+				}
+			}
+		}
+
+		//  ---  DELETE OLD FILES ---------------
+		if (count($configurator->oldFiles) > 0) {
+			//    foreach (array_keys($GLOBALS['uploadFolders']) as $i) {
+			foreach (array_keys($configurator->oldFiles) as $i) {
+				$tempFile = $GLOBALS['xoops']->path('modules/' . $moduleDirName . $configurator->oldFiles[$i]);
+				if (is_file($tempFile)) {
+					unlink($tempFile);
+				}
+			}
+		}
+
+		//  ---  DELETE OLD FOLDERS ---------------
+		xoops_load('XoopsFile');
+		if (count($configurator->oldFolders) > 0) {
+			//    foreach (array_keys($GLOBALS['uploadFolders']) as $i) {
+			foreach (array_keys($configurator->oldFolders) as $i) {
+				$tempFolder = $GLOBALS['xoops']->path('modules/' . $moduleDirName . $configurator->oldFolders[$i]);
+				/* @var $folderHandler XoopsObjectHandler */
+				$folderHandler = XoopsFile::getHandler('folder', $tempFolder);
+				$folderHandler->delete($tempFolder);
+			}
+		}
+
+		//  ---  CREATE FOLDERS ---------------
+		if (count($configurator->uploadFolders) > 0) {
+			//    foreach (array_keys($GLOBALS['uploadFolders']) as $i) {
+			foreach (array_keys($configurator->uploadFolders) as $i) {
+				$utilityClass::createFolder($configurator->uploadFolders[$i]);
+			}
+		}
+
+		//  ---  COPY blank.png FILES ---------------
+		if (count($configurator->copyBlankFiles) > 0) {
+			$file = __DIR__ . '/../assets/images/blank.png';
+			foreach (array_keys($configurator->copyBlankFiles) as $i) {
+				$dest = $configurator->copyBlankFiles[$i] . '/blank.png';
+				$utilityClass::copyFile($file, $dest);
+			}
+		}
+
+		//delete .html entries from the tpl table
+		$sql = 'DELETE FROM ' . $GLOBALS['xoopsDB']->prefix('tplfile') . " WHERE `tpl_module` = '" . $module->getVar('dirname', 'n') . '\' AND `tpl_file` LIKE \'%.html%\'';
+		$GLOBALS['xoopsDB']->queryF($sql);
+
+		/** @var XoopsGroupPermHandler $gpermHandler */
+		$gpermHandler = xoops_getHandler('groupperm');
+		return $gpermHandler->deleteByModule($module->getVar('mid'), 'item_read');
+	}
+	return true;
 }

Spacing +18 added lines, -18 removed lines

@@ -20,7 +20,7 @@ 
 if ((!defined('XOOPS_ROOT_PATH')) || !($GLOBALS['xoopsUser'] instanceof \XoopsUser)
     || !$GLOBALS['xoopsUser']->IsAdmin()
 ) {
-    exit('Restricted access' . PHP_EOL);
+    exit('Restricted access'.PHP_EOL);
 }
 
 /**
@@ -32,7 +32,7 @@ 
 {
     $result = $GLOBALS['xoopsDB']->queryF("SHOW TABLES LIKE '$tablename'");
 
-    return ($GLOBALS['xoopsDB']->getRowsNum($result) > 0) ? true : false;
+    return ($GLOBALS['xoopsDB']->getRowsNum($result)>0) ? true : false;
 }
 
 /**
@@ -76,7 +76,7 @@ 
     $utility = new Planet\Utility();
     $configurator = new Planet\Configurator();
 
-    if ($previousVersion < 240) {
+    if ($previousVersion<240) {
 
         //rename column EXAMPLE
         $tables     = new Tables();
@@ -87,21 +87,21 @@ 
         if ($tables->useTable($table)) {
             $tables->alterColumn($table, $column, $attributes, $newName);
             if (!$tables->executeQueue()) {
-                echo '<br>' . _AM_XXXXX_UPGRADEFAILED0 . ' ' . $migrate->getLastError();
+                echo '<br>'._AM_XXXXX_UPGRADEFAILED0.' '.$migrate->getLastError();
             }
         }
 
         //delete old HTML templates
-        if (count($configurator->templateFolders) > 0) {
+        if (count($configurator->templateFolders)>0) {
             foreach ($configurator->templateFolders as $folder) {
-                $templateFolder = $GLOBALS['xoops']->path('modules/' . $moduleDirName . $folder);
+                $templateFolder = $GLOBALS['xoops']->path('modules/'.$moduleDirName.$folder);
                 if (is_dir($templateFolder)) {
                     $templateList = array_diff(scandir($templateFolder, SCANDIR_SORT_NONE), ['..', '.']);
                     foreach ($templateList as $k => $v) {
-                        $fileInfo = new \SplFileInfo($templateFolder . $v);
+                        $fileInfo = new \SplFileInfo($templateFolder.$v);
                         if ('html' === $fileInfo->getExtension() && 'index.html' !== $fileInfo->getFilename()) {
-                            if (file_exists($templateFolder . $v)) {
-                                unlink($templateFolder . $v);
+                            if (file_exists($templateFolder.$v)) {
+                                unlink($templateFolder.$v);
                             }
                         }
                     }
@@ -110,10 +110,10 @@ 
         }
 
         //  ---  DELETE OLD FILES ---------------
-        if (count($configurator->oldFiles) > 0) {
+        if (count($configurator->oldFiles)>0) {
             //    foreach (array_keys($GLOBALS['uploadFolders']) as $i) {
             foreach (array_keys($configurator->oldFiles) as $i) {
-                $tempFile = $GLOBALS['xoops']->path('modules/' . $moduleDirName . $configurator->oldFiles[$i]);
+                $tempFile = $GLOBALS['xoops']->path('modules/'.$moduleDirName.$configurator->oldFiles[$i]);
                 if (is_file($tempFile)) {
                     unlink($tempFile);
                 }
@@ -122,10 +122,10 @@ 
 
         //  ---  DELETE OLD FOLDERS ---------------
         xoops_load('XoopsFile');
-        if (count($configurator->oldFolders) > 0) {
+        if (count($configurator->oldFolders)>0) {
             //    foreach (array_keys($GLOBALS['uploadFolders']) as $i) {
             foreach (array_keys($configurator->oldFolders) as $i) {
-                $tempFolder = $GLOBALS['xoops']->path('modules/' . $moduleDirName . $configurator->oldFolders[$i]);
+                $tempFolder = $GLOBALS['xoops']->path('modules/'.$moduleDirName.$configurator->oldFolders[$i]);
                 /* @var $folderHandler XoopsObjectHandler */
                 $folderHandler = XoopsFile::getHandler('folder', $tempFolder);
                 $folderHandler->delete($tempFolder);
@@ -133,7 +133,7 @@ 
         }
 
         //  ---  CREATE FOLDERS ---------------
-        if (count($configurator->uploadFolders) > 0) {
+        if (count($configurator->uploadFolders)>0) {
             //    foreach (array_keys($GLOBALS['uploadFolders']) as $i) {
             foreach (array_keys($configurator->uploadFolders) as $i) {
                 $utilityClass::createFolder($configurator->uploadFolders[$i]);
@@ -141,16 +141,16 @@ 
         }
 
         //  ---  COPY blank.png FILES ---------------
-        if (count($configurator->copyBlankFiles) > 0) {
-            $file = __DIR__ . '/../assets/images/blank.png';
+        if (count($configurator->copyBlankFiles)>0) {
+            $file = __DIR__.'/../assets/images/blank.png';
             foreach (array_keys($configurator->copyBlankFiles) as $i) {
-                $dest = $configurator->copyBlankFiles[$i] . '/blank.png';
+                $dest = $configurator->copyBlankFiles[$i].'/blank.png';
                 $utilityClass::copyFile($file, $dest);
             }
         }
 
         //delete .html entries from the tpl table
-        $sql = 'DELETE FROM ' . $GLOBALS['xoopsDB']->prefix('tplfile') . " WHERE `tpl_module` = '" . $module->getVar('dirname', 'n') . '\' AND `tpl_file` LIKE \'%.html%\'';
+        $sql = 'DELETE FROM '.$GLOBALS['xoopsDB']->prefix('tplfile')." WHERE `tpl_module` = '".$module->getVar('dirname', 'n').'\' AND `tpl_file` LIKE \'%.html%\'';
         $GLOBALS['xoopsDB']->queryF($sql);
 
         /** @var XoopsGroupPermHandler $gpermHandler */