XoopsModules25x / myiframe

preloads/core.php

Indentation +8 added lines, -8 removed lines

@@ -20,12 +20,12 @@
  */
 class MyiframeCorePreload extends \XoopsPreloadItem
 {
-    // to add PSR-4 autoloader
-    /**
-     * @param $args
-     */
-    public static function eventCoreIncludeCommonEnd($args): void
-    {
-        require_once __DIR__ . '/autoloader.php';
-    }
+	// to add PSR-4 autoloader
+	/**
+	 * @param $args
+	 */
+	public static function eventCoreIncludeCommonEnd($args): void
+	{
+		require_once __DIR__ . '/autoloader.php';
+	}
 }

include/config.php

Indentation +56 added lines, -56 removed lines

@@ -21,63 +21,63 @@
 $moduleDirNameUpper = \mb_strtoupper($moduleDirName);
 
 return (object)[
-    'name'           => $moduleDirNameUpper . ' Module Configurator',
-    'paths'          => [
-        'dirname'    => $moduleDirName,
-        'admin'      => XOOPS_ROOT_PATH . '/modules/' . $moduleDirName . '/admin',
-        'modPath'    => XOOPS_ROOT_PATH . '/modules/' . $moduleDirName,
-        'modUrl'     => XOOPS_URL . '/modules/' . $moduleDirName,
-        'uploadPath' => XOOPS_UPLOAD_PATH . '/' . $moduleDirName,
-        'uploadUrl'  => XOOPS_UPLOAD_URL . '/' . $moduleDirName,
-    ],
-    'uploadFolders'  => [
-        XOOPS_UPLOAD_PATH . '/' . $moduleDirName,
-        XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/category',
-        XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/screenshots',
-        //XOOPS_UPLOAD_PATH . '/flags'
-    ],
-    'copyBlankFiles' => [
-        XOOPS_UPLOAD_PATH . '/' . $moduleDirName,
-        XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/category',
-        XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/screenshots',
-        //XOOPS_UPLOAD_PATH . '/flags'
-    ],
+	'name'           => $moduleDirNameUpper . ' Module Configurator',
+	'paths'          => [
+		'dirname'    => $moduleDirName,
+		'admin'      => XOOPS_ROOT_PATH . '/modules/' . $moduleDirName . '/admin',
+		'modPath'    => XOOPS_ROOT_PATH . '/modules/' . $moduleDirName,
+		'modUrl'     => XOOPS_URL . '/modules/' . $moduleDirName,
+		'uploadPath' => XOOPS_UPLOAD_PATH . '/' . $moduleDirName,
+		'uploadUrl'  => XOOPS_UPLOAD_URL . '/' . $moduleDirName,
+	],
+	'uploadFolders'  => [
+		XOOPS_UPLOAD_PATH . '/' . $moduleDirName,
+		XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/category',
+		XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/screenshots',
+		//XOOPS_UPLOAD_PATH . '/flags'
+	],
+	'copyBlankFiles' => [
+		XOOPS_UPLOAD_PATH . '/' . $moduleDirName,
+		XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/category',
+		XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/screenshots',
+		//XOOPS_UPLOAD_PATH . '/flags'
+	],
 
-    'copyTestFolders' => [
-        //[
-        //    constant($moduleDirNameUpper . '_PATH') . '/testdata/images',
-        //    XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/images',
-        //]
-    ],
+	'copyTestFolders' => [
+		//[
+		//    constant($moduleDirNameUpper . '_PATH') . '/testdata/images',
+		//    XOOPS_UPLOAD_PATH . '/' . $moduleDirName . '/images',
+		//]
+	],
 
-    'templateFolders' => [
-        '/templates/',
-        '/templates/blocks/',
-        '/templates/admin/',
-    ],
-    'oldFiles'        => [
-        '/class/request.php',
-        '/class/registry.php',
-        '/class/utilities.php',
-        '/class/util.php',
-        // '/include/constants.php',
-        // '/include/functions.php',
-        '/ajaxrating.txt',
-    ],
-    'oldFolders'      => [
-        '/images',
-        '/css',
-        '/js',
-        '/tcpdf',
-        '/images',
-    ],
-    'renameTables'    => [//         'XX_archive'     => 'ZZZZ_archive',
-    ],
-    'moduleStats'     => [
-        //            'totalcategories' => $helper->getHandler('Category')->getCategoriesCount(-1),
-        //            'totalitems'      => $helper->getHandler('Item')->getItemsCount(),
-        //            'totalsubmitted'  => $helper->getHandler('Item')->getItemsCount(-1, [Constants::PUBLISHER_STATUS_SUBMITTED]),
-    ],
-    'modCopyright'    => "<a href='https://xoops.org' title='XOOPS Project' target='_blank'>
+	'templateFolders' => [
+		'/templates/',
+		'/templates/blocks/',
+		'/templates/admin/',
+	],
+	'oldFiles'        => [
+		'/class/request.php',
+		'/class/registry.php',
+		'/class/utilities.php',
+		'/class/util.php',
+		// '/include/constants.php',
+		// '/include/functions.php',
+		'/ajaxrating.txt',
+	],
+	'oldFolders'      => [
+		'/images',
+		'/css',
+		'/js',
+		'/tcpdf',
+		'/images',
+	],
+	'renameTables'    => [//         'XX_archive'     => 'ZZZZ_archive',
+	],
+	'moduleStats'     => [
+		//            'totalcategories' => $helper->getHandler('Category')->getCategoriesCount(-1),
+		//            'totalitems'      => $helper->getHandler('Item')->getItemsCount(),
+		//            'totalsubmitted'  => $helper->getHandler('Item')->getItemsCount(-1, [Constants::PUBLISHER_STATUS_SUBMITTED]),
+	],
+	'modCopyright'    => "<a href='https://xoops.org' title='XOOPS Project' target='_blank'>
                      <img src='" . Admin::iconUrl('xoopsmicrobutton.gif') . "' alt='XOOPS Project'></a>",
 ];

