defer addition of new method argument to 8.0

xabbuh · xabbuh · commit 8fadaca4db15 · 2024-04-09T08:05:14.000+02:00
diff --git a/src/Symfony/Component/HttpFoundation/UriSigner.php b/src/Symfony/Component/HttpFoundation/UriSigner.php
@@ -51,8 +51,18 @@ public function __construct(#[\SensitiveParameter] string $secret, string $hashP
      *
      * The expiration is added as a query string parameter.
      */
-    public function sign(string $uri, \DateTimeInterface|\DateInterval|int|null $expiration = null): string
+    public function sign(string $uri/*, \DateTimeInterface|\DateInterval|int|null $expiration = null*/): string
     {
+        $expiration = null;
+
+        if (1 < \func_num_args()) {
+            $expiration = func_get_arg(1);
+        }
+
+        if (null !== $expiration && !$expiration instanceof \DateTimeInterface && !$expiration instanceof \DateInterval && !\is_int($expiration)) {
+            throw new \TypeError(sprintf('The second argument of %s() must be an instance of %s or %s, an integer or null (%s given).', __METHOD__, \DateTimeInterface::class, \DateInterval::class, get_debug_type($expiration)));
+        }
+
         $url = parse_url($uri);
         $params = [];
 
