[open-ils-commits] [GIT] Evergreen ILS branch rel_2_4 updated. 7e78cc659467a543b7d37cbe29e5675517ed346b

[Evergreen Git]

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_4 has been updated
       via  7e78cc659467a543b7d37cbe29e5675517ed346b (commit)
      from  20a28c05dd6c56080596f262a8cb1d830d2ec967 (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 7e78cc659467a543b7d37cbe29e5675517ed346b
Author: Galen Charlton <gmc at esilibrary.com>
Date:   Fri Oct 4 14:44:56 2013 -0700

    LP#1235474: fix failure of authority browse context menu to render
    
    This patch fixes a situation where if an authority record that would
    be brought up by the authority browse context menu in the MARC editor
    lacks an 003 field, the context menu would not be rendered all, meaning
    that the default context menu (e.g., cut/copy/paste/etc.) would be
    displayed instead.
    
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>
    Signed-off-by: Dan Scott <dscott at laurentian.ca>

diff --git a/Open-ILS/xul/staff_client/server/cat/marcedit.js b/Open-ILS/xul/staff_client/server/cat/marcedit.js
index ba222b4..a00f4fe 100644
--- a/Open-ILS/xul/staff_client/server/cat/marcedit.js
+++ b/Open-ILS/xul/staff_client/server/cat/marcedit.js
@@ -1805,7 +1805,10 @@ function browseAuthority (sf_popup, menu_id, target, sf, limit, page) {
             var see_from = [];
             var see_also = [];
             var auth_id = dojox.xml.parser.textContent(dojo.query('datafield[tag="901"]', record).query('subfield[code="c"]')[0]);
-            var auth_org = dojox.xml.parser.textContent(dojo.query('controlfield[tag="003"]', record)[0]);
+            var auth_org = '';
+            if (dojo.query('controlfield[tag="003"]', record).length > 0) {
+                auth_org = dojox.xml.parser.textContent(dojo.query('controlfield[tag="003"]', record)[0]);
+            }
 
             // Grab the fields with tags beginning with 1 (main entries) and iterate through the subfields
             dojo.query('datafield[tag^="1"]', record).forEach(function(field) {

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

Summary of changes:
 Open-ILS/xul/staff_client/server/cat/marcedit.js |    5 ++++-
 1 files changed, 4 insertions(+), 1 deletions(-)


hooks/post-receive
-- 
Evergreen ILS