Skip to content

[5.3] Handle fields event in EmailCloak plugin#44923

Merged
HLeithner merged 5 commits intojoomla:5.3-devfrom
rdeutz:fix_42824_cloakemail
Feb 27, 2025
Merged

[5.3] Handle fields event in EmailCloak plugin#44923
HLeithner merged 5 commits intojoomla:5.3-devfrom
rdeutz:fix_42824_cloakemail

Conversation

@rdeutz
Copy link
Contributor

@rdeutz rdeutz commented Feb 16, 2025

Pull Request for Issue #42824

Summary of Changes

Handeling the onCustomFieldsAfterPrepareField in the EmailCloak plugin so that the value of a field when it is an email get's
cloaked.

Testing Instructions

Create a text field for an article
Enter an email address into the article
Check the output of the article and see if the email address is cloaked

Actual result BEFORE applying this Pull Request

Email address is plain text

Expected result AFTER applying this Pull Request

Tthe email address is cloaked

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

Co-authored-by: Richard Fath <richard67@users.noreply.github.com>
rdeutz and others added 2 commits February 16, 2025 16:40
Co-authored-by: Richard Fath <richard67@users.noreply.github.com>
@MacJoom MacJoom added the PBF Pizza, Bugs and Fun label Feb 17, 2025
@ghost
Copy link

ghost commented Feb 20, 2025

I have tested this item ✅ successfully on 763e374

Before After
1 2


@LukasHH Can you test the Pull request too (#42824)?


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

@LukasHH
Copy link
Contributor

LukasHH commented Feb 20, 2025

I have tested this item ✅ successfully on 763e374

grafik

@richard67
Copy link
Member

I have tested this item ✅ successfully on 763e374

@LukasHH Please mark your test result by going to the issue tracker https://issues.joomla.org/tracker/joomla-cms/44923 and using the blue "Test this" button at the top left corner and chosing the test result and finally submitting. Thanks in advance.

@LukasHH
Copy link
Contributor

LukasHH commented Feb 20, 2025

I have tested this item ✅ successfully on 763e374

i have test this item successfully
grafik


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

@richard67
Copy link
Member

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Feb 20, 2025
@richard67 richard67 removed the PBF Pizza, Bugs and Fun label Feb 20, 2025
@tekvishal
Copy link

I have tested this item ✅ successfully on 763e374

I have tested this item ✅ successfully on 763e374


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

@ghost
Copy link

ghost commented Feb 22, 2025

@tekvishal Its enough to mark your test and hit the "Submit test result" button. Easier for you ;-)

Untitled

@ceus1984
Copy link

I have tested this item ✅ successfully on 763e374

it turned to "joomla hidden mail"


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

@Sabine-2023
Copy link

I have tested this item ✅ successfully on 763e374

Testet: works fine


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

@alikon
Copy link
Contributor

alikon commented Feb 22, 2025

RTC


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

@HLeithner HLeithner merged commit dcbaacd into joomla:5.3-dev Feb 27, 2025
4 checks passed
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Feb 27, 2025
@rdeutz rdeutz added this to the Joomla! 5.3.0 milestone Feb 28, 2025
janschoenherr pushed a commit to janschoenherr/joomla-cms that referenced this pull request Mar 10, 2025
Co-authored-by: Richard Fath <richard67@users.noreply.github.com>
@rdeutz rdeutz deleted the fix_42824_cloakemail branch May 14, 2025 10:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants