From: Filippo Valsorda Date: Tue, 27 Nov 2012 16:57:12 +0000 (+0100) Subject: make all IEs return 'upload_date' and 'uploader', even if only u'NA' X-Git-Url: http://git.bitcoin.ninja/?a=commitdiff_plain;h=9ce5d9ee7513c433cbbb8949214ada078ffddea6;p=youtube-dl make all IEs return 'upload_date' and 'uploader', even if only u'NA' --- diff --git a/youtube_dl/InfoExtractors.py b/youtube_dl/InfoExtractors.py index 8ea03ea01..3adf56a0b 100644 --- a/youtube_dl/InfoExtractors.py +++ b/youtube_dl/InfoExtractors.py @@ -40,7 +40,7 @@ class InfoExtractor(object): id: Video identifier. url: Final video URL. - uploader: Nickname of the video uploader. + uploader: Nickname of the video uploader, unescaped. upload_date: Video upload date (YYYYMMDD). title: Video title, unescaped. ext: Video filename extension. @@ -52,6 +52,8 @@ class InfoExtractor(object): description: One-line video description. player_url: SWF Player URL (used for rtmpdump). + The fields should all be Unicode strings. + Subclasses of this one should re-define the _real_initialize() and _real_extract() methods and define a _VALID_URL regexp. Probably, they should also be added to the list of extractors. @@ -1018,7 +1020,6 @@ class YahooIE(InfoExtractor): 'ext': video_extension.decode('utf-8'), 'thumbnail': video_thumbnail.decode('utf-8'), 'description': video_description, - 'thumbnail': video_thumbnail, }] @@ -2129,6 +2130,8 @@ class BlipTVIE(InfoExtractor): info = { 'id': title, 'url': url, + 'uploader': u'NA', + 'upload_date': u'NA', 'title': title, 'ext': ext, 'urlhandle': urlh @@ -2486,7 +2489,7 @@ class EscapistIE(InfoExtractor): 'id': videoId, 'url': videoUrl, 'uploader': showName, - 'upload_date': None, + 'upload_date': u'NA', 'title': showName, 'ext': 'flv', 'thumbnail': imgUrl, @@ -2535,6 +2538,8 @@ class CollegeHumorIE(InfoExtractor): info = { 'id': video_id, 'internal_id': internal_video_id, + 'uploader': u'NA', + 'upload_date': u'NA', } self.report_extraction(video_id) @@ -2619,8 +2624,8 @@ class XVideosIE(InfoExtractor): info = { 'id': video_id, 'url': video_url, - 'uploader': None, - 'upload_date': None, + 'uploader': u'NA', + 'upload_date': u'NA', 'title': video_title, 'ext': 'flv', 'thumbnail': video_thumbnail, @@ -2700,7 +2705,7 @@ class SoundcloudIE(InfoExtractor): description = mobj.group(1) # upload date - upload_date = None + upload_date = u'NA' mobj = re.search("pretty-date'>on ([\w]+ [\d]+, [\d]+ \d+:\d+)", webpage) if mobj: try: @@ -2781,8 +2786,8 @@ class InfoQIE(InfoExtractor): info = { 'id': video_id, 'url': video_url, - 'uploader': None, - 'upload_date': None, + 'uploader': u'NA', + 'upload_date': u'NA', 'title': video_title, 'ext': extension, # Extension is always(?) mp4, but seems to be flv 'thumbnail': None, @@ -2929,6 +2934,8 @@ class StanfordOpenClassroomIE(InfoExtractor): video = mobj.group('video') info = { 'id': course + '_' + video, + 'uploader': u'NA', + 'upload_date': u'NA', } self.report_extraction(info['id']) @@ -2953,6 +2960,8 @@ class StanfordOpenClassroomIE(InfoExtractor): info = { 'id': course, 'type': 'playlist', + 'uploader': u'NA', + 'upload_date': u'NA', } self.report_download_webpage(info['id']) @@ -2989,6 +2998,8 @@ class StanfordOpenClassroomIE(InfoExtractor): info = { 'id': 'Stanford OpenClassroom', 'type': 'playlist', + 'uploader': u'NA', + 'upload_date': u'NA', } self.report_download_webpage(info['id']) @@ -3097,6 +3108,7 @@ class MTVIE(InfoExtractor): 'id': video_id, 'url': video_url, 'uploader': performer, + 'upload_date': u'NA', 'title': video_title, 'ext': ext, 'format': format, @@ -3217,7 +3229,8 @@ class YoukuIE(InfoExtractor): info = { 'id': '%s_part%02d' % (video_id, index), 'url': download_url, - 'uploader': None, + 'uploader': u'NA', + 'upload_date': u'NA', 'title': video_title, 'ext': ext, } @@ -3280,8 +3293,8 @@ class XNXXIE(InfoExtractor): return [{ 'id': video_id, 'url': video_url, - 'uploader': None, - 'upload_date': None, + 'uploader': u'NA', + 'upload_date': u'NA', 'title': video_title, 'ext': 'flv', 'thumbnail': video_thumbnail,