Deprecate passing null as $message or $code to exceptions

derrabus · derrabus · commit 6e5a390a2a3a · 2021-02-24T14:36:13.000+01:00
diff --git a/Exception/AccessDeniedHttpException.php b/Exception/AccessDeniedHttpException.php
@@ -24,6 +24,12 @@ class AccessDeniedHttpException extends HttpException
      */
     public function __construct(?string $message = '', \Throwable $previous = null, int $code = 0, array $headers = [])
     {
+        if (null === $message) {
+            trigger_deprecation('symfony/http-kernel', '5.3', 'Passing null as $message to "%s()" is deprecated, pass an empty string instead.', __METHOD__);
+
+            $message = '';
+        }
+
         parent::__construct(403, $message, $previous, $headers, $code);
     }
 }
diff --git a/Exception/BadRequestHttpException.php b/Exception/BadRequestHttpException.php
@@ -23,6 +23,12 @@ class BadRequestHttpException extends HttpException
      */
     public function __construct(?string $message = '', \Throwable $previous = null, int $code = 0, array $headers = [])
     {
+        if (null === $message) {
+            trigger_deprecation('symfony/http-kernel', '5.3', 'Passing null as $message to "%s()" is deprecated, pass an empty string instead.', __METHOD__);
+
+            $message = '';
+        }
+
         parent::__construct(400, $message, $previous, $headers, $code);
     }
 }
diff --git a/Exception/ConflictHttpException.php b/Exception/ConflictHttpException.php
@@ -23,6 +23,12 @@ class ConflictHttpException extends HttpException
      */
     public function __construct(?string $message = '', \Throwable $previous = null, int $code = 0, array $headers = [])
     {
+        if (null === $message) {
+            trigger_deprecation('symfony/http-kernel', '5.3', 'Passing null as $message to "%s()" is deprecated, pass an empty string instead.', __METHOD__);
+
+            $message = '';
+        }
+
         parent::__construct(409, $message, $previous, $headers, $code);
     }
 }
diff --git a/Exception/GoneHttpException.php b/Exception/GoneHttpException.php
@@ -23,6 +23,12 @@ class GoneHttpException extends HttpException
      */
     public function __construct(?string $message = '', \Throwable $previous = null, int $code = 0, array $headers = [])
     {
+        if (null === $message) {
+            trigger_deprecation('symfony/http-kernel', '5.3', 'Passing null as $message to "%s()" is deprecated, pass an empty string instead.', __METHOD__);
+
+            $message = '';
+        }
+
         parent::__construct(410, $message, $previous, $headers, $code);
     }
 }
diff --git a/Exception/HttpException.php b/Exception/HttpException.php
@@ -23,6 +23,17 @@ class HttpException extends \RuntimeException implements HttpExceptionInterface
 
     public function __construct(int $statusCode, ?string $message = '', \Throwable $previous = null, array $headers = [], ?int $code = 0)
     {
+        if (null === $message) {
+            trigger_deprecation('symfony/http-kernel', '5.3', 'Passing null as $message to "%s()" is deprecated, pass an empty string instead.', __METHOD__);
+
+            $message = '';
+        }
+        if (null === $code) {
+            trigger_deprecation('symfony/http-kernel', '5.3', 'Passing null as $code to "%s()" is deprecated, pass 0 instead.', __METHOD__);
+
+            $code = 0;
+        }
+
         $this->statusCode = $statusCode;
         $this->headers = $headers;
 
diff --git a/Exception/LengthRequiredHttpException.php b/Exception/LengthRequiredHttpException.php
@@ -23,6 +23,12 @@ class LengthRequiredHttpException extends HttpException
      */
     public function __construct(?string $message = '', \Throwable $previous = null, int $code = 0, array $headers = [])
     {
+        if (null === $message) {
+            trigger_deprecation('symfony/http-kernel', '5.3', 'Passing null as $message to "%s()" is deprecated, pass an empty string instead.', __METHOD__);
+
+            $message = '';
+        }
+
         parent::__construct(411, $message, $previous, $headers, $code);
     }
 }
