Summary: | Membership Expiry can not be limited by Patron Category | ||
---|---|---|---|
Product: | Koha | Reporter: | Kelly McElligott <kelly> |
Component: | Command-line Utilities | Assignee: | Tomás Cohen Arazi (tcohen) <tomascohen> |
Status: | RESOLVED DUPLICATE | QA Contact: | Testopia <testopia> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | amit.gupta, amitddng135, fridolin.somers, jheltibridle, nick, robin, sally.healey, tomascohen, wizzyrea |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
See Also: | https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28456 | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | --- | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: | ||
Attachments: | Bug 26062: Add --category option to membership_expiry.pl |
Description
Kelly McElligott
2020-07-24 18:17:10 UTC
Created attachment 107367 [details] [review] Bug 26062: Add --category option to membership_expiry.pl This patch adds a new option switch to the script, allowing to restrict the query for upcoming expirying patrons to a specific category. To test: 1. Apply this patch 2. Have upcoming expirying patrons with a certain category, and some others matching the expiry params (before/after), but with a different category 3. Run the script passing --category CAT (CAT being the chosen category) => SUCCESS: Only those on the specified category are picked 4. Sign off :-D Sponsored-by: ByWater Solutions The delay for sending the notice is already a system preference - can we have the limit as a system preference too please? This would make it much easier for installations with multiple instances. I agree with Katrin here - from a sysadmin standpoint it's much better for libraries to be able to configure these from the interface (it's easier for package maintenance/upgrades too, since users won't be inclined to modify the default cron files and then have to resolve those differences at upgrade time in apt) I think this is obsolete since Bug 28456 added --where arg. Our library has patrons who we would like to exclude from notices to renew their accounts. It would make more sense to have this be something we could control from the staff interface, since we do not have control of the crons as a supported library. On command line you can already achieve the limit on patron category using the --where parameter. I believe we should really move this into the GUI. |