--- loncom/enrollment/Autoenroll.pl	2003/12/08 15:47:22	1.2
+++ loncom/enrollment/Autoenroll.pl	2003/12/11 03:24:20	1.4
@@ -13,7 +13,7 @@
 # Determine the library server's domain
     my $perlvarref = LONCAPA::Configuration::read_conf('loncapa.conf');
     my $dom = $$perlvarref{'lonDefDomain'};
-    my $tmpdir = $$perlvarref{'lonDaemons'}.'/tmp';
+    my $logfile = $$perlvarref{'lonDaemons'}.'/logs/autoenroll.log';
     $ENV{'user.domain'} = $dom;
 
 # Determine the present time;
@@ -23,7 +23,6 @@
     my %courses = &Apache::lonnet::courseiddump($dom,'.',1); 
     my %affiliates = ();
     my %enrollvar = ();
-    my %logmsg = ();
     my %reply = ();
     my %LC_code = ();
     foreach my $key (sort keys %courses) {
@@ -87,17 +86,18 @@
     &localenroll::fetch_enrollment($dom,\%affiliates,\%reply);
 
 # Now go through classes and perform required enrollment changes.
+    open (my $fh,">>$logfile");
+    print $fh "********************\n".localtime(time)." Enrollment messages start --\n";
     foreach my $crs (sort keys %enrollvar) {
+        my $logmsg = '';
+        my $newusermsg = '';
         if ($reply{$crs} > 0) {
             if (($enrollvar{$crs}{autostart} < $timenow) && ($enrollvar{$crs}{autoend} > $timenow)) {
                 if (($enrollvar{$crs}{autoadds} == 1) || ($enrollvar{$crs}{autodrops} == 1)) {
-                    my ($changecount,$response) = &LONCAPA::Enrollment::update_LC($dom,$crs,$enrollvar{$crs}{autoadds},$enrollvar{$crs}{autodrops},$enrollvar{$crs}{startdate},$enrollvar{$crs}{enddate},$enrollvar{$crs}{authtype},$enrollvar{$crs}{autharg},\@{$affiliates{$crs}},\%{$LC_code{$crs}},\$logmsg{$crs},'automated');
-                    my $logfile = $crs."_enrollment_log.txt";
-                    open (FILE,">>$tmpdir/$logfile");
-                    print FILE "********************\n".localtime(time)." Enrollment messages start --\n";
-                    print FILE "$logmsg{$crs}\n";
-                    print FILE "-- ".localtime(time)." Enrollment messages end\n********************\n\n"; 
-                    close (FILE);
+                    my ($changecount,$response) = &LONCAPA::Enrollment::update_LC($dom,$crs,$enrollvar{$crs}{autoadds},$enrollvar{$crs}{autodrops},$enrollvar{$crs}{startdate},$enrollvar{$crs}{enddate},$enrollvar{$crs}{authtype},$enrollvar{$crs}{autharg},\@{$affiliates{$crs}},\%{$LC_code{$crs}},\$logmsg,\$newusermsg,'automated');
+                    print $fh "Messages start for $crs\n";
+                    print $fh "$logmsg\n";
+                    print $fh "Messages end for $crs\n";
                     if ($changecount > 0) {
                         unless ($enrollvar{$crs}{notifylist}  eq '') {
 # Send message about enrollment changes to notifylist.
@@ -108,6 +108,10 @@
 
                                 my $subject = "Student enrollment changes in $enrollvar{$crs}{coursecode}";
                                 my $message = "The following $changecount change(s) occurred in $enrollvar{$crs}{description} - $enrollvar{$crs}{coursecode} as a result of the automated classlist update:\n\n".$response;
+                                unless ($newusermsg eq '') 
+                                {
+                                   $message .= "\n".$newusermsg;
+                                }
                                 my @to_notify = ();
                                 if ($enrollvar{$crs}{notifylist} =~ m/,/) {
                                     @to_notify = split/,/,$enrollvar{$crs}{notifylist};
@@ -118,6 +122,10 @@
                                     my ($ccname,$ccdom) = split/@/,$cc;
                                     my $status =  &Apache::lonmsg::user_normal_msg($ccname,$ccdom,$subject,$message);
                                 }
+                                if ( ($enrollvar{$crs}{notifylist} eq '') && ($newusermsg ne '') ) {
+                                     my $subject = "New user accounts in  $enrollvar{$crs}{'coursecode'}";
+                                     my $status =  &Apache::lonmsg::user_normal_msg($ENV{'user.name'},$dom,$subject,$newusermsg);
+                                }
                                 delete($ENV{'user.name'});
                                 delete($ENV{'user.home'});
                             }
@@ -126,9 +134,11 @@
                 }
             }
         } else {
-            print STDERR "No institutional classlist data could be retrieved for $crs\n";
+            print $fh "No institutional classlist data could be retrieved for $crs\n";
         }
     }
+    print $fh "-- ".localtime(time)." Enrollment messages end\n*******************\n\n";
+    close($fh);
     delete($ENV{'user.domain'});
 
 # Check for photos