Opened 3 months ago

Last modified 3 months ago

#23517 new defect (bug)

Shortcode: if last parameter ends with '/', it is mistaken for self-closing shortcode closure — at Version 4

Reported by: sergem Owned by:
Priority: normal Milestone: Awaiting Review
Component: Shortcodes Version: 3.5.1
Severity: normal Keywords: has-patch
Cc:

Description (last modified by SergeyBiryukov)

In example:

[shortcode urlparam=http://somesite/path/]content text[/shortcode]
"[shortcode urlparam=http://somesite/path/]" 

is interpreted as self closing:

[shortcode/] with urlparam="http://somesite/path"

All examples in http://codex.wordpress.org/Shortcode_API include whitespace before '/]'

This prevents implementation of bbcode parser for people to paste flickr photo snippets.

Change History (4)

Similar to HTML, the [shortcode/] syntax is a self-closing shortcode, and there's no easy way to distinguish that from the argument value, unless you enclose that in quotes, which should work:

[shortcode urlparam="http://somesite/path/"]

Suggesting to close as invalid.

  • Keywords has-patch added

lookbehind non-whitespace does it nicely!

  • Description modified (diff)
Note: See TracTickets for help on using tickets.