Skip to content

Conversation

AndrewMast
Copy link
Contributor

Why

This is a very minor change. Recently #54876 removed unneeded function existence checks. Two of the calls were for json_validate, but only one of them passed in a depth value (512, which is the php default). This PR is removes the passing of a default value, normalizes the code to be the same in both cases of json_validate, and is based on this comment: #54876 (comment).

What

-return json_validate($value, 512);
+return json_validate($value);

@AndrewMast AndrewMast changed the title [13.x] Removed unneeded argument (very minor fix) [13.x] Removed unneeded default argument Mar 5, 2025
@taylorotwell taylorotwell merged commit adf340d into laravel:master Mar 5, 2025
35 checks passed
@AndrewMast AndrewMast deleted the am-fix-json-validate branch March 5, 2025 17:54
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.

2 participants