Arp\LaminasDoctrine\Factory\Cache\Adapter\ArrayAdapterFactory

Complexity

Total Complexity

2

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	__invoke()	0	16	2

<?php

declare(strict_types=1);

namespace Arp\LaminasDoctrine\Factory\Cache\Adapter;

use Arp\LaminasFactory\AbstractFactory;
use Laminas\ServiceManager\Exception\ServiceNotCreatedException;
use Laminas\ServiceManager\Exception\ServiceNotFoundException;
use Psr\Container\ContainerExceptionInterface;
use Psr\Container\ContainerInterface;
use Symfony\Component\Cache\Adapter\ArrayAdapter;
use Symfony\Component\Cache\Exception\InvalidArgumentException;

final class ArrayAdapterFactory extends AbstractFactory
{
    /**
     * @param array<string, mixed>|null $options
     *
     * @throws ServiceNotCreatedException
     * @throws ServiceNotFoundException
     * @throws ContainerExceptionInterface
     */
    public function __invoke(ContainerInterface $container, string $requestedName, array $options = null): ArrayAdapter
    {
        $options = $options ?? $this->getServiceOptions($container, $requestedName, 'cache');

        try {
            return new ArrayAdapter(
                $options['default_lifetime'] ?? 0,
                $options['store_serialized'] ?? 0,

It seems like $options['store_serialized'] ?? 0 can also be of type integer; however, parameter $storeSerialized of Symfony\Component\Cache\Adapter\ArrayAdapter::__construct() does only seem to accept boolean, maybe add an additional type check?

                $options['max_lifetime'] ?? 0,
                $options['max_items'] ?? 0,
            );
        } catch (InvalidArgumentException $e) {
            throw new ServiceNotCreatedException(
                sprintf('Failed to create cache adapter \'%s\' due to invalid configuration options', $requestedName),
                $e->getCode(),
                $e,
            );
        }
    }
}