include/onupdate.php

Indentation +102 added lines, -102 removed lines

@@ -19,8 +19,8 @@ 
 use XoopsModules\Myiframe;
 
 if ((!defined('XOOPS_ROOT_PATH')) || !($GLOBALS['xoopsUser'] instanceof \XoopsUser)
-    || !$GLOBALS['xoopsUser']->isAdmin()) {
-    exit('Restricted access' . PHP_EOL);
+	|| !$GLOBALS['xoopsUser']->isAdmin()) {
+	exit('Restricted access' . PHP_EOL);
 }
 
 /**
@@ -31,16 +31,16 @@ 
  */
 function xoops_module_pre_update_myiframe(\XoopsModule $module)
 {
-    $moduleDirName = \basename(\dirname(__DIR__));
-    /** @var Myiframe\Helper $helper */
-    /** @var Myiframe\Utility $utility */
-    $helper  = Myiframe\Helper::getInstance();
-    $utility = new Myiframe\Utility();
+	$moduleDirName = \basename(\dirname(__DIR__));
+	/** @var Myiframe\Helper $helper */
+	/** @var Myiframe\Utility $utility */
+	$helper  = Myiframe\Helper::getInstance();
+	$utility = new Myiframe\Utility();
 
-    $xoopsSuccess = $utility::checkVerXoops($module);
-    $phpSuccess   = $utility::checkVerPhp($module);
+	$xoopsSuccess = $utility::checkVerXoops($module);
+	$phpSuccess   = $utility::checkVerPhp($module);
 
-    return $xoopsSuccess && $phpSuccess;
+	return $xoopsSuccess && $phpSuccess;
 }
 
 /**
@@ -52,96 +52,96 @@ 
  */
 function xoops_module_update_myiframe(\XoopsModule $module, $previousVersion = null)
 {
-    $moduleDirName      = \basename(\dirname(__DIR__));
-    $moduleDirNameUpper = \mb_strtoupper($moduleDirName);
-
-    /** @var Myiframe\Helper $helper */ /** @var Myiframe\Utility $utility */
-    /** @var Myiframe\Common\Configurator $configurator */
-    $helper       = Myiframe\Helper::getInstance();
-    $utility      = new Myiframe\Utility();
-    $configurator = new Myiframe\Common\Configurator();
-
-    if ($previousVersion < 240) {
-        //rename column EXAMPLE
-        $tables     = new Tables();
-        $table      = 'myiframex_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_MYIFRAME_UPGRADEFAILED0 . ' ' . $tables->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 (is_file($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 XoopsObjectHandler $folderHandler */
-                $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) {
-                $utility::createFolder($configurator->uploadFolders[$i]);
-            }
-        }
-
-        //  ---  COPY blank.png FILES ---------------
-        if (count($configurator->copyBlankFiles) > 0) {
-            $file = \dirname(__DIR__) . '/assets/images/blank.png';
-            foreach (array_keys($configurator->copyBlankFiles) as $i) {
-                $dest = $configurator->copyBlankFiles[$i] . '/blank.png';
-                $utility::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 $grouppermHandler */
-        $grouppermHandler = xoops_getHandler('groupperm');
-
-        return $grouppermHandler->deleteByModule($module->getVar('mid'), 'item_read');
-    }
-
-    return true;
+	$moduleDirName      = \basename(\dirname(__DIR__));
+	$moduleDirNameUpper = \mb_strtoupper($moduleDirName);
+
+	/** @var Myiframe\Helper $helper */ /** @var Myiframe\Utility $utility */
+	/** @var Myiframe\Common\Configurator $configurator */
+	$helper       = Myiframe\Helper::getInstance();
+	$utility      = new Myiframe\Utility();
+	$configurator = new Myiframe\Common\Configurator();
+
+	if ($previousVersion < 240) {
+		//rename column EXAMPLE
+		$tables     = new Tables();
+		$table      = 'myiframex_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_MYIFRAME_UPGRADEFAILED0 . ' ' . $tables->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 (is_file($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 XoopsObjectHandler $folderHandler */
+				$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) {
+				$utility::createFolder($configurator->uploadFolders[$i]);
+			}
+		}
+
+		//  ---  COPY blank.png FILES ---------------
+		if (count($configurator->copyBlankFiles) > 0) {
+			$file = \dirname(__DIR__) . '/assets/images/blank.png';
+			foreach (array_keys($configurator->copyBlankFiles) as $i) {
+				$dest = $configurator->copyBlankFiles[$i] . '/blank.png';
+				$utility::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 $grouppermHandler */
+		$grouppermHandler = xoops_getHandler('groupperm');
+
+		return $grouppermHandler->deleteByModule($module->getVar('mid'), 'item_read');
+	}
+
+	return true;
 }

