Fixed: Record status for notifications on tests

pull/9394/head v5.1.3.8246
Bogdan 11 months ago
parent 1650ce17fb
commit 3376a467ca

@ -1,6 +1,7 @@
using System; using System;
using System.Collections.Generic; using System.Collections.Generic;
using System.Linq; using System.Linq;
using FluentValidation.Results;
using NLog; using NLog;
using NzbDrone.Core.Messaging.Events; using NzbDrone.Core.Messaging.Events;
using NzbDrone.Core.ThingiProvider; using NzbDrone.Core.ThingiProvider;
@ -192,5 +193,26 @@ namespace NzbDrone.Core.Notifications
definition.SupportsOnApplicationUpdate = provider.SupportsOnApplicationUpdate; definition.SupportsOnApplicationUpdate = provider.SupportsOnApplicationUpdate;
definition.SupportsOnManualInteractionRequired = provider.SupportsOnManualInteractionRequired; definition.SupportsOnManualInteractionRequired = provider.SupportsOnManualInteractionRequired;
} }
public override ValidationResult Test(NotificationDefinition definition)
{
var result = base.Test(definition);
if (definition.Id == 0)
{
return result;
}
if (result == null || result.IsValid)
{
_notificationStatusService.RecordSuccess(definition.Id);
}
else
{
_notificationStatusService.RecordFailure(definition.Id);
}
return result;
}
} }
} }

Loading…
Cancel
Save