Merge tag 'upstream/2014.10.30'

[youtubedl] / youtube_dl / extractor / videofyme.py

diff --git a/youtube_dl/extractor/videofyme.py b/youtube_dl/extractor/videofyme.py
index 04106672b583443fb3785a60a283e555a7f9fdfc..f75169041b4f958b9f345daba99a4a1ba575cf4e 100644 (file)
--- a/youtube_dl/extractor/videofyme.py
+++ b/youtube_dl/extractor/videofyme.py
@@ -1,5 +1,4 @@
 import re
 import re

-import xml.etree.ElementTree

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


@@ -8,13 +7,13 @@ from ..utils import (
 )
 
 class VideofyMeIE(InfoExtractor):
 )
 
 class VideofyMeIE(InfoExtractor):

-    _VALID_URL = r'https?://(www.videofy.me/.+?|p.videofy.me/v)/(?P<id>\d+)(&|#|$)'
+    _VALID_URL = r'https?://(www\.videofy\.me/.+?|p\.videofy\.me/v)/(?P<id>\d+)(&|#|$)'

     IE_NAME = u'videofy.me'
 
     _TEST = {
         u'url': u'http://www.videofy.me/thisisvideofyme/1100701',
         u'file':  u'1100701.mp4',
     IE_NAME = u'videofy.me'
 
     _TEST = {
         u'url': u'http://www.videofy.me/thisisvideofyme/1100701',
         u'file':  u'1100701.mp4',

-        u'md5': u'2046dd5758541d630bfa93e741e2fd79',
+        u'md5': u'c77d700bdc16ae2e9f3c26019bd96143',

         u'info_dict': {
             u'title': u'This is VideofyMe',
             u'description': None,
         u'info_dict': {
             u'title': u'This is VideofyMe',
             u'description': None,


@@ -27,14 +26,12 @@ class VideofyMeIE(InfoExtractor):
     def _real_extract(self, url):
         mobj = re.match(self._VALID_URL, url)
         video_id = mobj.group('id')
     def _real_extract(self, url):
         mobj = re.match(self._VALID_URL, url)
         video_id = mobj.group('id')

-        config_xml = self._download_webpage('http://sunshine.videofy.me/?videoId=%s' % video_id,
+        config = self._download_xml('http://sunshine.videofy.me/?videoId=%s' % video_id,

                                             video_id)
                                             video_id)

-        config = xml.etree.ElementTree.fromstring(config_xml.encode('utf-8'))

         video = config.find('video')
         sources = video.find('sources')
         video = config.find('video')
         sources = video.find('sources')

-        url_node = find_xpath_attr(sources, 'source', 'id', 'HQ on')
-        if url_node is None:
-            url_node = find_xpath_attr(sources, 'source', 'id', 'HQ off')
+        url_node = next(node for node in [find_xpath_attr(sources, 'source', 'id', 'HQ %s' % key) 
+            for key in ['on', 'av', 'off']] if node is not None)

         video_url = url_node.find('url').text
 
         return {'id': video_id,
         video_url = url_node.find('url').text
 
         return {'id': video_id,