gitea
/
Koha-clms
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Bug 33556: Path parameters are handled explicitly in the controllers 

In the case of $c->objects->search_rs, the variable is just not used.
In the case of /acq/orders, it's a leftover from when we removed in the
helper. Check there are tests with path params everywhere (including
orders) and it has no effect.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
(cherry picked from commit dcbd3e6929)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
This commit is contained in:
Tomas Cohen Arazi 2023-06-08 11:52:11 -03:00 committed by Fridolin Somers
parent 2b1b8884cc
commit 7796cf1e30
2 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
Koha/REST/Plugin/Objects.pm
View File

@ -157,8 +157,7 @@ controller, and thus shouldn't be called twice in it.
$query_fixers //= [];
# Extract reserved params
my ( $filtered_params, $reserved_params, $path_params ) =
$c->extract_reserved_params($args);
my ( $filtered_params, $reserved_params ) = $c->extract_reserved_params($args);
if ( exists $reserved_params->{_order_by} ) {

6
Koha/REST/V1/Acquisitions/Orders.pm
View File

@ -46,8 +46,10 @@ sub list {
return try {
my $only_active = delete $c->validation->output->{only_active};
my $order_id = delete $c->validation->output->{order_id};
my $only_active = $c->param('only_active');
my $order_id = $c->param('order_id');
$c->req->params->remove('only_active')->remove('order_id');
my $orders_rs;