From c65e3b63f556be95d9e3725d872c79654c09b23e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Andreas=20M=C3=B6ller?= Date: Mon, 21 Dec 2020 16:11:20 +0100 Subject: [PATCH] Enhancement: Enable no_superfluous_phpdoc_tags fixer (#199) * Enhancement: Enable no_superfluous_phpdoc_tags fixer * Fix: Run 'make cs' * Fix: Run 'make baseline' --- .php_cs.dist | 1 + phpstan-baseline.neon | 30 ++++++++++---------- psalm.baseline.xml | 5 +--- src/Faker/Calculator/Isbn.php | 4 --- src/Faker/DefaultGenerator.php | 4 --- src/Faker/Documentor.php | 3 -- src/Faker/Generator.php | 4 --- src/Faker/Guesser/Name.php | 3 -- src/Faker/ORM/CakePHP/Populator.php | 3 -- src/Faker/ORM/Doctrine/ColumnTypeGuesser.php | 2 -- src/Faker/ORM/Doctrine/EntityPopulator.php | 16 +---------- src/Faker/ORM/Doctrine/Populator.php | 4 +-- src/Faker/ORM/Mandango/ColumnTypeGuesser.php | 3 -- src/Faker/ORM/Mandango/EntityPopulator.php | 5 ---- src/Faker/ORM/Mandango/Populator.php | 4 --- src/Faker/ORM/Propel/ColumnTypeGuesser.php | 5 ---- src/Faker/ORM/Propel/EntityPopulator.php | 6 ---- src/Faker/ORM/Propel/Populator.php | 3 -- src/Faker/ORM/Propel2/ColumnTypeGuesser.php | 5 ---- src/Faker/ORM/Propel2/EntityPopulator.php | 6 ---- src/Faker/ORM/Propel2/Populator.php | 3 -- src/Faker/ORM/Spot/ColumnTypeGuesser.php | 4 --- src/Faker/ORM/Spot/EntityPopulator.php | 12 +------- src/Faker/ORM/Spot/Populator.php | 3 -- src/Faker/Provider/Base.php | 9 ------ src/Faker/Provider/DateTime.php | 1 - src/Faker/Provider/de_AT/Person.php | 2 -- src/Faker/Provider/hy_AM/PhoneNumber.php | 3 -- src/Faker/Provider/pl_PL/LicensePlate.php | 7 ----- src/Faker/Provider/pt_BR/PhoneNumber.php | 6 ---- src/Faker/Provider/ro_RO/Person.php | 5 ++-- src/Faker/UniqueGenerator.php | 7 +---- src/Faker/ValidGenerator.php | 5 ---- 33 files changed, 23 insertions(+), 160 deletions(-) diff --git a/.php_cs.dist b/.php_cs.dist index 4bcec150e3..2dacaf74cd 100644 --- a/.php_cs.dist +++ b/.php_cs.dist @@ -83,6 +83,7 @@ return $config 'no_leading_import_slash' => true, 'no_leading_namespace_whitespace' => true, 'no_superfluous_elseif' => true, + 'no_superfluous_phpdoc_tags' => true, 'no_unneeded_curly_braces' => true, 'no_unneeded_final_method' => true, 'no_unreachable_default_argument_value' => true, diff --git a/phpstan-baseline.neon b/phpstan-baseline.neon index 2225f37f0e..7a8f697198 100644 --- a/phpstan-baseline.neon +++ b/phpstan-baseline.neon @@ -47,7 +47,7 @@ parameters: - message: "#^Parameter \\$class of method Faker\\\\ORM\\\\Doctrine\\\\ColumnTypeGuesser\\:\\:guessFormat\\(\\) has invalid typehint type Doctrine\\\\Common\\\\Persistence\\\\Mapping\\\\ClassMetadata\\.$#" - count: 2 + count: 1 path: src/Faker/ORM/Doctrine/ColumnTypeGuesser.php - @@ -67,7 +67,7 @@ parameters: - message: "#^Parameter \\$class of method Faker\\\\ORM\\\\Doctrine\\\\EntityPopulator\\:\\:__construct\\(\\) has invalid typehint type Doctrine\\\\Common\\\\Persistence\\\\Mapping\\\\ClassMetadata\\.$#" - count: 2 + count: 1 path: src/Faker/ORM/Doctrine/EntityPopulator.php - @@ -157,7 +157,7 @@ parameters: - message: "#^Parameter \\$manager of method Faker\\\\ORM\\\\Doctrine\\\\EntityPopulator\\:\\:execute\\(\\) has invalid typehint type Doctrine\\\\Common\\\\Persistence\\\\ObjectManager\\.$#" - count: 2 + count: 1 path: src/Faker/ORM/Doctrine/EntityPopulator.php - @@ -187,7 +187,7 @@ parameters: - message: "#^Parameter \\$manager of method Faker\\\\ORM\\\\Doctrine\\\\EntityPopulator\\:\\:generateId\\(\\) has invalid typehint type Doctrine\\\\Common\\\\Persistence\\\\ObjectManager\\.$#" - count: 2 + count: 1 path: src/Faker/ORM/Doctrine/EntityPopulator.php - @@ -202,7 +202,7 @@ parameters: - message: "#^Parameter \\$manager of method Faker\\\\ORM\\\\Doctrine\\\\Populator\\:\\:__construct\\(\\) has invalid typehint type Doctrine\\\\Common\\\\Persistence\\\\ObjectManager\\.$#" - count: 2 + count: 1 path: src/Faker/ORM/Doctrine/Populator.php - @@ -227,7 +227,7 @@ parameters: - message: "#^Parameter \\$mandango of method Faker\\\\ORM\\\\Mandango\\\\EntityPopulator\\:\\:guessColumnFormatters\\(\\) has invalid typehint type Mandango\\\\Mandango\\.$#" - count: 2 + count: 1 path: src/Faker/ORM/Mandango/EntityPopulator.php - @@ -237,7 +237,7 @@ parameters: - message: "#^Parameter \\$mandango of method Faker\\\\ORM\\\\Mandango\\\\EntityPopulator\\:\\:execute\\(\\) has invalid typehint type Mandango\\\\Mandango\\.$#" - count: 2 + count: 1 path: src/Faker/ORM/Mandango/EntityPopulator.php - @@ -252,12 +252,12 @@ parameters: - message: "#^Parameter \\$mandango of method Faker\\\\ORM\\\\Mandango\\\\Populator\\:\\:__construct\\(\\) has invalid typehint type Mandango\\\\Mandango\\.$#" - count: 2 + count: 1 path: src/Faker/ORM/Mandango/Populator.php - message: "#^Parameter \\$column of method Faker\\\\ORM\\\\Propel\\\\ColumnTypeGuesser\\:\\:guessFormat\\(\\) has invalid typehint type ColumnMap\\.$#" - count: 2 + count: 1 path: src/Faker/ORM/Propel/ColumnTypeGuesser.php - @@ -402,7 +402,7 @@ parameters: - message: "#^Parameter \\$columnMap of method Faker\\\\ORM\\\\Propel\\\\EntityPopulator\\:\\:isColumnBehavior\\(\\) has invalid typehint type ColumnMap\\.$#" - count: 2 + count: 1 path: src/Faker/ORM/Propel/EntityPopulator.php - @@ -447,7 +447,7 @@ parameters: - message: "#^Parameter \\$column of method Faker\\\\ORM\\\\Propel2\\\\ColumnTypeGuesser\\:\\:guessFormat\\(\\) has invalid typehint type Propel\\\\Runtime\\\\Map\\\\ColumnMap\\.$#" - count: 2 + count: 1 path: src/Faker/ORM/Propel2/ColumnTypeGuesser.php - @@ -597,7 +597,7 @@ parameters: - message: "#^Parameter \\$columnMap of method Faker\\\\ORM\\\\Propel2\\\\EntityPopulator\\:\\:isColumnBehavior\\(\\) has invalid typehint type Propel\\\\Runtime\\\\Map\\\\ColumnMap\\.$#" - count: 2 + count: 1 path: src/Faker/ORM/Propel2/EntityPopulator.php - @@ -652,12 +652,12 @@ parameters: - message: "#^Parameter \\$locator of method Faker\\\\ORM\\\\Spot\\\\EntityPopulator\\:\\:__construct\\(\\) has invalid typehint type Spot\\\\Locator\\.$#" - count: 2 + count: 1 path: src/Faker/ORM/Spot/EntityPopulator.php - message: "#^Parameter \\$mapper of method Faker\\\\ORM\\\\Spot\\\\EntityPopulator\\:\\:__construct\\(\\) has invalid typehint type Spot\\\\Mapper\\.$#" - count: 2 + count: 1 path: src/Faker/ORM/Spot/EntityPopulator.php - @@ -707,7 +707,7 @@ parameters: - message: "#^Parameter \\$locator of method Faker\\\\ORM\\\\Spot\\\\Populator\\:\\:__construct\\(\\) has invalid typehint type Spot\\\\Locator\\.$#" - count: 2 + count: 1 path: src/Faker/ORM/Spot/Populator.php - diff --git a/psalm.baseline.xml b/psalm.baseline.xml index e09c4805d3..5a10716611 100644 --- a/psalm.baseline.xml +++ b/psalm.baseline.xml @@ -45,9 +45,7 @@ - - ObjectManager|null - + $this->manager ObjectManager|null @@ -145,7 +143,6 @@ $this->locator - Locator|null Locator diff --git a/src/Faker/Calculator/Isbn.php b/src/Faker/Calculator/Isbn.php index b227b2d662..717fee7aa7 100644 --- a/src/Faker/Calculator/Isbn.php +++ b/src/Faker/Calculator/Isbn.php @@ -18,8 +18,6 @@ class Isbn * @param string $input ISBN without check-digit * * @throws \LengthException When wrong input length passed - * - * @return string */ public static function checksum(string $input): string { @@ -48,8 +46,6 @@ function (&$digit, $position) { * Checks whether the provided number is a valid ISBN-10 number * * @param string $isbn ISBN to check - * - * @return bool */ public static function isValid(string $isbn): bool { diff --git a/src/Faker/DefaultGenerator.php b/src/Faker/DefaultGenerator.php index 5382ca06f7..4cb7d1ad64 100644 --- a/src/Faker/DefaultGenerator.php +++ b/src/Faker/DefaultGenerator.php @@ -17,8 +17,6 @@ public function __construct($default = null) /** * @param string $attribute - * - * @return mixed */ public function __get($attribute) { @@ -28,8 +26,6 @@ public function __get($attribute) /** * @param string $method * @param array $attributes - * - * @return mixed */ public function __call($method, $attributes) { diff --git a/src/Faker/Documentor.php b/src/Faker/Documentor.php index ea39233ce6..0e51818119 100644 --- a/src/Faker/Documentor.php +++ b/src/Faker/Documentor.php @@ -6,9 +6,6 @@ class Documentor { protected $generator; - /** - * @param Generator $generator - */ public function __construct(Generator $generator) { $this->generator = $generator; diff --git a/src/Faker/Generator.php b/src/Faker/Generator.php index a510ec3dbf..415911f709 100644 --- a/src/Faker/Generator.php +++ b/src/Faker/Generator.php @@ -287,8 +287,6 @@ protected function callFormatWithMatches($matches) /** * @param string $attribute - * - * @return mixed */ public function __get($attribute) { @@ -298,8 +296,6 @@ public function __get($attribute) /** * @param string $method * @param array $attributes - * - * @return mixed */ public function __call($method, $attributes) { diff --git a/src/Faker/Guesser/Name.php b/src/Faker/Guesser/Name.php index 6bf10cda2f..7d1ef12073 100644 --- a/src/Faker/Guesser/Name.php +++ b/src/Faker/Guesser/Name.php @@ -8,9 +8,6 @@ class Name { protected $generator; - /** - * @param \Faker\Generator $generator - */ public function __construct(\Faker\Generator $generator) { $this->generator = $generator; diff --git a/src/Faker/ORM/CakePHP/Populator.php b/src/Faker/ORM/CakePHP/Populator.php index b25b0b241e..ac195fbdd4 100644 --- a/src/Faker/ORM/CakePHP/Populator.php +++ b/src/Faker/ORM/CakePHP/Populator.php @@ -9,9 +9,6 @@ class Populator protected $quantities = []; protected $guessers = []; - /** - * @param \Faker\Generator $generator - */ public function __construct(\Faker\Generator $generator) { $this->generator = $generator; diff --git a/src/Faker/ORM/Doctrine/ColumnTypeGuesser.php b/src/Faker/ORM/Doctrine/ColumnTypeGuesser.php index d971aa766b..594a524973 100644 --- a/src/Faker/ORM/Doctrine/ColumnTypeGuesser.php +++ b/src/Faker/ORM/Doctrine/ColumnTypeGuesser.php @@ -15,8 +15,6 @@ public function __construct(Generator $generator) } /** - * @param ClassMetadata $class - * * @return \Closure|null */ public function guessFormat($fieldName, ClassMetadata $class) diff --git a/src/Faker/ORM/Doctrine/EntityPopulator.php b/src/Faker/ORM/Doctrine/EntityPopulator.php index 0202ec15bc..22997078cb 100644 --- a/src/Faker/ORM/Doctrine/EntityPopulator.php +++ b/src/Faker/ORM/Doctrine/EntityPopulator.php @@ -23,9 +23,6 @@ class EntityPopulator */ protected $modifiers = []; - /** - * @param ClassMetadata $class - */ public function __construct(ClassMetadata $class) { $this->class = $class; @@ -57,9 +54,6 @@ public function mergeColumnFormattersWith($columnFormatters) $this->columnFormatters = array_merge($this->columnFormatters, $columnFormatters); } - /** - * @param array $modifiers - */ public function setModifiers(array $modifiers) { $this->modifiers = $modifiers; @@ -73,17 +67,12 @@ public function getModifiers() return $this->modifiers; } - /** - * @param array $modifiers - */ public function mergeModifiersWith(array $modifiers) { $this->modifiers = array_merge($this->modifiers, $modifiers); } /** - * @param \Faker\Generator $generator - * * @return array */ public function guessColumnFormatters(\Faker\Generator $generator) @@ -177,8 +166,7 @@ public function guessColumnFormatters(\Faker\Generator $generator) /** * Insert one new record using the Entity class. * - * @param ObjectManager $manager - * @param bool $generateId + * @param bool $generateId * * @return EntityPopulator */ @@ -238,8 +226,6 @@ private function callMethods($obj, $insertedEntities) } /** - * @param ObjectManager $manager - * * @return int|null */ private function generateId($obj, $column, ObjectManager $manager) diff --git a/src/Faker/ORM/Doctrine/Populator.php b/src/Faker/ORM/Doctrine/Populator.php index b097bfb228..9e7a2030b3 100644 --- a/src/Faker/ORM/Doctrine/Populator.php +++ b/src/Faker/ORM/Doctrine/Populator.php @@ -32,9 +32,7 @@ class Populator /** * Populator constructor. * - * @param Generator $generator - * @param ObjectManager|null $manager - * @param int $batchSize + * @param int $batchSize */ public function __construct(Generator $generator, ObjectManager $manager = null, $batchSize = 1000) { diff --git a/src/Faker/ORM/Mandango/ColumnTypeGuesser.php b/src/Faker/ORM/Mandango/ColumnTypeGuesser.php index 06fece41b9..111d208048 100644 --- a/src/Faker/ORM/Mandango/ColumnTypeGuesser.php +++ b/src/Faker/ORM/Mandango/ColumnTypeGuesser.php @@ -11,9 +11,6 @@ class ColumnTypeGuesser */ protected $generator; - /** - * @param Generator $generator - */ public function __construct(Generator $generator) { $this->generator = $generator; diff --git a/src/Faker/ORM/Mandango/EntityPopulator.php b/src/Faker/ORM/Mandango/EntityPopulator.php index 0d2030b30d..b54e5ee830 100644 --- a/src/Faker/ORM/Mandango/EntityPopulator.php +++ b/src/Faker/ORM/Mandango/EntityPopulator.php @@ -48,9 +48,6 @@ public function mergeColumnFormattersWith($columnFormatters) } /** - * @param \Faker\Generator $generator - * @param Mandango $mandango - * * @return array */ public function guessColumnFormatters(\Faker\Generator $generator, Mandango $mandango) @@ -97,8 +94,6 @@ public function guessColumnFormatters(\Faker\Generator $generator, Mandango $man /** * Insert one new record using the Entity class. - * - * @param Mandango $mandango */ public function execute(Mandango $mandango, $insertedEntities) { diff --git a/src/Faker/ORM/Mandango/Populator.php b/src/Faker/ORM/Mandango/Populator.php index 8af6ab33fc..6dc26c035d 100644 --- a/src/Faker/ORM/Mandango/Populator.php +++ b/src/Faker/ORM/Mandango/Populator.php @@ -15,10 +15,6 @@ class Populator protected $entities = []; protected $quantities = []; - /** - * @param \Faker\Generator $generator - * @param Mandango $mandango - */ public function __construct(\Faker\Generator $generator, Mandango $mandango) { $this->generator = $generator; diff --git a/src/Faker/ORM/Propel/ColumnTypeGuesser.php b/src/Faker/ORM/Propel/ColumnTypeGuesser.php index 574dc55afd..8e13ca123a 100644 --- a/src/Faker/ORM/Propel/ColumnTypeGuesser.php +++ b/src/Faker/ORM/Propel/ColumnTypeGuesser.php @@ -9,17 +9,12 @@ class ColumnTypeGuesser { protected $generator; - /** - * @param \Faker\Generator $generator - */ public function __construct(\Faker\Generator $generator) { $this->generator = $generator; } /** - * @param ColumnMap $column - * * @return \Closure|null */ public function guessFormat(ColumnMap $column) diff --git a/src/Faker/ORM/Propel/EntityPopulator.php b/src/Faker/ORM/Propel/EntityPopulator.php index 9101a0e208..7e7553d1a1 100644 --- a/src/Faker/ORM/Propel/EntityPopulator.php +++ b/src/Faker/ORM/Propel/EntityPopulator.php @@ -49,8 +49,6 @@ public function mergeColumnFormattersWith($columnFormatters) } /** - * @param \Faker\Generator $generator - * * @return array */ public function guessColumnFormatters(\Faker\Generator $generator) @@ -98,8 +96,6 @@ public function guessColumnFormatters(\Faker\Generator $generator) } /** - * @param ColumnMap $columnMap - * * @return bool */ protected function isColumnBehavior(ColumnMap $columnMap) @@ -150,8 +146,6 @@ public function mergeModifiersWith($modifiers) } /** - * @param \Faker\Generator $generator - * * @return array */ public function guessModifiers(\Faker\Generator $generator) diff --git a/src/Faker/ORM/Propel/Populator.php b/src/Faker/ORM/Propel/Populator.php index d9299d9cb1..4a0c673a88 100644 --- a/src/Faker/ORM/Propel/Populator.php +++ b/src/Faker/ORM/Propel/Populator.php @@ -12,9 +12,6 @@ class Populator protected $entities = []; protected $quantities = []; - /** - * @param \Faker\Generator $generator - */ public function __construct(\Faker\Generator $generator) { $this->generator = $generator; diff --git a/src/Faker/ORM/Propel2/ColumnTypeGuesser.php b/src/Faker/ORM/Propel2/ColumnTypeGuesser.php index caebc5d6da..fe08b63f0d 100644 --- a/src/Faker/ORM/Propel2/ColumnTypeGuesser.php +++ b/src/Faker/ORM/Propel2/ColumnTypeGuesser.php @@ -9,17 +9,12 @@ class ColumnTypeGuesser { protected $generator; - /** - * @param \Faker\Generator $generator - */ public function __construct(\Faker\Generator $generator) { $this->generator = $generator; } /** - * @param ColumnMap $column - * * @return \Closure|null */ public function guessFormat(ColumnMap $column) diff --git a/src/Faker/ORM/Propel2/EntityPopulator.php b/src/Faker/ORM/Propel2/EntityPopulator.php index 9c01bac481..391dfed4b5 100644 --- a/src/Faker/ORM/Propel2/EntityPopulator.php +++ b/src/Faker/ORM/Propel2/EntityPopulator.php @@ -49,8 +49,6 @@ public function mergeColumnFormattersWith($columnFormatters) } /** - * @param \Faker\Generator $generator - * * @return array */ public function guessColumnFormatters(\Faker\Generator $generator) @@ -100,8 +98,6 @@ public function guessColumnFormatters(\Faker\Generator $generator) } /** - * @param ColumnMap $columnMap - * * @return bool */ protected function isColumnBehavior(ColumnMap $columnMap) @@ -152,8 +148,6 @@ public function mergeModifiersWith($modifiers) } /** - * @param \Faker\Generator $generator - * * @return array */ public function guessModifiers(\Faker\Generator $generator) diff --git a/src/Faker/ORM/Propel2/Populator.php b/src/Faker/ORM/Propel2/Populator.php index 17ecb5e918..25cc5e33fa 100644 --- a/src/Faker/ORM/Propel2/Populator.php +++ b/src/Faker/ORM/Propel2/Populator.php @@ -15,9 +15,6 @@ class Populator protected $entities = []; protected $quantities = []; - /** - * @param \Faker\Generator $generator - */ public function __construct(\Faker\Generator $generator) { $this->generator = $generator; diff --git a/src/Faker/ORM/Spot/ColumnTypeGuesser.php b/src/Faker/ORM/Spot/ColumnTypeGuesser.php index fa6f0bcca3..0b75497f91 100644 --- a/src/Faker/ORM/Spot/ColumnTypeGuesser.php +++ b/src/Faker/ORM/Spot/ColumnTypeGuesser.php @@ -10,8 +10,6 @@ class ColumnTypeGuesser /** * ColumnTypeGuesser constructor. - * - * @param Generator $generator */ public function __construct(Generator $generator) { @@ -19,8 +17,6 @@ public function __construct(Generator $generator) } /** - * @param array $field - * * @return \Closure|null */ public function guessFormat(array $field) diff --git a/src/Faker/ORM/Spot/EntityPopulator.php b/src/Faker/ORM/Spot/EntityPopulator.php index d995353671..c0afbb7cae 100644 --- a/src/Faker/ORM/Spot/EntityPopulator.php +++ b/src/Faker/ORM/Spot/EntityPopulator.php @@ -43,9 +43,7 @@ class EntityPopulator protected $useExistingData = false; /** - * @param Mapper $mapper - * @param Locator $locator - * @param bool $useExistingData + * @param bool $useExistingData */ public function __construct(Mapper $mapper, Locator $locator, $useExistingData = false) { @@ -80,9 +78,6 @@ public function mergeColumnFormattersWith($columnFormatters) $this->columnFormatters = array_merge($this->columnFormatters, $columnFormatters); } - /** - * @param array $modifiers - */ public function setModifiers(array $modifiers) { $this->modifiers = $modifiers; @@ -96,17 +91,12 @@ public function getModifiers() return $this->modifiers; } - /** - * @param array $modifiers - */ public function mergeModifiersWith(array $modifiers) { $this->modifiers = array_merge($this->modifiers, $modifiers); } /** - * @param Generator $generator - * * @return array */ public function guessColumnFormatters(Generator $generator) diff --git a/src/Faker/ORM/Spot/Populator.php b/src/Faker/ORM/Spot/Populator.php index 1d0a7be632..67202bb3b7 100644 --- a/src/Faker/ORM/Spot/Populator.php +++ b/src/Faker/ORM/Spot/Populator.php @@ -16,9 +16,6 @@ class Populator /** * Populator constructor. - * - * @param \Faker\Generator $generator - * @param Locator|null $locator */ public function __construct(\Faker\Generator $generator, Locator $locator = null) { diff --git a/src/Faker/Provider/Base.php b/src/Faker/Provider/Base.php index 6aa0cc6831..5572264f08 100644 --- a/src/Faker/Provider/Base.php +++ b/src/Faker/Provider/Base.php @@ -19,9 +19,6 @@ class Base */ protected $unique; - /** - * @param \Faker\Generator $generator - */ public function __construct(Generator $generator) { $this->generator = $generator; @@ -153,10 +150,6 @@ public static function numberBetween($int1 = 0, $int2 = 2147483647) /** * Returns the passed value - * - * @param mixed $value - * - * @return mixed */ public static function passthrough($value) { @@ -236,8 +229,6 @@ public static function randomElements($array = ['a', 'b', 'c'], $count = 1, $all * Returns a random element from a passed array * * @param array $array - * - * @return mixed */ public static function randomElement($array = ['a', 'b', 'c']) { diff --git a/src/Faker/Provider/DateTime.php b/src/Faker/Provider/DateTime.php index 4ca8275bfb..9875e21cce 100644 --- a/src/Faker/Provider/DateTime.php +++ b/src/Faker/Provider/DateTime.php @@ -340,7 +340,6 @@ public static function timezone() /** * Internal method to set the time zone on a DateTime. * - * @param \DateTime $dt * @param string|null $timezone * * @return \DateTime diff --git a/src/Faker/Provider/de_AT/Person.php b/src/Faker/Provider/de_AT/Person.php index 36e1eb06c1..474848e471 100644 --- a/src/Faker/Provider/de_AT/Person.php +++ b/src/Faker/Provider/de_AT/Person.php @@ -125,8 +125,6 @@ public static function suffix() * * @link https://de.wikipedia.org/wiki/Sozialversicherungsnummer#.C3.96sterreich * - * @param \DateTime|null $birthdate - * * @return string */ public static function ssn(\DateTime $birthdate = null) diff --git a/src/Faker/Provider/hy_AM/PhoneNumber.php b/src/Faker/Provider/hy_AM/PhoneNumber.php index f4317c9cf5..c3ee853f5b 100644 --- a/src/Faker/Provider/hy_AM/PhoneNumber.php +++ b/src/Faker/Provider/hy_AM/PhoneNumber.php @@ -29,9 +29,6 @@ public function code() return static::randomElement(static::$codes); } - /** - * @return mixed - */ public function numberFormat() { return static::randomElement(static::$numberFormats); diff --git a/src/Faker/Provider/pl_PL/LicensePlate.php b/src/Faker/Provider/pl_PL/LicensePlate.php index ce8ed6a632..d59c93dec4 100644 --- a/src/Faker/Provider/pl_PL/LicensePlate.php +++ b/src/Faker/Provider/pl_PL/LicensePlate.php @@ -507,8 +507,6 @@ class LicensePlate extends Base * @param bool $special whether special license plates should be included * @param array|null $voivodeships list of voivodeships license plate should be generated from * @param array|null $counties list of counties license plate should be generated from - * - * @return string */ public static function licensePlate( bool $special = false, @@ -530,11 +528,6 @@ public static function licensePlate( /** * Selects random area from the list of available and requested. - * - * @param array $available - * @param array|null $requested - * - * @return mixed */ protected static function selectRandomArea(array $available, ?array $requested) { diff --git a/src/Faker/Provider/pt_BR/PhoneNumber.php b/src/Faker/Provider/pt_BR/PhoneNumber.php index cb04d8abb9..f1d3e44989 100644 --- a/src/Faker/Provider/pt_BR/PhoneNumber.php +++ b/src/Faker/Provider/pt_BR/PhoneNumber.php @@ -74,8 +74,6 @@ public static function landline($formatted = true) * Randomizes between cellphone and landline numbers. * * @param bool $formatted [def: true] If it should return a formatted number or not. - * - * @return mixed */ public static function phone($formatted = true) { @@ -132,8 +130,6 @@ public static function landlineNumber($formatted = true) /** * Randomizes between complete cellphone and landline numbers. - * - * @return mixed */ public function phoneNumber() { @@ -144,8 +140,6 @@ public function phoneNumber() /** * Randomizes between complete cellphone and landline numbers, cleared from formatting symbols. - * - * @return mixed */ public static function phoneNumberCleared() { diff --git a/src/Faker/Provider/ro_RO/Person.php b/src/Faker/Provider/ro_RO/Person.php index 65b160212d..51963cdab2 100644 --- a/src/Faker/Provider/ro_RO/Person.php +++ b/src/Faker/Provider/ro_RO/Person.php @@ -193,9 +193,8 @@ protected function getDateOfBirth($dateOfBirth) /** * https://ro.wikipedia.org/wiki/Cod_numeric_personal#S * - * @param \DateTime $dateOfBirth - * @param bool $isResident - * @param string $gender + * @param bool $isResident + * @param string $gender * * @return int */ diff --git a/src/Faker/UniqueGenerator.php b/src/Faker/UniqueGenerator.php index c6eb9bcf6a..00f30e03c1 100644 --- a/src/Faker/UniqueGenerator.php +++ b/src/Faker/UniqueGenerator.php @@ -13,8 +13,7 @@ class UniqueGenerator protected $uniques = []; /** - * @param Generator $generator - * @param int $maxRetries + * @param int $maxRetries */ public function __construct(Generator $generator, $maxRetries = 10000) { @@ -26,8 +25,6 @@ public function __construct(Generator $generator, $maxRetries = 10000) * Catch and proxy all generator calls but return only unique values * * @param string $attribute - * - * @return mixed */ public function __get($attribute) { @@ -39,8 +36,6 @@ public function __get($attribute) * * @param string $name * @param array $arguments - * - * @return mixed */ public function __call($name, $arguments) { diff --git a/src/Faker/ValidGenerator.php b/src/Faker/ValidGenerator.php index 66daf4a018..cfee85315f 100644 --- a/src/Faker/ValidGenerator.php +++ b/src/Faker/ValidGenerator.php @@ -13,7 +13,6 @@ class ValidGenerator protected $maxRetries; /** - * @param Generator $generator * @param callable|null $validator * @param int $maxRetries */ @@ -35,8 +34,6 @@ public function __construct(Generator $generator, $validator = null, $maxRetries * Catch and proxy all generator calls but return only valid values * * @param string $attribute - * - * @return mixed */ public function __get($attribute) { @@ -48,8 +45,6 @@ public function __get($attribute) * * @param string $name * @param array $arguments - * - * @return mixed */ public function __call($name, $arguments) {