[buddypress-trac] [BuddyPress] #2707: Support oembed

buddypress-trac at lists.automattic.com buddypress-trac at lists.automattic.com
Sat Jul 9 02:57:49 UTC 2011


#2707: Support oembed
---------------------+--------------------------------------------------
  Reporter:  DJPaul  |      Owner:
      Type:  defect  |     Status:  new
  Priority:  major   |  Milestone:  1.3
 Component:  Core    |    Version:  1.3
Resolution:          |   Keywords:  has-patch needs-refresh dev-feedback
---------------------+--------------------------------------------------
Changes (by r-a-y):

 * keywords:  has-patch dev-feedback => has-patch needs-refresh dev-feedback


Comment:

 I just need a little dev feedback.

 My sentiment is we can remove author_can() from the BP patch and iterate.
 The BP patch uses its own oEmbed discovery filter
 (bp_embed_oembed_discover) instead of the WP one (embed_oembed_discover)
 anyway.

 In addition, I think we should switch the "embed_oembed_html" filter to a
 new one (like "bp_embed_oembed_html"). "embed_oembed_html" handles the
 output of the embedded content; I'd like to use a new one to avoid any
 possible conflicts with existing WP Embed plugins that deal with posts.

 I don't have a problem with punting this ticket if necessary, as I can
 always add this code into my oEmbed plugin regardless.

 ---

 Patch needs to be refreshed again due to the newer changes in trunk.  Once
 everyone has chimed in, I'll create a new patch.

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/2707#comment:21>
BuddyPress <http://buddypress.org/>
BuddyPress


More information about the buddypress-trac mailing list