From 7e78cc659467a543b7d37cbe29e5675517ed346b Mon Sep 17 00:00:00 2001 From: Galen Charlton Date: Fri, 4 Oct 2013 14:44:56 -0700 Subject: [PATCH] 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 Signed-off-by: Dan Scott --- Open-ILS/xul/staff_client/server/cat/marcedit.js | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/Open-ILS/xul/staff_client/server/cat/marcedit.js b/Open-ILS/xul/staff_client/server/cat/marcedit.js index ba222b4f21..a00f4fe0a5 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) { -- 2.11.0