Use unescapeHTML for OpenGraph properties
authorPhilipp Hagemeister <phihag@phihag.de>
Wed, 17 Jul 2013 08:38:23 +0000 (10:38 +0200)
committerPhilipp Hagemeister <phihag@phihag.de>
Wed, 17 Jul 2013 08:38:23 +0000 (10:38 +0200)
These are attribute values, so we don't need the more complex and whitespace-destroying cleanHTML - we just need to unescape quotes, that's it.

youtube_dl/extractor/common.py

index ec988fc900c449d582c7fecc40649a21505ad9bd..da50abfc1cd492b8d360ef601b44841a938c055b 100644 (file)
@@ -14,6 +14,7 @@ from ..utils import (
     clean_html,
     compiled_regex_type,
     ExtractorError,
+    unescapeHTML,
 )
 
 class InfoExtractor(object):
@@ -270,7 +271,8 @@ class InfoExtractor(object):
     def _og_search_property(self, prop, html, name=None, **kargs):
         if name is None:
             name = 'OpenGraph %s' % prop
-        return self._html_search_regex(self._og_regex(prop), html, name, flags=re.DOTALL, **kargs)
+        escaped = self._search_regex(self._og_regex(prop), html, name, flags=re.DOTALL, **kargs)
+        return unescapeHTML(escaped)
 
     def _og_search_thumbnail(self, html, **kargs):
         return self._og_search_property('image', html, u'thumbnail url', fatal=False, **kargs)