From 741018e29057729652ca2ed4cb6eea9beea9e709 Mon Sep 17 00:00:00 2001
From: Jason Boyer <JBoyer@equinoxinitiative.org>
Date: Tue, 15 Sep 2020 14:00:54 -0400
Subject: [PATCH] LP1895660: HoldTarger.pm

"order_by" object in a query is not a JSON_HASH or JSON_ARRAY;no ORDER BY generated

json_query and CStoreEditor search_* functions don't want a string field name,
they work with hashes or arrays thereof.

Signed-off-by: Jason Boyer <JBoyer@equinoxinitiative.org>
Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>
---
 Open-ILS/src/perlmods/lib/OpenILS/Utils/HoldTargeter.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Open-ILS/src/perlmods/lib/OpenILS/Utils/HoldTargeter.pm b/Open-ILS/src/perlmods/lib/OpenILS/Utils/HoldTargeter.pm
index 3816822724..56e9c841a8 100644
--- a/Open-ILS/src/perlmods/lib/OpenILS/Utils/HoldTargeter.pm
+++ b/Open-ILS/src/perlmods/lib/OpenILS/Utils/HoldTargeter.pm
@@ -1230,7 +1230,7 @@ sub process_recalls {
             checkin_time => undef,
             duration => {'>' => $threshold}
         }, {
-            order_by => 'due_date',
+            order_by => [{ class => 'circ', field => 'due_date'}],
             limit => 1
         }
     ])->[0];
-- 
2.11.0