From: Jaime Marquínez Ferrándiz Date: Fri, 12 Jul 2013 12:53:28 +0000 (+0200) Subject: BrightcoveIE: Use parse_qs to extract the fields of the query (closes #1032) X-Git-Url: http://git.oshgnacknak.de/?a=commitdiff_plain;h=6543f0dca5e04530450fac5a64efa5d9ee294b1b;p=youtube-dl BrightcoveIE: Use parse_qs to extract the fields of the query (closes #1032) Add a compat_urlparse to utils. --- diff --git a/youtube_dl/extractor/brightcove.py b/youtube_dl/extractor/brightcove.py index 450b4a406..71e3c7883 100644 --- a/youtube_dl/extractor/brightcove.py +++ b/youtube_dl/extractor/brightcove.py @@ -6,6 +6,7 @@ from .common import InfoExtractor from ..utils import ( compat_urllib_parse, find_xpath_attr, + compat_urlparse, ) class BrightcoveIE(InfoExtractor): @@ -39,15 +40,15 @@ class BrightcoveIE(InfoExtractor): def _real_extract(self, url): mobj = re.match(self._VALID_URL, url) - query = mobj.group('query') + query_str = mobj.group('query') + query = compat_urlparse.parse_qs(query_str) - m_video_id = re.search(r'videoPlayer=(\d+)', query) - if m_video_id is not None: - video_id = m_video_id.group(1) - return self._get_video_info(video_id, query) + videoPlayer = query.get('@videoPlayer') + if videoPlayer: + return self._get_video_info(videoPlayer[0], query_str) else: - player_key = self._search_regex(r'playerKey=(.+?)(&|$)', query, 'playlist_id') - return self._get_playlist_info(player_key) + player_key = query['playerKey'] + return self._get_playlist_info(player_key[0]) def _get_video_info(self, video_id, query): request_url = self._FEDERATED_URL_TEMPLATE % query diff --git a/youtube_dl/utils.py b/youtube_dl/utils.py index 31db1bf8c..532ca585d 100644 --- a/youtube_dl/utils.py +++ b/youtube_dl/utils.py @@ -35,6 +35,11 @@ try: except ImportError: # Python 2 from urlparse import urlparse as compat_urllib_parse_urlparse +try: + import urllib.parse as compat_urlparse +except ImportError: # Python 2 + import urlparse as compat_urlparse + try: import http.cookiejar as compat_cookiejar except ImportError: # Python 2