Browse Source

[viki] Do not use a fallback language for title in the first try

In test_Viki_3, 'titles' gives a Hebrew title.
Yen Chi Hsuan 9 years ago
parent
commit
6d28c408cf
1 changed files with 5 additions and 2 deletions
  1. 5 2
      youtube_dl/extractor/viki.py

+ 5 - 2
youtube_dl/extractor/viki.py

@@ -101,10 +101,13 @@ class VikiBaseIE(InfoExtractor):
             self.report_warning('Unable to get session token, login has probably failed')
             self.report_warning('Unable to get session token, login has probably failed')
 
 
     @staticmethod
     @staticmethod
-    def dict_selection(dict_obj, preferred_key):
+    def dict_selection(dict_obj, preferred_key, allow_fallback=True):
         if preferred_key in dict_obj:
         if preferred_key in dict_obj:
             return dict_obj.get(preferred_key)
             return dict_obj.get(preferred_key)
 
 
+        if not allow_fallback:
+            return
+
         filtered_dict = list(filter(None, [dict_obj.get(k) for k in dict_obj.keys()]))
         filtered_dict = list(filter(None, [dict_obj.get(k) for k in dict_obj.keys()]))
         return filtered_dict[0] if filtered_dict else None
         return filtered_dict[0] if filtered_dict else None
 
 
@@ -218,7 +221,7 @@ class VikiIE(VikiBaseIE):
 
 
         self._check_errors(video)
         self._check_errors(video)
 
 
-        title = self.dict_selection(video.get('titles', {}), 'en')
+        title = self.dict_selection(video.get('titles', {}), 'en', allow_fallback=False)
         if not title:
         if not title:
             title = 'Episode %d' % video.get('number') if video.get('type') == 'episode' else video.get('id') or video_id
             title = 'Episode %d' % video.get('number') if video.get('type') == 'episode' else video.get('id') or video_id
             container_titles = video.get('container', {}).get('titles', {})
             container_titles = video.get('container', {}).get('titles', {})