Strip shortcodes from content before creating default broadcast

[alexkingorg]

http://wordpress.org/support/topic/brackets-show-in-twitter-post

[jondavidjohn]

@alexkingorg

2 options here...

https://github.com/crowdfavorite/wp-social/blob/master/lib/social/service.php#L411-L415

We can either strip_shortcodes($post->post_content) or do_shortcodes($post->post_content) the content as the first step here... part of me thinks the do_shortcodes() is more expected, obviously we're stripping tags here also so any shortcode that produces tags wouldn't have an impact, but if a shortcode is producing content, it will carry through in the broadcast.

Thoughts?

[alexkingorg]

Agreed, let's "do". We're stripping tags as part of the cleanup so we don't need to worry about junking things up that way.

[jondavidjohn]

This should do it then...

https://github.com/crowdfavorite/wp-social/compare/feature;fix-shortcodes-in-broadcasts-122

[alexkingorg]

Looks good to me.