-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
[5.3] Move code to log model in new scheduler history view #44587
base: 5.3-dev
Are you sure you want to change the base?
Conversation
administrator/components/com_scheduler/src/Controller/LogsController.php
Show resolved
Hide resolved
Please check the response of the ajax request in the browser console. What for an error did you get? |
|
Set your server into debug mode and post the whole stack trace please. |
my fault there was an obsolete plugin listening |
I have tested this item ✅ successfully on 7bbfc2a This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44587. |
I have tested this item ✅ successfully on f9bfbdf Thanks laoneo. This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44587. |
As I added only types, removed an obsolete string and alpha ordered the new strings, I mark this as RTC because it needs to go into an alpha. This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44587. |
RTC This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44587. |
Summary of Changes
This is a followup for #42530 which uses the existing functions of the MVC API and removes some duplicated code. Like that are all ACL checks done, the default events triggered and code looks more inline with the rest of the core.
Testing Instructions
Actual result BEFORE applying this Pull Request
All works.
Expected result AFTER applying this Pull Request
All works.
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