Initiate release.

[youtubedl] / youtube_dl / extractor / justintv.py

diff --git a/youtube_dl/extractor/justintv.py b/youtube_dl/extractor/justintv.py
index b2006e33499a31096d516b88e01c9f81f14392a6..27017e89f632880c21643c0b58f04d23e963fd39 100644 (file)
--- a/youtube_dl/extractor/justintv.py
+++ b/youtube_dl/extractor/justintv.py
@@ -1,10 +1,13 @@
+from __future__ import unicode_literals
+
+import itertools

 import json
 import os
 import re
 import json
 import os
 import re

-import xml.etree.ElementTree

 
 from .common import InfoExtractor
 from ..utils import (
 
 from .common import InfoExtractor
 from ..utils import (

+    compat_str,

     ExtractorError,
     formatSeconds,
 )
     ExtractorError,
     formatSeconds,
 )


@@ -25,23 +28,32 @@ class JustinTVIE(InfoExtractor):
         /?(?:\#.*)?$
         """
     _JUSTIN_PAGE_LIMIT = 100
         /?(?:\#.*)?$
         """
     _JUSTIN_PAGE_LIMIT = 100

-    IE_NAME = u'justin.tv'
-
-    def report_download_page(self, channel, offset):
-        """Report attempt to download a single page of videos."""
-        self.to_screen(u'%s: Downloading video information from %d to %d' %
-                (channel, offset, offset + self._JUSTIN_PAGE_LIMIT))
+    IE_NAME = 'justin.tv'
+    IE_DESC = 'justin.tv and twitch.tv'
+    _TEST = {
+        'url': 'http://www.twitch.tv/thegamedevhub/b/296128360',
+        'md5': 'ecaa8a790c22a40770901460af191c9a',
+        'info_dict': {
+            'id': '296128360',
+            'ext': 'flv',
+            'upload_date': '20110927',
+            'uploader_id': 25114803,
+            'uploader': 'thegamedevhub',
+            'title': 'Beginner Series - Scripting With Python Pt.1'
+        }
+    }

 
     # Return count of items, list of *valid* items
 
     # Return count of items, list of *valid* items

-    def _parse_page(self, url, video_id):
-        info_json = self._download_webpage(url, video_id,
-                                           u'Downloading video info JSON',
-                                           u'unable to download video info JSON')
+    def _parse_page(self, url, video_id, counter):
+        info_json = self._download_webpage(
+            url, video_id,
+            'Downloading video info JSON on page %d' % counter,
+            'Unable to download video info JSON %d' % counter)

 
         response = json.loads(info_json)
         if type(response) != list:
             error_text = response.get('error', 'unknown error')
 
         response = json.loads(info_json)
         if type(response) != list:
             error_text = response.get('error', 'unknown error')

-            raise ExtractorError(u'Justin.tv API: %s' % error_text)
+            raise ExtractorError('Justin.tv API: %s' % error_text)

         info = []
         for clip in response:
             video_url = clip['video_file_url']
         info = []
         for clip in response:
             video_url = clip['video_file_url']


@@ -52,7 +64,7 @@ class JustinTVIE(InfoExtractor):
                 video_id = clip['id']
                 video_title = clip.get('title', video_id)
                 info.append({
                 video_id = clip['id']
                 video_title = clip.get('title', video_id)
                 info.append({

-                    'id': video_id,
+                    'id': compat_str(video_id),

                     'url': video_url,
                     'title': video_title,
                     'uploader': clip.get('channel_name', video_uploader_id),
                     'url': video_url,
                     'title': video_title,
                     'uploader': clip.get('channel_name', video_uploader_id),


@@ -64,8 +76,6 @@ class JustinTVIE(InfoExtractor):
 
     def _real_extract(self, url):
         mobj = re.match(self._VALID_URL, url)
 
     def _real_extract(self, url):
         mobj = re.match(self._VALID_URL, url)

-        if mobj is None:
-            raise ExtractorError(u'invalid URL: %s' % url)

 
         api_base = 'http://api.justin.tv'
         paged = False
 
         api_base = 'http://api.justin.tv'
         paged = False


@@ -79,41 +89,41 @@ class JustinTVIE(InfoExtractor):
             webpage = self._download_webpage(url, chapter_id)
             m = re.search(r'PP\.archive_id = "([0-9]+)";', webpage)
             if not m:
             webpage = self._download_webpage(url, chapter_id)
             m = re.search(r'PP\.archive_id = "([0-9]+)";', webpage)
             if not m:

-                raise ExtractorError(u'Cannot find archive of a chapter')
+                raise ExtractorError('Cannot find archive of a chapter')

             archive_id = m.group(1)
 
             api = api_base + '/broadcast/by_chapter/%s.xml' % chapter_id
             archive_id = m.group(1)
 
             api = api_base + '/broadcast/by_chapter/%s.xml' % chapter_id

-            chapter_info_xml = self._download_webpage(api, chapter_id,
-                                             note=u'Downloading chapter information',
-                                             errnote=u'Chapter information download failed')
-            doc = xml.etree.ElementTree.fromstring(chapter_info_xml)
+            doc = self._download_xml(
+                api, chapter_id,
+                note='Downloading chapter information',
+                errnote='Chapter information download failed')

             for a in doc.findall('.//archive'):
                 if archive_id == a.find('./id').text:
                     break
             else:
             for a in doc.findall('.//archive'):
                 if archive_id == a.find('./id').text:
                     break
             else:

-                raise ExtractorError(u'Could not find chapter in chapter information')
+                raise ExtractorError('Could not find chapter in chapter information')

 
             video_url = a.find('./video_file_url').text
 
             video_url = a.find('./video_file_url').text

-            video_ext = video_url.rpartition('.')[2] or u'flv'
+            video_ext = video_url.rpartition('.')[2] or 'flv'

 
 

-            chapter_api_url = u'https://api.twitch.tv/kraken/videos/c' + chapter_id
-            chapter_info_json = self._download_webpage(chapter_api_url, u'c' + chapter_id,
-                                   note='Downloading chapter metadata',
-                                   errnote='Download of chapter metadata failed')
-            chapter_info = json.loads(chapter_info_json)
+            chapter_api_url = 'https://api.twitch.tv/kraken/videos/c' + chapter_id
+            chapter_info = self._download_json(
+                chapter_api_url, 'c' + chapter_id,
+                note='Downloading chapter metadata',
+                errnote='Download of chapter metadata failed')

 
             bracket_start = int(doc.find('.//bracket_start').text)
             bracket_end = int(doc.find('.//bracket_end').text)
 
             # TODO determine start (and probably fix up file)
             #  youtube-dl -v http://www.twitch.tv/firmbelief/c/1757457
 
             bracket_start = int(doc.find('.//bracket_start').text)
             bracket_end = int(doc.find('.//bracket_end').text)
 
             # TODO determine start (and probably fix up file)
             #  youtube-dl -v http://www.twitch.tv/firmbelief/c/1757457

-            #video_url += u'?start=' + TODO:start_timestamp
+            #video_url += '?start=' + TODO:start_timestamp

             # bracket_start is 13290, but we want 51670615
             # bracket_start is 13290, but we want 51670615

-            self._downloader.report_warning(u'Chapter detected, but we can just download the whole file. '
-                                            u'Chapter starts at %s and ends at %s' % (formatSeconds(bracket_start), formatSeconds(bracket_end)))
+            self._downloader.report_warning('Chapter detected, but we can just download the whole file. '
+                                            'Chapter starts at %s and ends at %s' % (formatSeconds(bracket_start), formatSeconds(bracket_end)))

 
             info = {
 
             info = {

-                'id': u'c' + chapter_id,
+                'id': 'c' + chapter_id,

                 'url': video_url,
                 'ext': video_ext,
                 'title': chapter_info['title'],
                 'url': video_url,
                 'ext': video_ext,
                 'title': chapter_info['title'],


@@ -122,23 +132,24 @@ class JustinTVIE(InfoExtractor):
                 'uploader': chapter_info['channel']['display_name'],
                 'uploader_id': chapter_info['channel']['name'],
             }
                 'uploader': chapter_info['channel']['display_name'],
                 'uploader_id': chapter_info['channel']['name'],
             }

-            return [info]
+            return info

         else:
             video_id = mobj.group('videoid')
             api = api_base + '/broadcast/by_archive/%s.json' % video_id
 
         else:
             video_id = mobj.group('videoid')
             api = api_base + '/broadcast/by_archive/%s.json' % video_id
 

-        self.report_extraction(video_id)
-
-        info = []
+        entries = []

         offset = 0
         limit = self._JUSTIN_PAGE_LIMIT
         offset = 0
         limit = self._JUSTIN_PAGE_LIMIT

-        while True:
-            if paged:
-                self.report_download_page(video_id, offset)
+        for counter in itertools.count(1):

             page_url = api + ('?offset=%d&limit=%d' % (offset, limit))
             page_url = api + ('?offset=%d&limit=%d' % (offset, limit))

-            page_count, page_info = self._parse_page(page_url, video_id)
-            info.extend(page_info)
+            page_count, page_info = self._parse_page(
+                page_url, video_id, counter)
+            entries.extend(page_info)

             if not paged or page_count != limit:
                 break
             offset += limit
             if not paged or page_count != limit:
                 break
             offset += limit

-        return info
+        return {
+            '_type': 'playlist',
+            'id': video_id,
+            'entries': entries,
+        }