Spatie\MediaLibraryPro\WithAccessingMedia::getMedia()

Complexity

Conditions	3
Paths	2

Size

Total Lines	19
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    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();
    }