include/functions.php

Indentation +44 added lines, -44 removed lines

@@ -21,33 +21,33 @@ 
  */
 function myiframe_getmoduleoption($option, $repmodule = 'myiframe')
 {
-    global $xoopsModuleConfig, $xoopsModule;
-    static $tbloptions = [];
-    if (is_array($tbloptions) && array_key_exists($option, $tbloptions)) {
-        return $tbloptions[$option];
-    }
+	global $xoopsModuleConfig, $xoopsModule;
+	static $tbloptions = [];
+	if (is_array($tbloptions) && array_key_exists($option, $tbloptions)) {
+		return $tbloptions[$option];
+	}
 
-    $retval = false;
-    if (isset($xoopsModuleConfig) && (is_object($xoopsModule) && $xoopsModule->getVar('dirname') == $repmodule && $xoopsModule->getVar('isactive'))) {
-        if (isset($xoopsModuleConfig[$option])) {
-            $retval = $xoopsModuleConfig[$option];
-        }
-    } else {
-        /** @var \XoopsModuleHandler $moduleHandler */
-        $moduleHandler = xoops_getHandler('module');
-        $module        = $moduleHandler->getByDirname($repmodule);
-        /** @var \XoopsConfigHandler $configHandler */
-        $configHandler = xoops_getHandler('config');
-        if ($module) {
-            $moduleConfig = $configHandler->getConfigsByCat(0, $module->getVar('mid'));
-            if (isset($moduleConfig[$option])) {
-                $retval = $moduleConfig[$option];
-            }
-        }
-    }
-    $tbloptions[$option] = $retval;
+	$retval = false;
+	if (isset($xoopsModuleConfig) && (is_object($xoopsModule) && $xoopsModule->getVar('dirname') == $repmodule && $xoopsModule->getVar('isactive'))) {
+		if (isset($xoopsModuleConfig[$option])) {
+			$retval = $xoopsModuleConfig[$option];
+		}
+	} else {
+		/** @var \XoopsModuleHandler $moduleHandler */
+		$moduleHandler = xoops_getHandler('module');
+		$module        = $moduleHandler->getByDirname($repmodule);
+		/** @var \XoopsConfigHandler $configHandler */
+		$configHandler = xoops_getHandler('config');
+		if ($module) {
+			$moduleConfig = $configHandler->getConfigsByCat(0, $module->getVar('mid'));
+			if (isset($moduleConfig[$option])) {
+				$retval = $moduleConfig[$option];
+			}
+		}
+	}
+	$tbloptions[$option] = $retval;
 
-    return $retval;
+	return $retval;
 }
 
 /**
@@ -61,9 +61,9 @@ 
  */
 function myiframe_FieldExists($fieldname, $table)
 {
-    $result = $GLOBALS['xoopsDB']->queryF("SHOW COLUMNS FROM    $table LIKE '$fieldname'");
+	$result = $GLOBALS['xoopsDB']->queryF("SHOW COLUMNS FROM    $table LIKE '$fieldname'");
 
-    return ($GLOBALS['xoopsDB']->getRowsNum($result) > 0);
+	return ($GLOBALS['xoopsDB']->getRowsNum($result) > 0);
 }
 
 /**
@@ -78,21 +78,21 @@ 
  */
 function myiframe_set_metas($page_title = '', $meta_description = '', $meta_keywords = ''): void
 {
-    global $xoTheme, $xoTheme, $xoopsTpl;
-    $xoopsTpl->assign('xoops_pagetitle', $page_title);
-    if (isset($xoTheme) && is_object($xoTheme)) {
-        if (!empty($meta_keywords)) {
-            $xoTheme->addMeta('meta', 'keywords', $meta_keywords);
-        }
-        if (!empty($meta_description)) {
-            $xoTheme->addMeta('meta', 'description', $meta_description);
-        }
-    } elseif (isset($xoopsTpl) && is_object($xoopsTpl)) {    // Compatibility for old Xoops versions
-        if (!empty($meta_keywords)) {
-            $xoopsTpl->assign('xoops_meta_keywords', $meta_keywords);
-        }
-        if (!empty($meta_description)) {
-            $xoopsTpl->assign('xoops_meta_description', $meta_description);
-        }
-    }
+	global $xoTheme, $xoTheme, $xoopsTpl;
+	$xoopsTpl->assign('xoops_pagetitle', $page_title);
+	if (isset($xoTheme) && is_object($xoTheme)) {
+		if (!empty($meta_keywords)) {
+			$xoTheme->addMeta('meta', 'keywords', $meta_keywords);
+		}
+		if (!empty($meta_description)) {
+			$xoTheme->addMeta('meta', 'description', $meta_description);
+		}
+	} elseif (isset($xoopsTpl) && is_object($xoopsTpl)) {    // Compatibility for old Xoops versions
+		if (!empty($meta_keywords)) {
+			$xoopsTpl->assign('xoops_meta_keywords', $meta_keywords);
+		}
+		if (!empty($meta_description)) {
+			$xoopsTpl->assign('xoops_meta_description', $meta_description);
+		}
+	}
 }

