From 773fb5d1ae1afbd2bb8d868d05a82062a0f27a1d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Torkel=20=C3=96degaard?= Date: Wed, 23 Nov 2016 14:55:10 +0100 Subject: [PATCH] fix(alerting): check if dashboard is saved before testing alert rule, fixes #6549 --- pkg/api/alerting.go | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/pkg/api/alerting.go b/pkg/api/alerting.go index 6d55f03bea3..dcc9b40b07a 100644 --- a/pkg/api/alerting.go +++ b/pkg/api/alerting.go @@ -103,6 +103,10 @@ func GetAlerts(c *middleware.Context) Response { // POST /api/alerts/test func AlertTest(c *middleware.Context, dto dtos.AlertTestCommand) Response { + if _, idErr := dto.Dashboard.Get("id").Int64(); idErr != nil { + return ApiError(400, "The dashboard needs to be saved at least once before you can test an alert rule", nil) + } + backendCmd := alerting.AlertTestCommand{ OrgId: c.OrgId, Dashboard: dto.Dashboard,