[engadget] remove support for legacy urls
parent
6d6536acb2
commit
d48352fb5d
|
@ -1,21 +1,13 @@
|
||||||
from __future__ import unicode_literals
|
from __future__ import unicode_literals
|
||||||
|
|
||||||
import re
|
|
||||||
|
|
||||||
from .common import InfoExtractor
|
from .common import InfoExtractor
|
||||||
from ..utils import (
|
|
||||||
url_basename,
|
|
||||||
)
|
|
||||||
|
|
||||||
|
|
||||||
class EngadgetIE(InfoExtractor):
|
class EngadgetIE(InfoExtractor):
|
||||||
_VALID_URL = r'''(?x)https?://www.engadget.com/
|
_VALID_URL = r'https?://www.engadget.com/video/(?P<id>\d+)'
|
||||||
(?:video(?:/5min)?/(?P<id>\d+)|
|
|
||||||
[\d/]+/.*?)
|
|
||||||
'''
|
|
||||||
|
|
||||||
_TEST = {
|
_TEST = {
|
||||||
'url': 'http://www.engadget.com/video/5min/518153925/',
|
'url': 'http://www.engadget.com/video/518153925/',
|
||||||
'md5': 'c6820d4828a5064447a4d9fc73f312c9',
|
'md5': 'c6820d4828a5064447a4d9fc73f312c9',
|
||||||
'info_dict': {
|
'info_dict': {
|
||||||
'id': '518153925',
|
'id': '518153925',
|
||||||
|
@ -27,15 +19,4 @@ class EngadgetIE(InfoExtractor):
|
||||||
|
|
||||||
def _real_extract(self, url):
|
def _real_extract(self, url):
|
||||||
video_id = self._match_id(url)
|
video_id = self._match_id(url)
|
||||||
|
return self.url_result('5min:%s' % video_id)
|
||||||
if video_id is not None:
|
|
||||||
return self.url_result('5min:%s' % video_id)
|
|
||||||
else:
|
|
||||||
title = url_basename(url)
|
|
||||||
webpage = self._download_webpage(url, title)
|
|
||||||
ids = re.findall(r'<iframe[^>]+?playList=(\d+)', webpage)
|
|
||||||
return {
|
|
||||||
'_type': 'playlist',
|
|
||||||
'title': title,
|
|
||||||
'entries': [self.url_result('5min:%s' % vid) for vid in ids]
|
|
||||||
}
|
|
||||||
|
|
Loading…
Reference in New Issue