include/oninstall.php

Indentation +54 added lines, -54 removed lines

@@ -28,20 +28,20 @@ 
  */
 function xoops_module_pre_install_myiframe(\XoopsModule $module)
 {
-    require_once \dirname(__DIR__) . '/preloads/autoloader.php';
-    /** @var \Utility $utility */
-    $utility      = new Utility();
-    $xoopsSuccess = $utility::checkVerXoops($module);
-    $phpSuccess   = $utility::checkVerPhp($module);
+	require_once \dirname(__DIR__) . '/preloads/autoloader.php';
+	/** @var \Utility $utility */
+	$utility      = new 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) . ';');
-        }
-    }
+	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;
+	return $xoopsSuccess && $phpSuccess;
 }
 
 /**
@@ -52,52 +52,52 @@ 
  */
 function xoops_module_install_myiframe(\XoopsModule $module)
 {
-    require \dirname(__DIR__, 3) . '/mainfile.php';
+	require \dirname(__DIR__, 3) . '/mainfile.php';
 
-    $moduleDirName = \basename(\dirname(__DIR__));
+	$moduleDirName = \basename(\dirname(__DIR__));
 
-    /** @var Myiframe\Helper $helper */
-    $helper = Myiframe\Helper::getInstance();
-    /** @var Myiframe\Utility $helper */
-    $utility = new Utility();
-    /** @var Myiframe\Common\Configurator $helper */
-    $configurator = new Myiframe\Common\Configurator();
-    // Load language files
-    $helper->loadLanguage('admin');
-    $helper->loadLanguage('modinfo');
+	/** @var Myiframe\Helper $helper */
+	$helper = Myiframe\Helper::getInstance();
+	/** @var Myiframe\Utility $helper */
+	$utility = new Utility();
+	/** @var Myiframe\Common\Configurator $helper */
+	$configurator = new Myiframe\Common\Configurator();
+	// Load language files
+	$helper->loadLanguage('admin');
+	$helper->loadLanguage('modinfo');
 
-    // default Permission Settings ----------------------
-    global $xoopsModule;
-    $moduleId = $xoopsModule->getVar('mid');
-    // $moduleId2        = $helper->getModule()->mid();
-    /** @var \XoopsGroupPermHandler $grouppermHandler */
-    $grouppermHandler = xoops_getHandler('groupperm');
-    // access rights ------------------------------------------
-    $grouppermHandler->addRight($moduleDirName . '_approve', 1, XOOPS_GROUP_ADMIN, $moduleId);
-    $grouppermHandler->addRight($moduleDirName . '_submit', 1, XOOPS_GROUP_ADMIN, $moduleId);
-    $grouppermHandler->addRight($moduleDirName . '_view', 1, XOOPS_GROUP_ADMIN, $moduleId);
-    $grouppermHandler->addRight($moduleDirName . '_view', 1, XOOPS_GROUP_USERS, $moduleId);
-    $grouppermHandler->addRight($moduleDirName . '_view', 1, XOOPS_GROUP_ANONYMOUS, $moduleId);
+	// default Permission Settings ----------------------
+	global $xoopsModule;
+	$moduleId = $xoopsModule->getVar('mid');
+	// $moduleId2        = $helper->getModule()->mid();
+	/** @var \XoopsGroupPermHandler $grouppermHandler */
+	$grouppermHandler = xoops_getHandler('groupperm');
+	// access rights ------------------------------------------
+	$grouppermHandler->addRight($moduleDirName . '_approve', 1, XOOPS_GROUP_ADMIN, $moduleId);
+	$grouppermHandler->addRight($moduleDirName . '_submit', 1, XOOPS_GROUP_ADMIN, $moduleId);
+	$grouppermHandler->addRight($moduleDirName . '_view', 1, XOOPS_GROUP_ADMIN, $moduleId);
+	$grouppermHandler->addRight($moduleDirName . '_view', 1, XOOPS_GROUP_USERS, $moduleId);
+	$grouppermHandler->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) {
-            $utility::createFolder($configurator->uploadFolders[$i]);
-        }
-    }
+	//  ---  CREATE FOLDERS ---------------
+	if (count($configurator->uploadFolders) > 0) {
+		//    foreach (array_keys($GLOBALS['uploadFolders']) as $i) {
+		foreach (array_keys($configurator->uploadFolders) as $i) {
+			$utility::createFolder($configurator->uploadFolders[$i]);
+		}
+	}
 
