Kelemen\ApiNette\Logger\Logger

Complexity

Total Complexity

5

Size/Duplication

Total Lines	66
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	setResolvedRoute()	0	4	1
A	setHandler()	0	4	1
A	start()	0	4	1
A	finish()	0	10	1

<?php

namespace Kelemen\ApiNette\Logger;

use Kelemen\ApiNette\Handler\BaseHandler;
use Kelemen\ApiNette\Logger\Storage\LoggerStorageInterface;
use Kelemen\ApiNette\Response\ApiResponse;
use Kelemen\ApiNette\Route\ResolvedRoute;
use Nette\Http\Request;

class Logger
{
    /** @var LoggerStorageInterface */
    private $storage;

    /** @var Request */
    private $httpRequest;

    /** @var ResolvedRoute */
    private $resolvedRoute;

    /** @var BaseHandler */
    private $handler;

    /** @var int */
    private $start = 0;

    /**
     * @param Request $httpRequest
     * @param LoggerStorageInterface $storage
     */
    public function __construct(Request $httpRequest, LoggerStorageInterface $storage)
    {
        $this->httpRequest = $httpRequest;
        $this->storage = $storage;
    }

    /**
     * @param ResolvedRoute $resolvedRoute
     */
    public function setResolvedRoute(ResolvedRoute $resolvedRoute)
    {
        $this->resolvedRoute = $resolvedRoute;
    }

    /**
     * @param BaseHandler $handler
     */
    public function setHandler(BaseHandler $handler)
    {
        $this->handler = $handler;
    }

    /**
     * Start logging
     */
    public function start()
    {
        $this->start = microtime(true);

The property $start was declared of type integer, but microtime(true) is of type double. Maybe add a type cast?

    }

    /**
     * Finish logging
     * @param ApiResponse $response
     */
    public function finish(ApiResponse $response)
    {
        $this->storage->store(
            $this->httpRequest,
            $response,
            round((microtime(true) - $this->start) * 1000),
            $this->resolvedRoute,
            $this->handler
        );
    }
}