From 70c18bae260162bb95fd66a231a2d33793ec577d Mon Sep 17 00:00:00 2001 From: Bill Erickson Date: Wed, 22 Sep 2021 11:04:18 -0400 Subject: [PATCH] LP1919465 Pull list only shows holds with current_copy This avoids a scenario where copy-level holds would appear in the pull list even if they are not pull-able, because they have "a copy" (i.e. the requested copy), but no current copy (i.e. targeted copy). Signed-off-by: Bill Erickson Signed-off-by: Jennifer Weston Signed-off-by: Chris Sharp --- Open-ILS/src/eg2/src/app/staff/share/holds/grid.component.ts | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/Open-ILS/src/eg2/src/app/staff/share/holds/grid.component.ts b/Open-ILS/src/eg2/src/app/staff/share/holds/grid.component.ts index 4b7b776b04..70ad1783df 100644 --- a/Open-ILS/src/eg2/src/app/staff/share/holds/grid.component.ts +++ b/Open-ILS/src/eg2/src/app/staff/share/holds/grid.component.ts @@ -249,6 +249,10 @@ export class HoldsGridComponent implements OnInit { filters.capture_time = null; filters.frozen = 'f'; + // cp.* fields are set for copy-level holds even if they + // have no current_copy. Make sure current_copy is set. + filters.current_copy = {'is not': null}; + // There are aliases for these (cp_status, cp_circ_lib), // but the API complains when I use them. filters['cp.status'] = [0, 7]; -- 2.11.0