mambax7 / planet

include/oninstall.php

<?php
/*
 * You may not change or alter any portion of this comment or credits
 * of supporting developers from this source code or any supporting source code
 * which is considered copyrighted (c) material of the original comment or credit authors.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
 */

/**
 * @copyright    XOOPS Project https://xoops.org/
 * @license      GNU GPL 2 or later (http://www.gnu.org/licenses/gpl-2.0.html)
 * @package
 * @since
 * @author       XOOPS Development Team
 */

//require_once __DIR__ . '/setup.php';

/**
 *
 * Prepares system prior to attempting to install module
 * @param XoopsModule $module {@link XoopsModule}
 *
 * @return bool true if ready to install, false if not
 */
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;
}

/**
 *
 * Performs tasks required during installation of the module
 * @param XoopsModule $module {@link XoopsModule}
 *
 * @return bool true if installation successful, false if not
 */
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();

$utility is not used, you could remove the assignment.

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

$moduleId2 is not used, you could remove the assignment.

    $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) {

This code seems to be duplicated across your project.

        //    foreach (array_keys($GLOBALS['uploadFolders']) as $i) {
        foreach (array_keys($configurator->uploadFolders) as $i) {
            $utilityClass::createFolder($configurator->uploadFolders[$i]);

The variable $utilityClass does not exist. Did you mean $utility?

        }
    }

    //  ---  COPY blank.png FILES ---------------
    if (count($configurator->copyBlankFiles) > 0) {

This code seems to be duplicated across your project.

        $file = __DIR__ . '/../assets/images/blank.png';
        foreach (array_keys($configurator->copyBlankFiles) as $i) {
            $dest = $configurator->copyBlankFiles[$i] . '/blank.png';
            $utilityClass::copyFile($file, $dest);

The variable $utilityClass does not exist. Did you mean $utility?

        }
    }
    //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%'";

The variable $xoopsDB does not exist. Did you forget to declare it?

    $xoopsDB->queryF($sql);

    return true;
}

//======================================================

$indexFile = 'index.html';
$blankFile = $GLOBALS['xoops']->path('modules/randomquote/assets/images/icons/blank.gif');

//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);
}
chmod($module_uploads, 0777);
copy($indexFile, $GLOBALS['xoops']->path('uploads/randomquote/index.html'));

//Creation du fichier citas dans uploads
$module_uploads = $GLOBALS['xoops']->path('uploads/randomquote/citas');
if (!is_dir($module_uploads)) {
    mkdir($module_uploads, 0777);
}
chmod($module_uploads, 0777);
copy($indexFile, $GLOBALS['xoops']->path('uploads/randomquote/citas/index.html'));