summary refs log tree commit diff
diff options
context:
space:
mode:
authorPhilipp Hagemeister <phihag@phihag.de>2015-02-01 15:08:33 +0100
committerPhilipp Hagemeister <phihag@phihag.de>2015-02-01 15:08:33 +0100
commitc1f06d6307109acfa4434a38131c7d9a07d58ab6 (patch)
treeccb1bee60907e28ad3465a1618621c0130c2077d
parentc4e817ce4aeeb4012b0e128afe6c138d3d905a24 (diff)
downloadyoutube-dl-c1f06d6307109acfa4434a38131c7d9a07d58ab6.tar.gz
youtube-dl-c1f06d6307109acfa4434a38131c7d9a07d58ab6.tar.xz
youtube-dl-c1f06d6307109acfa4434a38131c7d9a07d58ab6.zip
[macgamestore] Modernize
-rw-r--r--youtube_dl/extractor/macgamestore.py17
1 files changed, 8 insertions, 9 deletions
diff --git a/youtube_dl/extractor/macgamestore.py b/youtube_dl/extractor/macgamestore.py
index b818cf50c..3cd4a3a19 100644
--- a/youtube_dl/extractor/macgamestore.py
+++ b/youtube_dl/extractor/macgamestore.py
@@ -1,7 +1,5 @@
 from __future__ import unicode_literals
 
-import re
-
 from .common import InfoExtractor
 from ..utils import ExtractorError
 
@@ -13,21 +11,22 @@ class MacGameStoreIE(InfoExtractor):
 
     _TEST = {
         'url': 'http://www.macgamestore.com/mediaviewer.php?trailer=2450',
-        'file': '2450.m4v',
         'md5': '8649b8ea684b6666b4c5be736ecddc61',
         'info_dict': {
+            'id': '2450',
+            'ext': 'm4v',
             'title': 'Crow',
         }
     }
 
     def _real_extract(self, url):
-        mobj = re.match(self._VALID_URL, url)
-        video_id = mobj.group('id')
-
-        webpage = self._download_webpage(url, video_id, 'Downloading trailer page')
+        video_id = self._match_id(url)
+        webpage = self._download_webpage(
+            url, video_id, 'Downloading trailer page')
 
-        if re.search(r'>Missing Media<', webpage) is not None:
-            raise ExtractorError('Trailer %s does not exist' % video_id, expected=True)
+        if '>Missing Media<' in webpage:
+            raise ExtractorError(
+                'Trailer %s does not exist' % video_id, expected=True)
 
         video_title = self._html_search_regex(
             r'<title>MacGameStore: (.*?) Trailer</title>', webpage, 'title')