Improve fixed template checks

This commit is contained in:
Ricardo Garcia 2008-07-22 09:45:49 +02:00 committed by Ricardo Garcia
parent 14c300687e
commit 22899cea59

View file

@ -153,8 +153,15 @@ class FileDownloader(object):
"""Print message to stderr.""" """Print message to stderr."""
sys.stderr.write('%s\n' % message) sys.stderr.write('%s\n' % message)
def fixed_template(self):
"""Checks if the output template is fixed."""
return (re.search(ur'%\(.+?\)s', self._params['outtmpl']) is None)
def download(self, url_list): def download(self, url_list):
"""Download a given list of URLs.""" """Download a given list of URLs."""
if len(url_list) > 1 and self.fixed_template():
sys.exit('ERROR: fixed output name but more than one file to download')
for url in url_list: for url in url_list:
suitable_found = False suitable_found = False
for ie in self._ies: for ie in self._ies:
@ -164,7 +171,7 @@ class FileDownloader(object):
suitable_found = True suitable_found = True
results = [x for x in ie.extract(url) if x is not None] results = [x for x in ie.extract(url) if x is not None]
if (len(url_list) > 1 or len(results) > 1) and re.search(r'%\(.+?\)s', self._params['outtmpl']) is None: if len(results) > 1 and self.fixed_template():
sys.exit('ERROR: fixed output name but more than one file to download') sys.exit('ERROR: fixed output name but more than one file to download')
if self._params.get('simulate', False): if self._params.get('simulate', False):