Merge branch '2.7' into 2.8

nicolas-grekas · nicolas-grekas · commit 5778b5ea3182 · 2017-10-24T15:48:52.000+02:00
* 2.7:
  [DI] minor docblock fixes
diff --git a/ConstraintValidatorFactoryInterface.php b/ConstraintValidatorFactoryInterface.php
@@ -21,8 +21,6 @@ interface ConstraintValidatorFactoryInterface
      * Given a Constraint, this returns the ConstraintValidatorInterface
      * object that should be used to verify its validity.
      *
-     * @param Constraint $constraint The source constraint
-     *
      * @return ConstraintValidatorInterface
      */
     public function getInstance(Constraint $constraint);
diff --git a/ConstraintViolation.php b/ConstraintViolation.php
@@ -18,54 +18,15 @@
  */
 class ConstraintViolation implements ConstraintViolationInterface
 {
-    /**
-     * @var string
-     */
     private $message;
-
-    /**
-     * @var string
-     */
     private $messageTemplate;
-
-    /**
-     * @var array
-     */
     private $parameters;
-
-    /**
-     * @var int|null
-     */
     private $plural;
-
-    /**
-     * @var mixed
-     */
     private $root;
-
-    /**
-     * @var string
-     */
     private $propertyPath;
-
-    /**
-     * @var mixed
-     */
     private $invalidValue;
-
-    /**
-     * @var Constraint|null
-     */
     private $constraint;
-
-    /**
-     * @var mixed
-     */
     private $code;
-
-    /**
-     * @var mixed
-     */
     private $cause;
 
     /**
diff --git a/ConstraintViolationListInterface.php b/ConstraintViolationListInterface.php
@@ -20,15 +20,11 @@ interface ConstraintViolationListInterface extends \Traversable, \Countable, \Ar
 {
     /**
      * Adds a constraint violation to this list.
-     *
-     * @param ConstraintViolationInterface $violation The violation to add
      */
     public function add(ConstraintViolationInterface $violation);
 
     /**
      * Merges an existing violation list into this list.
-     *
-     * @param ConstraintViolationListInterface $otherList The list to merge
      */
     public function addAll(ConstraintViolationListInterface $otherList);
 
diff --git a/Constraints/IbanValidator.php b/Constraints/IbanValidator.php
@@ -35,8 +35,6 @@ class IbanValidator extends ConstraintValidator
      * included within it, a bank identifier with a fixed position and a fixed length per country
      *
      * @see https://www.swift.com/sites/default/files/resources/iban_registry.pdf
-     *
-     * @var array
      */
     private static $formats = array(
         'AD' => 'AD\d{2}\d{4}\d{4}[\dA-Z]{12}', // Andorra
diff --git a/Mapping/Cache/CacheInterface.php b/Mapping/Cache/CacheInterface.php
@@ -38,8 +38,6 @@ public function read($class);
 
     /**
      * Stores a class metadata in the cache.
-     *
-     * @param ClassMetadata $metadata A Class Metadata
      */
     public function write(ClassMetadata $metadata);
 }
diff --git a/Mapping/Cache/DoctrineCache.php b/Mapping/Cache/DoctrineCache.php
@@ -23,21 +23,11 @@ final class DoctrineCache implements CacheInterface
 {
     private $cache;
 
-    /**
-     * Creates a new Doctrine cache.
-     *
-     * @param Cache $cache The cache to adapt
-     */
     public function __construct(Cache $cache)
     {
         $this->cache = $cache;
     }
 
-    /**
-     * Sets the cache to adapt.
-     *
-     * @param Cache $cache The cache to adapt
-     */
     public function setCache(Cache $cache)
     {
         $this->cache = $cache;
diff --git a/Mapping/ClassMetadata.php b/Mapping/ClassMetadata.php
@@ -376,8 +376,6 @@ public function addGetterMethodConstraints($property, $method, array $constraint
 
     /**
      * Merges the constraints of the given metadata into this object.
-     *
-     * @param ClassMetadata $source The source metadata
      */
     public function mergeConstraints(ClassMetadata $source)
     {
@@ -590,11 +588,6 @@ public function getCascadingStrategy()
         return CascadingStrategy::NONE;
     }
 
-    /**
-     * Adds a property metadata.
-     *
-     * @param PropertyMetadataInterface $metadata
-     */
     private function addPropertyMetadata(PropertyMetadataInterface $metadata)
     {
         $property = $metadata->getPropertyName();
diff --git a/Mapping/GenericMetadata.php b/Mapping/GenericMetadata.php
@@ -119,8 +119,6 @@ public function __clone()
      *    if $traverse is enabled, but $deep is disabled
      *  - {@link TraversalStrategy::NONE} if $traverse is disabled
      *
-     * @param Constraint $constraint The constraint to add
-     *
      * @return $this
      *
      * @throws ConstraintDefinitionException When trying to add the
diff --git a/Mapping/Loader/AbstractLoader.php b/Mapping/Loader/AbstractLoader.php
@@ -32,9 +32,6 @@ abstract class AbstractLoader implements LoaderInterface
      */
     const DEFAULT_NAMESPACE = '\\Symfony\\Component\\Validator\\Constraints\\';
 
-    /**
-     * @var array
-     */
     protected $namespaces = array();
 
     /**
diff --git a/Mapping/Loader/LoaderInterface.php b/Mapping/Loader/LoaderInterface.php
@@ -23,8 +23,6 @@ interface LoaderInterface
     /**
      * Loads validation metadata into a {@link ClassMetadata} instance.
      *
-     * @param ClassMetadata $metadata The metadata to load
-     *
      * @return bool Whether the loader succeeded
      */
     public function loadClassMetadata(ClassMetadata $metadata);
diff --git a/Tests/Validator/Abstract2Dot5ApiTest.php b/Tests/Validator/Abstract2Dot5ApiTest.php
@@ -40,9 +40,6 @@ abstract class Abstract2Dot5ApiTest extends AbstractValidatorTest
     protected $validator;
 
     /**
-     * @param MetadataFactoryInterface $metadataFactory
-     * @param array                    $objectInitializers
-     *
      * @return ValidatorInterface
      */
     abstract protected function createValidator(MetadataFactoryInterface $metadataFactory, array $objectInitializers = array());
diff --git a/Validator/ValidatorInterface.php b/Validator/ValidatorInterface.php
@@ -91,8 +91,6 @@ public function startContext();
      * The returned validator adds all generated violations to the given
      * context.
      *
-     * @param ExecutionContextInterface $context The execution context
-     *
      * @return ContextualValidatorInterface The validator for that context
      */
     public function inContext(ExecutionContextInterface $context);
diff --git a/ValidatorBuilderInterface.php b/ValidatorBuilderInterface.php
@@ -26,16 +26,14 @@ interface ValidatorBuilderInterface
     /**
      * Adds an object initializer to the validator.
      *
-     * @param ObjectInitializerInterface $initializer The initializer
-     *
      * @return $this
      */
     public function addObjectInitializer(ObjectInitializerInterface $initializer);
 
     /**
      * Adds a list of object initializers to the validator.
      *
-     * @param array $initializers The initializer
+     * @param ObjectInitializerInterface[] $initializers
      *
      * @return $this
      */
@@ -53,7 +51,7 @@ public function addXmlMapping($path);
     /**
      * Adds a list of XML constraint mapping files to the validator.
      *
-     * @param array $paths The paths to the mapping files
+     * @param string[] $paths The paths to the mapping files
      *
      * @return $this
      */
@@ -71,7 +69,7 @@ public function addYamlMapping($path);
     /**
      * Adds a list of YAML constraint mappings file to the validator.
      *
-     * @param array $paths The paths to the mapping files
+     * @param string[] $paths The paths to the mapping files
      *
      * @return $this
      */
@@ -89,7 +87,7 @@ public function addMethodMapping($methodName);
     /**
      * Enables constraint mapping using the given static methods.
      *
-     * @param array $methodNames The names of the methods
+     * @param string[] $methodNames The names of the methods
      *
      * @return $this
      */
@@ -98,8 +96,6 @@ public function addMethodMappings(array $methodNames);
     /**
      * Enables annotation based constraint mapping.
      *
-     * @param Reader $annotationReader The annotation reader to be used
-     *
      * @return $this
      */
     public function enableAnnotationMapping(Reader $annotationReader = null);
@@ -114,35 +110,27 @@ public function disableAnnotationMapping();
     /**
      * Sets the class metadata factory used by the validator.
      *
-     * @param MetadataFactoryInterface $metadataFactory The metadata factory
-     *
      * @return $this
      */
     public function setMetadataFactory(MetadataFactoryInterface $metadataFactory);
 
     /**
      * Sets the cache for caching class metadata.
      *
-     * @param CacheInterface $cache The cache instance
-     *
      * @return $this
      */
     public function setMetadataCache(CacheInterface $cache);
 
     /**
      * Sets the constraint validator factory used by the validator.
      *
-     * @param ConstraintValidatorFactoryInterface $validatorFactory The validator factory
-     *
      * @return $this
      */
     public function setConstraintValidatorFactory(ConstraintValidatorFactoryInterface $validatorFactory);
 
     /**
      * Sets the translator used for translating violation messages.
      *
-     * @param TranslatorInterface $translator The translator instance
-     *
      * @return $this
      */
     public function setTranslator(TranslatorInterface $translator);

Original file line number	Diff line number	Diff line change
`@@ -21,8 +21,6 @@ interface ConstraintValidatorFactoryInterface`
`21`	`21`	`* Given a Constraint, this returns the ConstraintValidatorInterface`
`22`	`22`	`* object that should be used to verify its validity.`
`23`	`23`	`*`
`24`		`- * @param Constraint $constraint The source constraint`
`25`		`- *`
`26`	`24`	`* @return ConstraintValidatorInterface`
`27`	`25`	`*/`
`28`	`26`	`public function getInstance(Constraint $constraint);`
Original file line number	Diff line number	Diff line change
`@@ -35,8 +35,6 @@ class IbanValidator extends ConstraintValidator`
`35`	`35`	`* included within it, a bank identifier with a fixed position and a fixed length per country`
`36`	`36`	`*`
`37`	`37`	`* @see https://www.swift.com/sites/default/files/resources/iban_registry.pdf`
`38`		`- *`
`39`		`- * @var array`
`40`	`38`	`*/`
`41`	`39`	`private static $formats = array(`
`42`	`40`	`'AD' => 'AD\d{2}\d{4}\d{4}[\dA-Z]{12}', // Andorra`
Original file line number	Diff line number	Diff line change
`@@ -38,8 +38,6 @@ public function read($class);`
`38`	`38`
`39`	`39`	`/**`
`40`	`40`	`* Stores a class metadata in the cache.`
`41`		`- *`
`42`		`- * @param ClassMetadata $metadata A Class Metadata`
`43`	`41`	`*/`
`44`	`42`	`public function write(ClassMetadata $metadata);`
`45`	`43`	`}`
Original file line number	Diff line number	Diff line change
`@@ -23,21 +23,11 @@ final class DoctrineCache implements CacheInterface`
`23`	`23`	`{`
`24`	`24`	`private $cache;`
`25`	`25`
`26`		`- /**`
`27`		`- * Creates a new Doctrine cache.`
`28`		`- *`
`29`		`- * @param Cache $cache The cache to adapt`
`30`		`- */`
`31`	`26`	`public function __construct(Cache $cache)`
`32`	`27`	`{`
`33`	`28`	`$this->cache = $cache;`
`34`	`29`	`}`
`35`	`30`
`36`		`- /**`
`37`		`- * Sets the cache to adapt.`
`38`		`- *`
`39`		`- * @param Cache $cache The cache to adapt`
`40`		`- */`
`41`	`31`	`public function setCache(Cache $cache)`
`42`	`32`	`{`
`43`	`33`	`$this->cache = $cache;`
Original file line number	Diff line number	Diff line change
`@@ -376,8 +376,6 @@ public function addGetterMethodConstraints($property, $method, array $constraint`
`376`	`376`
`377`	`377`	`/**`
`378`	`378`	`* Merges the constraints of the given metadata into this object.`
`379`		`- *`
`380`		`- * @param ClassMetadata $source The source metadata`
`381`	`379`	`*/`
`382`	`380`	`public function mergeConstraints(ClassMetadata $source)`
`383`	`381`	`{`
`@@ -590,11 +588,6 @@ public function getCascadingStrategy()`
`590`	`588`	`return CascadingStrategy::NONE;`
`591`	`589`	`}`
`592`	`590`
`593`		`- /**`
`594`		`- * Adds a property metadata.`
`595`		`- *`
`596`		`- * @param PropertyMetadataInterface $metadata`
`597`		`- */`
`598`	`591`	`private function addPropertyMetadata(PropertyMetadataInterface $metadata)`
`599`	`592`	`{`
`600`	`593`	`$property = $metadata->getPropertyName();`
Original file line number	Diff line number	Diff line change
`@@ -119,8 +119,6 @@ public function __clone()`
`119`	`119`	`* if $traverse is enabled, but $deep is disabled`
`120`	`120`	`* - {@link TraversalStrategy::NONE} if $traverse is disabled`
`121`	`121`	`*`
`122`		`- * @param Constraint $constraint The constraint to add`
`123`		`- *`
`124`	`122`	`* @return $this`
`125`	`123`	`*`
`126`	`124`	`* @throws ConstraintDefinitionException When trying to add the`
Original file line number	Diff line number	Diff line change
`@@ -23,8 +23,6 @@ interface LoaderInterface`
`23`	`23`	`/**`
`24`	`24`	`* Loads validation metadata into a {@link ClassMetadata} instance.`
`25`	`25`	`*`
`26`		`- * @param ClassMetadata $metadata The metadata to load`
`27`		`- *`
`28`	`26`	`* @return bool Whether the loader succeeded`
`29`	`27`	`*/`
`30`	`28`	`public function loadClassMetadata(ClassMetadata $metadata);`