Bug 38512 - Item table status column display is wrong when record has recalls
Summary: Item table status column display is wrong when record has recalls
Status: Pushed to main
Alias: None
Product: Koha
Classification: Unclassified
Component: Circulation (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal
Assignee: Lucas Gass (lukeg)
QA Contact: Martin Renvoize (ashimema)
URL:
Keywords:
Depends on:
Blocks: 37334 38767
  Show dependency treegraph
 
Reported: 2024-11-21 22:01 UTC by Lucas Gass (lukeg)
Modified: 2024-12-23 09:37 UTC (History)
6 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
25.05.00
Circulation function: recalls


Attachments
Bug 38512: Only show status of recall when it is item_level (2.67 KB, patch)
2024-11-21 22:40 UTC, Lucas Gass (lukeg)
Details | Diff | Splinter Review
Bug 38512: Fix display of recalls in items table (3.57 KB, patch)
2024-12-02 17:42 UTC, Lucas Gass (lukeg)
Details | Diff | Splinter Review
Bug 38512: Fix display of recalls in items table (3.63 KB, patch)
2024-12-03 12:14 UTC, Emmi Takkinen
Details | Diff | Splinter Review
Bug 38512: Fix display of recalls in items table (3.69 KB, patch)
2024-12-16 19:08 UTC, Martin Renvoize (ashimema)
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Lucas Gass (lukeg) 2024-11-21 22:01:36 UTC
To recreate:
1. Turn on  UseRecalls 
2. Set circ rules to allow for recalls ( Recalls allowed(total) and Recalls allowed(count))
3. Check something out to a patron, choose a record with multiple items on it 
4. With another patron recall the item from the OPAC
5. Choose "Recall next available item"
6. Now go back to the item table and see that all items on the bib say something like: "recalled by koha (42) on 11/21/2024"

That seems incorrect, the items should still show as 'Available'.
Comment 1 Lucas Gass (lukeg) 2024-11-21 22:40:47 UTC
Created attachment 174904 [details] [review]
Bug 38512: Only show status of recall when it is item_level

To test:
1. Turn on  UseRecalls
2. Set circ rules to allow for recalls ( Recalls allowed(total) and Recalls allowed(count))
3. Check something out to a patron, choose a record with multiple items on it
4. With another patron recall the item from the OPAC
5. Choose "Recall next available item"
6. Now go back to the item table and see that all items on the bib say something like: "recalled by koha (42) on 11/21/2024".
7. APPLY PATCH
8. Do steps 3 - 6 again and now the items should correcylt show as available.
9. Place a recall again but choose 'Recall a specific item'.
10. Look at the table again, it should accuretly show the recall in the status column. "recalled by koha (42) on 11/21/2024"
11. Turn off UseRecalls, make sure the table still loads properly.
Comment 2 Lucas Gass (lukeg) 2024-11-22 16:52:09 UTC
With this patch, If I check out all items on the record and then place a recall on the next available item there is no indication anywhere on the staff detail page that there is a 'next available' recall.

I think there should be some indication of that on the record. But is the status column ( item row ) the place to do it?
Comment 3 Lucas Gass (lukeg) 2024-11-22 16:56:14 UTC
(In reply to Lucas Gass (lukeg) from comment #2)
> With this patch, If I check out all items on the record and then place a
> recall on the next available item there is no indication anywhere on the
> staff detail page that there is a 'next available' recall.
> 
> I think there should be some indication of that on the record. But is the
> status column ( item row ) the place to do it?

If I compare to holds, when all items are checked out and I make a bib level hold, there is no indication in the status column ( item row ) that there is a hold. 

So, I think this patch is correct as is.
Comment 4 Lucas Gass (lukeg) 2024-12-02 17:42:58 UTC
Created attachment 175057 [details] [review]
Bug 38512: Fix display of recalls in items table

1. Turn on  UseRecalls
2. Set circ rules to allow for recalls ( Recalls allowed(total) and Recalls allowed(count))
3. Check something out to a patron, choose a record with multiple items on it
4. With another patron recall the item from the OPAC
5. Choose "Recall next available item"
6. Now go back to the item table and see that all items on the bib say something like: "Recalled by koha (42) on 11/21/2024"
7. APPLY PATCH
8. Do steps 3 - 6 again and now the items should correctly show as available.
9. Place a recall again but choose 'Recall a specific item'.
10. Look at the table again, it should accuretly show the recall in the status column. "recalled by koha (42) on 11/21/2024"
11. Turn off UseRecalls, make sure the table still loads properly.
Comment 5 Lucas Gass (lukeg) 2024-12-02 17:45:52 UTC
Note: There as also a problem with the display of the holding library name, I fixed that here.
Comment 6 Emmi Takkinen 2024-12-03 12:14:35 UTC
Created attachment 175068 [details] [review]
Bug 38512: Fix display of recalls in items table

1. Turn on  UseRecalls
2. Set circ rules to allow for recalls ( Recalls allowed(total) and Recalls allowed(count))
3. Check something out to a patron, choose a record with multiple items on it
4. With another patron recall the item from the OPAC
5. Choose "Recall next available item"
6. Now go back to the item table and see that all items on the bib say something like: "Recalled by koha (42) on 11/21/2024"
7. APPLY PATCH
8. Do steps 3 - 6 again and now the items should correctly show as available.
9. Place a recall again but choose 'Recall a specific item'.
10. Look at the table again, it should accuretly show the recall in the status column. "recalled by koha (42) on 11/21/2024"
11. Turn off UseRecalls, make sure the table still loads properly.

Signed-off-by: Emmi Takkinen <emmi.takkinen@koha-suomi.fi>
Comment 7 Martin Renvoize (ashimema) 2024-12-16 19:08:33 UTC
Created attachment 175571 [details] [review]
Bug 38512: Fix display of recalls in items table

1. Turn on  UseRecalls
2. Set circ rules to allow for recalls ( Recalls allowed(total) and Recalls allowed(count))
3. Check something out to a patron, choose a record with multiple items on it
4. With another patron recall the item from the OPAC
5. Choose "Recall next available item"
6. Now go back to the item table and see that all items on the bib say something like: "Recalled by koha (42) on 11/21/2024"
7. APPLY PATCH
8. Do steps 3 - 6 again and now the items should correctly show as available.
9. Place a recall again but choose 'Recall a specific item'.
10. Look at the table again, it should accuretly show the recall in the status column. "recalled by koha (42) on 11/21/2024"
11. Turn off UseRecalls, make sure the table still loads properly.

Signed-off-by: Emmi Takkinen <emmi.takkinen@koha-suomi.fi>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 8 Katrin Fischer 2024-12-23 09:37:36 UTC
Pushed for 25.05!

Well done everyone, thank you!