From 274df2f1ed066bef8bb3483711cee03cec61dfc3 Mon Sep 17 00:00:00 2001 From: Qstick Date: Tue, 17 Jul 2018 21:04:10 -0400 Subject: [PATCH] Fixed: Skip torrents in Deluge api that don't have hashes --- .../DelugeTests/DelugeFixture.cs | 18 ++++++++++++++++++ .../Download/Clients/Deluge/Deluge.cs | 2 ++ 2 files changed, 20 insertions(+) diff --git a/src/NzbDrone.Core.Test/Download/DownloadClientTests/DelugeTests/DelugeFixture.cs b/src/NzbDrone.Core.Test/Download/DownloadClientTests/DelugeTests/DelugeFixture.cs index 19488834c..fb819d1a9 100644 --- a/src/NzbDrone.Core.Test/Download/DownloadClientTests/DelugeTests/DelugeFixture.cs +++ b/src/NzbDrone.Core.Test/Download/DownloadClientTests/DelugeTests/DelugeFixture.cs @@ -290,6 +290,24 @@ namespace NzbDrone.Core.Test.Download.DownloadClientTests.DelugeTests item.CanBeRemoved.Should().Be(canBeRemoved); } + [Test] + public void GetItems_should_ignore_items_without_hash() + { + _downloading.Hash = null; + + GivenTorrents(new List + { + _downloading, + _queued + }); + + var items = Subject.GetItems().ToList(); + + items.Should().HaveCount(1); + + items.First().Status.Should().Be(DownloadItemStatus.Queued); + } + [Test] public void should_return_status_with_outputdirs() { diff --git a/src/NzbDrone.Core/Download/Clients/Deluge/Deluge.cs b/src/NzbDrone.Core/Download/Clients/Deluge/Deluge.cs index 4910c64ab..9cdfd49bb 100644 --- a/src/NzbDrone.Core/Download/Clients/Deluge/Deluge.cs +++ b/src/NzbDrone.Core/Download/Clients/Deluge/Deluge.cs @@ -104,6 +104,8 @@ namespace NzbDrone.Core.Download.Clients.Deluge foreach (var torrent in torrents) { + if (torrent.Hash == null) continue; + var item = new DownloadClientItem(); item.DownloadId = torrent.Hash.ToUpper(); item.Title = torrent.Name;