ByTIC\MediaLibrary\FileAdder\Traits\FileAdderProcessesTrait

Complexity

Total Complexity

9

Size/Duplication

Total Lines	63
Duplicated Lines	0 %

Test Coverage

Coverage

96.97%

Importance

Changes	2
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	processMediaItem()	0	11	2
A	guardAgainstDisallowedFileAdditions()	0	8	3
A	copyMediaToFilesystem()	0	10	1
A	toMediaCollection()	0	6	2
A	createMediaConversions()	0	6	1

<?php

namespace ByTIC\MediaLibrary\FileAdder\Traits;

use ByTIC\MediaLibrary\Collections\Collection;
use ByTIC\MediaLibrary\Exceptions\FileCannotBeAdded;
use ByTIC\MediaLibrary\Exceptions\FileCannotBeAdded\FileUnacceptableForCollection;
use ByTIC\MediaLibrary\Media\Manipulators\ManipulatorFactory;
use ByTIC\MediaLibrary\Media\Media;
use ByTIC\MediaLibrary\PathGenerator\PathGeneratorFactory;
use ByTIC\MediaLibrary\Validation\Violations\ViolationsBag;
use Nip\Filesystem\File;

/**
 * Trait FileAdderProcessesTrait.
 */
trait FileAdderProcessesTrait
{
    /**
     * @param string|Collection $collection
     */
    public function toMediaCollection($collection)
    {
        if (is_string($collection)) {
            $collection = $this->getMediaRepository()->getCollection($collection);

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

        }
        $this->processMediaItem($collection);
    }

    /**
     * @param Collection $collection
     * @param Media      $media
     */
    protected function processMediaItem(Collection $collection, Media $media = null)
    {
        $this->guardAgainstDisallowedFileAdditions($collection, $this->getFile());

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

        $media = $media ? $media : $this->getMedia();

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

        $media->setCollection($collection);

        $this->copyMediaToFilesystem();
        $this->createMediaConversions();

        $collection->appendMedia($media);
    }

    protected function copyMediaToFilesystem()
    {
        $media = $this->getMedia();
        $destination = PathGeneratorFactory::create()::getBasePathForMediaOriginal($media);
        $destination .= DIRECTORY_SEPARATOR . $media->getCollection()->getStrategy()::makeFileName($this);

        $media->generateFileFromContent($destination, fopen($this->getPathToFile(), 'r'));

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

        $file = new File($media->getCollection()->getFilesystem(), $destination);
        $media->setFile($file);
    }

    protected function createMediaConversions()
    {
        $media = $this->getMedia();
        ManipulatorFactory::createForMedia($media)->performConversions(
            $this->getSubject()->getMediaConversions()->forCollection($media->getCollection()->getName()),

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

            $media
        );
    }

    /**
     * @param Collection $collection
     * @param mixed      $file
     */
    protected function guardAgainstDisallowedFileAdditions(Collection $collection, $file)
    {
        $guardTest = ($collection->acceptsMedia)($file);
        if ($guardTest instanceof ViolationsBag) {
            throw FileUnacceptableForCollection::createFromViolationsBag($file, $collection, $guardTest);
        }
        if ($guardTest === false) {
            throw new FileCannotBeAdded();
        }
    }
}