IMPORTANT MESSAGE REGARDING THE FORUM

As you may already know, the JoomlaWorks Forum is powered by Kunena (the only decent forum component for Joomla).

Unfortunately, the latest update for Kunena has seemingly broken all code snippets (old and new) in the forum.

Until this message is removed, please paste any code as regular text - in other words, don't use the [code]...[/code] tags in Kunena's post editor.

Keyword

System plugin shouldn't modify <html> tag on /adminstrator

  • Casper Voogt
  • Casper Voogt's Avatar Topic Author
  • Offline
  • New Member
More
6 years 4 months ago - 6 years 4 months ago #143657 by Casper Voogt
We just upgraded our CiviCRM to 4.6.2 and were then unable to use its Mailing feature. I traced this down to a syntax error in the HTML code produced by Joomla/K2, specifically "function onAfterRender()" in /plugins/system/k2.php. The relevant section of code was:
ogp.me......' was being passed into a CiviCRM JavaScript variable verbatim without escaping the quotes, even though other HTML elements with quotes do get escaped properly by CiviCRM. Seems that K2 alters the HTML output at a later stage than when CiviCRM grabs the HTML output - something along those lines. I could have probably solved this in CiviCRM's code too, but felt it made sense to do this in the K2 plugin, because why would we want OpenGraph HTML code in our admin area?

So, I changed this to:
		Last edit: 6 years 4 months ago  by Casper Voogt.		Reason: notify of replies	

Please Log in or Create an account to join the conversation.

More
6 years 4 months ago #143735 by Lefteris
Hi,

I just applied the fix : github.com/joomlaworks/k2/commit/f5cea21896647bf5be0cbe7ee9498272f298b27c .

Thank you for reporting.

IMPORTANT: Please search the forum before posting a question!

JoomlaWorks Support Team Member

---
JoomlaWorks
www.joomlaworks.net/

Please Log in or Create an account to join the conversation.

More
6 years 1 month ago #146137 by Jeremy Nicholls
Caspar, thanks for finding this! I had the same problem, but in a different part of CiviCRM, and never thought of checking K2.

Please Log in or Create an account to join the conversation.