WordPress.org

Make WordPress Core

Opened 9 years ago

Closed 8 years ago

#22346 closed feature request (maybelater)

Add Videojug.com to internal whitelist

Reported by: Videojug Owned by:
Milestone: Priority: normal
Severity: normal Version: 3.4.2
Component: Embeds Keywords: has-patch
Focuses: Cc:

Description

Dear Wordpress,

I am writing on behalf of Videojug. We create and distribute hiqh quality "How-To" online content, and have been producing professional, instructional, short-form video content since 2006. We receive many requests monthly from our Contributors and Users asking for assistance in embedding our videos in their Wordpress sites. Currently it is a convoluted process involving plugins adding complexities.

Other video providers in our sector are already whitelisted. We would like to be added please to your internal whitelist, allowing any Wordpress blogger to embed our videos by copying the URL just like they would when embedding a Youtube video. Our own embed code has been stable for a long time.

We have our oembeds working in the following wordpress account:
http://chloeluper.co.uk/index.php/embed-test-page/

This is the add oembed provider code:

wp_oembed_add_provider( '#http://(www\.)?videojug.com/(film|interview)/.*#i', 'http://www.videojug.com/oembed.{format}', true );

I have attached the modified default theme.

Thank you
Imad Salhi

Attachments (2)

functions.php (20.3 KB) - added by Videojug 9 years ago.
The modified default theme.
22346.diff (830 bytes) - added by Ipstenu 9 years ago.
Adds it in as an oebmed.

Download all attachments as: .zip

Change History (6)

@Videojug
9 years ago

The modified default theme.

@Ipstenu
9 years ago

Adds it in as an oebmed.

#1 @Ipstenu
9 years ago

  • Keywords has-patch added

Probably best done as a plugin for now, but here's a patch FWIW :)

#2 follow-up: @Videojug
9 years ago

  • Cc Videojug added

Hi Ipstenu,

Thank you very much for your reply, what do you mean by "Has patch"?

How do we create the patch? Is this a placeholder to support the oembed before it is added to the oembed whitelist?

Thank you
Imad

#3 in reply to: ↑ 2 @rmccue
9 years ago

Replying to Videojug:

Thank you very much for your reply, what do you mean by "Has patch"?

ipstenu added a patch as an attachment above, which provides the code to add just the change to the file, rather than the full replacement for the file. The has-patch keyword lets those browsing the issue list to see which tickets have the code changes.

#4 @SergeyBiryukov
8 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to maybelater
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.