Cmp\Cache\Traits\MultiCacheTrait

Complexity

Total Complexity

8

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	setItems()	0	9	3
A	getItems()	0	9	2
A	deleteItems()	0	9	3

<?php

namespace Cmp\Cache\Traits;

/**
 * Interface MultiCacheTrait
 *
 * Apply to a non multi cache backend to convert it
 *
 * @package Cmp\Cache
 * @mixin \Cmp\Cache\Traits
 */
trait MultiCacheTrait
{
    /**
     * {@inheritdoc}
     */
    public function setItems(array $items, $timeToLive = null)
    {
        $success = true;
        foreach ($items as $key => $item) {
            $success = $success && $this->set($key, $item, $timeToLive);

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

        }

        return $success;
    }

    /**
     * {@inheritdoc}
     */
    public function getItems(array $keys)
    {
        $items = [];
        foreach ($keys as $key) {
            $items[$key] = $this->get($key);

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

        }

        return $items;
    }

    /**
     * {@inheritdoc}
     */
    public function deleteItems(array $keys)
    {
        $success = true;
        foreach ($keys as $key) {
            $success = $success && $this->delete($key);

The method delete() does not exist on Cmp\Cache\Traits\MultiCacheTrait. Did you maybe mean deleteItems()?

        }

        return $success;
    }
}