Browse Source

fix(alerting): check if dashboard is saved before testing alert rule, fixes #6549

Torkel Ödegaard 9 years ago
parent
commit
773fb5d1ae
1 changed files with 4 additions and 0 deletions
  1. 4 0
      pkg/api/alerting.go

+ 4 - 0
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,