--- loncom/interface/loncoursequeueadmin.pm 2009/08/11 00:39:45 1.1 +++ loncom/interface/loncoursequeueadmin.pm 2010/01/15 15:38:10 1.12.2.6 @@ -1,7 +1,7 @@ # The LearningOnline Network # 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.12.2.6 2010/01/15 15:38:10 raeburn 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,14 @@ described at http://www.lon-capa.org. =item get_student_counts() +=item course_creation() + +=item build_batchcreatehash() + +=item can_clone_course() + +=item queued_selfenrollment() + =back =cut @@ -57,78 +67,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 Apache::lonuserutils; use LONCAPA; 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 +187,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 +217,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 +240,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 = '
'. - ''. - ''. + $output = ''."\n". + ''."\n". + $nextelement."\n". &Apache::loncommon::start_data_table(). &Apache::loncommon::start_data_table_header_row(). ''.&mt('Action').''. @@ -224,12 +264,12 @@ sub display_queued_requests { if ($context eq 'course') { $output .= ''.&mt('Section').''. ''.&mt('Date requested').''; + } else { %crstypes = &Apache::lonlocal::texthash ( official => 'Official course', unofficial => 'Unofficial course', community => 'Community', ); - } else { $output .= ''.&mt('Type').''. ''.&mt('Date requested').''. ''.&mt('Details').''; @@ -243,7 +283,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 +321,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 +361,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 +370,13 @@ 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, + @rejections,@rejectionerrors,@nopermissions,%courseroles, + %communityroles,%domdefs,%approvalmsg,%rejectionmsg,$crstype, + @warn_coursereqs); @approvals = &Apache::loncommon::get_env_multiple('form.approvereq'); @rejections = &Apache::loncommon::get_env_multiple('form.rejectreq'); $now = time; @@ -342,6 +385,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 +396,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 +408,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 +465,10 @@ sub update_request_queue { my $uhome = &Apache::lonnet::homeserver($uname,$udom); if ($uhome ne 'no_host') { if (exists($requesthash{$uname.':'.$udom})) { - + if ($cdom eq 'gci' && $cnum eq '9615072b469884921gcil1') { + my $enresult = &enable_gci_submission($udom,$uname, + $access_end,$access_start); + } if (exists($classlist->{$uname.':'.$udom})) { if (ref($classlist->{$uname.':'.$udom}) eq 'ARRAY') { if (($classlist->{$uname.':'.$udom}->[$idx->{'status'}] eq 'Active') || @@ -402,46 +497,140 @@ 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); + } elsif ($udom eq 'gci') { + my %changehash = ( + 'reqcrsotherdom.unofficial' => 'gcitest:autolimit=', + ); + my $reqresult = &Apache::lonnet::put('environment',\%changehash, + $udom,$uname); + if ($reqresult ne 'ok') { + push(@warn_coursereqs,$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); + &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 +645,60 @@ 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); + } + } else { + push(@warn_rejects,$cnum); + } + } else { + push(@existing,$cnum); + } + } else { + push(@rejectionerrors,$cnum); + } } } } @@ -473,10 +715,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:').'

'; + } else { + $output .= '

'.&mt('The following course/community creation requests were ignored because the request is no longer in the queue:').'

'; + } } if (@invalidusers) { @@ -528,8 +828,6 @@ sub update_request_queue { $output .= '
  • '.$user.'
  • '; } $output .= '

    '; - } else { - } } if (@limitexceeded) { @@ -539,22 +837,56 @@ sub update_request_queue { $output .= '
  • '.$user.'
  • '; } $output .= '

    '; - } else { - } } - if (@enrollerrors) { + if (@nopermissions) { + $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:').'

    '; + } + if (@processing_errors) { if ($context eq 'course') { $output .= '

    '.&mt('The following enrollment requests could not be processed because an error occurred:').'

    '; } else { - + $output .= '

    '.&mt('The following course/community creation requests could not be processed because an error occurred:').'

    '; } } - if (@warn_approves) { + if (@rejectionerrors) { + $output .= '

    '.&mt('The following course/community creation request rejections could not be fully processed because an error occurred:').'

    '; + } + if (@warn_approves || @warn_rejects) { if ($context eq 'course') { $output .= '

    '.&mt("For the following users, an error occurred when updating the user's own self-enroll requests record:").'

    '; } else { - + $output .= '

    '.&mt("For the following course/community requests an error occurred when updating the requestor's own requests record:").'

    '; } } - if (@warn_rejects) { - if ($context eq 'course') { - $output .= '

    '.&mt("For the following users, an error occurred when updating the user's own self-enroll requests record:").'