-    //  ---  COPY blank.png FILES ---------------
-    if (count($configurator->copyBlankFiles) > 0) {
-        $file = \dirname(__DIR__) . '/assets/images/blank.png';
-        foreach (array_keys($configurator->copyBlankFiles) as $i) {
-            $dest = $configurator->copyBlankFiles[$i] . '/blank.png';
-            $utility::copyFile($file, $dest);
-        }
-    }
-    //delete .html entries from the tpl table
-    $sql = 'DELETE FROM ' . $GLOBALS['xoopsDB']->prefix('tplfile') . " WHERE `tpl_module` = '" . $xoopsModule->getVar('dirname', 'n') . "' AND `tpl_file` LIKE '%.html%'";
-    $GLOBALS['xoopsDB']->queryF($sql);
+	//  ---  COPY blank.png FILES ---------------
+	if (count($configurator->copyBlankFiles) > 0) {
+		$file = \dirname(__DIR__) . '/assets/images/blank.png';
+		foreach (array_keys($configurator->copyBlankFiles) as $i) {
+			$dest = $configurator->copyBlankFiles[$i] . '/blank.png';
+			$utility::copyFile($file, $dest);
+		}
+	}
+	//delete .html entries from the tpl table
+	$sql = 'DELETE FROM ' . $GLOBALS['xoopsDB']->prefix('tplfile') . " WHERE `tpl_module` = '" . $xoopsModule->getVar('dirname', 'n') . "' AND `tpl_file` LIKE '%.html%'";
+	$GLOBALS['xoopsDB']->queryF($sql);
 
