Hide related videos on youtube

levavie - February 20, 2009 - 14:20
Project:Video Filter
Version:6.x-2.x-dev
Component:Code
Category:task
Priority:normal
Assigned:Unassigned
Status:closed
Description

The following patch hides the related videos displayed on youtube films. This is the default option for emfield. But this module is better since it allows to embed videos in the middle of the post.

AttachmentSize
video_filter.codecs.inc_.patch670 bytes

#1

scottrigby - August 10, 2009 - 03:06
Version:6.x-2.4» 6.x-2.x-dev
Status:needs review» reviewed & tested by the community

@levavie: Nice work :) I want to mark RTBC - the patch above is against 6.x-2.4, so needs to be updated.

Rerolling against 6.x-2.x-dev

AttachmentSize
video_filter_378148_1.patch 809 bytes

#2

blackdog - August 11, 2009 - 19:40
Status:reviewed & tested by the community» postponed (maintainer needs more info)

I wonder if we should make this a setting? I'm not sure, but if we change this now, when people are 'used' to having related content show up, some might be surprised.

#3

scottrigby - August 12, 2009 - 17:42

@blackdog I fully agree. Do you have any preferences for where / how this should be configured? I assume this is only for youtube, just a new variable like variable_set('video_filter_related', 1) defaulting to 'Show related videos'?

#4

blackdog - August 13, 2009 - 07:39

@scottrigby - Yeah, something like that would be great.

#5

scottrigby - August 14, 2009 - 02:19
Status:postponed (maintainer needs more info)» needs review

@blackdog how's this?

AttachmentSize
video_filter_378148_5.patch 2.15 KB

#6

blackdog - August 16, 2009 - 11:09
Status:needs review» fixed

Committed. Thanks!

#7

scottrigby - August 25, 2009 - 19:15

Sweet!

#8

System Message - September 8, 2009 - 19:20
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.