Summary: | See highlighted items below link broken | ||
---|---|---|---|
Product: | Koha | Reporter: | Lisette Scheer <lisette> |
Component: | Patrons | Assignee: | 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
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. 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? Created attachment 171441 [details] [review] Bug 35987: Fix broken See highlighted items below link Can we get a test plan? https://wiki.koha-community.org/wiki/Commit_messages 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 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> |