[open-ils-commits] [GIT] Evergreen ILS branch rel_2_3 updated. 6f9892fc38372d812fd2d8e3cd1be4523fd9e89b

Evergreen Git git at git.evergreen-ils.org
Thu Aug 30 12:36:36 EDT 2012


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Evergreen ILS".

The branch, rel_2_3 has been updated
       via  6f9892fc38372d812fd2d8e3cd1be4523fd9e89b (commit)
      from  746f6bd12298b0ceb510a307cce11f62fdeeff75 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 6f9892fc38372d812fd2d8e3cd1be4523fd9e89b
Author: Lebbeous Fogle-Weekley <lebbeous at esilibrary.com>
Date:   Thu Aug 30 12:20:13 2012 -0400

    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 at esilibrary.com>
    Signed-off-by: Bill Erickson <berick at esilibrary.com>

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 6f6142f..c0a1fa9 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({

-----------------------------------------------------------------------

Summary of changes:
 Open-ILS/web/js/ui/default/acq/search/unified.js |    8 +++++++-
 1 files changed, 7 insertions(+), 1 deletions(-)


hooks/post-receive
-- 
Evergreen ILS


More information about the open-ils-commits mailing list