[Koha-bugs] [Bug 36103] "Cancel hold" link broken for item level holds

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36103

Pedro Amorim <pedro.amorim at ptfs-europe.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |kyle at bywatersolutions.com,
                   |                            |oleonard at myacpl.org

--- Comment #3 from Pedro Amorim <pedro.amorim at ptfs-europe.com> ---
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.