Fix function name and use out var for try get in DownloadClientProvider

pull/8509/head
Mark McDowall 2 years ago committed by Bogdan
parent 38d946ed79
commit a953d1a6c5

@ -92,7 +92,7 @@ namespace NzbDrone.Core.Download
if (filterBlockedClients) if (filterBlockedClients)
{ {
return FilterBlockedIndexers(enabledClients).ToList(); return FilterBlockedDownloadClients(enabledClients).ToList();
} }
return enabledClients; return enabledClients;
@ -103,15 +103,13 @@ namespace NzbDrone.Core.Download
return _downloadClientFactory.GetAvailableProviders().Single(d => d.Definition.Id == id); return _downloadClientFactory.GetAvailableProviders().Single(d => d.Definition.Id == id);
} }
private IEnumerable<IDownloadClient> FilterBlockedIndexers(IEnumerable<IDownloadClient> clients) private IEnumerable<IDownloadClient> FilterBlockedDownloadClients(IEnumerable<IDownloadClient> clients)
{ {
var blockedClients = _downloadClientStatusService.GetBlockedProviders().ToDictionary(v => v.ProviderId, v => v); var blockedClients = _downloadClientStatusService.GetBlockedProviders().ToDictionary(v => v.ProviderId, v => v);
foreach (var client in clients) foreach (var client in clients)
{ {
DownloadClientStatus blockedClientStatus; if (blockedClients.TryGetValue(client.Definition.Id, out var blockedClientStatus))
if (blockedClients.TryGetValue(client.Definition.Id, out blockedClientStatus))
{ {
_logger.Debug("Temporarily ignoring client {0} till {1} due to recent failures.", client.Definition.Name, blockedClientStatus.DisabledTill.Value.ToLocalTime()); _logger.Debug("Temporarily ignoring client {0} till {1} due to recent failures.", client.Definition.Name, blockedClientStatus.DisabledTill.Value.ToLocalTime());
continue; continue;

Loading…
Cancel
Save