-    return true;
+	return true;
 }

class/Utility.php

Indentation +1 added lines, -1 removed lines

@@ -30,5 +30,5 @@
  */
 class Utility extends Common\SysUtility
 {
-    //--------------- Custom module methods -----------------------------
+	//--------------- Custom module methods -----------------------------
 }

class/Common/Breadcrumb.php

Indentation +27 added lines, -27 removed lines

@@ -25,39 +25,39 @@ 
  */
 class Breadcrumb
 {
-    public  $dirname;
-    private $bread = [];
+	public  $dirname;
+	private $bread = [];
 
-    public function __construct()
-    {
-        $this->dirname = \basename(\dirname(__DIR__, 2));
-    }
+	public function __construct()
+	{
+		$this->dirname = \basename(\dirname(__DIR__, 2));
+	}
 
-    /**
-     * Add link to breadcrumb
-     *
-     * @param string $title
-     * @param string $link
-     */
-    public function addLink($title = '', $link = ''): void
-    {
-        $this->bread[] = [
-            'link'  => $link,
-            'title' => $title,
-        ];
-    }
+	/**
+	 * Add link to breadcrumb
+	 *
+	 * @param string $title
+	 * @param string $link
+	 */
+	public function addLink($title = '', $link = ''): void
+	{
+		$this->bread[] = [
+			'link'  => $link,
+			'title' => $title,
+		];
+	}
 
-    /**
-     * Render BreadCrumb
-     */
-    public function render(): void
-    {
-        /*
+	/**
+	 * Render BreadCrumb
+	 */
+	public function render(): void
+	{
+		/*
         TODO if you want to use the render code below,
         1) create ./templates/chess_common_breadcrumb.tpl)
         2) add declaration to  xoops_version.php
         */
-        /*
+		/*
         if (!isset($GLOBALS['xoTheme']) || !\is_object($GLOBALS['xoTheme'])) {
             require $GLOBALS['xoops']->path('class/theme.php');
 
@@ -76,5 +76,5 @@ 
 
         return $html;
         */
-    }
+	}
 }

