--- loncom/interface/lonconfigsettings.pm 2011/03/31 15:35:54 1.18 +++ loncom/interface/lonconfigsettings.pm 2011/10/19 21:15:53 1.20 @@ -1,7 +1,7 @@ # The LearningOnline Network with CAPA # Handler to set domain-wide configuration settings # -# $Id: lonconfigsettings.pm,v 1.18 2011/03/31 15:35:54 raeburn Exp $ +# $Id: lonconfigsettings.pm,v 1.20 2011/10/19 21:15:53 www Exp $ # # Copyright Michigan State University Board of Trustees # @@ -38,7 +38,7 @@ use Apache::lonlocal; use Apache::lonparmset(); sub print_header { - my ($r,$phase,$context,$jscript) = @_; + my ($r,$phase,$context,$jscript,$container) = @_; my ($pagetitle,$brcrumtitle,$action,$call_category_check); if ($context eq 'domain') { ($pagetitle, $brcrumtitle) = ('View/Modify Domain Settings','View/Modify Domain Settings'); @@ -120,9 +120,6 @@ $jscript } $r->print(&Apache::loncommon::start_page($pagetitle,$js,$additem)); $r->print(&Apache::lonhtmlcommon::breadcrumbs($brcrumtitle)); - if ($context eq 'course') { - &Apache::lonparmset::startSettingsScreen($r,'coursepref'); - } $r->print('
@@ -130,13 +127,16 @@ $jscript
'); + if ($container) { + &Apache::lonparmset::startSettingsScreen($r,$container); + } $r->print('
'); return; } sub print_footer { - my ($r,$context,$phase,$newphase,$button_text,$actions) = @_; + my ($r,$phase,$newphase,$button_text,$actions,$container) = @_; $button_text = &mt($button_text); $r->print(''); if (defined($env{'form.origin'})) { @@ -164,8 +164,8 @@ sub print_footer { } if ($phase eq 'process') { $r->print('
'); - if ($context eq 'course') { - &Apache::lonparmset::endSettingsScreen($r); + if ($container) { + &Apache::lonparmset::endSettingsScreen($r); } $r->print(&Apache::loncommon::end_page()); } @@ -173,7 +173,7 @@ sub print_footer { } sub make_changes { - my ($r,$dom,$phase,$context,$prefs_order,$prefs,$values,$confname,$roles,$allitems) = @_; + my ($r,$dom,$phase,$context,$prefs_order,$prefs,$values,$confname,$roles,$allitems,$container) = @_; my %brcrumtext = &get_crumb_text(); my @actions = &Apache::loncommon::get_env_multiple('form.actions'); my ($numchanged,%changes,%disallowed); @@ -182,7 +182,7 @@ sub make_changes { text=>$brcrumtext{$context}}, {href=>"javascript:changePage(document.$phase,'$phase')", text=>"Updated"}); - &print_header($r,$phase,$context); + &print_header($r,$phase,$context,undef,$container); my $crstype; if ($context eq 'course') { $crstype = &Apache::loncommon::course_type(); @@ -237,19 +237,19 @@ sub make_changes { if ($context eq 'course') { $footer_text = 'Back to display/edit settings'; } - &print_footer($r,$context,$phase,'display',$footer_text,\@actions); + &print_footer($r,$phase,'display',$footer_text,\@actions,$container); $r->print('

'); } sub display_settings { my ($r,$dom,$phase,$context,$prefs_order,$prefs,$values,$confname,$jscript, - $allitems,$crstype) = @_; + $allitems,$crstype,$container) = @_; my %brcrumtext = &get_crumb_text(); my @actions = &Apache::loncommon::get_env_multiple('form.actions'); &Apache::lonhtmlcommon::add_breadcrumb ({href=>"javascript:changePage(document.$phase,'display')", text=>"Display/Edit Settings"}); - &print_header($r,$phase,$context,$jscript); + &print_header($r,$phase,$context,$jscript,$container); my $divwidth = 900; if ((ref($prefs_order) eq 'ARRAY') && (ref($prefs) eq 'HASH') && (ref($values) eq 'HASH')) { if (@actions > 0) { @@ -282,7 +282,7 @@ sub display_settings { $r->print($output{$items[$i]}); } $r->print(''); - $r->print(&print_footer($r,$context,$phase,'process','Save Changes',\@actions)); + $r->print(&print_footer($r,$phase,'process','Save Changes',\@actions,$container)); } else { $r->print(''. ''.&mt('No settings chosen'). @@ -290,45 +290,17 @@ sub display_settings { } $r->print(''); } - if ($context eq 'course') { - $r->print(' - -'); - } $r->print(&Apache::loncommon::end_page()); return; } sub display_choices { - my ($r,$phase,$context,$prefs_order,$prefs) = @_; + my ($r,$phase,$context,$prefs_order,$prefs,$container) = @_; if ($phase eq '') { $phase = 'pickactions'; } my %helphash; - &print_header($r,$phase,$context); + &print_header($r,$phase,$context,undef,$container); $r->print('