Remove 'local' pickup points

composer.json

@@ -24,7 +24,6 @@
         "symfony/http-foundation": "^5.4 || ^6.0",
         "symfony/http-kernel": "^5.4 || ^6.0",
         "symfony/intl": "^5.4 || ^6.0",
-        "symfony/messenger": "^5.4 || ^6.0",
         "symfony/options-resolver": "^5.4 || ^6.0",
         "symfony/routing": "^5.4 || ^6.0",
         "symfony/security-bundle": "^5.4 || ^6.0",

src/DependencyInjection/Compiler/RegisterProvidersPass.php

@@ -6,7 +6,6 @@
 
 use InvalidArgumentException;
 use Setono\SyliusPickupPointPlugin\Provider\CachedProvider;
-use Setono\SyliusPickupPointPlugin\Provider\LocalProvider;
 use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\DependencyInjection\Definition;
@@ -22,7 +21,6 @@ public function process(ContainerBuilder $container): void
 
         $registry = $container->getDefinition('setono_sylius_pickup_point.registry.provider');
         $cacheEnabled = $container->getParameter('setono_sylius_pickup_point.cache.enabled') === true;
-        $localEnabled = $container->getParameter('setono_sylius_pickup_point.local') === true;
 
         $typeToLabelMap = [];
         foreach ($container->findTaggedServiceIds('setono_sylius_pickup_point.provider') as $id => $tagged) {
@@ -45,18 +43,6 @@ public function process(ContainerBuilder $container): void
                     $container->setDefinition($id, $cachedDefinition);
                 }
 
-                if ($localEnabled) {
-                    $decoratedId = $id;
-                    $id .= '.local'; // overwrite the id
-                    $cachedDefinition = new Definition(LocalProvider::class, [
-                        new Reference($id . '.inner'),
-                        new Reference('setono_sylius_pickup_point.repository.pickup_point'),
-                    ]);
-                    $cachedDefinition->setDecoratedService($decoratedId, null, 512);
-
-                    $container->setDefinition($id, $cachedDefinition);
-                }
-
                 $registry->addMethodCall('register', [$attributes['code'], new Reference($id)]);
             }
         }

src/DependencyInjection/Configuration.php

@@ -7,13 +7,7 @@
 use Setono\DAOBundle\SetonoDAOBundle;
 use Setono\GlsWebserviceBundle\SetonoGlsWebserviceBundle;
 use Setono\PostNordBundle\SetonoPostNordBundle;
-use Setono\SyliusPickupPointPlugin\Doctrine\ORM\PickupPointRepository;
-use Setono\SyliusPickupPointPlugin\Model\PickupPoint;
-use Sylius\Bundle\ResourceBundle\Controller\ResourceController;
-use Sylius\Bundle\ResourceBundle\Form\Type\DefaultResourceType;
 use Sylius\Bundle\ResourceBundle\SyliusResourceBundle;
-use Sylius\Component\Resource\Factory\Factory;
-use Symfony\Component\Config\Definition\Builder\ArrayNodeDefinition;
 use Symfony\Component\Config\Definition\Builder\TreeBuilder;
 use Symfony\Component\Config\Definition\ConfigurationInterface;
 
@@ -40,11 +34,6 @@ public function getConfigTreeBuilder(): TreeBuilder
                         ->end()
                     ->end()
                 ->end()
-                ->booleanNode('local')
-                    ->defaultValue(true)
-                    ->info('Whether to use the local database when timeouts occur in third party HTTP calls. Remember to run the setono-sylius-pickup-point:load-pickup-points command periodically to populate the local database with pickup points')
-                    ->example(true)
-                ->end()
                 ->arrayNode('providers')
                     ->addDefaultsIfNotSet()
                     ->children()
@@ -72,31 +61,6 @@ public function getConfigTreeBuilder(): TreeBuilder
             ->end()
         ;
 
-        $this->addResourcesSection($rootNode);
-
         return $treeBuilder;
     }
-
-    private function addResourcesSection(ArrayNodeDefinition $node): void
-    {
-        /** @psalm-suppress MixedMethodCall,PossiblyUndefinedMethod,PossiblyNullReference */
-        $node
-            ->children()
-                ->arrayNode('resources')
-                    ->addDefaultsIfNotSet()
-                    ->children()
-                        ->arrayNode('pickup_point')
-                            ->addDefaultsIfNotSet()
-                            ->children()
-                                ->variableNode('options')->end()
-                                ->arrayNode('classes')
-                                    ->addDefaultsIfNotSet()
-                                    ->children()
-                                        ->scalarNode('model')->defaultValue(PickupPoint::class)->cannotBeEmpty()->end()
-                                        ->scalarNode('controller')->defaultValue(ResourceController::class)->cannotBeEmpty()->end()
-                                        ->scalarNode('repository')->defaultValue(PickupPointRepository::class)->cannotBeEmpty()->end()
-                                        ->scalarNode('form')->defaultValue(DefaultResourceType::class)->end()
-                                        ->scalarNode('factory')->defaultValue(Factory::class)->end()
-        ;
-    }
 }

src/DependencyInjection/SetonoSyliusPickupPointExtension.php

@@ -19,9 +19,6 @@ public function load(array $configs, ContainerBuilder $container): void
         /** @psalm-suppress PossiblyNullArgument */
         $config = $this->processConfiguration($this->getConfiguration([], $container), $configs);
         $loader = new XmlFileLoader($container, new FileLocator(__DIR__ . '/../Resources/config'));
-        $container->setParameter('setono_sylius_pickup_point.local', $config['local']);
-
-        $this->registerResources('setono_sylius_pickup_point', $config['driver'], $config['resources'], $container);
 
         $loader->load('services.xml');
 

src/Factory/PickupPointFactory.php

@@ -0,0 +1,16 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Setono\SyliusPickupPointPlugin\Factory;
+
+use Setono\SyliusPickupPointPlugin\Model\PickupPoint;
+use Setono\SyliusPickupPointPlugin\Model\PickupPointInterface;
+
+final class PickupPointFactory implements PickupPointFactoryInterface
+{
+    public function createNew(): PickupPointInterface
+    {
+        return new PickupPoint();
+    }
+}

src/Factory/PickupPointFactoryInterface.php

@@ -0,0 +1,12 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Setono\SyliusPickupPointPlugin\Factory;
+
+use Setono\SyliusPickupPointPlugin\Model\PickupPointInterface;
+
+interface PickupPointFactoryInterface
+{
+    public function createNew(): PickupPointInterface;
+}