From 251717941bae2a4606d968065d7aa241349bb0b5 Mon Sep 17 00:00:00 2001 From: Jason Etheridge Date: Wed, 28 Mar 2012 15:19:42 -0400 Subject: [PATCH] fix a Down action bug in Configure Toolbars where the list selection index could be set to an invalid value, which could cause errors for actions acting upon that index Signed-off-by: Jason Etheridge Signed-off-by: Thomas Berezansky --- Open-ILS/xul/staff_client/server/admin/toolbar.js | 1 + 1 file changed, 1 insertion(+) diff --git a/Open-ILS/xul/staff_client/server/admin/toolbar.js b/Open-ILS/xul/staff_client/server/admin/toolbar.js index 8e899be046..8a0e17afbc 100644 --- a/Open-ILS/xul/staff_client/server/admin/toolbar.js +++ b/Open-ILS/xul/staff_client/server/admin/toolbar.js @@ -303,6 +303,7 @@ function populate_list2_list3(list3_idx) { if (list3_idx) { if (list3_idx < 0) { list3_idx = 0; } + if (list3_idx >= g.list3.node.view.rowCount) { list3_idx = g.list3.node.view.rowCount - 1; } g.list3.node.view.selection.select(list3_idx); } -- 2.43.2