ByJG\RestServer\ErrorHandler

Complexity

Total Complexity

6

Size/Duplication

Total Lines	65
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	setHandler()	0	6	1
A	register()	0	4	1
A	unregister()	0	4	1
A	addExtraInfo()	0	7	2

<?php

namespace ByJG\RestServer;

use ByJG\DesignPattern\Singleton;
use ByJG\RestServer\Whoops\PlainResponseHandler;
use Whoops\Handler\Handler;
use Whoops\Run;

class ErrorHandler
{

    use Singleton;

    /**
     *
     * @var Run
     */
    protected $whoops = null;

    /**
     *
     * @var Handler
     */
    protected $handler = null;

    protected function __construct()
    {
        $this->whoops = new Run();
        $this->setHandler(new PlainResponseHandler());
    }

    /**
     * Set the proper Error Handler based on the Output of the page
     *
     * @param Handler $handler
     */
    public function setHandler(Handler $handler)
    {
        $this->whoops->popHandler();
        $this->handler = $handler;
        $this->whoops->pushHandler($this->handler);

$this->handler is of type object<Whoops\Handler\Handler>, but the function expects a callable.

    }

    /**
     * Set Whoops as the default error and exception handler used by PHP:
     */
    public function register()
    {
        $this->whoops->register();
    }

    /**
     * Disable Whoops as the default error and exception handler used by PHP:
     */
    public function unregister()
    {
        $this->whoops->unregister();
    }

    /**
     * Added extra information for debug purposes on the error handler screen
     *
     * @param string $name
     * @param string $value
     */
    public function addExtraInfo($name, $value)
    {
        if (method_exists($this->handler, 'addDataTable')) {
            $data = $this->handler->getDataTable();

It seems like you code against a specific sub-type and not the parent class Whoops\Handler\Handler as the method getDataTable() does only exist in the following sub-classes of Whoops\Handler\Handler: ByJG\RestServer\Whoops\JsonResponseHandler, ByJG\RestServer\Whoops\PlainResponseHandler. Maybe you want to instanceof check for one of these explicitly?

            $this->handler->addDataTable('Info #' . (count($data) + 1), array($name => $value));

It seems like you code against a specific sub-type and not the parent class Whoops\Handler\Handler as the method addDataTable() does only exist in the following sub-classes of Whoops\Handler\Handler: ByJG\RestServer\Whoops\JsonResponseHandler, ByJG\RestServer\Whoops\PlainResponseHandler, Whoops\Handler\PrettyPageHandler. Maybe you want to instanceof check for one of these explicitly?

        }
    }
}