Spatie\MediaLibraryPro\Http\Livewire\Concerns\WithMedia

Complexity

Total Complexity

15

Size/Duplication

Total Lines	68
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	getMediaComponentNames()	0	3	1
A	renderingWithMedia()	0	6	2
A	onMediaChanged()	0	5	1
A	makeSureCustomPropertiesUseRightCasing()	0	14	3
A	hydrateWithMedia()	0	4	2
A	clearMedia()	0	14	4
A	mountWithMedia()	0	4	2

<?php

namespace Spatie\MediaLibraryPro\Http\Livewire\Concerns;

/** @mixin \Livewire\Component */
trait WithMedia
{
    public function getMediaComponentNames(): array
    {
        return $this->mediaComponentNames ?? [];
    }

    public function mountWithMedia(): void
    {
        foreach ($this->getMediaComponentNames() as $mediaComponentName) {
            $this->$mediaComponentName = null;
        }
    }

    public function hydrateWithMedia()
    {
        foreach ($this->getMediaComponentNames() as $mediaComponent) {
            $this->listeners["$mediaComponent:mediaChanged"] = 'onMediaChanged';

The property listeners does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        }
    }

    public function onMediaChanged($name, $media): void
    {
        $media = $this->makeSureCustomPropertiesUseRightCasing($media);

        $this->$name = $media;
    }

    public function renderingWithMedia(): void
    {
        $errorBag = $this->getErrorBag();

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

        foreach ($this->getMediaComponentNames() as $mediaComponentName) {
            $this->emit("$mediaComponentName:mediaComponentValidationErrors", $mediaComponentName, $errorBag->toArray());

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

        }
    }

    public function clearMedia($mediaComponentNames = null)
    {
        if (is_null($mediaComponentNames)) {
            $mediaComponentNames = $this->getMediaComponentNames();
        }

        if (is_string($mediaComponentNames)) {
            $mediaComponentNames = [$mediaComponentNames];
        }

        foreach ($mediaComponentNames as $mediaComponentName) {
            $this->emit("$mediaComponentName:clearMedia", $mediaComponentName);

            $this->$mediaComponentName = [];
        }
    }

    protected function makeSureCustomPropertiesUseRightCasing(array $media): array
    {
        $media = collect($media)
            ->map(function (array $mediaItemAttributes) {
                if (! isset($mediaItemAttributes['custom_properties']) && isset($mediaItemAttributes['customProperties'])) {
                    $mediaItemAttributes['custom_properties'] = $mediaItemAttributes['customProperties'];
                    unset($mediaItemAttributes['customProperties']);
                }

                return $mediaItemAttributes;
            })
            ->toArray();

        return $media;
    }
}