Upgrade phpcs-tool to 9.0.0 (#71)

Author: janlam7

composer.json

@@ -19,7 +19,7 @@
     },
     "require-dev": {
         "composer/composer":  "^2.0.0",
-        "hostnet/phpcs-tool": "^8.3.21",
+        "hostnet/phpcs-tool": "^9.0.0",
         "phpunit/phpunit":    "^8.5.0"
     },
     "conflict": {

src/Annotation/Generate.php

@@ -23,29 +23,21 @@ class Generate
 {
     /**
      * No method should be generated.
-     *
-     * @var string
      */
     public const VISIBILITY_NONE = 'none';
 
     /**
      * A public method should be generated.
-     *
-     * @var string
      */
     public const VISIBILITY_PUBLIC = 'public';
 
     /**
      * A protected method should be generated.
-     *
-     * @var string
      */
     public const VISIBILITY_PROTECTED = 'protected';
 
     /**
      * A private method should be generated.
-     *
-     * @var string
      */
     public const VISIBILITY_PRIVATE = 'private';
 
@@ -218,8 +210,6 @@ public function getEnumerators(): array
      *  - public
      *
      * @param array ...$requirements
-     *
-     * @return string
      */
     public static function getMostLimitedVisibility(...$requirements): string
     {

src/AnnotationProcessor/AnnotationProcessorInterface.php

@@ -21,7 +21,6 @@ interface AnnotationProcessorInterface
      * @param object              $annotation  class annotated with @annotation
      * @param PropertyInformation $information location to store new
      *                                          information about the property
-     * @return void
      */
     public function processAnnotation($annotation, PropertyInformation $information): void;
 
@@ -31,8 +30,6 @@ public function processAnnotation($annotation, PropertyInformation $information)
      *
      * This value can be used to feed it into a doc parser to only parse
      * the annotations that will actually be used.
-     *
-     * @return string
      */
     public function getProcessableAnnotationNamespace(): string;
 }

src/AnnotationProcessor/DoctrineAnnotationProcessor.php

@@ -51,8 +51,6 @@ class DoctrineAnnotationProcessor implements AnnotationProcessorInterface
      *
      * @param mixed $annotation object of a class annotated with @annotation
      * @param PropertyInformation $information
-     *
-     * @return void
      */
 
     public function processAnnotation($annotation, PropertyInformation $information): void
@@ -98,9 +96,6 @@ public function processAnnotation($annotation, PropertyInformation $information)
         // Do nothing for other types
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getProcessableAnnotationNamespace(): string
     {
         return 'Doctrine\ORM\Mapping';
@@ -214,6 +209,8 @@ private function processJoinColumn(JoinColumn $join_column, PropertyInformation
      *
      * @param mixed $type
      *
+     * @phpcsSuppress SlevomatCodingStandard.TypeHints.ReturnTypeHint
+     *
      * @return mixed A valid PHP type
      */
     private function transformType($type)
@@ -279,8 +276,6 @@ private function transformType($type)
      * is in the current namespace and is left as-is.
      *
      * @param string $type
-     *
-     * @return string
      */
     private function transformComplexType($type): string
     {
@@ -303,8 +298,6 @@ private function transformComplexType($type): string
      * @see http://doctrine-dbal.readthedocs.org/en/latest/reference/types.html
      *
      * @param string $type
-     *
-     * @return int
      */
     private function getIntegerSizeForType(string $type): int
     {

src/AnnotationProcessor/GenerateAnnotationProcessor.php

@@ -78,9 +78,6 @@ public function processAnnotation($annotation, PropertyInformation $info): void
         $info->setIsGenerator(true);
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getProcessableAnnotationNamespace(): string
     {
         return 'Hostnet\Component\AccessorGenerator\Annotation';

src/AnnotationProcessor/PropertyInformation.php

@@ -22,133 +22,133 @@ class PropertyInformation implements PropertyInformationInterface
 {
     /**
      * {@inheritdoc}
-
+     *
      * @var string|null
      */
     private $type;
 
     /**
      * {@inheritdoc}
-
+     *
      * @var string
      */
     private $type_hint = '';
 
     /**
      * {@inheritdoc}
-
+     *
      * @var string
      */
     private $fully_qualified_type = '';
 
     /**
      * {@inheritdoc}
-
+     *
      * @var string
      */
     private $encryption_alias;
 
     /**
      * {@inheritdoc}
-
+     *
      * @var int
      */
     private $integer_size = 32; // Be on the safe side for database interaction.
 
     /**
      * {@inheritdoc}
-
+     *
      * @var int
      */
     private $length = 0;
 
     /**
      * {@inheritdoc}
-
+     *
      * @var int
      */
     private $precision = 0;
 
     /**
      * {@inheritdoc}
-
+     *
      * @var int
      */
     private $scale = 0;
 
     /**
      * {@inheritdoc}
-
+     *
      * @var bool|null
      */
     private $nullable;
 
     /**
      * {@inheritdoc}
-
+     *
      * @var bool|null
      */
     private $unique;
 
     /**
      * {@inheritdoc}
-
+     *
      * @var bool
      */
     private $is_generator = false;
 
     /**
      * {@inheritdoc}
-
+     *
      * @var bool
      */
     private $is_fixed_point_number = false;
 
     /**
      * {@inheritdoc}
-
+     *
      * @var string
      */
     private $referenced_property = '';
 
     /**
      * {@inheritdoc}
-
+     *
      * @var bool
      */
     private $is_collection = false;
 
     /**
      * {@inheritdoc}
-
+     *
      * @var bool
      */
     private $is_referencing_collection = false;
 
     /**
      * {@inheritdoc}
-
+     *
      * @var bool
      */
     private $generate_strict = true;
 
     /**
      * {@inheritdoc}
-
+     *
      * @var string|null
      */
     private $generate_get;
 
     /**
      * {@inheritdoc}
-
+     *
      * @var string|null
      */
     private $generate_set;
 
     /**
      * {@inheritdoc}
-
+     *
      * @var string|null
      */
     private $generate_add;
@@ -162,14 +162,14 @@ class PropertyInformation implements PropertyInformationInterface
 
     /**
      * {@inheritdoc}
-
+     *
      * @var string|null
      */
     private $index;
 
     /**
      * {@inheritdoc}
-
+     *
      * @var string|null
      */
     private $generate_remove;
@@ -230,7 +230,6 @@ public function registerAnnotationProcessor(AnnotationProcessorInterface $proces
     /**
      * Start the processing of processAnnotations
      *
-     * @return void
      * @throws \OutOfBoundsException
      * @throws \Hostnet\Component\AccessorGenerator\Reflection\Exception\ClassDefinitionNotFoundException
      * @throws \RuntimeException
@@ -399,7 +398,6 @@ public function getFullyQualifiedType(): string
      *
      * @throws \DomainException
      * @throws \InvalidArgumentException
-     * @return string
      */
     private function validateType(string $type): string
     {
@@ -613,9 +611,6 @@ public function setIsGenerator($bool): void
         $this->is_generator = $bool;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function isGenerator(): bool
     {
         return $this->is_generator;
@@ -782,7 +777,7 @@ public function setPrecision(?int $precision): self
      *
      * @return number
      */
-    public function getScale()
+    public function getScale(): int
     {
         return $this->scale;
     }
@@ -813,12 +808,7 @@ public function setScale(?int $scale): self
         return $this;
     }
 
-    /**
-     * {@inheritdoc}
-     *
-     * @return bool|null
-     */
-    public function isNullable()
+    public function isNullable(): ?bool
     {
         if (null === $this->nullable) {
             return null;
@@ -1022,42 +1012,22 @@ private static function getValidTypes(): array
         ];
     }
 
-    /**
-     * {@inheritdoc}
-     *
-     * @return string|null
-     */
-    public function getGetVisibility()
+    public function getGetVisibility(): ?string
     {
         return $this->generate_get;
     }
 
-    /**
-     * {@inheritdoc}
-     *
-     * @return string|null
-     */
-    public function getSetVisibility()
+    public function getSetVisibility(): ?string
     {
         return $this->generate_set;
     }
 
-    /**
-     * {@inheritdoc}
-     *
-     * @return string|null
-     */
-    public function getAddVisibility()
+    public function getAddVisibility(): ?string
     {
         return $this->generate_add;
     }
 
-    /**
-     * {@inheritdoc}
-     *
-     * @return string|null
-     */
-    public function getRemoveVisibility()
+    public function getRemoveVisibility(): ?string
     {
         return $this->generate_remove;
     }
@@ -1082,8 +1052,6 @@ public function addEnumeratorToGenerate(Enumerator $enumerator): void
 
     /**
      * Returns true if an enumerator accessor will be generated for this property.
-     *
-     * @return bool
      */
     public function willGenerateEnumeratorAccessors(): bool
     {