Larrock\Core\Traits\GetFilesAndImages

Complexity

Total Complexity

18

Size/Duplication

Total Lines	95
Duplicated Lines	0 %

Importance

Changes	4
Bugs	1	Features	1

7 Methods

Rating	Name	Duplication	Size	Complexity
A	getFiles()	0	4	1
A	getFirstImage()	0	4	1
A	registerMediaConversions()	0	16	3
A	getImages()	0	4	1
A	getFirstImage110Attribute()	0	15	4
A	getFirstImageAttribute()	0	15	4
A	getFirstImage140Attribute()	0	15	4

<?php

namespace Larrock\Core\Traits;

use Cache;
use Spatie\MediaLibrary\Models\Media;
use Spatie\MediaLibrary\HasMedia\HasMediaTrait;

trait GetFilesAndImages
{
    use HasMediaTrait;

    /**
     * @param Media|null $media
     * @throws \Spatie\Image\Exceptions\InvalidManipulation
     */
    public function registerMediaConversions(Media $media = null)

The parameter $media is not used and could be removed.

    {
        $this->addMediaConversion('110x110')
            ->height(110)->width(110)
            ->performOnCollections('images');

        $this->addMediaConversion('140x140')
            ->height(140)->width(140)
            ->performOnCollections('images');

        if ($this->config->customMediaConversions) {
            foreach ($this->config->customMediaConversions as $conversion) {
                $explode = explode('x', $conversion);
                $this->addMediaConversion($conversion)
                    ->height($explode[0])->width($explode[1])
                    ->performOnCollections('images');
            }
        }
    }

    public function getFiles()
    {
        return $this->hasMany('Spatie\MediaLibrary\Models\Media', 'model_id', 'id')

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

            ->where([['model_type', '=', $this->config->model], ['collection_name', '=', 'files']])->orderBy('order_column', 'DESC');
    }

    public function getImages()
    {
        return $this->hasMany('Spatie\MediaLibrary\Models\Media', 'model_id', 'id')
            ->where([['model_type', '=', $this->config->model], ['collection_name', '=', 'images']])->orderBy('order_column', 'DESC');
    }

    public function getFirstImage()
    {
        return $this->hasOne('Spatie\MediaLibrary\Models\Media', 'model_id', 'id')

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

            ->where([['model_type', '=', $this->config->model], ['collection_name', '=', 'images']])->orderBy('order_column', 'DESC');
    }

    public function getFirstImageAttribute()
    {
        $value = Cache::rememberForever(sha1('image_f_category'.$this->id.'_'.$this->config->model), function () {
            if ($get_image = $this->getMedia('images')->sortByDesc('order_column')->first()) {
                return $get_image->getUrl();
            }

            if (method_exists($this, 'getCategoryActive') && $this->getCategoryActive) {
                return $this->getCategoryActive->first()->first_image;
            }

            return '/_assets/_front/_images/empty_big.png';
        });

        return $value;
    }

    public function getFirstImage110Attribute()
    {
        $value = Cache::rememberForever(sha1('image_f110_category'.$this->id.'_'.$this->config->model), function () {
            if ($get_image = $this->getMedia('images')->sortByDesc('order_column')->first()) {
                return $get_image->getUrl('110x110');
            }

            if (method_exists($this, 'getCategoryActive') && $this->getCategoryActive) {
                return $this->getCategoryActive->first()->first_image_110;
            }

            return '/_assets/_front/_images/empty_big.png';
        });

        return $value;
    }

    public function getFirstImage140Attribute()
    {
        $value = Cache::rememberForever(sha1('image_f140_category'.$this->id.'_'.$this->config->model), function () {
            if ($get_image = $this->getMedia('images')->sortByDesc('order_column')->first()) {
                return $get_image->getUrl('140x140');
            }

            if (method_exists($this, 'getCategoryActive') && $this->getCategoryActive) {
                return $this->getCategoryActive->first()->first_image_140;
            }

            return '/_assets/_front/_images/empty_big.png';
        });

        return $value;
    }
}