Yiisoft\Yii\Debug\Proxy\ProxyLogTrait

Complexity

Total Complexity

9

Size/Duplication

Total Lines	81
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	logToEvent()	0	20	1
A	processLogData()	0	12	4
A	logToCollector()	0	19	1
A	logProxy()	0	17	3

<?php

declare(strict_types=1);

namespace Yiisoft\Yii\Debug\Proxy;

use Yiisoft\Yii\Debug\Event\ProxyMethodCallEvent;
use function get_class;

trait ProxyLogTrait
{
    private ContainerProxyConfig $config;

    protected function logProxy(
        string $service,
        object $instance,
        string $method,
        array $arguments,
        $result,
        float $timeStart
    ): void {
        $error = $this->getCurrentError();

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

        $this->processLogData($arguments, $result, $error);

        if ($this->config->getCollector() !== null) {
            $this->logToCollector($service, $instance, $method, $arguments, $result, $error, $timeStart);
        }

        if ($this->config->getDispatcher() !== null) {
            $this->logToEvent($service, $instance, $method, $arguments, $result, $error, $timeStart);
        }
    }

    private function processLogData(array &$arguments, &$result, ?object &$error): void
    {
        if (!($this->config->getLogLevel() & ContainerInterfaceProxy::LOG_ARGUMENTS)) {
            $arguments = null;
        }

        if (!($this->config->getLogLevel() & ContainerInterfaceProxy::LOG_RESULT)) {
            $result = null;
        }

        if (!($this->config->getLogLevel() & ContainerInterfaceProxy::LOG_ERROR)) {
            $error = null;
        }
    }

    private function logToCollector(
        string $service,
        object $instance,
        string $method,
        ?array $arguments,
        $result,
        ?object $error,
        float $timeStart
    ): void {
        $this->config->getCollector()->collect(
            $service,
            get_class($instance),
            $method,
            $arguments,
            $result,
            $this->getCurrentResultStatus(),

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

            $error,
            $timeStart,
            microtime(true),
        );
    }

    private function logToEvent(
        string $service,
        object $instance,
        string $method,
        ?array $arguments,
        $result,
        ?object $error,
        float $timeStart
    ): void {
        $this->config->getDispatcher()->dispatch(
            new ProxyMethodCallEvent(
                $service,
                get_class($instance),
                $method,
                $arguments,
                $result,
                $this->getCurrentResultStatus(),
                $error,
                $timeStart,
                microtime(true),
            )
        );
    }
}