[Components] Convert to native return types

Author: wouterj

Command/DebugCommand.php

@@ -43,10 +43,7 @@ public function __construct(MetadataFactoryInterface $validator)
         $this->validator = $validator;
     }
 
-    /**
-     * @return void
-     */
-    protected function configure()
+    protected function configure(): void
     {
         $this
             ->addArgument('class', InputArgument::REQUIRED, 'A fully qualified class name or a path')

Constraint.php

@@ -51,8 +51,6 @@ abstract class Constraint
 
     /**
      * Domain-specific data attached to a constraint.
-     *
-     * @var mixed
      */
     public $payload;
 
@@ -181,11 +179,9 @@ protected function normalizeOptions(mixed $options): array
      * this method will be called at most once per constraint instance and
      * option name.
      *
-     * @return void
-     *
      * @throws InvalidOptionsException If an invalid option name is given
      */
-    public function __set(string $option, mixed $value)
+    public function __set(string $option, mixed $value): void
     {
         if ('groups' === $option) {
             $this->groups = (array) $value;
@@ -223,10 +219,8 @@ public function __isset(string $option): bool
 
     /**
      * Adds the given group if this constraint is in the Default group.
-     *
-     * @return void
      */
-    public function addImplicitGroupName(string $group)
+    public function addImplicitGroupName(string $group): void
     {
         if (null === $this->groups && \array_key_exists('groups', (array) $this)) {
             throw new \LogicException(sprintf('"%s::$groups" is set to null. Did you forget to call "%s::__construct()"?', static::class, self::class));
@@ -242,11 +236,9 @@ public function addImplicitGroupName(string $group)
      *
      * Override this method to define a default option.
      *
-     * @return string|null
-     *
      * @see __construct()
      */
-    public function getDefaultOption()
+    public function getDefaultOption(): ?string
     {
         return null;
     }
@@ -260,7 +252,7 @@ public function getDefaultOption()
      *
      * @see __construct()
      */
-    public function getRequiredOptions()
+    public function getRequiredOptions(): array
     {
         return [];
     }
@@ -271,10 +263,8 @@ public function getRequiredOptions()
      * By default, this is the fully qualified name of the constraint class
      * suffixed with "Validator". You can override this method to change that
      * behavior.
-     *
-     * @return string
      */
-    public function validatedBy()
+    public function validatedBy(): string
     {
         return static::class.'Validator';
     }
@@ -288,7 +278,7 @@ public function validatedBy()
      *
      * @return string|string[] One or more constant values
      */
-    public function getTargets()
+    public function getTargets(): string|array
     {
         return self::PROPERTY_CONSTRAINT;
     }

ConstraintValidator.php

@@ -36,10 +36,7 @@ abstract class ConstraintValidator implements ConstraintValidatorInterface
      */
     protected $context;
 
-    /**
-     * @return void
-     */
-    public function initialize(ExecutionContextInterface $context)
+    public function initialize(ExecutionContextInterface $context): void
     {
         $this->context = $context;
     }

ConstraintValidatorInterface.php

@@ -20,15 +20,11 @@ interface ConstraintValidatorInterface
 {
     /**
      * Initializes the constraint validator.
-     *
-     * @return void
      */
-    public function initialize(ExecutionContextInterface $context);
+    public function initialize(ExecutionContextInterface $context): void;
 
     /**
      * Checks if the passed value is valid.
-     *
-     * @return void
      */
-    public function validate(mixed $value, Constraint $constraint);
+    public function validate(mixed $value, Constraint $constraint): void;
 }

ConstraintViolationList.php

@@ -58,18 +58,12 @@ public function __toString(): string
         return $string;
     }
 
-    /**
-     * @return void
-     */
-    public function add(ConstraintViolationInterface $violation)
+    public function add(ConstraintViolationInterface $violation): void
     {
         $this->violations[] = $violation;
     }
 
-    /**
-     * @return void
-     */
-    public function addAll(ConstraintViolationListInterface $otherList)
+    public function addAll(ConstraintViolationListInterface $otherList): void
     {
         foreach ($otherList as $violation) {
             $this->violations[] = $violation;
@@ -90,18 +84,12 @@ public function has(int $offset): bool
         return isset($this->violations[$offset]);
     }
 
-    /**
-     * @return void
-     */
-    public function set(int $offset, ConstraintViolationInterface $violation)
+    public function set(int $offset, ConstraintViolationInterface $violation): void
     {
         $this->violations[$offset] = $violation;
     }
 
-    /**
-     * @return void
-     */
-    public function remove(int $offset)
+    public function remove(int $offset): void
     {
         unset($this->violations[$offset]);
     }

ConstraintViolationListInterface.php

@@ -25,17 +25,13 @@ interface ConstraintViolationListInterface extends \Traversable, \Countable, \Ar
 {
     /**
      * Adds a constraint violation to this list.
-     *
-     * @return void
      */
-    public function add(ConstraintViolationInterface $violation);
+    public function add(ConstraintViolationInterface $violation): void;
 
     /**
      * Merges an existing violation list into this list.
-     *
-     * @return void
      */
-    public function addAll(self $otherList);
+    public function addAll(self $otherList): void;
 
     /**
      * Returns the violation at a given offset.
@@ -57,19 +53,15 @@ public function has(int $offset): bool;
      * Sets a violation at a given offset.
      *
      * @param int $offset The violation offset
-     *
-     * @return void
      */
-    public function set(int $offset, ConstraintViolationInterface $violation);
+    public function set(int $offset, ConstraintViolationInterface $violation): void;
 
     /**
      * Removes a violation at a given offset.
      *
      * @param int $offset The offset to remove
-     *
-     * @return void
      */
-    public function remove(int $offset);
+    public function remove(int $offset): void;
 
     /**
      * Converts the violation into a string for debugging purposes.

Constraints/AbstractComparisonValidator.php

@@ -34,10 +34,7 @@ public function __construct(PropertyAccessorInterface $propertyAccessor = null)
         $this->propertyAccessor = $propertyAccessor;
     }
 
-    /**
-     * @return void
-     */
-    public function validate(mixed $value, Constraint $constraint)
+    public function validate(mixed $value, Constraint $constraint): void
     {
         if (!$constraint instanceof AbstractComparison) {
             throw new UnexpectedTypeException($constraint, AbstractComparison::class);

Constraints/AllValidator.php

@@ -21,10 +21,7 @@
  */
 class AllValidator extends ConstraintValidator
 {
-    /**
-     * @return void
-     */
-    public function validate(mixed $value, Constraint $constraint)
+    public function validate(mixed $value, Constraint $constraint): void
     {
         if (!$constraint instanceof All) {
             throw new UnexpectedTypeException($constraint, All::class);

Constraints/AtLeastOneOfValidator.php

@@ -20,10 +20,7 @@
  */
 class AtLeastOneOfValidator extends ConstraintValidator
 {
-    /**
-     * @return void
-     */
-    public function validate(mixed $value, Constraint $constraint)
+    public function validate(mixed $value, Constraint $constraint): void
     {
         if (!$constraint instanceof AtLeastOneOf) {
             throw new UnexpectedTypeException($constraint, AtLeastOneOf::class);

Constraints/BicValidator.php

@@ -63,10 +63,7 @@ public function __construct(PropertyAccessor $propertyAccessor = null)
         $this->propertyAccessor = $propertyAccessor;
     }
 
-    /**
-     * @return void
-     */
-    public function validate(mixed $value, Constraint $constraint)
+    public function validate(mixed $value, Constraint $constraint): void
     {
         if (!$constraint instanceof Bic) {
             throw new UnexpectedTypeException($constraint, Bic::class);