about summary refs log tree commit diff
path: root/youtube_dl/extractor/niconico.py
diff options
context:
space:
mode:
authorhakatashi <hakatasiloving@gmail.com>2014-07-02 03:13:12 +0900
committerhakatashi <hakatasiloving@gmail.com>2014-07-02 03:13:12 +0900
commit64ed7a38f98e9b01feae757bec25b81db80b29f6 (patch)
tree1fde03a43854fbe51eb434d9c0ebd7a6e48abd67 /youtube_dl/extractor/niconico.py
parentdc2fc736911b5b8d769becd9227976e5caf267dc (diff)
downloadyoutube-dl-64ed7a38f98e9b01feae757bec25b81db80b29f6.tar.gz
youtube-dl-64ed7a38f98e9b01feae757bec25b81db80b29f6.tar.xz
youtube-dl-64ed7a38f98e9b01feae757bec25b81db80b29f6.zip
[niconico] Add support for channel video
Diffstat (limited to 'youtube_dl/extractor/niconico.py')
-rw-r--r--youtube_dl/extractor/niconico.py18
1 files changed, 8 insertions, 10 deletions
diff --git a/youtube_dl/extractor/niconico.py b/youtube_dl/extractor/niconico.py
index 517a72561..43d8644a4 100644
--- a/youtube_dl/extractor/niconico.py
+++ b/youtube_dl/extractor/niconico.py
@@ -37,7 +37,7 @@ class NiconicoIE(InfoExtractor):
         },
     }
 
-    _VALID_URL = r'^https?://(?:www\.|secure\.)?nicovideo\.jp/watch/([a-z][a-z][0-9]+)(?:.*)$'
+    _VALID_URL = r'^https?://(?:www\.|secure\.)?nicovideo\.jp/watch/((?:[a-z][a-z])?[0-9]+)(?:.*)$'
     _NETRC_MACHINE = 'niconico'
 
     def _real_initialize(self):
@@ -91,20 +91,18 @@ class NiconicoIE(InfoExtractor):
         video_format = video_extension.upper()
         video_thumbnail = video_info.find('.//thumbnail_url').text
         video_description = video_info.find('.//description').text
-        video_uploader_id = video_info.find('.//user_id').text
         video_upload_date = unified_strdate(video_info.find('.//first_retrieve').text.split('+')[0])
         video_view_count = video_info.find('.//view_counter').text
         video_webpage_url = video_info.find('.//watch_url').text
 
         # uploader
-        video_uploader = video_uploader_id
-        url = 'http://seiga.nicovideo.jp/api/user/info?id=' + video_uploader_id
-        try:
-            user_info = self._download_xml(
-                url, video_id, note='Downloading user information')
-            video_uploader = user_info.find('.//nickname').text
-        except ExtractorError as err:
-            self._downloader.report_warning('Unable to download user info webpage: %s' % compat_str(err))
+        # No need to fetch extra resources...new API has field for uploader's name
+        if video_info.find('.//ch_id') is not None:
+            video_uploader_id = video_info.find('.//ch_id').text
+            video_uploader = video_info.find('.//ch_name').text
+        elif video_info.find('.//user_id') is not None:
+            video_uploader_id = video_info.find('.//user_id').text
+            video_uploader = video_info.find('.//user_nickname').text
 
         return {
             'id': video_id,