[open-ils-commits] r10904 - branches/rel_1_4/Open-ILS/web/opac/skin/default/js

[svn at svn.open-ils.org]

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

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

Modified: branches/rel_1_4/Open-ILS/web/opac/skin/default/js/holds.js
===================================================================
--- branches/rel_1_4/Open-ILS/web/opac/skin/default/js/holds.js	2008-10-24 18:30:02 UTC (rev 10903)
+++ branches/rel_1_4/Open-ILS/web/opac/skin/default/js/holds.js	2008-10-24 19:03:25 UTC (rev 10904)
@@ -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) {