Merge tag 'upstream/2017.03.26'

[youtubedl] / youtube_dl / postprocessor / metadatafromtitle.py

diff --git a/youtube_dl/postprocessor/metadatafromtitle.py b/youtube_dl/postprocessor/metadatafromtitle.py
index a56077f206b5133f2fae3fadbcad10523353ed5a..164edd3a820af4d0c3d1af48b9cf81a6b5460e9b 100644 (file)
--- a/youtube_dl/postprocessor/metadatafromtitle.py
+++ b/youtube_dl/postprocessor/metadatafromtitle.py
@@ -3,11 +3,6 @@ from __future__ import unicode_literals
 import re
 
 from .common import PostProcessor
 import re
 
 from .common import PostProcessor

-from ..utils import PostProcessingError
-
-
-class MetadataFromTitlePPError(PostProcessingError):
-    pass

 
 
 class MetadataFromTitlePP(PostProcessor):
 
 
 class MetadataFromTitlePP(PostProcessor):


@@ -17,14 +12,14 @@ class MetadataFromTitlePP(PostProcessor):
         self._titleregex = self.format_to_regex(titleformat)
 
     def format_to_regex(self, fmt):
         self._titleregex = self.format_to_regex(titleformat)
 
     def format_to_regex(self, fmt):

-        """
+        r"""

         Converts a string like
            '%(title)s - %(artist)s'
         to a regex like
            '(?P<title>.+)\ \-\ (?P<artist>.+)'
         """
         lastpos = 0
         Converts a string like
            '%(title)s - %(artist)s'
         to a regex like
            '(?P<title>.+)\ \-\ (?P<artist>.+)'
         """
         lastpos = 0

-        regex = ""
+        regex = ''

         # replace %(..)s with regex group and escape other string parts
         for match in re.finditer(r'%\((\w+)\)s', fmt):
             regex += re.escape(fmt[lastpos:match.start()])
         # replace %(..)s with regex group and escape other string parts
         for match in re.finditer(r'%\((\w+)\)s', fmt):
             regex += re.escape(fmt[lastpos:match.start()])


@@ -38,7 +33,8 @@ class MetadataFromTitlePP(PostProcessor):
         title = info['title']
         match = re.match(self._titleregex, title)
         if match is None:
         title = info['title']
         match = re.match(self._titleregex, title)
         if match is None:

-            raise MetadataFromTitlePPError('Could not interpret title of video as "%s"' % self._titleformat)
+            self._downloader.to_screen('[fromtitle] Could not interpret title of video as "%s"' % self._titleformat)
+            return [], info

         for attribute, value in match.groupdict().items():
             value = match.group(attribute)
             info[attribute] = value
         for attribute, value in match.groupdict().items():
             value = match.group(attribute)
             info[attribute] = value