@@ -, +, @@ pickup locations GET /api/v1/app.pl/api/v1/holds/5/pickup_locations 403 Forbidden --- api/v1/swagger/paths/holds.json | 2 +- t/db_dependent/api/v1/holds.t | 12 +++++++++++- 2 files changed, 12 insertions(+), 2 deletions(-) --- a/api/v1/swagger/paths/holds.json +++ a/api/v1/swagger/paths/holds.json @@ -672,7 +672,7 @@ }, "x-koha-authorization": { "permissions": { - "reserveforothers": "1" + "reserveforothers": "place_holds" } } } --- a/t/db_dependent/api/v1/holds.t +++ a/t/db_dependent/api/v1/holds.t @@ -686,11 +686,21 @@ subtest 'pickup_locations() tests' => sub { my $patron = $builder->build_object( { class => 'Koha::Patrons', - value => { userid => 'tomasito', flags => 1 } + value => { userid => 'tomasito', flags => 0 } } ); $patron->set_password( { password => $password, skip_validation => 1 } ); my $userid = $patron->userid; + $builder->build( + { + source => 'UserPermission', + value => { + borrowernumber => $patron->borrowernumber, + module_bit => 6, + code => 'place_holds', + }, + } + ); my $item_class = Test::MockModule->new('Koha::Item'); $item_class->mock( --