[#44709] - [5.2.3] User Action Logs Email
- Fixed in Code Base
- 26 Jan 2025
- Medium
- Build: 5.2-dev
- # 44709
- Diff
- brianteeman:actionlogsmail
- Pending continuous-integration/drone/pr Build is pending Details
User tests: Successful: Unsuccessful:
Pull Request for Issue #39328 .
Summary of Changes
- the htmlbody string is sanitized by the html editor when it is loaded which is why {messages} is moved outside of the table. This PR moves {messages} to wrap the table instead of the row to avoid this sanitization and make the email work
- {messages} is missing from the params field in the database which is why it doesnt appear in the list of tags to insert and thus without that tag nothing is in the email
Testing Instructions Part 1
Set Mail Templates to send mail as html
Enable receiving action log emails for your user and do something to trigger an email eg saving an article
Actual result BEFORE applying this Pull Request
Expected result AFTER applying this Pull Request
Testing Instructions Part 2
Open the User Actions Log: Notification Mail template
Actual result BEFORE applying this Pull Request
Expected result AFTER applying this Pull Request
NOTE: If a user has already saved a User Actions Log: Notification Mail template they will need to press the reset HTML Body button.
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
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | SQL Administration com_admin Postgresql Language & Strings Installation |
I have tested this item ✅ successfully on 9f3f7d3
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44709.
| Status | Pending | ⇒ | Ready to Commit |
| Labels |
Added:
bug
Language Change
PR-5.2-dev
|
||
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44709.
| Labels |
Added:
RTC
|
||
I'm unhappy with moving the MESSAGES outside of the table, since it means it is duplicating the complete table instead of just the row, but it seems right now this is the only solution we have. Merging this now. Thank you!
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2025-01-26 17:56:01 |
| Closed_By | ⇒ | Hackwar |
I have tested this item ✅ successfully on 9f3f7d3
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44709.