- Koha-bugs - lists.koha-community.org

[Bug 14604] Offline circulation processor doesn't handle large sets of actions well
by bugzilla-daemon＠bugs.koha-community.org 17 Sep '26

17 Sep '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14604 Kyle M Hall (khall) <kyle(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #206192|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14604] Offline circulation processor doesn't handle large sets of actions well
by bugzilla-daemon＠bugs.koha-community.org 17 Sep '26

17 Sep '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14604 Kyle M Hall (khall) <kyle(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #206191|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14604] Offline circulation processor doesn't handle large sets of actions well
by bugzilla-daemon＠bugs.koha-community.org 17 Sep '26

17 Sep '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14604 Kyle M Hall (khall) <kyle(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #206190|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14604] Offline circulation processor doesn't handle large sets of actions well
by bugzilla-daemon＠bugs.koha-community.org 17 Sep '26

17 Sep '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14604 --- Comment #5 from Kyle M Hall (khall) <kyle(a)bywatersolutions.com> --- Created attachment 206192 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=206192&action=e… Bug 14604: Process offline circulation actions in the background This patch makes the Process button on the pending offline circulation actions page enqueue a background job rather than send one synchronous ajax request per action. Test Plan: 1) Apply this patch 2) Restart all the things! 3) Upload a Koha offline circulation ( .koc ) file with a few hundred actions in it A file that works with default koha-testing-docker data is attached to the ticket. 4) Go to Circulation > Pending offline circulation actions 5) Click "Check all", then click "Process" 6) Note the page comes back right away instead of locking up the browser! 7) Note the actions you selected are now listed as "Queued" and can no longer be checked! 8) Follow the "View detail of the enqueued job" link 9) Note the job reports how many actions succeeded, and lists the date, action, barcode, card number and result of every one of them! 10) Go back to Circulation > Pending offline circulation actions 11) Note the actions that were processed are gone from the list 12) Check a couple of the remaining actions and click "Delete" 13) Note they are gone and the number deleted is reported! Sponsored-by: ByWater Solutions Signed-off-by: Kyle M Hall <kyle(a)bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14604] Offline circulation processor doesn't handle large sets of actions well
by bugzilla-daemon＠bugs.koha-community.org 17 Sep '26

17 Sep '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14604 --- Comment #4 from Kyle M Hall (khall) <kyle(a)bywatersolutions.com> --- Created attachment 206191 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=206191&action=e… Bug 14604: Add unit tests Sponsored-by: ByWater Solutions Signed-off-by: Kyle M Hall <kyle(a)bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14604] Offline circulation processor doesn't handle large sets of actions well
by bugzilla-daemon＠bugs.koha-community.org 17 Sep '26

17 Sep '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14604 --- Comment #3 from Kyle M Hall (khall) <kyle(a)bywatersolutions.com> --- Created attachment 206190 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=206190&action=e… Bug 14604: Add background job to process offline circulation actions Sponsored-by: ByWater Solutions Signed-off-by: Kyle M Hall <kyle(a)bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14604] Offline circulation processor doesn't handle large sets of actions well
by bugzilla-daemon＠bugs.koha-community.org 17 Sep '26

17 Sep '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14604 Kyle M Hall (khall) <kyle(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14604] Offline circulation processor doesn't handle large sets of actions well
by bugzilla-daemon＠bugs.koha-community.org 17 Sep '26

17 Sep '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14604 --- Comment #2 from Kyle M Hall (khall) <kyle(a)bywatersolutions.com> --- Created attachment 206188 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=206188&action=e… Example .koc to use with koha-testing-docker -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 43549] New: Convert misc/cronjobs/cleanup_database.pl to the Getopt::Long::Descriptive convention (see bug 43546)
by bugzilla-daemon＠bugs.koha-community.org 17 Sep '26

17 Sep '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43549 Bug ID: 43549 Summary: Convert misc/cronjobs/cleanup_database.pl to the Getopt::Long::Descriptive convention (see bug 43546) Initiative type: --- Sponsorship --- status: Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Command-line Utilities Assignee: koha-bugs(a)lists.koha-community.org Reporter: martin.renvoize(a)openfifth.co.uk QA Contact: testopia(a)bugs.koha-community.org CC: jake.deery(a)openfifth.co.uk, robin(a)catalyst.net.nz Depends on: 43546 Target Milestone: --- This is one of a set of bugs filed alongside bug 43546 (Standardize command-line scripts on Getopt::Long::Descriptive) to convert individual, commonly-used cron scripts to whatever declarative option-parsing convention that bug settles on. Depends on 43546. Current state: cleanup_database.pl uses plain Getopt::Long (GetOptions call around line 198) with roughly 50 options. No single option is required; instead the script requires at least one of a large set of purge-type flags to be set, enforced by a long "unless ($sessions || $zebraqueue_days || ... )" condition after parsing (around line 280). What this bug covers: convert the GetOptions call to the new convention for consistent declaration and auto-generated --help output. None of these options map cleanly onto a simple per-option required constraint, since the actual requirement is "at least one of roughly 50 flags", not any single mandatory option -- that check should remain a manual post-parse validation, same as today. Given the option count, this is a larger and more involved conversion than the others in this set. Suggest treating it as lower priority -- do it once the convention is proven out on a smaller script first. Test plan: - --help output covers all documented options and matches current behavior - Running with no purge flags still fails with the same "nothing to do" message - Running with any single purge flag still behaves exactly as before Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43546 [Bug 43546] Standardize command-line scripts on Getopt::Long::Descriptive for safer, more consistent option handling -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 6

[Bug 43551] New: Convert misc/cronjobs/process_message_queue.pl to the Getopt::Long::Descriptive convention (see bug 43546)
by bugzilla-daemon＠bugs.koha-community.org 17 Sep '26

17 Sep '26

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43551 Bug ID: 43551 Summary: Convert misc/cronjobs/process_message_queue.pl to the Getopt::Long::Descriptive convention (see bug 43546) Initiative type: --- Sponsorship --- status: Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Command-line Utilities Assignee: koha-bugs(a)lists.koha-community.org Reporter: martin.renvoize(a)openfifth.co.uk QA Contact: testopia(a)bugs.koha-community.org CC: jake.deery(a)openfifth.co.uk, robin(a)catalyst.net.nz Depends on: 43546 Target Milestone: --- This is one of a set of bugs filed alongside bug 43546 (Standardize command-line scripts on Getopt::Long::Descriptive) to convert individual, commonly-used cron scripts to whatever declarative option-parsing convention that bug settles on. Depends on 43546. Current state: process_message_queue.pl uses plain Getopt::Long (GetOptions call around line 40). No option is required -- every option (--username, --password, --limit, --method, --type, --code, --where, --exit-on-plugin-failure) is genuinely optional, and the script runs correctly with none of them supplied. What this bug covers: convert the GetOptions call to the new convention purely for consistency and self-documenting --help output, matching bug 43546's convention. No behavior change is expected, since nothing here is actually a missing-required-option bug to fix. Test plan: - --help output covers all documented options and matches current behavior - Running with no options still behaves exactly as before Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43546 [Bug 43546] Standardize command-line scripts on Getopt::Long::Descriptive for safer, more consistent option handling -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 6