From 3255525b59fa0ecd728a3b5942db89a21ab52f43 Mon Sep 17 00:00:00 2001 From: Andrew Isherwood Date: Tue, 23 Apr 2019 10:09:33 +0100 Subject: [PATCH] Bug 21460: Simplify the code to have only 1 assignment Signed-off-by: Josef Moravec Signed-off-by: Nick Clemens --- Koha/REST/V1/Illrequests.pm | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/Koha/REST/V1/Illrequests.pm b/Koha/REST/V1/Illrequests.pm index c3c51a592d..f747390792 100644 --- a/Koha/REST/V1/Illrequests.pm +++ b/Koha/REST/V1/Illrequests.pm @@ -56,14 +56,11 @@ sub list { # Get all requests # If necessary, only get those from a specified patron - my @requests; - if ($args->{borrowernumber}) { - @requests = Koha::Illrequests->search( - { borrowernumber => $args->{borrowernumber} } - ); - } else { - @requests = Koha::Illrequests->as_list; - } + my @requests = Koha::Illrequests->search({ + args->{borrowernumber} + ? ( borrowernumber => $args->{borrowernumber} ) + : () + })->as_list; # Identify patrons & branches that # we're going to need and get them -- 2.39.5