--- loncom/publisher/lonretrieve.pm	2011/10/23 01:27:34	1.43
+++ loncom/publisher/lonretrieve.pm	2011/10/31 01:30:38	1.45
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Handler to retrieve an old version of a file
 #
-# $Id: lonretrieve.pm,v 1.43 2011/10/23 01:27:34 www Exp $
+# $Id: lonretrieve.pm,v 1.45 2011/10/31 01:30:38 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -121,7 +121,7 @@ sub phaseone {
     
     if (-e $resfn) {  
 	$r->print('<form action="/adm/retrieve" method="post">'.
-		  '<input type="hidden" name="filename" value="/~'.$uname.$fn.'" />'.
+		  '<input type="hidden" name="filename" value="/priv/'.$udom.'/'.$uname.$fn.'" />'.
 		  '<input type="hidden" name="phase" value="two" />'.
 		  &Apache::loncommon::start_data_table().
 		  &Apache::loncommon::start_data_table_header_row().
@@ -162,8 +162,8 @@ sub phaseone {
 		if ($is_meta
 		    || &Apache::loncommon::fileembstyle($suffix) eq 'ssi') {
 		    $r->print(
-			      '<td><a target="cat" href="/adm/diff?filename=/~'.
-			      $uname.$fn.
+			      '<td><a target="cat" href="/adm/diff?filename=/priv/'.
+			      $udom,'/'.$uname.$fn.
 			      '&amp;versiontwo=priv&amp;versionone='.$version.
 			      '">'.&mt('Diffs with Version').' '.$version.
 			      '</a></td>');
@@ -185,8 +185,8 @@ sub phaseone {
 	if ($is_meta 
 	    || &Apache::loncommon::fileembstyle($suffix) eq 'ssi') {
 	    $r->print(
-		      '<td><a target="cat" href="/adm/diff?filename=/~'.
-		      $uname.$fn.
+		      '<td><a target="cat" href="/adm/diff?filename=/priv/'.
+		      $udom.'/'.$uname.$fn.
 		      '&amp;versiontwo=priv'.
 		      '">'.&mt('Diffs with current Version').'</a></td>');
 	}
@@ -209,7 +209,7 @@ sub phaseone {
               .'/';
     $r->print(&Apache::lonhtmlcommon::start_funclist()
              .&Apache::lonhtmlcommon::add_item_funclist(
-                  '<a href="/priv/'.$uname.$fn.'">'
+                  '<a href="/priv/'.$udom.'/'.$uname.$fn.'">'
                  .&mt('Back to Resource')
                  .'</a>')
              .&Apache::lonhtmlcommon::add_item_funclist(
@@ -266,7 +266,7 @@ sub phasetwo {
             print $logfile "Copy failed: $error\n\n";
         }
         $r->print('</p>'
-                 .'<p><a href="/priv/'.$uname.$fn.'">'
+                 .'<p><a href="/priv/'.$udom.'/'.$uname.$fn.'">'
                  .&mt('Back to Resource')
                  .'</a></p>');
     } else {
@@ -319,9 +319,8 @@ sub handler {
   my $uname;
   my $udom;
 
-  ($uname,$udom)=
-    &Apache::loncacc::constructaccess($fn,$r->dir_config('lonDefDomain'));
-  unless (($uname) && ($udom)) {
+  ($uname,$udom) = &Apache::loncacc::constructaccess($fn);
+  unless (($uname ne '') && ($udom ne '')) {
      $r->log_reason($uname.' at '.$udom.
          ' trying to publish file '.$env{'form.filename'}.
          ' ('.$fn.') - not authorized', 
@@ -329,7 +328,7 @@ sub handler {
      return HTTP_NOT_ACCEPTABLE;
   }
 
-  $fn=~s{/~($LONCAPA::username_re)}{};
+  $fn=~s{/priv/$LONCAPA::domain_re/$LONCAPA::username_re}{};
 
   &Apache::loncommon::content_type($r,'text/html');
   $r->send_http_header;