WebinoAppLib\Service\Debugger

Complexity

Total Complexity

13

Size/Duplication

Total Lines	90
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	4

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	21	4
A	getBarPanel()	0	4	1
A	setBarPanel()	0	5	1
A	setBarInfo()	0	10	4
A	dump()	0	4	1
A	barDump()	0	4	1
A	timer()	0	4	1

<?php
/**
 * Webino (http://webino.sk)
 *
 * @link        https://github.com/webino for the canonical source repository
 * @copyright   Copyright (c) 2015-2017 Webino, s.r.o. (http://webino.sk)
 * @author      Peter Bačinský <peter@bacinsky.sk>
 * @license     BSD-3-Clause
 */

namespace WebinoAppLib\Service;

use Tracy\Debugger as Tracy;
use Tracy\IBarPanel;
use WebinoAppLib\Options\DebuggerOptions;

/**
 * Class Debugger
 */
class Debugger implements DebuggerInterface
{
    /**
     * @var DebuggerOptions
     */
    private $options;

    /**
     * @param array|DebuggerOptions $options
     */
    public function __construct($options = null)
    {
        $options instanceof DebuggerOptions
            or $options = new DebuggerOptions((array) $options);

Using logical operators such as or instead of || is generally not recommended.

        $this->options = $options;

It seems like $options can also be of type array. However, the property $options is declared as type object<WebinoAppLib\Options\DebuggerOptions>. Maybe add an additional type check?

        if ($options->isDisabled() || Tracy::isEnabled()) {

It seems like $options is not always an object, but can also be of type array|null. Maybe add an additional type check?

            return;
        }

        Tracy::enable(
            $options->getMode(),
            $options->getLog(),
            $options->getEmail()
        );

        Tracy::$showBar    = $options->hasBar();
        Tracy::$strictMode = $options->isStrict();
        Tracy::$maxDepth   = $options->getMaxDepth();
        Tracy::$maxLength  = $options->getMaxLen();
    }

    /**
     * @param string $id
     * @return IBarPanel|null
     */
    public function getBarPanel($id)
    {
        return Tracy::getBar()->getPanel($id);
    }

    /**
     * @param object|IBarPanel $panel
     * @param string|null $id
     * @return $this
     */
    public function setBarPanel(IBarPanel $panel, $id = null)
    {
        Tracy::getBar()->addPanel($panel, $id);
        return $this;
    }

    /**
     * {@inheritdoc}
     */
    public function setBarInfo($info, $value = null)
    {
        $_info = is_string($info) ? $info = [$info => (string) $value] : (array) $info;

        /** @var \Tracy\DefaultBarPanel $panel */
        $panel = $this->getBarPanel('Tracy:info');
        $panel and $panel->data = array_replace(is_array($panel->data) ? $panel->data : [], $_info);

Using logical operators such as and instead of && is generally not recommended.

        return $this;
    }

    /**
     * {@inheritdoc}
     */
    public function dump($subject, $return = false)
    {
        return Tracy::dump($subject, $return);
    }

    /**
     * {@inheritdoc}
     */
    public function barDump($subject, $title = null, array $options = null)
    {
        Tracy::barDump($subject, $title, $options);
    }

    /**
     * {@inheritdoc}
     */
    public function timer($name = null)
    {
        return Tracy::timer($name);
    }
}