Added notes for handling Full Season releases, decisions to be made on how to handle them and how to correct multi-episode releases (as long as the first episode is needed we mark it as so).

Currently full season releases can be parsed, but will not process.
pull/4/head
Mark McDowall 14 years ago
parent 21e14fbb84
commit ee00ef8c17

@ -75,6 +75,8 @@ namespace NzbDrone.Core.Providers
/// <returns></returns> /// <returns></returns>
public virtual bool IsNeeded(EpisodeParseResult parsedReport) public virtual bool IsNeeded(EpisodeParseResult parsedReport)
{ {
//Todo: Fix this so it properly handles multi-epsiode releases (Currently as long as the first episode is needed we download it)
//Todo: for small releases this is less of an issue, but for Full Season Releases this could be an issue if we only need the first episode (or first few)
foreach (var episode in parsedReport.Episodes) foreach (var episode in parsedReport.Episodes)
{ {
var episodeInfo = GetEpisode(parsedReport.SeriesId, parsedReport.SeasonNumber, episode); var episodeInfo = GetEpisode(parsedReport.SeriesId, parsedReport.SeasonNumber, episode);
@ -128,7 +130,6 @@ namespace NzbDrone.Core.Providers
Logger.Trace("Quality is past cut-off skipping."); Logger.Trace("Quality is past cut-off skipping.");
continue; continue;
} }
} }
} }

@ -135,6 +135,9 @@ namespace NzbDrone.Core.Providers.Indexer
return; return;
} }
//Todo: How to handle full season files? Currently the episode list is completely empty for these releases
//Todo: Should we assume that the release contains all the episodes that belong to this season and add them from the DB?
if (!_episodeProvider.IsNeeded(parseResult)) if (!_episodeProvider.IsNeeded(parseResult))
{ {
_logger.Debug("Episode {0} is not needed. skipping.", parseResult); _logger.Debug("Episode {0} is not needed. skipping.", parseResult);

Loading…
Cancel
Save