Feeling Lucky
PR-4.3-dev
[#41582] - Replace the special characters in site name for mail templates
- Fixed in Code Base
- 4 Sep 2023
- Medium
- Build: 4.3-dev
- # 41582
- Diff
- Digital-Peak:mailer/replace
User tests: Successful: Unsuccessful:
Pull Request for pr #41469.
Summary of Changes
This fixes a regression of #41469. When a test mail is sent without an @ or | sign in the site name, then the preg replace call returns null.
Testing Instructions
- Make sure you are on PHP 8.2
- Set reporting to maximum in the joomla configuration
- Set a site name without any special character
- Send a test mail
Actual result BEFORE applying this Pull Request
A javascript error where the response shows some deprecated warnings that null should not be passed. The test mail doesn't contain the site name in the subject.
Expected result AFTER applying this Pull Request
Test mail is correctly sent with the sitename in the subject and no JS error.
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
joomla-cms-bot
- | Category | ⇒ | Administration com_config |
| Status | New | ⇒ | Pending |
HLeithner
- | Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2023-09-04 11:37:02 |
| Closed_By | ⇒ | HLeithner | |
| Labels |
Added:
PR-4.3-dev
|
||