mambax7 / planet

include/comment.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/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/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 */