Switch from yaml to php format services definition

Author: norkunas

.editorconfig

@@ -8,5 +8,18 @@ indent_style = space
 insert_final_newline = true
 trim_trailing_whitespace = true
 
-[*.{yml,yaml}]
+[.github/**/*.{yml,yaml}]
 indent_size = 2
+
+[*.{yml,yaml}]
+trim_trailing_whitespace = false
+
+[**.md]
+indent_size = unset
+indent_style = unset
+
+[*.json]
+insert_final_newline = false
+
+[*.{neon,neon.dist}]
+indent_style = tab

config/profiling.php

@@ -0,0 +1,14 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Symfony\Component\DependencyInjection\Loader\Configurator;
+
+use Bazinga\GeocoderBundle\DataCollector\GeocoderDataCollector;
+
+return static function (ContainerConfigurator $container) {
+    $services = $container->services();
+
+    $services->set(GeocoderDataCollector::class)
+        ->tag('data_collector', ['template' => '@BazingaGeocoder/Collector/geocoder.html.twig', 'id' => 'geocoder']);
+};

config/profiling.yml

@@ -0,0 +1,55 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Symfony\Component\DependencyInjection\Loader\Configurator;
+
+use Bazinga\GeocoderBundle\Command\GeocodeCommand;
+use Bazinga\GeocoderBundle\Plugin\FakeIpPlugin;
+use Bazinga\GeocoderBundle\Validator\Constraint\AddressValidator;
+use Geocoder\Dumper\Dumper;
+use Geocoder\Dumper\GeoArray;
+use Geocoder\Dumper\GeoJson;
+use Geocoder\Dumper\Gpx;
+use Geocoder\Dumper\Kml;
+use Geocoder\Dumper\Wkb;
+use Geocoder\Dumper\Wkt;
+use Geocoder\ProviderAggregator;
+
+return static function (ContainerConfigurator $container) {
+    $services = $container->services();
+    $services->instanceof(Dumper::class)
+        ->tag('bazinga_geocoder.dumper')
+        ->public();
+
+    $services
+        ->set(GeoArray::class)
+        ->set(GeoJson::class)
+        ->set(Gpx::class)
+        ->set(Kml::class)
+        ->set(Wkb::class)
+        ->set(Wkt::class)
+
+        ->load('Bazinga\\GeocoderBundle\\ProviderFactory\\', __DIR__.'/../src/ProviderFactory')
+            ->autowire()
+            ->autoconfigure()
+            ->private()
+
+        ->set(ProviderAggregator::class)
+
+        ->set(FakeIpPlugin::class)
+            ->args([null, null, false])
+
+        ->set(GeocodeCommand::class)
+            ->args([
+                service(ProviderAggregator::class),
+            ])
+            ->tag('console.command', ['command' => 'geocoder:geocode', 'description' => 'Geocode an address or a ip address'])
+
+        ->set(AddressValidator::class)
+            ->args([
+                service(ProviderAggregator::class),
+            ])
+            ->tag('validator.constraint_validator')
+    ;
+};

config/services.php

@@ -31,7 +31,7 @@
 use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\DependencyInjection\Exception\ServiceNotFoundException;
 use Symfony\Component\DependencyInjection\Extension\Extension;
-use Symfony\Component\DependencyInjection\Loader\YamlFileLoader;
+use Symfony\Component\DependencyInjection\Loader\PhpFileLoader;
 use Symfony\Component\DependencyInjection\Reference;
 
 /**
@@ -50,11 +50,11 @@ public function load(array $configs, ContainerBuilder $container)
         $configuration = $this->getConfiguration($configs, $container);
         $config = $processor->processConfiguration($configuration, $configs);
 
-        $loader = new YamlFileLoader($container, new FileLocator(__DIR__.'/../../config'));
-        $loader->load('services.yml');
+        $loader = new PhpFileLoader($container, new FileLocator(__DIR__.'/../../config'));
+        $loader->load('services.php');
 
         if (true === $config['profiling']['enabled']) {
-            $loader->load('profiling.yml');
+            $loader->load('profiling.php');
         }
 
         if ($config['fake_ip']['enabled']) {

config/services.yml

@@ -160,28 +160,6 @@ public function testBundleWithPluginsYml(): void
         self::assertInstanceOf(LoggerPlugin::class, $plugins[0]);
     }
 
-    public function testBundleWithPluginXml(): void
-    {
-        self::bootKernel(['config' => static function (TestKernel $kernel) {
-            $kernel->addTestConfig(__DIR__.'/config/framework.yml');
-
-            if ($kernel::VERSION_ID >= 60000) {
-                $kernel->addTestConfig(__DIR__.'/config/framework_sf6.yml');
-            }
-
-            $kernel->addTestConfig(__DIR__.'/config/service_plugin.xml');
-        }]);
-
-        $container = self::getContainer();
-
-        self::assertTrue($container->has('bazinga_geocoder.provider.acme'));
-        $service = $container->get('bazinga_geocoder.provider.acme');
-        self::assertInstanceOf(PluginProvider::class, $service);
-        $plugins = NSA::getProperty($service, 'plugins');
-        self::assertNotEmpty($plugins);
-        self::assertInstanceOf(LoggerPlugin::class, $plugins[0]);
-    }
-
     public function testBundleHasRegisteredDumpers(): void
     {
         self::bootKernel(['config' => static function (TestKernel $kernel) {