--- loncom/interface/lonrequestcourse.pm 2009/08/11 01:39:34 1.8
+++ loncom/interface/lonrequestcourse.pm 2009/08/17 04:39:15 1.15
@@ -1,7 +1,7 @@
# The LearningOnline Network
# Request a course
#
-# $Id: lonrequestcourse.pm,v 1.8 2009/08/11 01:39:34 raeburn Exp $
+# $Id: lonrequestcourse.pm,v 1.15 2009/08/17 04:39:15 raeburn Exp $
#
# Copyright Michigan State University Board of Trustees
#
@@ -44,6 +44,62 @@ described at http://www.lon-capa.org.
=item handler()
+=item header()
+
+=item form_elements()
+
+=item onload_action()
+
+=item check_can_request()
+
+=item course_types()
+
+=item print_main_menu()
+
+=item request_administration()
+
+=item print_request_form()
+
+=item print_enrollment_menu()
+
+=item inst_section_selector()
+
+=item date_setting_table()
+
+=item print_personnel_menu()
+
+=item print_request_status()
+
+=item print_request_logs()
+
+=item print_review()
+
+=item dates_from_form()
+
+=item courseinfo_form()
+
+=item clone_form()
+
+=item clone_text()
+
+=item coursecode_form()
+
+=item get_course_dom()
+
+=item display_navbuttons()
+
+=item print_request_outcome()
+
+=item get_processtype()
+
+=item check_autolimit()
+
+=item build_batchcreatehash()
+
+=item retrieve_settings()
+
+=item get_request_settings()
+
=back
=cut
@@ -114,13 +170,14 @@ sub handler {
}
my %trail = (
- crstype => 'Course Request Action',
- codepick => 'Category',
- courseinfo => 'Description',
- enrollment => 'Enrollment',
- personnel => 'Personnel',
- review => 'Review',
- process => 'Result',
+ crstype => 'Course Request Action',
+ codepick => 'Category',
+ courseinfo => 'Description',
+ enrollment => 'Enrollment',
+ personnel => 'Personnel',
+ review => 'Review',
+ process => 'Result',
+ pick_request => 'Display Summary',
);
my $page = 0;
@@ -179,7 +236,12 @@ sub handler {
'</div>'.&Apache::loncommon::end_page());
}
} elsif ($action eq 'view') {
- &print_request_status($jscript,$loaditems,$crumb);
+ if ($state eq 'crstype') {
+ &print_main_menu($r,\%can_request,\%states,$dom,$jscript,'',$crumb);
+ } else {
+ &request_administration($r,$action,$state,$page,\%states,$dom,$jscript,
+ $loaditems,$crumb);
+ }
} elsif ($action eq 'log') {
&print_request_logs($jscript,$loaditems,$crumb);
} else {
@@ -214,30 +276,33 @@ sub form_elements {
},
courseinfo => {
cdescr => 'text',
- clonecourse => 'text',
- clonedomain => 'selectbox',
+ clonecrs => 'text',
+ clonedom => 'selectbox',
datemode => 'radio',
dateshift => 'text',
},
enrollment => {
- startaccess_month => 'selectbox',
- startaccess_hour => 'selectbox',
- endaccess_month => 'selectbox',
- endaccess_hour => 'selectbox',
- startaccess_day => 'text',
- startaccess_year => 'text',
- startaccess_minute => 'text',
- startaccess_second => 'text',
- endaccess_day => 'text',
- endaccess_year => 'text',
- endaccess_minute => 'text',
- endaccess_second => 'text',
+ accessstart_month => 'selectbox',
+ accessstart_hour => 'selectbox',
+ accessend_month => 'selectbox',
+ accessend_hour => 'selectbox',
+ accessstart_day => 'text',
+ accessstart_year => 'text',
+ accessstart_minute => 'text',
+ accessstart_second => 'text',
+ accessend_day => 'text',
+ accessend_year => 'text',
+ accessend_minute => 'text',
+ accessend_second => 'text',
no_end_date => 'checkbox',
},
personnel => {
persontotal => 'hidden',
addperson => 'checkbox',
},
+ review => {
+ cnum => 'hidden',
+ },
},
view => {
crstype => {
@@ -246,6 +311,13 @@ sub form_elements {
},
},
);
+ my %servers = &Apache::lonnet::get_servers($dom,'library');
+ my $numlib = keys(%servers);
+ if ($numlib > 1) {
+ $elements{'new'}{'courseinfo'}{'chome'} = 'selectbox';
+ } else {
+ $elements{'new'}{'courseinfo'}{'chome'} = 'hidden';
+ }
my (@codetitles,%cat_titles,%cat_order,@code_order,$lastitem);
&Apache::lonnet::auto_possible_instcodes($dom,\@codetitles,\%cat_titles,
\%cat_order,\@code_order);
@@ -262,18 +334,18 @@ sub form_elements {
if (&Apache::lonnet::auto_run('',$dom)) {
my %extras = (
sectotal => 'hidden',
- startenroll_month => 'selectbox',
- startenroll_hour => 'selectbox',
- endenroll_month => 'selectbox',
- endenroll_hour => 'selectbox',
- startenroll_day => 'text',
- startenroll_year => 'text',
- startenroll_minute => 'text',
- startenroll_second => 'text',
- endenroll_day => 'text',
- endenroll_year => 'text',
- endenroll_minute => 'text',
- endenroll_second => 'text',
+ enrollstart_month => 'selectbox',
+ enrollstart_hour => 'selectbox',
+ enrollend_month => 'selectbox',
+ enrollend_hour => 'selectbox',
+ enrollstart_day => 'text',
+ enrollstart_year => 'text',
+ enrollstart_minute => 'text',
+ enrollstart_second => 'text',
+ enrollend_day => 'text',
+ enrollend_year => 'text',
+ enrollend_minute => 'text',
+ enrollend_second => 'text',
crosslisttotal => 'hidden',
addcrosslist => 'checkbox',
autoadds => 'radio',
@@ -314,16 +386,15 @@ sub form_elements {
$persontotal = 1;
}
for (my $i=0; $i<$persontotal; $i++) {
- $people{'person_'.$i.'_uname'} = 'text',
- $people{'person_'.$i.'_dom'} = 'selectbox',
- $people{'person_'.$i.'_hidedom'} = 'hidden',
- $people{'person_'.$i.'_first'} = 'text',
- $people{'person_'.$i.'_last'} = 'text',
- $people{'person_'.$i.'_email'} = 'text',
- $people{'person_'.$i.'_role'} = 'selectbox',
- $people{'person_'.$i.'_sec'} = 'selectbox',
- $people{'person_'.$i.'_newsec'} = 'text',
- $people{'person_'.$i.'_sections'} = 'hidden',
+ $people{'person_'.$i.'_uname'} = 'text',
+ $people{'person_'.$i.'_dom'} = 'selectbox',
+ $people{'person_'.$i.'_hidedom'} = 'hidden',
+ $people{'person_'.$i.'_firstname'} = 'text',
+ $people{'person_'.$i.'_lastname'} = 'text',
+ $people{'person_'.$i.'_emailaddr'} = 'text',
+ $people{'person_'.$i.'_role'} = 'selectbox',
+ $people{'person_'.$i.'_sec'} = 'selectbox',
+ $people{'person_'.$i.'_newsec'} = 'text',
}
my %personnelhash = (%{$elements{'new'}{'personnel'}},%people);
%{$elements{'new'}{'personnel'}} = %personnelhash;
@@ -481,7 +552,8 @@ END
my $formname = 'requestcrs';
my $nexttext = &mt('Next');
- $r->print('<div><form name="'.$formname.'" method="post" action="/adm/requestcourse">'.
+ $r->print(
+ '<div><form name="'.$formname.'" method="post" action="/adm/requestcourse">'.
&Apache::lonhtmlcommon::start_pick_box().
&Apache::lonhtmlcommon::row_title('Action').'
<input type="hidden" name="showdom" value="'.$dom.'" />
@@ -540,8 +612,11 @@ END
&print_request_form($r,$action,$state,$page,$states,$dom);
} elsif ($action eq 'view') {
$r->print(&header('Manage course requests',$js.$jscript,$loaditems).$crumb);
+ if ($state eq 'pick_request') {
+ $r->print(&print_request_status($dom));
+ }
} elsif ($action eq 'log') {
- $r->print(&coursereq_log('View request log',$jscript,$loaditems).$crumb);
+ $r->print(&coursereq_log('View request log',$jscript,$loaditems).$crumb);
}
$r->print(&Apache::loncommon::end_page());
return;
@@ -554,7 +629,7 @@ sub print_request_form {
$prev = $states->{$action}[$page-1];
$next = $states->{$action}[$page+1];
my %navtxt = &Apache::lonlocal::texthash (
- prev => 'Previous',
+ prev => 'Back',
next => 'Next',
);
$crstype = $env{'form.crstype'};
@@ -614,13 +689,13 @@ sub print_request_form {
$r->print('<div>'.&Apache::lonhtmlcommon::start_pick_box().$codepicker.
&Apache::lonhtmlcommon::end_pick_box().'</div>');
} else {
- $r->print(&courseinfo_form($dom,$formname,$crstype));
+ $r->print(&courseinfo_form($dom,$formname,$crstype,$next));
}
} else {
- $r->print(&courseinfo_form($dom,$formname,$crstype));
+ $r->print(&courseinfo_form($dom,$formname,$crstype,$next));
}
} elsif ($prev eq 'codepick') {
- $r->print(&courseinfo_form($dom,$formname,$crstype));
+ $r->print(&courseinfo_form($dom,$formname,$crstype,$next));
} elsif ($state eq 'enrollment') {
if ($crstype eq 'official') {
&Apache::lonnet::auto_possible_instcodes($dom,\@codetitles,\%cat_titles,
@@ -636,8 +711,13 @@ sub print_request_form {
$r->print(&print_review($formname,$dom,\@codetitles,\%cat_titles,\%cat_order,
\@code_order));
$navtxt{'next'} = &mt('Submit course request');
- } elsif ($state eq '') {
- my $result = &print_request_outcome($dom);
+ } elsif ($state eq 'process') {
+ if ($crstype eq 'official') {
+ &Apache::lonnet::auto_possible_instcodes($dom,\@codetitles,\%cat_titles,
+ \%cat_order,\@code_order);
+ }
+ my $result = &print_request_outcome($dom,\@codetitles,\@code_order);
+ $r->print($result);
}
my @excluded = ('counter');
my %elements = &form_elements($dom);
@@ -663,7 +743,7 @@ sub print_request_form {
push(@excluded,'instcode');
}
$r->print(&Apache::lonhtmlcommon::echo_form_input(\@excluded).'</form>');
- &display_navbuttons($r,$formname,$prev,$navtxt{'prev'},$next,$navtxt{'next'});
+ &display_navbuttons($r,$formname,$prev,$navtxt{'prev'},$next,$navtxt{'next'},$state);
return;
}
@@ -775,13 +855,13 @@ sub inst_section_selector {
}
sub date_setting_table {
- my ($starttime,$endtime,$formname,$suffix,%datetitles) = @_;
+ my ($starttime,$endtime,$formname,$prefix,%datetitles) = @_;
my ($perpetual,$table);
- my $startform = &Apache::lonhtmlcommon::date_setter($formname,'start'.$suffix,
+ my $startform = &Apache::lonhtmlcommon::date_setter($formname,$prefix.'start',
$starttime,'','','',1,'','','',1);
- my $endform = &Apache::lonhtmlcommon::date_setter($formname,'end'.$suffix,
+ my $endform = &Apache::lonhtmlcommon::date_setter($formname,$prefix.'end',
$endtime,'','','',1,'','','',1);
- if ($suffix eq 'access') {
+ if ($prefix eq 'access') {
$perpetual = ' <span class="LC_nobreak"><label>'.
'<input type="checkbox" name="no_end_date" />'.
&mt('No end date').'</label></span>';
@@ -806,7 +886,7 @@ sub print_personnel_menu {
$persontotal ++;
}
my $userlinktxt = &mt('Set User');
- my @items = ('uname','dom','last','first','email','hidedom');
+ my @items = ('uname','dom','lastname','firstname','emailaddr','hidedom');
my $roleoptions;
my @roles = &Apache::lonuserutils::roles_by_context('course');
@@ -885,18 +965,16 @@ sub print_personnel_menu {
$existmult.'>'."\n".$existops.'</select>'.(' ' x3);
}
$sectionselector .= $newtitle.
- '<input type="text" name="person_'.$i.'_newsec" size="15" value="" />'.
- '<input type="hidden" name="person_'.$i.'_sections" value="" />'."\n";
-
+ '<input type="text" name="person_'.$i.'_newsec" size="15" value="" />'."\n";
$output .=
&Apache::lonhtmlcommon::row_title(&mt('Additional Personnel').'<br />'.
'<span class="LC_nobreak">'.$userlink.
'</span>').
'<table><tr><td align="center" valign="top">'.&mt('Username').'<br />'.$uname_form.'</td>'."\n".
'<td align="center" valign="top" colspan="2">'.&mt('Domain').'<br />'.$udom_form.'</td></tr><tr>'."\n".
- '<td align="center" valign="top">'.&mt('First Name').'<br />'.$form_elems{'first'}.'</td>'."\n".
- '<td align="center" valign="top">'.&mt('Last Name').'<br />'.$form_elems{'last'}.'</td>'."\n".
- '<td align="center" valign="top">'.&mt('E-mail').'<br />'.$form_elems{email}.'</td></tr>'."\n".
+ '<td align="center" valign="top">'.&mt('First Name').'<br />'.$form_elems{'firstname'}.'</td>'."\n".
+ '<td align="center" valign="top">'.&mt('Last Name').'<br />'.$form_elems{'lastname'}.'</td>'."\n".
+ '<td align="center" valign="top">'.&mt('E-mail').'<br />'.$form_elems{'emailaddr'}.'</td></tr>'."\n".
'<tr><td align="center" valign="top">'.&mt('Role').'<br />'.$roleselector.'</td>'."\n".
'<td'.$seccolspan.' align="center" valign="top">'.&mt('Section(s)').'<br />'.$sectionselector.'</td>'."\n".
'</tr></table>'.&Apache::lonhtmlcommon::row_closure();
@@ -911,10 +989,119 @@ sub print_personnel_menu {
}
sub print_request_status {
- return;
+ my ($dom) = @_;
+ my %statusinfo = &Apache::lonnet::dump('courserequests',$env{'user.domain'},
+ $env{'user.name'},'^status:'.$dom);
+ my ($output,$formname,%queue_by_date,%typenames);
+ if ($env{'form.crstype'} eq 'any') {
+ %typenames = &Apache::lonlocal::texthash (
+ official => 'Official course',
+ unofficial => 'Unofficial course',
+ community => 'Community',
+ );
+ }
+ foreach my $key (keys(%statusinfo)) {
+ if (($statusinfo{$key} eq 'approval') || ($statusinfo{$key} eq 'pending')) {
+ (undef,my($cdom,$cnum)) = split(':',$key);
+ next if ($cdom ne $dom);
+ my $requestkey = $cdom.'_'.$cnum;
+ if ($requestkey =~ /^($match_domain)_($match_courseid)$/) {
+ my %history = &Apache::lonnet::restore($requestkey,'courserequests',
+ $env{'user.domain'},$env{'user.name'});
+ my $entry;
+ my $timestamp = $history{'reqtime'};
+ my $crstype = $history{'crstype'};
+ my $disposition = $history{'disposition'};
+ next if ((exists($history{'status'})) && ($history{'status'} eq 'created'));
+ next unless (($env{'form.crstype'} eq 'any') ||
+ ($env{'form.crstype'} eq $crstype));
+ next unless (($disposition eq 'approval') ||
+ ($disposition eq 'pending'));
+ if (ref($history{'details'}) eq 'HASH') {
+ $entry = $requestkey.':'.$crstype.':'.
+ &escape($history{'details'}{'cdescr'});
+ if ($crstype eq 'official') {
+ $entry .= ':'.&escape($history{'details'}{'instcode'});
+ }
+ }
+ if ($entry ne '') {
+ if (exists($queue_by_date{$timestamp})) {
+ if (ref($queue_by_date{$timestamp}) eq 'ARRAY') {
+ push(@{$queue_by_date{$timestamp}},$entry);
+ }
+ } else {
+ @{$queue_by_date{$timestamp}} = ($entry);
+ }
+ }
+ }
+ }
+ }
+ $formname = 'requestcrs';
+ my @sortedtimes = sort {$a <=> $b} (keys(%queue_by_date));
+ $output = '<h3>'.&mt('Pending course requests').'</h3><div>'."\n".
+ '<form method="post" name="'.$formname.'" action="/adm/requestcourse" />'."\n".
+ '<input type="hidden" name="action" value="'.$env{'form.action'}.'" />'."\n".
+ '<input type="hidden" name="state" value="'.$env{'form.state'}.'" />'."\n".
+ '<input type="hidden" name="crstype" value="'.$env{'form.crstype'}.'" />'."\n";
+ if (@sortedtimes > 0) {
+ $output .= &Apache::loncommon::start_data_table().
+ &Apache::loncommon::start_data_table_header_row().
+ '<th>'.&mt('Action').'</th>'.
+ '<th>'.&mt('Course Description').'</th>'.
+ '<th>'.&mt('Domain').'</th>';
+ if ($env{'form.crstype'} eq 'any') {
+ $output .= '<th>'.&mt('Type').'</th>';
+ }
+ if (($env{'form.crstype'} eq 'any') || ($env{'form.crstype'} eq 'official')) {
+ $output .= '<th>'.&mt('Institutional Code').'</th>';
+ }
+ $output .= '<th>'.&mt('Date requested').'</th>'.
+ &Apache::loncommon::end_data_table_header_row();
+ my $count = 0;
+ foreach my $item (@sortedtimes) {
+ my $showtime = &Apache::lonlocal::locallocaltime($item);
+ if (ref($queue_by_date{$item}) eq 'ARRAY') {
+ foreach my $request (sort(@{$queue_by_date{$item}})) {
+ my ($key,$type,$desc,$instcode) = split(':',$request);
+ my ($cdom,$cnum) = split('_',$key);
+ $output .= &Apache::loncommon::start_data_table_row().
+ '<td><input type="button" value="'.&mt('Select').'" onclick="javascript:viewrequest('."'$cdom','$cnum'".')" /></td>'.
+ '<td>'.&unescape($desc).'</td>'.
+ '<td>'.$cdom.'</td>';
+ if ($env{'form.crstype'} eq 'any') {
+ my $typename = $typenames{$type};
+ if ($typename eq '') {
+ $typename = &mt('Unknown type');
+ }
+ $output .= '<td>'.$typename.'</td>';
+ }
+ if (($env{'form.crstype'} eq 'any') ||
+ ($env{'form.crstype'} eq 'official')) {
+ my $showinstcode;
+ if ($type eq 'official') {
+ $showinstcode = &unescape($instcode);
+ } else {
+ $showinstcode = &mt('Not applicable');
+ }
+ $output .= '<td>'.$showinstcode.'</td>';
+ }
+ $output .= '<td>'.$showtime.'</td>'.
+ &Apache::loncommon::end_data_table_row();
+ }
+ }
+ }
+ $output .= &Apache::loncommon::end_data_table();
+ } else {
+ $output .= '<div>'.&mt('You have no matching course requests awaiting approval by a Domain Coordinator or held in a queue pending administrative action at your institution.').'</div>';
+ }
+ $output .= '
+<br /><input type="button" name="prev" value="'.&mt('Back').'" onclick="javascript:backPage(document.'.$formname.",'crstype'".')" />
+</form></div>';
+ return $output;
}
sub print_request_logs {
+ my ($jscript,$loaditems,$crumb) = @_;
return;
}
@@ -966,12 +1153,12 @@ sub print_review {
$section_headers = '<th>'.&mt('Sections').'</th>'.
'<th>'.&mt('Crosslistings').'</th>';
- my ($startenroll,$endenroll) = &dates_from_form('startenroll','endenroll');
+ my ($enrollstart,$enrollend) = &dates_from_form('enrollstart','enrollend');
my @autoroster = (&mt('No'),&mt('Yes'));
$enroll_values = '<td>'.$autoroster[$env{'form.autoadds'}].'</td>'.
'<td>'.$autoroster[$env{'form.autodrops'}].'</td>'.
- '<td>'.&Apache::lonlocal::locallocaltime($startenroll).'</td>'.
- '<td>'.&Apache::lonlocal::locallocaltime($endenroll).'</td>';
+ '<td>'.&Apache::lonlocal::locallocaltime($enrollstart).'</td>'.
+ '<td>'.&Apache::lonlocal::locallocaltime($enrollend).'</td>';
$section_values = '<td><table class="LC_innerpickbox"><tr><th>'.
&mt('Institutional section').'</th>'.
'<th>'.&mt('LON-CAPA section').'</th></tr>';
@@ -1027,46 +1214,48 @@ sub print_review {
my %ctxt = &clone_text();
$inst_headers .= '<th>'.&mt('Clone From').'</th>';
- if (($env{'form.clonecourse'} =~ /^$match_name$/) &&
- ($env{'form.clonedomain'} =~ /^$match_domain$/)) {
- my %coursehash =
- &Apache::lonnet::courseiddump($env{'form.clonedomain'},'.',1,'.','.',
- $env{'form.clonecourse'},undef,undef,'.');
- my $cloneid = $env{'form.clonedomain'}.'_'.$env{'form.clonecourse'};
- if (ref($coursehash{$cloneid}) eq 'HASH') {
- $inst_headers .= '<th>'.$ctxt{'dsh'}.'</th>';
- my $clonedesc = $coursehash{$cloneid}{'description'};
- my $cloneinst = $coursehash{$cloneid}{'inst_code'};
-
- $inst_values .= '<td>'.$clonedesc.' ';
- if ($cloneinst ne '') {
- $inst_values .= &mt('([_1] in [_2])',$cloneinst,$env{'form.clonedomain'});
- } else {
- $inst_values .= &mt('(from [_1])',$env{'form.clonedomain'});
- }
- $inst_values .= '</td><td>';
- if ($env{'form.datemode'} eq 'preserve') {
- $inst_values .= $ctxt{'pcd'};
- } elsif ($env{'form.datemode'} eq 'shift') {
- $inst_values .= &mt('Shift dates by [_1] days',$env{'form.dateshift'});
- } else {
- $inst_values .= $ctxt{'ncd'};
- }
- $inst_values .= '</td>';
- } else {
- $inst_values .= '<td>'.&mt('Unknown').'</td>';
- }
+ if (($env{'form.clonecrs'} =~ /^$match_name$/) &&
+ ($env{'form.clonedom'} =~ /^$match_domain$/)) {
+ my $canclone = &Apache::loncoursequeueadmin::can_clone_course($env{'user.name'},
+ $env{'user.domain'},$env{'form.clonecrs'}, $env{'form.clonedom'});
+ if ($canclone) {
+ my %courseenv = &Apache::lonnet::userenvironment($env{'form.clonedom'},
+ $env{'form.clonecrs'},('description','internal.coursecode'));
+ if (keys(%courseenv) > 0) {
+ $inst_headers .= '<th>'.$ctxt{'dsh'}.'</th>';
+ $inst_values .= '<td>'.$courseenv{'description'}.' ';
+ my $cloneinst = $courseenv{'internal.coursecode'};
+ if ($cloneinst ne '') {
+ $inst_values .= &mt('([_1] in [_2])',$cloneinst,$env{'form.clonedom'});
+ } else {
+ $inst_values .= &mt('(from [_1])',$env{'form.clonedom'});
+ }
+ $inst_values .= '</td><td>';
+ if ($env{'form.datemode'} eq 'preserve') {
+ $inst_values .= $ctxt{'pcd'};
+ } elsif ($env{'form.datemode'} eq 'shift') {
+ $inst_values .= &mt('Shift dates by [_1] days',$env{'form.dateshift'});
+ } else {
+ $inst_values .= $ctxt{'ncd'};
+ }
+ $inst_values .= '</td>';
+ } else {
+ $inst_values .= '<td>'.&mt('Unknown').'</td>';
+ }
+ } else {
+ $inst_values .= '<td>'.&mt('Not permitted'),'</td>';
+ }
} else {
$inst_values .= '<td>'.&mt('None').'</td>';
}
$enroll_headers .= '<th>'.&mt('Access Starts').'</th>'.
'<th>'.&mt('Access Ends').'</th>';
- my ($startaccess,$endaccess) = &dates_from_form('startaccess','endaccess');
- $enroll_values .= '<td>'.&Apache::lonlocal::locallocaltime($startaccess).'</td>';
- if ($endaccess == 0) {
+ my ($accessstart,$accessend) = &dates_from_form('accessstart','accessend');
+ $enroll_values .= '<td>'.&Apache::lonlocal::locallocaltime($accessstart).'</td>';
+ if ($accessend == 0) {
$enroll_values .= '<td>'.&mt('No end date').'</td>';
} else {
- $enroll_values .= '<td>'.&Apache::lonlocal::locallocaltime($endaccess).'</td>';
+ $enroll_values .= '<td>'.&Apache::lonlocal::locallocaltime($accessend).'</td>';
}
my $container = 'Course';
@@ -1082,14 +1271,32 @@ sub print_review {
'<td>'.&mt('None').'</td></tr>';
for (my $i=0; $i<$env{'form.persontotal'}; $i++) {
if ($env{'form.person_'.$i.'_uname'} ne '') {
+ my @allsecs = &Apache::loncommon::get_env_multiple('form.person_'.$i.'_sec');
+ my $newsec = $env{'form.person_'.$i.'_newsec'};
+ $newsec =~ s/^\s+//;
+ $newsec =~s/\s+$//;
+ my @newsecs = split(/[\s,;]+/,$newsec);
+ foreach my $sec (@newsecs) {
+ next if ($sec =~ /\W/);
+ next if ($newsec eq 'none');
+ if ($sec ne '') {
+ unless (grep(/^\Q$sec\E$/,@allsecs)) {
+ push(@allsecs,$sec);
+ }
+ }
+ }
+ my $showsec = 'None';
+ if (@allsecs) {
+ $showsec = join(', ',@allsecs);
+ }
$personnel_values .=
- '<tr><td>'.$env{'form.person_'.$i.'_first'}.' '.
- $env{'form.person_'.$i.'_last'}.'</td>'.
+ '<tr><td>'.$env{'form.person_'.$i.'_firstname'}.' '.
+ $env{'form.person_'.$i.'_lastname'}.'</td>'.
'<td>'.$env{'form.person_'.$i.'_uname'}.':'.
$env{'form.person_'.$i.'_dom'}.'</td>'.
'<td>'.&Apache::lonnet::plaintext($env{'form.person_'.$i.'_role'},
$container).'</td>'.
- '<td>'.$env{'form.person_'.$i.'_sections'}.'</td></tr>';
+ '<td>'.$showsec.'</td></tr>';
}
}
my $output = '<p>'.&mt('Review the details of the course request before submission.').'</p>'.
@@ -1132,7 +1339,7 @@ sub dates_from_form {
my ($startname,$endname) = @_;
my $startdate = &Apache::lonhtmlcommon::get_date_from_form($startname);
my $enddate = &Apache::lonhtmlcommon::get_date_from_form($endname);
- if ($endname eq 'endaccess') {
+ if ($endname eq 'accessend') {
if (exists($env{'form.no_end_date'}) ) {
$enddate = 0;
}
@@ -1141,13 +1348,39 @@ sub dates_from_form {
}
sub courseinfo_form {
- my ($dom,$formname,$crstype) = @_;
- my $output = '<div>'.&Apache::lonhtmlcommon::start_pick_box().
- &Apache::lonhtmlcommon::row_title('Course Description').
- '<input type="text" size="40" name="cdescr" />'.
- &Apache::lonhtmlcommon::row_closure(1).
- &Apache::lonhtmlcommon::end_pick_box().'</div>'.
- '<div>'.&clone_form($dom,$formname,$crstype).'</div>'."\n";
+ my ($dom,$formname,$crstype,$next) = @_;
+ my $nodescr = &mt('You must provide a (brief) course description.');
+ my $js_validate = <<"ENDJS";
+<script type="text/javascript">
+// <![CDATA['
+
+function validateForm() {
+ if ((document.$formname.cdescr.value == "") || (document.$formname.cdescr.value == "undefined")) {
+ alert('$nodescr');
+ return;
+ }
+ nextPage(document.$formname,'$next');
+}
+// ]]
+</script>
+
+ENDJS
+
+ my $output .= $js_validate."\n".'<div>'.&Apache::lonhtmlcommon::start_pick_box().
+ &Apache::lonhtmlcommon::row_title('Course Description').
+ '<input type="text" size="40" name="cdescr" />';
+ my ($home_server_pick,$numlib) =
+ &Apache::loncommon::home_server_form_item($dom,'chome',
+ 'default','hide');
+ if ($numlib > 1) {
+ $output .= &Apache::lonhtmlcommon::row_closure().
+ &Apache::lonhtmlcommon::row_title(
+ &mt('Home Server for Course'));
+ }
+ $output .= $home_server_pick.
+ &Apache::lonhtmlcommon::row_closure(1).
+ &Apache::lonhtmlcommon::end_pick_box().'</div>'.
+ '<div>'.&clone_form($dom,$formname,$crstype).'</div>'."\n";
return $output;
}
@@ -1157,14 +1390,14 @@ sub clone_form {
if ($crstype eq 'community') {
$type = 'Community';
}
- my $cloneform = &Apache::loncommon::select_dom_form($dom,'clonedomain').
- &Apache::loncommon::selectcourse_link($formname,'clonecourse','clonedomain','','','',$type);
+ my $cloneform = &Apache::loncommon::select_dom_form($dom,'clonedom').
+ &Apache::loncommon::selectcourse_link($formname,'clonecrs','clonedom','','','',$type);
my %lt = &clone_text();
my $output .=
&Apache::lonhtmlcommon::start_pick_box().
&Apache::lonhtmlcommon::row_title($lt{'cid'}).'<label>'.
- '<input type="text" size="25" name="clonecourse" value="" onfocus="this.blur();'.
- 'opencrsbrowser('."'$formname','clonecourse','clonedomain','','','','$type'".');" />'.
+ '<input type="text" size="25" name="clonecrs" value="" onfocus="this.blur();'.
+ 'opencrsbrowser('."'$formname','clonecrs','clonedom','','','','$type'".');" />'.
'</label>'.&Apache::lonhtmlcommon::row_closure(1).'<label>'.
&Apache::lonhtmlcommon::row_title($lt{'dmn'}).'</label>'.
$cloneform.'</label>'.&Apache::lonhtmlcommon::row_closure().
@@ -1296,7 +1529,7 @@ sub get_course_dom {
}
sub display_navbuttons {
- my ($r,$formname,$prev,$prevtext,$next,$nexttext) = @_;
+ my ($r,$formname,$prev,$prevtext,$next,$nexttext,$state) = @_;
$r->print('<div class="LC_navbuttons">');
if ($prev) {
$r->print('
@@ -1309,7 +1542,10 @@ sub display_navbuttons {
onclick="javascript:history.back()"/>
');
}
- if ($next) {
+ if ($state eq 'courseinfo') {
+ $r->print('
+ <input type="button" name="next" value="'.$nexttext.'" onclick="javascript:validateForm();" />');
+ } elsif ($next) {
$r->print('
<input type="button" name="next" value="'.$nexttext.'"
onclick="javascript:nextPage(document.'.$formname.','."'".$next."'".')" />');
@@ -1318,52 +1554,168 @@ sub display_navbuttons {
}
sub print_request_outcome {
- my ($dom) = @_;
- my $output;
- my $cnum = $env{'form.cnum'};
+ my ($dom,$codetitles,$code_order) = @_;
+ my ($output,$cnum,$now,$req_notifylist,$crstype,$enrollstart,$enrollend,
+ %sections,%crosslistings,%personnel,@baduname,@missingdom,%domconfig,);
+ $cnum = $env{'form.cnum'};
unless ($cnum =~ /^$match_courseid$/) {
$output = &mt('Invalid LON-CAPA course number for the new course')."\n";
return $output;
}
- my $req_notifylist = &Apache::lonnet::get_dom('configuration',['requestcourses'],
- $dom);
- my $now = time;
- my $crstype = $env{'form.crstype'};
- my ($startenroll,$endenroll,%sections,%crosslistings,%personnel);
+
+ %domconfig = &Apache::lonnet::get_dom('configuration',['requestcourses'],$dom);
+ if (ref($domconfig{'requestcourses'}) eq 'HASH') {
+ if (ref($domconfig{'requestcourses'}{'notify'}) eq 'HASH') {
+ $req_notifylist = $domconfig{'requestcourses'}{'notify'}{'approval'};
+ }
+ }
+ $now = time;
+ $crstype = $env{'form.crstype'};
if ($crstype eq 'official') {
if (&Apache::lonnet::auto_run('',$dom)) {
- ($startenroll,$endenroll)=&dates_from_form('startenroll','endenroll');
+ ($enrollstart,$enrollend)=&dates_from_form('enrollstart','enrollend');
+ }
+ for (my $i=0; $i<$env{'form.sectotal'}; $i++) {
+ if ($env{'form.sec_'.$i}) {
+ if ($env{'form.secnum_'.$i} ne '') {
+ $sections{$i}{'inst'} = $env{'form.secnum_'.$i};
+ $sections{$i}{'loncapa'} = $env{'form.loncapasec_'.$i};
+ }
+ }
+ }
+ for (my $i=0; $i<$env{'form.crosslisttotal'}; $i++) {
+ if ($env{'form.crosslist_'.$i}) {
+ my $xlistinfo = '';
+ if (ref($code_order) eq 'ARRAY') {
+ if (@{$code_order} > 0) {
+ foreach my $item (@{$code_order}) {
+ $xlistinfo .= $env{'form.crosslist_'.$i.'_'.$item};
+ }
+ }
+ }
+ $xlistinfo .= $env{'form.crosslist_'.$i.'_instsec'};
+ $crosslistings{$i}{'inst'} = $xlistinfo;
+ $crosslistings{$i}{'loncapa'} = $env{'form.crosslist_'.$i.'_lcsec'};
+ }
+ }
+ } else {
+ $enrollstart = '';
+ $enrollend = '';
+ }
+ for (my $i=0; $i<$env{'form.persontotal'}; $i++) {
+ my $uname = $env{'form.person_'.$i.'_uname'};
+ my $udom = $env{'form.person_'.$i.'_uname'};
+ if (($uname =~ /^$match_username$/) && ($udom =~ /^$match_domain$/)) {
+ if (&Apache::lonnet::domain($udom) ne '') {
+ unless (ref($personnel{$uname.':'.$udom}) eq 'HASH') {
+ $personnel{$uname.':'.$udom} = {
+ firstname => $env{'form.person_'.$i.'_firstname'},
+ lastname => $env{'form.person_'.$i.'_lastname'},
+ emailaddr => $env{'form.person_'.$i.'_emailaddr'},
+ };
+ }
+ my $role = $env{'form.person_'.$i.'_role'};
+ unless ($role eq '') {
+ if (ref(@{$personnel{$uname.':'.$udom}{'roles'}}) eq 'ARRAY') {
+ my @curr_roles = @{$personnel{$uname.':'.$udom}{'roles'}};
+ unless (grep(/^\Q$role\E$/,@curr_roles)) {
+ push(@{$personnel{$uname.':'.$udom}{'roles'}},$role);
+ }
+ } else {
+ @{$personnel{$uname.':'.$udom}{'roles'}} = ($role);
+ }
+ if ($role eq 'cc') {
+ @{$personnel{$uname.':'.$udom}{$role}{'usec'}} = ();
+ } else {
+ my @currsec = &Apache::loncommon::get_env_multiple('form.person_'.$i.'_sec');
+ my $newsec = $env{'form.person_'.$i.'_newsec'};
+ $newsec =~ s/^\s+//;
+ $newsec =~s/\s+$//;
+ my @newsecs = split(/[\s,;]+/,$newsec);
+ foreach my $sec (@newsecs) {
+ next if ($sec =~ /\W/);
+ next if ($newsec eq 'none');
+ if ($sec ne '') {
+ unless (grep(/^\Q$sec\E$/,@currsec)) {
+ push(@currsec,$sec);
+ }
+ }
+ }
+ @{$personnel{$uname.':'.$udom}{$role}{'usec'}} = @currsec;
+ }
+ }
+ } else {
+ push(@missingdom,$uname.':'.$udom);
+ }
+ } else {
+ push(@baduname,$uname.':'.$udom);
+ }
+ }
+ my ($accessstart,$accessend) = &dates_from_form('accessstart','accessend');
+ my $autodrops = 0;
+ if ($env{'form.autodrops'}) {
+ $autodrops = $env{'form.autodrops'};
+ }
+ my $autoadds = 0;
+ if ($env{'form.autoadds'}) {
+ $autodrops = $env{'form.autoadds'};
+ }
+ my $sectotal = 0;
+ if ($env{'form.sectotal'}) {
+ $sectotal = $env{'form.sectotal'};
+ }
+ if ($env{'form.autoadds'}) {
+ $autodrops = $env{'form.autoadds'};
+ }
+ my $crosslisttotal = 0;
+ if ($env{'form.crosslisttotal'}) {
+ $crosslisttotal = $env{'form.crosslisttotal'};
+ }
+ my $instcode = '';
+ if (exists($env{'form.instcode'})) {
+ $instcode = $env{'form.instcode'};
+ }
+ my $clonecrs = '';
+ my $clonedom = '';
+ if (($env{'form.clonecrs'} =~ /^($match_courseid)$/) &&
+ ($env{'form.clonedom'} =~ /^($match_domain)$/)) {
+ my $clonehome = &Apache::lonnet::homeserver($env{'form.clonedom'},$env{'form.clonecrs'});
+ if ($clonehome ne 'no_host') {
+ my $canclone = &can_clone_course($env{'user.name'},$env{'user.domain'},
+ $env{'form.clonecrs'}, $env{'form.clonedom'});
+ if ($canclone) {
+ $clonecrs = $env{'form.clonecrs'};
+ $clonedom = $env{'form.clonedom'};
+ }
}
- %sections = ();
- %crosslistings = ();
}
- my ($startaccess,$endaccess) = &dates_from_form('startaccess','endacess');
my $details = {
- owner => $env{'user.name'},
- domain => $env{'user.domain'},
- cdom => $dom,
- cnum => $cnum,
- cdesc => $env{'form.cdesc'},
- crstype => $env{'form.crstype'},
- instcode => $env{'form.instcode'},
- clonedomain => $env{'form.clonedomain'},
- clonecourse => $env{'form.clonecourse'},
- datemode => $env{'form.datemode'},
- dateshift => $env{'form.datshift'},
- sectotal => $env{'form.sectotal'},
- sections => \%sections,
- crosstotal => $env{'form.crosstotal'},
- crosslistings => \%crosslistings,
- autoadds => $env{'form.autoadds'},
- autodrops => $env{'form.autodrops'},
- startenroll => $startenroll,
- endenroll => $endenroll,
- startaccess => $startaccess,
- endaccess => $endaccess,
- personnel => \%personnel
+ owner => $env{'user.name'},
+ domain => $env{'user.domain'},
+ cdom => $dom,
+ cnum => $cnum,
+ coursehome => $env{'form.chome'},
+ cdescr => $env{'form.cdescr'},
+ crstype => $env{'form.crstype'},
+ instcode => $instcode,
+ clonedom => $clonedom,
+ clonecrs => $clonecrs,
+ datemode => $env{'form.datemode'},
+ dateshift => $env{'form.dateshift'},
+ sectotal => $sectotal,
+ sections => \%sections,
+ crosslisttotal => $crosslisttotal,
+ crosslists => \%crosslistings,
+ autoadds => $autoadds,
+ autodrops => $autodrops,
+ enrollstart => $enrollstart,
+ enrollend => $enrollend,
+ accessstart => $accessstart,
+ accessend => $accessend,
+ personnel => \%personnel,
};
my @inststatuses;
- my $val = &get_processtype($dom,$crstype,\@inststatuses);
+ my $val = &get_processtype($dom,$crstype,\@inststatuses,\%domconfig);
if ($val eq '') {
if ($crstype eq 'official') {
$output = &mt('You are not permitted to request creation of official courses');
@@ -1375,15 +1727,19 @@ sub print_request_outcome {
$output = &mt('Unrecognized course type: [_1]',$crstype);
}
} else {
- my ($disposition,$message);
+ my ($disposition,$message,$reqstatus);
my %reqhash = (
- crstype => $crstype,
- details => $details,
+ reqtime => $now,
+ crstype => $crstype,
+ details => $details,
);
my $requestkey = $dom.'_'.$cnum;
- if ($val =~ /^autolimit=/) {
+ if ($val eq 'autolimit=') {
+ $disposition = 'process';
+ } elsif ($val =~ /^autolimit=(\d+)$/) {
+ my $limit = $1;
$disposition = &check_autolimit($env{'user.name'},$env{'user.domain'},
- $dom,$crstype,$val,\$message);
+ $dom,$crstype,$limit,\$message);
} elsif ($val eq 'validate') {
$disposition =
&Apache::lonnet::auto_courserequest_validation($dom,$details,
@@ -1391,30 +1747,39 @@ sub print_request_outcome {
} else {
$disposition = 'approval';
}
- $reqhash{'status'} = $disposition;
+ $reqhash{'disposition'} = $disposition;
+ $reqstatus = $disposition;
if ($disposition eq 'rejected') {
$output = &mt('Your course request was rejected.');
if ($message) {
$output .= '<div class="LC_warning">'.$message.'</div>';
}
} elsif ($disposition eq 'process') {
+ my %domdefs = &Apache::lonnet::get_domain_defaults($dom);
+ my ($logmsg,$newusermsg,$addresult,$enrollcount,$response,$keysmsg,%longroles);
+ my @roles = &Apache::lonuserutils::roles_by_context('course');
my $type = 'Course';
if ($crstype eq 'community') {
$type = 'Community';
}
- my ($logmsg,$newusermsg,$addresult,$enrollcount,$output,$keysmsg,%longroles);
- my @roles = &Apache::lonuserutils::roles_by_context('course');
foreach my $role (@roles) {
$longroles{$role}=&Apache::lonnet::plaintext($role,$type);
}
- my %reqdetails = &build_batchcreatehash($details);
- my $cid = &LONCAPA::batchcreatecourse::build_course($dom,$cnum,'request',\%reqdetails,\%longroles,\$logmsg,\$newusermsg,\$addresult,\$enrollcount,\$output,\$keysmsg,$env{'user.domain'},$env{'user.name'},$cnum);
- $disposition = 'created';
- if ($cid eq $cnum) {
+ my $result = &Apache::loncoursequeueadmin::course_creation($dom,$cnum,
+ 'autocreate',$details,\$logmsg,\$newusermsg,\$addresult,
+ \$enrollcount,\$response,\$keysmsg,\%domdefs,\%longroles);
+ if ($result eq 'created') {
$disposition = 'created';
- $output = &mt('Your course request has been processed and the course has been created.').'<br />'.&mt('You will need to logout and log-in again to be able to select a role in the course.');
+ $reqstatus = 'created';
+ $output = &mt('Your course request has been processed and the course has been created.').
+ '<br />'.
+ &mt('You will need to logout and log-in again to be able to select a role in the course.');
} else {
- $output = &mt('An error occurred when processing your course request.').'<br />'.&mt('You may want to review the request details and submit the request again.');
+ $output = '<span class="LC_error">'.
+ &mt('An error occurred when processing your course request.').
+ '<br />'.
+ &mt('You may want to review the request details and submit the request again.').
+ '</span>';
}
} else {
my $requestid = $cnum.'_'.$disposition;
@@ -1424,7 +1789,7 @@ sub print_request_outcome {
crstype => $crstype,
ownername => $env{'user.name'},
ownerdom => $env{'user.domain'},
- description => $env{'form.cdesc'},
+ description => $env{'form.cdescr'},
},
};
my $putresult = &Apache::lonnet::newput_dom('courserequestqueue',$request,
@@ -1440,16 +1805,16 @@ sub print_request_outcome {
}
$output = &mt('Your course request has been recorded.').'<br />';
if ($disposition eq 'approval') {
- $output .= &mt('Your course request has been recorded.').'<br />'.
- &mt('A message will be sent to your LON-CAPA account when a domain coordinator takes action on your request.').'<br />'.
- &mt('To access your LON-CAPA message, go to the Main Menu and click on "Send and Receive Messages".').'<br />';
+ $output .= &mt('A message will be sent to your LON-CAPA account when a domain coordinator takes action on your request.').'<br />'.
+ &mt('To access your LON-CAPA message, go to the Main Menu and click on "Send and Receive Messages".').'<br />';
if ($address ne '') {
$output.= &mt('An e-mail will also be sent to: [_1] when this occurs.',$address).'<br />';
}
if ($req_notifylist) {
my $fullname = &Apache::loncommon::plainname($env{'user.name'},
$env{'user.domain'});
- &Apache::loncoursequeueadmin::send_selfserve_notification($req_notifylist,$fullname,$now,$dom,$details);
+ my $sender = $env{'user.name'}.':'.$env{'user.domain'};
+ &Apache::loncoursequeueadmin::send_selfserve_notification($req_notifylist,"$fullname ($env{'user.name'}:$env{'user.domain'})",$cnum,$env{'form.cdescr'},$now,'coursereq',$sender);
}
} else {
$output .= '<div class="LC_info">'.
@@ -1459,25 +1824,40 @@ sub print_request_outcome {
'</div>';
}
} else {
- $reqhash{'status'} = 'domainerror';
+ $reqstatus = 'domainerror';
$reqhash{'disposition'} = $disposition;
my $warning = &mt('An error occurred saving your request in the pending requests queue.');
$output = '<span class"LC_warning">'.$warning.'</span><br />';
}
}
- my $storeresult = &Apache::lonnet::store_coursereq($requestkey,\%reqhash);
+ my ($storeresult,$statusresult);
+ if ($requestkey =~ /^($match_domain)_($match_courseid)$/) {
+ $storeresult = &Apache::lonnet::store_userdata(\%reqhash,$requestkey,
+ 'courserequests');
+ if ($storeresult eq 'ok') {
+ my %status = (
+ 'status:'.$dom.':'.$cnum => $reqstatus,
+ );
+ $statusresult = &Apache::lonnet::put('courserequests',\%status);
+ }
+ } else {
+ $storeresult = 'error: invalid requestkey format';
+ }
if ($storeresult ne 'ok') {
- $output .= '<span class="LC_warning">'.&mt('An error occurred saving a record of the details of your request.').'</span><br />';
- &logthis("Error saving course request - $requestkey for $env{'user.name'}:$env{'user.domain'} - $storeresult");
+ $output .= '<span class="LC_warning">'.&mt('An error occurred saving a record of the details of your request: [_1].',$storeresult).'</span><br />';
+ &Apache::lonnet::logthis("Error saving course request - $requestkey for $env{'user.name'}:$env{'user.domain'} - $storeresult");
+ } elsif ($statusresult ne 'ok') {
+ $output .= '<span class="LC_warning">'.&mt('An error occurred saving a record of the status of your request: [_1].',$statusresult).'</span><br />';
+ &Apache::lonnet::logthis("Error saving course request status for $requestkey (for $env{'user.name'}:$env{'user.domain'}) - $statusresult");
}
}
return $output;
}
sub get_processtype {
- my ($dom,$crstype,$inststatuses) = @_;
- return unless (ref($inststatuses) eq 'ARRAY');
+ my ($dom,$crstype,$inststatuses,$domconfig) = @_;
+ return unless ((ref($inststatuses) eq 'ARRAY') && (ref($domconfig) eq 'HASH'));
my (%userenv,%settings,$val);
my @options = ('autolimit','validate','approve');
if ($dom eq $env{'user.domain'}) {
@@ -1488,10 +1868,9 @@ sub get_processtype {
$val = $userenv{'requestcourses.'.$crstype};
@{$inststatuses} = ('_custom_');
} else {
- my %domconfig = &Apache::lonnet::get_dom('configuration',['requestcourses'],$dom);
my ($task,%alltasks);
- if (ref($domconfig{'requestcourses'}) eq 'HASH') {
- %settings = %{$domconfig{'requestcourses'}};
+ if (ref($domconfig->{'requestcourses'}) eq 'HASH') {
+ %settings = %{$domconfig->{'requestcourses'}};
if (ref($settings{$crstype}) eq 'HASH') {
if (($env{'user.adv'}) && (exists($settings{$crstype}{'_LC_adv'}))) {
$val = $settings{$crstype}{'_LC_adv'};
@@ -1500,7 +1879,7 @@ sub get_processtype {
if ($userenv{'inststatus'} ne '') {
@{$inststatuses} = split(',',$userenv{'inststatus'});
} else {
- @{$inststatuses} = ('other');
+ @{$inststatuses} = ('default');
}
foreach my $status (@{$inststatuses}) {
if (exists($settings{$crstype}{$status})) {
@@ -1518,6 +1897,7 @@ sub get_processtype {
}
}
my $maxlimit = 0;
+
foreach my $key (sort(keys(%alltasks))) {
if ($key =~ /^autolimit=(\d*)$/) {
if ($1 eq '') {
@@ -1558,16 +1938,37 @@ sub get_processtype {
}
sub check_autolimit {
- return;
-}
-
-sub build_batchcreatehash {
- my ($details) = @_;
- my %batchhash;
- if (ref($details) eq 'HASH') {
-
+ my ($uname,$udom,$dom,$crstype,$limit,$message) = @_;
+ my %crsroles = &Apache::lonnet::get_my_roles($env{'user.name'},$env{'user.domain'},
+ 'userroles',['active','future'],['cc'],[$dom]);
+ my ($types,$typename) = &course_types();
+ my %requests = &Apache::lonnet::dumpstore('courserequests',$udom,$uname);
+ my %count;
+ if (ref($types) eq 'ARRAY') {
+ foreach my $type (@{$types}) {
+ $count{$type} = 0;
+ }
}
- return %batchhash;
+ foreach my $key (keys(%requests)) {
+ my ($cdom,$cnum) = split('_',$key);
+ if (exists($crsroles{$cnum.':'.$cdom.':cc'})) {
+ if (ref($requests{$key}) eq 'HASH') {
+ my $type = $requests{$key}{'crstype'};
+ if ($type =~ /^official|unofficial|community$/) {
+ $count{$type} ++;
+ }
+ }
+ }
+ }
+ if ($count{$crstype} < $limit) {
+ return 'process';
+ } else {
+ if (ref($typename) eq 'HASH') {
+ $$message = &mt('Your request has not been processed because you have reached the limit for the number of courses of this type.').'<br />'.&mt("Your $typename->{$crstype} limit is [_1].",$limit);
+ }
+ return 'rejected';
+ }
+ return;
}
sub retrieve_settings {
@@ -1575,16 +1976,16 @@ sub retrieve_settings {
my %reqinfo = &get_request_settings($request_id,$dom);
my %stored;
$stored{'cdescr'} = &unescape($reqinfo{'description'});
- $stored{'startaccess'} = $reqinfo{'startaccess'};
- $stored{'endaccess'} = $reqinfo{'endaccess'};
- if ($stored{'endaccess'} == 0) {
+ $stored{'accessstart'} = $reqinfo{'accessstart'};
+ $stored{'accessend'} = $reqinfo{'accessend'};
+ if ($stored{'accessend'} == 0) {
$stored{'no_end_date'} = 1;
}
- $stored{'startenroll'} = $reqinfo{'startenroll'};
- $stored{'endenroll'} = $reqinfo{'endenroll'};
+ $stored{'enrollstart'} = $reqinfo{'enrollstart'};
+ $stored{'enrollend'} = $reqinfo{'enrollend'};
$stored{'crosslist'} = $reqinfo{'crosslist'};
- $stored{'clonecourse'} = $reqinfo{'clonecourse'};
- $stored{'clonedomain'} = $reqinfo{'clonedomain'};
+ $stored{'clonecrs'} = $reqinfo{'clonecrs'};
+ $stored{'clonedom'} = $reqinfo{'clonedom'};
$stored{'sections'} = $reqinfo{'sections'};
$stored{'personnel'} = $reqinfo{'personnel'};