Phile\Plugin\Phile\ErrorHandler\Plugin

Complexity

Total Complexity

3

Size/Duplication

Total Lines	29
Duplicated Lines	0 %

Test Coverage

Coverage

69.23%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	onPluginsLoaded()	0	16	3

<?php
/**
 * Plugin class
 */
namespace Phile\Plugin\Phile\ErrorHandler;

use Phile\Core\Router;
use Phile\Core\ServiceLocator;
use Phile\Plugin\AbstractPlugin;
use Phile\Plugin\Phile\ErrorHandler\Development;
use Phile\Plugin\Phile\ErrorHandler\ErrorLog;

/**
 * Class Plugin
 * Default Phile parser plugin for Markdown
 *
 * @author  PhileCMS
 * @link    https://philecms.com
 * @license http://opensource.org/licenses/MIT
 * @package Phile\Plugin\Phile\ParserMarkdown
 */
class Plugin extends AbstractPlugin
{
    const HANDLER_ERROR_LOG = 'error_log';
    const HANDLER_DEVELOPMENT = 'development';

    protected $events = ['plugins_loaded' => 'onPluginsLoaded'];

    /**
     * called on 'plugins_loaded' event
     *
     * @param  null $data

Are you sure the doc-type for parameter $data is correct as it would always require null to be passed?

     * @throws \Phile\Exception\ServiceLocatorException
     */
    public function onPluginsLoaded($data = null)

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

    {
        $this->settings['base_url'] = (new Router)->getBaseUrl();
        switch ($this->settings['handler']) {
            case Plugin::HANDLER_ERROR_LOG:
                ServiceLocator::registerService(
                    'Phile_ErrorHandler',
                    new ErrorLog
                );
                break;
            case Plugin::HANDLER_DEVELOPMENT:
                ServiceLocator::registerService(
                    'Phile_ErrorHandler',
                    new Development($this->settings)
                );
                break;
        }
    }
}