From 14f7dba6f6dc938e75c1fab3093e5b5a9c94e0b3 Mon Sep 17 00:00:00 2001 From: Lebbeous Fogle-Weekley 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 --- 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