From 0aa178e5ba30ab72f7cbbd6688e51c4075f1745a 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 --- 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.11.0