--- loncom/publisher/lonupload.pm	2011/10/26 22:38:49	1.55
+++ loncom/publisher/lonupload.pm	2011/11/14 00:20:31	1.59
@@ -1,8 +1,7 @@
-
 # The LearningOnline Network with CAPA
 # Handler to upload files into construction space
 #
-# $Id: lonupload.pm,v 1.55 2011/10/26 22:38:49 www Exp $
+# $Id: lonupload.pm,v 1.59 2011/11/14 00:20:31 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -163,7 +162,7 @@ sub upfile_store {
 }
 
 sub phaseone {
-    my ($r,$fn,$uname,$udom,$mode)=@_;
+    my ($r,$fn,$mode)=@_;
     my $action = '/adm/upload';
     if ($mode eq 'testbank') {
         $action = '/adm/testbank';
@@ -240,7 +239,7 @@ sub phaseone {
 }
 
 sub phasetwo {
-    my ($r,$fn,$uname,$udom,$mode)=@_;
+    my ($r,$fn,$mode)=@_;
 
     my $output;
     my $action = '/adm/upload';
@@ -252,7 +251,7 @@ sub phasetwo {
     }
     $fn=~s/\/+/\//g;
     if ($fn) {
-	my $target='/home/httpd/html/'.$fn;
+	my $target= $r->dir_config('lonDocRoot').'/'.$fn;
 	&Debug($r, "target -> ".$target);
 #     target is the full filesystem path of the destination file.
 	my $base = &File::Basename::basename($fn);
@@ -404,9 +403,10 @@ sub phasethree {
     } elsif ($mode eq 'imsimport') {
         $action = '/adm/imsimport';
     }
-    my $dir_root = '/home/httpd/html/priv/'.$udom.'/'.$uname;
-    my $url_root = '/priv/'.$udom.'/'.$uname;
+    my $url_root = "/priv/$udom/$uname";
+    my $dir_root = $r->dir_config('lonDocRoot').$url_root;
     my $path = &File::Basename::dirname($fn);
+    $path =~ s{^\Q$url_root\E}{};
     my $filename = &HTML::Entities::encode($env{'form.filename'},'<>&"');
     my $state = &embedded_form_elems('modify_orightml',$filename,$mode).
                 '<input type="hidden" name="phase" value="four" />';
@@ -415,7 +415,7 @@ sub phasethree {
                                             $dir_root,$url_root,undef,
                                             undef,undef,$state,$action);
     if ($mode ne 'imsimport' && $mode ne 'testbank') {
-        $result .= '<br /><h3><a href="'.$url_root.$fn.'">'.
+        $result .= '<br /><h3><a href="'.$fn.'">'.
                   &mt('View main file').'</a></h3>'.
                   '<h3><a href="'.$url_root.$path.'">'.
                   &mt('Back to Directory').'</a></h3><br />';
@@ -442,13 +442,14 @@ sub phasefour {
         $action = '/adm/imsimport';
     }
     my $result;
-    my $dir_root = '/home/httpd/html/priv/'.$udom.'/'.$uname;
-    my $url_root = '/priv/'.$udom.'/'.$uname;
+    my $url_root = "/priv/$udom/$uname";
+    my $dir_root = $r->dir_config('lonDocRoot').$url_root;
     my $path = &File::Basename::dirname($fn);
+    $path =~ s{^\Q$url_root\E}{};
     $result .= &Apache::loncommon::modify_html_refs($mode,$path,
                               $uname,$udom,$dir_root);
     if ($mode ne 'imsimport' && $mode ne 'testbank') {
-        $result .= '<br /><h3><a href="'.$url_root.$fn.'">'.
+        $result .= '<br /><h3><a href="'.$fn.'">'.
                   &mt('View main file').'</a></h3>'.
                   '<h3><a href="'.$url_root.$path.'">'.
                   &mt('Back to Directory').'</a></h3><br />';
@@ -460,11 +461,7 @@ sub phasefour {
 sub handler {
 
     my $r=shift;
-
-    my $uname;
-    my $udom;
     my $javascript = '';
-
     my $fn=$env{'form.filename'};
 
     if ($env{'form.filename1'}) {
@@ -478,8 +475,25 @@ sub handler {
         return HTTP_NOT_FOUND;
     }
 
+    my ($uname,$udom)=&Apache::loncacc::constructaccess($fn);
+
+    unless (($uname) && ($udom)) {
+        $r->log_reason($uname.' at '.$udom.
+                       ' trying to publish file '.$env{'form.filename'}.
+                       ' - not authorized',
+                       $r->filename);
+        return HTTP_NOT_ACCEPTABLE;
+    }
+
+# ----------------------------------------------------------- Start page output
+
+    &Apache::loncommon::content_type($r,'text/html');
+    $r->send_http_header;
+
     unless ($env{'form.phase'} eq 'two') {
-        $javascript = qq|
+        $javascript = <<"ENDJS";
+<script type="text/javascript">
+// <![CDATA[
 function verifyForm() {
     var mode = document.fileupload.filetype.options[document.fileupload.filetype.selectedIndex].value
     if (mode == "testbank") {
@@ -493,29 +507,17 @@ function verifyForm() {
     }
     document.fileupload.submit();
 }
-	|;
+// ]]>
+</script>
+ENDJS
     }
-    ($uname,$udom)=&Apache::loncacc::constructaccess($fn);
-
-    unless (($uname) && ($udom)) {
-	$r->log_reason($uname.' at '.$udom.
-		       ' trying to publish file '.$env{'form.filename'}.
-		       ' - not authorized', 
-		       $r->filename); 
-	return HTTP_NOT_ACCEPTABLE;
-    }
-    
-# ----------------------------------------------------------- Start page output
-
-
-    &Apache::loncommon::content_type($r,'text/html');
-    $r->send_http_header;
 
-   $javascript = "<script type=\"text/javascript\">\n//<!--\n".
-	$javascript."\n// --></script>\n";
+    my $londocroot = $r->dir_config('lonDocRoot');
+    my $trailfile = $fn;
+    $trailfile =~ s{^/(priv/)}{$londocroot/$1};
 
     # Breadcrumbs
-    my $brcrum = [{'href' => &Apache::loncommon::authorspace(),
+    my $brcrum = [{'href' => &Apache::loncommon::authorspace($fn),
                    'text' => 'Construction Space'},
                   {'href' => '/adm/upload',
                    'text' => 'Upload file to Construction Space'}];
@@ -523,7 +525,7 @@ function verifyForm() {
                                              $javascript,
                                              {'bread_crumbs' => $brcrum,})
              .&Apache::loncommon::head_subbox(
-                &Apache::loncommon::CSTR_pageheader())
+                &Apache::loncommon::CSTR_pageheader($trailfile))
     );
   
     if (($uname ne $env{'user.name'}) || ($udom ne $env{'user.domain'})) {
@@ -539,10 +541,10 @@ function verifyForm() {
         my ($output,$rtnflag) = &phasethree($r,$fn,$uname,$udom,'author');
         $r->print($output);
     } elsif ($env{'form.phase'} eq 'two') {
-	my ($output,$returnflag) = &phasetwo($r,$fn,$uname,$udom);
+	my ($output,$returnflag) = &phasetwo($r,$fn);
         $r->print($output);
     } else {
-	&phaseone($r,$fn,$uname,$udom);
+	&phaseone($r,$fn);
     }
 
     $r->print(&Apache::loncommon::end_page());