-
Notifications
You must be signed in to change notification settings - Fork 214
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
Closes #6760: E_ERROR in 3.16.2: Argument #1 ($microseconds) must be of type int. In line 164 of PreloadUrl.php #6766
Closes #6760: E_ERROR in 3.16.2: Argument #1 ($microseconds) must be of type int. In line 164 of PreloadUrl.php #6766
Conversation
Coverage summary from CodacySee diff coverage on Codacy
Coverage variation details
Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: Diff coverage details
Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: See your quality gate settings Change summary preferences🚀 Don’t miss a bit, follow what’s new on Codacy. Codacy stopped sending the deprecated coverage status on June 5th, 2024. Learn more Footnotes
|
I have proactively tested this filter with the steps below to show that the fatal error is no more thrown:
|
Nice! Thanks @jeawhanlee |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM other than the test missing as mentioned by @MathieuLamiot
Description
Fixes #6760
Documentation
User documentation
Fixes error thrown about unexpected argument type parsed.
Technical documentation
Guards the filter value from unexpected type other than
int
Type of change
New dependencies
None
Risks
None
Checklists
Feature validation
Documentation
Code style
Observability
Risks