getParam function improvements

[intelrug]

laravel-paybox/src/Paybox.php

Line 131 in 5ff87ed

return Arr::get($params, $key) ?: Arr::get($params, $fallbackKey);

laravel-paybox/src/Paybox.php

Line 134 in 5ff87ed

return Arr::get($this->config, $key) ?: Arr::get($this->config, $fallbackKey);

I think it would be better to use Null coalescing (??) operator here, because the current code does not work correctly with boolean or zero values.
And we need an extra if because Arr::get returns an array if the provided key is null.

So the final code will look something like this:

protected function getParam(array $params, string $key, ?string $fallbackKey = null): mixed
{
    if (Arr::hasAny($params, [$key, $fallbackKey])) {
        return Arr::get($params, $key) ?? Arr::get($params, $fallbackKey);
    }

    if (Arr::hasAny($this->config, [$key, $fallbackKey])) {
        return Arr::get($this->config, $key) ?? Arr::get($this->config, $fallbackKey);
    }

    return null;
}

What do you think?