https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36347
Lucas Gass <lu...@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #163747|0 |1 is obsolete| | --- Comment #5 from Lucas Gass <lu...@bywatersolutions.com> --- Created attachment 163849 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=163849&action=edit Bug 36347: Avoid return claims table being loaded twice The issue is caused by these lines in circulation.js: $("#finesholdsissues a[data-toggle='tab']").on("shown.bs.tab", function(e){ $(this).click(); }); When the user clicks on a hidden tab, 'shown.bs.tab' is triggered and then another click event is triggered. And because the table was loaded on tab click, it was loaded twice. Visually it could have caused the rows to be duplicated (it was random, probably due to timing, network latency, ...) I don't know why we need to trigger a click in this case, but removing this will probably break something else, so the fix here is to load the table on 'shown.bs.tab' event instead of 'click' This patch also contains a small fix in refreshReturnClaimsTable which was reloading the table right after the table initialization. Test plan: 1. Check out an item, and in the checkouts table, click "Claim returned" 2. Open the network tab of your browser's developer tools 3. Click on the "Claims" tab, verify that only one call is made to svc/return_claims 4. Verify that the claims table is displayed correctly Signed-off-by: David Nind <da...@davidnind.com> Signed-off-by: Lucas Gass <lu...@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/