From 359b49c99a78e83ebbcab1937548bebdc461b3e0 Mon Sep 17 00:00:00 2001
From: Andrew Isherwood <andrew.isherwood@ptfs-europe.com>
Date: Tue, 23 Apr 2019 10:09:33 +0100
Subject: [PATCH] Bug 21460: Simplify the code to have only 1 assignment

---
 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