Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[4.4] Fix Banners application crashes #42869

Merged
merged 4 commits into from Feb 28, 2024

Conversation

joomdonation
Copy link
Contributor

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:

  • No documentation changes for docs.joomla.org needed
  • No documentation changes for manual.joomla.org needed

@alikon
Copy link
Contributor

alikon commented Feb 24, 2024

I have tested this item ✅ successfully on 183b562

code review


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

1 similar comment
@chmst
Copy link
Contributor

chmst commented Feb 24, 2024

I have tested this item ✅ successfully on 183b562

code review


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

@chmst chmst removed the PR-4.4-dev label Feb 24, 2024
@chmst
Copy link
Contributor

chmst commented Feb 24, 2024

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Feb 24, 2024
@Quy Quy added this to the Joomla! 4.4.4 milestone Feb 24, 2024
@laoneo laoneo merged commit 4132f18 into joomla:4.4-dev Feb 28, 2024
3 checks passed
@laoneo
Copy link
Member

laoneo commented Feb 28, 2024

Thanks!

@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Feb 28, 2024
@joomdonation joomdonation deleted the fix_banner_helper branch March 7, 2024 06:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants