Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit 18793b7

Browse files
TobionTobion
committed
feature #31996 [5.0] Add return types in final classes (dFayet)
This PR was merged into the 5.0-dev branch. Discussion ---------- [5.0] Add return types in final classes | Q | A | ------------- | --- | Branch? | master | Bug fix? | no | New feature? | yes/no <!-- please update src/**/CHANGELOG.md files --> | BC breaks? | no | Deprecations? | no | Tests pass? | no | Fixed tickets | #31981 | License | MIT | Doc PR | symfony/symfony-docs#... <!-- required for new features --> This is the first step for the issue #31981 I have some questions: - ~I have not added type for methods with `@inheritdoc` annotation, should I?~ - ~Don't we want to type also functions without `@return` annotation? (still in `final` classes)~ - ~If yes is the answer of the previous one, do we also want the `void` return type?~ - ~I have also added the return type in the `DependencyInjection` PhpDumper, but is it also wanted? (if yes, I will clean a bit the code changed)~ - ~Should we update the documentation's code samples when they display `final` classes?~ Todo: - [x] Adjust the PR, following the answers of the questions - [x] Add return type also when there is no `@return`, or with `@inheritdoc` - [x] [src/Symfony/Component/Debug/ErrorHandler.php#L383](https://github.com/symfony/symfony/blob/master/src/Symfony/Component/Debug/ErrorHandler.php#L383) `@return` annotation is not correct according to the return, investigate and adjust if needed - [x] [src/Symfony/Component/HttpKernel/ControllerMetadata/ArgumentMetadataFactory.php#L50](https://github.com/symfony/symfony/blob/master/src/Symfony/Component/HttpKernel/ControllerMetadata/ArgumentMetadataFactory.php#L50) `@return` annotation is not correct according to the return, investigate and adjust if needed - [x] Do a PR on documentation to add return type on code snippets with final classes => unneeded as they were already typed Commits ------- ca5ae19 Replace @return annotation by return type in final classes
2 parents 2f9e69e + ca5ae19 commit 18793b7

File tree

147 files changed

+326
-398
lines changed

Some content is hidden

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

147 files changed

+326
-398
lines changed

