pulsejet / memories

Fast, modern and advanced photo management suite. Runs as a Nextcloud app.
https://memories.gallery
GNU Affero General Public License v3.0
2.82k stars 76 forks source link

android: show local files on older devices #1122

Closed yamanq closed 1 month ago

yamanq commented 1 month ago

I found that the previous PR was only one step to fixing the issue. Just like before, android:requestLegacyExternalStorage is needed for permission purposes, which fixes the EXIF data and full image on zoom. However, the thumbnail generation was still not working as the URI returned by ContentUris.withAppendedId was incorrect, so I used a different approach to get the correct URI.

I could not easily test on newer devices but on my Android 10, this fixes #1000.

pulsejet commented 1 month ago

Tested on 14, works well. Thanks!