RTC Language Change PR-5.3-dev Pending

User tests: Successful: Unsuccessful:

avatar laoneo
laoneo
6 Dec 2024

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

  • Run a task
  • Open the new task history view
  • Delete a log entry
  • Clear the whole history

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

avatar laoneo laoneo - open - 6 Dec 2024
avatar laoneo laoneo - change - 6 Dec 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 6 Dec 2024
Category Administration Language & Strings Front End Plugins
avatar laoneo laoneo - change - 6 Dec 2024
Title
Move to log model
[5.3] Move code to log model in new scheduler history view
avatar laoneo laoneo - edited - 6 Dec 2024
avatar laoneo laoneo - change - 6 Dec 2024
The description was changed
avatar laoneo laoneo - edited - 6 Dec 2024
avatar laoneo laoneo - change - 6 Dec 2024
The description was changed
avatar laoneo laoneo - edited - 6 Dec 2024
avatar laoneo laoneo - change - 6 Dec 2024
The description was changed
avatar laoneo laoneo - edited - 6 Dec 2024
avatar alikon
alikon - comment - 7 Dec 2024

just applied the pr and runned a default task

image

avatar laoneo
laoneo - comment - 7 Dec 2024

Please check the response of the ajax request in the browser console. What for an error did you get?

avatar alikon
alikon - comment - 7 Dec 2024

Call to undefined method Joomla\CMS\Document\JsonDocument::getToolbar()

avatar laoneo
laoneo - comment - 7 Dec 2024

Se your server into debug mode and post the whole stack trace please.

avatar laoneo
laoneo - comment - 7 Dec 2024

Set your server into debug mode and post the whole stack trace please.

avatar alikon
alikon - comment - 7 Dec 2024

my fault there was an obsolete plugin listening onBeforeRender

avatar alikon alikon - test_item - 7 Dec 2024 - Tested successfully
avatar alikon
alikon - comment - 7 Dec 2024

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.

avatar laoneo laoneo - change - 7 Dec 2024
Labels Added: Language Change PR-5.3-dev
avatar exlemor exlemor - test_item - 16 Dec 2024 - Tested successfully
avatar exlemor
exlemor - comment - 16 Dec 2024

I have tested this item ✅ successfully on f9bfbdf

I have successfully tested this as instructed. Worked beautifully.

Thanks laoneo.


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

avatar laoneo
laoneo - comment - 16 Dec 2024

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.

avatar laoneo laoneo - change - 16 Dec 2024
Status Pending Ready to Commit
avatar laoneo
laoneo - comment - 16 Dec 2024

RTC


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

avatar laoneo laoneo - change - 17 Dec 2024
Labels Added: RTC
avatar rdeutz rdeutz - change - 7 Jan 2025
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2025-01-07 18:08:14
Closed_By rdeutz
avatar rdeutz rdeutz - close - 7 Jan 2025
avatar rdeutz rdeutz - merge - 7 Jan 2025
avatar rdeutz
rdeutz - comment - 7 Jan 2025

Thanks.

Add a Comment

Login with GitHub to post a comment