[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