From c24e34676132640e5c652383290e4d5a5f705fc3 Mon Sep 17 00:00:00 2001 From: Dan Scott Date: Tue, 9 Oct 2012 16:49:09 -0400 Subject: [PATCH] Authority popups are empty: fix dojo.query() usage When you right-click on a controllable field to display the contents of the matching authority records, the contents of the matching authority records simply appear as empty grey boxes. This is another case of needing to iterate over multiple values separately rather than using one convenient CSS selector. Signed-off-by: Dan Scott Signed-off-by: Bill Erickson --- Open-ILS/xul/staff_client/server/cat/marcedit.js | 8 +++++++- 1 file changed, 7 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 6abd005ff9..39fcca57b4 100644 --- a/Open-ILS/xul/staff_client/server/cat/marcedit.js +++ b/Open-ILS/xul/staff_client/server/cat/marcedit.js @@ -1853,7 +1853,13 @@ function buildAuthorityPopup (entry_text, record, auth_org, auth_id, sf_popup, t } submenu.appendChild(popup); - dojo.query('datafield[tag^="1"], datafield[tag^="4"], datafield[tag^="5"]', record).forEach(function(field) { + dojo.query('datafield[tag^="1"]', record).forEach(function(field) { + buildAuthorityPopupSelector(field, grid, auth_org, auth_id); + }); + dojo.query('datafield[tag^="4"]', record).forEach(function(field) { + buildAuthorityPopupSelector(field, grid, auth_org, auth_id); + }); + dojo.query('datafield[tag^="5"]', record).forEach(function(field) { buildAuthorityPopupSelector(field, grid, auth_org, auth_id); }); -- 2.11.0