mostafaznv / larupload

src/Concerns/Standalone/StandaloneLaruploadCallables.php

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