diff --git a/Exception/MethodNotAllowedHttpException.php b/Exception/MethodNotAllowedHttpException.php
@@ -24,6 +24,17 @@ class MethodNotAllowedHttpException extends HttpException
      */
     public function __construct(array $allow, ?string $message = '', \Throwable $previous = null, ?int $code = 0, array $headers = [])
     {
+        if (null === $message) {
+            trigger_deprecation('symfony/http-kernel', '5.3', 'Passing null as $message to "%s()" is deprecated, pass an empty string instead.', __METHOD__);
+
+            $message = '';
+        }
+        if (null === $code) {
+            trigger_deprecation('symfony/http-kernel', '5.3', 'Passing null as $code to "%s()" is deprecated, pass 0 instead.', __METHOD__);
+
+            $code = 0;
+        }
+
         $headers['Allow'] = strtoupper(implode(', ', $allow));
 
         parent::__construct(405, $message, $previous, $headers, $code);
diff --git a/Exception/NotAcceptableHttpException.php b/Exception/NotAcceptableHttpException.php
@@ -23,6 +23,12 @@ class NotAcceptableHttpException extends HttpException
      */
     public function __construct(?string $message = '', \Throwable $previous = null, int $code = 0, array $headers = [])
     {
+        if (null === $message) {
+            trigger_deprecation('symfony/http-kernel', '5.3', 'Passing null as $message to "%s()" is deprecated, pass an empty string instead.', __METHOD__);
+
+            $message = '';
+        }
+
         parent::__construct(406, $message, $previous, $headers, $code);
     }
 }
diff --git a/Exception/NotFoundHttpException.php b/Exception/NotFoundHttpException.php
@@ -23,6 +23,12 @@ class NotFoundHttpException extends HttpException
      */
     public function __construct(?string $message = '', \Throwable $previous = null, int $code = 0, array $headers = [])
     {
+        if (null === $message) {
+            trigger_deprecation('symfony/http-kernel', '5.3', 'Passing null as $message to "%s()" is deprecated, pass an empty string instead.', __METHOD__);
+
+            $message = '';
+        }
+
         parent::__construct(404, $message, $previous, $headers, $code);
     }
 }
diff --git a/Exception/PreconditionFailedHttpException.php b/Exception/PreconditionFailedHttpException.php
@@ -23,6 +23,12 @@ class PreconditionFailedHttpException extends HttpException
      */
     public function __construct(?string $message = '', \Throwable $previous = null, int $code = 0, array $headers = [])
     {
+        if (null === $message) {
+            trigger_deprecation('symfony/http-kernel', '5.3', 'Passing null as $message to "%s()" is deprecated, pass an empty string instead.', __METHOD__);
+
+            $message = '';
+        }
+
         parent::__construct(412, $message, $previous, $headers, $code);
     }
 }
diff --git a/Exception/PreconditionRequiredHttpException.php b/Exception/PreconditionRequiredHttpException.php
@@ -25,6 +25,12 @@ class PreconditionRequiredHttpException extends HttpException
      */
     public function __construct(?string $message = '', \Throwable $previous = null, int $code = 0, array $headers = [])
     {
+        if (null === $message) {
+            trigger_deprecation('symfony/http-kernel', '5.3', 'Passing null as $message to "%s()" is deprecated, pass an empty string instead.', __METHOD__);
+
+            $message = '';
+        }
+
         parent::__construct(428, $message, $previous, $headers, $code);
     }
 }
