--- loncom/interface/loncoursequeueadmin.pm 2009/08/11 00:39:45 1.1
+++ loncom/interface/loncoursequeueadmin.pm 2010/02/22 09:47:56 1.20
@@ -1,7 +1,7 @@
# The LearningOnline Network
-# Utilities to administer domain course requests and course self-enroll requests
+# Utilities to administer domain course requests and course self-enroll requests
#
-# $Id: loncoursequeueadmin.pm,v 1.1 2009/08/11 00:39:45 raeburn Exp $
+# $Id: loncoursequeueadmin.pm,v 1.20 2010/02/22 09:47:56 bisitz Exp $
#
# Copyright Michigan State University Board of Trustees
#
@@ -33,7 +33,9 @@ Apache::loncoursequeueadmin.pm
=head1 SYNOPSIS
-Adminitsration utilities used by domain coordinators for queued course creation requests, and by course coordinators for queued self-enrollment requests.
+Utilities used by domain coordinators to administer queued course creation requests,
+and by course coordinators for queued self-enrollment requests, and by general
+users to display their queued self-enrollment requests.
This is part of the LearningOnline Network with CAPA project
described at http://www.lon-capa.org.
@@ -50,6 +52,16 @@ described at http://www.lon-capa.org.
=item get_student_counts()
+=item course_creation()
+
+=item build_batchcreatehash()
+
+=item can_clone_course()
+
+=item get_processtype()
+
+=item queued_selfenrollment()
+
=back
=cut
@@ -57,78 +69,106 @@ described at http://www.lon-capa.org.
package Apache::loncoursequeueadmin;
use strict;
-use Apache::Constants qw(:common :http);
use Apache::lonnet;
use Apache::loncommon;
use Apache::lonmsg;
use Apache::lonlocal;
-use LONCAPA;
+use Apache::lonuserutils;
+use LONCAPA qw(:DEFAULT :match);
sub send_selfserve_notification {
- my ($notifylist,$textstr,$cid,$coursedesc,$timestamp,$context,$sender,
- $approvedlist,$rejectedlist) = @_;
+ my ($notifylist,$textstr,$cid,$contextdesc,$timestamp,$context,$sender,
+ $approvedlist,$rejectedlist,$crstype) = @_;
# FIXME locallocaltime needs to be able to take $sender_lh as an argument
# so this can be localized to the recipients date display format/time zone
$timestamp =&Apache::lonlocal::locallocaltime($timestamp);
my $msgcc;
- my ($rawsubj,@rawmsg,$subject,$message,$namelink);
- $namelink = &Apache::loncommon::aboutmewrapper(
- &Apache::loncommon::plainname($env{'user.name'},$env{'user.domain'}));
+ my ($rawsubj,@rawmsg,$subject,$message,$reviewer,$msgtxt);
if ($context eq 'coursemanagers') {
$rawsubj = 'Self-enrollment requests processed';
push(@rawmsg,{
mt => 'Enrollment requests in the following course: [_1] have been processed.',
- args => ["\n$coursedesc\n"],
+ args => ["\n $contextdesc"],
});
} elsif ($context eq 'domainmanagers') {
- $rawsubj = 'Course request';
+ $rawsubj = 'Course/Community requests reviewed';
push(@rawmsg,{
- mt => 'Your request for creation of the following course: [_1], requested on [_2], has been reviewed by a Domain Coordinator.',
- args => ["\n$coursedesc\n","$timestamp\n"],
-
+ mt => 'Course/Community creation requests in the following domain: [_1] have been reviewed.',
+ args => ["\n $contextdesc"],
});
if (ref($textstr) eq 'ARRAY') {
push(@rawmsg,@{$textstr});
}
} elsif ($context eq 'enroller') {
$rawsubj = 'Enrollment request';
+ if ($crstype eq 'community') {
+ $msgtxt = 'Your request for enrollment in the following community: [_1]requested on [_2]has been reviewed by a Coordinator.'
+ } else {
+ $msgtxt = 'Your request for enrollment in the following course: [_1]requested on [_2]has been reviewed by a Course Coordinator.';
+ }
push(@rawmsg,{
- mt => 'Your request for enrollment in the following course: [_1], requested on [_2], has been reviewed by a Course Coordinator.',
- args => ["\n$coursedesc\n","$timestamp\n"],
+ mt => $msgtxt,
+ args => ["\n ".$contextdesc.",\n",$timestamp.",\n"],
});
if (ref($textstr) eq 'ARRAY') {
push(@rawmsg,@{$textstr});
}
} elsif ($context eq 'courserequestor') {
- $rawsubj = 'Course request';
+ if ($crstype eq 'Community') {
+ $rawsubj = 'Community request';
+ $msgtxt = 'Your request for creation of the following community: [_1]requested on [_2]has been reviewed by a Domain Coordinator.';
+ } else {
+ $rawsubj = 'Course request';
+ $msgtxt = 'Your request for creation of the following course: [_1]requested on [_2]has been reviewed by a Domain Coordinator.';
+ }
push(@rawmsg,{
- mt => 'Your request for creation of the following course: [_1], requested on [_2], has been reviewed by a Domain Coordinator.',
- args => ["\n$coursedesc\n","$timestamp\n"],
+ mt => $msgtxt,
+ args => ["\n".$contextdesc.",\n",$timestamp.",\n"],
});
if (ref($textstr) eq 'ARRAY') {
push(@rawmsg,@{$textstr});
}
} elsif ($context eq 'coursereq') {
+ if ($crstype eq 'community') {
+ $rawsubj = 'Community request to review';
+ $msgtxt = 'Creation of the following community: [_1]was requested by [_2] on [_3].';
+ } else {
+ $rawsubj = 'Course request to review';
+ $msgtxt = 'Creation of the following course: [_1]was requested by [_2] on [_3].';
+ }
push(@rawmsg,{
- mt => 'Creation of the following course: [_1] was requested by [_2] on [_3].',
- args => ["\n$coursedesc\n",$textstr,$timestamp],
+ mt => $msgtxt,
+ args => ["\n $contextdesc\n",$textstr,$timestamp],
},
{
- mt =>'As Domain Coordinator, use: [_1]Main Menu -> Create a new course -> Manage Course Requests[_1] to display a list of pending course requests,[_1] which you can either approve or reject.',
- args => ["\n"],
+ mt =>'[_1]As Domain Coordinator, use: [_2]Main Menu -> Course and community creation -> Approve or reject requests[_3]to display a list of pending requests, which you can either approve or reject.',
+ args => ["\n","\n\n ","\n\n"],
});
} elsif ($context eq 'selfenrollreq') {
$rawsubj = 'Self-enrollment request';
+ if ($crstype eq 'community') {
+ $msgtxt = 'Enrollment in the following community: [_1] was requested by [_2] on [_3].'
+ } else {
+ $msgtxt = 'Enrollment in the following course: [_1] was requested by [_2] on [_3].'
+ }
push(@rawmsg,{
- mt => 'Enrollment in the following course: [_1] was requested by [_2] on [_3].',
- args => ["\n$coursedesc\n",$textstr,$timestamp],
- },
+ mt => $msgtxt,
+ args => ["\n $contextdesc\n",$textstr,$timestamp."\n"],
+ });
+ my $directions;
+ if ($crstype eq 'community') {
+ $directions = 'As Coordinator, use: [_1]Main Menu -> Manage Community Users -> Enrollment Requests[_2]to display a list of pending enrollment requests, which you can either approve or reject.';
+ } else {
+ $directions = 'As Course Coordinator, use: [_1]Main Menu -> Manage Course Users -> Enrollment Requests[_2]to display a list of pending enrollment requests, which you can either approve or reject.';
+ }
+ push(@rawmsg,
{
- mt =>'As Course Coordinator, use: [_1]Main Menu -> Manage Course Users -> Enrollment Requests[_1] to display a list of pending enrollment requests,[_1] which you can either approve or reject.',
- args => ["\n"],
+ mt => $directions,
+ args => [" \n\n","\n"],
});
+
}
my @to_notify = split(/,/,$notifylist);
my $numsent = 0;
@@ -149,7 +189,7 @@ sub send_selfserve_notification {
my ($uname,$udom);
if ($sender =~ /:/) {
($uname,$udom) = split(/:/,$sender);
- } else {
+ } elsif ($context eq 'course') {
$uname = $sender;
my %courseinfo = &Apache::lonnet::coursedescription($cid);
$udom = $courseinfo{'num'};
@@ -179,7 +219,7 @@ sub send_selfserve_notification {
@{$item->{args}})."\n";
}
}
- if ($context eq 'managers') {
+ if ($context eq 'coursemanagers') {
if ($approvedlist) {
$message .= "\n\n".&Apache::lonlocal::mt_user($sender_lh,'Approved enrollments:')."\n".$approvedlist;
}
@@ -202,21 +242,23 @@ sub send_selfserve_notification {
sub display_queued_requests {
my ($context,$dom,$cnum) = @_;
- my ($namespace,$formaction,%requesthash);
+ my ($namespace,$formaction,$nextelement,%requesthash);
if ($context eq 'course') {
$formaction = '/adm/createuser';
$namespace = 'selfenrollrequests';
%requesthash = &Apache::lonnet::dump($namespace,$dom,$cnum);
+ $nextelement = '';
} else {
$formaction = '/adm/createcourse';
$namespace = 'courserequestqueue';
- %requesthash = &Apache::lonnet::dump_dom($namespace,$dom,undef,'_approval');
+ %requesthash = &Apache::lonnet::dump_dom($namespace,$dom,'_approval');
+ $nextelement = '';
}
my ($output,%queue_by_date,%crstypes);
if (keys(%requesthash) > 0) {
- $output = '
';
@@ -243,7 +285,8 @@ sub display_queued_requests {
} else {
$timestamp = $requesthash{$item}{'timestamp'};
if (ref($requesthash{$item}) eq 'HASH') {
- $entry = $item.':'.$requesthash{$item}{'ownername'}.':'.
+ my ($cnum,$disposition) = split('_',$item);
+ $entry = $cnum.':'.$requesthash{$item}{'ownername'}.':'.
$requesthash{$item}{'ownerdom'}.':'.
$requesthash{$item}{'crstype'}.':'.
$requesthash{$item}{'description'};
@@ -280,9 +323,9 @@ sub display_queued_requests {
$puname,$pudom);
} else {
- my ($cnum,$ownername,$ownerdom,$type,$cdesc)=split(/:/,$request);
- $detailslink=''.$cdesc.'';
+ my ($cnum,$ownername,$ownerdom,$type,$cdesc)=split(/:/,$request,5);
+ $detailslink=''.$cdesc.'';
$crstype = $type;
if (defined($crstypes{$type})) {
$crstype = $crstypes{$type};
@@ -320,7 +363,7 @@ sub display_queued_requests {
if ($context eq 'course') {
$output .= &mt('There are currently no enrollment requests.');
} else {
- $output .= &mt('There are currently no course requests awaiting approval.');
+ $output .= &mt('There are currently no course or community requests awaiting approval.');
}
}
return $output;
@@ -329,11 +372,12 @@ sub display_queued_requests {
sub update_request_queue {
my ($context,$cdom,$cnum,$coursedesc) = @_;
my ($output,$access_start,$access_end,$limit,$cap,$notifylist,$namespace,
- $stucounts,$idx,$classlist,%requesthash,$coursedesc,$cid,$hostname,
- $protocol,$now,$sender,$approvedmsg,$rejectedmsg,$beneficiary,
- @existing,@missingreq,@invalidusers,@limitexceeded,@enrolled,@created,
- @enrollerrors,@create_errors,@warn_approves,@warn_rejects,@approvals,
- @rejections);
+ $stucounts,$idx,$classlist,%requesthash,$cid,$hostname,$protocol,
+ $domdesc,$now,$sender,$approvedmsg,$rejectedmsg,$beneficiary,
+ @existing,@missingreq,@invalidusers,@limitexceeded,@completed,
+ @processing_errors,@warn_approves,@warn_rejects,@approvals,@warn_dels,
+ @rejections,@rejectionerrors,@nopermissions,%courseroles,
+ %communityroles,%domdefs,%approvalmsg,%rejectionmsg,$crstype);
@approvals = &Apache::loncommon::get_env_multiple('form.approvereq');
@rejections = &Apache::loncommon::get_env_multiple('form.rejectreq');
$now = time;
@@ -342,6 +386,7 @@ sub update_request_queue {
$namespace = 'selfenrollrequests';
$beneficiary = 'enroller';
$cid = $env{'request.course.id'};
+ $crstype = lc(&Apache::loncommon::course_type());
my $chome = &Apache::lonnet::homeserver($cnum,$cdom);
$hostname = &Apache::lonnet::hostname($chome);
$protocol = $Apache::lonnet::protocol{$chome};
@@ -352,7 +397,6 @@ sub update_request_queue {
$limit = $env{'course.'.$cid.'.internal.selfenroll_limit'};
$cap = $env{'course.'.$cid.'.internal.selfenroll_cap'};
$notifylist = $env{'course.'.$cid.'.internal.selfenroll_notifylist'};
- $namespace = 'selfenrollrequests';
($stucounts,$idx,$classlist) = &get_student_counts($cdom,$cnum);
$approvedmsg = [{
mt => 'Your request for enrollment has been approved.',
@@ -365,7 +409,56 @@ sub update_request_queue {
mt => 'Your request for enrollment has not been approved.',
}];
} else {
- $beneficiary = 'requestor';
+ $domdesc = &Apache::lonnet::domain($cdom);
+ $namespace = 'courserequestqueue';
+ $beneficiary = 'courserequestor';
+ %requesthash = &Apache::lonnet::dump_dom($namespace,$cdom,'_approval');
+ my $chome = &Apache::lonnet::domain($cdom,'primary');
+ $hostname = &Apache::lonnet::hostname($chome);
+ $protocol = $Apache::lonnet::protocol{$chome};
+ $protocol = 'http' if ($protocol ne 'https');
+ my %domconfig = &Apache::lonnet::get_dom('configuration',['requestcourses'],$cdom);
+ if (ref($domconfig{'requestcourses'}) eq 'HASH') {
+ if (ref($domconfig{'requestcourses'}{'notify'}) eq 'HASH') {
+ $notifylist = $domconfig{'requestcourses'}{'notify'}{'approval'};
+ }
+ }
+ $approvalmsg{'course'} =
+ [{
+ mt => 'Your course request has been approved.',
+ },
+ {
+ mt => 'Visit [_1], to log-in and access the course',
+ args => [$protocol.'://'.$hostname],
+ }];
+ $rejectionmsg{'course'} =
+ [{
+ mt => 'Your course request has not been approved.',
+ }];
+
+ $approvalmsg{'community'} =
+ [{
+ mt => 'Your community request has been approved.',
+ },
+ {
+ mt => 'Visit [_1], to log-in and access the community',
+ args => [$protocol.'://'.$hostname],
+ }];
+
+ $rejectionmsg{'community'} =
+ [{
+ mt => 'Your community request has not been approved.',
+ }];
+
+ %domdefs = &Apache::lonnet::get_domain_defaults($cdom);
+ my @roles = &Apache::lonuserutils::roles_by_context('course');
+ foreach my $role (@roles) {
+ $courseroles{$role}=&Apache::lonnet::plaintext($role,'Course');
+ }
+ foreach my $role (@roles) {
+ $communityroles{$role}=&Apache::lonnet::plaintext($role,'Community');
+ }
+
}
foreach my $item (sort {$a <=> $b} @approvals) {
if ($context eq 'course') {
@@ -373,7 +466,6 @@ sub update_request_queue {
my $uhome = &Apache::lonnet::homeserver($uname,$udom);
if ($uhome ne 'no_host') {
if (exists($requesthash{$uname.':'.$udom})) {
-
if (exists($classlist->{$uname.':'.$udom})) {
if (ref($classlist->{$uname.':'.$udom}) eq 'ARRAY') {
if (($classlist->{$uname.':'.$udom}->[$idx->{'status'}] eq 'Active') ||
@@ -402,46 +494,135 @@ sub update_request_queue {
my $result =
&Apache::lonnet::modify_student_enrollment($udom,$uname,undef,undef,undef,undef,undef,$usec,$access_end,$access_start,'selfenroll',undef,$cdom.'_'.$cnum,1);
if ($result eq 'ok') {
- push(@enrolled,$uname.':'.$udom);
+ push(@completed,$uname.':'.$udom);
$stucounts->{'allstudents'} ++;
$stucounts->{'selfenrolled'} ++;
&send_selfserve_notification($uname.':'.$udom,$approvedmsg,
- $cid,$coursedesc,$now,$beneficiary,$sender);
- my %userrequest;
- if ($context eq 'course') {
- %userrequest = (
- $cdom.'_'.$cnum => {
- timestamp => $now,
- section => $usec,
- adjudicator => $env{'user.name'}.':'.$env{'user.domain'},
- status => 'approved',
- }
- );
- } else {
- %userrequest = ();
- }
+ $cid,$coursedesc,$now,$beneficiary,$sender,undef,undef,$crstype);
+ my %userrequest = (
+ $cdom.'_'.$cnum => {
+ timestamp => $now,
+ section => $usec,
+ adjudicator => $env{'user.name'}.':'.$env{'user.domain'},
+ status => 'approved',
+ }
+ );
my $userresult =
&Apache::lonnet::put($namespace,\%userrequest,$udom,$uname);
if ($userresult ne 'ok') {
push(@warn_approves,$uname.':'.$udom);
}
} else {
- push(@enrollerrors,$uname.':'.$udom);
+ push(@processing_errors,$uname.':'.$udom);
}
}
} else {
push(@invalidusers,$uname.':'.$udom);
}
} else {
-
+ my ($num,$cnum) = split(':',$item);
+ if (ref($requesthash{$cnum.'_approval'}) eq 'HASH') {
+ if (&Apache::lonnet::homeserver($cnum,$cdom) eq 'no_host') {
+ my $ownername = $requesthash{$cnum.'_approval'}{'ownername'};
+ my $ownerdom = $requesthash{$cnum.'_approval'}{'ownerdom'};
+ $crstype = $requesthash{$cnum.'_approval'}{'crstype'};
+ my $coursedesc = $requesthash{$cnum.'_approval'}{'description'};
+ my $longroles = \%courseroles;
+ if ($crstype eq 'community') {
+ $longroles = \%communityroles;
+ }
+ my $cancreate;
+ if ($cdom eq $ownerdom) {
+ if (&Apache::lonnet::usertools_access($ownername,$ownerdom,$crstype,
+ undef,'requestcourses')) {
+ $cancreate = 1;
+ }
+ } else {
+ my %userenv = &Apache::lonnet::userenvironment($ownerdom,$ownername,'reqcrsotherdom.'.$crstype);
+ if ($userenv{'reqcrsotherdom.'.$crstype}) {
+ my @doms = split(',',$userenv{'reqcrsotherdom.'.$crstype});
+ if (grep(/^\Q$cdom\E:/,@doms)) {
+ $cancreate = 1;
+ }
+ }
+ }
+ if ($cancreate) {
+ my $requestkey = $cdom.'_'.$cnum;
+ my %history =
+ &Apache::lonnet::restore($requestkey,'courserequests',
+ $ownerdom,$ownername);
+ if ((ref($history{'details'}) eq 'HASH') &&
+ ($history{'disposition'} eq 'approval')) {
+ my ($logmsg,$newusermsg,$addresult,$enrollcount,$response,$keysmsg);
+ my $result = &course_creation($cdom,$cnum,$context,$history{'details'},\$logmsg,
+ \$newusermsg,\$addresult,\$enrollcount,
+ \$response,\$keysmsg,\%domdefs,$longroles);
+ if ($result eq 'created') {
+ if ($crstype eq 'community') {
+ $approvedmsg = $approvalmsg{'community'};
+ } else {
+ $approvedmsg = $approvalmsg{'course'};
+ }
+ push(@completed,$cnum);
+
+ unless (&Apache::lonnet::del_dom($namespace,[$cnum.'_approval'],$cdom) eq 'ok') {
+ push(@warn_dels,$cnum);
+ }
+ &send_selfserve_notification($ownername.':'.$ownerdom,$approvedmsg,
+ $cid,$coursedesc,$now,$beneficiary,$sender,undef,undef,$crstype);
+ my %reqhash = (
+ reqtime => $history{'reqtime'},
+ crstype => $history{'crstype'},
+ details => $history{'details'},
+ disposition => $history{'disposition'},
+ status => 'created',
+ adjudicator => $env{'user.name'}.':'.
+ $env{'user.domain'},
+ );
+ my $userresult =
+ &Apache::lonnet::store_userdata(\%reqhash,$requestkey,
+ 'courserequests',$ownerdom,$ownername);
+ if ($userresult eq 'ok') {
+ my %status = (
+ 'status:'.$cdom.':'.$cnum => 'created'
+ );
+ my $statusresult =
+ &Apache::lonnet::put('courserequests',\%status,
+ $ownerdom,$ownername);
+ if ($statusresult ne 'ok') {
+ push(@warn_approves,$cnum);
+ }
+ }
+ if ($userresult ne 'ok') {
+ push(@warn_approves,$cnum);
+ }
+ } else {
+ push(@processing_errors,$cnum);
+ }
+ } else {
+ push(@processing_errors,$cnum);
+ }
+ } else {
+ push(@nopermissions,$cnum);
+ }
+ } else {
+ push(@existing,$cnum);
+ }
+ } else {
+ push(@missingreq,$cnum);
+ }
}
}
- my @changes = (@enrolled,@rejections);
+ my @changes = (@completed,@rejections);
+ if ($context eq 'domain') {
+ @changes = map {$_.'_approval'} (@changes);
+ }
if (@rejections) {
- foreach my $user (@rejections) {
- &send_selfserve_notification($user,$rejectedmsg,$cid,$coursedesc,
- $now,$beneficiary,$sender);
+ foreach my $item (@rejections) {
if ($context eq 'course') {
+ my $user = $item;
+ &send_selfserve_notification($user,$rejectedmsg,$cid,$coursedesc,
+ $now,$beneficiary,$sender,undef,undef,$crstype);
my ($uname,$udom) = split(/:/,$user);
my %userrequest = (
$cdom.'_'.$cnum => {
@@ -456,7 +637,63 @@ sub update_request_queue {
push(@warn_rejects,$user);
}
} else {
-
+ my $cnum = $item;
+ if (ref($requesthash{$cnum.'_approval'}) eq 'HASH') {
+ if (&Apache::lonnet::homeserver($cnum,$cdom) eq 'no_host') {
+ my $requestkey = $cdom.'_'.$cnum;
+ my $ownername = $requesthash{$cnum.'_approval'}{'ownername'};
+ my $ownerdom = $requesthash{$cnum.'_approval'}{'ownerdom'};
+ my $coursedesc = $requesthash{$cnum.'_approval'}{'description'};
+ $crstype = $requesthash{$cnum.'_approval'}{'crstype'};
+ if ($crstype eq 'community') {
+ $rejectedmsg = $rejectionmsg{'community'};
+ } else {
+ $rejectedmsg = $rejectionmsg{'course'};
+ }
+ &send_selfserve_notification($ownername.':'.$ownerdom,$rejectedmsg,
+ $cid,$coursedesc,$now,$beneficiary,
+ $sender,undef,undef,$crstype);
+ my %history =
+ &Apache::lonnet::restore($requestkey,'courserequests',
+ $ownerdom,$ownername);
+ if ((ref($history{'details'}) eq 'HASH') &&
+ ($history{'disposition'} eq 'approval')) {
+ my %reqhash = (
+ reqtime => $history{'reqtime'},
+ crstype => $history{'crstype'},
+ details => $history{'details'},
+ disposition => $history{'disposition'},
+ status => 'rejected',
+ adjudicator => $env{'user.name'}.':'.$env{'user.domain'},
+ );
+ my $userresult =
+ &Apache::lonnet::store_userdata(\%reqhash,$requestkey,
+ 'courserequests',$ownerdom,$ownername);
+ if ($userresult eq 'ok') {
+ my %status = (
+ 'status:'.$cdom.':'.$cnum => 'rejected'
+ );
+ my $statusresult =
+ &Apache::lonnet::put('courserequests',\%status,
+ $ownerdom,$ownername);
+ if ($statusresult ne 'ok') {
+ push(@warn_rejects,$cnum);
+ }
+ } else {
+ push(@warn_rejects,$cnum);
+ }
+ unless (&Apache::lonnet::del_dom($namespace,[$cnum.'_approval'],$cdom) eq 'ok') {
+ push(@warn_dels,$cnum);
+ }
+ } else {
+ push(@warn_rejects,$cnum);
+ }
+ } else {
+ push(@existing,$cnum);
+ }
+ } else {
+ push(@rejectionerrors,$cnum);
+ }
}
}
}
@@ -473,10 +710,14 @@ sub update_request_queue {
my ($chgmsg,$approvedlist,$rejectedlist);
if ($context eq 'course') {
$chgmsg = "'Action was taken on the following enrollment requests by [_1].',$namelink";
- if (@enrolled) {
- $approvedlist = join("\n",@enrolled);
- $output .= '
'.&mt('The following were enrolled in the course:').'
';
- foreach my $user (@enrolled) {
+ if (@completed) {
+ $approvedlist = join("\n",@completed);
+ if ($crstype eq 'community') {
+ $output .= '
'.&mt('The following were enrolled in the community:').'
';
+ } else {
+ $output .= '
'.&mt('The following were enrolled in the course:').'
';
+ }
+ foreach my $user (@completed) {
my ($uname,$udom) = split(/:/,$user);
my $userlink =
&Apache::loncommon::aboutmewrapper(&Apache::loncommon::plainname($uname,$udom),$uname,$udom);
@@ -492,13 +733,49 @@ sub update_request_queue {
}
$output .= '
';
}
+ if ($notifylist ne '') {
+ &send_selfserve_notification($notifylist,$chgmsg,$cid,$coursedesc,
+ $now,'coursemanagers',$sender,
+ $approvedlist,$rejectedlist,$crstype);
+ }
} else {
-
-
+ $chgmsg = "'Action was taken on the following course and community requests by [_1].',$namelink";
+ if (@completed) {
+ $approvedlist = join("\n",@completed);
+ $output .= '
'.&mt('The following courses/communities were created:').'
';
+ foreach my $cnum (@completed) {
+ my $showcourse;
+ if (ref($requesthash{$cnum.'_approval'})) {
+ $showcourse = $requesthash{$cnum.'_approval'}{'description'};
+ } else {
+ $showcourse = $cnum;
+ }
+ my $syllabuslink =
+ &Apache::loncommon::syllabuswrapper($showcourse,$cnum,$cdom);
+ $output .= '
'.&mt('The following course/community creation requests could not be processed because the owner does not have rights to create this type of course:').'