Function bbp_make_clickable() problem with shortcodes

[GDragoN]

When URL is inside the shortcode, in some cases it will break the shortcode processing and whole content. This is caused by bbp_make_clickable() function that fails to properly process URL.

This is OK:

[shortcode]https://www.dev4press.com/ [/shortcode]

And this is OK:

[shortcode]https://www.dev4press.com[/shortcode]

But, this is not OK:

[shortcode]https://www.dev4press.com/[/shortcode]

When the URL ends with / character and shortcode is closed immediately after, bbp_make_clickable() will include shortcode ending in URL and break the layout later when shortcode parser takes over.

Milan

[netweb]

Milan,

I've tried replicating this with a few shotcodes here on my test site but cannot reproduce the issue with bbPress 2.5.8

What shortcode are you using where this happens? Could you maybe add some code here for that shortcut please?

[netweb]

@GDragoN Could you test this against /trunk now, the changes in [6014] may also have fixed this issue of yours.

[GDragoN]

It works now. Thanks!

[johnjamesjacoby]

Assigning all closed & unassigned tickets in the 2.6 milestone to myself.