Incorrect sort when you use pagination

[ScharfViktor]

Sort when file more than 100:

1. Open subfolder "Check Sort/CheckPagination"
2. Choose the sort by name
3. Go to page 2

Expected behaviour:
files on page 2: 101, 102, 103, 104

Actual behaviour:
files on page 2: the last 4 files from the response (random)

Server configuration

branch: release-4.0.0
backend: ocis 1.9.0 with docker reachable at https://host.docker.internal:9200/
browser: chrome Version 92.0.4515.107

[kulmann]

Urgh. we're sorting inside OcTableFiles (ownCloud Design System). Since we're handing only the paginated data to the OcTableFiles component we're only sorting the data of the current page. :-(

[kulmann]

We should remove the sort logic from the design system and only emit events for sorting. Sorting needs to happen within web (and later on server side).

[individual-it]

probably same problem:

1. upload many files, so that pagination is needed
2. navigate to the first page
3. create a folder
   => folder is selected, but not viewed on the first page. After a page reload the folder is listed at the top of the list like all folders

[kulmann]

Wait with fixing until #4893 is merged

[dschmidt]

Wait with fixing until #4893 is merged

You probably meant:
owncloud/owncloud-design-system#1817
#6106

We should remove the sort logic from the design system and only emit events for sorting. Sorting needs to happen within web (and later on server side).

What would sorting in web look like?
I don't see any reasonable client side solution for sorting (except for retrieving the whole folder in background ... uuuuuurgh, not reasonable)

edit:
oh lol, that's exactly what we do - there is no server side pagination for now, as far as I can tell. That obviously makes it way easier to implement sorting in web for now, as it's not removing any functionality. I'll give it a shot.

[dschmidt]

Nice! Thanks a lot for bringing this over the finish line 💖

[kulmann]

Thanks for doing the heavy lifting @dschmidt 😍