From: Ricardo Garcia <sarbalap+freshmeat@gmail.com>
Date: Mon, 21 Jul 2008 21:29:06 +0000 (+0200)
Subject: Check the output name is not fixed when there are several videos to download
X-Git-Url: http://git.oshgnacknak.de/?a=commitdiff_plain;h=b46347267a8c460561d1bc3589c7eab55b6a4655;p=youtube-dl

Check the output name is not fixed when there are several videos to download
---

diff --git a/youtube-dl b/youtube-dl
index 45773fda2..dfbdc4a47 100755
--- a/youtube-dl
+++ b/youtube-dl
@@ -38,7 +38,7 @@ class FileDownloader(object):
 	For this, file downloader objects have a method that allows
 	InfoExtractors to be registered in a given order. When it is passed
 	a URL, the file downloader handles it to the first InfoExtractor it
-	finds that reports it's able to handle it. The InfoExtractor returns
+	finds that reports being able to handle it. The InfoExtractor returns
 	all the information to the FileDownloader and the latter downloads the
 	file or does whatever it's instructed to do.
 
@@ -153,9 +153,12 @@ class FileDownloader(object):
 					continue
 				# Suitable InfoExtractor found
 				suitable_found = True
-				for result in ie.extract(url):
-					if result is None:
-						continue
+				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:
+					sys.exit('ERROR: fixed output name but more than one file to download')
+
+				for result in results:
 					try:
 						filename = self._params['outtmpl'] % result
 					except (KeyError), err: