[Bug 43638] New: Add core support for scheduled/automated patron image import
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43638 Bug ID: 43638 Summary: Add core support for scheduled/automated patron image import Initiative type: --- Sponsorship --- status: Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Patrons Assignee: koha-bugs@lists.koha-community.org Reporter: martin.renvoize@openfifth.co.uk QA Contact: testopia@bugs.koha-community.org CC: gmcharlt@gmail.com, kyle@bywatersolutions.com Target Milestone: --- Depends on Bug 43637 (refactor patron image upload into a reusable engine class and background job), which this builds on. This bug is the scheduling half of that work: automatically importing patron images from a remote/local file source on a schedule, the same way Bug 43616 added scheduled/automated patron record import on top of the existing, unmodified Koha::Patrons::Import engine. Planned approach: - New patron_image_import_accounts and patron_image_import_log tables, plus Koha::PatronImageImportAccount(s)/Koha::PatronImportLog(s) object classes, mirroring Bug 43616's own account/log model (name, file transport, matchpoint, active flag, per-(account, filename) idempotency). - A new Tools page ("Schedule patron image imports") for configuring accounts - a smaller option surface than the patron-import case, since image import has no equivalent of default values or preserved fields. - A new misc/cronjobs/patron_image_imports.pl cronjob polling each active account's configured Koha::File::Transport, calling Koha::Patron::Image::Import->import_batch (from Bug 43637) for zip archives it finds, and also handling loose image files sitting directly on the transport (matched by filename stem against the account's chosen matchpoint) - not just zips, since an unattended feed may not always package images into an archive. No code has been written yet for this bug - filing it now to track the plan and its dependency on Bug 43637 before design/implementation begins. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43638 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |43637 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43637 [Bug 43637] Refactor patron image upload into a reusable engine class and background job -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43638 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43638 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |martin.renvoize@openfifth.c |ity.org |o.uk -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43638 --- Comment #1 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 206826 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=206826&action=edit Bug 43638: Automated Schema Update Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43638 --- Comment #2 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 206827 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=206827&action=edit Bug 43638: Add is_boolean annotation to PatronImageImportAccount Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43638 --- Comment #3 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 206828 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=206828&action=edit Bug 43638: Add Koha::PatronImageImportAccount(s) and Koha::PatronImageImportLog(s) Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43638 --- Comment #4 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 206829 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=206829&action=edit Bug 43638: Add MAX_ZIP_MEMBERS cap to Koha::Patron::Image::Import This commit adds a safety cap to prevent extraction and processing of zip archives with more than 1000 non-directory members, bounding worst-case extraction work from oversized or malicious zip files. The cap is enforced in import_batch before any extraction occurs, and returns a TOO_MANY_MEMBERS error when exceeded. Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43638 --- Comment #5 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 206830 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=206830&action=edit Bug 43638: Add patron_image_imports.pl cron job Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43638 --- Comment #6 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 206831 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=206831&action=edit Bug 43638: Add 'Schedule patron image imports' page under Tools Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43638 --- Comment #7 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 206832 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=206832&action=edit Bug 43638: Register patron_image_imports.pl in the default crontab templates Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43638 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43638 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |37248 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37248 [Bug 37248] [Omnibus] Power to the user -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43638 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |37248 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37248 [Bug 37248] [Omnibus] Power to the user -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43638 Andrew Fuerste-Henry <andrew@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA CC| |andrew@bywatersolutions.com --- Comment #8 from Andrew Fuerste-Henry <andrew@bywatersolutions.com> --- Can you please add a test plan here? -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org