Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

custom fields - emails are not cloaked #42824

Closed
LukasHH opened this issue Feb 16, 2024 · 3 comments
Closed

custom fields - emails are not cloaked #42824

LukasHH opened this issue Feb 16, 2024 · 3 comments

Comments

@LukasHH
Copy link
Contributor

LukasHH commented Feb 16, 2024

Steps to reproduce the issue

  • Create custom fields for users or articles as text or url.
  • Enter an email address as plain text or mailto link.
  • Display the fields in the frontend.

Expected result

The emails are cloaked by the "emailcloak" plugin and are protected from spambots.

Actual result

The emails are not cloaked by the "emailcloak" plugin and are displayed in plain text. They are not protected from spambots.

System information (as much as possible)

Joomla! 4.4.2 Stable
PHP 8.2.12

Additional comments

If I write e-mails directly into an article, they are cloaked. However, if they come via a custom field, they are not cloaked.

If I have emails in a custom module, I can set the "Prepare Content" option so that the emails are cloaked. However, this is not possible with custom fields.

@brianteeman
Copy link
Contributor

"prepare content" should happen on fields already. There was a small bug that prevented it in some circumstances that was fixed with #42744 I suspect that this will fix your issue so please check and report back

@LukasHH
Copy link
Contributor Author

LukasHH commented Feb 17, 2024

I have checked this with the fix #42744, but it only affects the field plugin. But it's good to know that it now works correctly there too.

In the upper part of my screenshot, the fields are displayed via the field settings "automatic display: before content". The fix has no effect here.

grafik

@alikon
Copy link
Contributor

alikon commented Dec 10, 2024

please test #44597

@alikon alikon closed this as completed Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants