Skip to content

Commit 8a07748

Browse files
wouterjwouterj
committed
Convert void return types to natives ones
1 parent ead93bf commit 8a07748

File tree

71 files changed

+109
-424
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

71 files changed

+109
-424
lines changed

‎src/Symfony/Bridge/Doctrine/DataCollector/DoctrineDataCollector.php

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -39,10 +39,7 @@ public function __construct(
3939
$this->managers = $registry->getManagerNames();
4040
}
4141

42-
/**
43-
* @return void
44-
*/
45-
public function collect(Request $request, Response $response, \Throwable $exception = null)
42+
public function collect(Request $request, Response $response, \Throwable $exception = null): void
4643
{
4744
$this->data = [
4845
'queries' => $this->collectQueries(),
@@ -62,10 +59,7 @@ private function collectQueries(): array
6259
return $queries;
6360
}
6461

65-
/**
66-
* @return void
67-
*/
68-
public function reset()
62+
public function reset(): void
6963
{
7064
$this->data = [];
7165
$this->debugDataHolder->reset();

‎src/Symfony/Bridge/Doctrine/DependencyInjection/AbstractDoctrineExtension.php

Lines changed: 6 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -38,11 +38,9 @@ abstract class AbstractDoctrineExtension extends Extension
3838
/**
3939
* @param array $objectManager A configured object manager
4040
*
41-
* @return void
42-
*
4341
* @throws \InvalidArgumentException
4442
*/
45-
protected function loadMappingInformation(array $objectManager, ContainerBuilder $container)
43+
protected function loadMappingInformation(array $objectManager, ContainerBuilder $container): void
4644
{
4745
if ($objectManager['auto_mapping']) {
4846
// automatically register bundle mappings
@@ -107,10 +105,8 @@ protected function loadMappingInformation(array $objectManager, ContainerBuilder
107105
* Register the alias for this mapping driver.
108106
*
109107
* Aliases can be used in the Query languages of all the Doctrine object managers to simplify writing tasks.
110-
*
111-
* @return void
112108
*/
113-
protected function setMappingDriverAlias(array $mappingConfig, string $mappingName)
109+
protected function setMappingDriverAlias(array $mappingConfig, string $mappingName): void
114110
{
115111
if (isset($mappingConfig['alias'])) {
116112
$this->aliasMap[$mappingConfig['alias']] = $mappingConfig['prefix'];
@@ -122,11 +118,9 @@ protected function setMappingDriverAlias(array $mappingConfig, string $mappingNa
122118
/**
123119
* Register the mapping driver configuration for later use with the object managers metadata driver chain.
124120
*
125-
* @return void
126-
*
127121
* @throws \InvalidArgumentException
128122
*/
129-
protected function setMappingDriverConfig(array $mappingConfig, string $mappingName)
123+
protected function setMappingDriverConfig(array $mappingConfig, string $mappingName): void
130124
{
131125
$mappingDirectory = $mappingConfig['dir'];
132126
if (!is_dir($mappingDirectory)) {
@@ -179,9 +173,8 @@ protected function getMappingDriverBundleConfigDefaults(array $bundleConfig, \Re
179173
/**
180174
* Register all the collected mapping information with the object manager by registering the appropriate mapping drivers.
181175
*
182-
* @return void
183176
*/
184-
protected function registerMappingDrivers(array $objectManager, ContainerBuilder $container)
177+
protected function registerMappingDrivers(array $objectManager, ContainerBuilder $container): void
185178
{
186179
// configure metadata driver for each bundle based on the type of mapping files found
187180
if ($container->hasDefinition($this->getObjectManagerElementName($objectManager['name'].'_metadata_driver'))) {
@@ -235,11 +228,9 @@ protected function registerMappingDrivers(array $objectManager, ContainerBuilder
235228
/**
236229
* Assertion if the specified mapping information is valid.
237230
*
238-
* @return void
239-
*
240231
* @throws \InvalidArgumentException
241232
*/
242-
protected function assertValidMappingConfiguration(array $mappingConfig, string $objectManagerName)
233+
protected function assertValidMappingConfiguration(array $mappingConfig, string $objectManagerName): void
243234
{
244235
if (!$mappingConfig['type'] || !$mappingConfig['dir'] || !$mappingConfig['prefix']) {
245236
throw new \InvalidArgumentException(sprintf('Mapping definitions for Doctrine manager "%s" require at least the "type", "dir" and "prefix" options.', $objectManagerName));
@@ -325,11 +316,9 @@ private function detectMappingType(string $directory, ContainerBuilder $containe
325316
/**
326317
* Loads a configured object manager metadata, query or result cache driver.
327318
*
328-
* @return void
329-
*
330319
* @throws \InvalidArgumentException in case of unknown driver type
331320
*/
332-
protected function loadObjectManagerCacheDriver(array $objectManager, ContainerBuilder $container, string $cacheName)
321+
protected function loadObjectManagerCacheDriver(array $objectManager, ContainerBuilder $container, string $cacheName): void
333322
{
334323
$this->loadCacheDriver($cacheName, $objectManager['name'], $objectManager[$cacheName.'_driver'], $container);
335324
}

‎src/Symfony/Bridge/Doctrine/DependencyInjection/CompilerPass/DoctrineValidationPass.php

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -28,10 +28,7 @@ public function __construct(string $managerType)
2828
$this->managerType = $managerType;
2929
}
3030

31-
/**
32-
* @return void
33-
*/
34-
public function process(ContainerBuilder $container)
31+
public function process(ContainerBuilder $container): void
3532
{
3633
$this->updateValidatorMappingFiles($container, 'xml', 'xml');
3734
$this->updateValidatorMappingFiles($container, 'yaml', 'yml');

‎src/Symfony/Bridge/Doctrine/DependencyInjection/CompilerPass/RegisterEventListenersAndSubscribersPass.php

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -53,10 +53,7 @@ public function __construct(string $connectionsParameter, string $managerTemplat
5353
$this->tagPrefix = $tagPrefix;
5454
}
5555

56-
/**
57-
* @return void
58-
*/
59-
public function process(ContainerBuilder $container)
56+
public function process(ContainerBuilder $container): void
6057
{
6158
if (!$container->hasParameter($this->connectionsParameter)) {
6259
return;

‎src/Symfony/Bridge/Doctrine/DependencyInjection/CompilerPass/RegisterMappingsPass.php

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -130,10 +130,8 @@ public function __construct(Definition|Reference $driver, array $namespaces, arr
130130

131131
/**
132132
* Register mappings and alias with the metadata drivers.
133-
*
134-
* @return void
135133
*/
136-
public function process(ContainerBuilder $container)
134+
public function process(ContainerBuilder $container): void
137135
{
138136
if (!$this->enabled($container)) {
139137
return;

‎src/Symfony/Bridge/Doctrine/DependencyInjection/Security/UserProvider/EntityFactory.php

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -33,10 +33,7 @@ public function __construct(string $key, string $providerId)
3333
$this->providerId = $providerId;
3434
}
3535

36-
/**
37-
* @return void
38-
*/
39-
public function create(ContainerBuilder $container, string $id, array $config)
36+
public function create(ContainerBuilder $container, string $id, array $config): void
4037
{
4138
$container
4239
->setDefinition($id, new ChildDefinition($this->providerId))
@@ -51,10 +48,7 @@ public function getKey(): string
5148
return $this->key;
5249
}
5350

54-
/**
55-
* @return void
56-
*/
57-
public function addConfiguration(NodeDefinition $node)
51+
public function addConfiguration(NodeDefinition $node): void
5852
{
5953
$node
6054
->children()

‎src/Symfony/Bridge/Doctrine/Form/EventListener/MergeDoctrineCollectionListener.php

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -38,10 +38,7 @@ public static function getSubscribedEvents(): array
3838
];
3939
}
4040

41-
/**
42-
* @return void
43-
*/
44-
public function onSubmit(FormEvent $event)
41+
public function onSubmit(FormEvent $event): void
4542
{
4643
$collection = $event->getForm()->getData();
4744
$data = $event->getData();

‎src/Symfony/Bridge/Doctrine/Form/Type/DoctrineType.php

Lines changed: 3 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -97,10 +97,7 @@ public function __construct(ManagerRegistry $registry)
9797
$this->registry = $registry;
9898
}
9999

100-
/**
101-
* @return void
102-
*/
103-
public function buildForm(FormBuilderInterface $builder, array $options)
100+
public function buildForm(FormBuilderInterface $builder, array $options): void
104101
{
105102
if ($options['multiple'] && interface_exists(Collection::class)) {
106103
$builder
@@ -110,10 +107,7 @@ public function buildForm(FormBuilderInterface $builder, array $options)
110107
}
111108
}
112109

113-
/**
114-
* @return void
115-
*/
116-
public function configureOptions(OptionsResolver $resolver)
110+
public function configureOptions(OptionsResolver $resolver): void
117111
{
118112
$choiceLoader = function (Options $options) {
119113
// Unless the choices are given explicitly, load them on demand
@@ -238,10 +232,7 @@ public function getParent(): string
238232
return ChoiceType::class;
239233
}
240234

241-
/**
242-
* @return void
243-
*/
244-
public function reset()
235+
public function reset(): void
245236
{
246237
$this->idReaders = [];
247238
$this->entityLoaders = [];

‎src/Symfony/Bridge/Doctrine/Form/Type/EntityType.php

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -21,10 +21,7 @@
2121

2222
class EntityType extends DoctrineType
2323
{
24-
/**
25-
* @return void
26-
*/
27-
public function configureOptions(OptionsResolver $resolver)
24+
public function configureOptions(OptionsResolver $resolver): void
2825
{
2926
parent::configureOptions($resolver);
3027

‎src/Symfony/Bridge/Doctrine/Messenger/DoctrineClearEntityManagerWorkerSubscriber.php

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -30,18 +30,12 @@ public function __construct(ManagerRegistry $managerRegistry)
3030
$this->managerRegistry = $managerRegistry;
3131
}
3232

33-
/**
34-
* @return void
35-
*/
36-
public function onWorkerMessageHandled()
33+
public function onWorkerMessageHandled(): void
3734
{
3835
$this->clearEntityManagers();
3936
}
4037

41-
/**
42-
* @return void
43-
*/
44-
public function onWorkerMessageFailed()
38+
public function onWorkerMessageFailed(): void
4539
{
4640
$this->clearEntityManagers();
4741
}

0 commit comments

Comments
 (0)
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy