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

WordPress Trac noreply at wordpress.org
Thu Dec 22 16:28:38 UTC 2016


#37359: wp_filter_pre_oembed_result() breaks SoundCloud embeds
--------------------------+-------------------------
 Reporter:  MaffooClock   |       Owner:
     Type:  defect (bug)  |      Status:  closed
 Priority:  normal        |   Milestone:
Component:  Embeds        |     Version:  4.5.3
 Severity:  normal        |  Resolution:  worksforme
 Keywords:                |     Focuses:
--------------------------+-------------------------

Comment (by jkhongusc):

 Our Enterprise just hit this exact issue.  We upgraded from 4.4.2 to 4.7.
 After the upgrade we noticed SoundCloud embeds was broken on one of our
 sites exactly as the OP reports:
 > resulting in the post embedding itself where the SoundCloud Iframe is
 expected to be

 Through debugging, yes by modifying WP 4.7 core, on a staging system.  I
 was able to backtrace to wp_filter_pre_oembed_result().  IMO the bug
 resides in url_to_postid().  url_to_postid ('https://soundcloud.com/path')
 is returning the postid of the post that contains the SoundCloud embed.
 The configuration of the site has url_to_postid() returning self
 referencing postids.  If I put the same SoundCloud embed on another site,
 it works perfectly.  I am going to open a new bug for url_to_postid().

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


More information about the wp-trac mailing list