Photok icon indicating copy to clipboard operation
Photok copied to clipboard

Export not working properly

Open jerryn70 opened this issue 2 years ago • 12 comments

I have 2199 files in photok gallery both images and videos . When I select all files and try to export it to the gallery only 799 images is exported

jerryn70 avatar Sep 11 '21 15:09 jerryn70

@jerryn70 if you select all, are really all of them selected?

If not, this could be a bug related to paging.

leonlatsch avatar Sep 11 '21 17:09 leonlatsch

@leonlatsch all files are selected. I scrolled through all the files and confirmed.

jerryn70 avatar Sep 12 '21 02:09 jerryn70

@jerryn70 is the exporting dialog displaying the right amount of photos?

Eg: Exporting: 1/2199

Or is it displaying: 1/799

leonlatsch avatar Sep 12 '21 10:09 leonlatsch

@jerryn70 the configured max for photos loaded at once is 800.

This would match pretty well 🤔

leonlatsch avatar Sep 12 '21 10:09 leonlatsch

@leonlatsch 1/799

jerryn70 avatar Sep 12 '21 10:09 jerryn70

@jerryn70 Well, ok.

I will Label this a bug and fix it once I find the time.

As a workaround:

  • Select all
  • export
  • Scroll down
  • Export again

This should work, I guess.

leonlatsch avatar Sep 12 '21 10:09 leonlatsch

@leonlatsch i tried this but it creates too many duplicate file exports.

jerryn70 avatar Sep 12 '21 10:09 jerryn70

@leonlatsch Another export error in the latest fdroid version. It shows finished but nothing is exported to the gallery.

jerryn70 avatar Sep 23 '21 04:09 jerryn70

The Problem

The multi selection is index based.

When selecting all, the item count is used to add all items. The item count also includes all placeholders, added by paging.

So the placeholders are selected, but no item lays behind them. Therefore they cannot be used.

leonlatsch avatar Dec 07 '21 21:12 leonlatsch

We may think about dropping paging, since the data that is currently loaded paged, is not big. Loading the thumbnails happens in the ViewHolder's onBind anyway. What do you think? @comparesheriff

leonlatsch avatar Dec 07 '21 21:12 leonlatsch

This issue is still present for me

SeleDreams avatar Sep 23 '22 07:09 SeleDreams

I know, sorry about that.

This is a complex issue wich involves rewriting a bug chunk of logic and Photok is currently just maintained.

leonlatsch avatar Sep 29 '22 15:09 leonlatsch