feature #51245 [Scheduler] Only use toString if defined for message (fabpot)

fabpot · fabpot · commit 20cefc8ac2c2 · 2023-08-03T12:42:21.000+02:00
This PR was merged into the 6.4 branch. Discussion ---------- [Scheduler] Only use toString if defined for message | Q | A | ------------- | --- | Branch? | 6.4 | Bug fix? | no | New feature? | yes  | Deprecations? | no  | Tickets | n/a | License | MIT | Doc PR | n/a If the message is stringable, let's only use that instead of having the class name and the message as string (more flexible). For the trigger, it's always stringable anyway. Commits ------- 9d2ca60 [Scheduler] Only use toString if defined for message
diff --git a/src/Symfony/Component/Scheduler/Command/DebugCommand.php b/src/Symfony/Component/Scheduler/Command/DebugCommand.php
@@ -113,21 +113,10 @@ private static function renderRecurringMessage(RecurringMessage $recurringMessag
             $message = $message->getMessage();
         }
 
-        $messageName = (new \ReflectionClass($message))->getShortName();
-        $triggerName = (new \ReflectionClass($trigger))->getShortName();
-
-        if ($message instanceof \Stringable) {
-            $messageName .= ": {$message}";
-        }
-
-        if ($trigger instanceof \Stringable) {
-            $triggerName .= ": {$trigger}";
-        }
-
         return [
-            $messageName,
-            $triggerName,
-            $recurringMessage->getTrigger()->getNextRunDate($date)?->format(\DateTimeInterface::ATOM) ?? '-',
+            $message instanceof \Stringable ? (string) $message : (new \ReflectionClass($message))->getShortName(),
+            (string) $trigger,
+            $trigger->getNextRunDate($date)?->format(\DateTimeInterface::ATOM) ?? '-',
         ];
     }
 }
