Merge pull request #7691 from ryandesign/use-PYTHON-env-var

[youtube-dl] / youtube_dl / extractor / dump.py

diff --git a/youtube_dl/extractor/dump.py b/youtube_dl/extractor/dump.py
index cfa71bd38d9809cebc429f9197017b93e80682cd..ff78d4fd2e84c390a929f29a052b1dd87abe03d1 100644 (file)
--- a/youtube_dl/extractor/dump.py
+++ b/youtube_dl/extractor/dump.py
@@ -4,36 +4,36 @@ from __future__ import unicode_literals
 import re
 
 from .common import InfoExtractor
-from ..utils import (
-    ExtractorError,
-)
+
 
 class DumpIE(InfoExtractor):
     _VALID_URL = r'^https?://(?:www\.)?dump\.com/(?P<id>[a-zA-Z0-9]+)/'
 
+    _TEST = {
+        'url': 'http://www.dump.com/oneus/',
+        'md5': 'ad71704d1e67dfd9e81e3e8b42d69d99',
+        'info_dict': {
+            'id': 'oneus',
+            'ext': 'flv',
+            'title': "He's one of us.",
+            'thumbnail': 're:^https?://.*\.jpg$',
+        },
+    }
+
     def _real_extract(self, url):
         m = re.match(self._VALID_URL, url)
         video_id = m.group('id')
 
-        # Note: There is an easier-to-parse configuration at
-        # http://www.aparat.com/video/video/config/videohash/%video_id
-        # but the URL in there does not work
-
         webpage = self._download_webpage(url, video_id)
+        video_url = self._search_regex(
+            r's1.addVariable\("file",\s*"([^"]+)"', webpage, 'video URL')
 
-        try:
-            video_url = re.findall(r'file","(.+?.flv)"', webpage)[-1]
-        except IndexError:
-            raise ExtractorError(u'No video URL found')
-
-        thumb = re.findall('<meta property="og:image" content="(.+?)"',webpage)[0]
-
-        title = self._search_regex(r'<b>([^"]+)</b>', webpage, u'title')
+        title = self._og_search_title(webpage)
+        thumbnail = self._og_search_thumbnail(webpage)
 
         return {
             'id': video_id,
             'title': title,
             'url': video_url,
-            'ext': 'flv',
-            'thumbnail': thumb,
+            'thumbnail': thumbnail,
         }