I believe that the YouTube search tab doesn't add a UX improvement that's proportional to the amount of dev time required to implement, improve and maintain it compared to simply pasting in a video URL. These statements are also echoed in #1871964-4: YouTube search adds results thumbnail images to library.

I propose moving the YouTube search tab into a separate module, called something like Media: YouTube Extras which would deal with the more edge-case YouTube functionality such as searching for videos and uploading videos to YouTube from within your site, etc.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Devin Carlson’s picture

Status: Active » Needs review
FileSize
16.26 KB

A patch to remove the search tab.

I also created a sandbox project which adds the functionality back in. See Media: YouTube Extras. I would be happy to promote the sandbox to a full project and give anyone who was interested in maintaining it access.

aaron’s picture

Status: Needs review » Needs work

After applying the patch to an existing installation, I get the following error:

Fatal error: require_once() [function.require]: Failed opening required 'C:\wamp\www\drupal/sites/all/modules/media_youtube/includes/MediaYouTubeBrowser.inc' (include_path='.;C:\php\pear') in C:\wamp\www\drupal\includes\bootstrap.inc on line 3080

Rob C’s picture

I had the same idea about a year ago, created some basic hooks for this, but didn't found the time to get something to work.

@Devin: message / IRC me if you need a hand, got some details / ideas about other non-youtube searches.

Maybe we should rename / rebuild this and call it Media: search i figured. (?)

guillaumev’s picture

Issue summary: View changes
FileSize
676 bytes

Just a quick & dirty patch for people who want to remove the "Youtube" tab from the media browser.

Devin Carlson’s picture

Version: 7.x-2.x-dev » 7.x-3.x-dev
Status: Needs work » Needs review
FileSize
17.87 KB

An updated patch to remove the search tab now that it is no longer functional due to changes in the YouTube API.

Devin Carlson’s picture

Status: Needs review » Fixed

Tested and committed #5 to Media: YouTube 7.x-3.x.

  • Devin Carlson committed e68c605 on 7.x-3.x
    Issue #2072327 by Devin Carlson, guillaumev: Spin off YouTube search tab...
zonesny’s picture

The YouTube search tab does work. Please see the patch we've implemented here:
https://www.drupal.org/node/2410027

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

Chris Charlton’s picture

What was the final spinoff module?