ByTIC\MediaLibrary\Media\Traits\HasConversionsTrait

Complexity

Total Complexity

9

Size/Duplication

Total Lines	47
Duplicated Lines	0 %

Test Coverage

Coverage

95%

Importance

Changes	1
Bugs	0	Features	1

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getConversions()	0	6	2
A	getConversionNames()	0	6	2
A	generateConversions()	0	8	3
A	generateConversionNames()	0	9	2

<?php

namespace ByTIC\MediaLibrary\Media\Traits;

use ByTIC\MediaLibrary\Conversions\ConversionCollection;

/**
 * Trait HasConversionsTrait
 * @package ByTIC\MediaLibrary\Media\Traits
 */
trait HasConversionsTrait
{
    protected $conversions = null;

    protected $conversionNames;

    public function getConversionNames(): array
    {
        if ($this->conversionNames == null) {
            $this->conversionNames = $this->generateConversionNames();
        }
        return $this->conversionNames;
    }

    /**
     * @return array
     */
    protected function generateConversionNames()
    {
        $conversions = $this->getConversions();

        $return = [];
        foreach ($conversions as $conversion) {
            $return[] = $conversion->getName();
        }
        return $return;
    }

    public function getConversions(): ConversionCollection
    {
        if ($this->conversions == null) {
            $this->conversions = $this->generateConversions();
        }
        return $this->conversions;
    }

    /**
     * @return ConversionCollection
     */
    protected function generateConversions()
    {
        $collection = $this->getCollection();

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

        $model = $this->getModel();

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

        if ($collection && $model) {
            return $model->getMediaConversions()->forCollection($collection->getName());
        }
        return new ConversionCollection();
    }

}