--- rat/lonwrapper.pm	2020/07/18 20:34:58	1.49.2.7.2.3
+++ rat/lonwrapper.pm	2017/11/30 14:41:30	1.65
@@ -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.7.2.3 2020/07/18 20:34:58 raeburn Exp $
+# $Id: lonwrapper.pm,v 1.65 2017/11/30 14:41:30 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -43,20 +43,19 @@ use HTML::Entities();
 
 # ================================================================ Main Handler
 sub wrapper {
-    my ($r,$url,$brcrum,$absolute,$is_ext,$is_pdf,$exttool,$linktext,$explanation,
+    my ($url,$brcrum,$absolute,$is_ext,$is_pdf,$exttool,$linktext,$explanation,
         $title,$width,$height) = @_;
 
     my $forcereg;
     unless ($env{'form.folderpath'}) {
         $forcereg = 1;
     }
-
     my %lt = &Apache::lonlocal::texthash(
                                           'noif' => 'No iframe support.',
                                           'show' => 'Show content in pop-up window',
                                         );
 
-    my ($anchor,$uselink);
+    my $anchor;
     if ($is_ext) {
         if ($env{'form.symb'}) {
             (undef,undef,my $res) = &Apache::lonnet::decode_symb($env{'form.symb'});
@@ -66,12 +65,6 @@ sub wrapper {
         } elsif ($env{'form.anchor'} ne '') {
             $anchor = '#'.$env{'form.anchor'};
         }
-        unless (($is_pdf) && ($env{'browser.mobile'})) {
-            my $hostname = $r->hostname();
-            my $lonhost = $r->dir_config('lonHostID');
-            my $ip = &Apache::lonnet::get_host_ip($lonhost);
-            $uselink = &Apache::loncommon::is_nonframeable($url,$absolute,$hostname,$ip);
-        }
     }
 
     my $noiframe = &Apache::loncommon::modal_link($url.$anchor,$lt{'show'},500,400);
@@ -89,43 +82,25 @@ sub wrapper {
         $args->{'only_body'} = $env{'form.only_body'};
     }
 
-    my $headjs;
-
-#
-# Where iframe is in use, if window.onload() executes before the custom resize function
-# has been defined (jQuery), two global javascript vars (LCnotready and LCresizedef)
-# are used to ensure document.ready() triggers a call to resize, so the iframe contents
-# do not obscure the Functions menu.
-#
-
-    unless (($env{'browser.mobile'}) || ($exttool eq 'window') || ($exttool eq 'tab') || $uselink) {
-        $headjs = '
-<script type="text/javascript">
-// <![CDATA[
-var LCnotready = 0;
-var LCresizedef = 0;
-// ]]>
-</script>'."\n";
-    }
-
-    my $startpage = &Apache::loncommon::start_page('Menu',$headjs,$args);
+    my $startpage = &Apache::loncommon::start_page('Menu',undef,$args);
     my $endpage = &Apache::loncommon::end_page();
-
-    if (($uselink) && ($title eq '')) {
-        if ($env{'form.symb'}) {
-            $title=&Apache::lonnet::gettitle($env{'form.symb'});
-        } else {
-            my $symb=&Apache::lonnet::symbread($r->uri);
-            if ($symb) {
-                $title=&Apache::lonnet::gettitle($symb);
-            }
-        }
-    }
+    
     if (($env{'browser.mobile'}) || ($exttool eq 'window') || ($exttool eq 'tab')) {
         my $output = $startpage;
         if ($is_pdf) {
-            $linktext = &mt('Link to PDF (for mobile devices)');
-            $output .= &create_link($url,$anchor,$title,$linktext);
+            if ($title eq '') {
+                $title = $env{'form.title'};
+                if ($title eq '') {
+                    unless ($env{'request.enc'}) {
+                        ($title) = ($url =~ m{/([^/]+)$});
+                        $title =~ s/(\?[^\?]+)$//;
+                    }
+                }
+            }
+            unless ($title eq '') {
+                $output .= $title.'<br />';
+            }
+            $output .= '<a href="'.$url.'">'.&mt('Link to PDF (for mobile devices)').'</a>';
         } elsif (($exttool eq 'window') || ($exttool eq 'tab')) {
             if ($linktext eq '') {
                 $linktext = &mt('Launch External Tool');
@@ -140,9 +115,9 @@ var LCresizedef = 0;
                 $output .= <<"ENDLINK";
 <script type="text/javascript">
 // <![CDATA[
-var windowObjectReference = null;
-var PreviousUrl;
-
+var windowObjectReference = null; 
+var PreviousUrl; 
+                           
 function openSinglePopup(strUrl) {
     if (windowObjectReference == null || windowObjectReference.closed) {
         windowObjectReference = window.open(strUrl, "LCExternalToolPopUp",
@@ -168,23 +143,15 @@ ENDLINK
                 $output .= '<div>'.$explanation.'</div>';
             }
         } else {
-            if ($uselink) {
-                $linktext = &mt('Link to resource');
-                $output .= &create_link($url,$anchor,$title,$linktext);
-            } else {
-                my $dest = &HTML::Entities::encode($url.$anchor,'&<>"');
-                $output .= '<div style="overflow:scroll; -webkit-overflow-scrolling:touch;">'."\n".
-                           '<iframe src="'.$dest.'" height="100%" width="100%" frameborder="0">'."\n".
-                           "$lt{'noif'} $noiframe\n".
-                           "</iframe>\n".
-                           "</div>\n";
-            }
+            my $dest = &HTML::Entities::encode($url.$anchor,'&<>"');
+            $output .= '<div style="overflow:scroll; -webkit-overflow-scrolling:touch;">'."\n".
+                       '<iframe src="'.$dest.'" height="100%" width="100%" frameborder="0">'."\n".
+                       "$lt{'noif'} $noiframe\n".
+                       "</iframe>\n".
+                       "</div>\n";
         }
         $output .= $endpage;
         return $output;
-    } elsif ($uselink) {
-        $linktext = &mt('Link to resource');
-        return $startpage.&create_link($url,$anchor,$title,$linktext).$endpage;
     } else {
         my $offset = 5;
         &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},['inhibitmenu']);
@@ -213,20 +180,8 @@ ENDLINK
                 var pos = height + hdrtop + offset;
                 \$('.LC_iframecontainer').css('top', pos);
             });
-            LCresizedef = 1;
-            if (LCnotready == 1) {
-                LCnotready = 0;
-                \$(window).trigger('resize');
-            }
         });
-        window.onload = function(){
-            if (LCresizedef) {
-                LCnotready = 0;
-                \$(window).trigger('resize');
-            } else {
-                LCnotready = 1;
-            }
-        };
+        window.onload = function(){  \$(window).trigger('resize') };
 SCRIPT
         # javascript will position the iframe if window was resized (or zoomed)
         my $dest = &HTML::Entities::encode($url.$anchor,'&<>"');
@@ -241,26 +196,6 @@ ENDFRAME
     }
 }
 
