[#48045] - [5.4] Improve error logging in Mail API
- Pending
- Medium
- Build: 5.4-dev
- # 48045
- Diff
- alikon:patch-2
User tests: Successful: Unsuccessful:
Enhance error handling in Mail API by differentiating between error and debug messages.
Pull Request resolves #45963 .
- I read the Generative AI policy and my contribution is either not created with the help of AI or is compatible with the policy and GNU/GPL 2 or later.
Summary of Changes
change LOG priority based if is an error or not
Testing Instructions
- Enable 'Log Almost Everything' and 'Debug System'
- Home Dashboard > Server > Send Test Mail
- Check administrator/logs/everything.php
Actual result BEFORE applying this Pull Request
All log entries for sending e-mails have the priority ERROR
Expected result AFTER applying this Pull Request
if no errors logs priority DEBUG
Link to documentations
Please select:
-
Documentation link for guide.joomla.org:
-
No documentation changes for guide.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 | ⇒ | Libraries |
I have not tested this item.
I have successfully tested this during Friday PR Testing Session July 3rd, 2026.
Thanks @alikon
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/48045.
I have tested this item ✅ successfully on a3e4c0d
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/48045.
I have tested this item ✅ successfully on a3e4c0d
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/48045.
I have tested this item ✅ successfully on a3e4c0d
I have tested this successfully during Friday's PR Testing Session July 3rd.
Thanks @alikon
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/48045.
I have tested this item ✅ successfully on a3e4c0d
I have tested this successfully during Friday's PR Testing Session July 3rd.
Thanks @alikon
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/48045.
| Status | Pending | ⇒ | Ready to Commit |
| Labels |
Added:
bug
PR-5.4-dev
|
||
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/48045.
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/48045.
| Status | Ready to Commit | ⇒ | Pending |
Returned to “Pending” status due to concerns
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/48045.
Returned to “Pending” status due to concerns
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/48045.
| Labels |
Added:
Updates Requested
|
||
I have not tested this item.
I have successfully tested this during Friday PR Testing Session July 3rd, 2026.
Thanks @alikon
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/48045.