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

[11.x] Fix argument type in Cache\Store #51100

Merged
merged 1 commit into from Apr 17, 2024

Conversation

GromNaN
Copy link
Contributor

@GromNaN GromNaN commented Apr 17, 2024

The cache stores only accept a string for the $key parameter, an array produces an error. This type was introduced by bbd825b without details.

The methods increment/decrement can leverage the type false that is more precise than bool.

@taylorotwell taylorotwell merged commit 33e3c47 into laravel:11.x Apr 17, 2024
28 checks passed
@GromNaN GromNaN deleted the cache-store-type branch April 17, 2024 14:22
@GromNaN GromNaN changed the title Fix argument type in Cache\Store [11.x] Fix argument type in Cache\Store Apr 24, 2024
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.

None yet

2 participants