From cf77a2f46bf7e379da3b4376db19bdf1ff1df011 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jos=C3=A9=20A=2EP?= <53834183+Jossec101@users.noreply.github.com> Date: Mon, 11 Dec 2023 22:39:06 +0100 Subject: [PATCH] refactor(Program.cs): move 'PauseAll' method call outside switch statement to avoid redundancy fix(Program.cs): remove 'all' case from switch statement as 'PauseAll' is now called by default before the switch --- src/Program.cs | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/src/Program.cs b/src/Program.cs index 4217bf15..6ed2735d 100644 --- a/src/Program.cs +++ b/src/Program.cs @@ -61,6 +61,8 @@ async Task ManageJobs(ISchedulerFactory schedulerFactory) var pauseJobs = Environment.GetEnvironmentVariable("PAUSED_JOBS"); Console.WriteLine($"PAUSED_JOBS env var is {pauseJobs}"); var scheduler = Task.Run(() => schedulerFactory.GetScheduler()).Result; + Console.WriteLine("Pausing all jobs..."); + await scheduler.PauseAll(); var triggers = (await scheduler.GetTriggerKeys(GroupMatcher.AnyGroup())).ToList(); triggers = triggers.OrderBy(x => x.Name).ToList(); @@ -75,18 +77,13 @@ async Task ManageJobs(ISchedulerFactory schedulerFactory) switch (pauseJobs) { - case "all": - await scheduler.PauseAll(); - Console.WriteLine("All jobs paused"); - break; - + case null: //Resume all await scheduler.ResumeAll(); break; default: - Console.WriteLine("Pausing all jobs..."); - await scheduler.PauseAll(); + // Get a list of all triggers foreach (var triggerKey in triggers)