Use main image options defined in plugin configuration

webgriffe · Jul 4, 2024 · 1dcfe13 · 1dcfe13
1 parent 806d200
commit 1dcfe13
Show file tree

Hide file tree

Showing 5 changed files with 53 additions and 6 deletions.
diff --git a/src/Converter/CommonOrderConverterTrait.php b/src/Converter/CommonOrderConverterTrait.php
@@ -32,6 +32,12 @@ abstract private function getCacheManager(): CacheManager;
 
     abstract private function getTranslator(): TranslatorInterface;
 
+    abstract private function getSchema(): string;
+
+    abstract private function getImageFilter(): string;
+
+    abstract private function getMainImageType(): string;
+
     private function getCustomer(OrderInterface $order): ?Customer
     {
         $customer = $order->getCustomer();
@@ -106,7 +112,7 @@ private function createOrderLineFromOrderItem(OrderInterface $order, OrderItemIn
                 '',
                 'GET',
                 $hostname,
-                'https',
+                $this->getSchema(),
             ));
         }
         $slug = $orderItem->getProduct()?->getSlug();
@@ -123,7 +129,7 @@ private function createOrderLineFromOrderItem(OrderInterface $order, OrderItemIn
         if ($productImagePath !== null) {
             $imageUrl = $this->getCacheManager()->getBrowserPath(
                 $productImagePath,
-                'sylius_shop_product_thumbnail',
+                $this->getImageFilter(),
             );
         }
         $this->getUrlGenerator()->setContext($previousContext);
@@ -206,7 +212,7 @@ private function getProductImagePath(?ProductInterface $product): ?string
         if ($product === null) {
             return null;
         }
-        $images = $product->getImagesByType('main_image');
+        $images = $product->getImagesByType($this->getMainImageType());
         foreach ($images as $image) {
             return $image->getPath();
         }

diff --git a/src/Converter/OrderConverter.php b/src/Converter/OrderConverter.php
@@ -24,6 +24,8 @@ public function __construct(
         private TranslatorInterface $translator,
         private UrlGeneratorInterface $urlGenerator,
         private CacheManager $cacheManager,
+        private string $mainImageType,
+        private string $imageFilter,
     ) {
     }
 
@@ -78,4 +80,19 @@ private function getTranslator(): TranslatorInterface
     {
         return $this->translator;
     }
+
+    private function getSchema(): string
+    {
+        return 'https';
+    }
+
+    private function getImageFilter(): string
+    {
+        return $this->imageFilter;
+    }
+
+    private function getMainImageType(): string
+    {
+        return $this->mainImageType;
+    }
 }
diff --git a/src/Converter/PaymentConverter.php b/src/Converter/PaymentConverter.php
@@ -27,6 +27,8 @@ public function __construct(
         private TranslatorInterface $translator,
         private UrlGeneratorInterface $urlGenerator,
         private CacheManager $cacheManager,
+        private string $mainImageType,
+        private string $imageFilter,
     ) {
     }
 
@@ -96,4 +98,19 @@ private function getTranslator(): TranslatorInterface
     {
         return $this->translator;
     }
+
+    private function getSchema(): string
+    {
+        return 'https';
+    }
+
+    private function getImageFilter(): string
+    {
+        return $this->imageFilter;
+    }
+
+    private function getMainImageType(): string
+    {
+        return $this->mainImageType;
+    }
 }
diff --git a/src/DependencyInjection/Configuration.php b/src/DependencyInjection/Configuration.php
@@ -11,7 +11,7 @@ final class Configuration implements ConfigurationInterface
 {
     public function getConfigTreeBuilder(): TreeBuilder
     {
-        $treeBuilder = new TreeBuilder('acme_sylius_example_plugin');
+        $treeBuilder = new TreeBuilder('webgriffe_sylius_klarna_plugin');
         $rootNode = $treeBuilder->getRootNode();
 
         $rootNode

diff --git a/src/DependencyInjection/WebgriffeSyliusKlarnaExtension.php b/src/DependencyInjection/WebgriffeSyliusKlarnaExtension.php
@@ -25,7 +25,7 @@ public function load(array $configs, ContainerBuilder $container): void
 
         $loader->load('services.php');
 
-        $this->addImageOptionsConverters($container, $config);
+        $this->addImageOptionsOnConverters($container, $config);
     }
 
     public function prepend(ContainerBuilder $container): void
@@ -50,7 +50,14 @@ protected function getNamespacesOfMigrationsExecutedBefore(): array
         ];
     }
 
-    private function addImageOptionsConverters(ContainerBuilder $container, array $config): void
+    private function addImageOptionsOnConverters(ContainerBuilder $container, array $config): void
     {
+        $definition = $container->getDefinition('webgriffe_sylius_klarna.converter.order');
+        $definition->setArgument('$mainImageType', $config['product_images']['type']);
+        $definition->setArgument('$imageFilter', $config['product_images']['filter']);
+
+        $definition = $container->getDefinition('webgriffe_sylius_klarna.converter.payment');
+        $definition->setArgument('$mainImageType', $config['product_images']['type']);
+        $definition->setArgument('$imageFilter', $config['product_images']['filter']);
     }
 }