bug #60745 [Scheduler] Add validation for comma-separated weekdays in PeriodicalTrigger

wazum · wazum · commit 186da57ce0f3 · 2025-07-17T09:09:40.000+02:00
diff --git a/src/Symfony/Component/Scheduler/Tests/Trigger/PeriodicalTriggerTest.php b/src/Symfony/Component/Scheduler/Tests/Trigger/PeriodicalTriggerTest.php
@@ -76,6 +76,7 @@ public static function getInvalidIntervals(): iterable
         yield [-3600, 'The "$interval" argument must be greater than zero.'];
         yield ['0', 'The "$interval" argument must be greater than zero.'];
         yield [0, 'The "$interval" argument must be greater than zero.'];
+        yield ['Monday, Thursday, Saturday', 'Comma-separated values are not supported'];
     }
 
     /**
diff --git a/src/Symfony/Component/Scheduler/Trigger/PeriodicalTrigger.php b/src/Symfony/Component/Scheduler/Trigger/PeriodicalTrigger.php
@@ -51,6 +51,10 @@ public function __construct(
 
             $i = $interval;
             if (\is_string($interval)) {
+                if (\str_contains($interval, ',')) {
+                    throw new InvalidArgumentException(\sprintf('Invalid interval "%s": Comma-separated values are not supported', $interval));
+                }
+
                 $this->description = \sprintf('every %s', $interval);
                 $i = @\DateInterval::createFromDateString($interval);
 

Original file line number	Diff line number	Diff line change
`@@ -76,6 +76,7 @@ public static function getInvalidIntervals(): iterable`
`76`	`76`	`yield [-3600, 'The "$interval" argument must be greater than zero.'];`
`77`	`77`	`yield ['0', 'The "$interval" argument must be greater than zero.'];`
`78`	`78`	`yield [0, 'The "$interval" argument must be greater than zero.'];`
	`79`	`+ yield ['Monday, Thursday, Saturday', 'Comma-separated values are not supported'];`
`79`	`80`	`}`
`80`	`81`
`81`	`82`	`/**`