Change recommended way to access array query string parameters

[jontjs]

$request->query->all()['foo']; triggers an 'Undefined array key "foo"' error when foo doesn't exist. However $request->query->all('foo'); return null when foo doesn't exist (consistent with $request->query->get('bar'); for accessing a string parameter bar which might not exist).

[mdoutreluingne]

return null when foo doesn't exist.

Return empty array to be more precise.

I approuve this PR, the change of the signature of this method all(string $key = null) which was introduced in Symfony 6.0, now lets put a key as a parameter to this method, which makes it possible to avoid an error if the key does not exist.

[jontjs]

return null when foo doesn't exist.

Return empty array to be more precise.

Thank you for that pointing that out. Given that it's a different to what ->get('bar') returns if bar does not exist (that returns null), I thought it would be a good idea to update my PR to explicitly state that an empty array is returned.

[javiereguiluz]

Thank you Jon.