moveepisode is less side-effecty!

This commit is contained in:
kay.one 2013-08-29 23:41:49 -07:00
parent 954a79639f
commit c25b92955f
1 changed files with 3 additions and 5 deletions

View File

@ -34,14 +34,12 @@ namespace NzbDrone.Core.MediaFiles
{ {
var renamed = new List<EpisodeFile>(); var renamed = new List<EpisodeFile>();
foreach (var file in episodeFiles) foreach (var episodeFile in episodeFiles)
{ {
try try
{ {
var episodeFile = file;
_logger.Trace("Renaming episode file: {0}", episodeFile); _logger.Trace("Renaming episode file: {0}", episodeFile);
episodeFile = _episodeFileMover.MoveEpisodeFile(episodeFile, series); episodeFile.Path = _episodeFileMover.MoveEpisodeFile(episodeFile, series);
_mediaFileService.Update(episodeFile); _mediaFileService.Update(episodeFile);
renamed.Add(episodeFile); renamed.Add(episodeFile);
@ -54,7 +52,7 @@ namespace NzbDrone.Core.MediaFiles
} }
catch (Exception ex) catch (Exception ex)
{ {
_logger.ErrorException("Failed to rename file: " + file.Path, ex); _logger.ErrorException("Failed to rename file: " + episodeFile.Path, ex);
} }
} }