Add types declarations

Author: ker0x

src/BazingaGeocoderBundle.php

@@ -23,10 +23,7 @@
  */
 class BazingaGeocoderBundle extends Bundle
 {
-    /**
-     * @return void
-     */
-    public function build(ContainerBuilder $container)
+    public function build(ContainerBuilder $container): void
     {
         parent::build($container);
 

src/Command/GeocodeCommand.php

@@ -34,10 +34,7 @@ public function __construct(ProviderAggregator $geocoder)
         parent::__construct();
     }
 
-    /**
-     * @return void
-     */
-    protected function configure()
+    protected function configure(): void
     {
         $this
             ->setName('geocoder:geocode')
@@ -55,10 +52,7 @@ protected function configure()
             );
     }
 
-    /**
-     * @return int
-     */
-    protected function execute(InputInterface $input, OutputInterface $output)
+    protected function execute(InputInterface $input, OutputInterface $output): int
     {
         if ($input->getOption('provider')) {
             $this->geocoder->using($input->getOption('provider'));

src/DataCollector/GeocoderDataCollector.php

@@ -34,20 +34,14 @@ public function __construct()
         $this->data['providers'] = [];
     }
 
-    /**
-     * @return void
-     */
-    public function reset()
+    public function reset(): void
     {
         $this->instances = [];
         $this->data['queries'] = [];
         $this->data['providers'] = [];
     }
 
-    /**
-     * @return void
-     */
-    public function collect(Request $request, Response $response, \Throwable $exception = null)
+    public function collect(Request $request, Response $response, \Throwable $exception = null): void
     {
         if (!empty($this->data['queries'])) {
             // To avoid collection more that once.
@@ -101,15 +95,10 @@ public function getProviders(): array
      */
     public function getProviderQueries(string $provider): array
     {
-        return array_filter($this->data['queries'], static function ($data) use ($provider) {
-            return $data['providerName'] === $provider;
-        });
+        return array_filter($this->data['queries'], static fn ($data): bool => $data['providerName'] === $provider);
     }
 
-    /**
-     * @return void
-     */
-    public function addInstance(ProfilingPlugin $instance)
+    public function addInstance(ProfilingPlugin $instance): void
     {
         $this->instances[] = $instance;
         $this->data['providers'][] = $instance->getName();

src/DependencyInjection/BazingaGeocoderExtension.php

@@ -41,10 +41,8 @@ class BazingaGeocoderExtension extends Extension
 {
     /**
      * @param array<mixed, mixed> $configs
-     *
-     * @return void
      */
-    public function load(array $configs, ContainerBuilder $container)
+    public function load(array $configs, ContainerBuilder $container): void
     {
         $processor = new Processor();
         $configuration = $this->getConfiguration($configs, $container);
@@ -78,10 +76,8 @@ public function load(array $configs, ContainerBuilder $container)
 
     /**
      * @param array<mixed, mixed> $config
-     *
-     * @return void
      */
-    private function loadProviders(ContainerBuilder $container, array $config)
+    private function loadProviders(ContainerBuilder $container, array $config): void
     {
         foreach ($config['providers'] as $providerName => $providerConfig) {
             try {
@@ -181,7 +177,7 @@ public function configureProviderPlugins(ContainerBuilder $container, array $con
                 ->addTag('bazinga_geocoder.profiling_plugin');
         }
 
-        return array_map(static fn (string $id) => new Reference($id), $plugins);
+        return array_map(static fn (string $id): Reference => new Reference($id), $plugins);
     }
 
     /**

src/DependencyInjection/Compiler/AddProvidersPass.php

@@ -25,10 +25,8 @@ class AddProvidersPass implements CompilerPassInterface
     /**
      * Get all providers based on their tag (`bazinga_geocoder.provider`) and
      * register them.
-     *
-     * @return void
      */
-    public function process(ContainerBuilder $container)
+    public function process(ContainerBuilder $container): void
     {
         if (!$container->hasDefinition(ProviderAggregator::class)) {
             return;

src/DependencyInjection/Compiler/FactoryValidatorPass.php

@@ -26,12 +26,9 @@ class FactoryValidatorPass implements CompilerPassInterface
     /**
      * @var string[]
      */
-    private static $factoryServiceIds = [];
+    private static array $factoryServiceIds = [];
 
-    /**
-     * @return void
-     */
-    public function process(ContainerBuilder $container)
+    public function process(ContainerBuilder $container): void
     {
         foreach (self::$factoryServiceIds as $id) {
             if (!$container->hasAlias($id) && !$container->hasDefinition($id)) {
@@ -40,12 +37,7 @@ public function process(ContainerBuilder $container)
         }
     }
 
-    /**
-     * @param string $factoryServiceId
-     *
-     * @return void
-     */
-    public static function addFactoryServiceId($factoryServiceId)
+    public static function addFactoryServiceId(string $factoryServiceId): void
     {
         self::$factoryServiceIds[] = $factoryServiceId;
     }

src/DependencyInjection/Compiler/ProfilerPass.php

@@ -24,10 +24,7 @@
  */
 class ProfilerPass implements CompilerPassInterface
 {
-    /**
-     * @return void
-     */
-    public function process(ContainerBuilder $container)
+    public function process(ContainerBuilder $container): void
     {
         if (!$container->hasDefinition(GeocoderDataCollector::class)) {
             return;

src/DependencyInjection/Configuration.php

@@ -30,10 +30,8 @@ public function __construct(bool $debug)
 
     /**
      * Generates the configuration tree builder.
-     *
-     * @return TreeBuilder The tree builder
      */
-    public function getConfigTreeBuilder()
+    public function getConfigTreeBuilder(): TreeBuilder
     {
         $treeBuilder = new TreeBuilder('bazinga_geocoder');
         $rootNode = $treeBuilder->getRootNode();
@@ -58,9 +56,7 @@ public function getConfigTreeBuilder()
             ->arrayNode('fake_ip')
                 ->beforeNormalization()
                 ->ifString()
-                    ->then(function ($value) {
-                        return ['ip' => $value];
-                    })
+                    ->then(fn ($value): array => ['ip' => $value])
                 ->end()
                 ->canBeEnabled()
                 ->children()
@@ -75,10 +71,7 @@ public function getConfigTreeBuilder()
         return $treeBuilder;
     }
 
-    /**
-     * @return ArrayNodeDefinition
-     */
-    private function getProvidersNode()
+    private function getProvidersNode(): ArrayNodeDefinition
     {
         $treeBuilder = new TreeBuilder('providers');
         $rootNode = $treeBuilder->getRootNode();
@@ -113,10 +106,8 @@ private function getProvidersNode()
 
     /**
      * Create plugin node of a client.
-     *
-     * @return ArrayNodeDefinition The plugin node
      */
-    private function createClientPluginNode()
+    private function createClientPluginNode(): ArrayNodeDefinition
     {
         $treeBuilder = new TreeBuilder('plugins');
         $rootNode = $treeBuilder->getRootNode();

src/Doctrine/ORM/GeocoderListener.php

@@ -45,10 +45,7 @@ public function getSubscribedEvents(): array
         ];
     }
 
-    /**
-     * @return void
-     */
-    public function onFlush(OnFlushEventArgs $args)
+    public function onFlush(OnFlushEventArgs $args): void
     {
         $em = method_exists($args, 'getObjectManager') ? $args->getObjectManager() : $args->getEntityManager();
         $uow = $em->getUnitOfWork();
@@ -92,10 +89,8 @@ public function onFlush(OnFlushEventArgs $args)
 
     /**
      * @param object $entity
-     *
-     * @return void
      */
-    private function geocodeEntity(ClassMetadata $metadata, $entity)
+    private function geocodeEntity(ClassMetadata $metadata, $entity): void
     {
         if (null !== $metadata->addressGetter) {
             $address = $metadata->addressGetter->invoke($entity);

src/Mapping/ClassMetadata.php

@@ -17,23 +17,11 @@
  */
 class ClassMetadata
 {
-    /**
-     * @var \ReflectionProperty
-     */
-    public $addressProperty;
+    public \ReflectionProperty $addressProperty;
 
-    /**
-     * @var \ReflectionProperty
-     */
-    public $latitudeProperty;
+    public \ReflectionProperty $latitudeProperty;
 
-    /**
-     * @var \ReflectionProperty
-     */
-    public $longitudeProperty;
+    public \ReflectionProperty $longitudeProperty;
 
-    /**
-     * @var \ReflectionMethod
-     */
-    public $addressGetter;
+    public ?\ReflectionMethod $addressGetter = null;
 }