bytic / MediaLibrary

src/Collections/Traits/MediaOperationsTraits.php

<?php

namespace ByTIC\MediaLibrary\Collections\Traits;

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

/**
 * Trait MediaOperationsTraits.
 *
 * @method Media get($key)
 */
trait MediaOperationsTraits
{
    /** @inheritDoc
     */
    public function filter(callable $callback = null): \Nip\Collections\CollectionInterface
    {
        $filtered = parent::filter($callback);
        $filtered->rehydrateFromSibling($this);
        return $filtered;
    }

    /**
     * @param string $key
     */
    public function deleteMediaByKey($key)
    {
        $this->get($key)->delete();
        $this->unset($key);

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

    }

    public function delete()
    {
        foreach ($this as $key => $file) {
            $this->deleteMediaByKey($key);
        }

        if (isset($file)) {
            $directory = dirname($file->getPath());
            $this->deleteDirIfEmpty($directory);
        }
    }

    /**
     * @param $directory
     */
    protected function deleteDirIfEmpty($directory)
    {
        $contents = $this->getFilesystem()->listContents($directory);

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

        if (empty($contents)) {
            $this->getFilesystem()->deleteDir($directory);
        }
    }

    /**
     * @param static $sibling
     */
    protected function rehydrateFromSibling($sibling)
    {
        $this->setName($sibling->getName());

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

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

        $this->setMediaRepository($sibling->getMediaRepository());

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

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

        $this->setMediaType($sibling->getMediaType());

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

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

        $this->setLoader($sibling->getLoader());

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

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

    }
}