Bug 29021

Summary: Automatic renewal due to RenewAccruingItemWhenPaid should not be considered Seen
Product: Koha Reporter: Andrew Fuerste-Henry <andrewfh>
Component: CirculationAssignee: Martin Renvoize (ashimema) <martin.renvoize>
Status: CLOSED FIXED QA Contact: Kyle M Hall (khall) <kyle>
Severity: normal    
Priority: P5 - low CC: arthur.suzuki, gmcharlt, kyle.m.hall, kyle, lucas, martin.renvoize, matt.blenkinsop
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:
23.05.00,22.11.03
Circulation function:
Bug Depends on: 24083, 30275    
Bug Blocks:    
Attachments: Bug 29021: RenewAccruingItemWhenPaid renewals are not seen
Bug 29021: RenewAccruingItemWhenPaid renewals are not seen
Bug 29021: RenewAccruingItemWhenPaid renewals are not seen
Bug 29021: (QA follow-up) Fix for new parameter, add unit test
Bug 29021: (QA follow-up) Remove useless warnings

Description Andrew Fuerste-Henry 2021-09-14 17:59:46 UTC
If one is using Seen/Unseen renewals and has the RenewAccruingItemWhenPaid system preference enabled, upon accepting payment for an accruing overdue fine the item will be automatically renewed and that renewal will be considered "seen," as if the patron had actually brought the item to the library. That seems like a faulty assumption. An automatically-triggered renewal should be unseen.
Comment 1 Martin Renvoize (ashimema) 2022-05-11 14:53:26 UTC
Created attachment 134897 [details] [review]
Bug 29021: RenewAccruingItemWhenPaid renewals are not seen

This patch updates the call to AddRewewal from Koha::Accounts to makr
the renewal as unseen.
Comment 2 PTFS Europe Sandboxes 2022-12-08 15:52:49 UTC
Created attachment 144503 [details] [review]
Bug 29021: RenewAccruingItemWhenPaid renewals are not seen

This patch updates the call to AddRewewal from Koha::Accounts to makr
the renewal as unseen.

Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
Comment 3 Kyle M Hall (khall) 2023-01-27 16:59:08 UTC
Created attachment 145761 [details] [review]
Bug 29021: RenewAccruingItemWhenPaid renewals are not seen

This patch updates the call to AddRewewal from Koha::Accounts to makr
the renewal as unseen.

Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 4 Kyle M Hall (khall) 2023-01-27 16:59:55 UTC
Created attachment 145762 [details] [review]
Bug 29021: (QA follow-up) Fix for new parameter, add unit test

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 5 Tomás Cohen Arazi (tcohen) 2023-01-30 15:05:07 UTC
Created attachment 145799 [details] [review]
Bug 29021: (QA follow-up) Remove useless warnings

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Comment 6 Tomás Cohen Arazi (tcohen) 2023-01-30 15:15:33 UTC
Pushed to master for 23.05.

Nice work everyone, thanks!
Comment 7 Jacob O'Mara 2023-01-31 17:19:21 UTC
Nice work, thanks everyone!

Pushed to 22.11.x for the next release.
Comment 8 Lucas Gass (lukeg) 2023-02-01 19:10:57 UTC
Backported to 22.05.x for upcoming 22.05.10
Comment 9 Lucas Gass (lukeg) 2023-02-02 16:53:27 UTC
I should not have backported this, tests are failing because there is no 'Koha::Checkouts::Renewals' in 22.05. That looks like it was added by Bug 30275. I will revert this patchset from 22.05.x.
Comment 10 Lucas Gass (lukeg) 2023-02-02 16:57:07 UTC
Patchset reverted from 22.05.x
Comment 11 Arthur Suzuki 2023-02-17 09:45:26 UTC
Same as Lukas, can't backport