scubajeff / lespas

Les Pas, photo album app for Nextcloud user
Apache License 2.0
450 stars 22 forks source link

Bug: File names are reset when adding photos to new remote album #105

Closed marcus-k closed 1 year ago

marcus-k commented 1 year ago

Describe the bug When adding pictures to a new remote album from the Les Pas app, the file names are not preserved when they are uploaded to the Nextcloud server. This did not used to happen before; my old uploaded photos have their file names as they are on my phone (i.e. yyyymmdd_hhmmss.jpg). However, any new photos I add to a new remote album get renamed to some random number that increments up for that album (see screenshot). Perhaps a new update changed this behaviour, I am not sure. I would like the file names to be preserved like prior photos I have uploaded.

To Reproduce Steps to reproduce the behavior:

  1. Click the '+' icon to create select photos
  2. Create a new remote album for those photos
  3. Observe the uploading where all the file names are renamed to the random numbers.

Expected behavior The file names to be preserved upon uploading.

Screenshots File names when uploaded in Nextcloud: image

Smartphone (please complete the following information):

scubajeff commented 1 year ago

Bug confirmed. Pretty sure this is the side effect of adapting the new Android Photo Picker API. Will revert to the old implementation or try to find a way preserving the original filename.

Meanwhile you can try adding pictures to Lespas using another way: select files in filemanager app then share them to Les Pas.

marcus-k commented 1 year ago

Ah I see thanks, that's good to know. Yeah I noticed the photo picker was different whenever I updated. One down side of the new photo picker is that it does not seem to have a scroll bar to quickly drag to the end of my photos. I ended up having to spend a good minute or two scrolling all the way down to the earlier pictures I had.

I didn't know about selecting photos and sharing them to Les Pas before. I just tested that out and it worked as expected with the file names preserved. I will probably end up doing that in the meantime while this problem is being sorted out.

scubajeff commented 1 year ago

Sharing data among apps is one of the beauty of mobile OS. I use sharing a lot to collect pictures/videos into Les Pas from space like twitter and reddit.