NGSTACK-901 CS fixes

Author: hknezevic

bundle/Core/Imagine/VariationPathGenerator/WebpFormatVariationPathGenerator.php

@@ -7,6 +7,7 @@
 use Ibexa\Contracts\Core\Variation\VariationPathGenerator;
 use Liip\ImagineBundle\Imagine\Filter\FilterConfiguration;
 
+use function array_key_exists;
 use function pathinfo;
 use function preg_replace;
 
@@ -21,7 +22,7 @@ final class WebpFormatVariationPathGenerator implements VariationPathGenerator
 
     public function __construct(
         VariationPathGenerator $innerVariationPathGenerator,
-        FilterConfiguration $filterConfiguration
+        FilterConfiguration $filterConfiguration,
     ) {
         $this->innerVariationPathGenerator = $innerVariationPathGenerator;
         $this->filterConfiguration = $filterConfiguration;
@@ -38,10 +39,10 @@ public function getVariationPath($originalPath, $filter): string
 
         $info = pathinfo($originalPath);
 
-        if (!is_string($info['extension']) || strlen($info['extension']) === 0) {
+        if (!array_key_exists('extension', $info) || $info['extension'] === '') {
             return $variationPath . '.webp';
         }
 
-        return preg_replace("/\.{$info['extension']}$/", '.webp', $variationPath);
+        return preg_replace("/\\.{$info['extension']}$/", '.webp', $variationPath) ?? $variationPath;
     }
 }

bundle/DependencyInjection/Compiler/WebpFormatVariationPathGeneratorDecoratorPass.php

@@ -4,20 +4,27 @@
 
 namespace Netgen\Bundle\SiteBundle\DependencyInjection\Compiler;
 
-use Netgen\Bundle\SiteBundle\Core\Imagine\VariationPathGenerator\WebpFormatVariationPathGenerator;
 use Ibexa\Bundle\Core\Imagine\VariationPathGenerator\WebpFormatVariationPathGenerator as BaseWebpFormatVariationPathGenerator;
+use Netgen\Bundle\SiteBundle\Core\Imagine\VariationPathGenerator\WebpFormatVariationPathGenerator;
 use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
-use Symfony\Component\DependencyInjection\Reference;
+
+use function array_keys;
+use function in_array;
+use function is_array;
 
 class WebpFormatVariationPathGeneratorDecoratorPass implements CompilerPassInterface
 {
     /**
      * Overrides default Webp image alias variation path generator decorator to comply with legacy variation URL pattern
-     * We do this only if we have Netgen AdminUI installed (legacy-based administration)
+     * We do this only if we have Netgen AdminUI installed (legacy-based administration).
      */
     public function process(ContainerBuilder $container): void
     {
+        if (!$container->hasParameter('kernel.bundles') || !is_array($container->getParameter('kernel.bundles'))) {
+            return;
+        }
+
         $activatedBundles = array_keys($container->getParameter('kernel.bundles'));
 
         if (!in_array('NetgenAdminUIBundle', $activatedBundles, true)) {