File-Suffixes duplicated in Downloaded Album

[ildyria]

Discussed in #2440

^{Originally posted by fmaack May 20, 2024}

Lychee version

v5.3

Which PHP version are you using?

PHP 8.3

Detailed description of the problem

I am using the "Download Album"-Feature which creates a zip-file containing all the images.

The filenames of the images inside the zip-file contains the filetype two times.

Steps to reproduce the issue

1. upload an image IMG_123.jpeg into an album.
2. use the "Download Album" Button.
3. unpack the zip-File.

Result: The downloaded filename ist IMG_123.jpeg.jpeg.

Diagnostics [REQUIRED]

No response

Browser & System [REQUIRED]

No response

Please confirm (incomplete submissions will not be addressed)

• I have provided easy and step-by-step instructions to reproduce the bug.
• I understand my bug report will be removed if I haven't met the criteria above.

[trailfog]

I wanted to report this bug as well. I might have a few additional things to add, however:

• This also happens if you download a single image in the original size.
• This does not happen with images, that have been downsized during the import.
• This only happens, when the image was uploaded using the new, v5 layout.
  • When uploading the image using the v5 layout, the full filename + extension is used as the images "title". The old layout did not save the extension in the title.
  • When uploading an image with the v5 and switching back to the old layout, the same issue arises with the extension being added twice when downloading the image uploaded with the v5 interface.

Thus, I believe the issue lies with the full filename being saved during the upload, not with the download.

My quick & dirty SQL fix

Running a query, such as the following, will remove the extension from the title and result in the expected filename when downloading.

The extension in the query will have to be adjusted to your use case. Be careful when using this query, as it may delete other information from the title than expected, as it only checks for the presence of ".jpg" (anywhere) in the title and removes the last 4 characters of such titles.

USE AT YOUR OWN RISK!

UPDATE photos SET title = SUBSTRING(title, 1, LENGTH(title) - 4) WHERE title LIKE '%.jpg';

[ildyria]

@trailfog thank you for your investigations.
It seems indeed that it is the upload with the new front-end that is cause of the problem.
I am not sure whether to drop the extension from the title of an image is a good or a bad thing.

In the mean time, there is this solution: #2474
Opinion?