From 7e14b263877371085069bf0cf5a72a51ac0d0d01 Mon Sep 17 00:00:00 2001 From: Jane Sandberg Date: Sun, 11 Jul 2021 15:49:19 -0700 Subject: [PATCH] LP1919483: Fix the queries for secondary admin pages To test, go to the following screens, and make sure that their secondary admin pages don't show errors and filter properly. * Authority Control Set * Authority Thesaurus * Course list * Hard due dates * MARC import remove fields * MARC Search/Facet fields * Z39.50 servers Signed-off-by: Jane Sandberg Signed-off-by: Christine Burns Signed-off-by: Galen Charlton --- .../eg2/src/app/staff/share/admin-page/admin-page.component.ts | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/Open-ILS/src/eg2/src/app/staff/share/admin-page/admin-page.component.ts b/Open-ILS/src/eg2/src/app/staff/share/admin-page/admin-page.component.ts index 7cc1650d74..d9242bdf7b 100644 --- a/Open-ILS/src/eg2/src/app/staff/share/admin-page/admin-page.component.ts +++ b/Open-ILS/src/eg2/src/app/staff/share/admin-page/admin-page.component.ts @@ -275,7 +275,7 @@ export class AdminPageComponent implements OnInit { this.idlClass, searchOps, {fleshSelectors: true}); } - const search: any = new Array(); + const search: any[] = new Array(); const orgFilter: any = {}; if (this.orgField && (this.searchOrgs || this.contextOrg)) { @@ -290,13 +290,15 @@ export class AdminPageComponent implements OnInit { }); }); - // FIXME - do we want to remove this, whose only present user - // is the booking grid, in favor of switching it to the built-in + // FIXME - do we want to remove this, which is used in several + // secondary admin pages, in favor of switching it to the built-in // grid filtering? if (this.gridFilters) { // Lay the URL grid filters over our search object. Object.keys(this.gridFilters).forEach(key => { - search[key] = this.gridFilters[key]; + const urlProvidedFilters = {}; + urlProvidedFilters[key] = this.gridFilters[key]; + search.push(urlProvidedFilters); }); } -- 2.11.0