[open-ils-commits] r10905 - trunk/Open-ILS/web/opac/skin/default/js

[svn at svn.open-ils.org]

Author: erickson
Date: 2008-10-24 15:03:50 -0400 (Fri, 24 Oct 2008)
New Revision: 10905

Modified:
   trunk/Open-ILS/web/opac/skin/default/js/holds.js
Log:
fixed bug caused by null expire date

Modified: trunk/Open-ILS/web/opac/skin/default/js/holds.js
===================================================================
--- trunk/Open-ILS/web/opac/skin/default/js/holds.js	2008-10-24 19:03:25 UTC (rev 10904)
+++ trunk/Open-ILS/web/opac/skin/default/js/holds.js	2008-10-24 19:03:50 UTC (rev 10905)
@@ -726,12 +726,15 @@
 	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) {