Overblog\GraphQLBundle\Relay\Connection\TotalCountCache

Complexity

Total Complexity

5

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	reset()	0	3	1
A	__invoke()	0	9	3

<?php

declare(strict_types=1);

namespace Overblog\GraphQLBundle\Relay\Connection;

use Symfony\Contracts\Service\ResetInterface;

class TotalCountCache implements ResetInterface
{
    /**
     * @var int|callable
     */
    private $total;
    private int $totalCount;

    /**
     * @param int|callable $total
     */
    public function __construct($total)
    {
        $this->total = $total;
    }

    public function __invoke(array $callableArgs = []): int
    {
        if (isset($this->totalCount)) {
            return $this->totalCount;
        }

        $this->totalCount = is_callable($this->total) ? call_user_func_array($this->total, $callableArgs) : $this->total;

It seems like $this->total can also be of type integer; however, parameter $callback of call_user_func_array() does only seem to accept callable, maybe add an additional type check?

        return $this->totalCount;
    }

    public function reset(): void
    {
        unset($this->totalCount);
    }
}