--- loncom/interface/lonhtmlcommon.pm	2003/10/15 18:01:10	1.32
+++ loncom/interface/lonhtmlcommon.pm	2003/10/17 19:56:13	1.33
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # a pile of common html routines
 #
-# $Id: lonhtmlcommon.pm,v 1.32 2003/10/15 18:01:10 www Exp $
+# $Id: lonhtmlcommon.pm,v 1.33 2003/10/17 19:56:13 matthew Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -288,7 +288,7 @@ sub get_date_from_form {
     }
     if (defined($ENV{'form.'.$dname.'_hour'})) {
         my $tmphour = $ENV{'form.'.$dname.'_hour'};
-        if (($tmphour =~ /^\d+$/) && ($tmphour > 0) && ($tmphour < 32)) {
+        if (($tmphour =~ /^\d+$/) && ($tmphour >= 0) && ($tmphour < 24)) {
             $hour = $tmphour;
         }
     }
@@ -311,7 +311,9 @@ sub get_date_from_form {
         }
     }
     if (($year<70) || ($year>137)) { return undef; }
-    if (eval(&timelocal($sec,$min,$hour,$day,$month,$year))) {
+    if (defined($sec) && defined($min)   && defined($hour) &&
+        defined($day) && defined($month) && defined($year) &&
+        eval(&timelocal($sec,$min,$hour,$day,$month,$year))) {
         return &timelocal($sec,$min,$hour,$day,$month,$year);
     } else {
         return undef;