-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[5.4] fix update notification scheduled task #46315
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
Conversation
AutoupdateRegisterState is int enum
|
I have tested this item ✅ successfully on f5aa259 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46315. |
1 similar comment
|
I have tested this item ✅ successfully on f5aa259 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46315. |
|
RTC This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46315. |
|
✅ Final test before merge with JBT recreate 5.3.3 (as for 5.3.4
|
|
Thank you @heelc29 for your contribution. Thank you @richard67 and @alikon for testing. |
|
I think after update to 5.4.0 it have many sites with locked task "Update Notification" and this block all other tasks in the Task Scheduler. with the next Joomla-Update-Version to fix this problem? PR #46315 not change locked status of "Update Notification" of the Task Scheduler in the database. |
|
I don’t think we should do that for all sites on update. By the way, you can also unlock the locked tasks which have that running man icon by clicking on that icon. |
|
I know this all. Other minimal solution: Information after next Joomla-Update to check the Task Scheduler of locked tasks and unlock information. I think it is better to do anything before many amateur websites crashed with full database. |
Pull Request for Issue #46308 .
Summary of Changes
Fix data type for AutoupdateRegisterState
must be of type int, string givenThis happens only on updated sites, because the param entry for
update_statusis not present in database and so the default parameter of->get()is used.Now the same integer default value is used like here (
Unsubscribed):joomla-cms/administrator/components/com_joomlaupdate/src/Controller/UpdateController.php
Line 738 in 08ae2c6
joomla-cms/administrator/components/com_joomlaupdate/src/Enum/AutoupdateRegisterState.php
Lines 20 to 26 in 08ae2c6
Testing Instructions
Update NotificationtaskActual result BEFORE applying this Pull Request
Scheduled task fails

PHP error message in server response

Task is locked

Expected result AFTER applying this Pull Request
Scheduled task should work

Link to documentations
Please select: