CoiSA\ErrorHandler\Handler\DebugHandler

Complexity

Total Complexity

3

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__invoke()	0	5	2
A	__construct()	0	4	1

<?php
/**
 * @author Felipe Sayão Lobato Abreu <contato@felipeabreu.com.br>
 * @package CoiSA\ErrorHandler\Handler
 * @since 2017-07-24
 */

namespace CoiSA\ErrorHandler\Handler;

use CoiSA\ErrorHandler\Helper\DebugHelper;

/**
 * Class DebugHandler
 * @package CoiSA\ErrorHandler\Handler
 */
abstract class DebugHandler implements HandlerInterface
{
    /**
     * @var bool Should add HTML <pre> TAG?
     */
    protected $pretty;

    /** @var  bool Send exit code? */
    protected $exit;

    /**
     * VarExportHandler constructor.
     *
     * @param bool $pretty
     */
    public function __construct(bool $pretty = true, $exit = false)
    {
        $this->pretty = $pretty;
        $this->exit = $exit;
    }

    /**
     * @param \Throwable $throwable
     * @return int|null
     */
    public function __invoke(\Throwable $throwable): ?int
    {
        DebugHelper::export($throwable, $this->pretty);

        return $this->exit ?: null;

The expression return $this->exit ?: null could return the type true which is incompatible with the type-hinted return null|integer. Consider adding an additional type-check to rule them out.

    }
}