Twitter sharing html tags

[aosmichenko]

Hello everyone,

I have found an issue or maybe it's not, but when you are trying to share post via twitter it doesn't esc html for example: http://okmagazine.com/dress-up/trend-to-try/ok-look-of-the-day-mariah-careys-santa-baby-style/ try to share on twitter.

I have found where it creates sharing text :/modules/sharedaddy/sharing-sources.php
on line 412-417 code says:

if ( 140 < $strlen( $post_title ) + $suffix_length ) {
// The -1 is for "\xE2\x80\xA6", a UTF-8 ellipsis.
$text = $substr( $post_title, 0, 140 - $suffix_length - 1 ) . "\xE2\x80\xA6";
} else {
$text = $post_title;
}

What if we use strip_tags() http://www.php.net/strip_tags function to remove them, I have seen many times, when people add to the wordpress post's and page's titles.

Thanks, Alex.

[georgestephanis]

Aha, good call. I'm not quite sure if that's the best place to fix the issue, but it should be a relatively simple fix. Let me check -- one moment.

[aosmichenko]

Good way, I didn't find this function in codex(

[nb]

We should also try to convert HTML entities to Unicode characters. I recently noticed an HTML-encoded em dash in a tweet.

[aosmichenko]

#32