@formfranska please test the PR

Sorry, Not sure exactly what I'm supposed to do. I wasn't able to copy the text from your test article (couldn't scroll all the way down). Complete newbie in github. Sorry ?

You can use the "Raw" button, then the raw data will be displayed. Or the button "Copy", then the source text is copied.

Otherwise simply set an article in which an image is included, whose file name contains an @ sign.

@LukasHH Could you extend your testing instructions so that people paste the test content multiple times? We should have each test object multiple times to make sure that each occurrence of a pattern is cloaked, not only the first one, and that we don’t get into an endless loop with that while loop which we recently discussed at another place.

Sorry, Not sure exactly what I'm supposed to do. I wasn't able to copy the text from your test article (couldn't scroll all the way down). Complete newbie in github. Sorry ?

You can use the "Raw" button, then the raw data will be displayed. Or the button "Copy", then the source text is copied.

Otherwise simply set an article in which an image is included, whose file name contains an @ sign.

Thank you! OK, so I didn't have the Copy button but the Raw button was present so used that.

I published the article and only Sample 21 looks broken to me:

Sample 21:

Image filename in the form of an email

<img src="/images/demofiles/%3Cjoomla-hidden-mail%20%20is-link="1" is-email="1" first="dGVzdA==" last="aW1hZ2UucG5n" text="dGVzdEBpbWFnZS5wbmc=" base="" >This email address is being protected from spambots. You need JavaScript enabled to view it." width="215" height="121" loading="lazy" data-path="local-images:/demofiles/test@image.png" />

This email address is being protected from spambots. You need JavaScript enabled to view it." alt="" width="215" height="121" loading="lazy" data-path="local-images:/demofiles/test@image.png" />

<img src="/images/demofiles/%3Cjoomla-hidden-mail%20%20is-link="1" is-email="1" first="dGVzdA==" last="aW1hZ2UucG5n" text="dGVzdEBpbWFnZS5wbmc=" base="" >This email address is being protected from spambots. You need JavaScript enabled to view it." width="100" height="100" loading="lazy" data-path="local-images:/demofiles/test@image.svg" />

This email address is being protected from spambots. You need JavaScript enabled to view it." alt="" width="100" height="100" loading="lazy" data-path="local-images:/demofiles/test@image.svg" />

<img src="/images/demofiles/%3Cjoomla-hidden-mail%20%20is-link="1" is-email="1" first="dGVzdA==" last="aW1hZ2UucG5n" text="dGVzdEBpbWFnZS5wbmc=" base="" >This email address is being protected from spambots. You need JavaScript enabled to view it." alt="test@image.svg" width="100" height="100" loading="lazy" data-path="local-images:/demofiles/test@image.svg" />

test@image.svg

This email address is being protected from spambots. You need JavaScript enabled to view it." media="(min-width: 38em)" /> This email address is being protected from spambots. You need JavaScript enabled to view it." /> This email address is being protected from spambots. You need JavaScript enabled to view it." alt="" />

Was this helping you in any way? Or did I do it wrong?

This is the initial situation without the patch, which allows you to test all variants. You would now have to apply the changes and then it should be displayed correctly.
https://github.com/joomla/joomla-cms/pull/39932/files

You can also use the Patch Tester component for testing: https://docs.joomla.org/Component_Patchtester_for_Testers

This is the initial situation without the patch, which allows you to test all variants. You would now have to apply the changes and then it should be displayed correctly. https://github.com/joomla/joomla-cms/pull/39932/files

You can also use the Patch Tester component for testing: https://docs.joomla.org/Component_Patchtester_for_Testers

OK, thanks for your help every step of the way ?

Applied patch

Sample 21 now looks like this (with patch enabled)

Please note that when I try to look at the images I get

404 Not Found The resource requested could not be found on this server!

I have just updated and extended the page with the test code. Here I have also added the sample images. I have also added a readme with further explanation.
https://github.com/LukasHH/other-test

The images are currently not displayed for you, because they are not available. It is important that the file name is not cloaked.

I have just updated and extended the page with the test code. Here I have also added the sample images. I have also added a readme with further explanation. https://github.com/LukasHH/other-test

The images are currently not displayed for you, because they are not available. It is important that the file name is not cloaked.

Thank you! OK, so with this https://github.com/LukasHH/other-test it looks like this:

Is that how it should look?

Yes - this is how it should look. In the browser source code you can also see that the images with an @ are not cloaked, while emails are still cloaked by the plugin.

You can also test it with the as you had described in the forum.
https://forum.joomla.org/viewtopic.php?p=3683408#p3683408

I have tested this item ✅ successfully on cd6ae6b

Tested successfully using the sample page in Joomla 4.2.9-dev of 26 February using PHP 8.1.10 in Wampserver

@LukasHH

Status now: Everything done OK so far. Also images loaded etc.

But when applying Patchtester, get response:
The file marked for modification does not exist: plugins/content/emailcloak/emailcloak.php

But: Content – Email Cloacking is enabled!

@ChristineWk Can it be that you have tested on a 4.3-dev installation where that plugin has been converted to the new structure with PR #38466 ? This would explain your problem with the patchtester. This PR here is for the 4.2-dev branch where the file still is at the old place.

@richard67
Yes, you're right, I had the thought too... sorry.

@ChristineWk Hmm unfortunately you can't really test this PR on a 4.3-dev. But thanks for trying. Do you have a 4.2.8 or so for testing, too? On that you could test this PR.

I have a 4.2.9-dev, because for 4.2.8 there was a security release. Will try.

@LukasHH

Could you check again my test-site please? It seems OK now. Tks

@LukasHH

Could you check again my test-site please? It seems OK now. Tks
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39932.

Hello Christine,

I just see on your page that the Sample 17a and Sample 17b versions are not cloaked. They should still be cloaked.
I'll have a look at it again to see if it's the same for me.

I need to undo the change in line 444.
1e46b07

I have noticed that this change no longer cloaks the versions under Sample 17. Even without the change, this does not affect functionality with images that contain an @ in the filename.

@LukasHH

Hv reverted Patch, fetched and applied again.

@LukasHH

Hv reverted Patch, fetched and applied again.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39932.

This looks better now.

I have tested this item ✅ successfully on e033de6

I have tested this item ✅ successfully on e033de6

RTC

Thank you