Modules\Page\Repositories\Cache\CachePageDecorator

Complexity

Total Complexity

4

Size/Duplication

Total Lines	61
Duplicated Lines	32.79 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes	2
Bugs	0	Features	1

4 Methods

Rating	Name	Duplication	Size	Complexity
A	countAll()	10	10	1
A	findBySlugInLocale()	0	10	1
A	__construct()	0	6	1
A	findHomepage()	10	10	1

<?php namespace Modules\Page\Repositories\Cache;

use Modules\Core\Repositories\Cache\BaseCacheDecorator;
use Modules\Page\Repositories\PageRepository;

class CachePageDecorator extends BaseCacheDecorator implements PageRepository
{
    /**
     * @var PageRepository
     */
    protected $repository;

    public function __construct(PageRepository $page)
    {
        parent::__construct();
        $this->entityName = 'pages';
        $this->repository = $page;
    }

    /**
     * Find the page set as homepage
     *
     * @return object
     */
    public function findHomepage()

This method seems to be duplicated in your project.

    {
        return $this->cache
            ->tags($this->entityName, 'global')

The call to Repository::tags() has too many arguments starting with 'global'.

            ->remember("{$this->locale}.{$this->entityName}.findHomepage", $this->cacheTime,
                function () {
                    return $this->repository->findHomepage();
                }
            );
    }

    /**
     * Count all records
     * @return int
     */
    public function countAll()

This method seems to be duplicated in your project.

    {
        return $this->cache
            ->tags($this->entityName, 'global')

The call to Repository::tags() has too many arguments starting with 'global'.

            ->remember("{$this->locale}.{$this->entityName}.countAll", $this->cacheTime,
                function () {
                    return $this->repository->countAll();
                }
            );
    }

    /**
     * @param $slug
     * @param $locale
     * @return object
     */
    public function findBySlugInLocale($slug, $locale)
    {
        return $this->cache
            ->tags($this->entityName, 'global')

The call to Repository::tags() has too many arguments starting with 'global'.

            ->remember("{$this->locale}.{$this->entityName}.findBySlugInLocale.{$slug}.{$locale}", $this->cacheTime,
                function () use ($slug, $locale) {
                    return $this->repository->findBySlugInLocale($slug, $locale);
                }
            );
    }
}