From c1ac88c9433d3897b14bf4bfb6a6885021572f9b 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 --- 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.11.0