--- loncom/publisher/lonretrieve.pm	2011/10/31 01:30:38	1.45
+++ loncom/publisher/lonretrieve.pm	2012/10/29 17:38:55	1.49
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Handler to retrieve an old version of a file
 #
-# $Id: lonretrieve.pm,v 1.45 2011/10/31 01:30:38 raeburn Exp $
+# $Id: lonretrieve.pm,v 1.49 2012/10/29 17:38:55 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -99,7 +99,6 @@ use strict;
 use Apache::File;
 use File::Copy;
 use Apache::Constants qw(:common :http :methods);
-use Apache::loncacc;
 use Apache::loncommon();
 use Apache::lonlocal;
 use Apache::lonnet;
@@ -108,19 +107,17 @@ use LONCAPA();
 # ------------------------------------ Interface for selecting previous version
 sub phaseone {
     my ($r,$fn,$uname,$udom)=@_;
-    my $docroot=$r->dir_config('lonDocRoot');
 
-    my $urldir='/res/'.$udom.'/'.$uname.$fn;
-    $urldir=~s/\/[^\/]+$/\//;
+    my $urldir = "/res/$udom/$uname".$fn;
+    my $resfn = $r->dir_config('lonDocRoot').$urldir;
 
-    my $resfn=$docroot.'/res/'.$udom.'/'.$uname.$fn;
-    my $resdir=$resfn;
-    $resdir=~s/\/[^\/]+$/\//;
+    $urldir =~ s{[^/]+$}{};
+    my $resdir = $r->dir_config('lonDocRoot').$urldir;
 
     my ($main,$suffix,$is_meta) = &get_file_info($fn);
     
     if (-e $resfn) {  
-	$r->print('<form action="/adm/retrieve" method="post">'.
+	$r->print('<form action="/adm/retrieve?inhibitmenu=yes" method="post">'.
 		  '<input type="hidden" name="filename" value="/priv/'.$udom.'/'.$uname.$fn.'" />'.
 		  '<input type="hidden" name="phase" value="two" />'.
 		  &Apache::loncommon::start_data_table().
@@ -156,17 +153,18 @@ sub phaseone {
               '<td>'.&Apache::lonlocal::locallocaltime($rmtime).'</td>');
 		
 		if (!$is_meta) {
-		    $r->print('<td><a href="'.$urldir.$filename.'.meta" target="cat">'.
-			      &mt('Metadata Version').' '.$version.'</a></td>');
+		    $r->print('<td>'.
+                              &Apache::loncommon::modal_link($urldir.$filename.'.meta',
+			      &mt('Metadata Version [_1]',$version),550,450).'</td>');
 		}
 		if ($is_meta
 		    || &Apache::loncommon::fileembstyle($suffix) eq 'ssi') {
-		    $r->print(
-			      '<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>');
+		    $r->print('<td>'.
+                              &Apache::loncommon::modal_link(
+                               '/adm/diff?filename=/priv/'.$udom,'/'.$uname.$fn.
+			      '&amp;versiontwo=priv&amp;versionone='.$version,
+			       &mt('Diffs with Version [_1]',$version),550,450).
+			      '</td>');
 		}
 		$r->print(&Apache::loncommon::end_data_table_row());
 	    }
@@ -179,16 +177,19 @@ sub phaseone {
           '<td>'.&Apache::lonlocal::locallocaltime($rmtime).'</td>'
     );
 	if (!$is_meta) {
-	    $r->print('<td><a href="'.$urldir.$main.'.'.$suffix.'.meta" target="cat">'.
-		      &mt('Metadata current version').'</a></td>');           
+	    $r->print('<td>',
+                      &Apache::loncommon::modal_link($urldir.$main.'.'.$suffix.'.meta',
+		      &mt('Metadata current version'),550,450).'</td>');           
 	}
 	if ($is_meta 
 	    || &Apache::loncommon::fileembstyle($suffix) eq 'ssi') {
-	    $r->print(
-		      '<td><a target="cat" href="/adm/diff?filename=/priv/'.
+	    $r->print('<td>'.
+                      &Apache::loncommon::modal_link(
+                       '/adm/diff?filename=/priv/'.
 		      $udom.'/'.$uname.$fn.
-		      '&amp;versiontwo=priv'.
-		      '">'.&mt('Diffs with current Version').'</a></td>');
+		      '&amp;versiontwo=priv',
+		      &mt('Diffs with current Version'),550,450).
+                      '</td>');
 	}
 	$r->print(&Apache::loncommon::end_data_table_row().
 		  &Apache::loncommon::end_data_table().
@@ -204,20 +205,6 @@ sub phaseone {
 	$r->print('<p class="LC_warning">'.&mt('No previous versions published.').'</p>');
     }
 
-    my $dir =  &Apache::loncommon::authorspace()
-              .&File::Basename::dirname($fn)
-              .'/';
-    $r->print(&Apache::lonhtmlcommon::start_funclist()
-             .&Apache::lonhtmlcommon::add_item_funclist(
-                  '<a href="/priv/'.$udom.'/'.$uname.$fn.'">'
-                 .&mt('Back to Resource')
-                 .'</a>')
-             .&Apache::lonhtmlcommon::add_item_funclist(
-                  '<a href="'.$dir.'">'
-                 .&mt('Back to Directory')
-                 .'</a>')
-             .&Apache::lonhtmlcommon::end_funclist()
-    );
 }
 
 # ---------------------------------- Interface for presenting specified version
@@ -233,13 +220,13 @@ sub phasetwo {
 	my ($main,$suffix,$is_meta) = &get_file_info($fn);
 
         my $logfile;
-        my $ctarget=$r->dir_config('lonDocRoot').'/priv/'.$udom.'/'.$uname.$fn;
+        my $ctarget=$r->dir_config('lonDocRoot')."/priv/$udom/$uname".$fn;
         my $vfn=$fn;
         if ($version ne 'new') {
 	    $vfn=~s/\.(\Q$suffix\E)$/\.$version\.$1/;
         }
 
-        my $csource=$r->dir_config('lonDocRoot').'/res/'.$udom.'/'.$uname.$vfn;
+        my $csource=$r->dir_config('lonDocRoot')."/res/$udom/$uname".$vfn;
 
 	my $logname = $ctarget;
 	if ($is_meta) { $logname =~ s/\.meta$//; }
@@ -258,6 +245,7 @@ sub phasetwo {
                      .&mt('ok')
                      .'</span>');
             print $logfile "Copied sucessfully.\n\n";
+            $r->print(&Apache::lonhtmlcommon::scripttag('parent.location.reload();'));
         } else {
             my $error=$!;
 	    $r->print('<span class="LC_error">'
@@ -265,10 +253,6 @@ sub phasetwo {
                      .'</span>');
             print $logfile "Copy failed: $error\n\n";
         }
-        $r->print('</p>'
-                 .'<p><a href="/priv/'.$udom.'/'.$uname.$fn.'">'
-                 .&mt('Back to Resource')
-                 .'</a></p>');
     } else {
        $r->print('<p class="LC_info">'.&mt('Please pick a version to retrieve:').'</p>');
        &phaseone($r,$fn,$uname,$udom);
@@ -302,7 +286,7 @@ sub handler {
 
   if ($env{'form.filename'}) {
       $fn=$env{'form.filename'};
-      $fn=~s/^https?\:\/\/[^\/]+//;
+      $fn =~ s{^https?\://[^/]+}{};
   } else {
      $r->log_reason($env{'user.name'}.' at '.$env{'user.domain'}.
          ' unspecified filename for retrieval', $r->filename); 
@@ -319,7 +303,7 @@ sub handler {
   my $uname;
   my $udom;
 
-  ($uname,$udom) = &Apache::loncacc::constructaccess($fn);
+  ($uname,$udom) = &Apache::lonnet::constructaccess($fn);
   unless (($uname ne '') && ($udom ne '')) {
      $r->log_reason($uname.' at '.$udom.
          ' trying to publish file '.$env{'form.filename'}.
@@ -328,8 +312,6 @@ sub handler {
      return HTTP_NOT_ACCEPTABLE;
   }
 
-  $fn=~s{/priv/$LONCAPA::domain_re/$LONCAPA::username_re}{};
-
   &Apache::loncommon::content_type($r,'text/html');
   $r->send_http_header;
 
@@ -337,19 +319,25 @@ sub handler {
     &Apache::lonhtmlcommon::clear_breadcrumbs();
     &Apache::lonhtmlcommon::add_breadcrumb({
         'text'  => 'Construction Space',
-        'href'  => &Apache::loncommon::authorspace(),
+        'href'  => &Apache::loncommon::authorspace($fn),
     });
     &Apache::lonhtmlcommon::add_breadcrumb({
         'text'  => 'Retrieve previous version',
         'href'  => '',
     });
 
+    my $londocroot = $r->dir_config('lonDocRoot');
+    my $trailfile = $fn;
+    $trailfile =~ s{^/(priv/)}{$londocroot/$1};
+
   $r->print(&Apache::loncommon::start_page('Retrieve Published Resources')
            .&Apache::lonhtmlcommon::breadcrumbs()
            .&Apache::loncommon::head_subbox(
-                &Apache::loncommon::CSTR_pageheader()) # FIXME crumbs broken?
+                &Apache::loncommon::CSTR_pageheader($trailfile))
     );
 
+  $fn=~s{/priv/$LONCAPA::domain_re/$LONCAPA::username_re}{};
+
   $r->print('<p>'
            .&mt('Retrieve previous versions of [_1]'
                    ,'<span class="LC_filename">'.$fn.'</span>')