Review changes

Author: konarshankar07

MediaGalleryRenditions/Model/GenerateRenditionImages.php

@@ -7,52 +7,51 @@
 
 namespace Magento\MediaGalleryRenditions\Model;
 
-use Magento\MediaGallerySynchronizationApi\Api\ImportFileInterface;
-use Magento\MediaGallerySynchronization\Model\Filesystem\SplFileInfoFactory;
-use Magento\MediaGallerySynchronization\Model\CreateAssetFromFile;
+use Magento\Framework\Exception\LocalizedException;
+use Magento\MediaGallerySynchronization\Model\GetAssetFromPath;
+use Magento\MediaGallerySynchronizationApi\Model\ImportFilesInterface;
 use Magento\MediaGalleryRenditionsApi\Api\GenerateRenditionsInterface;
 
 /**
  * Generate Rendition Images
  */
-class GenerateRenditionImages implements ImportFileInterface
+class GenerateRenditionImages implements ImportFilesInterface
 {
     /**
-     * @var SplFileInfoFactory
+     * @var GetAssetFromPath
      */
-    private $splFileInfoFactory;
-
-    /**
-     * @var CreateAssetFromFile
-     */
-    private $createAssetFromFile;
+    private $getAssetFromPath;
 
     /**
      * @var GenerateRenditionsInterface
      */
     private $generateRenditions;
 
     /**
-     * @param SplFileInfoFactory $splFileInfoFactory
-     * @param CreateAssetFromFile $createAssetFromFile
+     * @param GetAssetFromPath $getAssetFromPath
      * @param GenerateRenditionsInterface $generateRenditions
      */
     public function __construct(
-        SplFileInfoFactory $splFileInfoFactory,
-        CreateAssetFromFile $createAssetFromFile,
+        GetAssetFromPath $getAssetFromPath,
         GenerateRenditionsInterface $generateRenditions
     ) {
-        $this->splFileInfoFactory = $splFileInfoFactory;
-        $this->createAssetFromFile = $createAssetFromFile;
+        $this->getAssetFromPath = $getAssetFromPath;
         $this->generateRenditions = $generateRenditions;
     }
 
     /**
-     * @inheritdoc
+     * Save media files data
+     *
+     * @param string[] $paths
+     * @throws LocalizedException
      */
-    public function execute(string $path): void
+    public function execute(array $paths): void
     {
-        $file = $this->splFileInfoFactory->create($path);
-        $this->generateRenditions->execute([$this->createAssetFromFile->execute($file)]);
+        $assets = [];
+
+        foreach ($paths as $path) {
+            $assets[] = $this->getAssetFromPath->execute($path);
+        }
+        $this->generateRenditions->execute($assets);
     }
 }

MediaGalleryRenditions/etc/di.xml

@@ -9,7 +9,7 @@
     <preference for="Magento\MediaGalleryRenditionsApi\Model\ConfigInterface" type="Magento\MediaGalleryRenditions\Model\Config"/>
     <preference for="Magento\MediaGalleryRenditionsApi\Api\GenerateRenditionsInterface" type="Magento\MediaGalleryRenditions\Model\GenerateRenditions"/>
     <preference for="Magento\MediaGalleryRenditionsApi\Api\GetRenditionPathInterface" type="Magento\MediaGalleryRenditions\Model\GetRenditionPath"/>
-    <type name="Magento\MediaGallerySynchronizationApi\Model\ImportFileComposite">
+    <type name="Magento\MediaGallerySynchronizationApi\Model\ImportFilesComposite">
         <arguments>
             <argument name="importers" xsi:type="array">
                 <item name="2" xsi:type="object">Magento\MediaGalleryRenditions\Model\GenerateRenditionImages</item>