--- rat/lonwrapper.pm	2014/05/20 11:47:06	1.47
+++ rat/lonwrapper.pm	2016/08/05 21:34:54	1.49.2.1
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Wrapper for external and binary files as standalone resources
 #
-# $Id: lonwrapper.pm,v 1.47 2014/05/20 11:47:06 raeburn Exp $
+# $Id: lonwrapper.pm,v 1.49.2.1 2016/08/05 21:34:54 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -40,7 +40,7 @@ use Apache::lonextresedit();
 
 # ================================================================ Main Handler
 sub wrapper {
-    my ($url,$brcrum,$absolute,$is_ext,$is_pdf) = @_;
+    my ($url,$brcrum,$absolute,$is_ext,$is_pdf,$title) = @_;
 
     my $forcereg;
     unless ($env{'form.folderpath'}) {
@@ -52,7 +52,15 @@ sub wrapper {
                                           'show' => 'Show content in pop-up window',
                                         );
 
-    my $noiframe = &Apache::loncommon::modal_link($url,$lt{'show'},500,400);
+    my $anchor;
+    if (($is_ext) && ($env{'form.symb'})) {
+        (undef,undef,my $res) = &Apache::lonnet::decode_symb($env{'form.symb'});
+        if ($res =~ /(\#.+)$/) {
+            $anchor = $1;
+        }
+    }
+
+    my $noiframe = &Apache::loncommon::modal_link($url.$anchor,$lt{'show'},500,400);
     my $args = {'bgcolor' => '#FFFFFF'};
     if ($forcereg) {
         $args->{'force_register'} = $forcereg;
@@ -70,11 +78,13 @@ sub wrapper {
     if ($env{'browser.mobile'}) {
         my $output = $startpage;
         if ($is_pdf) {
-            my $title = $env{'form.title'};
             if ($title eq '') {
-                unless ($env{'request.enc'}) {
-                    ($title) = ($url =~ m{/([^/]+)$});
-                    $title =~ s/(\?[^\?]+)$//;
+                $title = $env{'form.title'};
+                if ($title eq '') {
+                    unless ($env{'request.enc'}) {
+                        ($title) = ($url =~ m{/([^/]+)$});
+                        $title =~ s/(\?[^\?]+)$//;
+                    }
                 }
             }
             unless ($title eq '') {
@@ -83,13 +93,10 @@ sub wrapper {
             $output .= '<a href="'.$url.'">'.&mt('Link to PDF (for mobile devices)').'</a>';
         } else {
             $output .= '<div style="overflow:scroll; -webkit-overflow-scrolling:touch;">'."\n".
-                       '<iframe src="$url" height="100%" width="100%" frameborder="0">'."\n".
+                       '<iframe src="'.$url.$anchor.'" height="100%" width="100%" frameborder="0">'."\n".
                        "$lt{'noif'} $noiframe\n".
                        "</iframe>\n".
                        "</div>\n";
-            $output .= '<iframe src="$url" height="100%" width="100%" frameborder="0">'."\n".
-                       "$lt{'noif'} $noiframe\n".
-                       '</iframe>';
         }
         $output .= $endpage;
         return $output;
@@ -124,7 +131,7 @@ SCRIPT
         $startpage
         $script
         <div class="LC_iframecontainer">
-            <iframe src="$url">$lt{'noif'} $noiframe</iframe>
+            <iframe src="$url$anchor">$lt{'noif'} $noiframe</iframe>
         </div>
         $endpage
 ENDFRAME
@@ -153,15 +160,6 @@ sub handler {
     }
  
     if ($is_ext) {
-        my $hostname = $r->hostname();
-        my $lonhost = &Apache::lonnet::host_from_dns($hostname);
-        if ($lonhost) {
-            my $actual = &Apache::lonnet::absolute_url($hostname);
-            my $expected = $Apache::lonnet::protocol{$lonhost}.'://'.$hostname; 
-            unless ($actual eq $expected) {
-                $absolute = $expected;
-            }
-        }
         &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},
             ['forceedit','register','folderpath','symb','idx','title']);
         if (($env{'form.forceedit'}) &&
@@ -198,6 +196,7 @@ sub handler {
 # This is not homework
 #
         if ($is_ext) {
+            $absolute = $env{'request.use_absolute'};
             $ENV{'QUERY_STRING'} =~ s/(^|\&)symb=[^\&]*/$1/;
             $ENV{'QUERY_STRING'} =~ s/\&$//;
         }
@@ -236,7 +235,7 @@ described at http://www.lon-capa.org.
 
 =over
 
-=item wrapper($url,$brcrum,$absolute,$is_ext,$is_pdf))
+=item wrapper($url,$brcrum,$absolute,$is_ext,$is_pdf,$title))
 
 =over
 
@@ -261,7 +260,9 @@ relative URLs.
 That will be the case where an external resource has been 
 served from port 80, when the server customarily serves
 requests using Apache/SSL (i.e., port 443). mod_rewrite 
-is used to switch requests for external resources 
+is used to switch requests for external resources and
+the syllabus: /public/<domain>/<courseid>/syllabus
+(which might also point at an external resource)
 from https:// to http:// where the the URL of the remote site 
 specified in the resource itself is http://.
 
@@ -276,6 +277,14 @@ true if URL is for an external resource.
 
 true if URL is for a PDF (based on file extension).
 
+=item $title
+
+optional. If wrapped item is a PDF, and $env{'browser.mobile'} 
+is true, a link to a PDF is shown. The "title" will be displayed
+above the link, but if not provided as an arg, $env{'form.title'}
+will be used, otherwise, the filename will be displayed (unless
+hidden URL set for the resource).
+
 =back
 
 Returns markup for the entire page.