joomla-cms
joomla-cms copied to clipboard
[4.4] Fix Banners application crashes
Pull Request for Issue #42841.
Summary of Changes
The current code in BannerHelper classs bind a Joomla\CMS\Date\Date object to :date parameter and could cause application crashes in some environments as reported in the issue. This PR changes code so that the parameter is bound by a string in Datetime format instead (same with the code in other places in Joomla). I also rename the variable to $nowDate to imply that this is current date time (also, same with similar codes use in other places in Joomla)
Testing Instructions
- Use Joomla 4.4
- Apply patch
- Access to Banners Management screen from administrator area of your site
Actual result BEFORE applying this Pull Request
Sometime application crashes
Expected result AFTER applying this Pull Request
Always works, no crashes anymore.
Link to documentations
Please select:
- [x] No documentation changes for docs.joomla.org needed
- [x] No documentation changes for manual.joomla.org needed
I have tested this item :white_check_mark: successfully on 183b5621e7ade4f0e926580eedd0ea848edea98e
code review
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42869.
I have tested this item :white_check_mark: successfully on 183b5621e7ade4f0e926580eedd0ea848edea98e
code review
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42869.
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42869.
Thanks!