From 1136e5c833694cdf5e47fee4fd497a9da7ac93a6 Mon Sep 17 00:00:00 2001 From: echel0n Date: Sat, 9 Aug 2014 01:43:02 +0000 Subject: [PATCH] Fixed IndexError: list index out of range issue --- lib/subliminal/services/tvsubtitles.py | 288 +++++++++++++------------ 1 file changed, 145 insertions(+), 143 deletions(-) diff --git a/lib/subliminal/services/tvsubtitles.py b/lib/subliminal/services/tvsubtitles.py index f6b2fd52..5d056aaa 100644 --- a/lib/subliminal/services/tvsubtitles.py +++ b/lib/subliminal/services/tvsubtitles.py @@ -1,143 +1,145 @@ -# -*- coding: utf-8 -*- -# Copyright 2012 Nicolas Wack -# -# This file is part of subliminal. -# -# subliminal is free software; you can redistribute it and/or modify it under -# the terms of the GNU Lesser General Public License as published by -# the Free Software Foundation; either version 3 of the License, or -# (at your option) any later version. -# -# subliminal is distributed in the hope that it will be useful, -# but WITHOUT ANY WARRANTY; without even the implied warranty of -# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the -# GNU Lesser General Public License for more details. -# -# You should have received a copy of the GNU Lesser General Public License -# along with subliminal. If not, see . -from . import ServiceBase -from ..cache import cachedmethod -from ..language import language_set, Language -from ..subtitles import get_subtitle_path, ResultSubtitle -from ..utils import get_keywords -from ..videos import Episode -from bs4 import BeautifulSoup -import logging -import re - - -logger = logging.getLogger("subliminal") - - -def match(pattern, string): - try: - return re.search(pattern, string).group(1) - except AttributeError: - logger.debug(u'Could not match %r on %r' % (pattern, string)) - return None - - -class TvSubtitles(ServiceBase): - server_url = 'http://www.tvsubtitles.net' - site_url = 'http://www.tvsubtitles.net' - api_based = False - languages = language_set(['ar', 'bg', 'cs', 'da', 'de', 'el', 'en', 'es', 'fi', 'fr', 'hu', - 'it', 'ja', 'ko', 'nl', 'pl', 'pt', 'ro', 'ru', 'sv', 'tr', 'uk', - 'zh', 'pb']) - #TODO: Find more exceptions - language_map = {'gr': Language('gre'), 'cz': Language('cze'), 'ua': Language('ukr'), - 'cn': Language('chi'), 'br': Language('pob')} - videos = [Episode] - require_video = False - required_features = ['permissive'] - - @cachedmethod - def get_likely_series_id(self, name): - r = self.session.post('%s/search.php' % self.server_url, data={'q': name}) - soup = BeautifulSoup(r.content, self.required_features) - maindiv = soup.find('div', 'left') - results = [] - for elem in maindiv.find_all('li'): - sid = int(match('tvshow-([0-9]+)\.html', elem.a['href'])) - show_name = match('(.*) \(', elem.a.text) - results.append((show_name, sid)) - #TODO: pick up the best one in a smart way - result = results[0] - return result[1] - - @cachedmethod - def get_episode_id(self, series_id, season, number): - """Get the TvSubtitles id for the given episode. Raises KeyError if none - could be found.""" - # download the page of the season, contains ids for all episodes - episode_id = None - r = self.session.get('%s/tvshow-%d-%d.html' % (self.server_url, series_id, season)) - soup = BeautifulSoup(r.content, self.required_features) - table = soup.find('table', id='table5') - for row in table.find_all('tr'): - cells = row.find_all('td') - if not cells: - continue - episode_number = match('x([0-9]+)', cells[0].text) - if not episode_number: - continue - episode_number = int(episode_number) - episode_id = int(match('episode-([0-9]+)', cells[1].a['href'])) - # we could just return the id of the queried episode, but as we - # already downloaded the whole page we might as well fill in the - # information for all the episodes of the season - self.cache_for(self.get_episode_id, args=(series_id, season, episode_number), result=episode_id) - # raises KeyError if not found - return self.cached_value(self.get_episode_id, args=(series_id, season, number)) - - # Do not cache this method in order to always check for the most recent - # subtitles - def get_sub_ids(self, episode_id): - subids = [] - r = self.session.get('%s/episode-%d.html' % (self.server_url, episode_id)) - epsoup = BeautifulSoup(r.content, self.required_features) - for subdiv in epsoup.find_all('a'): - if 'href' not in subdiv.attrs or not subdiv['href'].startswith('/subtitle'): - continue - subid = int(match('([0-9]+)', subdiv['href'])) - lang = self.get_language(match('flags/(.*).gif', subdiv.img['src'])) - result = {'subid': subid, 'language': lang} - for p in subdiv.find_all('p'): - if 'alt' in p.attrs and p['alt'] == 'rip': - result['rip'] = p.text.strip() - if 'alt' in p.attrs and p['alt'] == 'release': - result['release'] = p.text.strip() - subids.append(result) - return subids - - def list_checked(self, video, languages): - return self.query(video.path or video.release, languages, get_keywords(video.guess), video.series, video.season, video.episode) - - def query(self, filepath, languages, keywords, series, season, episode): - logger.debug(u'Getting subtitles for %s season %d episode %d with languages %r' % (series, season, episode, languages)) - self.init_cache() - sid = self.get_likely_series_id(series.lower()) - try: - ep_id = self.get_episode_id(sid, season, episode) - except KeyError: - logger.debug(u'Could not find episode id for %s season %d episode %d' % (series, season, episode)) - return [] - subids = self.get_sub_ids(ep_id) - # filter the subtitles with our queried languages - subtitles = [] - for subid in subids: - language = subid['language'] - if language not in languages: - continue - path = get_subtitle_path(filepath, language, self.config.multi) - subtitle = ResultSubtitle(path, language, self.__class__.__name__.lower(), '%s/download-%d.html' % (self.server_url, subid['subid']), - keywords=[subid['rip'], subid['release']]) - subtitles.append(subtitle) - return subtitles - - def download(self, subtitle): - self.download_zip_file(subtitle.link, subtitle.path) - return subtitle - - -Service = TvSubtitles +# -*- coding: utf-8 -*- +# Copyright 2012 Nicolas Wack +# +# This file is part of subliminal. +# +# subliminal is free software; you can redistribute it and/or modify it under +# the terms of the GNU Lesser General Public License as published by +# the Free Software Foundation; either version 3 of the License, or +# (at your option) any later version. +# +# subliminal is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU Lesser General Public License for more details. +# +# You should have received a copy of the GNU Lesser General Public License +# along with subliminal. If not, see . +from . import ServiceBase +from ..cache import cachedmethod +from ..language import language_set, Language +from ..subtitles import get_subtitle_path, ResultSubtitle +from ..utils import get_keywords +from ..videos import Episode +from bs4 import BeautifulSoup +import logging +import re + + +logger = logging.getLogger("subliminal") + + +def match(pattern, string): + try: + return re.search(pattern, string).group(1) + except AttributeError: + logger.debug(u'Could not match %r on %r' % (pattern, string)) + return None + + +class TvSubtitles(ServiceBase): + server_url = 'http://www.tvsubtitles.net' + site_url = 'http://www.tvsubtitles.net' + api_based = False + languages = language_set(['ar', 'bg', 'cs', 'da', 'de', 'el', 'en', 'es', 'fi', 'fr', 'hu', + 'it', 'ja', 'ko', 'nl', 'pl', 'pt', 'ro', 'ru', 'sv', 'tr', 'uk', + 'zh', 'pb']) + #TODO: Find more exceptions + language_map = {'gr': Language('gre'), 'cz': Language('cze'), 'ua': Language('ukr'), + 'cn': Language('chi'), 'br': Language('pob')} + videos = [Episode] + require_video = False + required_features = ['permissive'] + + @cachedmethod + def get_likely_series_id(self, name): + r = self.session.post('%s/search.php' % self.server_url, data={'q': name}) + soup = BeautifulSoup(r.content, self.required_features) + maindiv = soup.find('div', 'left') + results = [] + for elem in maindiv.find_all('li'): + sid = int(match('tvshow-([0-9]+)\.html', elem.a['href'])) + show_name = match('(.*) \(', elem.a.text) + results.append((show_name, sid)) + + if len(results): + #TODO: pick up the best one in a smart way + result = results[0] + return result[1] + + @cachedmethod + def get_episode_id(self, series_id, season, number): + """Get the TvSubtitles id for the given episode. Raises KeyError if none + could be found.""" + # download the page of the season, contains ids for all episodes + episode_id = None + r = self.session.get('%s/tvshow-%d-%d.html' % (self.server_url, series_id, season)) + soup = BeautifulSoup(r.content, self.required_features) + table = soup.find('table', id='table5') + for row in table.find_all('tr'): + cells = row.find_all('td') + if not cells: + continue + episode_number = match('x([0-9]+)', cells[0].text) + if not episode_number: + continue + episode_number = int(episode_number) + episode_id = int(match('episode-([0-9]+)', cells[1].a['href'])) + # we could just return the id of the queried episode, but as we + # already downloaded the whole page we might as well fill in the + # information for all the episodes of the season + self.cache_for(self.get_episode_id, args=(series_id, season, episode_number), result=episode_id) + # raises KeyError if not found + return self.cached_value(self.get_episode_id, args=(series_id, season, number)) + + # Do not cache this method in order to always check for the most recent + # subtitles + def get_sub_ids(self, episode_id): + subids = [] + r = self.session.get('%s/episode-%d.html' % (self.server_url, episode_id)) + epsoup = BeautifulSoup(r.content, self.required_features) + for subdiv in epsoup.find_all('a'): + if 'href' not in subdiv.attrs or not subdiv['href'].startswith('/subtitle'): + continue + subid = int(match('([0-9]+)', subdiv['href'])) + lang = self.get_language(match('flags/(.*).gif', subdiv.img['src'])) + result = {'subid': subid, 'language': lang} + for p in subdiv.find_all('p'): + if 'alt' in p.attrs and p['alt'] == 'rip': + result['rip'] = p.text.strip() + if 'alt' in p.attrs and p['alt'] == 'release': + result['release'] = p.text.strip() + subids.append(result) + return subids + + def list_checked(self, video, languages): + return self.query(video.path or video.release, languages, get_keywords(video.guess), video.series, video.season, video.episode) + + def query(self, filepath, languages, keywords, series, season, episode): + logger.debug(u'Getting subtitles for %s season %d episode %d with languages %r' % (series, season, episode, languages)) + self.init_cache() + sid = self.get_likely_series_id(series.lower()) + try: + ep_id = self.get_episode_id(sid, season, episode) + except KeyError: + logger.debug(u'Could not find episode id for %s season %d episode %d' % (series, season, episode)) + return [] + subids = self.get_sub_ids(ep_id) + # filter the subtitles with our queried languages + subtitles = [] + for subid in subids: + language = subid['language'] + if language not in languages: + continue + path = get_subtitle_path(filepath, language, self.config.multi) + subtitle = ResultSubtitle(path, language, self.__class__.__name__.lower(), '%s/download-%d.html' % (self.server_url, subid['subid']), + keywords=[subid['rip'], subid['release']]) + subtitles.append(subtitle) + return subtitles + + def download(self, subtitle): + self.download_zip_file(subtitle.link, subtitle.path) + return subtitle + + +Service = TvSubtitles \ No newline at end of file