Config: Add settings for album downloads

[gusmank7]

As a user I want to be able to configure album downloads in the same way as I can configure file downloads.

---

Based on the documents for setting.yml. We can modify the name for file download with file, original or share.

But I found one issue that still unsolvable, and it stated in the note:

Note that your choice will not affect the file names in ZIP archives when you download complete albums, as they always use share-friendly names. However, we may add settings for this in a future release.

I use Photoprism to culling & share event images folder with friends, and most of the time we will just press the download button on the top bar. This result the file name use share-friendly format.

As I use my own file name format like yymmdd-eventname-0001.jpg it will be great if the global setting also affect the download complete albums/folder.

[gusmank7]

Hi is there any possibilities for implementing this? @lastzero

[graciousgrey]

Additional download settings, including settings for album downloads, are planned. However, we cannot yet say when we will get to this.

[gusmank7]

Thank you for your feedback @graciousgrey

If i’m not wrong, I think I found the configuration for this in the source code in internal/api/download_album.go file,

fileName := photoprism.FileName(file.FileRoot, file.FileName)
alias := file.ShareBase(0)
key := strings.ToLower(alias)

if seq := aliases[key]; seq > 0 {
	alias = file.ShareBase(seq)
			}

aliases[key] += 1

But i don't know anything about Go. I tried a stupid way to change the ShareBase to OriginalBase but the build-go command return an error. 😅

is there any short term solution to force the use of OriginalBase in the code, before the official system is release?

[graciousgrey]

@gusmank7 please see: #4626

[per2jensen]

I have a wish for the file name of the download zip file, when downloading from an album.

I suggest the file name should be of the form: <album>-download-YYYYMMDD-HHmmss.zip

The string after the timestamp as seen on the attached example should in my opinion be discarded. I understand it is for making the file name unique, but in the normal use case that is just noise.

[lastzero]

@gusmank7 These changes will add album download settings as shown in this example (you will need to manually edit your settings.yml for this as there is no frontend yet and we currently do not have time to add this to the UI):

Albums:
  Order:
    Album: oldest
    Folder: added
    Moment: oldest
    State: newest
    Month: oldest
  Download:
    Name: file
    Disabled: false
    Originals: true
    MediaRaw: false
    MediaSidecar: false

It would be great if you could do some more testing on all of these options so we can release this with confidence!

The string after the timestamp as seen on the attached example should in my opinion be discarded. I understand it is for making the file name unique, but in the normal use case that is just noise.

@per2jensen From what I remember, it is there to (a) cache the file without conflicts, and (b) make it difficult/impossible to download the file without knowing the random filename. So without investing time in other security measures, we cannot remove it, although it might be possible to tell the browser to save it under a different name... Anyway, we need to take a look at the AI features next, as these need improvement too.