Diff for /loncom/interface/courseprefs.pm between versions 1.119 and 1.123

version 1.119, 2023/04/13 15:21:00 version 1.123, 2023/06/03 03:32:16
Line 1661  sub process_linkprot { Line 1661  sub process_linkprot {
         } elsif (exists($env{$urlparamname})) {          } elsif (exists($env{$urlparamname})) {
             $env{$urlparamname} = '';              $env{$urlparamname} = '';
         }          }
           my $passback = 'form.linkprot_passback_'.$idx;
           my $passbackparamname = 'form.linkprot_passbackformat_'.$idx;
           if ($env{$passback} == 1) {
               unless ($env{$passbackparamname} =~ /^1\.(0|1)$/) {
                   $env{$passbackparamname} = '';
               }
           } elsif (exists($env{$passbackparamname})) {
               $env{$passbackparamname} = '';
           }
         unless ($idx eq 'add') {          unless ($idx eq 'add') {
             if ((!$current{'returnurl'} && ($env{$urlparamname} ne '')) ||              if ((!$current{'returnurl'} && ($env{$urlparamname} ne '')) ||
                 ($current{'returnurl'} && ($env{$urlparamname} eq ''))) {                  ($current{'returnurl'} && ($env{$urlparamname} eq ''))) {
                 $haschanges{$itemid} = 1;                  $haschanges{$itemid} = 1;
             }              }
               if ((!$current{'passback'} && ($env{$passbackparamname} ne '')) ||
                   ($current{'passback'} && ($env{$passbackparamname} eq ''))) {
                   $haschanges{$itemid} = 1;
               }
         }          }
         if ($env{$urlparamname} ne '') {          if ($env{$urlparamname} ne '') {
             $linkprot{$itemid}{'returnurl'} = $env{$urlparamname};              $linkprot{$itemid}{'returnurl'} = $env{$urlparamname};
         }          }
           if ($env{$passbackparamname} ne '') {
               $linkprot{$itemid}{'passback'} = 1;
               $linkprot{$itemid}{'passbackformat'} = $env{$passbackparamname};
           }
         if ($ltiauth) {          if ($ltiauth) {
             my $reqitem = 'form.linkprot_requser_'.$idx;              my $reqitem = 'form.linkprot_requser_'.$idx;
             $env{$reqitem} =~ s/(`)/'/g;              $env{$reqitem} =~ s/(`)/'/g;
Line 1883  sub process_ltitools { Line 1900  sub process_ltitools {
     my @allfields = ('fullname','firstname','lastname','email','user','roles');      my @allfields = ('fullname','firstname','lastname','email','user','roles');
     map { $possfield{$_} = 1; } @allfields;      map { $possfield{$_} = 1; } @allfields;
   
     my ($dest,$privnum,$cipher,$errors,%ltitools);      my ($dest,$privnum,$cipher);
   
     ($cipher,$privnum) = &get_credentials($cdom,$cnum,'ltitools',$context);      ($cipher,$privnum) = &get_credentials($cdom,$cnum,'ltitools',$context);
     if ($context eq 'domain') {      if ($context eq 'domain') {
Line 3088  sub store_linkprot { Line 3105  sub store_linkprot {
                     my %values = %{$changes->{$id}};                      my %values = %{$changes->{$id}};
                     my %desc = &linkprot_names();                      my %desc = &linkprot_names();
                     my $display;                      my $display;
                     foreach my $title ('name','lifetime','version','key','secret','returnurl') {                      foreach my $title ('name','lifetime','version','key','secret','returnurl','passbackformat') {
                         if (($title eq 'key') || ($title eq 'secret')) {                          if (($title eq 'key') || ($title eq 'secret')) {
                             if (ref($ltienc{$id}) eq 'HASH') {                              if (ref($ltienc{$id}) eq 'HASH') {
                                 if (exists($ltienc{$id}{$title})) {                                  if (exists($ltienc{$id}{$title})) {
Line 3108  sub store_linkprot { Line 3125  sub store_linkprot {
                             if ($values{$title}) {                              if ($values{$title}) {
                                 $display .= &mt('Return URL parameter').': '.$values{$title}.', ';                                   $display .= &mt('Return URL parameter').': '.$values{$title}.', '; 
                             }                              }
                           } elsif ($title eq 'passbackformat') {
                               if ($values{$title} eq '1.0') {
                                   $display .= &mt('Can return grades to Launcher with Outcomes Service 1.0 format').', ';
                               } elsif ($values{$title} eq '1.1') {
                                   $display .= &mt('Can return grades to Launcher with Outcomes Service 1.1 format').', ';
                               }
                         } else {                          } else {
                             $display .= $desc{$title}.': '.$values{$title}.', ';                              $display .= $desc{$title}.': '.$values{$title}.', ';
                         }                          }
Line 3223  sub store_ltitools { Line 3246  sub store_ltitools {
                 &Apache::lonnet::devalidate_cache_new('courseltitools',$hashid);                  &Apache::lonnet::devalidate_cache_new('courseltitools',$hashid);
                 unless (($home eq 'no_host') || ($home eq '')) {                  unless (($home eq 'no_host') || ($home eq '')) {
                     if (grep(/^\Q$home\E$/,@ids)) {                      if (grep(/^\Q$home\E$/,@ids)) {
                         &Apache::lonnet::devalidate_cache_new('courseltitoolsenc',$hashid);                          &Apache::lonnet::devalidate_cache_new('crsltitoolsenc',$hashid);
                     }                      }
                 }                  }
             } else {              } else {
Line 6310  sub print_bridgetasks { Line 6333  sub print_bridgetasks {
 sub print_ltitools {  sub print_ltitools {
     my ($cdom,$cnum,$settings,$rowtotal,$crstype,$noedit,$context) = @_;      my ($cdom,$cnum,$settings,$rowtotal,$crstype,$noedit,$context) = @_;
     my ($datatable,$disabled,$css_class,$dest);      my ($datatable,$disabled,$css_class,$dest);
     my %lt = &ltitools_names();      if ($noedit) {
           $disabled = ' disabled="disabled"';
       }
     my $itemcount = 1;      my $itemcount = 1;
       unless ($context eq 'domain') {
           my %tooltypes = &Apache::loncommon::usable_exttools();
           unless ($tooltypes{'crs'}) {
               my $showtype = 'course';
               if ($crstype eq 'Community') {
                   $showtype = lc($crstype);
               }
               $css_class = $itemcount%2?' class="LC_odd_row"':'';
               $datatable = '<tr '.$css_class.'><td colspan="2">'.
                            &mt("Definition of external tools is not enabled for this $showtype.").'<br />';
               if ($tooltypes{'dom'}) {
                            $datatable .= &mt("Contact an administrator for the $showtype domain ([_1]) to request this feature be enabled.",
                                              '<i>'.$cdom.'</i>').
                            '<br /><br />'.
                            &mt("Use of external tools defined at a domain level is enabled, so the $showtype editor can be used to add tool(s), if any have been defined.");
               } else {
                   $datatable .= &mt("Use of external tools defined at a domain level is not enabled, either, for this $showtype.").
                                 '<br /><br />'.
                                 &mt("Contact an administrator for the $showtype domain ([_1]) to request changes.",
                                     '<i>'.$cdom.'</i>');
   
               }
               $datatable .= '</tr>';
               $itemcount ++;
               return $datatable;
           }
       }
       my %lt = &ltitools_names();
     my $maxnum = 0;      my $maxnum = 0;
     my %ordered;      my %ordered;
     if (ref($settings) eq 'HASH') {      if (ref($settings) eq 'HASH') {
Line 6393  sub print_ltitools { Line 6446  sub print_ltitools {
                 '<option value="HMAC-SHA1"'.$sigsel{'HMAC-SHA1'}.'>HMAC-SHA1</option>'.                  '<option value="HMAC-SHA1"'.$sigsel{'HMAC-SHA1'}.'>HMAC-SHA1</option>'.
                 '<option value="HMAC-SHA256"'.$sigsel{'HMAC-SHA256'}.'>HMAC-SHA256</option></select></span>'.                  '<option value="HMAC-SHA256"'.$sigsel{'HMAC-SHA256'}.'>HMAC-SHA256</option></select></span>'.
                 '<br /><br />'.                  '<br /><br />'.
                 '<span class="LC_nobreak">'.$lt{'url'}.':<input type="text" size="40" name="ltitools_url_'.$i.'"'.                  '<span class="LC_nobreak">'.$lt{'url'}.':<input type="text" size="60" name="ltitools_url_'.$i.'"'.
                 ' value="'.$url.'" /></span>'.                  ' value="'.$url.'" /></span>'.
                 ('&nbsp;'x2).                  ('&nbsp;'x2).
                 '<span class="LC_nobreak">'.$lt{'lifetime'}.':'.                  '<span class="LC_nobreak">'.$lt{'lifetime'}.':'.
Line 6671  sub print_ltitools { Line 6724  sub print_ltitools {
                   '<option value="HMAC-SHA1" selected="selected">HMAC-SHA1</option>'.                    '<option value="HMAC-SHA1" selected="selected">HMAC-SHA1</option>'.
                   '<option value="HMAC-SHA256">HMAC-SHA256</option></select></span>'.                    '<option value="HMAC-SHA256">HMAC-SHA256</option></select></span>'.
                   '<br />'.                    '<br />'.
                   '<span class="LC_nobreak">'.$lt{'url'}.':<input type="text" size="40" name="ltitools_add_url" value="" /></span> '."\n".                    '<span class="LC_nobreak">'.$lt{'url'}.':<input type="text" size="60" name="ltitools_add_url" value="" /></span> '."\n".
                   ('&nbsp;'x2).                    ('&nbsp;'x2).
                   '<span class="LC_nobreak">'.$lt{'lifetime'}.':<input type="text" size="5" name="ltitools_add_lifetime" value="300" /></span><br />';                    '<span class="LC_nobreak">'.$lt{'lifetime'}.':<input type="text" size="5" name="ltitools_add_lifetime" value="300" /></span><br />';
     if ($switchserver) {      if ($switchserver) {
Line 7286  sub print_linkprotection { Line 7339  sub print_linkprotection {
                     '<tr '.$css_class.'><td><span class="LC_nobreak">'.                      '<tr '.$css_class.'><td><span class="LC_nobreak">'.
                     '<label><input type="checkbox" name="linkprot_del" value="'.$i.'"'.$disabled.' />'.                      '<label><input type="checkbox" name="linkprot_del" value="'.$i.'"'.$disabled.' />'.
                     &mt('Delete?').'</label></span></td><td>';                      &mt('Delete?').'</label></span></td><td>';
                 my ($usersty,$onclickrequser,%checkedrequser,$onclickreturnurl,%checkedreturnurl);                  my ($usersty,$onclickrequser,%checkedrequser,$onclickreturnurl,%checkedreturnurl,
                       $onclickpassback,%checkedpassback,$passbacksty,%checkedpassbackfmt);
                   $passbacksty = 'none';
                   $onclickpassback = ' onclick="toggleLinkProtExtra(this.form,'."'passback','passbackparam','1','inline-block','$i'".');"';
                   %checkedpassback = (
                       'no' => ' checked="checked"',
                       'yes' => '',
                   );                
                   %checkedpassbackfmt = (
                       '1p1' => ' checked="checked"',
                       '1p0' => '',
                   );
                   if ($values{'passback'} ne '') {
                       $passbacksty = 'inline-block';
                       $checkedpassback{'yes'} = ' checked="checked"';
                       $checkedpassback{'no'} = '';
                       if ($values{'passbackformat'} eq '1.0') {
                           $checkedpassbackfmt{'1p0'} = ' checked="checked"';
                           $checkedpassbackfmt{'1p1'} = '';
                       }
                   }
                 if ($ltiauth) {                  if ($ltiauth) {
                     $usersty = 'display:none';                      $usersty = 'display:none';
                     $onclickrequser = ' onclick="toggleLinkProtExtra(this.form,'."'requser','optional','1','block','$i'".');"';                      $onclickrequser = ' onclick="toggleLinkProtExtra(this.form,'."'requser','optional','1','block','$i'".');"';
Line 7375  sub print_linkprotection { Line 7448  sub print_linkprotection {
                     }                      }
                 }                  }
                 $datatable .= '<br /><br />'.                  $datatable .= '<br /><br />'.
                                 '<span class="LC_nobreak">'.$desc{'passback'}.'?'.
                                 '<label><input type="radio" name="linkprot_passback_'.$i.'" value="0"'.
                                 $onclickpassback.$checkedpassback{'no'}.$disabled.' />'.&mt('No').'</label>&nbsp;'.
                                 '<label><input type="radio" name="linkprot_passback_'.$i.'" value="1"'.
                                 $onclickpassback.$checkedpassback{'yes'}.$disabled.' />'.&mt('Yes').'</label></span><br />'.
                                 '<div id="linkprot_passbackparam_'.$i.'" class="LC_floatleft" style="display:'.$passbacksty.'" />'.
                                 '<span class="LC_nobreak">'.&mt('Grade format').
                                 '<label><input type="radio" name="linkprot_passbackformat_'.$i.'" value="1.1"'.$checkedpassbackfmt{'1p1'}.' />'.
                                 &mt('Outcomes Service (1.1)').'</label>'.('&nbsp;'x2).
                                 '<label><input type="radio" name="linkprot_passbackformat_'.$i.'" value="1.0"'.$checkedpassbackfmt{'1p0'}.'/>'.
                                 &mt('Outcomes Extension (1.0)').'</label></span></div>'.
                                 '<div style="padding:0;clear:both;margin:0;border:0"></div>'.
                                 '<br />'.
                               '<span class="LC_nobreak">'.$desc{'returnurl'}.'?'.                                '<span class="LC_nobreak">'.$desc{'returnurl'}.'?'.
                               '<label><input type="radio" name="linkprot_returnurl_'.$i.'" value="0"'.                                '<label><input type="radio" name="linkprot_returnurl_'.$i.'" value="0"'.
                               $onclickreturnurl.$checkedreturnurl{'no'}.$disabled.' />'.&mt('No').'</label>&nbsp;'.                                $onclickreturnurl.$checkedreturnurl{'no'}.$disabled.' />'.&mt('No').'</label>&nbsp;'.
Line 7405  sub print_linkprotection { Line 7491  sub print_linkprotection {
                   '<input type="hidden" name="linkprot_maxnum" value="'.$next.'" />'."\n".                    '<input type="hidden" name="linkprot_maxnum" value="'.$next.'" />'."\n".
                   '<input type="checkbox" name="linkprot_add" value="1"'.$disabled.' />'.&mt('Add').'</span></td>'."\n".                    '<input type="checkbox" name="linkprot_add" value="1"'.$disabled.' />'.&mt('Add').'</span></td>'."\n".
                   '<td width="100%">';                    '<td width="100%">';
     my ($usersty,$onclickrequser,%checkedrequser,$onclickreturnurl,%checkedreturnurl);      my ($usersty,$onclickrequser,%checkedrequser,$onclickreturnurl,%checkedreturnurl,
          $onclickpassback,%checkedpassback,%checkedpassbackfmt);
     if ($ltiauth) {      if ($ltiauth) {
         $usersty = 'display:none';          $usersty = 'display:none';
         $onclickrequser = ' onclick="toggleLinkProtExtra(this.form,'."'requser','optional','1','block','add'".');"';          $onclickrequser = ' onclick="toggleLinkProtExtra(this.form,'."'requser','optional','1','block','add'".');"';
Line 7415  sub print_linkprotection { Line 7502  sub print_linkprotection {
         );          );
         $datatable .= '<fieldset><legend>'.$lt{'requ'}.'</legend>';          $datatable .= '<fieldset><legend>'.$lt{'requ'}.'</legend>';
     }      }
       $onclickpassback = ' onclick="toggleLinkProtExtra(this.form,'."'passback','passbackparam','1','inline-block','add'".');"';
       %checkedpassback = (
           'no' => ' checked="checked"',
           'yes' => '',
       );
       %checkedpassbackfmt = (
           '1p1' => ' checked="checked"',
           '1p0' => '',
       );
     $onclickreturnurl = ' onclick="toggleLinkProtExtra(this.form,'."'returnurl','divurlparam','1','inline-block','add'".');"';      $onclickreturnurl = ' onclick="toggleLinkProtExtra(this.form,'."'returnurl','divurlparam','1','inline-block','add'".');"';
     %checkedreturnurl = (      %checkedreturnurl = (
         no => ' checked="checked"',          no => ' checked="checked"',
Line 7437  sub print_linkprotection { Line 7533  sub print_linkprotection {
                       '<label><input type="checkbox" name="linkprot_visible_add" id="linkprot_visible_add" onclick="if (this.checked) { this.form.linkprot_secret_add.type='."'text'".' } else { this.form.linkprot_secret_add.type='."'password'".' }"'.$disabled.' />'.&mt('Visible input').'</label></span> '."\n";                        '<label><input type="checkbox" name="linkprot_visible_add" id="linkprot_visible_add" onclick="if (this.checked) { this.form.linkprot_secret_add.type='."'text'".' } else { this.form.linkprot_secret_add.type='."'password'".' }"'.$disabled.' />'.&mt('Visible input').'</label></span> '."\n";
     }      }
     $datatable .= '<br /><br />'.      $datatable .= '<br /><br />'.
                     '<span class="LC_nobreak">'.$desc{'passback'}.'?'.
                     '<label><input type="radio" name="linkprot_passback_add" value="0"'.
                     $onclickpassback.$checkedpassback{'no'}.$disabled.' />'.&mt('No').'</label>&nbsp;'.
                     '<label><input type="radio" name="linkprot_passback_add" value="1"'.
                     $onclickpassback.$checkedpassback{'yes'}.$disabled.' />'.&mt('Yes').'</label></span>'.
                     '<br /><div id="linkprot_passbackparam_add" class="LC_floatleft" style="display:none" />'.
                     '<span class="LC_nobreak">'.&mt('Grade format').
                     '<label><input type="radio" name="linkprot_passbackformat_add" value="1.1"'.$checkedpassbackfmt{'1p1'}.$disabled.' />'.
                     &mt('Outcomes Service (1.1)').'</label>'.('&nbsp;'x2).
                     '<label><input type="radio" name="linkprot_passbackformat_add" value="1.0"'.$checkedpassbackfmt{'1p0'}.$disabled.' />'.
                     &mt('Outcomes Extension (1.0)').'</label></span></div>'.
                     '<div style="padding:0;clear:both;margin:0;border:0"></div>'.
                     '<br />'.
                   '<span class="LC_nobreak">'.$desc{'returnurl'}.'?'.                    '<span class="LC_nobreak">'.$desc{'returnurl'}.'?'.
                   '<label><input type="radio" name="linkprot_returnurl_add" value="0"'.                    '<label><input type="radio" name="linkprot_returnurl_add" value="0"'.
                   $onclickreturnurl.$checkedreturnurl{'no'}.$disabled.' />'.&mt('No').'</label>&nbsp;'.                    $onclickreturnurl.$checkedreturnurl{'no'}.$disabled.' />'.&mt('No').'</label>&nbsp;'.
Line 7469  sub linkprot_names { Line 7578  sub linkprot_names {
                                           'lifetime'  => 'Nonce lifetime (s)',                                            'lifetime'  => 'Nonce lifetime (s)',
                                           'name'      => 'Launcher Application',                                            'name'      => 'Launcher Application',
                                           'secret'    => 'Secret',                                            'secret'    => 'Secret',
                                             'passback'  => 'Can return grades to Launcher',
                                           'returnurl' => 'Launcher return URL',                                            'returnurl' => 'Launcher return URL',
                                           'requser'   => 'Use identity',                                            'requser'   => 'Use identity',
                                           'email'     => 'Email address',                                            'email'     => 'Email address',
Line 8224  sub devalidate_remote_courseprefs { Line 8334  sub devalidate_remote_courseprefs {
     my %servers = &Apache::lonnet::internet_dom_servers($cdom);      my %servers = &Apache::lonnet::internet_dom_servers($cdom);
     my %thismachine;      my %thismachine;
     map { $thismachine{$_} = 1; } &Apache::lonnet::current_machine_ids();      map { $thismachine{$_} = 1; } &Apache::lonnet::current_machine_ids();
     my @posscached = ('courselti');      my @posscached = ('courselti','courseltitools');
     if (keys(%servers)) {      if (keys(%servers)) {
         foreach my $server (keys(%servers)) {          foreach my $server (keys(%servers)) {
             next if ($thismachine{$server});              next if ($thismachine{$server});

Removed from v.1.119  
changed lines
  Added in v.1.123


FreeBSD-CVSweb <freebsd-cvsweb@FreeBSD.org>