[#47213] - [5.4] Fix null value for date in publish_up
- Fixed in Code Base
- 31 Aug 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 |
| Status | Closed | ⇒ | New |
| Closed_Date | 2026-03-14 08:56:55 | ⇒ | |
| Closed_By | chmst | ⇒ |
| Status | New | ⇒ | Pending |
I have tested this item ✅ successfully on ae89430
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47213.
I have tested this item ✅ successfully on ae89430
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47213.
I have tested this item ✅ successfully on ae89430
Create an article with Unpublished status.
Go to Home Dashboard.
Scroll to Popular Articles section.
See deprecated message before PR.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47213.
I have tested this item ✅ successfully on ae89430
Create an article with Unpublished status.
Go to Home Dashboard.
Scroll to Popular Articles section.
See deprecated message before PR.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47213.
| Status | Pending | ⇒ | Ready to Commit |
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47213.
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47213.
| Labels |
Added:
RTC
|
||
Final test before merge: ✅
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2026-08-31 14:10:21 |
| Closed_By | ⇒ | richard67 |
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.