Add void return types

Author: wouterj

ContainerAwareEventManager.php

@@ -167,7 +167,7 @@ public function removeEventSubscriber(EventSubscriber $subscriber): void
         parent::removeEventSubscriber($subscriber);
     }
 
-    private function initializeListeners(string $eventName)
+    private function initializeListeners(string $eventName): void
     {
         $this->initialized[$eventName] = true;
         foreach ($this->listeners[$eventName] as $hash => $listener) {
@@ -179,7 +179,7 @@ private function initializeListeners(string $eventName)
         }
     }
 
-    private function initializeSubscribers()
+    private function initializeSubscribers(): void
     {
         $this->initializedSubscribers = true;
         foreach ($this->subscribers as $subscriber) {

DataCollector/DoctrineDataCollector.php

@@ -47,12 +47,17 @@ public function __construct(
 
     /**
      * Adds the stack logger for a connection.
+     *
+     * @return void
      */
     public function addLogger(string $name, DebugStack $logger)
     {
         $this->loggers[$name] = $logger;
     }
 
+    /**
+     * @return void
+     */
     public function collect(Request $request, Response $response, \Throwable $exception = null)
     {
         $this->data = [
@@ -81,6 +86,9 @@ private function collectQueries(): array
         return $queries;
     }
 
+    /**
+     * @return void
+     */
     public function reset()
     {
         $this->data = [];

DataFixtures/ContainerAwareLoader.php

@@ -32,6 +32,9 @@ public function __construct(ContainerInterface $container)
         $this->container = $container;
     }
 
+    /**
+     * @return void
+     */
     public function addFixture(FixtureInterface $fixture)
     {
         if ($fixture instanceof ContainerAwareInterface) {

DependencyInjection/AbstractDoctrineExtension.php

@@ -38,6 +38,8 @@ abstract class AbstractDoctrineExtension extends Extension
     /**
      * @param array $objectManager A configured object manager
      *
+     * @return void
+     *
      * @throws \InvalidArgumentException
      */
     protected function loadMappingInformation(array $objectManager, ContainerBuilder $container)
@@ -105,6 +107,8 @@ protected function loadMappingInformation(array $objectManager, ContainerBuilder
      * Register the alias for this mapping driver.
      *
      * Aliases can be used in the Query languages of all the Doctrine object managers to simplify writing tasks.
+     *
+     * @return void
      */
     protected function setMappingDriverAlias(array $mappingConfig, string $mappingName)
     {
@@ -118,6 +122,8 @@ protected function setMappingDriverAlias(array $mappingConfig, string $mappingNa
     /**
      * Register the mapping driver configuration for later use with the object managers metadata driver chain.
      *
+     * @return void
+     *
      * @throws \InvalidArgumentException
      */
     protected function setMappingDriverConfig(array $mappingConfig, string $mappingName)
@@ -172,6 +178,8 @@ protected function getMappingDriverBundleConfigDefaults(array $bundleConfig, \Re
 
     /**
      * Register all the collected mapping information with the object manager by registering the appropriate mapping drivers.
+     *
+     * @return void
      */
     protected function registerMappingDrivers(array $objectManager, ContainerBuilder $container)
     {
@@ -227,6 +235,8 @@ protected function registerMappingDrivers(array $objectManager, ContainerBuilder
     /**
      * Assertion if the specified mapping information is valid.
      *
+     * @return void
+     *
      * @throws \InvalidArgumentException
      */
     protected function assertValidMappingConfiguration(array $mappingConfig, string $objectManagerName)
@@ -315,6 +325,8 @@ private function detectMappingType(string $directory, ContainerBuilder $containe
     /**
      * Loads a configured object manager metadata, query or result cache driver.
      *
+     * @return void
+     *
      * @throws \InvalidArgumentException in case of unknown driver type
      */
     protected function loadObjectManagerCacheDriver(array $objectManager, ContainerBuilder $container, string $cacheName)

DependencyInjection/CompilerPass/DoctrineValidationPass.php

@@ -28,6 +28,9 @@ public function __construct(string $managerType)
         $this->managerType = $managerType;
     }
 
+    /**
+     * @return void
+     */
     public function process(ContainerBuilder $container)
     {
         $this->updateValidatorMappingFiles($container, 'xml', 'xml');
@@ -38,7 +41,7 @@ public function process(ContainerBuilder $container)
      * Gets the validation mapping files for the format and extends them with
      * files matching a doctrine search pattern (Resources/config/validation.orm.xml).
      */
-    private function updateValidatorMappingFiles(ContainerBuilder $container, string $mapping, string $extension)
+    private function updateValidatorMappingFiles(ContainerBuilder $container, string $mapping, string $extension): void
     {
         if (!$container->hasParameter('validator.mapping.loader.'.$mapping.'_files_loader.mapping_files')) {
             return;

DependencyInjection/CompilerPass/RegisterMappingsPass.php

@@ -130,6 +130,8 @@ public function __construct(Definition|Reference $driver, array $namespaces, arr
 
     /**
      * Register mappings and alias with the metadata drivers.
+     *
+     * @return void
      */
     public function process(ContainerBuilder $container)
     {

DependencyInjection/CompilerPass/RegisterUidTypePass.php

@@ -19,7 +19,7 @@
 
 final class RegisterUidTypePass implements CompilerPassInterface
 {
-    public function process(ContainerBuilder $container)
+    public function process(ContainerBuilder $container): void
     {
         if (!class_exists(AbstractUid::class)) {
             return;

DependencyInjection/Security/UserProvider/EntityFactory.php

@@ -33,6 +33,9 @@ public function __construct(string $key, string $providerId)
         $this->providerId = $providerId;
     }
 
+    /**
+     * @return void
+     */
     public function create(ContainerBuilder $container, string $id, array $config)
     {
         $container
@@ -51,6 +54,9 @@ public function getKey()
         return $this->key;
     }
 
+    /**
+     * @return void
+     */
     public function addConfiguration(NodeDefinition $node)
     {
         $node

Form/EventListener/MergeDoctrineCollectionListener.php

@@ -38,6 +38,9 @@ public static function getSubscribedEvents(): array
         ];
     }
 
+    /**
+     * @return void
+     */
     public function onSubmit(FormEvent $event)
     {
         $collection = $event->getForm()->getData();

Form/Type/DoctrineType.php

@@ -97,6 +97,9 @@ public function __construct(ManagerRegistry $registry)
         $this->registry = $registry;
     }
 
+    /**
+     * @return void
+     */
     public function buildForm(FormBuilderInterface $builder, array $options)
     {
         if ($options['multiple'] && interface_exists(Collection::class)) {
@@ -107,6 +110,9 @@ public function buildForm(FormBuilderInterface $builder, array $options)
         }
     }
 
+    /**
+     * @return void
+     */
     public function configureOptions(OptionsResolver $resolver)
     {
         $choiceLoader = function (Options $options) {
@@ -232,6 +238,9 @@ public function getParent(): string
         return ChoiceType::class;
     }
 
+    /**
+     * @return void
+     */
     public function reset()
     {
         $this->idReaders = [];