Feeling Lucky
?
[#39856] - [4] PHP8. Argument #1 ($string) must be of type string. JLayout form.field.text.
- Closed
- 13 Feb 2023
- Medium
- Build: 4.2-dev
- # 39856
- Diff
- ReLater:patch-1
- Pending continuous-integration/drone/pr Build is pending Details
User tests: Successful: Unsuccessful:
Pull Request for Issue #39854.
Summary of Changes
Add is_string() check inside htmlspecialchars.
Testing Instructions
See #39854 (comment)
Actual result BEFORE applying this Pull Request
PHP8 error htmlspecialchars(): Argument #1 ($string) must be of type string, array given Line 99.
Expected result AFTER applying this Pull Request
No error.
Link to documentations
Please select:
-
Documentation link for docs.joomla.org:
-
No documentation changes for docs.joomla.org needed
-
Pull Request link for manual.joomla.org:
-
No documentation changes for manual.joomla.org needed
joomla-cms-bot
- | Category | ⇒ | Layout |
| Status | New | ⇒ | Pending |
| Title |
|
||||||
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2023-02-13 13:14:07 |
| Closed_By | ⇒ | ReLater | |
| Labels |
Added:
?
|
||
I do not like it,
it should explode if it has wrong value, otherwise it will be hard to debug when there soething wrong.