Bug 35987

Summary: See highlighted items below link broken
Product: Koha Reporter: Lisette Scheer <lisette>
Component: PatronsAssignee: Bugs List <koha-bugs>
Status: Passed QA --- QA Contact: Aleisha Amohia <aleisha>
Severity: normal    
Priority: P5 - low CC: aleisha, gmcharlt, kyle.m.hall, lmstrand, lucas, shi-yao.wang
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Bug Depends on: 33528    
Bug Blocks:    
Attachments: Bug 35987: Fix broken See highlighted items below link
Bug 35987: Fix broken See highlighted items below link
Bug 35987: Fix broken See highlighted items below link

Description Lisette Scheer 2024-02-02 22:05:09 UTC
On circulation.pl and moremember.pl the "see highlighted items below" link goes to #checkouts, but there is no div #checkouts instead of #checkouts_panel
Comment 1 Lucas Gass (lukeg) 2024-02-05 14:54:36 UTC
Even with `*_panel` the behavior is still odd if you don't "Always show checkouts automatically". 

Clicking "See highlighted items below:" should expand the checkout table if it is not already.
Comment 2 Lari Strand 2024-06-05 12:33:19 UTC
Adding a double event onclick="$('#issues-table-load-immediately').click(); $('#issues-table-load-immediately').click();" to the checkouts_panel href text link fixes this without need to create a new function that checks the existence of the issues datatable (without the check I get a popup error DataTables warning: table id=issues-table - Cannot reinitialise DataTable if I press on the "See highlighted items below" text link again). It's a dirty fix that simulates two clicks on the Always show checkouts automatically checkbox. Is this acceptable?
Comment 3 Lari Strand 2024-09-13 10:50:23 UTC
Created attachment 171441 [details] [review]
Bug 35987: Fix broken See highlighted items below link
Comment 4 Shi Yao Wang 2024-09-26 19:33:56 UTC
Can we get a test plan?
https://wiki.koha-community.org/wiki/Commit_messages
Comment 5 Magnus Enger 2024-10-31 09:27:16 UTC
Created attachment 173769 [details] [review]
Bug 35987: Fix broken See highlighted items below link

Signed-off-by: Magnus Enger <magnus@libriotech.no>
Test plan with KTD:
- Issue a couple of items to a patron, and make sure one of them is
  overdue.
- Make sure "Always show checkouts automatically" is not checked
- Reload the page
- Click on the link in "Overdues: Patron has ITEMS OVERDUE See
  highlighted items below"
- Nothing happens
- Apply the patch, restart_all, reload the "Check out" page for the
  patron
- Click on the link again, and the active loans should be shown
Comment 6 Aleisha Amohia 2024-11-06 03:12:38 UTC
Created attachment 174060 [details] [review]
Bug 35987: Fix broken See highlighted items below link

Signed-off-by: Magnus Enger <magnus@libriotech.no>
Test plan with KTD:
- Issue a couple of items to a patron, and make sure one of them is
  overdue.
- Make sure "Always show checkouts automatically" is not checked
- Reload the page
- Click on the link in "Overdues: Patron has ITEMS OVERDUE See
  highlighted items below"
- Nothing happens
- Apply the patch, restart_all, reload the "Check out" page for the
  patron
- Click on the link again, and the active loans should be shown

Signed-off-by: Aleisha Amohia <aleishaamohia@hotmail.com>