Annotate changelog bug being closed.

[youtubedl] / youtube_dl / extractor / wistia.py

diff --git a/youtube_dl/extractor/wistia.py b/youtube_dl/extractor/wistia.py
index e1748c2613bbcf94bb36b706ce7ddfdde6b2b86c..13a079151c9c879561e3e538c49f3122f85b349b 100644 (file)
--- a/youtube_dl/extractor/wistia.py
+++ b/youtube_dl/extractor/wistia.py
@@ -1,30 +1,35 @@
-import json
-import re
+from __future__ import unicode_literals

 
 from .common import InfoExtractor
 
 from .common import InfoExtractor

+from ..compat import compat_urllib_request
+from ..utils import ExtractorError

 
 
 class WistiaIE(InfoExtractor):
 
 
 class WistiaIE(InfoExtractor):

-    _VALID_URL = r'^https?://(?:fast\.)?wistia\.net/embed/iframe/(?P<id>[a-z0-9]+)'
+    _VALID_URL = r'https?://(?:fast\.)?wistia\.net/embed/iframe/(?P<id>[a-z0-9]+)'
+    _API_URL = 'http://fast.wistia.com/embed/medias/{0:}.json'

 
     _TEST = {
 
     _TEST = {

-        u"url": u"http://fast.wistia.net/embed/iframe/sh7fpupwlt",
-        u"file": u"sh7fpupwlt.mov",
-        u"md5": u"cafeb56ec0c53c18c97405eecb3133df",
-        u"info_dict": {
-            u"title": u"cfh_resourceful_zdkh_final_1"
+        'url': 'http://fast.wistia.net/embed/iframe/sh7fpupwlt',
+        'md5': 'cafeb56ec0c53c18c97405eecb3133df',
+        'info_dict': {
+            'id': 'sh7fpupwlt',
+            'ext': 'mov',
+            'title': 'Being Resourceful',
+            'duration': 117,

         },
     }
 
     def _real_extract(self, url):
         },
     }
 
     def _real_extract(self, url):

-        mobj = re.match(self._VALID_URL, url)
-        video_id = mobj.group('id')
+        video_id = self._match_id(url)

 
 

-        webpage = self._download_webpage(url, video_id)
-        data_json = self._html_search_regex(
-            r'Wistia.iframeInit\((.*?), {}\);', webpage, u'video data')
-
-        data = json.loads(data_json)
+        request = compat_urllib_request.Request(self._API_URL.format(video_id))
+        request.add_header('Referer', url)  # Some videos require this.
+        data_json = self._download_json(request, video_id)
+        if data_json.get('error'):
+            raise ExtractorError('Error while getting the playlist',
+                                 expected=True)
+        data = data_json['media']

 
         formats = []
         thumbnails = []
 
         formats = []
         thumbnails = []


@@ -44,12 +49,15 @@ class WistiaIE(InfoExtractor):
                 'height': a['height'],
                 'filesize': a['size'],
                 'ext': a['ext'],
                 'height': a['height'],
                 'filesize': a['size'],
                 'ext': a['ext'],

+                'preference': 1 if atype == 'original' else None,

             })
             })

-        formats.sort(key=lambda a: a['filesize'])
+
+        self._sort_formats(formats)

 
         return {
             'id': video_id,
             'title': data['name'],
             'formats': formats,
             'thumbnails': thumbnails,
 
         return {
             'id': video_id,
             'title': data['name'],
             'formats': formats,
             'thumbnails': thumbnails,

+            'duration': data.get('duration'),

         }
         }