https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36103
Pedro Amorim <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |[email protected], | |[email protected] --- Comment #3 from Pedro Amorim <[email protected]> --- I'm not convinced by my patch but I'm not sure what the intend here is so I'm adding Owen and Kyle in the hopes for additional input. It seems the JS using .cancel-hold in reserve/request.tt is meant to be working on the .cancel-hold link coming from holds_table.inc, i.e. this table: http://localhost:8081/cgi-bin/koha/reserve/request.pl?biblionumber=76 But its also affecting the other table mentioned in the test plan, this table: http://localhost:8081/cgi-bin/koha/reserve/request.pl?biblionumber=76&borrowernumber=51 (To see the problematic "Cancel hold" link here you need to have a hold previously placed) So, do we remove this CSS class from this link and keep its behaviour as before? Or do we move this problematic link to also make use of the cancellation reason modal and request.pl?action=cancel or the older modRequest.pl as currently is? -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. _______________________________________________ Koha-bugs mailing list [email protected] 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/
