RTC PR-4.4-dev Pending

User tests: Successful: Unsuccessful:

avatar rdeutz
rdeutz
12 Sep 2024

Pull Request for Issue #42459 partly .

Summary of Changes

The calculation of the next execution is not working as expected. This fix is for the intervals:

  • interval-minutes
  • interval-hours
  • interval-days
  • interval-months

I haven't looked into the cron-execution but will do in the furture (seems to me working right).

Testing Instructions

Create a task (e.g. Sleep) and chose Interval Days and set "Interval in Days" to 1 and the "Execution Time (UTC)" to 11:11 and save it.

Actual result BEFORE applying this Pull Request

You will see that the day is the next day but the time is the current time.

Expected result AFTER applying this Pull Request

The day is the next day and the time is 11:11

Apply the patch and try again, now it should be as expected. Play around with the other options and settings.

For the calculation of the next execution I have used the current time, then I am adding the interval and finally set time and day if needed.

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar rdeutz rdeutz - open - 12 Sep 2024
avatar rdeutz rdeutz - change - 12 Sep 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 12 Sep 2024
Category Administration
avatar rdeutz rdeutz - change - 12 Sep 2024
Labels Added: PR-4.4-dev
avatar TLWebdesign TLWebdesign - test_item - 18 Sep 2024 - Tested successfully
avatar TLWebdesign
TLWebdesign - comment - 18 Sep 2024

I have tested this item ✅ successfully on 52f7cb7


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

avatar dautrich dautrich - test_item - 18 Sep 2024 - Tested successfully
avatar dautrich
dautrich - comment - 18 Sep 2024

I have tested this item ✅ successfully on 52f7cb7


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

avatar Quy
Quy - comment - 18 Sep 2024

Not sure to mark RTC per comment.

avatar rdeutz
rdeutz - comment - 18 Sep 2024

@Quy I would set RTC because it is a good step in the right direction and we only have a few days before the last 4.4.x Version before securtiy only fixes will be released. It would be a shame to have it so broken as it is now. I am going to make a 2nd PR and will work on the other issues in the comment.

avatar Quy Quy - change - 18 Sep 2024
Status Pending Ready to Commit
avatar Quy
Quy - comment - 18 Sep 2024

RTC


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

avatar MacJoom MacJoom - change - 20 Sep 2024
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2024-09-20 07:26:40
Closed_By MacJoom
Labels Added: RTC
avatar MacJoom MacJoom - close - 20 Sep 2024
avatar MacJoom MacJoom - merge - 20 Sep 2024
avatar MacJoom
MacJoom - comment - 20 Sep 2024

Thank you!

Add a Comment

Login with GitHub to post a comment