fixed bug caused by null expire date

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_4@10904 dcc99617-32d9-48b4-a31d-7c20da2025e4

diff --git a/Open-ILS/web/opac/skin/default/js/holds.js b/Open-ILS/web/opac/skin/default/js/holds.js
index 0cc9447..33b42f1 100644 (file)
--- a/Open-ILS/web/opac/skin/default/js/holds.js
+++ b/Open-ILS/web/opac/skin/default/js/holds.js
@@ -726,12 +726,15 @@ function holdsBuildHoldFromWindow() {
        hold.target(target);
        hold.hold_type(holdArgs.type);
 
-    var expireDate = dojo.date.stamp.toISOString(dijit.byId('holds_expire_time').getValue())
-    expireDate = holdsVerifyThawDate(expireDate); 
-    if(expireDate)
-        hold.expire_time(expireDate);
-    else 
-        return;
+    var expDate = dijit.byId('holds_expire_time').getValue();
+    if(expDate) {
+        var expireDate = dojo.date.stamp.toISOString(expDate);
+        expireDate = holdsVerifyThawDate(expireDate); 
+        if(expireDate)
+            hold.expire_time(expireDate);
+        else 
+            return;
+    }
 
     // see if this hold should be frozen and for how long
     if($('holds_frozen_chkbox').checked) {