--- rat/lonwrapper.pm	2023/10/06 17:35:54	1.49.2.11.2.5
+++ rat/lonwrapper.pm	2022/08/30 11:41:27	1.79
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Wrapper for external and binary files as standalone resources
 #
-# $Id: lonwrapper.pm,v 1.49.2.11.2.5 2023/10/06 17:35:54 raeburn Exp $
+# $Id: lonwrapper.pm,v 1.79 2022/08/30 11:41:27 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -47,7 +47,7 @@ use Digest::MD5();
 # ================================================================ Main Handler
 sub wrapper {
     my ($r,$url,$brcrum,$absolute,$is_ext,$is_pdf,$exttool,$linktext,$explanation,
-        $title,$width,$height,$reuse,$is_supp) = @_;
+        $title,$width,$height,$reuse) = @_;
 
     my $forcereg;
     unless ($env{'form.folderpath'}) {
@@ -95,8 +95,6 @@ sub wrapper {
     }
     if ($env{'form.only_body'}) {
         $args->{'only_body'} = $env{'form.only_body'};
-    } elsif (($is_supp) && ($env{'form.folderpath'} ne '')) {
-        $args->{'bread_crumbs_nomenu'} = 1;
     }
 
     my ($countdown,$donemsg,$headjs);
@@ -168,7 +166,7 @@ var LCresizedef = 0;
     if (($uselink) && ($title eq '')) {
         if ($env{'form.symb'}) {
             $title=&Apache::lonnet::gettitle($env{'form.symb'});
-        } elsif (!$is_supp) {
+        } else {
             my $symb=&Apache::lonnet::symbread($r->uri);
             if ($symb) {
                 $title=&Apache::lonnet::gettitle($symb);
@@ -227,7 +225,7 @@ var LCresizedef = 0;
                     unless ($reuse) {
                         my $resid;
                         if ($env{'request.course.id'}) {
-                            unless (($is_supp) || ($env{'form.folderpath'} =~ /^supplemental/)) {
+                            unless ($env{'form.folderpath'} =~ /^supplemental/) {
                                 my $symb=&Apache::lonnet::symbread($r->uri);
                                 if ($symb) {
                                     my $navmap = Apache::lonnavmaps::navmap->new();
@@ -304,7 +302,7 @@ ENDLINK
     } else {
         my $offset = 5;
         &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},['inhibitmenu']);
-        if (($env{'form.inhibitmenu'} eq 'yes') || ($env{'form.only_body'})) {
+        if ($env{'form.inhibitmenu'} eq 'yes') {
             $offset = 0;
         }
         my $script = &Apache::lonhtmlcommon::scripttag(<<SCRIPT);
@@ -336,12 +334,12 @@ ENDLINK
             }
         });
         window.onload = function(){
-            if (LCresizedef) {
-                LCnotready = 0;
-                \$(window).trigger('resize');
-            } else {
-                LCnotready = 1;
-            }
+             if (LCresizedef) {
+                 LCnotready = 0;
+                 \$(window).trigger('resize');
+             } else {
+                 LCnotready = 1;
+             }
         };
 SCRIPT
         # javascript will position the iframe if window was resized (or zoomed)
@@ -386,7 +384,7 @@ sub handler {
 
     my $url = $r->uri;
     my ($is_ext,$brcrum,$absolute,$is_pdf,$exttool,$cdom,$cnum,$hostname,
-        $linktext,$explanation,$width,$height,$reuse,$is_supp);
+        $linktext,$explanation,$width,$height,$reuse);
 
     for ($url){
         s|^/adm/wrapper||;
@@ -427,16 +425,9 @@ sub handler {
         my %toolhash = &Apache::lonnet::get('exttool_'.$marker,['target','linktext','explanation','id','width','height'],
                                             $cdom,$cnum);
         if ($toolhash{'id'}) {
-            my ($idx,%ltitools);
-            if ($toolhash{'id'} =~ /^c(\d+)$/) {
-                $idx = $1;
-                %ltitools = &Apache::lonnet::get_course_lti($cnum,$cdom,'consumer');
-            } else {
-                $idx = $toolhash{'id'};
-                %ltitools = &Apache::lonnet::get_domain_lti($cdom,'consumer');
-            }
-            if (ref($ltitools{$idx}) eq 'HASH') {
-                $exttoolremote = $ltitools{$idx}{'url'};
+            my %ltitools = &Apache::lonnet::get_domain_lti($cdom,'consumer');
+            if (ref($ltitools{$toolhash{'id'}}) eq 'HASH') {
+                $exttoolremote = $ltitools{$toolhash{'id'}}{'url'};
             }
         }
         if ($toolhash{'target'} eq 'window') {
@@ -455,7 +446,7 @@ sub handler {
     }
     if (($is_ext) || ($exttool)) {
         &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},
-            ['forceedit','register','folderpath','symb','idx','title','anchor','only_body']);
+            ['forceedit','register','folderpath','symb','idx','title','anchor']);
         if (($env{'form.forceedit'}) &&
             (&Apache::lonnet::allowed('mdc',$env{'request.course.id'})) &&
             (($env{'form.folderpath'} =~ /^supplemental/) ||
@@ -469,7 +460,6 @@ sub handler {
                 if ($env{'form.anchor'} ne '') {
                     $url .= '#'.$env{'form.anchor'};
                 }
-                $is_supp = 1;
             }
             my $type = 'ext';
             if ($exttool) {
@@ -493,37 +483,8 @@ sub handler {
                     $title = &mt('External Tool');
                 }
             }
-            $title = &HTML::Entities::encode($title,'\'"<>&');
-            $is_supp = 1;
-            if ($env{'request.course.id'}) {
-                $cnum = $env{'course.'.$env{'request.course.id'}.'.num'};
-                $cdom = $env{'course.'.$env{'request.course.id'}.'.domain'};
-                &Apache::loncommon::validate_folderpath(1,'',$cnum,$cdom);
-            }
-            unless ($env{'form.only_body'}) { 
-                $brcrum =
-                    &Apache::lonhtmlcommon::docs_breadcrumbs(undef,$crstype,undef,$title,1);
-            }
-        }
-    } elsif ($env{'request.course.id'}) {
-        my $courseurl = &Apache::lonnet::courseid_to_courseurl($env{'request.course.id'});
-        $courseurl =~ s{^/}{};
-        if ($url =~ m{^\Q/uploaded/$courseurl/supplemental/\E}) {
-            $is_supp = 1;
-            &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},
-                                                    ['folderpath','title','only_body']);
-            if ($env{'form.folderpath'}) {
-                $cnum = $env{'course.'.$env{'request.course.id'}.'.num'};
-                $cdom = $env{'course.'.$env{'request.course.id'}.'.domain'};
-                &Apache::loncommon::validate_folderpath(1,'',$cnum,$cdom);
-            }
-            my $title = $env{'form.title'};
-            $title = &HTML::Entities::encode($title,'\'"<>&');
-            my $crstype = &Apache::loncommon::course_type();
-            unless ($env{'form.only_body'}) {
-                $brcrum =
-                    &Apache::lonhtmlcommon::docs_breadcrumbs(undef,$crstype,undef,$title,1);
-            }
+            $brcrum =
+                &Apache::lonhtmlcommon::docs_breadcrumbs(undef,$crstype,undef,$title,1);
         }
     }
 
@@ -556,8 +517,7 @@ sub handler {
         }
 
         $r->print( wrapper($r,$url,$brcrum,$absolute,$is_ext,$is_pdf,$exttool,
-                           $linktext,$explanation,undef,$width,$height,$reuse,
-                           $is_supp) );
+                           $linktext,$explanation,undef,$width,$height,$reuse) );
 
     } # not just the menu
     
@@ -584,7 +544,7 @@ described at http://www.lon-capa.org.
 
 =over
 
-=item wrapper($r,$url,$brcrum,$absolute,$is_ext,$is_pdf,$exttool,$linktext,$explanation,$title,$width,$height,$reuse,$is_supp)
+=item wrapper($r,$url,$brcrum,$absolute,$is_ext,$is_pdf,$exttool,$linktext,$explanation,$title,$width,$height,$reuse)
 
 =over