Feeling Lucky
?
[#36282] - [4] php 8.1 deprecation
- Fixed in Code Base
- 11 Dec 2021
- Medium
- Build: 4.0-dev
- # 36282
- Diff
- alikon:patch-3
User tests: Successful: Unsuccessful:
Pull Request for Issue # .
- PHP Deprecated: Declaration of Joomla\CMS\Date\Date::format($format, $local = false, $translate = true) should be compatible with DateTime::format(string $format): string in ../joomla-cms/libraries/src/Date/Date.php
- PHP Deprecated: Declaration of Joomla\CMS\Date\Date::setTimezone($tz) should be compatible with DateTime::setTimezone(DateTimeZone $timezone): DateTime in ../joomla-cms/libraries/src/Date/Date.php
Summary of Changes
added annotation #[\ReturnTypeWillChange]
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Libraries |
| Labels |
Added:
?
|
||
wilsonge
- comment
- 10 Dec 2021
For J4 we're just going to have to add the annotations. As mentioned this is a b/c break for anything extending these classes. Annotations should work back to PHP 7.2 as well.
| Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-12-11 18:30:37 |
| Closed_By | ⇒ | wilsonge |
wilsonge
- comment
- 11 Dec 2021
Thanks!
Unfortunately it will lead to Fatal error, for everyone who extends our
Dateclass.We need a better idea, how to solve it.