discard invalid action plans

engine/action_plan.go

@@ -297,8 +297,7 @@ func (at *ActionPlan) Execute() (err error) {
 
 func (at *ActionPlan) IsASAP() bool {
 	if at.Timing == nil {
-		Logger.Warning(fmt.Sprintf("Nil timing on action plan: %+v, executing ASAP!", at))
-		return true
+		return false
 	}
 	return at.Timing.Timing.StartTime == utils.ASAP
 }

scheduler/scheduler.go

@@ -92,6 +92,10 @@ func (s *Scheduler) LoadActionPlans(storage engine.RatingStorage) {
 		isAsap := false
 		newApls := make([]*engine.ActionPlan, 0) // will remove the one time runs from the database
 		for _, ap := range aps {
+			if ap.Timing == nil {
+				Logger.Warning(fmt.Sprintf("Nil timing on action plan: %+v, discarding!", at))
+				continue
+			}
 			isAsap = ap.IsASAP()
 			toBeSaved = toBeSaved || isAsap
 			if isAsap {