Project

General

Profile

Actions

Feature proposal #6850

closed

Remove htmlspecialchars_decode in content bot

Added by krileon over 6 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
High
Assignee:
Target version:
Start date:
09 November 2017
Due date:
% Done:

100%

Estimated time:

Description

htmlspecialchars_decode is being applied to the entire article text which in some cases can break existing markup, but this is needed to allow substitution shortcode to match since so many extensions and Joomla mess with the markup (WYSIWYG editors are a huge culprit here). See if this can be removed or done in a way that won't break the original markup.

https://www.joomlapolis.com/forum/147-potential-bug/237747-htmlspecialchars-decode-in-cbcontentbot-php#298859

Actions #1

Updated by krileon about 6 years ago

Simple fix for this is to change the " matching in REGEXP to (?:"|") allowing it to match the encoded version as well. The reason this issue happens is if you put a substitution inside of a WYSIWYG link URL parameter it'll encode the " (which is correct), but we still want the substitution to work.

Actions #2

Updated by krileon about 6 years ago

  • Status changed from Assigned to Resolved
  • % Done changed from 0 to 100

Fixed in MR !1304

Individual content bots need to also be fixed and any other custom substitution parsing needs to handle encoded quotes.

Actions #3

Updated by beat about 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF