From f2165da9e402344c3810d871ae476f1a7a4af167 Mon Sep 17 00:00:00 2001 From: Alex Pott <alex.a.pott@googlemail.com> Date: Mon, 1 Apr 2024 15:22:01 +0100 Subject: [PATCH] Issue #3437351: follow-up Fix return typehinting for Validator related classes --- .../Validation/Constraint/FileExtensionConstraintValidator.php | 2 +- .../Constraint/FileExtensionSecureConstraintValidator.php | 2 +- .../Constraint/FileImageDimensionsConstraintValidator.php | 2 +- .../Validation/Constraint/FileIsImageConstraintValidator.php | 2 +- .../Validation/Constraint/FileNameLengthConstraintValidator.php | 2 +- 5 files changed, 5 insertions(+), 5 deletions(-) diff --git a/core/modules/file/src/Plugin/Validation/Constraint/FileExtensionConstraintValidator.php b/core/modules/file/src/Plugin/Validation/Constraint/FileExtensionConstraintValidator.php index b85db047366e..d12fbcae63db 100644 --- a/core/modules/file/src/Plugin/Validation/Constraint/FileExtensionConstraintValidator.php +++ b/core/modules/file/src/Plugin/Validation/Constraint/FileExtensionConstraintValidator.php @@ -15,7 +15,7 @@ class FileExtensionConstraintValidator extends BaseFileConstraintValidator { /** * {@inheritdoc} */ - public function validate(mixed $value, Constraint $constraint) { + public function validate(mixed $value, Constraint $constraint): void { $file = $this->assertValueIsFile($value); if (!$constraint instanceof FileExtensionConstraint) { throw new UnexpectedTypeException($constraint, FileExtensionConstraint::class); diff --git a/core/modules/file/src/Plugin/Validation/Constraint/FileExtensionSecureConstraintValidator.php b/core/modules/file/src/Plugin/Validation/Constraint/FileExtensionSecureConstraintValidator.php index e4c53f5bc8b3..351a237b77fc 100644 --- a/core/modules/file/src/Plugin/Validation/Constraint/FileExtensionSecureConstraintValidator.php +++ b/core/modules/file/src/Plugin/Validation/Constraint/FileExtensionSecureConstraintValidator.php @@ -36,7 +36,7 @@ public static function create(ContainerInterface $container) { /** * {@inheritdoc} */ - public function validate(mixed $value, Constraint $constraint) { + public function validate(mixed $value, Constraint $constraint): void { $file = $this->assertValueIsFile($value); if (!$constraint instanceof FileExtensionSecureConstraint) { throw new UnexpectedTypeException($constraint, FileExtensionSecureConstraint::class); diff --git a/core/modules/file/src/Plugin/Validation/Constraint/FileImageDimensionsConstraintValidator.php b/core/modules/file/src/Plugin/Validation/Constraint/FileImageDimensionsConstraintValidator.php index 9a7dde81ec54..6c5724e89d06 100644 --- a/core/modules/file/src/Plugin/Validation/Constraint/FileImageDimensionsConstraintValidator.php +++ b/core/modules/file/src/Plugin/Validation/Constraint/FileImageDimensionsConstraintValidator.php @@ -45,7 +45,7 @@ public static function create(ContainerInterface $container) { /** * {@inheritdoc} */ - public function validate(mixed $value, Constraint $constraint) { + public function validate(mixed $value, Constraint $constraint): void { $file = $this->assertValueIsFile($value); if (!$constraint instanceof FileImageDimensionsConstraint) { throw new UnexpectedTypeException($constraint, FileImageDimensionsConstraint::class); diff --git a/core/modules/file/src/Plugin/Validation/Constraint/FileIsImageConstraintValidator.php b/core/modules/file/src/Plugin/Validation/Constraint/FileIsImageConstraintValidator.php index 95183a56f330..395890c33cfb 100644 --- a/core/modules/file/src/Plugin/Validation/Constraint/FileIsImageConstraintValidator.php +++ b/core/modules/file/src/Plugin/Validation/Constraint/FileIsImageConstraintValidator.php @@ -35,7 +35,7 @@ public static function create(ContainerInterface $container) { /** * {@inheritdoc} */ - public function validate(mixed $value, Constraint $constraint) { + public function validate(mixed $value, Constraint $constraint): void { $file = $this->assertValueIsFile($value); if (!$constraint instanceof FileIsImageConstraint) { throw new UnexpectedTypeException($constraint, FileIsImageConstraint::class); diff --git a/core/modules/file/src/Plugin/Validation/Constraint/FileNameLengthConstraintValidator.php b/core/modules/file/src/Plugin/Validation/Constraint/FileNameLengthConstraintValidator.php index 12558433eeba..40ce7f21a9fa 100644 --- a/core/modules/file/src/Plugin/Validation/Constraint/FileNameLengthConstraintValidator.php +++ b/core/modules/file/src/Plugin/Validation/Constraint/FileNameLengthConstraintValidator.php @@ -13,7 +13,7 @@ class FileNameLengthConstraintValidator extends BaseFileConstraintValidator { /** * {@inheritdoc} */ - public function validate(mixed $value, Constraint $constraint) { + public function validate(mixed $value, Constraint $constraint): void { $file = $this->assertValueIsFile($value); if (!$constraint instanceof FileNameLengthConstraint) { throw new UnexpectedTypeException($constraint, FileNameLengthConstraint::class); -- GitLab