Bug: First 10 results are missing - Githubissues

nullxception / boorusphere

Simple, content-focused booru viewer for Android

BSD 3-Clause "New" or "Revised" License

402 stars 30 forks source link

Bug: First 10 results are missing #92

Closed Tau5 closed 1 year ago

Tau5 commented 1 year ago

Type

Booru parser issues

Bug description

The first 10 results (or what is set on "max content per-load" are missing (at least on rule34.xxx)

Steps to reproduce

Open rule34.xxx on a web browser and make a query which returns more than 10 items
Set in the app "Max content per-load" to 10
Search the same thing on the app
Observe that the first 10 results are missing

Android version

12.1

App version

1.3.1

Screenshots or videos

No response

Solution

No response

Additional context

No response

Acknowledgements

[X] I have searched the existing issues and this is a new and no duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] I filled out all of the requested information in this issue properly.

nullxception commented 1 year ago

Looks like it caused by PageState._fetch() starts in page=1 instead of 0 (after e9610e9b72cae8869a023dbed34161ae140504c2 changes). Nice found.