I was unable to fit your Instructions:

1. How to assign a Contact not to a Contact-Category? If i delete all Categories got "Invalid field: Category"
2. Can't find "Add Mailto: Link" in com_contact. Or you mean with "Go to the component settings ..." Plugin "Content - Email Cloaking", where a "Mode: Non-linkable Text" exist.

Please can you explain so i can Test or do i understand some completely wrong?

@franz-wohlkoenig thank you for your feedback.

What I ment was that you don't have to assign the contact to a category other that "Uncategories". You did not have to delete any categories.

The "Add Mailto: Link" parameter in the com_contact options dialoge is only available after you apply the patch There is no connection to the email cloak plugin anymore.

Of course you can also change the parameter in a contact item or in the menu item options it belongs to.

I have tested this item successfully on 90d51bd

The patch works!

I have tested this item successfully on 90d51bd

Tested on Contact General Options, Menu Options and Contact Options to show/hide "Add Mailto: Link".

@brianteeman can we get a RTC label here?

Someone else will have to do it. I wont personally set a new feature to RTC
if I dont understand it

On 27 March 2016 at 11:20, zero-24 notifications@github.com wrote:

@brianteeman https://github.com/brianteeman can we get a RTC label here?

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#9490 (comment)

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

no Problem. @wilsonge can you set it RTC?

I have tested this item unsuccessfully on 90d51bd

Tested the option for contact and component,
could not find the option in the menu option.

I have tested this item ✅ successfully on 90d51bd

I follow your test instructions and without the patch I have a permanent mailto link. After applying the patch I can switch the mailto link off or on.
It works for me correctly.

@icampus

I have tested this item ✅ successfully on 90d51bd

I followed the test steps and before the patch there was a permanent mailto link. After applying the patch the link was gone.
Worked for me.

@icampus

@matrikular Can you please update your PR so the merge conflicts are resolved? Thank you.

I have tested this item ✅ successfully on 90d51bd

I have tested this item ✅ successfully on 90d51bd

I have tested successfully. I think you should do RTC.

without conflicts solved there is no RTC

Could someone please confirm that the conflicts have been resolved by the last commit?

All good - conflicts resolved
This branch has no conflicts with the base branch

"This branch has conflicts that must be resolved"

Is there a possibility to communicate this change to admins, e.g. via post-installation message.
If one is using a template override for components/com_contact/views/contact/tmpl/default_address.php with "old"

<?php echo $this->contact->email_to; ?>

instead of "new"

<?php echo JHtml::_('email.cloak', $this->contact->email_to, (bool) $this->params->get('add_mailto_link', true)); ?>

the result will be a not cloaked email address in FE because cloaking has been removed from view.html.php with this PR.

It is not correct to move the cloaking code

@bertmert seems to be a good idea. Either that and / or introducing something similar to $item->email_to_raw, $this>contact->email_to_raw. What do you think?

As you may know, preventing access to the raw email address by cloaking it in the view was a bad choice in the first place.

As to the conflicts, I'll look into that.

Please see #15973