Summary: | TrackLastPatronActivity for SIP should track both patron status and patron information requests | ||
---|---|---|---|
Product: | Koha | Reporter: | Nick Clemens (kidclamp) <nick> |
Component: | SIP2 | Assignee: | Nick Clemens (kidclamp) <nick> |
Status: | Pushed to oldstable --- | QA Contact: | Martin Renvoize (ashimema) <martin.renvoize> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | fridolin.somers, lucas, magnus |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
24.05.00,23.11.06,23.05.12
|
|
Circulation function: | |||
Bug Depends on: | |||
Bug Blocks: | 38284 | ||
Attachments: |
Bug 36605: Add update_lastseen to handle_patron_status for SIP
Bug 36605: Add update_lastseen to handle_patron_status for SIP Bug 36605: [23.05.x] Add update_lastseen to handle_patron_status for SIP |
Description
Nick Clemens (kidclamp)
2024-04-16 03:49:33 UTC
Created attachment 164897 [details] [review] Bug 36605: Add update_lastseen to handle_patron_status for SIP This patch extends the TrackLastPatronActivity trigger to cover SIP status messages as well. Other SIP messages like Checkin/Checkout should be covered by those values in system preference, so should not need adjustment. To test: prove -v t/db_dependent/SIP/Message.t Created attachment 164898 [details] [review] Bug 36605: Add update_lastseen to handle_patron_status for SIP This patch extends the TrackLastPatronActivity trigger to cover SIP status messages as well. Other SIP messages like Checkin/Checkout should be covered by those values in system preference, so should not need adjustment. To test: prove -v t/db_dependent/SIP/Message.t Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Obvious use case here, works as described and is well tested. Going straight for QA Pushed for 24.05! Well done everyone, thank you! Created attachment 165921 [details] [review] Bug 36605: [23.05.x] Add update_lastseen to handle_patron_status for SIP TrackLastPatronActivityTriggers is only 23.11+ - 23.05 version supplied More a bug than enhancement ;) Pushed to 23.11.x for 23.11.06 Backported to 23.05.x for upcoming 23.05.12 |