Bug 35251

Summary: Issue table does not recalculate number of checkouts after a check in
Product: Koha Reporter: Lucas Gass (lukeg) <lucas>
Component: CirculationAssignee: Lucas Gass (lukeg) <lucas>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: normal    
Priority: P5 - low CC: david, fridolin.somers, gmcharlt, kyle.m.hall, m.de.rooy, oleonard, victor
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
This fixes the number of checkouts shown on a patron's check-in and details page. Previously, if items were checked-in from either of these tabs, the number of checkouts was not updated.
Version(s) released in:
23.11.00
Circulation function:
Bug Depends on: 35068    
Bug Blocks: 35313    
Attachments: Bug 35251: Update checkout count when refreshing issue table
Bug 35251: Update checkout count when refreshing issue table
Bug 35251: Update checkout count when refreshing issue table
Bug 35251: Update checkout count when refreshing issue table

Description Lucas Gass (lukeg) 2023-11-03 15:32:55 UTC
To recreate:

1. Have some checkouts 
2. Go to the issue-table and select some of those checkouts and check them in. 
3. Look at the tab, Checkouts(X)
4. This is not updated and now shows an incorrect number of checkouts.
Comment 1 Lucas Gass (lukeg) 2023-11-03 16:52:57 UTC
Created attachment 158355 [details] [review]
Bug 35251: Update checkout count when refreshing issue table

1. Have some checkouts
2. Go to the issue-table and select some of those checkouts and check them in.
3. Look at the tab, Checkouts(X)
4. This is not updated and now shows an incorrect number of checkouts.
4. APPLY PATCH, clear browser cache/
5. Try again, now when the table reloads the 'Checkouts(X)' tab should be correctly updated.
Comment 2 David Nind 2023-11-03 20:19:29 UTC
Created attachment 158367 [details] [review]
Bug 35251: Update checkout count when refreshing issue table

1. Have some checkouts
2. Go to the issue-table and select some of those checkouts and check them in.
3. Look at the tab, Checkouts(X)
4. This is not updated and now shows an incorrect number of checkouts.
4. APPLY PATCH, clear browser cache/
5. Try again, now when the table reloads the 'Checkouts(X)' tab should be correctly updated.

Signed-off-by: David Nind <david@davidnind.com>
Comment 3 Victor Grousset/tuxayo 2023-11-05 18:43:12 UTC
Created attachment 158474 [details] [review]
Bug 35251: Update checkout count when refreshing issue table

1. Have some checkouts
2. Go to the issue-table and select some of those checkouts and check them in.
3. Look at the tab, Checkouts(X)
4. This is not updated and now shows an incorrect number of checkouts.
4. APPLY PATCH, clear browser cache/
5. Try again, now when the table reloads the 'Checkouts(X)' tab should be correctly updated.

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
Comment 4 Victor Grousset/tuxayo 2023-11-05 19:22:36 UTC
Works, makes sense, QA script happy, code looks good :) Waiting for dependency to pass QA
Comment 5 Marcel de Rooy 2023-11-10 08:59:15 UTC
Looking here
Comment 6 Marcel de Rooy 2023-11-10 09:02:18 UTC
Created attachment 158793 [details] [review]
Bug 35251: Update checkout count when refreshing issue table

1. Have some checkouts
2. Go to the issue-table and select some of those checkouts and check them in.
3. Look at the tab, Checkouts(X)
4. This is not updated and now shows an incorrect number of checkouts.
4. APPLY PATCH, clear browser cache/
5. Try again, now when the table reloads the 'Checkouts(X)' tab should be correctly updated.

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 7 Tomás Cohen Arazi (tcohen) 2023-11-10 14:02:37 UTC
Pushed to master for 23.11.

Nice work everyone, thanks!
Comment 8 Fridolin Somers 2023-11-11 07:50:57 UTC
Depends on Bug 35068 not in 23.05.x