Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[10.x] Add keys length check to prevent mget error #46998

Merged
merged 2 commits into from
May 9, 2023

Conversation

tloader11
Copy link
Contributor

@tloader11 tloader11 commented May 9, 2023

Without this extra check Cache::many([]) would return the following error:

ErrorException : array_map(): Expected parameter 2 to be an array, bool given

If there are no keys given as input, we can safely return an empty array as response.

@taylorotwell taylorotwell merged commit a9bd3bd into laravel:10.x May 9, 2023
milwad-dev pushed a commit to milwad-dev/framework that referenced this pull request May 12, 2023
* [10.x] Add keys length check to prevent mget error

* [10.x] changed formatting to comply with PR standard

---------

Co-authored-by: Tristan ter Haar <t.terhaar@tflabs.nl>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants