Remove unused ProcessingResult::NoCheckResult

No one passes a NULL CR to Checkable#ProcessCheckResult() anymore.
This commit is contained in:
Alexander A. Klimov 2025-05-20 10:41:26 +02:00
parent 55fc0e51ff
commit 69d2a0442a
3 changed files with 2 additions and 6 deletions

View File

@ -130,8 +130,6 @@ Dictionary::Ptr ApiActions::ProcessCheckResult(const ConfigObject::Ptr& object,
switch (result) { switch (result) {
case Result::Ok: case Result::Ok:
return ApiActions::CreateResult(200, "Successfully processed check result for object '" + checkable->GetName() + "'."); return ApiActions::CreateResult(200, "Successfully processed check result for object '" + checkable->GetName() + "'.");
case Result::NoCheckResult:
return ApiActions::CreateResult(400, "Could not process check result for object '" + checkable->GetName() + "' because no check result was passed.");
case Result::CheckableInactive: case Result::CheckableInactive:
return ApiActions::CreateResult(503, "Could not process check result for object '" + checkable->GetName() + "' because the object is inactive."); return ApiActions::CreateResult(503, "Could not process check result for object '" + checkable->GetName() + "' because the object is inactive.");
case Result::NewerCheckResultPresent: case Result::NewerCheckResultPresent:

View File

@ -100,14 +100,13 @@ Checkable::ProcessingResult Checkable::ProcessCheckResult(const CheckResult::Ptr
{ {
using Result = Checkable::ProcessingResult; using Result = Checkable::ProcessingResult;
VERIFY(cr);
{ {
ObjectLock olock(this); ObjectLock olock(this);
m_CheckRunning = false; m_CheckRunning = false;
} }
if (!cr)
return Result::NoCheckResult;
double now = Utility::GetTime(); double now = Utility::GetTime();
if (cr->GetScheduleStart() == 0) if (cr->GetScheduleStart() == 0)

View File

@ -119,7 +119,6 @@ public:
enum class ProcessingResult enum class ProcessingResult
{ {
Ok, Ok,
NoCheckResult,
CheckableInactive, CheckableInactive,
NewerCheckResultPresent, NewerCheckResultPresent,
}; };