Opened 20 months ago

Last modified 13 months ago

#18776 new defect (bug)

balanceTags should ignore shortcodes & their content

Reported by: ciantic Owned by:
Priority: normal Milestone: Future Release
Component: Shortcodes Version: 3.2.1
Severity: normal Keywords: needs-patch
Cc:

Description

Take following input:

something [myshortcode]<b>[/myshortcode]

after balanceTags() it becomes:

something [myshortcode]<b>[/myshortcode]</b>

Well, this is not what the shortcode creator probably wanted.

Proposed fixes:

If you are feeling lazy: Convert [.*] to <$1> and since balanceTags handles XML it should work, afterwards convert appropriate <> back to []. Though this does not give a shortcode creator chance to disallow balancing tags inside it's content which is bad.

Better fix is to allow shortcode creators to decide wether their contents should be balanced:

  1. balanceTags() should ignore all shortcodes and their contents.
  2. Parse all shortcodes in content and on each shortcode call apply filter on so that shortcode creator can hook (if sees it appropriate) on to that. Shortcode creator then can call balanceTags() in his filter of that shortcode in order to run balanceTags() contents of shortcode separately if wanted.

Change History (1)

  • Keywords needs-patch added
  • Milestone changed from Awaiting Review to Future Release
  • Version set to 3.2.1
Note: See TracTickets for help on using tickets.