Bug 35030

Summary: Extend TrackLastPatronActivity with placing article request
Product: Koha Reporter: Marcel de Rooy <m.de.rooy>
Component: PatronsAssignee: Martin Renvoize <martin.renvoize>
Status: RESOLVED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: enhancement    
Priority: P5 - low CC: caroline.cyr-la-rose, fridolin.somers, gmcharlt, kyle.m.hall, martin.renvoize, matt.blenkinsop, tomascohen
Version: MainKeywords: rel_23_11_candidate
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Caroline Cyr La Rose Documentation submission: https://gitlab.com/koha-community/koha-manual/-/merge_requests/745
Text to go in the release notes:
Version(s) released in:
23.11.00
Bug Depends on: 15504, 35027    
Bug Blocks: 35001    
Attachments: Bug 35030: Add 'article' to patron activity triggers
Bug 35030: Add 'article' to patron activity triggers
Bug 35030: Add 'article' to patron activity triggers

Description Marcel de Rooy 2023-10-11 14:28:19 UTC
See bug 15504
Comment 1 Marcel de Rooy 2023-10-11 14:43:51 UTC
Note that bug 35027 adds holds.
Comment 2 Martin Renvoize 2023-10-12 11:58:30 UTC Comment hidden (obsolete)
Comment 3 Marcel de Rooy 2023-10-12 12:03:45 UTC
Could you please extend the unit test ?
Comment 4 Martin Renvoize 2023-10-12 13:05:12 UTC
Created attachment 156961 [details] [review]
Bug 35030: Add 'article' to patron activity triggers

This patch adds 'article' to the list of triggers available for
tracking patron activity.

Test plan
1) Select 'Placing an article request
   TrackPatronLastActivityTriggers system preference
2) As a staff member, place a hold on any item for a test user
3) Confirm that the borrowers.lastseen field is updated for that
   test borrower
Comment 5 Martin Renvoize 2023-10-12 13:07:50 UTC
Tests added :)
Comment 6 Marcel de Rooy 2023-10-12 13:50:47 UTC
Created attachment 156964 [details] [review]
Bug 35030: Add 'article' to patron activity triggers

This patch adds 'article' to the list of triggers available for
tracking patron activity.

Test plan
1) Select 'Placing an article request
   TrackPatronLastActivityTriggers system preference
2) As a staff member, place a hold on any item for a test user
3) Confirm that the borrowers.lastseen field is updated for that
   test borrower

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 7 Marcel de Rooy 2023-10-12 13:54:05 UTC
Moving to PQA, combining with SO
Comment 8 Tomás Cohen Arazi 2023-10-23 14:36:42 UTC
Marking as BLOCKED until bug 15504 is PQA. Please adjust dependencies if this is wrong.
Comment 9 Tomás Cohen Arazi 2023-10-24 13:06:56 UTC
Pushed to master for 23.11.

Nice work everyone, thanks!
Comment 10 Fridolin Somers 2023-10-26 06:50:49 UTC
Depends on Bug 15504 not in 23.05.x