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