From 33e9eec15baf9645b91801b6e36ef7b6aec4edd9 Mon Sep 17 00:00:00 2001 From: Nicolas Grekas Date: Sun, 22 Oct 2017 09:42:21 -0700 Subject: [PATCH] [DI] minor docblock fixes --- AbstractRendererEngine.php | 14 --------- AbstractTypeExtension.php | 2 -- Button.php | 2 -- ButtonBuilder.php | 10 ------- .../Factory/CachingFactoryDecorator.php | 8 ----- .../Factory/PropertyAccessDecorator.php | 13 --------- ChoiceList/View/ChoiceView.php | 2 -- Extension/Core/ChoiceList/ChoiceList.php | 8 ----- .../Core/DataMapper/PropertyPathMapper.php | 8 ----- .../ChoiceToValueTransformer.php | 3 -- .../ChoicesToValuesTransformer.php | 7 ----- .../DataTransformer/DataTransformerChain.php | 7 +---- .../ValueToDuplicatesTransformer.php | 2 -- .../Core/EventListener/ResizeFormListener.php | 29 +++++-------------- .../EventListener/DataCollectorListener.php | 4 --- Extension/DataCollector/FormDataCollector.php | 3 -- .../FormDataCollectorInterface.php | 16 ---------- Extension/DataCollector/FormDataExtractor.php | 2 -- .../FormDataExtractorInterface.php | 8 ----- .../Type/FormTypeHttpFoundationExtension.php | 10 ------- .../Validator/Constraints/FormValidator.php | 2 -- Extension/Validator/ValidatorTypeGuesser.php | 8 ----- .../ViolationMapper/ViolationMapper.php | 2 -- Form.php | 2 -- FormConfigBuilderInterface.php | 12 -------- FormError.php | 28 +----------------- FormFactory.php | 2 -- FormFactoryBuilderInterface.php | 10 ------- FormInterface.php | 4 --- FormRenderer.php | 22 -------------- FormTypeExtensionInterface.php | 11 ------- FormView.php | 4 --- Guess/TypeGuess.php | 11 ------- NativeRequestHandler.php | 7 ----- ReversedTransformer.php | 10 ------- 35 files changed, 9 insertions(+), 284 deletions(-) diff --git a/AbstractRendererEngine.php b/AbstractRendererEngine.php index 7519011225..452e24f1d1 100644 --- a/AbstractRendererEngine.php +++ b/AbstractRendererEngine.php @@ -23,24 +23,10 @@ abstract class AbstractRendererEngine implements FormRendererEngineInterface */ const CACHE_KEY_VAR = 'cache_key'; - /** - * @var array - */ protected $defaultThemes; - - /** - * @var array - */ protected $themes = array(); - - /** - * @var array - */ protected $resources = array(); - /** - * @var array - */ private $resourceHierarchyLevels = array(); /** diff --git a/AbstractTypeExtension.php b/AbstractTypeExtension.php index 27783a1a4d..268c94354c 100644 --- a/AbstractTypeExtension.php +++ b/AbstractTypeExtension.php @@ -54,8 +54,6 @@ public function setDefaultOptions(OptionsResolverInterface $resolver) /** * Configures the options for this type. - * - * @param OptionsResolver $resolver The resolver for the options */ public function configureOptions(OptionsResolver $resolver) { diff --git a/Button.php b/Button.php index 9a5f8a60bd..3dbe110fcb 100644 --- a/Button.php +++ b/Button.php @@ -273,8 +273,6 @@ public function getPropertyPath() /** * Unsupported method. * - * @param FormError $error - * * @throws BadMethodCallException */ public function addError(FormError $error) diff --git a/ButtonBuilder.php b/ButtonBuilder.php index ad311db290..ec3a421d54 100644 --- a/ButtonBuilder.php +++ b/ButtonBuilder.php @@ -184,8 +184,6 @@ public function addEventListener($eventName, $listener, $priority = 0) * * This method should not be invoked. * - * @param EventSubscriberInterface $subscriber - * * @throws BadMethodCallException */ public function addEventSubscriber(EventSubscriberInterface $subscriber) @@ -272,8 +270,6 @@ public function setAttributes(array $attributes) * * This method should not be invoked. * - * @param DataMapperInterface $dataMapper - * * @throws BadMethodCallException */ public function setDataMapper(DataMapperInterface $dataMapper = null) @@ -413,8 +409,6 @@ public function setCompound($compound) /** * Sets the type of the button. * - * @param ResolvedFormTypeInterface $type The type of the button - * * @return $this */ public function setType(ResolvedFormTypeInterface $type) @@ -457,8 +451,6 @@ public function setDataLocked($locked) * * This method should not be invoked. * - * @param FormFactoryInterface $formFactory - * * @throws BadMethodCallException */ public function setFormFactory(FormFactoryInterface $formFactory) @@ -493,8 +485,6 @@ public function setMethod($method) /** * Unsupported method. * - * @param RequestHandlerInterface $requestHandler - * * @throws BadMethodCallException */ public function setRequestHandler(RequestHandlerInterface $requestHandler) diff --git a/ChoiceList/Factory/CachingFactoryDecorator.php b/ChoiceList/Factory/CachingFactoryDecorator.php index 5ce851096c..d7a62730ef 100644 --- a/ChoiceList/Factory/CachingFactoryDecorator.php +++ b/ChoiceList/Factory/CachingFactoryDecorator.php @@ -22,9 +22,6 @@ */ class CachingFactoryDecorator implements ChoiceListFactoryInterface { - /** - * @var ChoiceListFactoryInterface - */ private $decoratedFactory; /** @@ -89,11 +86,6 @@ private static function flatten(array $array, &$output) } } - /** - * Decorates the given factory. - * - * @param ChoiceListFactoryInterface $decoratedFactory The decorated factory - */ public function __construct(ChoiceListFactoryInterface $decoratedFactory) { $this->decoratedFactory = $decoratedFactory; diff --git a/ChoiceList/Factory/PropertyAccessDecorator.php b/ChoiceList/Factory/PropertyAccessDecorator.php index 1b68fd8924..82b2082f33 100644 --- a/ChoiceList/Factory/PropertyAccessDecorator.php +++ b/ChoiceList/Factory/PropertyAccessDecorator.php @@ -41,22 +41,9 @@ */ class PropertyAccessDecorator implements ChoiceListFactoryInterface { - /** - * @var ChoiceListFactoryInterface - */ private $decoratedFactory; - - /** - * @var PropertyAccessorInterface - */ private $propertyAccessor; - /** - * Decorates the given factory. - * - * @param ChoiceListFactoryInterface $decoratedFactory The decorated factory - * @param null|PropertyAccessorInterface $propertyAccessor The used property accessor - */ public function __construct(ChoiceListFactoryInterface $decoratedFactory, PropertyAccessorInterface $propertyAccessor = null) { $this->decoratedFactory = $decoratedFactory; diff --git a/ChoiceList/View/ChoiceView.php b/ChoiceList/View/ChoiceView.php index 5078de789a..5f32b5bbf4 100644 --- a/ChoiceList/View/ChoiceView.php +++ b/ChoiceList/View/ChoiceView.php @@ -70,8 +70,6 @@ class ChoiceView extends LegacyChoiceView { /** * Additional attributes for the HTML tag. - * - * @var array */ public $attr; diff --git a/Extension/Core/ChoiceList/ChoiceList.php b/Extension/Core/ChoiceList/ChoiceList.php index 6e0cea5539..14860a59f7 100644 --- a/Extension/Core/ChoiceList/ChoiceList.php +++ b/Extension/Core/ChoiceList/ChoiceList.php @@ -43,31 +43,23 @@ class ChoiceList implements ChoiceListInterface { /** * The choices with their indices as keys. - * - * @var array */ protected $choices = array(); /** * The choice values with the indices of the matching choices as keys. - * - * @var array */ protected $values = array(); /** * The preferred view objects as hierarchy containing also the choice groups * with the indices of the matching choices as bottom-level keys. - * - * @var array */ private $preferredViews = array(); /** * The non-preferred view objects as hierarchy containing also the choice * groups with the indices of the matching choices as bottom-level keys. - * - * @var array */ private $remainingViews = array(); diff --git a/Extension/Core/DataMapper/PropertyPathMapper.php b/Extension/Core/DataMapper/PropertyPathMapper.php index 736752a41e..25aba9a16d 100644 --- a/Extension/Core/DataMapper/PropertyPathMapper.php +++ b/Extension/Core/DataMapper/PropertyPathMapper.php @@ -23,16 +23,8 @@ */ class PropertyPathMapper implements DataMapperInterface { - /** - * @var PropertyAccessorInterface - */ private $propertyAccessor; - /** - * Creates a new property path mapper. - * - * @param PropertyAccessorInterface $propertyAccessor The property accessor - */ public function __construct(PropertyAccessorInterface $propertyAccessor = null) { $this->propertyAccessor = $propertyAccessor ?: PropertyAccess::createPropertyAccessor(); diff --git a/Extension/Core/DataTransformer/ChoiceToValueTransformer.php b/Extension/Core/DataTransformer/ChoiceToValueTransformer.php index 369a12343f..e8424b9299 100644 --- a/Extension/Core/DataTransformer/ChoiceToValueTransformer.php +++ b/Extension/Core/DataTransformer/ChoiceToValueTransformer.php @@ -22,9 +22,6 @@ class ChoiceToValueTransformer implements DataTransformerInterface { private $choiceList; - /** - * @param ChoiceListInterface $choiceList - */ public function __construct(ChoiceListInterface $choiceList) { $this->choiceList = $choiceList; diff --git a/Extension/Core/DataTransformer/ChoicesToValuesTransformer.php b/Extension/Core/DataTransformer/ChoicesToValuesTransformer.php index 05da291733..a6b2f3fd58 100644 --- a/Extension/Core/DataTransformer/ChoicesToValuesTransformer.php +++ b/Extension/Core/DataTransformer/ChoicesToValuesTransformer.php @@ -22,17 +22,12 @@ class ChoicesToValuesTransformer implements DataTransformerInterface { private $choiceList; - /** - * @param ChoiceListInterface $choiceList - */ public function __construct(ChoiceListInterface $choiceList) { $this->choiceList = $choiceList; } /** - * @param array $array - * * @return array * * @throws TransformationFailedException if the given value is not an array @@ -51,8 +46,6 @@ public function transform($array) } /** - * @param array $array - * * @return array * * @throws TransformationFailedException if the given value is not an array diff --git a/Extension/Core/DataTransformer/DataTransformerChain.php b/Extension/Core/DataTransformer/DataTransformerChain.php index f52bb97eb4..81ffc7e247 100644 --- a/Extension/Core/DataTransformer/DataTransformerChain.php +++ b/Extension/Core/DataTransformer/DataTransformerChain.php @@ -21,17 +21,12 @@ */ class DataTransformerChain implements DataTransformerInterface { - /** - * The value transformers. - * - * @var DataTransformerInterface[] - */ protected $transformers; /** * Uses the given value transformers to transform values. * - * @param array $transformers + * @param DataTransformerInterface[] $transformers */ public function __construct(array $transformers) { diff --git a/Extension/Core/DataTransformer/ValueToDuplicatesTransformer.php b/Extension/Core/DataTransformer/ValueToDuplicatesTransformer.php index 2c1d6d0f62..5b851af7dd 100644 --- a/Extension/Core/DataTransformer/ValueToDuplicatesTransformer.php +++ b/Extension/Core/DataTransformer/ValueToDuplicatesTransformer.php @@ -47,8 +47,6 @@ public function transform($value) /** * Extracts the duplicated value from an array. * - * @param array $array - * * @return mixed The value * * @throws TransformationFailedException if the given value is not an array or diff --git a/Extension/Core/EventListener/ResizeFormListener.php b/Extension/Core/EventListener/ResizeFormListener.php index 4b339181e9..e61ae5b8a3 100644 --- a/Extension/Core/EventListener/ResizeFormListener.php +++ b/Extension/Core/EventListener/ResizeFormListener.php @@ -24,35 +24,20 @@ */ class ResizeFormListener implements EventSubscriberInterface { - /** - * @var string - */ protected $type; - - /** - * @var array - */ protected $options; - - /** - * Whether children could be added to the group. - * - * @var bool - */ protected $allowAdd; - - /** - * Whether children could be removed from the group. - * - * @var bool - */ protected $allowDelete; - /** - * @var bool - */ private $deleteEmpty; + /** + * @param string $type + * @param array $options + * @param bool $allowAdd whether children could be added to the group + * @param bool $allowDelete whether children could be removed from the group + * @param bool $deleteEmpty + */ public function __construct($type, array $options = array(), $allowAdd = false, $allowDelete = false, $deleteEmpty = false) { $this->type = $type; diff --git a/Extension/DataCollector/EventListener/DataCollectorListener.php b/Extension/DataCollector/EventListener/DataCollectorListener.php index 0bd33d36a5..c80dd67141 100644 --- a/Extension/DataCollector/EventListener/DataCollectorListener.php +++ b/Extension/DataCollector/EventListener/DataCollectorListener.php @@ -49,8 +49,6 @@ public static function getSubscribedEvents() /** * Listener for the {@link FormEvents::POST_SET_DATA} event. - * - * @param FormEvent $event The event object */ public function postSetData(FormEvent $event) { @@ -65,8 +63,6 @@ public function postSetData(FormEvent $event) /** * Listener for the {@link FormEvents::POST_SUBMIT} event. - * - * @param FormEvent $event The event object */ public function postSubmit(FormEvent $event) { diff --git a/Extension/DataCollector/FormDataCollector.php b/Extension/DataCollector/FormDataCollector.php index a408dde230..8e7a3c61f5 100644 --- a/Extension/DataCollector/FormDataCollector.php +++ b/Extension/DataCollector/FormDataCollector.php @@ -25,9 +25,6 @@ */ class FormDataCollector extends DataCollector implements FormDataCollectorInterface { - /** - * @var FormDataExtractor - */ private $dataExtractor; /** diff --git a/Extension/DataCollector/FormDataCollectorInterface.php b/Extension/DataCollector/FormDataCollectorInterface.php index ee004a09f3..d2a7818cd0 100644 --- a/Extension/DataCollector/FormDataCollectorInterface.php +++ b/Extension/DataCollector/FormDataCollectorInterface.php @@ -24,37 +24,26 @@ interface FormDataCollectorInterface extends DataCollectorInterface { /** * Stores configuration data of the given form and its children. - * - * @param FormInterface $form A root form */ public function collectConfiguration(FormInterface $form); /** * Stores the default data of the given form and its children. - * - * @param FormInterface $form A root form */ public function collectDefaultData(FormInterface $form); /** * Stores the submitted data of the given form and its children. - * - * @param FormInterface $form A root form */ public function collectSubmittedData(FormInterface $form); /** * Stores the view variables of the given form view and its children. - * - * @param FormView $view A root form view */ public function collectViewVariables(FormView $view); /** * Specifies that the given objects represent the same conceptual form. - * - * @param FormInterface $form A form object - * @param FormView $view A view object */ public function associateFormWithView(FormInterface $form, FormView $view); @@ -63,8 +52,6 @@ public function associateFormWithView(FormInterface $form, FormView $view); * a tree-like data structure. * * The result can be queried using {@link getData()}. - * - * @param FormInterface $form A root form */ public function buildPreliminaryFormTree(FormInterface $form); @@ -85,9 +72,6 @@ public function buildPreliminaryFormTree(FormInterface $form); * tree, only the view data will be included in the result. If a * corresponding {@link FormInterface} exists otherwise, call * {@link associateFormWithView()} before calling this method. - * - * @param FormInterface $form A root form - * @param FormView $view A root view */ public function buildFinalFormTree(FormInterface $form, FormView $view); diff --git a/Extension/DataCollector/FormDataExtractor.php b/Extension/DataCollector/FormDataExtractor.php index 20b270d845..3e2ffd04f9 100644 --- a/Extension/DataCollector/FormDataExtractor.php +++ b/Extension/DataCollector/FormDataExtractor.php @@ -180,8 +180,6 @@ public function extractViewVariables(FormView $view) /** * Recursively builds an HTML ID for a form. * - * @param FormInterface $form The form - * * @return string The HTML ID */ private function buildId(FormInterface $form) diff --git a/Extension/DataCollector/FormDataExtractorInterface.php b/Extension/DataCollector/FormDataExtractorInterface.php index e0bc6dc0ca..5fd345fec0 100644 --- a/Extension/DataCollector/FormDataExtractorInterface.php +++ b/Extension/DataCollector/FormDataExtractorInterface.php @@ -24,8 +24,6 @@ interface FormDataExtractorInterface /** * Extracts the configuration data of a form. * - * @param FormInterface $form The form - * * @return array Information about the form's configuration */ public function extractConfiguration(FormInterface $form); @@ -33,8 +31,6 @@ public function extractConfiguration(FormInterface $form); /** * Extracts the default data of a form. * - * @param FormInterface $form The form - * * @return array Information about the form's default data */ public function extractDefaultData(FormInterface $form); @@ -42,8 +38,6 @@ public function extractDefaultData(FormInterface $form); /** * Extracts the submitted data of a form. * - * @param FormInterface $form The form - * * @return array Information about the form's submitted data */ public function extractSubmittedData(FormInterface $form); @@ -51,8 +45,6 @@ public function extractSubmittedData(FormInterface $form); /** * Extracts the view variables of a form. * - * @param FormView $view The form view - * * @return array Information about the view's variables */ public function extractViewVariables(FormView $view); diff --git a/Extension/HttpFoundation/Type/FormTypeHttpFoundationExtension.php b/Extension/HttpFoundation/Type/FormTypeHttpFoundationExtension.php index 9cb0dc4476..59e108673a 100644 --- a/Extension/HttpFoundation/Type/FormTypeHttpFoundationExtension.php +++ b/Extension/HttpFoundation/Type/FormTypeHttpFoundationExtension.php @@ -22,19 +22,9 @@ */ class FormTypeHttpFoundationExtension extends AbstractTypeExtension { - /** - * @var BindRequestListener - */ private $listener; - - /** - * @var RequestHandlerInterface - */ private $requestHandler; - /** - * @param RequestHandlerInterface $requestHandler - */ public function __construct(RequestHandlerInterface $requestHandler = null) { $this->listener = new BindRequestListener(); diff --git a/Extension/Validator/Constraints/FormValidator.php b/Extension/Validator/Constraints/FormValidator.php index 3bf57df048..bddb8a09e8 100644 --- a/Extension/Validator/Constraints/FormValidator.php +++ b/Extension/Validator/Constraints/FormValidator.php @@ -194,8 +194,6 @@ private static function allowDataWalking(FormInterface $form) /** * Returns the validation groups of the given form. * - * @param FormInterface $form The form - * * @return array The validation groups */ private static function getValidationGroups(FormInterface $form) diff --git a/Extension/Validator/ValidatorTypeGuesser.php b/Extension/Validator/ValidatorTypeGuesser.php index b04a9447fc..9f06a75d90 100644 --- a/Extension/Validator/ValidatorTypeGuesser.php +++ b/Extension/Validator/ValidatorTypeGuesser.php @@ -81,8 +81,6 @@ public function guessPattern($class, $property) /** * Guesses a field class name for a given constraint. * - * @param Constraint $constraint The constraint to guess for - * * @return TypeGuess|null The guessed field class and options */ public function guessTypeForConstraint(Constraint $constraint) @@ -167,8 +165,6 @@ public function guessTypeForConstraint(Constraint $constraint) /** * Guesses whether a field is required based on the given constraint. * - * @param Constraint $constraint The constraint to guess for - * * @return ValueGuess|null The guess whether the field is required */ public function guessRequiredForConstraint(Constraint $constraint) @@ -185,8 +181,6 @@ public function guessRequiredForConstraint(Constraint $constraint) /** * Guesses a field's maximum length based on the given constraint. * - * @param Constraint $constraint The constraint to guess for - * * @return ValueGuess|null The guess for the maximum length */ public function guessMaxLengthForConstraint(Constraint $constraint) @@ -215,8 +209,6 @@ public function guessMaxLengthForConstraint(Constraint $constraint) /** * Guesses a field's pattern based on the given constraint. * - * @param Constraint $constraint The constraint to guess for - * * @return ValueGuess|null The guess for the pattern */ public function guessPatternForConstraint(Constraint $constraint) diff --git a/Extension/Validator/ViolationMapper/ViolationMapper.php b/Extension/Validator/ViolationMapper/ViolationMapper.php index 14e7b521f1..8a3602e701 100644 --- a/Extension/Validator/ViolationMapper/ViolationMapper.php +++ b/Extension/Validator/ViolationMapper/ViolationMapper.php @@ -269,8 +269,6 @@ private function reconstructPath(ViolationPath $violationPath, FormInterface $or } /** - * @param FormInterface $form - * * @return bool */ private function acceptsErrors(FormInterface $form) diff --git a/Form.php b/Form.php index 7e662f8f1e..83f0055e0c 100644 --- a/Form.php +++ b/Form.php @@ -162,8 +162,6 @@ class Form implements \IteratorAggregate, FormInterface /** * Creates a new form based on the given configuration. * - * @param FormConfigInterface $config The form configuration - * * @throws LogicException if a data mapper is not provided for a compound form */ public function __construct(FormConfigInterface $config) diff --git a/FormConfigBuilderInterface.php b/FormConfigBuilderInterface.php index 3cb4e38482..bd9e8d8de7 100644 --- a/FormConfigBuilderInterface.php +++ b/FormConfigBuilderInterface.php @@ -35,8 +35,6 @@ public function addEventListener($eventName, $listener, $priority = 0); /** * Adds an event subscriber for events on this form. * - * @param EventSubscriberInterface $subscriber The subscriber to attach - * * @return $this The configuration object */ public function addEventSubscriber(EventSubscriberInterface $subscriber); @@ -98,8 +96,6 @@ public function setAttribute($name, $value); /** * Sets the attributes. * - * @param array $attributes The attributes - * * @return $this The configuration object */ public function setAttributes(array $attributes); @@ -107,8 +103,6 @@ public function setAttributes(array $attributes); /** * Sets the data mapper used by the form. * - * @param DataMapperInterface $dataMapper - * * @return $this The configuration object */ public function setDataMapper(DataMapperInterface $dataMapper = null); @@ -202,8 +196,6 @@ public function setCompound($compound); /** * Set the types. * - * @param ResolvedFormTypeInterface $type The type of the form - * * @return $this The configuration object */ public function setType(ResolvedFormTypeInterface $type); @@ -232,8 +224,6 @@ public function setDataLocked($locked); /** * Sets the form factory used for creating new forms. - * - * @param FormFactoryInterface $formFactory The form factory */ public function setFormFactory(FormFactoryInterface $formFactory); @@ -258,8 +248,6 @@ public function setMethod($method); /** * Sets the request handler used by the form. * - * @param RequestHandlerInterface $requestHandler - * * @return $this The configuration object */ public function setRequestHandler(RequestHandlerInterface $requestHandler); diff --git a/FormError.php b/FormError.php index c35a26f5a3..4bfd853d57 100644 --- a/FormError.php +++ b/FormError.php @@ -20,37 +20,11 @@ */ class FormError implements \Serializable { - /** - * @var string - */ - private $message; - - /** - * The template for the error message. - * - * @var string - */ protected $messageTemplate; - - /** - * The parameters that should be substituted in the message template. - * - * @var array - */ protected $messageParameters; - - /** - * The value for error message pluralization. - * - * @var int|null - */ protected $messagePluralization; - /** - * The cause for this error. - * - * @var mixed - */ + private $message; private $cause; /** diff --git a/FormFactory.php b/FormFactory.php index 34e3666dbd..e87e457a7b 100644 --- a/FormFactory.php +++ b/FormFactory.php @@ -142,8 +142,6 @@ public function createBuilderForProperty($class, $property, $data = null, array * Wraps a type into a ResolvedFormTypeInterface implementation and connects * it with its parent type. * - * @param FormTypeInterface $type The type to resolve - * * @return ResolvedFormTypeInterface The resolved type */ private function resolveType(FormTypeInterface $type) diff --git a/FormFactoryBuilderInterface.php b/FormFactoryBuilderInterface.php index c1e55dcc60..35cdc44d9d 100644 --- a/FormFactoryBuilderInterface.php +++ b/FormFactoryBuilderInterface.php @@ -21,8 +21,6 @@ interface FormFactoryBuilderInterface /** * Sets the factory for creating ResolvedFormTypeInterface instances. * - * @param ResolvedFormTypeFactoryInterface $resolvedTypeFactory - * * @return $this */ public function setResolvedTypeFactory(ResolvedFormTypeFactoryInterface $resolvedTypeFactory); @@ -30,8 +28,6 @@ public function setResolvedTypeFactory(ResolvedFormTypeFactoryInterface $resolve /** * Adds an extension to be loaded by the factory. * - * @param FormExtensionInterface $extension The extension - * * @return $this */ public function addExtension(FormExtensionInterface $extension); @@ -48,8 +44,6 @@ public function addExtensions(array $extensions); /** * Adds a form type to the factory. * - * @param FormTypeInterface $type The form type - * * @return $this */ public function addType(FormTypeInterface $type); @@ -66,8 +60,6 @@ public function addTypes(array $types); /** * Adds a form type extension to the factory. * - * @param FormTypeExtensionInterface $typeExtension The form type extension - * * @return $this */ public function addTypeExtension(FormTypeExtensionInterface $typeExtension); @@ -84,8 +76,6 @@ public function addTypeExtensions(array $typeExtensions); /** * Adds a type guesser to the factory. * - * @param FormTypeGuesserInterface $typeGuesser The type guesser - * * @return $this */ public function addTypeGuesser(FormTypeGuesserInterface $typeGuesser); diff --git a/FormInterface.php b/FormInterface.php index e28d1a102d..fcfbaaf96c 100644 --- a/FormInterface.php +++ b/FormInterface.php @@ -23,8 +23,6 @@ interface FormInterface extends \ArrayAccess, \Traversable, \Countable /** * Sets the parent form. * - * @param FormInterface|null $parent The parent form or null if it's the root - * * @return self * * @throws Exception\AlreadySubmittedException if the form has already been submitted @@ -296,8 +294,6 @@ public function isRoot(); /** * Creates a view. * - * @param FormView $parent The parent view - * * @return FormView The view */ public function createView(FormView $parent = null); diff --git a/FormRenderer.php b/FormRenderer.php index b731b740df..342a64d503 100644 --- a/FormRenderer.php +++ b/FormRenderer.php @@ -27,35 +27,13 @@ class FormRenderer implements FormRendererInterface { const CACHE_KEY_VAR = 'unique_block_prefix'; - /** - * @var FormRendererEngineInterface - */ private $engine; - - /** - * @var CsrfTokenManagerInterface - */ private $csrfTokenManager; - - /** - * @var array - */ private $blockNameHierarchyMap = array(); - - /** - * @var array - */ private $hierarchyLevelMap = array(); - - /** - * @var array - */ private $variableStack = array(); /** - * @param FormRendererEngineInterface $engine - * @param CsrfTokenManagerInterface|null $csrfTokenManager - * * @throws UnexpectedTypeException */ public function __construct(FormRendererEngineInterface $engine, $csrfTokenManager = null) diff --git a/FormTypeExtensionInterface.php b/FormTypeExtensionInterface.php index 095813d211..cc46b0a324 100644 --- a/FormTypeExtensionInterface.php +++ b/FormTypeExtensionInterface.php @@ -25,9 +25,6 @@ interface FormTypeExtensionInterface * further modify it. * * @see FormTypeInterface::buildForm() - * - * @param FormBuilderInterface $builder The form builder - * @param array $options The options */ public function buildForm(FormBuilderInterface $builder, array $options); @@ -38,10 +35,6 @@ public function buildForm(FormBuilderInterface $builder, array $options); * further modify it. * * @see FormTypeInterface::buildView() - * - * @param FormView $view The view - * @param FormInterface $form The form - * @param array $options The options */ public function buildView(FormView $view, FormInterface $form, array $options); @@ -52,10 +45,6 @@ public function buildView(FormView $view, FormInterface $form, array $options); * further modify it. * * @see FormTypeInterface::finishView() - * - * @param FormView $view The view - * @param FormInterface $form The form - * @param array $options The options */ public function finishView(FormView $view, FormInterface $form, array $options); diff --git a/FormView.php b/FormView.php index 8655bedf6e..21a3d5036a 100644 --- a/FormView.php +++ b/FormView.php @@ -20,8 +20,6 @@ class FormView implements \ArrayAccess, \IteratorAggregate, \Countable { /** * The variables assigned to this view. - * - * @var array */ public $vars = array( 'value' => null, @@ -30,8 +28,6 @@ class FormView implements \ArrayAccess, \IteratorAggregate, \Countable /** * The parent view. - * - * @var FormView */ public $parent; diff --git a/Guess/TypeGuess.php b/Guess/TypeGuess.php index a190d5bc03..1431b5e400 100644 --- a/Guess/TypeGuess.php +++ b/Guess/TypeGuess.php @@ -19,18 +19,7 @@ */ class TypeGuess extends Guess { - /** - * The guessed field type. - * - * @var string - */ private $type; - - /** - * The guessed options for creating an instance of the guessed class. - * - * @var array - */ private $options; /** diff --git a/NativeRequestHandler.php b/NativeRequestHandler.php index 3607feb99c..e28c2b4b10 100644 --- a/NativeRequestHandler.php +++ b/NativeRequestHandler.php @@ -21,9 +21,6 @@ */ class NativeRequestHandler implements RequestHandlerInterface { - /** - * @var ServerParams - */ private $serverParams; /** @@ -36,8 +33,6 @@ public function __construct(ServerParams $params = null) /** * The allowed keys of the $_FILES array. - * - * @var array */ private static $fileKeys = array( 'error', @@ -159,8 +154,6 @@ private static function getRequestMethod() * This method is identical to {@link \Symfony\Component\HttpFoundation\FileBag::fixPhpFilesArray} * and should be kept as such in order to port fixes quickly and easily. * - * @param array $data - * * @return array */ private static function fixPhpFilesArray($data) diff --git a/ReversedTransformer.php b/ReversedTransformer.php index fc5cd12bc3..8089e47bf0 100644 --- a/ReversedTransformer.php +++ b/ReversedTransformer.php @@ -21,18 +21,8 @@ */ class ReversedTransformer implements DataTransformerInterface { - /** - * The reversed transformer. - * - * @var DataTransformerInterface - */ protected $reversedTransformer; - /** - * Reverses this transformer. - * - * @param DataTransformerInterface $reversedTransformer - */ public function __construct(DataTransformerInterface $reversedTransformer) { $this->reversedTransformer = $reversedTransformer;