Latest Release breaks The Events Calendar by Tribe

  • Resolved karenratte

    (@karenratte)


    8 years, 8 months ago

    Either the latest release of EWWWW Image Optimizer has broken The Events Calendar or visa versa. Here is the error I get.

    [[ if(imageTooltipSrc.length) { ]]
[[=title]]
[[ } ]] [[ if(excerpt.length) { ]]
[[=raw excerpt]]
[[ } ]]

'//@ sourceURL' and '//@ sourceMappingURL' are deprecated, please use '//# sourceURL=' and '//# sourceMappingURL=' instead.
tribe_tmpl_tooltip:1 '//@ sourceURL' and '//@ sourceMappingURL' are deprecated, please use '//# sourceURL=' and '//# sourceMappingURL=' instead.

    I have fixed it on the original site. The problem still exists on my test site but is not open to the public. If you need to access it, please let me know and I will grant access to you.
    thanks!

    https://www.ads-software.com/plugins/ewww-image-optimizer/

Viewing 7 replies - 1 through 7 (of 7 total)
  • Plugin Author nosilver4u

    (@nosilver4u)

    None of that looks like it has anything to do with EWWW, that sounds like an issue within Events Calendar itself.

    Thread Starter karenratte

    (@karenratte)

    that could very well be, but I disabled this plugin and the events tooltips work again. I had to choose between the two. EWWW has been disabled. I cannot play go between. I hope you two work it out.

    Plugin Author nosilver4u

    (@nosilver4u)

    Before I can even attempt to replicate the issue, I’ll need you to turn on EWWW’s debugging option, and post the debug information from the setting page (via pastebin.com or using a code block).

    Plugin Author nosilver4u

    (@nosilver4u)

    Nevermind, I saw the problem, and was going back through old threads to find the fix when I ran across this: https://www.ads-software.com/support/topic/alternative-webp-rewriting-not-working-with-cloudflare?replies=10

    Basically, the html specification states that the very first time the parser (usually a browser) sees a closing tag starting like ‘</’, it is supposed to recognize that as the end of the inline <script> element.

    The unfortunate fact is that no browsers actually follow that rule, which causes developers to think they are following the spec (when they aren’t). On the flip side, the PHP parser DOES follow that rule, and will insert a closing </script> tag when it isn’t needed.

    It isn’t something I can fix, but the Events Calendar devs should be able to fix it simply. They just need to escape the slashes in their ending tags when they are inside a <script> element.

    So instead of using </a> they would need to make that <\/a> and so on for each html element within the script elements.

    Thread Starter karenratte

    (@karenratte)

    Thank you SOOO much. I will let them know.

    Thread Starter karenratte

    (@karenratte)

    Question: Under Conversion settings, Alternative WebP Rewriting is checked per instructions as I use MAXCDN and a Comet Cache. This is clearly the issue here. Do I need this option checked to make the best use of EWWW?
    Thanks!

    Plugin Author nosilver4u

    (@nosilver4u)

    Yes, if you want to use WebP, that’s the only way of doing it. The only other way I know of to do WebP behind a CDN/proxy is to use the Cache Enabler plugin from KeyCDN, but since you’re already using Comet Cache, you could request them to add in a WebP feature like Cache Enabler has. It’s much cleaner when they do it at the caching level too.

Viewing 7 replies - 1 through 7 (of 7 total)
  • The topic ‘Latest Release breaks The Events Calendar by Tribe’ is closed to new replies.