[#47213] - [5.4] Fix null value for date in publish_up
- Closed
- 14 Mar 2026
- Medium
- Build: 5.4-dev
- # 47213
- Diff
- chmst:54-mod-popular-fix
User tests: Successful: Unsuccessful:
Pull Request resolves # .
- I read the Generative AI policy and my contribution is either not created with the help of AI or is compatible with the policy and GNU/GPL 2 or later.
Summary of Changes
Fix deprecated warning in mod_popular when publish_up is null (php 8.4)
Testing Instructions
Code inspect.
Or set error-reporting to maximum, set in one of the popular articles the publish_up date to NULL (sql).
Actual result BEFORE applying this Pull Request
Deprecated: DateTime::__construct(): Passing null to parameter #1 ($datetime) of type string is deprecated
Expected result AFTER applying this Pull Request
No Warning.
Link to documentations
Please select:
-
Documentation link for guide.joomla.org:
-
No documentation changes for guide.joomla.org needed
-
Pull Request link for manual.joomla.org:
-
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Modules Administration |
I have tested this item 🔴 unsuccessfully on 3a15ab5
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47213.
I forgot to commit the helper file. Apologies.
| Labels |
Added:
bug
PR-5.4-dev
|
||
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2026-03-14 08:56:55 |
| Closed_By | ⇒ | chmst |
Not sure about this. Without the patch I get this:
But with the patch I get this:
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47213.