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

Backport of #3981: Plaintext mode of mailcloak #4359

Merged
merged 1 commit into from Sep 28, 2014

Conversation

Harmageddon
Copy link
Contributor

Hi folks!

PR #3981 fixed the plaintext functionality of the emailcloak plugin for 3.x. This PR fixes the same thing for 2.5.x.

Testing instructions

  • Edit the plugin "Content - Email Cloaking".
    • Enabled: Yes
    • Mode: Non-linkable text

Use cases of this plugin (every of them with javascript enabled and disabled):

  1. <a href="mailto:email@example.org">email@example.org</a>
  2. <a href="mailto:email@example.org">text</a>
  3. <a href="mailto:email@example.org"><img src="..." /></a>
  4. <a href="mailto:email@example.org?subject=Text">email@example.org</a>
  5. <a href="mailto:email@example.org?subject=Text">text</a>
  6. <a href="mailto:email@example.org?subject=Text"><img src="..." /></a>
  7. plain email@example.org

@infograf768
Copy link
Member

Makes sense.

@zero-24
Copy link
Contributor

zero-24 commented Sep 27, 2014

@infograf768 have you test it?

This comment was created with the J!Tracker Application at http://issues.joomla.org/.

@infograf768 infograf768 added the RTC This Pull Request is Ready To Commit label Sep 28, 2014
@mbabker mbabker added this to the Joomla! 2.5.26 milestone Sep 28, 2014
mbabker added a commit that referenced this pull request Sep 28, 2014
Backport of #3981: Plaintext mode of mailcloak
@mbabker mbabker merged commit 84a2b7b into joomla:2.5.x Sep 28, 2014
@zero-24 zero-24 removed the RTC This Pull Request is Ready To Commit label Oct 14, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants