Text links are stripped from image captions

[dnusim]

Basic HTML, like text links, should be allowed in image captions. Links are now stripped from the captions.

Text links in the image caption would be the most logical way to credit the photographer. In some cases, a link to the license covering the image is also mandatory. Creative Commons-licensed images are a great example of this.

[simonlawrence]

When you use the visual editor to add a link to the caption text it is stripped out when saving but you can edit the caption attribute in the HTML view and it will save.

I have found several people talking about it:

​http://www.priceplow.com/wordpress-link-in-image-caption
​http://www.transformationpowertools.com/wordpress/hyperlinks-in-wordpress-image-captions
​http://blogwip.com/images/adding-linked-captions-images-wordpress/

Looking through the code and it appears to be the "wpeditimage" TinyMCE JavaScript plugin that is replacing all HTML tags in the contents of the caption attribute:

h=h.replace(/<\S[^<>]*>/gi,"").replace(/'/g,"&#39;").replace(/"/g,"&quot;");

from /wp-includes/js/tinymce/wp-tinymce.php

[dnusim]

As it turns out: Links with singly-quoted attributes are only stripped if you are using the visual editor (in WP 2.9.2). If you are using the basic (HTML) editor, they are instead escaped before inserting them into the shortcode, which works fine.

I'm guessing a solution would involve doubly-quoted attributes being made singly-quoted before they are escaped, as I'm guessing the double quotes mess up the way shortcodes work. On top of that we need to figure out why the visual editor doesn't give the same result as the basic editor.

Changing milestone to 3.1 as this seems to affect a lot of people.

[dnusim]

This issue seems to be resolved in trunk. If someone will confirm, I think we can close this.

[blepoxp]

Single quotes work in HTML mode. Double quotes in HTML mode (escaped or not) break the caption if caption="..." uses double quotes. Tags are stripped if you use the caption field in the Image GUI but the caption still works.

[nacin]

I thought this was going to happen in 3.1 with the TinyMCE update. Moving there just because I think there might be a duplicate ticket or some code somewhere. Otherwise, punt.

[SergeyBiryukov]

Related: #18311

[ericlewis]

I'm not sure if this is perfect, but I think a step in the right direction here. This patch helps with adding links to captions in the visual editor mode.

The issue with links in the visual editor is that captions have all HTML entities filtered out of them. The patch filters out only non-anchor tags, leaving anchors there.

The other issue is the caption shortcode holds the text for caption between double quotes. To deal with this all quotes within the caption are converted to single quotes.

[ericlewis]

Fix replace ' instead of ' for single quotes in [caption] string

[ericlewis]

This has at this point been taken care of in #18311