diff options
author | Sergey M․ <dstftw@gmail.com> | 2015-10-01 22:44:51 +0600 |
---|---|---|
committer | Sergey M․ <dstftw@gmail.com> | 2015-10-01 22:44:51 +0600 |
commit | fb97809e64a145768346345ff2b5e5539880c014 (patch) | |
tree | e560ca2a46143bee46c7ee9d09ab907267866b42 /youtube_dl/extractor/videolecturesnet.py | |
parent | 0c996b9f488bfaa74d79e94739af80a0be38e125 (diff) |
[videolecturesnet] Improve playlist extraction
Diffstat (limited to 'youtube_dl/extractor/videolecturesnet.py')
-rw-r--r-- | youtube_dl/extractor/videolecturesnet.py | 34 |
1 files changed, 21 insertions, 13 deletions
diff --git a/youtube_dl/extractor/videolecturesnet.py b/youtube_dl/extractor/videolecturesnet.py index eadff8d18..704165406 100644 --- a/youtube_dl/extractor/videolecturesnet.py +++ b/youtube_dl/extractor/videolecturesnet.py @@ -3,8 +3,14 @@ from __future__ import unicode_literals import re from .common import InfoExtractor -from ..compat import compat_urlparse -from ..utils import parse_duration +from ..compat import ( + compat_HTTPError, + compat_urlparse, +) +from ..utils import ( + ExtractorError, + parse_duration, +) class VideoLecturesNetIE(InfoExtractor): @@ -28,17 +34,19 @@ class VideoLecturesNetIE(InfoExtractor): video_id = self._match_id(url) smil_url = 'http://videolectures.net/%s/video/1/smil.xml' % video_id - smil = self._download_smil(smil_url, video_id, fatal=False) - - # Probably a playlist - if smil is False: - webpage = self._download_webpage(url, video_id) - entries = [ - self.url_result(compat_urlparse.urljoin(url, video_url), 'VideoLecturesNet') - for _, video_url in re.findall(r'<a[^>]+href=(["\'])(.+?)\1[^>]+id=["\']lec=\d+', webpage)] - playlist_title = self._html_search_meta('title', webpage, 'title', fatal=True) - playlist_description = self._html_search_meta('description', webpage, 'description') - return self.playlist_result(entries, video_id, playlist_title, playlist_description) + + try: + smil = self._download_smil(smil_url, video_id) + except ExtractorError as e: + if isinstance(e.cause, compat_HTTPError) and e.cause.code == 404: + # Probably a playlist + webpage = self._download_webpage(url, video_id) + entries = [ + self.url_result(compat_urlparse.urljoin(url, video_url), 'VideoLecturesNet') + for _, video_url in re.findall(r'<a[^>]+href=(["\'])(.+?)\1[^>]+id=["\']lec=\d+', webpage)] + playlist_title = self._html_search_meta('title', webpage, 'title', fatal=True) + playlist_description = self._html_search_meta('description', webpage, 'description') + return self.playlist_result(entries, video_id, playlist_title, playlist_description) info = self._parse_smil(smil, smil_url, video_id) |