add return types for symfony 6

src/Command/MakerCommand.php

@@ -50,12 +50,12 @@ public function __construct(MakerInterface $maker, FileManager $fileManager, Gen
         parent::__construct();
     }
 
-    protected function configure()
+    protected function configure(): void
     {
         $this->maker->configureCommand($this, $this->inputConfig);
     }
 
-    protected function initialize(InputInterface $input, OutputInterface $output)
+    protected function initialize(InputInterface $input, OutputInterface $output): void
     {
         $this->io = new ConsoleStyle($input, $output);
         $this->fileManager->setIO($this->io);
@@ -74,7 +74,7 @@ protected function initialize(InputInterface $input, OutputInterface $output)
         }
     }
 
-    protected function interact(InputInterface $input, OutputInterface $output)
+    protected function interact(InputInterface $input, OutputInterface $output): void
     {
         if (!$this->fileManager->isNamespaceConfiguredToAutoload($this->generator->getRootNamespace())) {
             $this->io->note([
@@ -111,7 +111,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         return 0;
     }
 
-    public function setApplication(Application $application = null)
+    public function setApplication(Application $application = null): void
     {
         parent::setApplication($application);
 
@@ -127,7 +127,7 @@ public function setApplication(Application $application = null)
     /**
      * @internal Used for testing commands
      */
-    public function setCheckDependencies(bool $checkDeps)
+    public function setCheckDependencies(bool $checkDeps): void
     {
         $this->checkDependencies = $checkDeps;
     }

src/ConsoleStyle.php

@@ -30,12 +30,12 @@ public function __construct(InputInterface $input, OutputInterface $output)
         parent::__construct($input, $output);
     }
 
-    public function success($message)
+    public function success($message): void
     {
         $this->writeln('<fg=green;options=bold,underscore>OK</> '.$message);
     }
 
-    public function comment($message)
+    public function comment($message): void
     {
         $this->text($message);
     }

src/DependencyBuilder.php

@@ -25,7 +25,7 @@ final class DependencyBuilder
      * the user if other required dependencies are missing. An example
      * is the "validator" when trying to work with forms.
      */
-    public function addClassDependency(string $class, string $package, bool $required = true, bool $devDependency = false)
+    public function addClassDependency(string $class, string $package, bool $required = true, bool $devDependency = false): void
     {
         if ($devDependency) {
             $this->devDependencies[] = [
@@ -42,7 +42,7 @@ public function addClassDependency(string $class, string $package, bool $require
         }
     }
 
-    public function requirePHP71()
+    public function requirePHP71(): void
     {
         // no-op - MakerBundle now required PHP 7.1
     }

src/DependencyInjection/CompilerPass/MakeCommandRegistrationPass.php

@@ -25,7 +25,7 @@ class MakeCommandRegistrationPass implements CompilerPassInterface
 {
     public const MAKER_TAG = 'maker.command';
 
-    public function process(ContainerBuilder $container)
+    public function process(ContainerBuilder $container): void
     {
         foreach ($container->findTaggedServiceIds(self::MAKER_TAG) as $id => $tags) {
             $def = $container->getDefinition($id);

src/DependencyInjection/CompilerPass/RemoveMissingParametersPass.php

@@ -21,7 +21,7 @@
  */
 class RemoveMissingParametersPass implements CompilerPassInterface
 {
-    public function process(ContainerBuilder $container)
+    public function process(ContainerBuilder $container): void
     {
         if (!$container->hasParameter('twig.default_path')) {
             $container->getDefinition('maker.file_manager')

src/DependencyInjection/CompilerPass/SetDoctrineAnnotatedPrefixesPass.php

@@ -20,7 +20,7 @@
 
 class SetDoctrineAnnotatedPrefixesPass implements CompilerPassInterface
 {
-    public function process(ContainerBuilder $container)
+    public function process(ContainerBuilder $container): void
     {
         $annotatedPrefixes = null;
 

src/DependencyInjection/CompilerPass/SetDoctrineManagerRegistryClassPass.php

@@ -20,7 +20,7 @@
  */
 class SetDoctrineManagerRegistryClassPass implements CompilerPassInterface
 {
-    public function process(ContainerBuilder $container)
+    public function process(ContainerBuilder $container): void
     {
         if ($container->hasAlias(ManagerRegistry::class)) {
             $definition = $container->getDefinition('maker.entity_class_generator');

src/DependencyInjection/Configuration.php

@@ -19,7 +19,7 @@ class Configuration implements ConfigurationInterface
     /**
      * {@inheritdoc}
      */
-    public function getConfigTreeBuilder()
+    public function getConfigTreeBuilder(): TreeBuilder
     {
         $treeBuilder = new TreeBuilder('maker');
         if (method_exists($treeBuilder, 'getRootNode')) {

src/DependencyInjection/MakerExtension.php

@@ -34,7 +34,7 @@ class MakerExtension extends Extension
     /**
      * {@inheritdoc}
      */
-    public function load(array $configs, ContainerBuilder $container)
+    public function load(array $configs, ContainerBuilder $container): void
     {
         $loader = new Loader\XmlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
         $loader->load('services.xml');

src/Doctrine/BaseRelation.php

@@ -26,36 +26,36 @@ abstract class BaseRelation
 
     abstract public function isOwning(): bool;
 
-    public function getPropertyName()
+    public function getPropertyName(): string
     {
         return $this->propertyName;
     }
 
-    public function setPropertyName($propertyName)
+    public function setPropertyName(string $propertyName): self
     {
         $this->propertyName = $propertyName;
 
         return $this;
     }
 
-    public function getTargetClassName()
+    public function getTargetClassName(): string
     {
         return $this->targetClassName;
     }
 
-    public function setTargetClassName($targetClassName)
+    public function setTargetClassName(string $targetClassName): self
     {
         $this->targetClassName = $targetClassName;
 
         return $this;
     }
 
-    public function getTargetPropertyName()
+    public function getTargetPropertyName(): ?string
     {
         return $this->targetPropertyName;
     }
 
-    public function setTargetPropertyName($targetPropertyName)
+    public function setTargetPropertyName(?string $targetPropertyName): self
     {
         $this->targetPropertyName = $targetPropertyName;
 
@@ -67,7 +67,7 @@ public function isSelfReferencing(): bool
         return $this->isSelfReferencing;
     }
 
-    public function setIsSelfReferencing(bool $isSelfReferencing)
+    public function setIsSelfReferencing(bool $isSelfReferencing): self
     {
         $this->isSelfReferencing = $isSelfReferencing;
 
@@ -79,7 +79,7 @@ public function getMapInverseRelation(): bool
         return $this->mapInverseRelation;
     }
 
-    public function setMapInverseRelation(bool $mapInverseRelation)
+    public function setMapInverseRelation(bool $mapInverseRelation): self
     {
         $this->mapInverseRelation = $mapInverseRelation;
 
@@ -91,7 +91,7 @@ public function shouldAvoidSetter(): bool
         return $this->avoidSetter;
     }
 
-    public function avoidSetter(bool $avoidSetter = true)
+    public function avoidSetter(bool $avoidSetter = true): self
     {
         $this->avoidSetter = $avoidSetter;
 

src/Doctrine/BaseSingleRelation.php

@@ -27,7 +27,7 @@ public function isNullable(): bool
         return false;
     }
 
-    public function setIsNullable($isNullable)
+    public function setIsNullable(bool $isNullable): self
     {
         $this->isNullable = $isNullable;
 

src/Doctrine/DoctrineHelper.php

@@ -213,10 +213,7 @@ public function getMetadata(string $classOrNamespace = null, bool $disconnected
         return $metadata;
     }
 
-    /**
-     * @return EntityDetails|null
-     */
-    public function createDoctrineDetails(string $entityClassName)
+    public function createDoctrineDetails(string $entityClassName): ?EntityDetails
     {
         $metadata = $this->getMetadata($entityClassName);
 

src/Doctrine/EntityDetails.php

@@ -31,7 +31,7 @@ public function __construct($metadata)
         $this->metadata = $metadata;
     }
 
-    public function getRepositoryClass()
+    public function getRepositoryClass(): ?string
     {
         return $this->metadata->customRepositoryClassName;
     }
@@ -41,12 +41,12 @@ public function getIdentifier()
         return $this->metadata->identifier[0];
     }
 
-    public function getDisplayFields()
+    public function getDisplayFields(): array
     {
         return $this->metadata->fieldMappings;
     }
 
-    public function getFormFields()
+    public function getFormFields(): array
     {
         $fields = (array) $this->metadata->fieldNames;
         // Remove the primary key field if it's not managed manually

src/Doctrine/EntityRegenerator.php

@@ -40,7 +40,7 @@ public function __construct(DoctrineHelper $doctrineHelper, FileManager $fileMan
         $this->overwrite = $overwrite;
     }
 
-    public function regenerateEntities(string $classOrNamespace)
+    public function regenerateEntities(string $classOrNamespace): void
     {
         try {
             $metadata = $this->doctrineHelper->getMetadata($classOrNamespace);
@@ -215,7 +215,7 @@ private function getPathOfClass(string $class): string
         return (new \ReflectionClass($class))->getFileName();
     }
 
-    private function generateRepository(ClassMetadata $metadata)
+    private function generateRepository(ClassMetadata $metadata): void
     {
         if (!$metadata->customRepositoryClassName) {
             return;
@@ -235,7 +235,7 @@ private function generateRepository(ClassMetadata $metadata)
         $this->generator->writeChanges();
     }
 
-    private function getMappedFieldsInEntity(ClassMetadata $classMetadata)
+    private function getMappedFieldsInEntity(ClassMetadata $classMetadata): array
     {
         /* @var $classReflection \ReflectionClass */
         $classReflection = $classMetadata->reflClass;

src/Doctrine/EntityRelation.php

@@ -55,12 +55,12 @@ public function __construct(string $type, string $owningClass, string $inverseCl
         $this->isSelfReferencing = $owningClass === $inverseClass;
     }
 
-    public function setOwningProperty(string $owningProperty)
+    public function setOwningProperty(string $owningProperty): void
     {
         $this->owningProperty = $owningProperty;
     }
 
-    public function setInverseProperty(string $inverseProperty)
+    public function setInverseProperty(string $inverseProperty): void
     {
         if (!$this->mapInverseRelation) {
             throw new \Exception('Cannot call setInverseProperty() when the inverse relation will not be mapped.');
@@ -69,12 +69,12 @@ public function setInverseProperty(string $inverseProperty)
         $this->inverseProperty = $inverseProperty;
     }
 
-    public function setIsNullable(bool $isNullable)
+    public function setIsNullable(bool $isNullable): void
     {
         $this->isNullable = $isNullable;
     }
 
-    public function setOrphanRemoval(bool $orphanRemoval)
+    public function setOrphanRemoval(bool $orphanRemoval): void
     {
         $this->orphanRemoval = $orphanRemoval;
     }

src/Doctrine/ORMDependencyBuilder.php

@@ -23,7 +23,7 @@ final class ORMDependencyBuilder
     /**
      * Central method to add dependencies needed for Doctrine ORM.
      */
-    public static function buildDependencies(DependencyBuilder $dependencies)
+    public static function buildDependencies(DependencyBuilder $dependencies): void
     {
         $classes = [
             // guarantee DoctrineBundle

src/Doctrine/RelationManyToMany.php

@@ -25,7 +25,7 @@ public function isOwning(): bool
         return $this->isOwning;
     }
 
-    public function setIsOwning($isOwning)
+    public function setIsOwning($isOwning): self
     {
         $this->isOwning = $isOwning;
 

src/Doctrine/RelationOneToMany.php

@@ -25,7 +25,7 @@ public function getOrphanRemoval(): bool
         return $this->orphanRemoval;
     }
 
-    public function setOrphanRemoval($orphanRemoval)
+    public function setOrphanRemoval($orphanRemoval): self
     {
         $this->orphanRemoval = $orphanRemoval;
 

src/Doctrine/RelationOneToOne.php

@@ -25,19 +25,19 @@ public function isOwning(): bool
         return $this->isOwning;
     }
 
-    public function setIsOwning($isOwning)
+    public function setIsOwning($isOwning): self
     {
         $this->isOwning = $isOwning;
 
         return $this;
     }
 
-    public function getTargetGetterMethodName()
+    public function getTargetGetterMethodName(): string
     {
         return 'get'.Str::asCamelCase($this->getTargetPropertyName());
     }
 
-    public function getTargetSetterMethodName()
+    public function getTargetSetterMethodName(): string
     {
         return 'set'.Str::asCamelCase($this->getTargetPropertyName());
     }

src/Event/ConsoleErrorSubscriber.php

@@ -27,7 +27,7 @@ final class ConsoleErrorSubscriber implements EventSubscriberInterface
 {
     private $setExitCode = false;
 
-    public function onConsoleError(ConsoleErrorEvent $event)
+    public function onConsoleError(ConsoleErrorEvent $event): void
     {
         if (!$event->getError() instanceof RuntimeCommandException) {
             return;
@@ -43,7 +43,7 @@ public function onConsoleError(ConsoleErrorEvent $event)
         $io->error($event->getError()->getMessage());
     }
 
-    public function onConsoleTerminate(ConsoleTerminateEvent $event)
+    public function onConsoleTerminate(ConsoleTerminateEvent $event): void
     {
         if (!$this->setExitCode) {
             return;
@@ -53,7 +53,7 @@ public function onConsoleTerminate(ConsoleTerminateEvent $event)
         $event->setExitCode(1);
     }
 
-    public static function getSubscribedEvents()
+    public static function getSubscribedEvents(): array
     {
         return [
             ConsoleEvents::ERROR => 'onConsoleError',

src/EventRegistry.php

@@ -123,7 +123,7 @@ public function getAllActiveEvents(): array
     /**
      * Attempts to get the event class for a given event.
      */
-    public function getEventClassName(string $event)
+    public function getEventClassName(string $event): ?string
     {
         // if the event is already a class name, use it
         if (class_exists($event)) {
@@ -169,7 +169,7 @@ public function getEventClassName(string $event)
         return null;
     }
 
-    public function listActiveEvents(array $events)
+    public function listActiveEvents(array $events): array
     {
         foreach ($events as $key => $event) {
             $events[$key] = sprintf('%s (<fg=yellow>%s</>)', $event, self::$eventsMap[$event]);