--- loncom/interface/lonconfigsettings.pm 2015/05/21 23:26:35 1.30 +++ loncom/interface/lonconfigsettings.pm 2021/04/19 14:11:16 1.48 @@ -1,7 +1,7 @@ # The LearningOnline Network with CAPA # Handler to set domain-wide configuration settings # -# $Id: lonconfigsettings.pm,v 1.30 2015/05/21 23:26:35 raeburn Exp $ +# $Id: lonconfigsettings.pm,v 1.48 2021/04/19 14:11:16 raeburn Exp $ # # Copyright Michigan State University Board of Trustees # @@ -40,7 +40,7 @@ use Apache::courseclassifier(); use LONCAPA qw(:DEFAULT :match); sub print_header { - my ($r,$phase,$context,$jscript,$container) = @_; + my ($r,$phase,$context,$jscript,$container,$instcode,$dom,$values) = @_; my ($pagetitle,$brcrumtitle,$action,$call_category_check,$instcode_check, $crstype,@actions,@code_order); if ($phase eq 'display') { @@ -75,6 +75,7 @@ sub print_header { \@code_order); if (@code_order) { my $noinstcodestr = &mt('You indicated cloning based on category, but did not select any categories.'); + &js_escape(\$noinstcodestr); $instcode_check = <<"ENDSCRIPT"; if (formname == document.display) { if (formname.cloners_instcode.length) { @@ -118,6 +119,7 @@ ENDSCRIPT } } my $alert = &mt('You must select at least one functionality type to display.'); + &js_escape(\$alert); my $js = ' '."\n"); + if (($item eq 'usersessions') || ($item eq 'ssl')) { + unless ($got_check_uncheck) { + $r->print(''."\n"); + $got_check_uncheck = 1; + } } elsif ($item eq 'selfcreation') { if (ref($values) eq 'HASH') { $settings = $values->{'usercreation'}; @@ -382,7 +480,7 @@ sub display_settings { } else { ($output{$item},$rowtotal{$item}) = &Apache::courseprefs::print_config_box($r,$dom,$phase, - $item,$prefs->{$item},$values,$allitems,$crstype); + $item,$prefs->{$item},$values,$allitems,$crstype,$parm_permission); } $rowsum += $rowtotal{$item}; } @@ -392,7 +490,7 @@ sub display_settings { $r->print($output{$items[$i]}); } $r->print(''); - $r->print(&print_footer($r,$phase,'process','Save Changes',\@actions,$container)); + $r->print(&print_footer($r,$phase,'process','Save Changes',\@actions,$container,$parm_permission)); } else { $r->print(''. ''.&mt('No settings chosen'). @@ -408,7 +506,7 @@ sub display_settings { } sub display_choices { - my ($r,$phase,$context,$prefs_order,$prefs,$container) = @_; + my ($r,$phase,$context,$prefs_order,$prefs,$container,$parm_permission) = @_; if ($phase eq '') { $phase = 'pickactions'; } @@ -419,7 +517,13 @@ sub display_choices { &Apache::loncommon::check_uncheck_jscript()."\n". '// ]]>'."\n". ''."\n"); - $r->print('

'.&mt('Settings to display/modify').'

'. + my $heading = &mt('Settings to display/modify'); + if (ref($parm_permission) eq 'HASH') { + unless ($parm_permission->{'process'}) { + $heading = &mt('Settings to display'); + } + } + $r->print('

'.$heading.'

'. '
'.(' 'x2). @@ -457,7 +561,7 @@ sub display_choices { } $r->print('

'); } - $r->print(&print_footer($r,$phase,'display','Display',undef,$container)); + $r->print(&print_footer($r,$phase,'display','Display',undef,$container,$parm_permission)); $r->print(''); if ($container) { &Apache::lonparmset::endSettingsScreen($r);