class/Common/Configurator.php

Indentation +34 added lines, -34 removed lines

@@ -24,41 +24,41 @@
  */
 class Configurator
 {
-    public $name;
-    public $paths           = [];
-    public $uploadFolders   = [];
-    public $copyBlankFiles  = [];
-    public $copyTestFolders = [];
-    public $templateFolders = [];
-    public $oldFiles        = [];
-    public $oldFolders      = [];
-    public $renameTables    = [];
-    public $renameColumns   = [];
-    public $moduleStats     = [];
-    public $modCopyright;
-    public $icons;
+	public $name;
+	public $paths           = [];
+	public $uploadFolders   = [];
+	public $copyBlankFiles  = [];
+	public $copyTestFolders = [];
+	public $templateFolders = [];
+	public $oldFiles        = [];
+	public $oldFolders      = [];
+	public $renameTables    = [];
+	public $renameColumns   = [];
+	public $moduleStats     = [];
+	public $modCopyright;
+	public $icons;
 
-    /**
-     * Configurator constructor.
-     */
-    public function __construct()
-    {
-        $config = require \dirname(__DIR__, 2) . '/config/config.php';
+	/**
+	 * Configurator constructor.
+	 */
+	public function __construct()
+	{
+		$config = require \dirname(__DIR__, 2) . '/config/config.php';
 
-        $this->name            = $config->name;
-        $this->paths           = $config->paths;
-        $this->uploadFolders   = $config->uploadFolders;
-        $this->copyBlankFiles  = $config->copyBlankFiles;
-        $this->copyTestFolders = $config->copyTestFolders;
-        $this->templateFolders = $config->templateFolders;
-        $this->oldFiles        = $config->oldFiles;
-        $this->oldFolders      = $config->oldFolders;
-        $this->renameTables    = $config->renameTables;
-        $this->renameColumns   = $config->renameColumns;
-        $this->moduleStats     = $config->moduleStats;
-        $this->modCopyright    = $config->modCopyright;
+		$this->name            = $config->name;
+		$this->paths           = $config->paths;
+		$this->uploadFolders   = $config->uploadFolders;
+		$this->copyBlankFiles  = $config->copyBlankFiles;
+		$this->copyTestFolders = $config->copyTestFolders;
+		$this->templateFolders = $config->templateFolders;
+		$this->oldFiles        = $config->oldFiles;
+		$this->oldFolders      = $config->oldFolders;
+		$this->renameTables    = $config->renameTables;
+		$this->renameColumns   = $config->renameColumns;
+		$this->moduleStats     = $config->moduleStats;
+		$this->modCopyright    = $config->modCopyright;
 
-        $this->icons = require \dirname(__DIR__, 2) . '/config/icons.php';
-        $this->paths = require \dirname(__DIR__, 2) . '/config/paths.php';
-    }
+		$this->icons = require \dirname(__DIR__, 2) . '/config/icons.php';
+		$this->paths = require \dirname(__DIR__, 2) . '/config/paths.php';
+	}
 }

class/MyiframeBase.php

Indentation +44 added lines, -44 removed lines

