LP#647121 Event handling on override abort

Discussed on IRC, phasefx discovered that several events were not being properly
handled when they were empty strings.  Tested and working on Evergreen 2.0.4.

Signed-off-by: Michael Peters <mrpeters@library.in.gov>
Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>

diff --git a/Open-ILS/xul/staff_client/server/circ/checkout.js b/Open-ILS/xul/staff_client/server/circ/checkout.js
index c7d3813..11f5373 100644 (file)
--- a/Open-ILS/xul/staff_client/server/circ/checkout.js
+++ b/Open-ILS/xul/staff_client/server/circ/checkout.js
@@ -762,7 +762,7 @@ circ.checkout.prototype = {
 
                 for (var i = 0; i < test_permit.length; i++) {
                     dump('found [' + test_permit[i].ilsevent + ']\n');
-                    switch(test_permit[i].ilsevent == null ? null : Number(test_permit[i].ilsevent)) {
+                    switch(test_permit[i].ilsevent == null || test_permit[i].ilsevent == '' ? null : Number(test_permit[i].ilsevent)) {
                         case null /* custom event */ :
                             found_handled = true;
                         break;