[wp-trac] [WordPress Trac] #31502: Replace embed URL with error message if oEmbed provider returns an error
WordPress Trac
noreply at wordpress.org
Wed Sep 9 14:08:22 UTC 2015
#31502: Replace embed URL with error message if oEmbed provider returns an error
----------------------------------------+------------------------------
Reporter: mbootsman | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Embeds | Version: 2.9
Severity: normal | Resolution:
Keywords: needs-patch good-first-bug | Focuses:
----------------------------------------+------------------------------
Comment (by deremohan):
Replying to [comment:16 jesin]:
> Patch [attachment: 31502.diff] only checks when adding an embed URL.
OP's problem is what happens when an existing video is removed by the
video hosting service.
>
Hi Jesin,
As looking ajax call inside ajax-actions.php file at line 2818:
{{{
if ( $url && ! $parsed ) {
$parsed = $wp_embed->run_shortcode( $shortcode );
}
}}}
If existing video is removed from hosting, then video url is not get
parsed which leads to 'not-embeddable' type error. i mean this case should
be handle by WP_Embed class.
Also please have a look for patch which also handles "not-ssl" response
from ajax.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/31502#comment:17>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list