ByTIC\MediaLibrary\Collections\Traits\MediaOperationsTraits

Complexity

Total Complexity

8

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Importance

Changes	4
Bugs	3	Features	1

5 Methods

Rating	Name	Duplication	Size	Complexity
A	deleteDirIfEmpty()	0	5	2
A	deleteMediaByKey()	0	4	1
A	filter()	0	5	1
A	delete()	0	10	3
A	rehydrateFromSibling()	0	5	1

<?php

namespace ByTIC\MediaLibrary\Collections\Traits;

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

/**
 * Trait MediaOperationsTraits.
 */
trait MediaOperationsTraits
{
    /** @inheritDoc
     */
    public function filter(callable $callback = null)
    {
        $filtered = parent::filter($callback);
        $filtered->rehydrateFromSibling($this);
        return $filtered;
    }

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

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

        $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) {
            /* @var Media $file */
            $file->delete();
            $this->unset($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 setMediaType() must be provided by classes using this trait. How about adding it as abstract method to this trait?

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

    }
}