feature #49810 [Scheduler] Tweak debug:scheduler command output (fabpot)

fabpot · fabpot · commit 5e6ea1168190 · 2023-03-25T17:03:48.000+01:00
This PR was merged into the 6.3 branch. Discussion ---------- [Scheduler] Tweak debug:scheduler command output | Q | A | ------------- | --- | Branch? | 6.3 | Bug fix? | no | New feature? | no  | Deprecations? | no  | Tickets | n/a | License | MIT | Doc PR | n/a While trying to use the command on my test project, I realized that the output could be tweaked a little bit. cc `@kbond` Commits ------- 149798c [Scheduler] Tweak debug:scheduler command output
diff --git a/src/Symfony/Component/Scheduler/Command/DebugCommand.php b/src/Symfony/Component/Scheduler/Command/DebugCommand.php
@@ -27,6 +27,8 @@
  * Command to list/debug schedules.
  *
  * @author Kevin Bond <kevinbond@gmail.com>
+ *
+ * @experimental
  */
 #[AsCommand(name: 'debug:scheduler', description: 'List schedules and their recurring messages')]
 final class DebugCommand extends Command
@@ -63,16 +65,25 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         $io = new SymfonyStyle($input, $output);
         $io->title('Scheduler');
 
-        $names = $input->getArgument('schedule') ?: $this->scheduleNames;
+        if (!$names = $input->getArgument('schedule') ?: $this->scheduleNames) {
+            $io->error('No schedules found.');
+
+            return self::FAILURE;
+        }
 
         foreach ($names as $name) {
+            $io->section($name);
+
             /** @var ScheduleProviderInterface $schedule */
             $schedule = $this->schedules->get($name);
+            if (!$messages = $schedule->getSchedule()->getRecurringMessages()) {
+                $io->warning(sprintf('No recurring messages found for schedule "%s".', $name));
 
-            $io->section($name);
+                continue;
+            }
             $io->table(
                 ['Message', 'Trigger', 'Next Run'],
-                array_map(self::renderRecurringMessage(...), $schedule->getSchedule()->getRecurringMessages())
+                array_map(self::renderRecurringMessage(...), $messages),
             );
         }
 
