Skip to content

Conversation

@janschoenherr
Copy link
Contributor

@janschoenherr janschoenherr commented Nov 17, 2025

Pull Request for Issue #46356.

Summary of Changes

In Joomla 6 the algo26-matthias/idna-convert package got updated to version 4.2.1.

That library now throws an error when it encounters the ending com\.

With the change of this PR the thrown error is ignored and the email is kept.

This is the inital PR that introduced the emailToPunycode function:
#9041

Update

It seems to me, like the whole function was introduced because of the email cloaking plugin.
https://issues.joomla.org/tracker/joomla-cms/8881
That seems to be handling UTF-8 characters just fine now. Can we maybe remove that behavior from the ComponentHelper::filterText() function all together?

Testing Instructions

In Joomla 6 try to save an article with the following content

"mailto:[email protected]\"

Actual result BEFORE applying this Pull Request

Exception is thrown "Prohibited input U+0000005C"

Expected result AFTER applying this Pull Request

Article is saved.

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

@tecpromotion
Copy link
Contributor

I have tested this item ✅ successfully on 309a243


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

@michaelmaass
Copy link

I have tested this item ✅ successfully on 309a243

Tested successfully! No error anymore.


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

@tecpromotion
Copy link
Contributor

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Nov 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug PR-6.0-dev RTC This Pull Request is Ready To Commit

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants