Sulao\YiiHoneybadger\Component::getHoneybadger()

Complexity

Conditions	6
Paths	3

Size

Total Lines	31
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	18
CRAP Score	6

Importance

Changes	1
Bugs	0	Features	0

<?php
    protected function getHoneybadger(): Honeybadger
    {
        static $honeybadger;

        if ($honeybadger) {
            return $honeybadger;
        }

        $config = $this->config;

        // Globally disable error handlers, record the errors via the logger
        $config['handlers'] = [
            'exception' => false,
            'error'     => false,
            'shutdown'  => false,
        ];

        $context = ArrayHelper::remove($config, 'context');

        $honeybadger = Honeybadger::new($config);

        if (!empty($context) && is_array($context)) {
            foreach ($context as $key => $value) {
                if ($value instanceof Closure) {
                    $context[$key] = call_user_func($value);
                }
            }
            $honeybadger->context($context);

It seems like context() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }

        return $honeybadger;

The expression return $honeybadger returns the type Honeybadger\Concerns\Newable which includes types incompatible with the type-hinted return Honeybadger\Honeybadger.

    }