Mostafaznv\Larupload\Concerns\Standalone\StandaloneLaruploadCallables

Complexity

Total Complexity

8

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	url()	0	7	2
A	handleFFMpegQueue()	0	7	2
A	urls()	0	7	2
A	meta()	0	7	2

<?php

namespace Mostafaznv\Larupload\Concerns\Standalone;

use Mostafaznv\Larupload\Larupload;

trait StandaloneLaruploadCallables
{
    /**
     * @internal
     */
    public function url(string $style = Larupload::ORIGINAL_FOLDER): ?string
    {
        if ($this->internalFunctionIsCallable) {
            return parent::url($style);
        }

        $this->internalException();

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

    }

    /**
     * @internal
     */
    public function meta(?string $key = null): object|int|string|null
    {
        if ($this->internalFunctionIsCallable) {
            return parent::meta($key);
        }

        $this->internalException();
    }

    /**
     * @internal
     */
    public function urls(): object
    {
        if ($this->internalFunctionIsCallable) {
            return parent::urls();
        }

        $this->internalException();

In this branch, the function will implicitly return null which is incompatible with the type-hinted return object. Consider adding a return statement or allowing null as return value.

    }

    /**
     * @internal
     */
    public function handleFFMpegQueue(bool $isLastOne = false, bool $standalone = false): void
    {
        if ($this->internalFunctionIsCallable) {
            parent::handleFFMpegQueue($isLastOne, $standalone);
        }
        else {
            $this->internalException();
        }
    }
}