Flipbox\Stash\Pool

Complexity

Total Complexity

4

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

63.64%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	setItemDuration()	0	10	2
A	getItemDuration()	0	4	1
A	getItem()	0	10	1

<?php

/**
 * @copyright  Copyright (c) Flipbox Digital Limited
 * @license    https://github.com/flipbox/stash-cache/blob/master/LICENSE
 * @link       https://github.com/flipbox/stash-cache
 */

namespace Flipbox\Stash;

/**
 * @author Flipbox Factory <hello@flipboxfactory.com>
 * @since 1.0.0
 */
class Pool extends \Stash\Pool implements PoolDurationInterface
{

    /**
     * The cache duration
     *
     * @var null|int|\DateInterval
     */
    protected $itemDuration;

    /**
     * @inheritdoc
     */
    public function setItemDuration($duration)
    {
        if (!$duration instanceof \DateInterval) {
            $duration = (int)$duration;
        }

        $this->itemDuration = $duration;

        return $this;
    }

    /**
     * @inheritdoc
     */
    public function getItemDuration()
    {
        return $this->itemDuration;
    }

    /**
     * {@inheritdoc}
     */
    public function getItem($key)
    {
        // Get cache item
        $item = parent::getItem($key);

        // Set the expiration
        $item->expiresAfter($this->getItemDuration());

It seems like $this->getItemDuration() targeting Flipbox\Stash\Pool::getItemDuration() can also be of type null; however, Stash\Interfaces\ItemInterface::expiresAfter() does only seem to accept integer|object<DateInterval>, maybe add an additional type check?

        return $item;
    }
}