--- loncom/interface/lonmodifycourse.pm 2003/12/26 16:30:17 1.1
+++ loncom/interface/lonmodifycourse.pm 2010/03/29 17:32:28 1.52
@@ -1,504 +1,828 @@
+# The LearningOnline Network with CAPA
+# handler for DC-only modifiable course settings
+#
+# $Id: lonmodifycourse.pm,v 1.52 2010/03/29 17:32:28 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 Apache::lonmodifycourse;
use strict;
use Apache::Constants qw(:common :http);
use Apache::lonnet;
use Apache::loncommon;
+use Apache::lonhtmlcommon;
use Apache::lonlocal;
-use Apache::londropadd;
-use LONCAPA::Enrollment;
+use Apache::lonuserutils;
+use Apache::lonpickcourse;
use lib '/home/httpd/lib/perl';
-use localenroll;
+use LONCAPA;
-sub print_course_selection_page {
- my ($r,$tasklongref) = @_;
- my $dom = $ENV{'user.domain'};
- my %lt=&Apache::lonlocal::texthash(
- 'incs' => "Internal course settings",
- 'stat' => "Although the majority of course settings for LON-CAPA courses may be modified by a Course Coordinator, there are a small number of settings which may only be modified by a Domain Coordinator. These particular course parameters control automated student enrollment in LON-CAPA courses based on classlist data available from your institution's student information system.",
- 'chcs' => "Once you have chosen the course for which you wish to view/modify internal settings, click 'Go' to proceed",
- 'ccrs' => "Choose a course",
- 'gobt' => "Go"
- );
-
-# Determine the courses
- my %courseIDs = &Apache::lonnet::courseiddump($dom,'.',1);
- &print_header($r,$tasklongref);
- $r->print(<
-
'.&mt('Search for a community in the [_1] domain',$domdesc).'
');
+ } else {
+ $r->print('
'.&mt('Search for a course in the [_1] domain',$domdesc).'
');
+ }
+ $r->print(&Apache::lonpickcourse::build_filters($filterlist,$type,
+ undef,undef,$filter,$action,\$numtitles,'modifycourse'));
+ if ($type eq 'Community') {
+ $r->print(&mt('Actions available after searching for a community:').'
'.
+ '
'.&mt('Enter the community with the role of [_1]',$cctitle).'
'."\n".
+ '
'.&mt('View or modify community settings which only a [_1] may modify.',$dctitle).
+ '
'."\n".'
');
+ } else {
+ $r->print(&mt('Actions available after searching for a course:').'
'.
+ '
'.&mt('Enter the course with the role of [_1]',$cctitle).'
'."\n".
+ '
'.&mt('View or modify course settings which only a [_1] may modify.',$dctitle).
+ '
'."\n".'
');
+ }
+}
+
+sub print_course_selection_page {
+ my ($r,$dom,$domdesc) = @_;
+ my $type = $env{'form.type'};
+ if (!defined($type)) {
+ $type = 'Course';
+ }
+ &print_header($r,$type);
+
+# Criteria for course search
+ my $filterlist = ['descriptfilter',
+ 'instcodefilter','ownerfilter',
+ 'ownerdomfilter','coursefilter'];
+ my %filter;
+ my $action = '/adm/modifycourse';
+ my $dctitle = &Apache::lonnet::plaintext('dc');
+ my $numtitles;
+ $r->print(&Apache::lonpickcourse::js_changer());
+ $r->print(&mt('Revise your search criteria for this domain').' ('.$domdesc.'). ');
+ $r->print(&Apache::lonpickcourse::build_filters($filterlist,$type,
+ undef,undef,\%filter,$action,\$numtitles));
+ $filter{'domainfilter'} = $dom;
+ my %courses = &Apache::lonpickcourse::search_courses($r,$type,0,
+ \%filter,$numtitles);
+ &Apache::lonpickcourse::display_matched_courses($r,$type,0,$action,undef,undef,undef,
+ %courses);
+ return;
+}
+
+sub print_modification_menu {
+ my ($r,$cdesc,$domdesc,$dom,$type) = @_;
+ &print_header($r,$type);
+ my ($ccrole,$setquota_text,$setparams_text,$cat_text);
+ if ($type eq 'Community') {
+ $ccrole = 'co';
+ } else {
+ $ccrole = 'cc';
+ }
+ my $action = '/adm/modifycourse';
+ if ($type eq 'Community') {
+ $setquota_text = &mt('Total disk space allocated for storage of portfolio files in all groups in a community.');
+ $setparams_text = 'View/Modify community owner';
+ $cat_text = 'View/Modify catalog settings for community';
+ } else {
+ $setquota_text = &mt('Total disk space allocated for storage of portfolio files in all groups in a course.');
+ $setparams_text = 'View/Modify course owner, institutional code, and default authentication';
+ $cat_text = 'View/Modify catalog settings for course';
+ }
+ my @menu =
+ (
+ { text => $setparams_text,
+ phase => 'setparms',
+ },
+ { text => 'View/Modify quota for group portfolio files',
+ phase => 'setquota',
+ }
+ );
+ my %domconf = &Apache::lonnet::get_dom('configuration',['coursecategories'],$dom);
+ my @additional_params = &catalog_settable($domconf{'coursecategories'},$type);
+ if (@additional_params > 0) {
+ push (@menu, { text => $cat_text,
+ phase => 'catsettings',
+ });
+ }
+ unless ($type eq 'Community') {
+ push(@menu,
+ { text => 'Display current settings for automated enrollment',
+ phase => 'viewparms',
+ }
+ );
+ }
+ my $menu_html = '
'."\n";
+ if ($type eq 'Community') {
+ $menu_html .= &mt('Although almost all community settings can be modified by a Coordinator, the following may only be set or modified by a Domain Coordinator:');
+ } else {
+ $menu_html .= &mt('Although almost all course settings can be modified by a Course Coordinator, the following may only be set or modified by a Domain Coordinator:');
+ }
+ $menu_html .= '
'.&mt('Community owner (permitted to assign Coordinator roles in the community).').'
';
+ } else {
+ $menu_html .= '
'.&mt('Course owner (permitted to assign Course Coordinator roles in the course).').'
'.
+ '
'.&mt("Institutional code and default authentication (both required for auto-enrollment of students from institutional datafeeds).").'
';
+ }
+ $menu_html .= '
'.$setquota_text.'
'."\n";
+ foreach my $item (@additional_params) {
+ if ($type eq 'Community') {
+ if ($item eq 'togglecats') {
+ $menu_html .= '
'.&mt('Hiding/unhiding a community from the catalog (although can be [_1]configured[_2] to be modifiable by a Coordinator in community context).','','').'
'.&mt('Hiding/unhiding a course from the course catalog (although can be [_1]configured[_2] to be modifiable by a Course Coordinator in course context).','','').'
'.&mt('Manual cataloging of a course (although can be [_1]configured[_2] to be modifiable by a Course Coordinator in course context).','','').'
'."\n";
}
}
}
- unless ( (grep/^$enrollvar{'courseowner'}$/,@local_ccs) || ($enrollvar{'courseowner'} eq '') ) {
- push @local_ccs, $enrollvar{'coursecode'};
- my ($puname,$pdom) = split/\@/,$enrollvar{'courseowner'};
- $pname{$enrollvar{'courseowner'}} = &Apache::loncommon::plainname($puname,$pdom);
- $active_cc = &LONCAPA::Enrollment::check_user_status($pdom,$enrollvar{'coursecode'},$dom,$course,'cc');
- if ($active_cc eq 'ok') {
- $cc_status{$enrollvar{'courseowner'}} = 'Active';
- } else {
- $cc_status{$enrollvar{'courseowner'}} = 'Inactive';
- }
- }
- my $numlocalcc = @local_ccs;
- my $bodytag=&Apache::loncommon::bodytag('Modify Course Settings');
- my $helplink=&Apache::loncommon::help_open_topic('Modify_Course',&mt('Help on Modifying Courses'));
- my $defdom=$ENV{'request.role.domain'};
- my ($krbdef,$krbdefdom)=&Apache::loncommon::get_kerberos_defaults($defdom);
- my $javascript_validations=&Apache::londropadd::javascript_validations('createcourse',$krbdefdom);
- my %param = ( formname => 'document.cmods',
- kerb_def_dom => $krbdefdom,
- kerb_def_auth => $krbdef
- );
- my $selscript=&Apache::loncommon::studentbrowser_javascript();
- my $sellink=&Apache::loncommon::selectstudent_link
- ('cmod','crsown','ccdomain');
- my $krbform = &Apache::loncommon::authform_kerberos(%param);
- my $intform = &Apache::loncommon::authform_internal(%param);
- my $locform = &Apache::loncommon::authform_local(%param);
- my %lt=&Apache::lonlocal::texthash(
- 'aecs' => "Automated Enrollment Course Settings",
- 'cose' => "Course settings for LON-CAPA courses that control automated student enrollment based on classlist data available from your institution's student information system fall into two groups: (a) settings that can be modified by a Course Coordinator using the ",
- 'aenm' => "Automated Enrollment Manager",
- 'andb' => " and (b) settings that may only be modified by a Domain Coordinator via this page.",
- 'caes' => 'Current automated enrollment settings',
- 'cour' => "Course settings that control automated enrollment in this LON-CAPA course are currently:",
- 'cset' => "Course setting",
- 'valu' => "Current value",
- 'ccus' => "A course coordinator can use the 'Automated Enrollment Manager' to change all settings except course code, course owner, and default authentication method for students added to your course (who are also new to the LON-CAPA system at this domain).",
- 'mkch' => "Make changes to course settings set by Domain Coordinator",
- 'ccod' => "Course Code",
- 'ccus' => "The course code is used during automated enrollment to map the internal LON-CAPA course ID for this course to the corresponding course section ID(s) used by the office responsible for providing official class lists for courses at your institution.",
- 'cown' => "Course Owner",
- 'cous' => "The course owner is used in the retrieval of class lists from your institution's student information system when access to class lists data incorporates validation of instructor status.",
- 'tabl' => 'The table below contains a list of active course coordinators in this course,
-who are from this domain',
- 'usrd' => 'Use the radio buttons to select a different course owner.',
- 'deam' => "Default Authentication method",
- 'deus' => "The default authentication method, and default authentication parameter (domain, initial password or argument) are used when automatic enrollment of students in a course requires addition of new user accounts in your domain, and the class list file contains empty entries for the <authtype> and <autharg> properties for the new student.",
- 'gobt' => "Modify settings",
+ $menu_html .= '
'."\n".
+ &Apache::loncommon::end_data_table_row()."\n";
+ }
+ $ownertable .= &Apache::loncommon::end_data_table();
+ }
+ &print_header($r,$type,$javascript_validations);
+ my $dctitle = &Apache::lonnet::plaintext('dc');
+ my $mainheader = &modifiable_only_title($type);
+ my $hidden_elements = &hidden_form_elements();
+ $r->print('');
+ return;
+}
- my $description = $settings{'description'};
- my %cenv = ();
+sub modifiable_only_title {
+ my ($type) = @_;
+ my $dctitle = &Apache::lonnet::plaintext('dc');
+ if ($type eq 'Community') {
+ return &mt('Community settings modifiable only by [_1] for:',$dctitle);
+ } else {
+ return &mt('Course settings modifiable only by [_1] for:',$dctitle);
+ }
+}
- if ($ENV{'form.login'} eq 'krb') {
- $newattr{'authtype'} = $ENV{'form.login'};
- $newattr{'authtype'} .= $ENV{'form.krbver'};
- $newattr{'autharg'} = $ENV{'form.krbarg'};
- } elsif ($ENV{'form.login'} eq 'int') {
- $newattr{'authtype'} ='internal';
- if ((defined($ENV{'form.intarg'})) && ($ENV{'form.intarg'})) {
- $newattr{'autharg'} = $ENV{'form.intarg'};
- }
- } elsif ($ENV{'form.login'} eq 'loc') {
- $newattr{'authtype'} = 'localauth';
- if ((defined($ENV{'form.locarg'})) && ($ENV{'form.locarg'})) {
- $newattr{'autharg'} = $ENV{'form.locarg'};
- }
- }
- if ( $newattr{'authtype'}=~ /^krb/) {
- if ($newattr{'autharg'} eq '') {
- $warning = qq(''.
- &mt('As you did not include the default Kerberos domain to be used for authentication in this class, the institutional data used by the automated enrollment process must include the Kerberos domain for each new student').'');
+sub gather_authenitems {
+ my ($cdom,$enrollvar) = @_;
+ my ($krbdef,$krbdefdom)=&Apache::loncommon::get_kerberos_defaults($cdom);
+ my $curr_authtype = '';
+ my $curr_authfield = '';
+ if (ref($enrollvar) eq 'HASH') {
+ if ($enrollvar->{'authtype'} =~ /^krb/) {
+ $curr_authtype = 'krb';
+ } elsif ($enrollvar->{'authtype'} eq 'internal' ) {
+ $curr_authtype = 'int';
+ } elsif ($enrollvar->{'authtype'} eq 'localauth' ) {
+ $curr_authtype = 'loc';
}
}
+ unless ($curr_authtype eq '') {
+ $curr_authfield = $curr_authtype.'arg';
+ }
+ my $javascript_validations =
+ &Apache::lonuserutils::javascript_validations('modifycourse',$krbdefdom,
+ $curr_authtype,$curr_authfield);
+ my %param = ( formname => 'document.'.$env{'form.phase'},
+ kerb_def_dom => $krbdefdom,
+ kerb_def_auth => $krbdef,
+ mode => 'modifycourse',
+ curr_authtype => $curr_authtype,
+ curr_autharg => $enrollvar->{'autharg'}
+ );
+ my (%authform,$authenitems);
+ $authform{'krb'} = &Apache::loncommon::authform_kerberos(%param);
+ $authform{'int'} = &Apache::loncommon::authform_internal(%param);
+ $authform{'loc'} = &Apache::loncommon::authform_local(%param);
+ foreach my $item ('krb','int','loc') {
+ if ($authform{$item} ne '') {
+ $authenitems .= $authform{$item}.' ';
+ }
+ }
+ return($javascript_validations,$authenitems);
+}
+
+sub modify_course {
+ my ($r,$cdom,$cnum,$cdesc,$domdesc,$type) = @_;
+ my %longtype = &course_settings_descrip($type);
+ my @items = ('internal.courseowner','description','internal.co-owners',
+ 'internal.pendingco-owners');
+ unless ($type eq 'Community') {
+ push(@items,('internal.coursecode','internal.authtype','internal.autharg',
+ 'internal.sectionnums','internal.crosslistings'));
+ }
+ my %settings = &Apache::lonnet::get('environment',\@items,$cdom,$cnum);
+ my $description = $settings{'description'};
+ my ($ccrole,$response,$chgresponse,$nochgresponse,$reply,%currattr,%newattr,%cenv,%changed,
+ @changes,@nochanges,@sections,@xlists,@warnings);
+ my @modifiable_params = &get_dc_settable($type);
+ foreach my $param (@modifiable_params) {
+ $currattr{$param} = $settings{'internal.'.$param};
+ }
+ if ($type eq 'Community') {
+ %changed = ( owner => 0 );
+ $ccrole = 'co';
+ } else {
+ %changed = ( code => 0,
+ owner => 0,
+ );
+ $ccrole = 'cc';
+ unless ($settings{'internal.sectionnums'} eq '') {
+ if ($settings{'internal.sectionnums'} =~ m/,/) {
+ @sections = split/,/,$settings{'internal.sectionnums'};
+ } else {
+ $sections[0] = $settings{'internal.sectionnums'};
+ }
+ }
+ unless ($settings{'internal.crosslistings'} eq'') {
+ if ($settings{'internal.crosslistings'} =~ m/,/) {
+ @xlists = split/,/,$settings{'internal.crosslistings'};
+ } else {
+ $xlists[0] = $settings{'internal.crosslistings'};
+ }
+ }
+ if ($env{'form.login'} eq 'krb') {
+ $newattr{'authtype'} = $env{'form.login'};
+ $newattr{'authtype'} .= $env{'form.krbver'};
+ $newattr{'autharg'} = $env{'form.krbarg'};
+ } elsif ($env{'form.login'} eq 'int') {
+ $newattr{'authtype'} ='internal';
+ if ((defined($env{'form.intarg'})) && ($env{'form.intarg'})) {
+ $newattr{'autharg'} = $env{'form.intarg'};
+ }
+ } elsif ($env{'form.login'} eq 'loc') {
+ $newattr{'authtype'} = 'localauth';
+ if ((defined($env{'form.locarg'})) && ($env{'form.locarg'})) {
+ $newattr{'autharg'} = $env{'form.locarg'};
+ }
+ }
+ if ( $newattr{'authtype'}=~ /^krb/) {
+ if ($newattr{'autharg'} eq '') {
+ push(@warnings,
+ &mt('As you did not include the default Kerberos domain'
+ .' to be used for authentication in this class, the'
+ .' institutional data used by the automated'
+ .' enrollment process must include the Kerberos'
+ .' domain for each new student.'));
+ }
+ }
- if ( exists($ENV{'form.courseowner'}) ) {
- my $ownerparam = 'username_'.$ENV{'form.courseowner'};
- if ( exists($ENV{"form.$ownerparam"}) ) {
- $newattr{'courseowner'}=$ENV{"form.$ownerparam"};
- unless ( $newattr{'courseowner'} eq $currattr{'courseowner'} ) {
- $changeowner = 1;
+ if ( exists($env{'form.coursecode'}) ) {
+ $newattr{'coursecode'}=$env{'form.coursecode'};
+ unless ( $newattr{'coursecode'} eq $currattr{'coursecode'} ) {
+ $changed{'code'} = 1;
}
+ }
+ }
+
+ if ( exists($env{'form.courseowner'}) ) {
+ $newattr{'courseowner'}=$env{'form.courseowner'};
+ unless ( $newattr{'courseowner'} eq $currattr{'courseowner'} ) {
+ $changed{'owner'} = 1;
}
}
-
- if ( exists($ENV{'form.coursecode'}) ) {
- $newattr{'coursecode'}=$ENV{'form.coursecode'};
- unless ( $newattr{'coursecode'} eq $currattr{'coursecode'} ) {
- $changecode = 1;
+
+ if ($changed{'owner'} || $changed{'code'}) {
+ my %crsinfo = &Apache::lonnet::courseiddump($cdom,'.',1,'.','.',$cnum,
+ undef,undef,'.');
+ if (ref($crsinfo{$env{'form.pickedcourse'}}) eq 'HASH') {
+ if ($changed{'code'}) {
+ $crsinfo{$env{'form.pickedcourse'}}{'inst_code'} = $env{'form.coursecode'};
+ }
+ if ($changed{'owner'}) {
+ $crsinfo{$env{'form.pickedcourse'}}{'owner'} = $env{'form.courseowner'};
+ }
+ my $chome = &Apache::lonnet::homeserver($cnum,$cdom);
+ my $putres = &Apache::lonnet::courseidput($cdom,\%crsinfo,$chome,'notime');
+ if ($putres eq 'ok') {
+ &update_coowners($cdom,$cnum,$chome,\%settings,\%newattr);
+ }
}
}
-
- foreach (@params) {
- if ($currattr{$_} eq $newattr{$_}) {
- push @nochanges, $_;
+ foreach my $param (@modifiable_params) {
+ if ($currattr{$param} eq $newattr{$param}) {
+ push(@nochanges,$param);
} else {
- my $attr = 'internal.'.$_;
- $cenv{$attr} = $newattr{$_};
- push @changes, $_;
+ $cenv{'internal.'.$param} = $newattr{$param};
+ push(@changes,$param);
}
}
-
if (@changes > 0) {
- $chgresponse = "The following automated enrollment parameters have been changed:
";
+ $chgresponse = &mt("The following settings have been changed:
");
}
- if (@nochanges > 0) {
- $nochgresponse = "The following automated enrollment parameters remain unchanged:
";
+ if (@nochanges > 0) {
+ $nochgresponse = &mt("The following settings remain unchanged:
");
}
- if (@changes > 0) {
- my $putreply = &Apache::lonnet::put('environment',\%cenv,$dom,$crs);
+ if (@changes > 0) {
+ my $putreply = &Apache::lonnet::put('environment',\%cenv,$cdom,$cnum);
if ($putreply !~ /^ok$/) {
- $response = "There was a problem processing your requested changes. The automated enrollment settings for this course have been left unchanged. ";
+ $response = '
'.
+ &mt('There was a problem processing your requested changes.').' ';
+ if ($type eq 'Community') {
+ $response .= &mt('Settings for this community have been left unchanged.');
+ } else {
+ $response .= &mt('Settings for this course have been left unchanged.');
+ }
+ $response .= ' '.&mt('Error: ').$putreply.'
';
} else {
- foreach my $attr (@params) {
- if (grep/^$attr$/,@changes) {
- $chgresponse .= "
$$typeref{$attr} now set to $newattr{$attr}.
";
+ foreach my $attr (@modifiable_params) {
+ if (grep/^\Q$attr\E$/,@changes) {
+ $chgresponse .= '
'.$longtype{$attr}.' '.&mt('now set to:').' "'.$newattr{$attr}.'".
';
} else {
- $nochgresponse .= "
$$typeref{$attr} still set to $currattr{$attr}.
";
+ $nochgresponse .= '
'.$longtype{$attr}.' '.&mt('still set to:').' "'.$currattr{$attr}.'".
';
}
}
- if ($changecode || $changeowner) {
+ if (($type ne 'Community') && ($changed{'code'} || $changed{'owner'})) {
if ( $newattr{'courseowner'} eq '') {
- $warning .= "There is no owner associated with this LON-CAPA course. If automated enrollment in LON-CAPA courses at your institution requires validation of course owners, automated enrollment will fail for this course. ";
+ push(@warnings,&mt('There is no owner associated with this LON-CAPA course.').
+ ' '.&mt('If automated enrollment at your institution requires validation of course owners, automated enrollment will fail.'));
} else {
if (@sections > 0) {
- foreach my $sec (@sections) {
- if ($sec =~ m/^(.+):/) {
- my $course_id = $newattr{'coursecode'}.$1;
- if ($changecode) {
-# my $course_check = &localenroll::validate_courseID($course_id);
- my $course_check = 'ok';
- if ($course_check eq 'ok') {
-# my $outcome = &localenroll::new_course($course_id,$newattr{'courseowner'});
- my $outcome = 'ok';
- unless ($outcome eq 'ok') {
- $warning .= "If automatic enrollment is enabled for LON-CAPA course: $description, automated enrollment may fail for $newattr{'coursecode'} - section $1 for the following reason: $outcome. ";
+ if ($changed{'code'}) {
+ foreach my $sec (@sections) {
+ if ($sec =~ m/^(.+):/) {
+ my $instsec = $1;
+ my $inst_course_id = $newattr{'coursecode'}.$1;
+ my $course_check = &Apache::lonnet::auto_validate_courseID($cnum,$cdom,$inst_course_id);
+ if ($course_check eq 'ok') {
+ my $outcome = &Apache::lonnet::auto_new_course($cnum,$cdom,$inst_course_id,$newattr{'courseowner'});
+ unless ($outcome eq 'ok') {
+
+ push(@warnings,&mt('If automatic enrollment is enabled for "[_1]" automated enrollment may fail for "[_2]" - section: [_3], for the following reason: "[_4]"',$description,$newattr{'coursecode'},$instsec,$outcome).' ');
}
} else {
- $warning .= "If automatic enrollment is enabled for LON-CAPA course: $description, automated enrollment may fail for $newattr{'coursecode'} - section $1 for the following reason: $course_check. ";
+ push(@warnings,&mt('If automatic enrollment is enabled for "[_1]" automated enrollment may fail for "[_2]" - section: [_3], for the following reason: "[_4]"',$description,$newattr{'coursecode'},$instsec,$course_check));
}
} else {
- $warning .= "If automatic enrollment is enabled for LON-CAPA course: $description, automated enrollment may fail for $newattr{'coursecode'} - section $sec because this is not a valid section entry. ";
+ push(@warnings,&mt('If automatic enrollment is enabled for "[_1]", automated enrollment may fail for "[_2]" - section: [_3], because this is not a valid section entry.',$description,$newattr{'coursecode'},$sec));
}
}
- }
+ } elsif ($changed{'owner'}) {
+ foreach my $sec (@sections) {
+ if ($sec =~ m/^(.+):/) {
+ my $instsec = $1;
+ my $inst_course_id = $newattr{'coursecode'}.$instsec;
+ my $outcome = &Apache::lonnet::auto_new_course($cnum,$cdom,$inst_course_id,$newattr{'courseowner'});
+ unless ($outcome eq 'ok') {
+ push(@warnings,&mt('If automatic enrollment is enabled for "[_1]", automated enrollment may fail for "[_2]" - section [_3] for the following reason: "[_4]".',$description,$newattr{'coursecode'},$instsec,$outcome));
+ }
+ } else {
+ push(@warnings,&mt('If automatic enrollment is enabled for "[_1]", automated enrollment may fail for "[_2]" - section [_3] because this is not a valid section entry.',$description,$newattr{'coursecode'},$sec));
+ }
+ }
+ }
} else {
- $warning .= "As no section numbers are currently listed for LON-CAPA course: $description, automated enrollment will not occur for any sections of coursecode: $newattr{'coursecode'}. ";
+ push(@warnings,&mt('As no section numbers are currently listed for "[_1]" automated enrollment will not occur for any sections of institutional course code: "[_2]".',$description,$newattr{'coursecode'}));
}
- if ( (@xlists > 0) && ($changeowner) ) {
+ if ( (@xlists > 0) && ($changed{'owner'}) ) {
foreach my $xlist (@xlists) {
if ($xlist =~ m/^(.+):/) {
-# my $outcome = &localenroll::new_course($1,$newattr{'courseowner'});
- my $outcome = 'ok';
+ my $instxlist = $1;
+ my $outcome = &Apache::lonnet::auto_new_course($cnum,$cdom,$instxlist,$newattr{'courseowner'});
unless ($outcome eq 'ok') {
- $warning .= "If automatic enrollment is enabled for LON-CAPA course: $description, automated enrollment may fail for crosslisted class: $1 for the following reason: $outcome. ";
+ push(@warnings,&mt('If automatic enrollment is enabled for "[_1]", automated enrollment may fail for crosslisted class "[_2]" for the following reason: "[_3]".',$description,$instxlist,$outcome));
}
- }
+ }
}
}
}
}
}
+ } else {
+ foreach my $attr (@modifiable_params) {
+ $nochgresponse .= '
'.$longtype{$attr}.' '.&mt('still set to').' "'.$currattr{$attr}.'".
';
+ }
}
if (@changes > 0) {
@@ -507,83 +831,522 @@ sub modify_course {
if (@nochanges > 0) {
$nochgresponse .= "
";
}
- unless ($warning eq '') {
- $warning = "The following warning messages were generated as a result of applying the changes you specified to course settings that can affect the automated enrollment process:
".$warning;
+ my ($warning,$numwarnings);
+ my $numwarnings = scalar(@warnings);
+ if ($numwarnings) {
+ $warning = &mt('The following [quant,_1,warning was,warnings were] generated when applying your changes to automated enrollment:',$numwarnings).'