[#37463] - [4.1-dev] Fix DB query in `Scheduler::getTaskRecords()`
- Fixed in Code Base
- 4 Apr 2022
- Medium
- Build: 4.1-dev
- # 37463
- Diff
- ditsuke:4.1-dev
User tests: Successful: Unsuccessful:
Summary of Changes
Something broke the SELECT * query, resulting in a NULL id when
it was finally evaluated against the DB in TasksModel. This broke
the scheduler:run console command which is fixed by this commit.
Testing Instructions
- Create a scheduled task.
- Trigger the scheduler from the CLI:
php joomla.php scheduler:run.
Actual result BEFORE applying this Pull Request
Run tasks
=========
In OptionsResolver.php line 1060:
The option "id" with value null is expected to be of type "numeric", but is of type "null".
Expected result AFTER applying this Pull Request
Run tasks
=========
Task#02 'Some Task' processed in 5.12 seconds.
Finished running 1 tasks in 5.12 seconds.
Documentation Changes Required
None
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration |
| Title |
|
||||||
| Title |
|
||||||
| Labels |
Added:
?
|
||
I have tested this item
Works with MySQL as well as with PostgreSQL.
Hint for other testers: The error fixed by this PR only happens when the task needs to run. I.e. if the task is scheduled to run every 5 minutes, it might take up to 5 minutes until you get the error for the first time, and then again 5 minutes until next time. In between, when the task doesn't need to be run, you get a "No tasks due!" message without and with this PR.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37463.
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37463.
| Status | Pending | ⇒ | Ready to Commit |
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37463.
| Labels |
Added:
?
|
||
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2022-04-04 11:11:25 |
| Closed_By | ⇒ | laoneo |
Thanks!
Thanks!
/cc @bembelimen