Fixed: Ensure failing providers are marked as failed when testing all
This commit is contained in:
parent
6103c023de
commit
f6c05d4456
|
@ -74,10 +74,19 @@ namespace NzbDrone.Core.Download
|
||||||
{
|
{
|
||||||
var result = base.Test(definition);
|
var result = base.Test(definition);
|
||||||
|
|
||||||
if ((result == null || result.IsValid) && definition.Id != 0)
|
if (definition.Id == 0)
|
||||||
|
{
|
||||||
|
return result;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (result == null || result.IsValid)
|
||||||
{
|
{
|
||||||
_downloadClientStatusService.RecordSuccess(definition.Id);
|
_downloadClientStatusService.RecordSuccess(definition.Id);
|
||||||
}
|
}
|
||||||
|
else
|
||||||
|
{
|
||||||
|
_downloadClientStatusService.RecordFailure(definition.Id);
|
||||||
|
}
|
||||||
|
|
||||||
return result;
|
return result;
|
||||||
}
|
}
|
||||||
|
|
|
@ -75,10 +75,19 @@ namespace NzbDrone.Core.ImportLists
|
||||||
{
|
{
|
||||||
var result = base.Test(definition);
|
var result = base.Test(definition);
|
||||||
|
|
||||||
if ((result == null || result.IsValid) && definition.Id != 0)
|
if (definition.Id == 0)
|
||||||
|
{
|
||||||
|
return result;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (result == null || result.IsValid)
|
||||||
{
|
{
|
||||||
_importListStatusService.RecordSuccess(definition.Id);
|
_importListStatusService.RecordSuccess(definition.Id);
|
||||||
}
|
}
|
||||||
|
else
|
||||||
|
{
|
||||||
|
_importListStatusService.RecordFailure(definition.Id);
|
||||||
|
}
|
||||||
|
|
||||||
return result;
|
return result;
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue