From 23c0ff5327744f3ab1398203464237a266ba5817 Mon Sep 17 00:00:00 2001
From: phasefx <phasefx@dcc99617-32d9-48b4-a31d-7c20da2025e4>
Date: Mon, 29 Dec 2008 17:33:43 +0000
Subject: [PATCH] Javascript functions don't have an implicit return.  Inverted
 the logic (filter_list is backwards).  Added some defensive code

git-svn-id: svn://svn.open-ils.org/ILS/trunk@11685 dcc99617-32d9-48b4-a31d-7c20da2025e4
---
 Open-ILS/xul/staff_client/server/patron/bill_wizard.js | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/Open-ILS/xul/staff_client/server/patron/bill_wizard.js b/Open-ILS/xul/staff_client/server/patron/bill_wizard.js
index 73e0e7e022..9d65d7f063 100644
--- a/Open-ILS/xul/staff_client/server/patron/bill_wizard.js
+++ b/Open-ILS/xul/staff_client/server/patron/bill_wizard.js
@@ -98,7 +98,7 @@ function patron_bill_init() {
         JSAN.use('util.money');
         JSAN.use('util.widgets');
         JSAN.use('util.functional');
-        var billing_list = util.functional.filter_list( g.OpenILS.data.list.cbt, function (x) { x.id() < 100 } );
+        var billing_list = util.functional.filter_list( g.OpenILS.data.list.cbt, function (x) { return x.id() >= 100 } );
         var ml = util.widgets.make_menulist(
             util.functional.map_list(
                 billing_list.sort( function(a,b) { if (a.name()>b.name()) return 1; if (a.name()<b.name()) return -1; return 0; } ), //g.OpenILS.data.list.billing_type.sort(),
@@ -111,7 +111,9 @@ function patron_bill_init() {
         ml.addEventListener(
             'command',
             function() {
-                $('bill_amount').value = billing_list[ ml.value ].default_price();
+                if ( billing_list[ ml.value ] ) {
+                    $('bill_amount').value = billing_list[ ml.value ].default_price();
+                }
             },
             false
         ); 
@@ -120,7 +122,9 @@ function patron_bill_init() {
 
         $('billing_location').setAttribute('value', g.OpenILS.data.hash.aou[ g.OpenILS.data.list.au[0].ws_ou() ].name() );
 
-        $('bill_amount').value = billing_list[ ml.value ].default_price();
+        if ( billing_list[ ml.value ] ) {
+            $('bill_amount').value = billing_list[ ml.value ].default_price();
+        }
         $('bill_amount').select(); $('bill_amount').focus();
 
         if (xul_param('xact_id',{'modal_xulG':true})) { 
-- 
2.11.0