Bug 29737

Summary: Cannot suspend holds
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: Hold requestsAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Katrin Fischer <katrin.fischer>
Severity: major    
Priority: P5 - low CC: andrewfh, gmcharlt, kyle, tomascohen
Version: unspecifiedKeywords: regression
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:
22.05.00,21.11.02
Attachments: Bug 29737: Fix suspend from hold list
Bug 29737: Remove some unecessary parameters
Bug 29737: Fix suspend from hold list
Bug 29737: Remove some unecessary parameters
Bug 29737: Fix suspend from hold list
Bug 29737: Remove some unecessary parameters

Description Jonathan Druart 2021-12-20 12:49:36 UTC
From holds list of a bibliographic record, the "unsuspend" button does not work as expected. The form is submitted but the suspension is still there.

There are 3 requests, 2 GET and 1 POST. One of the GET is rejected by Firefox (NS_BINDING_ABORTED)
Comment 1 Jonathan Druart 2021-12-20 13:11:43 UTC
Created attachment 128754 [details] [review]
Bug 29737: Fix suspend from hold list

From holds list of a bibliographic record, the "unsuspend" button does not work as expected.
The form is submitted but the suspension is still there.

There are 3 requests, 2 GET and 1 POST.
One of the GET is rejected by Firefox (NS_BINDING_ABORTED)

Test plan:
Place some items on hold, play with suspend/unsuspend from the hold list
/cgi-bin/koha/reserve/request.pl?biblionumber=XX
Comment 2 Jonathan Druart 2021-12-20 13:11:47 UTC
Created attachment 128755 [details] [review]
Bug 29737: Remove some unecessary parameters

We don't need them. We could also remove the biblionumber but it
requires change to the controller I'd prefer to not do now.
Comment 3 Jonathan Druart 2021-12-20 13:12:18 UTC
This is a regression but no idea when it has been introduced!
Comment 4 David Nind 2021-12-20 17:29:54 UTC
Created attachment 128780 [details] [review]
Bug 29737: Fix suspend from hold list

From holds list of a bibliographic record, the "unsuspend" button does not work as expected.
The form is submitted but the suspension is still there.

There are 3 requests, 2 GET and 1 POST.
One of the GET is rejected by Firefox (NS_BINDING_ABORTED)

Test plan:
Place some items on hold, play with suspend/unsuspend from the hold list
/cgi-bin/koha/reserve/request.pl?biblionumber=XX

Signed-off-by: David Nind <david@davidnind.com>
Comment 5 David Nind 2021-12-20 17:30:00 UTC
Created attachment 128781 [details] [review]
Bug 29737: Remove some unecessary parameters

We don't need them. We could also remove the biblionumber but it
requires change to the controller I'd prefer to not do now.

Signed-off-by: David Nind <david@davidnind.com>
Comment 6 Katrin Fischer 2021-12-20 23:09:58 UTC
Created attachment 128803 [details] [review]
Bug 29737: Fix suspend from hold list

From holds list of a bibliographic record, the "unsuspend" button does not work as expected.
The form is submitted but the suspension is still there.

There are 3 requests, 2 GET and 1 POST.
One of the GET is rejected by Firefox (NS_BINDING_ABORTED)

Test plan:
Place some items on hold, play with suspend/unsuspend from the hold list
/cgi-bin/koha/reserve/request.pl?biblionumber=XX

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 7 Katrin Fischer 2021-12-20 23:10:02 UTC
Created attachment 128804 [details] [review]
Bug 29737: Remove some unecessary parameters

We don't need them. We could also remove the biblionumber but it
requires change to the controller I'd prefer to not do now.

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 8 Fridolin Somers 2022-01-10 07:05:03 UTC
Pushed to master for 22.05, thanks to everybody involved 🦄
Comment 9 Kyle M Hall 2022-01-14 13:38:31 UTC
Pushed to 21.11.x for 21.11.02
Comment 10 Andrew Fuerste-Henry 2022-01-21 16:46:33 UTC
Cannot recreate issue on 21.05.x. Not backporting. Please request if needed.