support custom filename

Author: QuentinGab

src/Definitions/MediaConversionAudio.php

@@ -17,6 +17,9 @@
 
 class MediaConversionAudio extends MediaConversionDefinition
 {
+    /**
+     * @param  null|string|(Closure(Media $media, ?MediaConversion $parent):string)  $fileName
+     */
     public function __construct(
         public string $name,
         public null|bool|Closure $when = null,
@@ -25,7 +28,7 @@ public function __construct(
         public bool $queued = true,
         public ?string $queue = null,
         public array $conversions = [],
-        public ?string $fileName = null,
+        public null|Closure|string $fileName = null,
         public FormatInterface $format = new Mp3,
     ) {
 
@@ -64,6 +67,17 @@ public function hasFileAudioStream(string $path): bool
         return (bool) $streams->audios()->count();
     }
 
+    public function getFileName(Media $media, ?MediaConversion $parent): string
+    {
+        if ($fileName = $this->fileName) {
+            return is_string($fileName) ? $fileName : $fileName($media, $parent);
+        }
+
+        $source = $parent ?? $media;
+
+        return "{$source->name}.mp3";
+    }
+
     public function handle(
         Media $media,
         ?MediaConversion $parent,
@@ -78,13 +92,11 @@ public function handle(
         /**
          * Videos do not always have an audio stream
          */
-        if (! $this->hasFileAudioStream(
-            $filesystem->path($file)
-        )) {
+        if (! $this->hasFileAudioStream($filesystem->path($file))) {
             return null;
         }
 
-        $fileName = $this->fileName ?? "{$media->name}.mp3";
+        $fileName = $this->getFileName($media, $parent);
 
         $ffmpeg = FFMpeg::fromFilesystem($filesystem)
             ->open($file)

src/Definitions/MediaConversionPoster.php

@@ -18,6 +18,9 @@
 
 class MediaConversionPoster extends MediaConversionDefinition
 {
+    /**
+     * @param  null|string|(Closure(Media $media, ?MediaConversion $parent):string)  $fileName
+     */
     public function __construct(
         public string $name,
         public null|bool|Closure $when = null,
@@ -26,7 +29,7 @@ public function __construct(
         public bool $queued = false,
         public ?string $queue = null,
         public array $conversions = [],
-        public ?string $fileName = null,
+        public null|Closure|string $fileName = null,
         public TimeCode|float $seconds = 0.0,
         public ?int $width = null,
         public ?int $height = null,
@@ -55,6 +58,17 @@ public function shouldExecute(Media $media, ?MediaConversion $parent): bool
         return ($parent ?? $media)->type === MediaType::Video;
     }
 
+    public function getFileName(Media $media, ?MediaConversion $parent): string
+    {
+        if ($fileName = $this->fileName) {
+            return is_string($fileName) ? $fileName : $fileName($media, $parent);
+        }
+
+        $source = $parent ?? $media;
+
+        return "{$source->name}.jpg";
+    }
+
     public function handle(
         Media $media,
         ?MediaConversion $parent,
@@ -66,7 +80,7 @@ public function handle(
             return null;
         }
 
-        $fileName = $this->fileName ?? "{$media->name}.jpg";
+        $fileName = $this->getFileName($media, $parent);
 
         FFMpeg::fromFilesystem($filesystem)
             ->open($file)