Spatie\MediaLibraryPro\WithAccessingMedia

Complexity

Total Complexity

3

Size/Duplication

Total Lines	21
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	getMedia()	0	19	3

<?php

namespace Spatie\MediaLibraryPro;

use Spatie\MediaLibrary\HasMedia;

The type Spatie\MediaLibrary\HasMedia was not found. Maybe you did not declare it correctly or list all dependencies?

use Spatie\MediaLibrary\MediaCollections\Models\Media;

The type Spatie\MediaLibrary\MediaCollections\Models\Media was not found. Maybe you did not declare it correctly or list all dependencies?

trait WithAccessingMedia
{
    protected function getMedia(string $name, HasMedia $model, string $collection): array
    {
        return old($name) ? old($name) : $model

The expression return old($name) ? old($name) : $model->getMedia($collection)->map(function(...) { /* ... */ })->keyBy('uuid')->toArray() could return the type null|string which is incompatible with the type-hinted return array. Consider adding an additional type-check to rule them out.

            ->getMedia($collection)
            ->map(function (Media $media) {
                return [
                    'name' => $media->name,
                    'fileName' => $media->file_name,
                    'uuid' => $media->uuid,
                    'previewUrl' => $media->hasGeneratedConversion('preview') ? $media->getUrl('preview') : '',
                    'order' => $media->order_column,
                    'customProperties' => $media->custom_properties,
                    'extension' => $media->extension,
                    'size' => $media->size,
                    'createdAt' => $media->created_at->timestamp,
                ];
            })
            ->keyBy('uuid')
            ->toArray();
    }
}