• Resolved NightL

    (@nightl)


    version 3.0.1
    I checked “plain emails by converting them to mailto links” and it scrambled everything on the page.

    I was not going to check other pages as I needed to revert it back asap

    Obviously I can’t show a demonstration of this while it totally disrupts the website but will check on a test site tomorrow with version 3.0.2.

    • This topic was modified 5 years, 2 months ago by NightL.
Viewing 5 replies - 1 through 5 (of 5 total)
  • Plugin Author Ironikus

    (@ironikus)

    Hey @nightl – thanks for reaching out!
    I tried the functionality again and it works great from my end.
    Would be nice if you can share later some more details on how the code on your page looks like and which emails get scrambled.
    Will be happy to assist you here to get it sorted!

    Thread Starter NightL

    (@nightl)

    Maybe Korean text characters triggering issue.

    WP Mailto Links version 3.0.2
    Wordress up to date
    Theme: Twenty Seventeen / Woocommerce
    everything up to date

    WP Mailto Links is on default settings except convert plain emails checkbox

    It was by chance I was on one particular page when I tested it, as it turns out all other pages are fine, including several other similar product pages.

    The only thing I can think that maybe triggering this issue is that there are Korean characters in the text.

    As it should look
    https://www.essentialvoice.com.au/voice-training-courses/seoul-november-2019/

    As it looks with convert plain email checked
    https://www.dropbox.com/s/iydf49e6ss25e9c/screencapture-essentialvoice-au-voice-training-courses-seoul-november-2019-2019-09-12-14_50_49.png?dl=0

    the product description html code
    https://www.dropbox.com/s/zamru6jdfniril0/html-code.jpg?dl=0

    As far as I can tell the only thing unique is the Korean text characters. The page is otherwise identical to about 7 other pages.

    I tested the page with the product/event description removed and it was OK – the plain text email at bottom of page was now active and without all the extra jumbled text/code appearing.

    Plugin Author Ironikus

    (@ironikus)

    Hi @nightl – thanks for your detailed description of the issue. I was able to reproduce it and down below you will find more details about it, as well as a solution.

    The problem:
    The issue happens because of an email that occurs within the Woocommerce variation attribute tag (JSON) within the HTML form element.
    This will break the site when you either:
    1. Activate the protection (with Javascript) and activating the “replace plain emails to mailto links)
    2. activate the admin icon that shows if the email is secured or not.

    I extended the logic to soft filter certain (critical) attributes with simple html entities or the protection text (depending on your settings).

    To offer you some proper tests, I created you a beta version here: https://ironikus.com/visit/wp-mailto-links-newest-beta

    In case this solves your problem, it will be great if you can confirm it, then I will push this functionality with our new release.

    In case you have any questions, feel free to reach out at any time!

    Thread Starter NightL

    (@nightl)

    Yes, this is working
    The following page is displaying correctly with the beta version currently running and convert plain emails checked.
    https://www.essentialvoice.com.au/voice-training-courses/seoul-november-2019/

    I check all other pages which were fine.

    On a side note and somewhat irrelevant – as I am logged in and follow through to the woocommerce checkout page, the billing form is already pre-filled. I note the padlock symbol also displays for the email form field. Obviously this is not an active email address. As I mentioned it is of zero issue as only admin would see it and has zero negative impact.

    Plugin Author Ironikus

    (@ironikus)

    Hey @nightl – Thank you a lot for your message, I’m happy to hear that it is solved!
    Regarding the other, non-existent email: This is correct since our plugin catches EVERY email on your page, to provide you the best security.
    In case you have any questions, feel free to reach out again!

Viewing 5 replies - 1 through 5 (of 5 total)
  • The topic ‘plain emails by converting them to mailto links – scrambled page’ is closed to new replies.