The batch size should be 50 when cron indexing is disabled

drunken monkey → made their first commit to this issue’s fork.

Thanks for suggesting this change, makes sense.
However, wouldn’t it make more sense to default to the default cron limit? Though I guess that might also be 0, so there needs to be a last fallback of 50 regardless. Anyways, changed the code to try the default cron limit first.
Please test/review and tell me what you think!

(Note: Seems that, annoyingly, both user experience and UX are commonly used as tags, so changing to use both here. (The latter is significantly more common, though.))

Sorry, I know there was a long delay since you created this, but would be great if you could still test/review before I merge it.

Sorry for the delayed response on this one, the fix works just fine. LGTM

No worries at all, thanks for reporting back and good to hear you are fine with my changes.
Merged.
Thanks again!

Thank you Thomas for fixing this issue.

Automatically closed - issue fixed for 2 weeks with no activity.