ByTIC\MediaLibrary\Collections\Traits\MediaOperationsTraits

Complexity

Total Complexity

8

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Test Coverage

Coverage

52%

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	9	3
A	rehydrateFromSibling()	0	6	1

<?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?

    }
}