[bild] Simplify (#3983)

diff --git a/youtube_dl/extractor/bild.py b/youtube_dl/extractor/bild.py
index 3a822a5c0afd695766ffa21c168cecec2d1eca78..0269d1174fd5c289ec49b593637b123d8bd536bf 100644 (file)
--- a/youtube_dl/extractor/bild.py
+++ b/youtube_dl/extractor/bild.py
@@ -1,46 +1,39 @@
-from __future__ import unicode_literals\r
-\r
-import re\r
-\r
-from .common import InfoExtractor\r
-\r
-\r
-class BildIE(InfoExtractor):\r
-    IE_NAME = 'bild'\r
-    _TEST = {\r
-        'url': 'http://www.bild.de/video/clip/apple-ipad-air/das-koennen-die-neuen-ipads-38184146.bild.html',\r
-        'info_dict': {\r
-            'id': '38184146',\r
-            'title': 'BILD hat sie getestet',\r
-            'thumbnail': 'http://bilder.bild.de/fotos/stand-das-koennen-die-neuen-ipads-38184138/Bild/1.bild.jpg',\r
-            'duration': 196,\r
-        }\r
-    }\r
-    \r
-    #http://www.bild.de/video/clip/apple-ipad-air/das-koennen-die-neuen-ipads-38184146.bild.html\r
-    _VALID_URL = r'http?://(?:www\.)?bild\.de/(?:[^/]+/)+(?P<display_id>[^/]+)-(?P<id>\d+)(?:,auto=true)?\.bild\.html'\r
-    \r
-    def _real_extract(self, url):\r
-        m = re.match(self._VALID_URL, url)\r
-        video_id = m.group('id')\r
-        \r
-        #webpage = self._download_webpage(url, video_id)\r
-        \r
-        xml_url = url.split(".bild.html")[0]+",view=xml.bild.xml"\r
-        \r
-        doc = self._download_xml(xml_url, video_id)\r
-        \r
-        video_url = doc.attrib['src']\r
-        title = doc.attrib['ueberschrift']\r
-        description = doc.attrib['text']\r
-        thumbnail = doc.attrib['img']\r
-        duration = int(doc.attrib['duration'])/1000\r
-\r
-        return {\r
-            'id': video_id,\r
-            'title': title,\r
-            'description': description,\r
-            'url': video_url,\r
-            'thumbnail': thumbnail,\r
-            'duration': duration,\r
-        }\r
+#coding: utf-8
+from __future__ import unicode_literals
+
+from .common import InfoExtractor
+from ..utils import int_or_none
+
+
+class BildIE(InfoExtractor):
+    _VALID_URL = r'https?://(?:www\.)?bild\.de/(?:[^/]+/)+(?P<display_id>[^/]+)-(?P<id>\d+)(?:,auto=true)?\.bild\.html'
+    IE_DESC = 'Bild.de'
+    _TEST = {
+        'url': 'http://www.bild.de/video/clip/apple-ipad-air/das-koennen-die-neuen-ipads-38184146.bild.html',
+        'md5': 'dd495cbd99f2413502a1713a1156ac8a',
+        'info_dict': {
+            'id': '38184146',
+            'ext': 'mp4',
+            'title': 'BILD hat sie getestet',
+            'thumbnail': 'http://bilder.bild.de/fotos/stand-das-koennen-die-neuen-ipads-38184138/Bild/1.bild.jpg',
+            'duration': 196,
+            'description': 'Mit dem iPad Air 2 und dem iPad Mini 3 hat Apple zwei neue Tablet-Modelle präsentiert. BILD-Reporter Sven Stein durfte die Geräte bereits testen. ',
+        }
+    }
+
+    def _real_extract(self, url):
+        video_id = self._match_id(url)
+
+        xml_url = url.split(".bild.html")[0] + ",view=xml.bild.xml"
+        doc = self._download_xml(xml_url, video_id)
+
+        duration = int_or_none(doc.attrib.get('duration'), scale=1000)
+
+        return {
+            'id': video_id,
+            'title': doc.attrib['ueberschrift'],
+            'description': doc.attrib.get('text'),
+            'url': doc.attrib['src'],
+            'thumbnail': doc.attrib.get('img'),
+            'duration': duration,
+        }