--- loncom/enrollment/Enrollment.pm 2003/12/05 17:04:37 1.1
+++ loncom/enrollment/Enrollment.pm 2003/12/11 03:26:15 1.8
@@ -1,15 +1,43 @@
+# Automated Enrollment manager
+# $Id: Enrollment.pm,v 1.8 2003/12/11 03:26:15 raeburn Exp $
+#
+# Copyright Michigan State University Board of Trustees
+#
+# This file is part of the LearningOnline Network with CAPA (LON-CAPA).
+#
+# LON-CAPA is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 2 of the License, or
+# (at your option) any later version.
+#
+# LON-CAPA is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with LON-CAPA; if not, write to the Free Software
+# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
+#
+# /home/httpd/html/adm/gpl.txt
+#
+# http://www.lon-capa.org/
+#
package LONCAPA::Enrollment;
use Apache::loncoursedata;
use Apache::lonnet;
+use Apache::lonmsg;
use HTML::Entities;
-use XML::Simple;
use LONCAPA::Configuration;
+use Time::Local;
+use lib '/home/httpd/lib/perl';
+use localenroll;
use strict;
sub update_LC {
- my ($dom,$crs,$adds,$drops,$startdate,$enddate,$authtype,$autharg,$classesref,$groupref,$logmsg,$context) = @_;
+ my ($dom,$crs,$adds,$drops,$startdate,$enddate,$authtype,$autharg,$classesref,$groupref,$logmsg,$newusermsg,$context) = @_;
# Get current LON-CAPA student enrollment for this class
my $configvars = &LONCAPA::Configuration::read_conf('loncapa.conf');
my $cid = $dom."_".$crs;
@@ -41,6 +69,8 @@ sub update_LC {
my $enrollcount = 0;
my $dropcount = 0;
+ srand( time() ^ ($$ + ($$ << 15)) ); # Seed rand in case initial passwords have to be generated for new users.
+
# Get mapping of IDs to usernames for current LON-CAPA student enrollment for this class
my @LCids = ();
my %unameFromLCid = ();
@@ -109,12 +139,12 @@ sub update_LC {
push @okusers, $uname;
}
elsif (@sections > 1) {
- $logmsg = "$uname appears in classlists for multiple sections of $crs -";
+ $$logmsg = "$uname appears in classlists for the more than one section of this course, i.e. in sections: ";
foreach (@sections) {
- $logmsg .= " $_,";
+ $$logmsg .= " $_,";
}
- chop($logmsg);
- $logmsg .= " No automated enrollment action taken for this student.\n";
+ chop($$logmsg);
+ $$logmsg .= ". Because of this ambiguity, no enrollment action was taken for this student.".$linefeed;
}
} else {
@{$enrollinfo{$uname}} = @{$allenrolled{$uname}[0]};
@@ -123,6 +153,7 @@ sub update_LC {
}
# Get mapping of student IDs to usernames for users in institutional data for this class
my @allINids = ();
+ my %unameFromINid = ();
foreach my $uname (@okusers) {
$enrollinfo{$uname}[ $place{'studentID'} ] =~ tr/A-Z/a-z/;
my $stuID = $enrollinfo{$uname}[ $place{'studentID'} ];
@@ -133,198 +164,218 @@ sub update_LC {
@{$unameFromINid{$stuID}} = $uname;
}
}
+# Explicitly allow access to creation/modification of students if called as an automated process.
+ if ($context eq 'automated') {
+ $ENV{'allowed.cst'}='F';
+ }
+
# Compare IDs with existing LON-CAPA enrollment for this class
foreach my $uname (@okusers) {
- my %uidhash=&Apache::lonnet::idrget($dom,$uname);
- my @stuinfo = @{$enrollinfo{$uname}};
- if (grep/^$uname$/,@localstudents) {
+ unless ($uname eq '') {
+ my %uidhash=&Apache::lonnet::idrget($dom,$uname);
+ my @stuinfo = @{$enrollinfo{$uname}};
+ if (grep/^$uname$/,@localstudents) {
# Check for studentID changes
- if ( ($uidhash{$uname}) && ($uidhash{$uname} !~ /error\:/) ) {
- unless ( ($uidhash{$uname}) eq ($stuinfo[ $place{studentID} ]) ) {
- $logmsg .= "Change in ID for $uname in class: $crs. StudentID in LON-CAPA system is $uidhash{$uname}, StudentID in institutional data is $stuinfo[ $place{studentID} ]\n";
+ if ( ($uidhash{$uname}) && ($uidhash{$uname} !~ /error\:/) ) {
+ unless ( ($uidhash{$uname}) eq ($stuinfo[ $place{studentID} ]) ) {
+ $$logmsg .= "Change in ID for $uname. StudentID in LON-CAPA system is $uidhash{$uname}; StudentID in institutional data is $stuinfo[ $place{studentID} ]".$linefeed;
+ }
}
- }
# Check for section changes
- unless ($$currlist{$uname}[$sec] eq $stuinfo[ $place{groupID} ]) {
- $logmsg .= "Found a section difference for $uname - ".$$currlist{$uname}[$sec] ."versus ".$stuinfo[ $place{groupID} ]." in class $crs\n";
- if ($$currlist{$uname}[$type] eq "auto") {
- my $modify_section_result = &Apache::lonnet::modifystudent($dom,$uname,'','','',undef,undef,undef,undef,$$currlist{$uname}[$sec],$nowtime,undef,undef,undef,undef,'auto',$cid);
- if ($modify_section_result !~ /^ok/) {
- $logmsg .= "An error occured during the attempt to expire the $uname from the old section $$currlist{$uname}[$sec] - $modify_section_result\n";
- }
-
-# Assign the role of student in the new section
- my $uurl='/'.$cid;
- $uurl=~s/\_/\//g;
- if ($stuinfo[ $place{groupID} ]) {
- $uurl.='/'.$stuinfo[ $place{groupID} ];
- }
- my $newend = $stuinfo[ $place{enddate} ];
- my $newstart = $stuinfo[ $place{startdate} ]);
- if ($newend eq '') {
- $end = $enddate;
- }
- if ($newstart eq '') {
- $start = $startdate;
+ unless ($$currlist{$uname}[$sec] eq $stuinfo[ $place{groupID} ]) {
+ if ( ($$currlist{$uname}[$type] eq "auto") && ($adds == 1) ) {
+ my $modify_section_result = &Apache::lonnet::modify_student_enrollment($dom,$uname,undef,undef,undef,undef,undef,$stuinfo[ $place{groupID} ],$$currlist{$uname}[$cend],$$currlist{$uname}[$cstart],'auto',$cid);
+ if ($modify_section_result =~ /^ok/) {
+ $$logmsg .= "Section for $uname switched from old section: ".$$currlist{$uname}[$sec] ." to new section: ".$stuinfo[ $place{groupID} ].".".$linefeed;
+ } else {
+ $$logmsg .= "Error when attempting section change for $uname from old section ".$$currlist{$uname}[$sec]." to new section: ".$stuinfo[ $place{groupID} ]." -error: $modify_section_result".$linefeed;
+ }
}
- &Apache::lonnet::assignrole($dom,$uname,$uurl,"st",$newend,$newstart);
}
- }
- }
- elsif ($uname ne '') {
+ } else {
# Check for changed usernames by checking studentIDs
- if ( ($stuinfo[ $place{studentID} ] ne '') && (grep/^$stuinfo[ $place{studentID} ]$/,@LCids) ) {
- if (grep/^$$currlist{$uname}[ $place{'studentID'} ]$/,@allINids) {
- foreach my $match ( @{ $unameFromLCid{ $stuinfo[ $place{studentID} ] } } ) {
- if (grep/^$match$/,@okusers) {
- $logmsg .= "A possible change in username has been detected for a student enrolled in $crs. The existing LON-CAPA classlist contains user: $uname and student ID: ".$$currlist{$uname}[ $place{studentID} ].". This username has been dropped from the institutional classlist, but the same student ID is used for user: $match who still appears in the institutional classlist. You may need to move the student data files for user: $uname to $match\n";
+ if ( ($stuinfo[ $place{studentID} ] ne '') && (grep/^$stuinfo[ $place{studentID} ]$/,@LCids) ) {
+ if (grep/^$$currlist{$uname}[ $place{'studentID'} ]$/,@allINids) {
+ foreach my $match ( @{ $unameFromLCid{ $stuinfo[ $place{studentID} ] } } ) {
+ if (grep/^$match$/,@okusers) {
+ $$logmsg .= "A possible change in username has been detected for a student enrolled in this course. The existing LON-CAPA classlist contains user: $uname and student ID: ".$$currlist{$uname}[ $place{studentID} ].". This username has been dropped from the institutional classlist, but the same student ID is used for user: $match who still appears in the institutional classlist. You may need to contact your Domain Coordinator to request a move of the student data files for user: $uname to $match".$linefeed;
+ }
}
}
- }
- } elsif ($adds == 1) {
+ } elsif ($adds == 1) {
# Add student to LON-CAPA classlist
- my $auth = $stuinfo[ $place{'authtype'} ];
- my $authparam = $stuinfo[ $place{'autharg'} ];
- my $first = $stuinfo[ $place{'firstname'} ];
- my $middle = $stuinfo[ $place{'middlename'} ];
- my $last = $stuinfo[ $place{'lastname'} ];
- my $gene = $stuinfo[ $place{'generation'} ];
- my $usec = $stuinfo[ $place{'groupID'} ];
- my $end = $stuinfo[ $place{'enddate'} ];
- my $start = $stuinfo[ $place{'startdate'} ];
- my $emailaddr = $stuinfo[ $place{'email'} ];
- my $pid = $stuinfo[ $place{'studentID'} ];
+ my $auth = $stuinfo[ $place{'authtype'} ];
+ my $authparam = $stuinfo[ $place{'autharg'} ];
+ my $first = $stuinfo[ $place{'firstname'} ];
+ my $middle = $stuinfo[ $place{'middlename'} ];
+ my $last = $stuinfo[ $place{'lastname'} ];
+ my $gene = $stuinfo[ $place{'generation'} ];
+ my $usec = $stuinfo[ $place{'groupID'} ];
+ my $end = $stuinfo[ $place{'enddate'} ];
+ my $start = $stuinfo[ $place{'startdate'} ];
+ my $emailaddr = $stuinfo[ $place{'email'} ];
+ my $pid = $stuinfo[ $place{'studentID'} ];
# remove non alphanumeric values from section
- $usec =~ s/\W//g;
+ $usec =~ s/\W//g;
- unless ($emailaddr =~/^[^\@]+\@[^\@]+$/) { $emailaddr =''; }
- my $emailenc = &HTML::Entities::encode($emailaddr);
+ unless ($emailaddr =~/^[^\@]+\@[^\@]+$/) { $emailaddr =''; }
+ my $emailenc = &HTML::Entities::encode($emailaddr);
# Use course defaults where entry is absent
- if ($auth eq '') {
- $auth = $authtype;
- }
- if ($authparam eq '') {
- $authparam = $autharg;
- }
- if ($end eq '') {
- $end = $enddate;
- }
- if ($start eq '') {
- $start = $startdate;
- }
+ if ( ($auth eq '') || (!defined($auth)) ) {
+ $auth = $authtype;
+ }
+ if ( ($authparam eq '') || (!defined($authparam)) ) {
+ $authparam = $autharg;
+ }
+ if ($auth =~ m/^krb/) {
+ $auth .= ":".$authparam;
+ }
+ if ( ($end eq '') || (!defined($end)) ) {
+ $end = $enddate;
+ }
+ if ( ($start eq '') || (!defined($start)) ) {
+ $start = $startdate;
+ }
# Clean up whitespace
- foreach (\$dom,\$uname,\$pid,\$first,\$middle,\$last,\$gene,\$usec) {
- $$_ =~ s/(\s+$|^\s+)//g;
- }
+ foreach (\$dom,\$uname,\$pid,\$first,\$middle,\$last,\$gene,\$usec) {
+ $$_ =~ s/(\s+$|^\s+)//g;
+ }
# Check for existing account in this LON-CAPA domain for this username
- my $uhome=&Apache::lonnet::homeserver($uname,$dom);
- if ($uhome eq 'no_host') { # User does not exist
- $create_passwd = 0;
- if ($passwd eq '') {
+ my $uhome=&Apache::lonnet::homeserver($uname,$dom);
+ if ($uhome eq 'no_host') { # User does not exist
+ my $create_passwd = 0;
+ my $authchk = '';
+ unless ($authparam eq '') { $authchk = 'ok'; };
# If no account exists and passwords should be generated
- if (($authtype eq "int") || ($authtype eq "loc")) {
- ($passwd,$create_passwd) = &create_password($authtype); }
+ if ($authtype eq "int") {
+ if ($authparam eq '') {
+ ($authparam) = &create_password();
+ if ($authparam eq '') {
+ $authchk = '';
+ } else {
+ $create_passwd = 1;
+ }
+ }
+ } elsif ($authtype eq "local") {
+ ($authparam,$create_passwd,$authchk) = &localenroll::create_password($authparam);
} elsif ($authtype =~ m/^krb/) {
- $passwd = $autharg;
+ if ($authparam eq '') {
+ $$logmsg .= "No Kerberos domain was provided for the new user - $uname, so the new student was not enrolled in the course.".$linefeed;
+ $authchk = 'invalid';
+ }
} else {
- $logmsg .= "Invalid authentication type for $uname in $crs\n";
- }
- }
+ $authchk = 'invalid';
+ $$logmsg .= "An invalid authentication type was provided for the new user - $uname, so the student was not enrolled in the course.".$linefeed;
+ }
+ if ($authchk eq 'ok') {
# Now create user.
- my $reply=&Apache::lonnet::modifystudent($dom,$uname,$pid,$auth,$authparam,$first,$middle,$last,$gene,$usec,$end,$start,'',undef,$emailaddr,'auto',$cid);
- if ($reply eq 'ok') {
- $enrollcount ++;
- $addresult .= "$first $last ($pid) - $uname enrolled in section/group $usec.".$linefeed;
- $logmsg .= "New user $uname added successfully. ";
- unless ($emailenc eq '') {
- my %emailHash;
- $emailHash{'critnotification'} = $emailenc;
- $emailHash{'notification'} = $emailenc;
- my $putresult = &Apache::lonnet::put('environment',\%emailHash,$dom,$uname);
- }
- if ($create_passwd) {
-# Send e-mail with inital password to new user at $emailaddr
- $logmsg .= "Initial password - - sent to $emailaddr\n";
- } else {
- $logmsg .= "\n";
+ my $reply=&Apache::lonnet::modifystudent($dom,$uname,$pid,$auth,$authparam,$first,$middle,$last,$gene,$usec,$end,$start,'',undef,$emailaddr,'auto',$cid);
+ if ($reply eq 'ok') {
+ $enrollcount ++;
+ $addresult .= "$first $last ($pid) - $uname enrolled in section/group $usec.".$linefeed;
+ if ($context eq 'automated') {
+ $$logmsg .= "New $dom user $uname added successfully.";
+ }
+ unless ($emailenc eq '') {
+ my %emailHash;
+ $emailHash{'critnotification'} = $emailenc;
+ $emailHash{'notification'} = $emailenc;
+ my $putresult = &Apache::lonnet::put('environment',\%emailHash,$dom,$uname);
+ }
+ if ($create_passwd) {
+# Send e-mail with initial password to new user at $emailaddr.
+# If e-mail address is invalid, send password via message to courseowner i
+# (if automated call) or to user if roster update.
+ if ($emailaddr eq '') {
+ $$newusermsg .= " username: $uname, password: ".$authparam.$linefeed."\n";
+ } else {
+ my $subject = "New LON-CAPA account";
+ my $body = "You have been enrolled in the LON-CAPA system at your school, because you are a registered student in a class that is using the LON-CAPA course management and online homework system.\n\nYou should log-in to the system using the following credentials:\nusername: $uname\npassword: $authparam\n\nThe URL you should use to access the LON-CAPA system at your school is: http://".$ENV{'SERVER_NAME'};
+ &Apache::lonmsg::sendemail($emailaddr,$subject,$body);
+ }
+ if ($context eq 'automated') {
+ $$logmsg .= " Initial password - - sent to ".$emailaddr.$linefeed;
+ }
+ } else {
+ if ($context eq 'automated') {
+ $$logmsg .= $linefeed;
+ }
+ }
+ } else {
+ $$logmsg .= "An error occurred adding new user $uname - ".$reply.$linefeed;
+ }
}
} else {
- $logmsg .= "An error occurred adding new user $uname - $reply\n";
- }
- } else {
# Get the user's information and authentication
- my %userenv = &Apache::lonnet::get('environment',['firstname','middlename','lastname','generation','id','critnotification','notification'],$dom,$uname);
- my ($tmp) = keys(%userenv);
- if ($tmp =~ /^(con_lost|error)/i) {
- %userenv = ();
- }
+ my %userenv = &Apache::lonnet::get('environment',['firstname','middlename','lastname','generation','id','critnotification','notification'],$dom,$uname);
+ my ($tmp) = keys(%userenv);
+ if ($tmp =~ /^(con_lost|error)/i) {
+ %userenv = ();
+ }
# Get the user's e-mail address
- if ($userenv{critnotification} =~ m/%40/) {
- unless ($emailenc eq $userenv{critnotification}) {
- $logmsg .= "Current critical notification e-mail - ".$userenv{critnotification}." for $uname is different to e-mail address in Institutional classlist - $emailenc\n";
- }
- if ($userenv{notification} =~ m/%40/) {
- unless ($emailenc eq $userenv{critnotification}) {
- $logmsg .= "Current standard notification e-mail - ".$userenv{notification}." for $uname is different to e-mail address in Institutional classlist - $emailenc\n";
- }
- }
- my $krbdefdom = '';
- my $currentauth=&Apache::lonnet::queryauthenticate($uname,$dom);
- if ($currentauth=~/^krb(4|5):/) {
- $currentauth=~/^krb(4|5):(.*)/;
- $krbdefdom=$1;
- }
- if ($currentauth=~/^krb(4|5):/ ||
- $currentauth=~/^unix:/ ||
- $currentauth=~/^internal:/ ||
- $currentauth=~/^localauth:/) {
+ if ($userenv{critnotification} =~ m/%40/) {
+ unless ($emailenc eq $userenv{critnotification}) {
+ $$logmsg .= "Current critical notification e-mail - ".$userenv{critnotification}." for $uname is different to e-mail address in Institutional classlist - ".$emailenc.$linefeed;
+ }
+ }
+ if ($userenv{notification} =~ m/%40/) {
+ unless ($emailenc eq $userenv{critnotification}) {
+ $$logmsg .= "Current standard notification e-mail - ".$userenv{notification}." for $uname is different to e-mail address in institutional classlist - ".$emailenc.$linefeed;
+ }
+ }
+ my $krbdefdom = '';
+ my $currentauth=&Apache::lonnet::queryauthenticate($uname,$dom);
+ if ($currentauth=~/^krb(4|5):/) {
+ $currentauth=~/^krb(4|5):(.*)/;
+ $krbdefdom=$1;
+ }
+ if ($currentauth=~/^krb(4|5):/ ||
+ $currentauth=~/^unix:/ ||
+ $currentauth=~/^internal:/ ||
+ $currentauth=~/^localauth:/) {
- } else {
- $logmsg .= "Invalid authentication method $currentauth for $uname.\n";
- }
+ } else {
+ $$logmsg .= "Invalid authentication method $currentauth for $uname.".$linefeed;
+ }
# Report if authentication methods are different.
- if ($currentauth ne $auth ) {
- $logmsg .= "Authentication mismatch for $uname - $currentauth in system, $auth for class $crs\n";
- }
+ if ($currentauth ne $auth ) {
+ $$logmsg .= "Authentication mismatch for $uname - $currentauth in system, $auth based on information in classlist or default for this course.".$linefeed;
+ }
# Check user data
- if ($first ne $userenv{'firstname'} ||
- $middle ne $userenv{'middlename'} ||
- $last ne $userenv{'lastname'} ||
- $gene ne $userenv{'generation'} ||
- $pid ne $userenv{'id'} ) {
+ if ($first ne $userenv{'firstname'} ||
+ $middle ne $userenv{'middlename'} ||
+ $last ne $userenv{'lastname'} ||
+ $gene ne $userenv{'generation'} ||
+ $pid ne $userenv{'id'} ) {
# Make the change(s)
- my %changeHash;
- $changeHash{'firstname'} = $first;
- $changeHash{'middlename'} = $middle;
- $changeHash{'lastname'} = $last;
- $changeHash{'generation'} = $gene;
- $changeHash{'id'} = $pid;
- my $putresult = &Apache::lonnet::put('environment',\%changeHash,$dom,$uname);
- if ($putresult eq 'ok') {
- $logmsg .= "User: $uname enrolled in $crs\n";
-# Assign the role of student
- $classlist_reply = &modify_student_enrollment($$configvars{'lonHostID'},$dom,$cid,$crs,$uname,$pid,$first,$middle,$last,$gene,$usec,$end,$start,'auto');
- if ($classlist_reply eq 'ok') {
- my $uurl='/'.$cid;
- $uurl=~s/\_/\//g;
- if ($usec) {
- $uurl.='/'.$usec;
- }
- &Apache::lonnet::assignrole($dom,$uname,$uurl,"st",$enddate,$startdate);
- $addresult .= "$first $last ($pid) - $uname enrolled in section/group $usec.".$linefeed;
- $enrollcount ++;
- $addresult .= "$first $last ($pid) - $uname enrolled in section/group $usec.".$linefeed;
- $logmsg .= "Existing user $uname enrolled successfully in $crs\n";
-
+ my %changeHash;
+ $changeHash{'firstname'} = $first;
+ $changeHash{'middlename'} = $middle;
+ $changeHash{'lastname'} = $last;
+ $changeHash{'generation'} = $gene;
+ $changeHash{'id'} = $pid;
+ my $putresult = &Apache::lonnet::put('environment',\%changeHash,$dom,$uname);
+ if ($putresult eq 'ok') {
+ $$logmsg .= "User information updated for user: $uname prior to enrollment.".$linefeed;
} else {
- $logmsg .= "There was a problem updating the classlist db file for user $uname to show the new enrollment, so no enrollment occurred in $crs\n";
+ $$logmsg .= "There was a problem modifying user data for existing user - $uname -error: $putresult, enrollment will still be attempted.".$linefeed;
+ }
+ }
+
+# Assign the role of student in the course.
+ my $classlist_reply = &Apache::lonnet::modify_student_enrollment($dom,$uname,$pid,$first,$middle,$last,$gene,$usec,$end,$start,'auto',$cid);
+ if ($classlist_reply eq 'ok') {
+ $enrollcount ++;
+ $addresult .= "$first $last ($pid) - $uname enrolled in section/group $usec.".$linefeed;
+ if ($context eq 'automated') {
+ $$logmsg .= "Existing $dom user $uname enrolled successfully.".$linefeed;
}
} else {
- $logmsg .= "There was a problem modifying user data for existing user - $uname, so no enrollment occurred in $crs.\n";
+ $$logmsg .= "There was a problem updating the classlist db file for user $uname to show the new enrollment -error: $classlist_reply, so no enrollment occurred for this user.".$linefeed;
}
}
}
@@ -340,25 +391,34 @@ sub update_LC {
# Check for changed usernames by checking studentIDs
if (grep/^$$currlist{$uname}[ $stuid ]$/,@allINids) {
foreach my $match (@{$unameFromINid{$$currlist{$uname}[ $stuid ]}} ) {
- $logmsg .= "A possible change in username has been detected for a student enrolled in $crs. The existing LON-CAPA classlist contains user: $uname and student ID: $$currlist{$uname}[ $place{studentID} ]. This username has been dropped from the institutional classlist, but the same student ID is used for user: $match who still appears in the institutional classlist. You may need to move the student data files for user: $uname to $match\n";
+ $$logmsg .= "A possible change in username has been detected for a student enrolled in this course. The existing LON-CAPA classlist contains user: $uname and student ID: $$currlist{$uname}[ $place{studentID} ]. This username has been dropped from the institutional classlist, but the same student ID is used for user: $match who still appears in the institutional classlist. You may need to move the student data files for user: $uname to $match.".$linefeed;
push @saved,$uname;
}
} elsif (@saved == 0) {
my $drop_reply = &Apache::lonnet::modifystudent($dom,$uname,'','','',undef,undef,undef,undef,$$currlist{$uname}[$sec],time,undef,undef,undef,undef,undef,$cid);
if ($drop_reply !~ /^ok/) {
- $logmsg .= "An error occured during the attempt to expire the $uname from the old section $$currlist{$uname}[$sec] - $drop_reply\n";
+ $$logmsg .= "An error occured during the attempt to expire the $uname from the old section $$currlist{$uname}[$sec] - $drop_reply.".$linefeed;
} else {
$dropcount ++;
my %userenv = &Apache::lonnet::get('environment',['firstname','lastname','id'],$dom,$uname);
$dropresult .= $userenv{'firstname'}." ".$userenv{'lastname'}." (".$userenv{'id'}.") - ".$uname." dropped from section/group ".$$currlist{$uname}[$sec].$linefeed;
+ if ($context eq 'automated') {
+ $$logmsg .= "User $uname student role expired from course.".$linefeed;
+ }
}
}
}
}
}
}
+
+# Terminated explictly allowed access to student creation/modification
+ if ($context eq 'automated') {
+ delete($ENV{'allowed.cst'});
+ }
if ($enrollcount > 0) {
if ($context eq "updatenow") {
+ $addresult = substr($addresult,0,rindex($addresult,""));
$addresult = "The following $enrollcount student(s) was/were added to this LON-CAPA course:
";
} else {
$addresult = "The following $enrollcount student(s) was/were added to this LON-CAPA course:\n\n".$addresult."\n\n";
@@ -366,6 +426,7 @@ sub update_LC {
}
if ($dropcount > 0) {
if ($context eq "updatenow") {
+ $dropresult = substr($dropresult,0,rindex($dropresult,""));
$dropresult = "The following $dropcount student(s) was/were expired from this LON-CAPA course:
";
} else {
$dropresult = "The following $dropcount student(s) was/were expired from this LON-CAPA course:\n\n".$dropresult."\n\n";
@@ -387,39 +448,101 @@ sub update_LC {
$dropresult .="\n";
}
}
- print STDERR $logmsg;
- return $addresult.$dropresult;
-}
+ my $changecount = $enrollcount + $dropcount;
+ return ($changecount,$addresult.$dropresult);
+}
sub parse_classlist {
- my ($tmpdir,$dom,$crs,$class,$placeref,$groupID,$studentsref) = @_;
- my $configvars = &LONCAPA::Configuration::read_conf();
- my $xmlfile = $tmpdir."/tmp/".$dom."_".$crs."_classlist.xml";
- my $enrolled = XMLin( $xmlfile, KeyAttr => ['username'] );
- foreach my $uname ( sort keys %{$$enrolled{'student'}} ) {
- @{ $$studentsref{$uname} } = ();
- foreach my $key (sort keys %{$$enrolled{'student'}{$uname}} ) {
- my $value = $$enrolled{'student'}{$uname}{$key};
- if (ref($value)) {
- $$studentsref{$uname}[ $$placeref{$key} ] = '';
- } else {
- if ($key eq 'groupID') {
- $$studentsref{$uname}[ $$placeref{$key} ] = $groupID;
- } else {
- $$studentsref{$uname}[ $$placeref{$key} ] = $value;
- }
- }
- }
- }
-# if (-e "$xmlfile") {
-# unlink $xmlfile;
-# }
- return;
+ my ($tmpdir,$dom,$crs,$class,$placeref,$groupID,$studentsref) = @_;
+ my $xmlfile = $tmpdir."/tmp/".$dom."_".$crs."_".$class."_classlist.xml";
+ my $uname = '';
+ my @state;
+ my @items = ('autharg','authtype','email','firstname','generation','lastname','middlename','studentID');
+ my $p = HTML::Parser->new
+ (
+ xml_mode => 1,
+ start_h =>
+ [sub {
+ my ($tagname, $attr) = @_;
+ push @state, $tagname;
+ if ("@state" eq "students student") {
+ $uname = $attr->{username};
+ }
+ }, "tagname, attr"],
+ text_h =>
+ [sub {
+ my ($text) = @_;
+ if ("@state" eq "students student groupID") {
+ $$studentsref{$uname}[ $$placeref{'groupID'} ] = $groupID;
+ } elsif ("@state" eq "students student startdate") {
+ my $start = $text;
+ unless ($text eq '') {
+ $start = &process_date($text);
+ }
+ $$studentsref{$uname}[ $$placeref{'startdate'} ] = $start;
+ } elsif ("@state" eq "students student enddate") {
+ my $end = $text;
+ unless ($text eq '') {
+ $end = &process_date($text);
+ }
+ $$studentsref{$uname}[ $$placeref{'enddate'} ] = $end;
+ } else {
+ foreach my $item (@items) {
+ if ("@state" eq "students student $item") {
+ $$studentsref{$uname}[ $$placeref{$item} ] = $text;
+ }
+ }
+ }
+ }, "dtext"],
+ end_h =>
+ [sub {
+ my ($tagname) = @_;
+ pop @state;
+ }, "tagname"],
+ );
+
+ $p->parse_file($xmlfile);
+ $p->eof;
+ if (-e "$xmlfile") {
+ unlink $xmlfile;
+ }
+ return;
}
-sub create_password {
-
+sub process_date {
+ my $timestr = shift;
+ my $timestamp = '';
+ if ($timestr =~ m/^\d{4}:\d{2}:\d{2}/) {
+ my @entries = split/:/,$timestr;
+ for (my $j=0; $j<@entries; $j++) {
+ if ( length($entries[$j]) > 1 ) {
+ $entries[$j] =~ s/^0//;
+ }
+ }
+ $entries[1] = $entries[1] - 1;
+ $timestamp = timelocal($entries[5],$entries[4],$entries[3],$entries[2],$entries[1],$entries[0]);
+ }
+ return $timestamp;
+}
+sub create_password {
+ my $passwd = '';
+ my @letts = "a","b","c","d","e","f","g","h","i","j","k","l","m","n","o","p","q","r","s","t","u","v","w","x","y","z";
+ for (my $i=0; $i<8; $i++) {
+ my $lettnum = int (rand 2);
+ my $item = '';
+ if ($lettnum) {
+ $item = $letts[int( rand(26) )];
+ my $uppercase = int(rand 2);
+ if ($uppercase) {
+ $item =~ tr/a-z/A-Z/;
+ }
+ } else {
+ $item = int( rand(10) );
+ }
+ $passwd .= $item;
+ }
+ return ($passwd);
}
sub CL_autharg { return 0; }