[Bug 24466] New: AllowHoldsOnPatronsPossessions and the REST API
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24466 Bug ID: 24466 Summary: AllowHoldsOnPatronsPossessions and the REST API Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Hold requests Assignee: koha-bugs@lists.koha-community.org Reporter: magnus@libriotech.no QA Contact: testopia@bugs.koha-community.org CC: gmcharlt@gmail.com A library in Sweden has reported the following problem: AllowHoldsOnPatronsPossessions = Do not allow. A patron borroes a book. A hold on the same book is then placed, for the same patron, through the REST API: curl --location --request POST 'https://{baseurl}/api/v1/holds/' \ --header 'Content-Type: text/plain' \ --header 'Authorization: Basic {base64-encoded user pwd}' \ --data-raw '{ "patron_id": {patronid}, "pickup_library_id": "{library}", "biblio_id": {biblioid} }' And voila, the patron has a hold on a record from which they already have an item on loan, contrary to the setting of the AllowHoldsOnPatronsPossessions syspref. (I have not tested this, but do trust that the report from the library is accurate.) -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24466 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@bugs.koha-c | |ommunity.org --- Comment #1 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- I guess we need to move that block from request.pl to CanBookBeReserved: 347 { # alreadypossession 348 # Check to see if patron is allowed to place holds on records where the 349 # patron already has an item from that record checked out 350 if ( !C4::Context->preference('AllowHoldsOnPatronsPossessions') 351 && CheckIfIssuedToPatron( $patron->borrowernumber, $biblionumber ) ) 352 { 353 $template->param( alreadypossession => 1, ); 354 } 355 } -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24466 --- Comment #2 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Maybe also get_opacitemholds_policy can be bypassed -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24466 --- Comment #3 from Magnus Enger <magnus@libriotech.no> --- Still a problem in 19.11.06.000. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24466 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia@bugs.koha-communit | |y.org | Component|Hold requests |REST API See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=25662 -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24466 Magnus Enger <magnus@libriotech.no> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution|--- |DUPLICATE --- Comment #4 from Magnus Enger <magnus@libriotech.no> --- This should be fixed by the patches on bug 22806, I think. *** This bug has been marked as a duplicate of bug 22806 *** -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org