PhileCMS / Phile

config/bootstrap.php

<?php
/**
 * Bootstraps the Phile-core
 *
 * There's no need to make changes here in an ordinary Phile installation.
 * It allows advanced configuration options if necessary.
 *
 * @author PhileCMS
 * @link https://github.com/PhileCMS/Phile
 * @license http://opensource.org/licenses/MIT
 */


/**
 * Set global definitions
 */
require_once 'constants.php';

/**
 * Setup container
 */
require 'container.php';
$container = Phile\Core\Container::getInstance();

/**
 * Register plugin directories
 *
 * Allows autoloading from plugin-directories and early usage in config.php
 *
 * @var \Phile\Plugin\PluginRepository $plugins
 */
$plugins = $container->get('Phile_Plugins');
$plugins->addDirectory(PLUGINS_DIR);

/**
 * Setup global application-object
 *
 * @var \Phile\Phile $app
 */
$app = $container->get('Phile_App');

/**
 * Define the bootstrap process
 */
use Phile\Core\Bootstrap;
use Phile\Core\Config;
use Phile\Core\Event;

This use statement conflicts with another class in this namespace, Event. Consider defining an alias.

use Phile\Core\Registry;

$app->addBootstrap(function (Event $eventBus, Config $config) use ($plugins): void {
    // Load configuration files into global $config configuration
    $configDir = $config->get('config_dir');
    Bootstrap::loadConfiguration($configDir . 'defaults.php', $config);
    Bootstrap::loadConfiguration($configDir . 'config.php', $config);

    // backwards-compatibility for deprecated global constants set in Config now
    // phpcs:disable PSR1.Files.SideEffects
    defined('CONTENT_DIR') || define('CONTENT_DIR', $config->get('content_dir'));
    defined('CONTENT_EXT') || define('CONTENT_EXT', $config->get('content_ext'));
    // phpcs:enable

    // Setup core folders
    Bootstrap::setupFolder($config->get('cache_dir'), $config);

It seems like $config->get('cache_dir') can also be of type null; however, parameter $directory of Phile\Core\Bootstrap::setupFolder() does only seem to accept string, maybe add an additional type check?

    Bootstrap::setupFolder($config->get('storage_dir'), $config);

    // backwards-compatibility for deprecated static Event access
    Event::setInstance($eventBus);

The function Phile\Core\Event::setInstance() has been deprecated: static use is deprecated

    // Load plug-ins
    $plugins->load($config);

    // Set error handler
    Bootstrap::setupErrorHandler($config);

    // Intialize global registry objects
    Registry::set('templateVars', []);

    // Set additional PHP environment variables from configuration
    date_default_timezone_set($config->get('timezone'));

It seems like $config->get('timezone') can also be of type null; however, parameter $timezoneId of date_default_timezone_set() does only seem to accept string, maybe add an additional type check?

});

/**
 * Add PSR-15 middleware
 */
use Phile\Http\MiddlewareQueue;

$app->addMiddleware(function (MiddlewareQueue $middleware, Event $eventBus, Config $config) use ($app): void {

The parameter $config is not used and could be removed.

    // Inject middleware from Phile-plugins
    $eventBus->trigger('phile.core.middleware.add', ['middleware' => $middleware]);

    // Add Phile itself as middleware (take request and render output)
    $middleware->add($app);
});