Sco\Admin\Display\Image

Complexity

Total Complexity

10

Size/Duplication

Total Lines	56
Duplicated Lines	21.43 %

Coupling/Cohesion

Components	1
Dependencies	6

Test Coverage

Coverage

0%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getImagePathAttribute()	0	4	1
A	setImagePathAttribute()	0	6	1
A	get()	9	9	2
A	parseRows()	0	16	3
A	getImageUrl()	3	8	3

<?php

namespace Sco\Admin\Display;

use Illuminate\Database\Eloquent\Model;
use Illuminate\Support\Collection;
use Sco\Admin\Display\Concerns\WithPagination;
use Sco\Admin\Traits\UploadStorageTrait;

class Image extends Display
{
    use UploadStorageTrait, WithPagination;

    protected $type = 'image';

    protected $imagePathAttribute;

    public function getImagePathAttribute()
    {
        return $this->imagePathAttribute;
    }

    public function setImagePathAttribute($value)
    {
        $this->imagePathAttribute = $value;

        return $this;
    }

    public function get()

This method seems to be duplicated in your project.

    {
        if ($this->isPagination()) {
            $data = $this->paginate();

            return $data->setCollection($this->parseRows($data->getCollection()));
        }
        return $this->parseRows($this->getQuery()->get());

It seems like $this->getQuery()->get() targeting Illuminate\Database\Eloquent\Builder::get() can also be of type array<integer,object<Illuminate\Database\Eloquent\Builder>>; however, Sco\Admin\Display\Image::parseRows() does only seem to accept object<Illuminate\Support\Collection>, maybe add an additional type check?

    }

    protected function parseRows(Collection $rows)
    {
        if (is_null($pathKey = $this->getImagePathAttribute())) {
            throw new \InvalidArgumentException('Must set image attribute');
        }

        return $rows->map(function (Model $row) use ($pathKey) {
            if (! isset($row->$pathKey)) {
                throw new \InvalidArgumentException("Not Found '{$pathKey}' attribute");
            }
            $row->setAttribute('_primary', $row->getKey());
            $row->setAttribute('_url', $this->getImageUrl($row->$pathKey));

            return $row;
        });
    }

    protected function getImageUrl($value)
    {
        if (! empty($value) && (strpos($value, '://') === false)) {

This code seems to be duplicated across your project.

            $value = $this->getFileUrl($value);
        }

        return $value;
    }
}