Merge pull request #2859 from pulpe/FunnyOrDie_thumb

[FunnyOrDie] fix thumbnails + add test (fixes #2856)
This commit is contained in:
Sergey M. 2014-05-06 19:46:40 +07:00
commit 84a2806c16
1 changed files with 22 additions and 21 deletions

View File

@ -8,18 +8,27 @@ from .common import InfoExtractor
class FunnyOrDieIE(InfoExtractor): class FunnyOrDieIE(InfoExtractor):
_VALID_URL = r'https?://(?:www\.)?funnyordie\.com/(?P<type>embed|videos)/(?P<id>[0-9a-f]+)(?:$|[?#/])' _VALID_URL = r'https?://(?:www\.)?funnyordie\.com/(?P<type>embed|videos)/(?P<id>[0-9a-f]+)(?:$|[?#/])'
_TEST = { _TESTS = [{
'url': 'http://www.funnyordie.com/videos/0732f586d7/heart-shaped-box-literal-video-version', 'url': 'http://www.funnyordie.com/videos/0732f586d7/heart-shaped-box-literal-video-version',
'file': '0732f586d7.mp4',
'md5': 'f647e9e90064b53b6e046e75d0241fbd', 'md5': 'f647e9e90064b53b6e046e75d0241fbd',
'info_dict': { 'info_dict': {
'description': ('Lyrics changed to match the video. Spoken cameo ' 'id': '0732f586d7',
'by Obscurus Lupa (from ThatGuyWithTheGlasses.com). Based on a ' 'ext': 'mp4',
'concept by Dustin McLean (DustFilms.com). Performed, edited, '
'and written by David A. Scott.'),
'title': 'Heart-Shaped Box: Literal Video Version', 'title': 'Heart-Shaped Box: Literal Video Version',
'description': 'md5:ea09a01bc9a1c46d9ab696c01747c338',
'thumbnail': 're:^http:.*\.jpg$',
}, },
} }, {
'url': 'http://www.funnyordie.com/embed/e402820827',
'md5': '0e0c5a7bf45c52b95cd16aa7f28be0b6',
'info_dict': {
'id': 'e402820827',
'ext': 'mp4',
'title': 'Please Use This Song (Jon Lajoie)',
'description': 'md5:2ed27d364f5a805a6dba199faaf6681d',
'thumbnail': 're:^http:.*\.jpg$',
},
}]
def _real_extract(self, url): def _real_extract(self, url):
mobj = re.match(self._VALID_URL, url) mobj = re.match(self._VALID_URL, url)
@ -31,23 +40,15 @@ class FunnyOrDieIE(InfoExtractor):
[r'type="video/mp4" src="(.*?)"', r'src="([^>]*?)" type=\'video/mp4\''], [r'type="video/mp4" src="(.*?)"', r'src="([^>]*?)" type=\'video/mp4\''],
webpage, 'video URL', flags=re.DOTALL) webpage, 'video URL', flags=re.DOTALL)
if mobj.group('type') == 'embed': post_json = self._search_regex(
post_json = self._search_regex( r'fb_post\s*=\s*(\{.*?\});', webpage, 'post details')
r'fb_post\s*=\s*(\{.*?\});', webpage, 'post details') post = json.loads(post_json)
post = json.loads(post_json)
title = post['name']
description = post.get('description')
thumbnail = post.get('picture')
else:
title = self._og_search_title(webpage)
description = self._og_search_description(webpage)
thumbnail = None
return { return {
'id': video_id, 'id': video_id,
'url': video_url, 'url': video_url,
'ext': 'mp4', 'ext': 'mp4',
'title': title, 'title': post['name'],
'description': description, 'description': post.get('description'),
'thumbnail': thumbnail, 'thumbnail': post.get('picture'),
} }