@@ -15,50 +15,50 @@
  */
 class MyiframeBase extends \XoopsObject
 {
-    //    /** @var \XoopsMySQLDatabase */
-    public $db;
+	//    /** @var \XoopsMySQLDatabase */
+	public $db;
 
-    /**
-     * myiframe constructor.
-     * @param null $id
-     */
-    public function __construct($id = null)
-    {
-        $this->db = \XoopsDatabaseFactory::getDatabaseConnection();
-        $this->initVar('frame_frameid', \XOBJ_DTYPE_INT, null, false, 10);
-        $this->initVar('frame_created', \XOBJ_DTYPE_INT, null, false, 10);
-        $this->initVar('frame_uid', \XOBJ_DTYPE_INT, null, false, 10);
-        $this->initVar('frame_description', \XOBJ_DTYPE_TXTBOX, null, false, 255);
-        $this->initVar('frame_width', \XOBJ_DTYPE_TXTBOX, null, false, 15);
-        $this->initVar('frame_height', \XOBJ_DTYPE_TXTBOX, null, false, 15);
-        $this->initVar('frame_align', \XOBJ_DTYPE_INT, null, false, 10);
-        $this->initVar('frame_frameborder', \XOBJ_DTYPE_INT, null, false, 10);
-        $this->initVar('frame_marginwidth', \XOBJ_DTYPE_INT, null, false, 10);
-        $this->initVar('frame_marginheight', \XOBJ_DTYPE_INT, null, false, 10);
-        $this->initVar('frame_scrolling', \XOBJ_DTYPE_INT, null, false, 10);
-        $this->initVar('frame_hits', \XOBJ_DTYPE_INT, null, false, 10);
-        $this->initVar('frame_url', \XOBJ_DTYPE_TXTBOX, null, false, 255);
-        if (!empty($id)) {
-            if (\is_array($id)) {
-                $this->assignVars($id);
-            } else {
-                $this->load((int)$id);
-            }
-        } else {
-            $this->setNew();
-        }
-    }
+	/**
+	 * myiframe constructor.
+	 * @param null $id
+	 */
+	public function __construct($id = null)
+	{
+		$this->db = \XoopsDatabaseFactory::getDatabaseConnection();
+		$this->initVar('frame_frameid', \XOBJ_DTYPE_INT, null, false, 10);
+		$this->initVar('frame_created', \XOBJ_DTYPE_INT, null, false, 10);
+		$this->initVar('frame_uid', \XOBJ_DTYPE_INT, null, false, 10);
+		$this->initVar('frame_description', \XOBJ_DTYPE_TXTBOX, null, false, 255);
+		$this->initVar('frame_width', \XOBJ_DTYPE_TXTBOX, null, false, 15);
+		$this->initVar('frame_height', \XOBJ_DTYPE_TXTBOX, null, false, 15);
+		$this->initVar('frame_align', \XOBJ_DTYPE_INT, null, false, 10);
+		$this->initVar('frame_frameborder', \XOBJ_DTYPE_INT, null, false, 10);
+		$this->initVar('frame_marginwidth', \XOBJ_DTYPE_INT, null, false, 10);
+		$this->initVar('frame_marginheight', \XOBJ_DTYPE_INT, null, false, 10);
+		$this->initVar('frame_scrolling', \XOBJ_DTYPE_INT, null, false, 10);
+		$this->initVar('frame_hits', \XOBJ_DTYPE_INT, null, false, 10);
+		$this->initVar('frame_url', \XOBJ_DTYPE_TXTBOX, null, false, 255);
+		if (!empty($id)) {
+			if (\is_array($id)) {
+				$this->assignVars($id);
+			} else {
+				$this->load((int)$id);
+			}
+		} else {
+			$this->setNew();
+		}
+	}
 
-    /**
-     * @param $id
-     */
-    public function load($id): void
-    {
-        $sql   = 'SELECT * FROM ' . $this->db->prefix('myiframe') . ' WHERE frame_frameid=' . (int)$id;
-        $myrow = $this->db->fetchArray($this->db->query($sql));
-        $this->assignVars($myrow);
-        if (!$myrow) {
-            $this->setNew();
-        }
-    }
+	/**
+	 * @param $id
+	 */
+	public function load($id): void
+	{
+		$sql   = 'SELECT * FROM ' . $this->db->prefix('myiframe') . ' WHERE frame_frameid=' . (int)$id;
+		$myrow = $this->db->fetchArray($this->db->query($sql));
+		$this->assignVars($myrow);
+		if (!$myrow) {
+			$this->setNew();
+		}
+	}
 }