Merge branch '7.4' into 8.0

nicolas-grekas · nicolas-grekas · commit 165f4ec117f2 · 2025-07-09T12:08:17.000+02:00
* 7.4:
  [ObjectMapper] handle non existing property errors
  Fix AsCronTask not passing arguments to command
  [HttpFoundation] Fix deprecation in tests on PHP 8.5
  Fix typo
  [Cache] Bump ext-redis to 6.2 and ext-relay to 0.11 minimum
  [VarExporter] Dump implicit-nullable types as explicit to prevent the corresponding deprecation
  [DependencyInjection] CS fix
  chore: PHP CS Fixer fixes
  Leverage get_error_handler()
  Various CS fixes
  Allow mixed root on CompoundConstraintTestCase validator
  [Serializer] Fix readonly property initialization from incorrect scope
  Update BrevoRequestParser.php
  [JsonPath] Improve escape sequence validation in name selector
diff --git a/IpUtils.php b/IpUtils.php
@@ -196,7 +196,7 @@ public static function anonymize(string $ip/* , int $v4Bytes = 1, int $v6Bytes =
             throw new \InvalidArgumentException('Cannot anonymize more than 4 bytes for IPv4 and 16 bytes for IPv6.');
         }
 
-        /**
+        /*
          * If the IP contains a % symbol, then it is a local-link address with scoping according to RFC 4007
          * In that case, we only care about the part before the % symbol, as the following functions, can only work with
          * the IP address itself. As the scope can leak information (containing interface name), we do not want to
diff --git a/Response.php b/Response.php
@@ -320,7 +320,7 @@ public function sendHeaders(?int $statusCode = null): static
             if (!\in_array(\PHP_SAPI, ['cli', 'phpdbg', 'embed'], true)) {
                 $statusCode ??= $this->statusCode;
                 trigger_deprecation('symfony/http-foundation', '7.4', 'Trying to use "%s::sendHeaders()" after headers have already been sent is deprecated will throw a PHP warning in 8.0. Use a "StreamedResponse" instead.', static::class);
-                //header(\sprintf('HTTP/%s %s %s', $this->version, $statusCode, $this->statusText), true, $statusCode);
+                // header(\sprintf('HTTP/%s %s %s', $this->version, $statusCode, $this->statusText), true, $statusCode);
             }
 
             return $this;
diff --git a/Tests/StreamedResponseTest.php b/Tests/StreamedResponseTest.php
@@ -32,7 +32,7 @@ public function testConstructorWithChunks()
 
         $buffer = '';
         ob_start(function (string $chunk) use (&$buffer) {
-            $buffer .= $chunk;
+            return $buffer .= $chunk;
         });
         $callback();
 

Original file line number	Diff line number	Diff line change
`@@ -196,7 +196,7 @@ public static function anonymize(string $ip/* , int $v4Bytes = 1, int $v6Bytes =`
`196`	`196`	`throw new \InvalidArgumentException('Cannot anonymize more than 4 bytes for IPv4 and 16 bytes for IPv6.');`
`197`	`197`	`}`
`198`	`198`
`199`		`- /**`
	`199`	`+ /*`
`200`	`200`	`* If the IP contains a % symbol, then it is a local-link address with scoping according to RFC 4007`
`201`	`201`	`* In that case, we only care about the part before the % symbol, as the following functions, can only work with`
`202`	`202`	`* the IP address itself. As the scope can leak information (containing interface name), we do not want to`
Original file line number	Diff line number	Diff line change
`@@ -320,7 +320,7 @@ public function sendHeaders(?int $statusCode = null): static`
`320`	`320`	`if (!\in_array(\PHP_SAPI, ['cli', 'phpdbg', 'embed'], true)) {`
`321`	`321`	`$statusCode ??= $this->statusCode;`
`322`	`322`	`trigger_deprecation('symfony/http-foundation', '7.4', 'Trying to use "%s::sendHeaders()" after headers have already been sent is deprecated will throw a PHP warning in 8.0. Use a "StreamedResponse" instead.', static::class);`
`323`		`- //header(\sprintf('HTTP/%s %s %s', $this->version, $statusCode, $this->statusText), true, $statusCode);`
	`323`	`+ // header(\sprintf('HTTP/%s %s %s', $this->version, $statusCode, $this->statusText), true, $statusCode);`
`324`	`324`	`}`
`325`	`325`
`326`	`326`	`return $this;`