Language Change PR-5.3-dev Pending

User tests: Successful: Unsuccessful:

avatar richard67
richard67
9 Feb 2025

Pull Request for Issue #44827 - alternative 2 .

The other alternative would be PR #44845 .

Summary of Changes

This pull request (PR) reverts the action log system plugin's configuration option and the corresponding database column in the #__action_logs_users table, which were added with PR #44640 and released with 5.3.0-alpha3, and changes code so that the current user is always excluded from email notifications.

The code could possibly be improved so that certain kinds of actions would not be excluded.

Testing Instructions

Will be added sooner or later.

Actual result BEFORE applying this Pull Request

Will be added sooner or later.

Expected result AFTER applying this Pull Request

Will be added sooner or later.

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 richard67 richard67 - open - 9 Feb 2025
avatar richard67 richard67 - change - 9 Feb 2025
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 9 Feb 2025
Category Administration SQL com_admin Postgresql Language & Strings Installation Front End Plugins
avatar richard67 richard67 - change - 9 Feb 2025
Labels Added: Language Change PR-5.3-dev
avatar richard67 richard67 - change - 9 Feb 2025
The description was changed
avatar richard67 richard67 - edited - 9 Feb 2025
avatar richard67 richard67 - change - 9 Feb 2025
Title
[5.3] Revert new option and db column from PR #44640 and always exclude current user from email notifications
[5.3] Revert new option and db column from PR 44640 and always exclude current user from email notifications
avatar richard67 richard67 - edited - 9 Feb 2025
avatar richard67 richard67 - change - 9 Feb 2025
The description was changed
avatar richard67 richard67 - edited - 9 Feb 2025
avatar richard67 richard67 - change - 9 Feb 2025
The description was changed
avatar richard67 richard67 - edited - 9 Feb 2025
avatar chmst
chmst - comment - 11 Feb 2025

I prefer the second solution. It is as usual in many applications.

avatar richard67 richard67 - change - 12 Feb 2025
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2025-02-12 20:14:48
Closed_By richard67
avatar richard67 richard67 - close - 12 Feb 2025
avatar richard67
richard67 - comment - 12 Feb 2025

We have discussed the issue and my 2 alternatives in the CMS Maintenance Team's weekly meeting and decided to use my other PR to completely revert PR #44640 before 5.3.0-beta1. Then we have all the time to work on a better solution.

Closing in favour of #44845 .

Add a Comment

Login with GitHub to post a comment