ByTIC\MediaLibrary\HasMedia\StandardCollections\FilesShortcodes

Complexity

Total Complexity

4

Size/Duplication

Total Lines	54
Duplicated Lines	0 %

Test Coverage

Coverage

83.33%

Importance

Changes	4
Bugs	1	Features	2

4 Methods

Rating	Name	Duplication	Size	Complexity
A	addFile()	0	3	1
A	addFileFromContent()	0	11	1
A	findFiles()	0	3	1
A	getFiles()	0	3	1

<?php

namespace ByTIC\MediaLibrary\HasMedia\StandardCollections;

use ByTIC\MediaLibrary\Collections\Collection;
use ByTIC\MediaLibrary\FileAdder\FileAdder;
use ByTIC\MediaLibrary\Media\Media;

/**
 * Trait StandardCollectionsShortcodes.
 */
trait FilesShortcodes
{
    /**
     * @return Collection|Media[]
     *
     * @deprecated Use getFiles
     */
    public function findFiles()
    {
        return $this->getFiles();
    }

    /**
     * @return Collection|Media[]
     */
    public function getFiles()
    {
        return $this->getMedia('files');
    }

    /**
     * @param $file
     *
     * @return FileAdder
     */
    public function addFile($file)
    {
        return $this->addMediaToCollection($file, 'files');

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

    }

    /**
     * @param $content
     * @param $name
     */
    public function addFileFromContent($content, $name)
    {
        $tmpFile = tempnam(sys_get_temp_dir(), 'bytic-media-library');
        file_put_contents($tmpFile, $content);

        $fileAdder = $this->addMedia($tmpFile);

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

        $fileAdder->setFileName($name);
        $fileAdder->toMediaCollection('files');

        unlink($tmpFile);
        return $fileAdder;
    }

    /**
     * @param string $collectionName
     * @param array  $filters
     *
     * @return Collection
     */
    abstract public function getMedia(string $collectionName = 'default', $filters = []): Collection;
}