Don’t change the extension if the format is the same

Sija · Jun 16, 2024 · a0d624c · a0d624c
1 parent 91c2753
commit a0d624c
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/src/ext/kemal_gphoto2.cr b/src/ext/kemal_gphoto2.cr
@@ -117,7 +117,8 @@ def send_file(env, file : GPhoto2::CameraFile, *, format : ImageOutputFormat?, w
   end
   format ||= ImageOutputFormat::JPEG
 
-  if format == path_format && !(width || height)
+  is_same_format = (format == path_format)
+  if is_same_format && !(width || height)
     send_file env, file,
       disposition: disposition
     return
@@ -151,7 +152,10 @@ def send_file(env, file : GPhoto2::CameraFile, *, format : ImageOutputFormat?, w
       str << "-w#{width}" if width
       str << "-h#{height}" if height
       str <<
-        if path.extension.chars.select!(&.letter?).all?(&.uppercase?)
+        case
+        when is_same_format
+          path.extension
+        when path.extension.chars.select!(&.letter?).all?(&.uppercase?)
           format.extension.upcase
         else
           format.extension