| Summary: | Item-level holds can show inaccurate transit status on the patron details page | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Emily Lamancusa (emlam) <emily.lamancusa> |
| Component: | Hold requests | Assignee: | Emily Lamancusa (emlam) <emily.lamancusa> |
| Status: | CLOSED FIXED | QA Contact: | Katrin Fischer <katrin.fischer> |
| Severity: | normal | ||
| Priority: | P5 - low | CC: | fridolin.somers, gmcharlt, katariina.hanhisalo |
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| Change sponsored?: | --- | Patch complexity: | Trivial patch |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: |
23.11.00,23.05.05,22.11.12
|
|
| Circulation function: | |||
| Attachments: |
Bug 34901: Display transit status on patron holds tab only if hold found value is "T"
Bug 34901: Display transit status on patron holds tab only if hold found value is "T" Bug 34901: Display transit status on patron holds tab only if hold found value is "T" Bug 34901: Display transit status on patron holds tab only if hold found value is "T" |
||
|
Description
Emily Lamancusa (emlam)
2023-09-23 21:17:27 UTC
Created attachment 156302 [details] [review] Bug 34901: Display transit status on patron holds tab only if hold found value is "T" Check the transit status of the *hold* in addition to the transit status of the *item*, to avoid displaying a misleading transit status on item-level holds when the item is actually in transit for a different hold To test: 1. Create a record-level hold for Patron A for pickup at a library other than the logged-in library 2. Check in an item to fill that hold 3. Put an item-level hold on that same item for Patron B at a different library other than the logged-in library 4. Open Patron A's and Patron B's account details pages in separate tabs --> Note that the Holds tab on Patron A's account detail page correctly shows that their hold is in-transit --> Note that the Holds tab on Patron B's account detail page incorrectly shows that their hold on the same item is also in-transit 4. Apply patch 5. Clear browser cache 6. Refresh both patrons' account details pages --> Confirm that the holds tab on Patron A's account still correctly says their hold is in-transit --> Confirm that the holds tab on Patron B's account now correctly shows a blank status for their hold 7. Cancel Patron A's hold 8. Check in the item again to put it in transit for Patron B's hold 9. Reload Patron B's account page --> Confirm that the holds tab on Patron B's account now correctly says their hold is in-transit Created attachment 157065 [details] [review] Bug 34901: Display transit status on patron holds tab only if hold found value is "T" Check the transit status of the *hold* in addition to the transit status of the *item*, to avoid displaying a misleading transit status on item-level holds when the item is actually in transit for a different hold To test: 1. Create a record-level hold for Patron A for pickup at a library other than the logged-in library 2. Check in an item to fill that hold 3. Put an item-level hold on that same item for Patron B at a different library other than the logged-in library 4. Open Patron A's and Patron B's account details pages in separate tabs --> Note that the Holds tab on Patron A's account detail page correctly shows that their hold is in-transit --> Note that the Holds tab on Patron B's account detail page incorrectly shows that their hold on the same item is also in-transit 4. Apply patch 5. Clear browser cache 6. Refresh both patrons' account details pages --> Confirm that the holds tab on Patron A's account still correctly says their hold is in-transit --> Confirm that the holds tab on Patron B's account now correctly shows a blank status for their hold 7. Cancel Patron A's hold 8. Check in the item again to put it in transit for Patron B's hold 9. Reload Patron B's account page --> Confirm that the holds tab on Patron B's account now correctly says their hold is in-transit Signed-off-by: Katariina Hanhisalo <katariina.hanhisalo@xamk.fi> Batch works as described. Just to notice, if you in the end also cancel Patron Bs hold and check in the book, it still wants to go to the palace where Patron B had the pick-up location. But this is another problem I think. Created attachment 157067 [details] [review] Bug 34901: Display transit status on patron holds tab only if hold found value is "T" Check the transit status of the *hold* in addition to the transit status of the *item*, to avoid displaying a misleading transit status on item-level holds when the item is actually in transit for a different hold To test: 1. Create a record-level hold for Patron A for pickup at a library other than the logged-in library 2. Check in an item to fill that hold 3. Put an item-level hold on that same item for Patron B at a different library other than the logged-in library 4. Open Patron A's and Patron B's account details pages in separate tabs --> Note that the Holds tab on Patron A's account detail page correctly shows that their hold is in-transit --> Note that the Holds tab on Patron B's account detail page incorrectly shows that their hold on the same item is also in-transit 4. Apply patch 5. Clear browser cache 6. Refresh both patrons' account details pages --> Confirm that the holds tab on Patron A's account still correctly says their hold is in-transit --> Confirm that the holds tab on Patron B's account now correctly shows a blank status for their hold 7. Cancel Patron A's hold 8. Check in the item again to put it in transit for Patron B's hold 9. Reload Patron B's account page --> Confirm that the holds tab on Patron B's account now correctly says their hold is in-transit Signed-off-by: Tuomas Kunttu <tuomas.kunttu@kouvola.fi> Created attachment 157162 [details] [review] Bug 34901: Display transit status on patron holds tab only if hold found value is "T" Check the transit status of the *hold* in addition to the transit status of the *item*, to avoid displaying a misleading transit status on item-level holds when the item is actually in transit for a different hold To test: 1. Create a record-level hold for Patron A for pickup at a library other than the logged-in library 2. Check in an item to fill that hold 3. Put an item-level hold on that same item for Patron B at a different library other than the logged-in library 4. Open Patron A's and Patron B's account details pages in separate tabs --> Note that the Holds tab on Patron A's account detail page correctly shows that their hold is in-transit --> Note that the Holds tab on Patron B's account detail page incorrectly shows that their hold on the same item is also in-transit 4. Apply patch 5. Clear browser cache 6. Refresh both patrons' account details pages --> Confirm that the holds tab on Patron A's account still correctly says their hold is in-transit --> Confirm that the holds tab on Patron B's account now correctly shows a blank status for their hold 7. Cancel Patron A's hold 8. Check in the item again to put it in transit for Patron B's hold 9. Reload Patron B's account page --> Confirm that the holds tab on Patron B's account now correctly says their hold is in-transit Signed-off-by: Katariina Hanhisalo <katariina.hanhisalo@xamk.fi> Signed-off-by: Tuomas Kunttu <tuomas.kunttu@kouvola.fi> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Pushed to master for 23.11. Nice work everyone, thanks! Pushed to 23.05.x for 23.05.05 Nice work everyone! Pushed to oldstable for 22.11.x |