[wp-trac] [WordPress Trac] #37359: wp_filter_pre_oembed_result() breaks SoundCloud embeds

WordPress Trac noreply at wordpress.org
Thu Jul 14 19:32:16 UTC 2016


#37359: wp_filter_pre_oembed_result() breaks SoundCloud embeds
-------------------------------+------------------------------
 Reporter:  MaffooClock        |       Owner:
     Type:  defect (bug)       |      Status:  closed
 Priority:  normal             |   Milestone:  Awaiting Review
Component:  Embeds             |     Version:  4.5.3
 Severity:  normal             |  Resolution:  invalid
 Keywords:  reporter-feedback  |     Focuses:
-------------------------------+------------------------------
Changes (by MaffooClock):

 * status:  new => closed
 * resolution:   => invalid


Comment:

 > First, you should never alter WordPress core code. Second, simply
 removing the return statement can cause more harm than good and lead to
 unexpected behaviour.
 Seriously?  Thanks, but I really don't need the lecture on how to go about
 troubleshooting; it insults my intelligence.  The point of sharing that
 was so that I share my observations, not to be criticized as a developer.
 I'd already implemented the `remove_filter()` workaround in production, by
 the way.

 Yes, it only happens to SoundCloud URLs (and all of them).  The only other
 oEmbed-enabled URLs I'm using are for YouTube, and those work fine.

 I've heard more than once that SoundCloud URLs are working fine on 4.5.3
 sites, so the problem must be local to me.  Since I don't have the time
 nor the will to setup a staging site to fiddle with disabling plugins and
 trying other themes, and since there's a 0% chance that I'll ever try to
 embed one of my own URLs, I'll just stick with the `remove_filter()` for
 now.

 Closing this bug report.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/37359#comment:2>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list