joomla-cms icon indicating copy to clipboard operation
joomla-cms copied to clipboard

[4.4] Fix Banners application crashes

Open joomdonation opened this issue 1 year ago • 3 comments

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

joomdonation avatar Feb 24 '24 07:02 joomdonation

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.

alikon avatar Feb 24 '24 08:02 alikon

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.

chmst avatar Feb 24 '24 11:02 chmst

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42869.

chmst avatar Feb 24 '24 11:02 chmst

Thanks!

laoneo avatar Feb 28 '24 11:02 laoneo