summary refs log tree commit diff
diff options
context:
space:
mode:
authorSergey M․ <dstftw@gmail.com>2014-03-29 15:55:03 +0700
committerSergey M․ <dstftw@gmail.com>2014-03-29 15:55:03 +0700
commit056b56688a2ee5ba39dd5b2cbb003fd98529180f (patch)
treeda2188f2489625bd2521b8def1e46512cef3e19d
parentb17418313f15f030dcc938f5618cdccf55752e39 (diff)
downloadyoutube-dl-056b56688a2ee5ba39dd5b2cbb003fd98529180f.tar.gz
youtube-dl-056b56688a2ee5ba39dd5b2cbb003fd98529180f.tar.xz
youtube-dl-056b56688a2ee5ba39dd5b2cbb003fd98529180f.zip
[ntv] Simplify
-rw-r--r--youtube_dl/extractor/ntv.py19
1 files changed, 9 insertions, 10 deletions
diff --git a/youtube_dl/extractor/ntv.py b/youtube_dl/extractor/ntv.py
index e998d156e..8447a9b86 100644
--- a/youtube_dl/extractor/ntv.py
+++ b/youtube_dl/extractor/ntv.py
@@ -5,7 +5,7 @@ import re
 
 from .common import InfoExtractor
 from ..utils import (
-    RegexNotFoundError,
+    ExtractorError,
     unescapeHTML
 )
 
@@ -98,16 +98,15 @@ class NTVIE(InfoExtractor):
 
         page = self._download_webpage(url, video_id, 'Downloading page')
 
-        def extract(patterns, name, page, fatal=False):
-            for pattern in patterns:
-                mobj = re.search(pattern, page)
-                if mobj:
-                    return mobj.group(1)
-            if fatal:
-                raise RegexNotFoundError(u'Unable to extract %s' % name)
-            return None
+        for pattern in self._VIDEO_ID_REGEXES:
+            mobj = re.search(pattern, page)
+            if mobj:
+                break
 
-        video_id = extract(self._VIDEO_ID_REGEXES, 'video id', page, fatal=True)
+        if not mobj:
+            raise ExtractorError('No media links available for %s' % video_id)
+
+        video_id = mobj.group(1)
 
         player = self._download_xml('http://www.ntv.ru/vi%s/' % video_id, video_id, 'Downloading video XML')
         title = unescapeHTML(player.find('./data/title').text)