-
-
Notifications
You must be signed in to change notification settings - Fork 436
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
Ensure null value not passed to str_replace #2901
Ensure null value not passed to str_replace #2901
Conversation
Default configuration values may be of null value, and should be typecasted to a string before passing to the str_replace function
e75902e
to
4378dcf
Compare
Duplicate of #2806. Please keep it open for now. Waiting for review for my PR. |
Btw ... i agree with tis PR, but if it is possible a w'd avoid type casting (i know its not possible everywhere). |
Agreed, I think the aforementioned PR is better here. |
The other PR is good but it changes the behavior by making config values with null in the database not overriding the default value in xml, if any. See the review comment in the PR. |
it seems it would make no harm to merge this PR. it's simple and shouldn't have any side effect |
cherry-picked to 20.0 since there were no conflicts |
Typecasing potentially default 'null' configuration values to a string before passing to
str_replace
, avoiding PHP 8.1 deprecation warnings.