‎src/Symfony/Bridge/Doctrine/Test/TestRepositoryFactory.php‎

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ final class TestRepositoryFactory implements RepositoryFactory
2929
/**
3030
* {@inheritdoc}
3131
*/
32-
public function getRepository(EntityManagerInterface $entityManager, $entityName)
32+
public function getRepository(EntityManagerInterface $entityManager, $entityName): ObjectRepository
3333
{
3434
$repositoryHash = $this->getRepositoryHash($entityManager, $entityName);
3535

@@ -47,10 +47,7 @@ public function setRepository(EntityManagerInterface $entityManager, $entityName
4747
$this->repositoryList[$repositoryHash] = $repository;
4848
}
4949

50-
/**
51-
* @return ObjectRepository
52-
*/
53-
private function createRepository(EntityManagerInterface $entityManager, $entityName)
50+
private function createRepository(EntityManagerInterface $entityManager, $entityName): ObjectRepository
5451
{
5552
/* @var $metadata ClassMetadata */
5653
$metadata = $entityManager->getClassMetadata($entityName);
@@ -59,7 +56,7 @@ private function createRepository(EntityManagerInterface $entityManager, $entity
5956
return new $repositoryClassName($entityManager, $metadata);
6057
}
6158

62-
private function getRepositoryHash(EntityManagerInterface $entityManager, $entityName)
59+
private function getRepositoryHash(EntityManagerInterface $entityManager, $entityName): string
6360
{
6461
return $entityManager->getClassMetadata($entityName)->getName().spl_object_hash($entityManager);
6562
}

‎src/Symfony/Bridge/Monolog/Handler/ChromePhpHandler.php‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ protected function sendHeader($header, $content)
7373
/**
7474
* Override default behavior since we check it in onKernelResponse.
7575
*/
76-
protected function headersAccepted()
76+
protected function headersAccepted(): bool
7777
{
7878
return true;
7979
}

‎src/Symfony/Bridge/Monolog/Handler/FirePHPHandler.php‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ protected function sendHeader($header, $content)
7575
/**
7676
* Override default behavior since we check the user agent in onKernelResponse.
7777
*/
78-
protected function headersAccepted()
78+
protected function headersAccepted(): bool
7979
{
8080
return true;
8181
}

‎src/Symfony/Bridge/Monolog/Processor/RouteProcessor.php‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ public function __construct(bool $includeParams = true)
3535
$this->reset();
3636
}
3737

38-
public function __invoke(array $records)
38+
public function __invoke(array $records): array
3939
{
4040
if ($this->routeData && !isset($records['extra']['requests'])) {
4141
$records['extra']['requests'] = array_values($this->routeData);
@@ -78,7 +78,7 @@ public function removeRouteData(FinishRequestEvent $event)
7878
unset($this->routeData[$requestId]);
7979
}
8080

81-
public static function getSubscribedEvents()
81+
public static function getSubscribedEvents(): array
8282
{
8383
return [
8484
KernelEvents::REQUEST => ['addRouteData', 1],

‎src/Symfony/Bridge/Monolog/Processor/WebProcessor.php‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ public function onKernelRequest(ResponseEvent $event)
3939
}
4040
}
4141

42-
public static function getSubscribedEvents()
42+
public static function getSubscribedEvents(): array
4343
{
4444
return [
4545
KernelEvents::REQUEST => ['onKernelRequest', 4096],

‎src/Symfony/Bridge/ProxyManager/LazyProxy/PhpDumper/ProxyDumper.php‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -40,15 +40,15 @@ public function __construct(string $salt = '')
4040
/**
4141
* {@inheritdoc}
4242
*/
43-
public function isProxyCandidate(Definition $definition)
43+
public function isProxyCandidate(Definition $definition): bool
4444
{
4545
return ($definition->isLazy() || $definition->hasTag('proxy')) && $this->proxyGenerator->getProxifiedClass($definition);
4646
}
4747

4848
/**
4949
* {@inheritdoc}
5050
*/
51-
public function getProxyFactoryCode(Definition $definition, $id, $factoryCode = null)
51+
public function getProxyFactoryCode(Definition $definition, $id, $factoryCode = null): string
5252
{
5353
$instantiation = 'return';
5454

@@ -82,7 +82,7 @@ public function getProxyFactoryCode(Definition $definition, $id, $factoryCode =
8282
/**
8383
* {@inheritdoc}
8484
*/
85-
public function getProxyCode(Definition $definition)
85+
public function getProxyCode(Definition $definition): string
8686
{
8787
$code = $this->classGenerator->generate($this->generateProxyClass($definition));
8888

‎src/Symfony/Bridge/Twig/Extension/RoutingExtension.php‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ public function getUrl($name, $parameters = [], $schemeRelative = false)
9393
*
9494
* @final
9595
*/
96-
public function isUrlGenerationSafe(Node $argsNode)
96+
public function isUrlGenerationSafe(Node $argsNode): array
9797
{
9898
// support named arguments
9999
$paramsNode = $argsNode->hasNode('parameters') ? $argsNode->getNode('parameters') : (

‎src/Symfony/Bridge/Twig/Extension/TranslationExtension.php‎

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ public function getTranslator(): ?TranslatorInterface
5858
/**
5959
* {@inheritdoc}
6060
*/
61-
public function getFilters()
61+
public function getFilters(): array
6262
{
6363
return [
6464
new TwigFilter('trans', [$this, 'trans']),
@@ -70,7 +70,7 @@ public function getFilters()
7070
*
7171
* @return AbstractTokenParser[]
7272
*/
73-
public function getTokenParsers()
73+
public function getTokenParsers(): array
7474
{
7575
return [
7676
// {% trans %}Symfony is great!{% endtrans %}
@@ -84,17 +84,17 @@ public function getTokenParsers()
8484
/**
8585
* {@inheritdoc}
8686
*/
87-
public function getNodeVisitors()
87+
public function getNodeVisitors(): array
8888
{
8989
return [$this->getTranslationNodeVisitor(), new TranslationDefaultDomainNodeVisitor()];
9090
}
9191

92-
public function getTranslationNodeVisitor()
92+
public function getTranslationNodeVisitor(): TranslationNodeVisitor
9393
{
9494
return $this->translationNodeVisitor ?: $this->translationNodeVisitor = new TranslationNodeVisitor();
9595
}
9696

97-
public function trans($message, array $arguments = [], $domain = null, $locale = null, $count = null)
97+
public function trans($message, array $arguments = [], $domain = null, $locale = null, $count = null): string
9898
{
9999
if (null !== $count) {
100100
$arguments['%count%'] = $count;
@@ -106,7 +106,7 @@ public function trans($message, array $arguments = [], $domain = null, $locale =
106106
/**
107107
* {@inheritdoc}
108108
*/
109-
public function getName()
109+
public function getName(): string
110110
{
111111
return 'translator';
112112
}

‎src/Symfony/Bridge/Twig/Mime/WrappedTemplatedEmail.php‎

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ public function attach(string $file, string $name = null, string $contentType =
6565
/**
6666
* @return $this
6767
*/
68-
public function setSubject(string $subject)
68+
public function setSubject(string $subject): object
6969
{
7070
$this->message->subject($subject);
7171

@@ -80,7 +80,7 @@ public function getSubject(): ?string
8080
/**
8181
* @return $this
8282
*/
83-
public function setReturnPath(string $address)
83+
public function setReturnPath(string $address): object
8484
{
8585
$this->message->returnPath($address);
8686

@@ -95,7 +95,7 @@ public function getReturnPath(): string
9595
/**
9696
* @return $this
9797
*/
98-
public function addFrom(string $address, string $name = null)
98+
public function addFrom(string $address, string $name = null): object
9999
{
100100
$this->message->addFrom($name ? new NamedAddress($address, $name) : new Address($address));
101101

@@ -113,7 +113,7 @@ public function getFrom(): array
113113
/**
114114
* @return $this
115115
*/
116-
public function addReplyTo(string $address)
116+
public function addReplyTo(string $address): object
117117
{
118118
$this->message->addReplyTo($address);
119119

@@ -131,7 +131,7 @@ public function getReplyTo(): array
131131
/**
132132
* @return $this
133133
*/
134-
public function addTo(string $address, string $name = null)
134+
public function addTo(string $address, string $name = null): object
135135
{
136136
$this->message->addTo($name ? new NamedAddress($address, $name) : new Address($address));
137137

@@ -149,7 +149,7 @@ public function getTo(): array
149149
/**
150150
* @return $this
151151
*/
152-
public function addCc(string $address, string $name = null)
152+
public function addCc(string $address, string $name = null): object
153153
{
154154
$this->message->addCc($name ? new NamedAddress($address, $name) : new Address($address));
155155

@@ -167,7 +167,7 @@ public function getCc(): array
167167
/**
168168
* @return $this
169169
*/
170-
public function addBcc(string $address, string $name = null)
170+
public function addBcc(string $address, string $name = null): object
171171
{
172172
$this->message->addBcc($name ? new NamedAddress($address, $name) : new Address($address));
173173

@@ -185,7 +185,7 @@ public function getBcc(): array
185185
/**
186186
* @return $this
187187
*/
188-
public function setPriority(int $priority)
188+
public function setPriority(int $priority): object
189189
{
190190
$this->message->setPriority($priority);
191191

‎src/Symfony/Bundle/FrameworkBundle/CacheWarmer/RouterCacheWarmer.php‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,15 +57,15 @@ public function warmUp($cacheDir)
5757
*
5858
* @return bool always true
5959
*/
60-
public function isOptional()
60+
public function isOptional(): bool
6161
{
6262
return true;
6363
}
6464

6565
/**
6666
* {@inheritdoc}
6767
*/
68-
public static function getSubscribedServices()
68+
public static function getSubscribedServices(): array
6969
{
7070
return [
7171
'router' => RouterInterface::class,

0 commit comments

Comments
 (0)