@@ -, +, @@ --- Koha/REST/Plugin/Query.pm | 11 ++--------- t/Koha/REST/Plugin/Query.t | 6 +++++- 2 files changed, 7 insertions(+), 10 deletions(-) --- a/Koha/REST/Plugin/Query.pm +++ a/Koha/REST/Plugin/Query.pm @@ -277,16 +277,9 @@ Merges parameters from $q_params into $filtered_params. my ( $c ) = @_; - my $override_header = $c->req->headers->header('x-koha-override'); + my $override_header = $c->req->headers->header('x-koha-override') || q{}; - my $overrides = {}; - - if ( $override_header ) { - my @overrides = (); - foreach my $override ( split /\s*,\s*/, $override_header ) { - $overrides->{$override} = 1; - } - } + my $overrides = { map { $_ => 1 } split /\s*,\s*/, $override_header }; $c->stash( 'koha.overrides' => $overrides ); --- a/t/Koha/REST/Plugin/Query.t +++ a/t/Koha/REST/Plugin/Query.t @@ -502,7 +502,7 @@ subtest 'stash_embed() tests' => sub { subtest 'stash_overrides() tests' => sub { - plan tests => 4; + plan tests => 6; my $t = Test::Mojo->new; @@ -511,4 +511,8 @@ subtest 'stash_overrides() tests' => sub { $t->get_ok( '/stash_overrides' => { 'x-koha-override' => '' } ) ->json_is( {} ); # empty string is skipped + + $t->get_ok( '/stash_overrides' => { } ) + ->json_is( {} ); # x-koha-ovverride not passed is skipped + }; --