Yaro\Jarboe\Table\Fields\Traits\Storage

Complexity

Total Complexity

14

Size/Duplication

Total Lines	75
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	2

Test Coverage

Coverage

15.63%

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	storeFile()	0	8	1
A	multiple()	0	6	1
A	isMultiple()	0	4	1
A	disk()	0	6	1
A	getDisK()	0	4	1
A	path()	0	6	1
A	getPath()	0	4	1
B	getUrl()	0	23	7

<?php

namespace Yaro\Jarboe\Table\Fields\Traits;

use Illuminate\Http\Request;
use Illuminate\Http\UploadedFile;
use Illuminate\Support\Facades\Storage as IlluminateStorage;

trait Storage
{
    protected $disk;
    protected $path = '';
    protected $multiple = false;

    protected function storeFile(UploadedFile $file, $filename, Request $request)
    {
        $path = $file->storeAs($this->getPath(), $filename, [
            'disk' => $this->getDisk(),
        ]);

        return $path;
    }

    public function multiple(bool $multiple = true)
    {
        $this->multiple = $multiple;

        return $this;
    }

    public function isMultiple()

The return type could not be reliably inferred; please add a @return annotation.

    {
        return $this->multiple;
    }

    public function disk(string $disk)
    {
        $this->disk = $disk;

        return $this;
    }

    public function getDisK()

The return type could not be reliably inferred; please add a @return annotation.

    {
        return $this->disk;
    }

    public function path(string $path)
    {
        $this->path = $path;

        return $this;
    }

    public function getPath()

The return type could not be reliably inferred; please add a @return annotation.

    {
        return $this->path;
    }

    public function getUrl($model)

The return type could not be reliably inferred; please add a @return annotation.

    {
        if (is_null($model)) {
            return null;
        }

        $filepath = $model->{$this->name()};

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

        if (!$filepath) {
            return null;
        }

        $disk = IlluminateStorage::disk($this->getDisk());

        $paths = [];
        if (is_array($filepath)) {
            foreach ($filepath as $path) {
                $paths[] = $this->isEncode() ? $path : $disk->url($path);

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

            }
            return $paths;
        }

        return $this->isEncode() ? $filepath : $disk->url($filepath);

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

    }
}