diff --git a/Exception/ServiceUnavailableHttpException.php b/Exception/ServiceUnavailableHttpException.php
@@ -24,6 +24,17 @@ class ServiceUnavailableHttpException extends HttpException
      */
     public function __construct($retryAfter = null, ?string $message = '', \Throwable $previous = null, ?int $code = 0, array $headers = [])
     {
+        if (null === $message) {
+            trigger_deprecation('symfony/http-kernel', '5.3', 'Passing null as $message to "%s()" is deprecated, pass an empty string instead.', __METHOD__);
+
+            $message = '';
+        }
+        if (null === $code) {
+            trigger_deprecation('symfony/http-kernel', '5.3', 'Passing null as $code to "%s()" is deprecated, pass 0 instead.', __METHOD__);
+
+            $code = 0;
+        }
+
         if ($retryAfter) {
             $headers['Retry-After'] = $retryAfter;
         }
diff --git a/Exception/TooManyRequestsHttpException.php b/Exception/TooManyRequestsHttpException.php
@@ -26,6 +26,17 @@ class TooManyRequestsHttpException extends HttpException
      */
     public function __construct($retryAfter = null, ?string $message = '', \Throwable $previous = null, ?int $code = 0, array $headers = [])
     {
+        if (null === $message) {
+            trigger_deprecation('symfony/http-kernel', '5.3', 'Passing null as $message to "%s()" is deprecated, pass an empty string instead.', __METHOD__);
+
+            $message = '';
+        }
+        if (null === $code) {
+            trigger_deprecation('symfony/http-kernel', '5.3', 'Passing null as $code to "%s()" is deprecated, pass 0 instead.', __METHOD__);
+
+            $code = 0;
+        }
+
         if ($retryAfter) {
             $headers['Retry-After'] = $retryAfter;
         }
diff --git a/Exception/UnauthorizedHttpException.php b/Exception/UnauthorizedHttpException.php
@@ -24,6 +24,17 @@ class UnauthorizedHttpException extends HttpException
      */
     public function __construct(string $challenge, ?string $message = '', \Throwable $previous = null, ?int $code = 0, array $headers = [])
     {
+        if (null === $message) {
+            trigger_deprecation('symfony/http-kernel', '5.3', 'Passing null as $message to "%s()" is deprecated, pass an empty string instead.', __METHOD__);
+
+            $message = '';
+        }
+        if (null === $code) {
+            trigger_deprecation('symfony/http-kernel', '5.3', 'Passing null as $code to "%s()" is deprecated, pass 0 instead.', __METHOD__);
+
+            $code = 0;
+        }
+
         $headers['WWW-Authenticate'] = $challenge;
 
         parent::__construct(401, $message, $previous, $headers, $code);
diff --git a/Exception/UnprocessableEntityHttpException.php b/Exception/UnprocessableEntityHttpException.php
@@ -23,6 +23,12 @@ class UnprocessableEntityHttpException extends HttpException
      */
     public function __construct(?string $message = '', \Throwable $previous = null, int $code = 0, array $headers = [])
     {
+        if (null === $message) {
+            trigger_deprecation('symfony/http-kernel', '5.3', 'Passing null as $message to "%s()" is deprecated, pass an empty string instead.', __METHOD__);
+
+            $message = '';
+        }
+
         parent::__construct(422, $message, $previous, $headers, $code);
     }
 }
diff --git a/Exception/UnsupportedMediaTypeHttpException.php b/Exception/UnsupportedMediaTypeHttpException.php
@@ -23,6 +23,12 @@ class UnsupportedMediaTypeHttpException extends HttpException
      */
     public function __construct(?string $message = '', \Throwable $previous = null, int $code = 0, array $headers = [])
     {
+        if (null === $message) {
+            trigger_deprecation('symfony/http-kernel', '5.3', 'Passing null as $message to "%s()" is deprecated, pass an empty string instead.', __METHOD__);
+
+            $message = '';
+        }
+
         parent::__construct(415, $message, $previous, $headers, $code);
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -24,6 +24,12 @@ class AccessDeniedHttpException extends HttpException`
`24`	`24`	`*/`
`25`	`25`	`public function __construct(?string $message = '', \Throwable $previous = null, int $code = 0, array $headers = [])`
`26`	`26`	`{`
	`27`	`+ if (null === $message) {`
	`28`	`+ trigger_deprecation('symfony/http-kernel', '5.3', 'Passing null as $message to "%s()" is deprecated, pass an empty string instead.', __METHOD__);`
	`29`	`+`
	`30`	`+ $message = '';`
	`31`	`+ }`
	`32`	`+`
`27`	`33`	`parent::__construct(403, $message, $previous, $headers, $code);`
`28`	`34`	`}`
`29`	`35`	`}`
Original file line number	Diff line number	Diff line change
`@@ -23,6 +23,12 @@ class BadRequestHttpException extends HttpException`
`23`	`23`	`*/`
`24`	`24`	`public function __construct(?string $message = '', \Throwable $previous = null, int $code = 0, array $headers = [])`
`25`	`25`	`{`
	`26`	`+ if (null === $message) {`
	`27`	`+ trigger_deprecation('symfony/http-kernel', '5.3', 'Passing null as $message to "%s()" is deprecated, pass an empty string instead.', __METHOD__);`
	`28`	`+`
	`29`	`+ $message = '';`
	`30`	`+ }`
	`31`	`+`
`26`	`32`	`parent::__construct(400, $message, $previous, $headers, $code);`
`27`	`33`	`}`
`28`	`34`	`}`
Original file line number	Diff line number	Diff line change
`@@ -23,6 +23,12 @@ class ConflictHttpException extends HttpException`
`23`	`23`	`*/`
`24`	`24`	`public function __construct(?string $message = '', \Throwable $previous = null, int $code = 0, array $headers = [])`
`25`	`25`	`{`
	`26`	`+ if (null === $message) {`
	`27`	`+ trigger_deprecation('symfony/http-kernel', '5.3', 'Passing null as $message to "%s()" is deprecated, pass an empty string instead.', __METHOD__);`
	`28`	`+`
	`29`	`+ $message = '';`
	`30`	`+ }`
	`31`	`+`
`26`	`32`	`parent::__construct(409, $message, $previous, $headers, $code);`
`27`	`33`	`}`
`28`	`34`	`}`
Original file line number	Diff line number	Diff line change
`@@ -23,6 +23,12 @@ class GoneHttpException extends HttpException`
`23`	`23`	`*/`
`24`	`24`	`public function __construct(?string $message = '', \Throwable $previous = null, int $code = 0, array $headers = [])`
`25`	`25`	`{`
	`26`	`+ if (null === $message) {`
	`27`	`+ trigger_deprecation('symfony/http-kernel', '5.3', 'Passing null as $message to "%s()" is deprecated, pass an empty string instead.', __METHOD__);`
	`28`	`+`
	`29`	`+ $message = '';`
	`30`	`+ }`
	`31`	`+`
`26`	`32`	`parent::__construct(410, $message, $previous, $headers, $code);`
`27`	`33`	`}`
`28`	`34`	`}`
Original file line number	Diff line number	Diff line change
`@@ -23,6 +23,12 @@ class LengthRequiredHttpException extends HttpException`
`23`	`23`	`*/`
`24`	`24`	`public function __construct(?string $message = '', \Throwable $previous = null, int $code = 0, array $headers = [])`
`25`	`25`	`{`
	`26`	`+ if (null === $message) {`
	`27`	`+ trigger_deprecation('symfony/http-kernel', '5.3', 'Passing null as $message to "%s()" is deprecated, pass an empty string instead.', __METHOD__);`
	`28`	`+`
	`29`	`+ $message = '';`
	`30`	`+ }`
	`31`	`+`
`26`	`32`	`parent::__construct(411, $message, $previous, $headers, $code);`
`27`	`33`	`}`
`28`	`34`	`}`
Original file line number	Diff line number	Diff line change
`@@ -23,6 +23,12 @@ class NotAcceptableHttpException extends HttpException`
`23`	`23`	`*/`
`24`	`24`	`public function __construct(?string $message = '', \Throwable $previous = null, int $code = 0, array $headers = [])`
`25`	`25`	`{`
	`26`	`+ if (null === $message) {`
	`27`	`+ trigger_deprecation('symfony/http-kernel', '5.3', 'Passing null as $message to "%s()" is deprecated, pass an empty string instead.', __METHOD__);`
	`28`	`+`
	`29`	`+ $message = '';`
	`30`	`+ }`
	`31`	`+`
`26`	`32`	`parent::__construct(406, $message, $previous, $headers, $code);`
`27`	`33`	`}`
`28`	`34`	`}`
Original file line number	Diff line number	Diff line change
`@@ -23,6 +23,12 @@ class NotFoundHttpException extends HttpException`
`23`	`23`	`*/`
`24`	`24`	`public function __construct(?string $message = '', \Throwable $previous = null, int $code = 0, array $headers = [])`
`25`	`25`	`{`
	`26`	`+ if (null === $message) {`
	`27`	`+ trigger_deprecation('symfony/http-kernel', '5.3', 'Passing null as $message to "%s()" is deprecated, pass an empty string instead.', __METHOD__);`
	`28`	`+`
	`29`	`+ $message = '';`
	`30`	`+ }`
	`31`	`+`
`26`	`32`	`parent::__construct(404, $message, $previous, $headers, $code);`
`27`	`33`	`}`
`28`	`34`	`}`
Original file line number	Diff line number	Diff line change
`@@ -23,6 +23,12 @@ class PreconditionFailedHttpException extends HttpException`
`23`	`23`	`*/`
`24`	`24`	`public function __construct(?string $message = '', \Throwable $previous = null, int $code = 0, array $headers = [])`
`25`	`25`	`{`
	`26`	`+ if (null === $message) {`
	`27`	`+ trigger_deprecation('symfony/http-kernel', '5.3', 'Passing null as $message to "%s()" is deprecated, pass an empty string instead.', __METHOD__);`
	`28`	`+`
	`29`	`+ $message = '';`
	`30`	`+ }`
	`31`	`+`
`26`	`32`	`parent::__construct(412, $message, $previous, $headers, $code);`
`27`	`33`	`}`
`28`	`34`	`}`