Support arrays of Imager X transforms in retconSrcset filter. Fixes #69

Author: mmikkel

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Retcon Changelog
 
+## 2.8.0 - 2024-06-28
+### Improved
+- Improves support for Imager X named transforms for the `retconSrcset` filter. [#69](https://github.com/mmikkel/Retcon-Craft/issues/69)
+
 ## 2.7.5 - 2023-11-11
 
 ### Fixed

composer.json

@@ -2,7 +2,7 @@
     "name": "mmikkel/retcon",
     "description": "Powerful Twig filters for mutating and querying HTML",
     "type": "craft-plugin",
-    "version": "2.7.5",
+    "version": "2.8.0",
     "keywords": [
         "craft",
         "cms",

src/helpers/RetconHelper.php

@@ -135,12 +135,12 @@ public static function getImageTransform($transform)
      * @param $transform
      * @param array|null $imagerTransformDefaults
      * @param array|null $imagerConfigOverrides
-     * @return RetconTransformedImage|null
+     * @return RetconTransformedImage|RetconTransformedImage[]|null
      * @throws Exception
      * @throws \craft\errors\ImageException
      * @throws \spacecatninja\imagerx\exceptions\ImagerException
      */
-    public static function getTransformedImage(string $src, $transform, ?array $imagerTransformDefaults = null, ?array $imagerConfigOverrides = null): ?RetconTransformedImage
+    public static function getTransformedImage(string $src, $transform, ?array $imagerTransformDefaults = null, ?array $imagerConfigOverrides = null): RetconTransformedImage|array|null
     {
 
         // TODO: In Retcon 3.0, we should try to get the asset via RetconHelper::getAssetFromRef(), and transform that directly
@@ -169,7 +169,17 @@ public static function getTransformedImage(string $src, $transform, ?array $imag
                 return null;
             }
             if (is_array($transformedImage)) {
-                $transformedImage = $transformedImage[0] ?? null;
+                return array_reduce($transformedImage, static function (array $carry, $transformedImage) {
+                    if (empty($transformedImage)) {
+                        return $carry;
+                    }
+                    $carry[] = new RetconTransformedImage([
+                        'url' => $transformedImage->getUrl(),
+                        'width' => $transformedImage->getWidth(),
+                        'height' => $transformedImage->getHeight(),
+                    ]);
+                    return $carry;
+                }, []) ?: null;
             }
             return new RetconTransformedImage([
                 'url' => $transformedImage->getUrl(),

src/services/RetconService.php

@@ -125,10 +125,15 @@ public function transform($input, $transform, $selector = null, ?array $imagerTr
                 continue;
             }
 
-            if (!$transformedImage = RetconHelper::getTransformedImage($src, $transform, $imagerTransformDefaults, $imagerConfigOverrides)) {
+            $transformedImage = RetconHelper::getTransformedImage($src, $transform, $imagerTransformDefaults, $imagerConfigOverrides);
+            if (empty($transformedImage)) {
                 continue;
             }
 
+            if (is_array($transformedImage)) {
+                $transformedImage = $transformedImage[0];
+            }
+
             $node->setAttribute('src', $transformedImage->url);
 
             // Should we set width and height attributes?
@@ -229,9 +234,14 @@ public function srcset($input, $transforms, $selector = null, $sizes = null, ?bo
 
             // Get transformed images
             $transformedImages = \array_reduce($transforms, static function ($carry, $transform) use ($src, $imagerTransformDefaults, $imagerConfigOverrides) {
-                if ($transformedImage = RetconHelper::getTransformedImage($src, $transform, $imagerTransformDefaults, $imagerConfigOverrides)) {
-                    $carry[] = $transformedImage;
+                $transformedImage = RetconHelper::getTransformedImage($src, $transform, $imagerTransformDefaults, $imagerConfigOverrides);
+                if (empty($transformedImage)) {
+                    return $carry;
+                }
+                if (is_array($transformedImage)) {
+                    return array_merge($carry, $transformedImage);
                 }
+                $carry[] = $transformedImage;
                 return $carry;
             }, []);