From 7036fcc024430cca1f3e2ae8e432fa8835800124 Mon Sep 17 00:00:00 2001 From: Lebbeous Fogle-Weekley Date: Thu, 14 Nov 2013 15:59:55 -0500 Subject: [PATCH] Serials: Batch Receive interface Copy Location widget was inert Choosing a value from the Copy Location widget wasn't working. You'd get a default copy location (or the one supplied by the Receive Unit Copy Template) no matter what you set that control too. This is fixed. One does not read from XUL menulist element by using its 'value' property. Signed-off-by: Lebbeous Fogle-Weekley Signed-off-by: Remington Steed --- Open-ILS/xul/staff_client/server/serial/batch_receive.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Open-ILS/xul/staff_client/server/serial/batch_receive.js b/Open-ILS/xul/staff_client/server/serial/batch_receive.js index 92d67d538b..f2a15f7691 100644 --- a/Open-ILS/xul/staff_client/server/serial/batch_receive.js +++ b/Open-ILS/xul/staff_client/server/serial/batch_receive.js @@ -428,7 +428,8 @@ function BatchReceiver() { var node = dojo.query("*", node_by_name(field, row))[0]; if (typeof(value) == "undefined") { - return node.value; + return node.selectedItem ? + node.selectedItem.value : node.value; } else { /* XXX The new two lines /should/ each do the same thing, but * apparently they don't. With only one or the other, I get -- 2.43.2