Minor cleanup: fixed wrong usage of sprintf, it only takes 2 placeholders here and w… #38628
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
…e gave it 3 arguments.
Description (*)
Found with running phpstan on level 0:
This PR fixes this, it doesn't fix anything major, as the code runs fine as-is and doesn't crash. So this is only a minor cleanup.
I've left the trailing comma, since that's accepted by PHP > 8.0 and it results in cleaner git diffs.
Related Pull Requests
Fixed Issues (if relevant)
Manual testing scenarios (*)
Nothing to test, the code wasn't behaving wrong before, this is just a minor cleanup.
The unit test for this class already tests for the correct exception message, so if unit tests are green, this can be considered good enough.
Questions or comments
Contribution checklist (*)