debian/control: Mark compliance with Debian policy 4.1.5.

[youtubedl] / youtube_dl / extractor / cbs.py

diff --git a/youtube_dl/extractor/cbs.py b/youtube_dl/extractor/cbs.py
index 58f258c54b059b09888cf0e26a4718a69c704faa..1799d63ea86ad9f9b1d7048792b52385928e20f4 100644 (file)
--- a/youtube_dl/extractor/cbs.py
+++ b/youtube_dl/extractor/cbs.py
@@ -2,6 +2,7 @@ from __future__ import unicode_literals
 
 from .theplatform import ThePlatformFeedIE
 from ..utils import (
 
 from .theplatform import ThePlatformFeedIE
 from ..utils import (

+    ExtractorError,

     int_or_none,
     find_xpath_attr,
     xpath_element,
     int_or_none,
     find_xpath_attr,
     xpath_element,


@@ -49,21 +50,22 @@ class CBSIE(CBSBaseIE):
         'only_matching': True,
     }]
 
         'only_matching': True,
     }]
 

-    def _extract_video_info(self, content_id):
+    def _extract_video_info(self, content_id, site='cbs', mpx_acc=2198311517):

         items_data = self._download_xml(
             'http://can.cbs.com/thunder/player/videoPlayerService.php',
         items_data = self._download_xml(
             'http://can.cbs.com/thunder/player/videoPlayerService.php',

-            content_id, query={'partner': 'cbs', 'contentId': content_id})
+            content_id, query={'partner': site, 'contentId': content_id})

         video_data = xpath_element(items_data, './/item')
         title = xpath_text(video_data, 'videoTitle', 'title', True)
         video_data = xpath_element(items_data, './/item')
         title = xpath_text(video_data, 'videoTitle', 'title', True)

-        tp_path = 'dJ5BDC/media/guid/2198311517/%s' % content_id
+        tp_path = 'dJ5BDC/media/guid/%d/%s' % (mpx_acc, content_id)

         tp_release_url = 'http://link.theplatform.com/s/' + tp_path
 
         asset_types = []
         subtitles = {}
         formats = []
         tp_release_url = 'http://link.theplatform.com/s/' + tp_path
 
         asset_types = []
         subtitles = {}
         formats = []

+        last_e = None

         for item in items_data.findall('.//item'):
             asset_type = xpath_text(item, 'assetType')
         for item in items_data.findall('.//item'):
             asset_type = xpath_text(item, 'assetType')

-            if not asset_type or asset_type in asset_types:
+            if not asset_type or asset_type in asset_types or asset_type in ('HLS_FPS', 'DASH_CENC'):

                 continue
             asset_types.append(asset_type)
             query = {
                 continue
             asset_types.append(asset_type)
             query = {


@@ -74,11 +76,17 @@ class CBSIE(CBSBaseIE):
                 query['formats'] = 'MPEG4,M3U'
             elif asset_type in ('RTMP', 'WIFI', '3G'):
                 query['formats'] = 'MPEG4,FLV'
                 query['formats'] = 'MPEG4,M3U'
             elif asset_type in ('RTMP', 'WIFI', '3G'):
                 query['formats'] = 'MPEG4,FLV'

-            tp_formats, tp_subtitles = self._extract_theplatform_smil(
-                update_url_query(tp_release_url, query), content_id,
-                'Downloading %s SMIL data' % asset_type)
+            try:
+                tp_formats, tp_subtitles = self._extract_theplatform_smil(
+                    update_url_query(tp_release_url, query), content_id,
+                    'Downloading %s SMIL data' % asset_type)
+            except ExtractorError as e:
+                last_e = e
+                continue

             formats.extend(tp_formats)
             subtitles = self._merge_subtitles(subtitles, tp_subtitles)
             formats.extend(tp_formats)
             subtitles = self._merge_subtitles(subtitles, tp_subtitles)

+        if last_e and not formats:
+            raise last_e

         self._sort_formats(formats)
 
         info = self._extract_theplatform_metadata(tp_path, content_id)
         self._sort_formats(formats)
 
         info = self._extract_theplatform_metadata(tp_path, content_id)