From 3390b5454048bc8c1b58c20b304f545510f7339b Mon Sep 17 00:00:00 2001
From: blake <blake@mobiusconsortium.org>
Date: Tue, 15 Mar 2022 11:48:12 -0500
Subject: [PATCH] LP#1964986: encourage distinct results when querying the
 ahopl IDL source

This patch addresses a now-theoretical issue where a direct PCRUD
query of ahopl could return duplicate rows in cases where the patron
linked to the hold request has multiple penalties attached.

The significance of this change is likely minor nowadays, as ahopl
was directly used by Evergreen only by the Dojo and AngularJS hold
pull lists, and as of 3.8 the Evergreen Angular pull list uses a
different approach for querying the pull list. However, until such
time as ahopl is deprecated and removed, this change may help
local integrations.

Signed-off-by: blake <blake@mobiusconsortium.org>
Signed-off-by: Jennifer Weston <jennifer.weston@equinoxOLI.org>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>
---
 Open-ILS/examples/fm_IDL.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Open-ILS/examples/fm_IDL.xml b/Open-ILS/examples/fm_IDL.xml
index 609a596628..7f63701165 100644
--- a/Open-ILS/examples/fm_IDL.xml
+++ b/Open-ILS/examples/fm_IDL.xml
@@ -6863,7 +6863,7 @@ SELECT  usr,
 	</class>
 	<class id="ahopl" controller="open-ils.cstore open-ils.pcrud" oils_obj:fieldmapper="action::hold_on_pull_list" reporter:label="Hold On Pull List" oils_persist:readonly="true">
 		<oils_persist:source_definition><![CDATA[
-		SELECT
+		SELECT DISTINCT
 			ahr.*,
 			COALESCE(acplo.position, acpl_ordered.fallback_position) AS
 				copy_location_order_position,
-- 
2.11.0