[#36794] - [3.10] [PHP 8.1] Make templates/system/html/modules.php PHP 8.1 compatible
- Fixed in Code Base
- 23 Jan 2022
- Medium
- Build: 3.10-dev
- # 36794
- Diff
- beat:patch-18
User tests: Successful: Unsuccessful:
Pull Request for Issue # none
Summary of Changes
Fixes Deprecated: htmlspecialchars(): Passing null to parameter #1 ($string) of type string is deprecated in /home/beat/www/j/templates/system/html/modules.php on line 101 and a few more (line 35, line 53, line 115)
and same in Protostar and Beez modules.php
Testing Instructions
Code review.
Or see front-end with PHP 8.1, debug ON, and all PHP errors on.
Actual result BEFORE applying this Pull Request
Deprecated: htmlspecialchars(): Passing null to parameter #1 ($string) of type string is deprecated in /home/beat/www/j/templates/system/html/modules.php on line 101
and in some other cases:
Deprecated: htmlspecialchars(): Passing null to parameter #1 ($string) of type string is deprecated in /home/beat/www/j/templates/system/html/modules.php on line 35
Deprecated: htmlspecialchars(): Passing null to parameter #1 ($string) of type string is deprecated in /home/beat/www/j/templates/system/html/modules.php on line 53
Deprecated: htmlspecialchars(): Passing null to parameter #1 ($string) of type string is deprecated in /home/beat/www/j/templates/system/html/modules.php on line 115
Expected result AFTER applying this Pull Request
not those warnings anymore.
Documentation Changes Required
None.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End Templates (site) |
| Title |
|
||||||
| Labels |
Added:
?
|
||
| Title |
|
||||||
Removed WIP, ready for review.
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36794.
I have tested this item
Code review.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36794.
| Status | Pending | ⇒ | Ready to Commit |
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36794.
Merging thanks :)
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2022-01-23 14:18:09 |
| Closed_By | ⇒ | zero-24 | |
| Labels |
Added:
?
?
|
||
Making WIP, protostar has same issue, maybe beez too,more commits coming.