-sub create_link {
-    my ($url,$anchor,$title,$linktext) = @_;
-    my $shownlink;
-    if ($title eq '') {
-        $title = $env{'form.title'};
-        if ($title eq '') {
-            unless ($env{'request.enc'}) {
-                ($title) = ($url =~ m{/([^/]+)$});
-                $title =~ s/(\?[^\?]+)$//;
-            }
-        }
-    }
-    unless ($title eq '') {
-        $shownlink = '<span style="font-weight:bold;">'.$title.'</span><br />';
-    }
-    my $dest = &HTML::Entities::encode($url.$anchor,'&<>"');
-    $shownlink .= '<a href="'.$dest.'">'.$linktext.'</a>';
-    return $shownlink;
-}
-
 sub handler {
     my $r=shift;
     &Apache::loncommon::content_type($r,'text/html');
@@ -274,11 +209,12 @@ sub handler {
 
     for ($url){
         s|^/adm/wrapper||;
-        $is_ext = $_ =~ s|^/ext/|http://|;
-        s|http://https://|https://| if ($is_ext);
-        s|&colon;|:|g;
+        $is_ext = $_ =~ s|^/ext/|http://|;         
+        s|http://https://|https://|;
+        s|&colon;|:|g;              
     }
 
+
     if ($url =~ /\.pdf$/i) {
         $is_pdf = 1;
     } elsif ($url =~ m{^/adm/($match_domain)/($match_courseid)/(\d+)/ext\.tool$}) {
@@ -300,10 +236,10 @@ sub handler {
             $width = $toolhash{'width'};
             $height = $toolhash{'height'};
         } elsif ($toolhash{'target'} eq 'tab') {
-            $exttool = 'tab';
+            $exttool = 'tab'; 
         }
         if (($exttool eq 'window') || ($exttool eq 'tab')) {
-            $linktext = $toolhash{'linktext'};
+            $linktext = $toolhash{'linktext'}; 
             $explanation = $toolhash{'explanation'};
         } elsif (($exttoolremote =~ /^http:/) && ($ENV{'SERVER_PORT'} == 443)) {
             $exttool = 'tab';
@@ -381,7 +317,7 @@ sub handler {
             &Apache::lonenc::check_encrypt(\$url);
         }
 
-        $r->print( wrapper($r,$url,$brcrum,$absolute,$is_ext,$is_pdf,$exttool,
+        $r->print( wrapper($url,$brcrum,$absolute,$is_ext,$is_pdf,$exttool,
                            $linktext,$explanation,undef,$width,$height) );
 
     } # not just the menu
@@ -409,14 +345,10 @@ described at http://www.lon-capa.org.
 
 =over
 
-=item wrapper($r,$url,$brcrum,$absolute,$is_ext,$is_pdf,$exttool,$linktext,$explanation,$title,$width,$height)
+=item wrapper($url,$brcrum,$absolute,$is_ext,$is_pdf,$linktext,$explanation,$title,$width,$height)
 
 =over
 
-=item $r
-
-request object
-
 =item $url
 
 url to display by including in an iframe within a