WordPress.org
Get WordPress

Make WordPress Core

Opened 9 years ago

Closed 9 years ago

#33522 closed defect (bug) (fixed)

Remove blip.tv from oEmbed Provider List

Reported by: tomthewebmaster's profile tomthewebmaster Owned by: pento's profile pento
Milestone: 4.4 Priority: normal
Severity: normal Version: 4.4
Component: Embeds Keywords: good-first-bug has-patch dev-feedback needs-docs
Focuses: Cc:

Description

The blip.tv service was shutdown on August 20, 2015 (more information: ​http://blip.tv/shutdown/message.html). As such, this service should probably be removed from the WordPress oEmbed provider list.

I assume line 43 needs to be removed in this file: https://core.trac.wordpress.org/browser/trunk/src/wp-includes/class-oembed.php.

Attachments (1)

33522-1.diff​ (1.1 KB) - added by rabmalin 9 years ago.
Remove blip.tv from oembed

Download all attachments as: .zip

Change History (9)

#1 @SergeyBiryukov
9 years ago

  • Milestone changed from Awaiting Review to 4.4

#2 @tomthewebmaster
9 years ago

Any backwards compatibility issues that would result from removing this service from the embed provider list should also be considered.

#3 @pento
9 years ago

From what I can tell, videos weren't migrated to Maker.tv, so I'm inclined to just remove Blip.tv. Existing embeds will be already broken.

#4 @wonderboymusic
9 years ago

  • Keywords needs-patch good-first-bug added

@rabmalin
9 years ago

Remove blip.tv from oembed

#5 @rabmalin
9 years ago

  • Keywords has-patch dev-feedback added; needs-patch removed

Patch added.

#6 @pento
9 years ago

  • Owner set to pento
  • Status changed from new to accepted

#7 @DrewAPicture
9 years ago

  • Keywords needs-docs added

We'll also need to adjust the hook doc for the oembed_providers filter to move the "Blip" entry down to the section for providers no longer supported.

#8 @pento
9 years ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In 33719:

oEmbed: Remove blip.tv as an oEmbed provider, the service has been shut down.

Vale, blip.tv.

Fixes #33522.

Props rabmalin.

Note: See TracTickets for help on using tickets.