Skip to content

Commit 1930a7b

Browse files
derrabusnicolas-grekas
authored andcommitted
Add missing return types to interfaces
1 parent 145830e commit 1930a7b

File tree

10 files changed

+19
-19
lines changed

10 files changed

+19
-19
lines changed

Tests/Command/AboutCommand/Fixture/TestAppKernel.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ public function getProjectDir(): string
3030
return __DIR__.'/test';
3131
}
3232

33-
public function registerContainerConfiguration(LoaderInterface $loader)
33+
public function registerContainerConfiguration(LoaderInterface $loader): void
3434
{
3535
$loader->load(static function (ContainerBuilder $container) {
3636
$container->loadFromExtension('framework', [
@@ -39,7 +39,7 @@ public function registerContainerConfiguration(LoaderInterface $loader)
3939
});
4040
}
4141

42-
protected function build(ContainerBuilder $container)
42+
protected function build(ContainerBuilder $container): void
4343
{
4444
}
4545
}

Tests/Command/CacheClearCommand/Fixture/TestAppKernel.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,12 +31,12 @@ public function getProjectDir(): string
3131
return __DIR__.'/test';
3232
}
3333

34-
public function registerContainerConfiguration(LoaderInterface $loader)
34+
public function registerContainerConfiguration(LoaderInterface $loader): void
3535
{
3636
$loader->load(__DIR__.\DIRECTORY_SEPARATOR.'config.yml');
3737
}
3838

39-
protected function build(ContainerBuilder $container)
39+
protected function build(ContainerBuilder $container): void
4040
{
4141
$container->register('logger', NullLogger::class);
4242
}

Tests/Controller/ControllerResolverTest.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -170,14 +170,14 @@ protected function createMockContainer()
170170

171171
class ContainerAwareController implements ContainerAwareInterface
172172
{
173-
private $container;
173+
private ?Container $container = null;
174174

175-
public function setContainer(?ContainerInterface $container)
175+
public function setContainer(?ContainerInterface $container): void
176176
{
177177
$this->container = $container;
178178
}
179179

180-
public function getContainer()
180+
public function getContainer(): ?Container
181181
{
182182
return $this->container;
183183
}
@@ -193,7 +193,7 @@ public function __invoke()
193193

194194
class DummyController extends AbstractController
195195
{
196-
public function getContainer()
196+
public function getContainer(): Psr11ContainerInterface
197197
{
198198
return $this->container;
199199
}

Tests/DependencyInjection/FrameworkExtensionTestCase.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2350,7 +2350,7 @@ private function assertCachePoolServiceDefinitionIsCreated(ContainerBuilder $con
23502350
*/
23512351
class TestAnnotationsPass implements CompilerPassInterface
23522352
{
2353-
public function process(ContainerBuilder $container)
2353+
public function process(ContainerBuilder $container): void
23542354
{
23552355
$container->setDefinition('annotation_reader', $container->getDefinition('annotations.cached_reader'));
23562356
$container->removeDefinition('annotations.cached_reader');

Tests/Functional/Bundle/DefaultConfigTestBundle/DependencyInjection/DefaultConfigTestExtension.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616

1717
class DefaultConfigTestExtension extends Extension
1818
{
19-
public function load(array $configs, ContainerBuilder $container)
19+
public function load(array $configs, ContainerBuilder $container): void
2020
{
2121
$configuration = new Configuration();
2222
$config = $this->processConfiguration($configuration, $configs);

Tests/Functional/Bundle/ExtensionWithoutConfigTestBundle/DependencyInjection/ExtensionWithoutConfigTestExtension.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616

1717
class ExtensionWithoutConfigTestExtension implements ExtensionInterface
1818
{
19-
public function load(array $configs, ContainerBuilder $container)
19+
public function load(array $configs, ContainerBuilder $container): void
2020
{
2121
}
2222

Tests/Functional/Bundle/TestBundle/DependencyInjection/AnnotationReaderPass.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616

1717
class AnnotationReaderPass implements CompilerPassInterface
1818
{
19-
public function process(ContainerBuilder $container)
19+
public function process(ContainerBuilder $container): void
2020
{
2121
// simulate using "annotation_reader" in a compiler pass
2222
$container->get('test.annotation_reader');

Tests/Functional/Bundle/TestBundle/DependencyInjection/TestExtension.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,15 +21,15 @@ class TestExtension extends Extension implements PrependExtensionInterface
2121
{
2222
private $customConfig;
2323

24-
public function load(array $configs, ContainerBuilder $container)
24+
public function load(array $configs, ContainerBuilder $container): void
2525
{
2626
$configuration = $this->getConfiguration($configs, $container);
2727
$this->processConfiguration($configuration, $configs);
2828

2929
$container->setAlias('test.annotation_reader', new Alias('annotation_reader', true));
3030
}
3131

32-
public function prepend(ContainerBuilder $container)
32+
public function prepend(ContainerBuilder $container): void
3333
{
3434
$container->prependExtensionConfig('test', ['custom' => 'foo']);
3535
}
@@ -39,7 +39,7 @@ public function getConfiguration(array $config, ContainerBuilder $container): ?C
3939
return new Configuration($this->customConfig);
4040
}
4141

42-
public function setCustomConfig($customConfig)
42+
public function setCustomConfig($customConfig): void
4343
{
4444
$this->customConfig = $customConfig;
4545
}

Tests/Functional/Bundle/TestBundle/TestBundle.php

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

2222
class TestBundle extends Bundle
2323
{
24-
public function build(ContainerBuilder $container)
24+
public function build(ContainerBuilder $container): void
2525
{
2626
parent::build($container);
2727

Tests/Functional/app/AppKernel.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -73,12 +73,12 @@ public function getLogDir(): string
7373
return sys_get_temp_dir().'/'.$this->varDir.'/'.$this->testCase.'/logs';
7474
}
7575

76-
public function registerContainerConfiguration(LoaderInterface $loader)
76+
public function registerContainerConfiguration(LoaderInterface $loader): void
7777
{
7878
$loader->load($this->rootConfig);
7979
}
8080

81-
protected function build(ContainerBuilder $container)
81+
protected function build(ContainerBuilder $container): void
8282
{
8383
$container->register('logger', NullLogger::class);
8484
$container->registerExtension(new TestDumpExtension());
@@ -117,7 +117,7 @@ public function getConfigTreeBuilder(): TreeBuilder
117117
return $treeBuilder;
118118
}
119119

120-
public function load(array $configs, ContainerBuilder $container)
120+
public function load(array $configs, ContainerBuilder $container): void
121121
{
122122
}
123123

0 commit comments

Comments
 (0)