mateoradman / bazarr-bulk

Bazarr Bulk Actions CLI - Bulk sync your subtitles with ease
https://crates.io/crates/bazarr-bulk
MIT License
30 stars 1 forks source link

Movies offset #24

Closed JuacOHI closed 1 month ago

JuacOHI commented 1 month ago

Hello. How are you doing ? First of all, thanks for creating this tool of mass sync.

Im having an issue with this tool, i had to restart my computer and i used the -offset to skip the ones that i already synced, but it seems that it didn't work. Is this option working?. Here is a screenshot: https://imgur.com/a/PGWc4fc bazarr-bulk

You can tell that my library its on the big side, almost 6000 and each one has 4 different languages, so if i resync everything its gonna take me a lot of days.

Thanks again for the tool.

mateoradman commented 1 month ago

Hi @JuacOHI, thanks for reporting the problem. I will test the feature again, and report back as soon as possible.

mateoradman commented 1 month ago

I've investigated this problem, and it seems that Bazarr's API only respects the start query parameter (I call it offset) in combination with the length query parameter (I call it limit). When start is set, but length is not, Bazarr sets length to -1 which retreives all results regardless of the start query param. I managed to come up with a workaround for this problem which does the following:

JuacOHI commented 1 month ago

Thank you for the quick reply and solution. I tried with the max integer and didn't work, but it did work when i put the limit number with the total of my library

bazarr-bulk bazarr-bulk2

mateoradman commented 1 month ago

Thank you for the quick reply and solution.

I tried with the max integer and didn't work, but it did work when i put the limit number with the total of my library

bazarr-bulk

bazarr-bulk2

Thank you for the confirmation. An official fix for this problem is going to be included in the next release.