Mostafaznv\Larupload\Concerns\Storage\UploadEntity\UploadEntityResponse

Complexity

Total Complexity

11

Size/Duplication

Total Lines	58
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	storageUrl()	0	19	5
A	withMeta()	0	5	1
A	storageDownload()	0	17	5

<?php

namespace Mostafaznv\Larupload\Concerns\Storage\UploadEntity;


use Illuminate\Http\RedirectResponse;
use Illuminate\Support\Facades\Storage;
use Mostafaznv\Larupload\UploadEntities;
use Symfony\Component\HttpFoundation\StreamedResponse;

trait UploadEntityResponse
{
    /**
     * Specify whether Larupload should return meta values on getAttribute or not.
     */
    protected bool $withMeta;

    /**
     * Specify whether Larupload should return responses in camel-case or not.
     */
    protected bool $camelCaseResponse;


    public function withMeta(bool $status): UploadEntities
    {
        $this->withMeta = $status;

        return $this;

The expression return $this returns the type Mostafaznv\Larupload\Concerns\Storage\UploadEntity\UploadEntityResponse which includes types incompatible with the type-hinted return Mostafaznv\Larupload\UploadEntities.

    }

    protected function storageUrl(string $path): ?string
    {
        if (isset($this->file) and $this->file == LARUPLOAD_NULL) {
            return null;
        }

        if ($this->driverIsLocal()) {

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

            $url = Storage::disk($this->disk)->url($path);

            return url($url);
        }

        $baseUrl = config("filesystems.disks.$this->disk.url");

        if ($baseUrl) {
            return "$baseUrl/$path";
        }

        return $path;
    }

    protected function storageDownload(string $path): StreamedResponse|RedirectResponse|null
    {
        if (isset($this->file) and $this->file == LARUPLOAD_NULL) {
            return null;
        }

        if ($this->driverIsLocal()) {
            return Storage::disk($this->disk)->download($path);
        }

        $baseUrl = config("filesystems.disks.$this->disk.url");

        if ($baseUrl) {
            return redirect("$baseUrl/$path");
        }

        return null;
    }
}