From 4385acef9967459b4fafaabe67698f2a32eb6fc0 Mon Sep 17 00:00:00 2001 From: geogolem <55031547+geogolem@users.noreply.github.com> Date: Thu, 2 Jan 2020 22:28:11 -0500 Subject: [PATCH] Fixed: Reverted a change that caused SQL error on NetImport https://github.com/Radarr/Radarr/commit/c300af82415c6f71af67a74d1f59ae9b2b30c674#diff-4199a537105793b9a05a6232cb0531bb that caused SQL error when doing netimport search update tests update tests revert tests then remove revert tests changes --> comment out the tests --- .../NetImport/NetImportSearchServiceFixture.cs | 3 ++- src/NzbDrone.Core/NetImport/NetImportSearchService.cs | 7 +------ 2 files changed, 3 insertions(+), 7 deletions(-) diff --git a/src/NzbDrone.Core.Test/NetImport/NetImportSearchServiceFixture.cs b/src/NzbDrone.Core.Test/NetImport/NetImportSearchServiceFixture.cs index a7dcababb..88e1cb3b9 100644 --- a/src/NzbDrone.Core.Test/NetImport/NetImportSearchServiceFixture.cs +++ b/src/NzbDrone.Core.Test/NetImport/NetImportSearchServiceFixture.cs @@ -363,6 +363,7 @@ namespace NzbDrone.Core.Test.NetImport .Verify(v => v.AddMovies(It.Is>(s => s.Count == 7 && !s.Any(m => m.TmdbId == _moviesList2[0].TmdbId))), Times.Once()); } + /* [Test] public void should_not_tmdb_map_movie_that_has_tmdbid_from_list() { @@ -400,6 +401,6 @@ namespace NzbDrone.Core.Test.NetImport Mocker.GetMock() .Verify(v => v.AddMovies(It.Is>(s => s.Count == 5)), Times.Once()); - } + }*/ } } diff --git a/src/NzbDrone.Core/NetImport/NetImportSearchService.cs b/src/NzbDrone.Core/NetImport/NetImportSearchService.cs index 46818dce3..2bcbb2be3 100644 --- a/src/NzbDrone.Core/NetImport/NetImportSearchService.cs +++ b/src/NzbDrone.Core/NetImport/NetImportSearchService.cs @@ -135,12 +135,7 @@ namespace NzbDrone.Core.NetImport foreach (var movie in listedMovies) { - var mapped = movie; - - if (mapped.TmdbId == 0) - { - mapped = _movieSearch.MapMovieToTmdbMovie(movie); - } + var mapped = _movieSearch.MapMovieToTmdbMovie(movie); if (mapped != null && mapped.TmdbId > 0) {