[sina] Modernize and simplify

This commit is contained in:
Philipp Hagemeister 2015-02-01 15:16:35 +01:00
parent 9f281cacd2
commit 437cac8cc1

View file

@ -11,7 +11,7 @@
class SinaIE(InfoExtractor): class SinaIE(InfoExtractor):
_VALID_URL = r'''https?://(.*?\.)?video\.sina\.com\.cn/ _VALID_URL = r'''(?x)https?://(.*?\.)?video\.sina\.com\.cn/
( (
(.+?/(((?P<pseudo_id>\d+).html)|(.*?(\#|(vid=)|b/)(?P<id>\d+?)($|&|\-)))) (.+?/(((?P<pseudo_id>\d+).html)|(.*?(\#|(vid=)|b/)(?P<id>\d+?)($|&|\-))))
| |
@ -23,9 +23,10 @@ class SinaIE(InfoExtractor):
_TESTS = [ _TESTS = [
{ {
'url': 'http://video.sina.com.cn/news/vlist/zt/chczlj2013/?opsubject_id=top12#110028898', 'url': 'http://video.sina.com.cn/news/vlist/zt/chczlj2013/?opsubject_id=top12#110028898',
'file': '110028898.flv',
'md5': 'd65dd22ddcf44e38ce2bf58a10c3e71f', 'md5': 'd65dd22ddcf44e38ce2bf58a10c3e71f',
'info_dict': { 'info_dict': {
'id': '110028898',
'ext': 'flv',
'title': '《中国新闻》 朝鲜要求巴拿马立即释放被扣船员', 'title': '《中国新闻》 朝鲜要求巴拿马立即释放被扣船员',
} }
}, },
@ -39,10 +40,6 @@ class SinaIE(InfoExtractor):
}, },
] ]
@classmethod
def suitable(cls, url):
return re.match(cls._VALID_URL, url, flags=re.VERBOSE) is not None
def _extract_video(self, video_id): def _extract_video(self, video_id):
data = compat_urllib_parse.urlencode({'vid': video_id}) data = compat_urllib_parse.urlencode({'vid': video_id})
url_doc = self._download_xml('http://v.iask.com/v_play.php?%s' % data, url_doc = self._download_xml('http://v.iask.com/v_play.php?%s' % data,
@ -59,7 +56,7 @@ def _extract_video(self, video_id):
} }
def _real_extract(self, url): def _real_extract(self, url):
mobj = re.match(self._VALID_URL, url, flags=re.VERBOSE) mobj = re.match(self._VALID_URL, url)
video_id = mobj.group('id') video_id = mobj.group('id')
if mobj.group('token') is not None: if mobj.group('token') is not None:
# The video id is in the redirected url # The video id is in the redirected url