[open-ils-commits] ***SPAM*** [GIT] Evergreen ILS branch rel_2_4 updated. 7b645e63b6a40f5808a3343a8eadbfbb5f9fc4d8

[Evergreen Git]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Evergreen ILS".

The branch, rel_2_4 has been updated
       via  7b645e63b6a40f5808a3343a8eadbfbb5f9fc4d8 (commit)
      from  89e6cc779711c629b97f60d917876034ab2b3242 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 7b645e63b6a40f5808a3343a8eadbfbb5f9fc4d8
Author: Lebbeous Fogle-Weekley <lebbeous at esilibrary.com>
Date:   Thu Nov 14 15:59:55 2013 -0500

    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 <lebbeous at esilibrary.com>
    Signed-off-by: Remington Steed <rjs7 at calvin.edu>

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 92d67d5..f2a15f7 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

-----------------------------------------------------------------------

Summary of changes:
 .../staff_client/server/serial/batch_receive.js    |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)


hooks/post-receive
-- 
Evergreen ILS