From c3176021f6b41a56ad6c529b90a615b682a76145 Mon Sep 17 00:00:00 2001 From: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com> Date: Thu, 30 Aug 2012 12:20:13 -0400 Subject: [PATCH] Acq: Fix General Search for Lineitem Attribute-based fields Reported by Ben Shum and Kathy Lussier, the Acq General Search interface was broken in that you couldn't use any of the search fields under Lineitem Attribute. This error was indeed introduced by recent commit 55a82b5 as Ben determined, but simply reverting that commit left other things broken. This commit actually targets the problem and seems to fix it in my testing. Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com> Signed-off-by: Bill Erickson <berick@esilibrary.com> --- Open-ILS/web/js/ui/default/acq/search/unified.js | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/Open-ILS/web/js/ui/default/acq/search/unified.js b/Open-ILS/web/js/ui/default/acq/search/unified.js index 6f6142fbc9..c0a1fa9f5f 100644 --- a/Open-ILS/web/js/ui/default/acq/search/unified.js +++ b/Open-ILS/web/js/ui/default/acq/search/unified.js @@ -129,7 +129,13 @@ function TermSelectorFactory(terms) { ) { var term = this.getTerm(); var widgetKey = this.uniq; - var target = termManager.getLinkTarget(term); + var target; + try { + target = termManager.getLinkTarget(term); + } catch (E) { + void(0); /* ok for this to fail (it doesn't handle acqlia right, + but we don't need it to in this case). */ + }; if (matchHow.getValue() == "__in") { new openils.widget.XULTermLoader({ -- 2.11.0