[embed] from structured video/audio post format not working with post_formats_compat.

[jbobich]

When using a theme that relies on post_formats_compat for video and audio posts (Twenty Twelve currently for example) --

If the user puts a video/audio file URL in through the new post formats UI with not video in the content of the page, the media will not embed properly within the final output of the_content().

[jbobich]

Apologies, what I meant to say was --

"If the user puts a video/audio file URL in through the new post formats UI with no embed in the content of the page, the media will not embed properly within the final output of the_content()."

[wonderboymusic]

The video does not display?

[jbobich]

Yup. I'm using latest trunk version with TwentyTwelve with no plugins installed. Can you not reproduce?

For example, I have a post setup like this:

​http://www.uploadblvd.com/uploads/image_5168921c85b8c.png

Before my patch, it displays like this:

​http://www.uploadblvd.com/uploads/image_516891bd6f206.png

After, it displays as you'd expect:

​http://www.uploadblvd.com/uploads/image_5168920ea737d.png

[wonderboymusic]

I see it now, that is really bizarre

[kovshenin]

This is interesting, and not limited to post formats, but the [embed] shortcode with audio and video files. The problem lies within the WP_Embed->shortcode method, which calls the wp_audio_embed and wp_video_embed functions, both of which return the URL, and not the embedded media, because WP_Embed->shortcode runs when all other shortcodes are not registered yet.

The reason it "works" is that the resulting URL is picked up by WP_Embed->autoembed and parsed again using wp_audio_embed or wp_video_embed, so if there's anything else on the same line that doesn't match the audio/video regexes, it fails. Even worse is when the resulting URL contains a </div> like in the content produced by the audio/video formats compat code. This breaks the theme's markup because both embed handlers don't escape the URL.

24070.2.diff​ makes both embed handlers return the (undone) shortcode, regardless of whether the shortcode has been registered or not.

[jbobich]

@kovshenin -- Awesome, that definitely solves it.

Is there a reason why wp_audio_embed and wp_video_embed need to parse the shorcodes to begin with (i.e. before 24070.2.diff​)? Meaning, does removing that functionality and just allowing the shortcodes to be parsed along with everything else later on effect anything else?

[kovshenin]

Replying to jbobich: The functions don't do any shortcode parsing, they're callbacks for WP_Embed, meaning you can insert a URL to an .mp3 file on a line by itself, and WordPress will figure it out and embed it in an audio player, sort of how you can insert links to YouTube videos or tweets (though that's a tiny bit different).

[kovshenin]

Refreshed in 24070.3.diff​ after [24000].

[markjaquith]

In 24083:

Fix issue with [embed] in post_format_compat.

props kovshenin. fixes #24070.