[#36797] - [3.10] [PHP 8.1] Fixes MVC BaseController Deprecated ucfirst() and Deprecated strtolower() null to string type warnings
- Fixed in Code Base
- 23 Jan 2022
- Medium
- Build: 3.10-dev
- # 36797
- Diff
- beat:patch-22
User tests: Successful: Unsuccessful:
Pull Request for Issue # none
Summary of Changes
Fixes MVC BaseController for warnings:
Deprecated: ucfirst(): Passing null to parameter #1 ($string) of type string is deprecated in /home/beat/www/j/libraries/src/MVC/Controller/BaseController.php on line 286
and
Deprecated: strtolower(): Passing null to parameter #1 ($string) of type string is deprecated in /home/beat/www/j/libraries/src/MVC/Controller/BaseController.php on line 684
Testing Instructions
Code review is probably enough, as trivial.
Check frontend with PHP 8.1 with all errors on and Joomla Debug ON.
Actual result BEFORE applying this Pull Request
Deprecated: ucfirst(): Passing null to parameter #1 ($string) of type string is deprecated in /home/beat/www/j/libraries/src/MVC/Controller/BaseController.php on line 286
Deprecated: strtolower(): Passing null to parameter #1 ($string) of type string is deprecated in /home/beat/www/j/libraries/src/MVC/Controller/BaseController.php on line 684
Expected result AFTER applying this Pull Request
Those warnings disappeared.
Documentation Changes Required
None.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Libraries |
Merging. Thanks.
| Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2022-01-23 17:19:21 |
| Closed_By | ⇒ | zero-24 | |
| Labels |
Added:
?
?
|
||
I have tested this item✅ successfully on d9c64d8
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36797.