CristalTeam/php-api-wrapper

array_merge() expects at least 1 parameter, 0 given ($this->scopes is null)

MiloDBD opened this issue · 0 comments

I'm doing a very basic "Transport" (not the basic one) Transport implementation, and I'm seeing $this->scopes is null on a basic GET request...

(src/Builder.php)

public function getQuery()
    {
        return array_merge(
            array_merge(...array_values($this->scopes)),
            $this->query
        );
    }

This got changes here.
https://github.com/CristalTeam/php-api-wrapper/blob/4de7a802f3df14155947b94b3fb0c415739b0e30/src/Builder.php#L33C5-L39C6

On this commit
d0afb95

If I revert the change to this.

    public function getQuery()
    {
        /*return array_merge(
            array_merge(...array_values($this->scopes)),
            $this->query
        );*/
        return $this->query;
    }

It seems to work.

I could be not implementing this correctly.

Any advice?

Thank you, good work here... appreciated.