--- loncom/interface/lonconfigsettings.pm 2016/08/04 17:34:35 1.21.4.8 +++ loncom/interface/lonconfigsettings.pm 2020/05/22 22:38:55 1.21.4.9.6.2 @@ -1,7 +1,7 @@ # The LearningOnline Network with CAPA # Handler to set domain-wide configuration settings # -# $Id: lonconfigsettings.pm,v 1.21.4.8 2016/08/04 17:34:35 raeburn Exp $ +# $Id: lonconfigsettings.pm,v 1.21.4.9.6.2 2020/05/22 22:38:55 raeburn Exp $ # # Copyright Michigan State University Board of Trustees # @@ -39,7 +39,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') { @@ -176,11 +176,26 @@ $jscript $additem = {'add_entries' => \%loaditems,}; } elsif ($phase eq 'display') { if ($context eq 'domain') { + my $onload; if (grep(/^coursedefaults$/,@actions)) { + $onload = "toggleDisplay(document.display,'cloneinstcode');". + "toggleDisplay(document.display,'credits');". + "toggleDisplay(document.display,'studentsubmission');"; + } + if (grep(/^ltitools$/,@actions)) { + $onload .= "toggleLTITools(document.display,'user','add');"; + if (ref($values) eq 'HASH') { + if (ref($values->{'ltitools'}) eq 'HASH') { + my $numltitools = scalar(keys(%{$values->{'ltitools'}})); + for (my $i=0; $i<$numltitools; $i++) { + $onload .= "toggleLTITools(document.display,'user','$i');"; + } + } + } + } + if ($onload) { my %loaditems = ( - ' onload' => "toggleDisplay(document.display,'cloneinstcode');". - "toggleDisplay(document.display,'credits');". - "toggleDisplay(document.display,'studentsubmission');", + 'onload' => $onload, ); $additem = {'add_entries' => \%loaditems,}; } @@ -209,7 +224,7 @@ $jscript } sub print_footer { - my ($r,$phase,$newphase,$button_text,$actions,$container) = @_; + my ($r,$phase,$newphase,$button_text,$actions,$container,$parm_permission) = @_; $button_text = &mt($button_text); $r->print(''); if (defined($env{'form.origin'})) { @@ -234,8 +249,16 @@ sub print_footer { } else { $onclick = '"javascript:changePage(document.'.$phase.','."'$newphase'".')"'; } - $r->print('
'); + my $showbutton = 1; + if (ref($parm_permission) eq 'HASH') { + unless (($parm_permission->{'process'}) || ($newphase eq 'display')) { + $showbutton = 0; + } + } + if ($showbutton) { + $r->print(''); + } } if ($phase eq 'process') { $r->print(''); @@ -245,7 +268,8 @@ sub print_footer { } sub make_changes { - my ($r,$dom,$phase,$context,$prefs_order,$prefs,$values,$confname,$roles,$allitems,$container) = @_; + my ($r,$dom,$phase,$context,$prefs_order,$prefs,$values,$confname,$roles, + $allitems,$container,$parm_permission) = @_; my %brcrumtext = &get_crumb_text(); my @actions = &Apache::loncommon::get_env_multiple('form.actions'); my ($numchanged,%changes,%disallowed); @@ -309,14 +333,14 @@ sub make_changes { if ($context eq 'course') { $footer_text = 'Back to display/edit settings'; } - &print_footer($r,$phase,'display',$footer_text,\@actions,$container); + &print_footer($r,$phase,'display',$footer_text,\@actions,$container,$parm_permission); $r->print(''); return \%lastact; } sub display_settings { my ($r,$dom,$phase,$context,$prefs_order,$prefs,$values,$confname,$jscript, - $allitems,$crstype,$container) = @_; + $allitems,$crstype,$container,$parm_permission) = @_; my %brcrumtext = &get_crumb_text(); my @actions = &Apache::loncommon::get_env_multiple('form.actions'); &Apache::lonhtmlcommon::add_breadcrumb @@ -326,7 +350,7 @@ sub display_settings { if (ref($values) eq 'HASH') { $instcode = $values->{'internal.coursecode'}; } - &print_header($r,$phase,$context,$jscript,$container,$instcode); + &print_header($r,$phase,$context,$jscript,$container,$instcode,$dom,$values); my $divwidth = 900; if ((ref($prefs_order) eq 'ARRAY') && (ref($prefs) eq 'HASH') && (ref($values) eq 'HASH')) { if (@actions > 0) { @@ -377,7 +401,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}; } @@ -387,7 +411,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'). @@ -400,7 +424,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'; } @@ -411,7 +435,13 @@ sub display_choices { &Apache::loncommon::check_uncheck_jscript()."\n". '// ]]>'."\n". ''."\n"); - $r->print('