--- loncom/publisher/londiff.pm	2004/03/11 22:44:01	1.15
+++ loncom/publisher/londiff.pm	2012/04/18 20:14:35	1.35
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Handler to show differences between file versions
 #
-# $Id: londiff.pm,v 1.15 2004/03/11 22:44:01 albertel Exp $
+# $Id: londiff.pm,v 1.35 2012/04/18 20:14:35 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -25,43 +25,31 @@
 #
 # http://www.lon-capa.org/
 #
-#
-# (Handler to retrieve an old version of a file
-#
-# (Publication Handler
-# 
-# (TeX Content Handler
-#
-# 05/29/00,05/30,10/11 Gerd Kortemeyer)
-#
-# 11/28,11/29,11/30,12/01,12/02,12/04,12/23 Gerd Kortemeyer
-# 03/23 Guy Albertelli
-# 03/24,03/29 Gerd Kortemeyer)
-#
-# 03/31,04/03 Gerd Kortemeyer)
-#
-# 05/02/01,05/09 Gerd Kortemeyer
-#
 ###
 
+
+
+
 package Apache::londiff;
 
 use strict;
 use Apache::File;
 use File::Copy;
+use File::Compare;
 use Algorithm::Diff qw(diff);
 use Apache::Constants qw(:common :http :methods);
-use Apache::loncacc;
-use Apache::lonnet();
+use Apache::loncacc();
+use Apache::lonnet;
 use Apache::loncommon();
+use Apache::lonretrieve();
 use Apache::lonlocal;
-
+use LONCAPA();
 
 sub get_split_file {
     my ($fn,$style)=@_;
     my $f1;
     my @f1;
-    if ($style='local') {
+    if ($style eq 'local') {
 	if (-e $fn) {
 	    my $fh=Apache::File->new($fn);
 	    my $line;	
@@ -70,141 +58,183 @@ sub get_split_file {
 	    }
 	}
     } elsif ($style eq 'remote') {
-	my $f1=&Apache::lonnet::getfile($fn);
-    }
-    if ($f1=~/\r/) {
-	@f1=split(/\r/,&Apache::lonnet::getfile($fn));      
-	foreach my $line (@f1) {
-	    $line=~s/\n//g;
-	}
-    } else {
-	@f1=split(/\n/,&Apache::lonnet::getfile($fn));      
+	$f1=&Apache::lonnet::getfile($fn);
     }
+    @f1=split(/\r\n|\r|\n/,$f1);
     return @f1;
 }
 
-sub handler {
+sub are_different_files {
+    my ($fileone,$filetwo)=@_;
+    return &compare($fileone,$filetwo);
+}
 
-  my $r=shift;
+sub handler {
 
+    my $r=shift;
 # Get query string for limited number of parameters
 
-   &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},
-                                       ['filename','versiontwo','versionone']);
-# Get the files
-
-  my $cuname=$ENV{'user.name'};
-  my $cudom=$ENV{'user.domain'};
+    &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},
+					    ['filename','versiontwo',
+					     'versionone','filetwo']);
+
+# Check permissions
+    my $allowed=0;
+    my $cuname=$env{'user.name'};
+    my $cudom=$env{'user.domain'};
+
+    if ($env{'form.filename'}=~ m{^/res/}) {
+        if (&Apache::lonnet::allowed('bre',$env{'form.filename'})) {
+            if ($env{'request.course.id'}) {
+                if (&Apache::lonnet::allowed('mdc',$env{'request.course.id'})) {
+                    $allowed = 1;
+                }
+            } else {
+                $allowed = 1;
+            }
+        } elsif (&Apache::lonnet::allowed('bro',$env{'form.filename'})) {
+            $allowed = 1;
+        }
+        if ($allowed) {
+            ($cudom,$cuname,$env{'form.filename'})=
+                ($env{'form.filename'}=~m{^/res/($LONCAPA::domain_re)/($LONCAPA::username_re)(/.*)$});
+
+            if (($env{'form.versionone'} eq 'priv') || ($env{'form.versiontwo'} eq 'priv')) {
+                my ($cstrname,$cstrdom) =
+                    &Apache::loncacc::constructaccess("/priv/$cudom/$cuname".$env{'form.filename'});
+                unless (($cstrname eq $cuname) && ($cstrdom eq $cudom)) {
+                    $allowed = 0;
+                }
+            }
+        }
+    } else {
+	($cuname,$cudom)=
+	    &Apache::loncacc::constructaccess($env{'form.filename'});
+        if ($cuname ne '' && $cudom ne '') {
+            $allowed = 1;
+        } else {
+	    $r->log_reason($env{'user.name'}.':'.$env{'user.domain'}.
+			   ' trying to get diffs file '.$env{'form.filename'}.
+			   '  - not authorized',
+			   $r->filename);
+	}
+    }
+    unless ($allowed) {
+        return HTTP_NOT_ACCEPTABLE;
+    }
 
-  if ($ENV{'form.filename'}=~/^\/res\//) {
-      ($cudom,$cuname,$ENV{'form.filename'})=
-       ($ENV{'form.filename'}=~/^\/res\/(\w+)\/(\w+)\/(.*)$/);
-  } else {
-   unless (($cuname,$cudom)=
-    &Apache::loncacc::constructaccess($ENV{'form.filename'},
-                                      $r->dir_config('lonDefDomain'))) {
-     $r->log_reason($cuname.' at '.$cudom.
-         ' trying to get diffs file '.$ENV{'form.filename'}.
-         '  - not authorized', 
-         $r->filename); 
-     return HTTP_NOT_ACCEPTABLE;
-   }
-  }
-
-  my $efn=$ENV{'form.filename'};
-
-  $efn=~s/\/\~(\w+)//g;
-
-  my @f1=();
-  my @f2=();
+# Get the files
 
-  &Apache::loncommon::content_type($r,'text/html');
-  $r->send_http_header;
+    my $efn=$env{'form.filename'};
+    $efn=~s{^/priv/$LONCAPA::domain_re/$LONCAPA::username_re}{};
 
-  $r->print('<html><head><title>LON-CAPA Construction Diffs</title></head>');
+    my @f1=();
+    my @f2=();
 
-  $r->print(&Apache::loncommon::bodytag('Resource Differences'));
+    &Apache::loncommon::content_type($r,'text/html');
+    $r->send_http_header;
 
+    $r->print(&Apache::loncommon::start_page('Resource Differences',undef,
+                                             {'no_nav_bar'  => 1, }));
   
-  $r->print('<h1>'.&mt('Compare versions of').' <tt>'.$efn.'</tt></h1>');
+    $r->print(($env{'form.filetwo'}?'':&mt('Compare versions of')).
+	      ' <span class="LC_filename">'.$efn.'</span>');
    
-       if (($cuname ne $ENV{'user.name'}) || ($cudom ne $ENV{'user.domain'})) {
-          $r->print('<h3><font color=red>Co-Author: '.$cuname.' at '.$cudom.
-               '</font></h3>');
-      }
-
-
- if (&Apache::loncommon::fileembstyle(($efn=~/\.(\w+)$/)) eq
-      'ssi') {
-  $r->print('<p><font color="red">');
-  if ($ENV{'form.versionone'} eq 'priv') {
-      my $fn='/home/'.$cuname.'/public_html/'.$efn;
-      @f1=&get_split_file($fn,'local');
-      $r->print('<b>'.&mt('Construction Space Version').'</b>');
-  } else {
-      my $fn=
-       '/home/httpd/html//res/'.$cudom.'/'.$cuname.'/';
-      if ($ENV{'form.versionone'}) {
-         my ($main,$suffix)=($efn=~/^(.+)\.(\w+)$/);
-         $fn.=$main.'.'.$ENV{'form.versionone'}.'.'.$suffix;
-	 $r->print('<b>'.&mt('Version').' '.$ENV{'form.versionone'}.'</b>');
-      } else {
-         $fn.=$efn;
-	 $r->print('<b>'.&mt('Current Version').'</b>');
-      }
-      @f1=&get_split_file($fn,'remote');
-  }
-
-  $r->print('</font><br />'.&mt('versus').'<br /><font color="green">');
-
-  if ($ENV{'form.versiontwo'} eq 'priv') {
-      my $fn='/home/'.$cuname.'/public_html/'.$efn;
-      @f2=&get_split_file($fn,'local');
-      $r->print('<b>'.&mt('Construction Space Version').'</b>');
-  } else {
-      my $fn=
-       '/home/httpd/html/res/'.$cudom.'/'.$cuname.'/';
-      if ($ENV{'form.versiontwo'}) {
-         my ($main,$suffix)=($efn=~/^(.+)\.(\w+)$/);
-         $fn.=$main.'.'.$ENV{'form.versiontwo'}.'.'.$suffix;
-	 $r->print('<b>'.&mt('Version').' '.$ENV{'form.versiontwo'}.'</b>');
-      } else {
-         $fn.=$efn;
-	 $r->print('<b>'.&mt('Current Version').'</b>');
-      }
-      @f2=&get_split_file($fn,'remote');
-  }
-  $r->print('</font></p>');
-# Run diff
-
-  my $diffs = diff(\@f1, \@f2);
-
-# Start page output
+    if (($cuname ne $env{'user.name'}) || ($cudom ne $env{'user.domain'})) {
+        my $nameshown = &Apache::loncommon::plainname($cuname,$cudom).
+                        ' ('.$cuname.':'.$cudom.')';
+        $r->print('<p><span class="LC_info">');
+        if ($env{'request.role'} =~ /^ca\./) {
+            $r->print(&mt('Co-Author in Construction Space for: [_1]',$nameshown));
+        } elsif ($env{'request.role'} =~ /^aa\./) {
+            $r->print(&mt('Assistant Author in Construction Space for: [_1]',$nameshown));
+        } else {
+            $r->print(&mt('Resource Author is: [_1]',$nameshown));
+        }
+        $r->print('</span></p>');
+    }
 
-  my $chunk;
-  my $line;
+    if (&Apache::loncommon::fileembstyle(($efn=~/\.(\w+)$/)) eq 'ssi'
+	|| $efn =~ /\.meta$/) {
+	$r->print('<p><span class="LC_diff_removed">');
+	if ($env{'form.versionone'} eq 'priv') {
+	    my $fn=$r->dir_config('lonDocRoot')."/priv/$cudom/$cuname".$efn;
+	    @f1=&get_split_file($fn,'local');
+	    $r->print('<b>'.&mt('Construction Space Version').'</b>');
+	} else {
+	    my $fn=$r->dir_config('lonDocRoot')."/res/$cudom/$cuname";
+	    if ($env{'form.versionone'}) {
+		my ($main,$suffix,$is_meta)=
+		    &Apache::lonretrieve::get_file_info($efn);
+		
+		$fn.=($efn =~m|(.*/)[^/]+|)[0];
+		# add on to $fn the path information in $efn
+		$fn.=$main.'.'.$env{'form.versionone'}.'.'.$suffix;
+		$r->print('<b>'.&mt('Version').' '.$env{'form.versionone'}.'</b>');
+	    } else {
+		$fn.=$efn;
+		$r->print('<b>'.&mt('Current Version').'</b>');
+	    }
+	    @f1=&get_split_file($fn,'remote');
+	}
+	
+	$r->print('</span><br />'.&mt('versus').'<br /><span class="LC_diff_added">');
 
-  $r->print('<pre>');
+	if ($env{'form.filetwo'}) {
+	    my $efn2=$env{'form.filetwo'};
+	    $efn2=~s{^/priv/$LONCAPA::domain_re/$LONCAPA::username_re}{};
+	    my $fn=$r->dir_config('lonDocRoot')."/priv/$cudom/$cuname".$efn2;
+	    @f2=&get_split_file($fn,'local');
+	    $r->print('<tt>'.$efn2.'</tt>');
+	} elsif ($env{'form.versiontwo'} eq 'priv') {
+	    my $fn=$r->dir_config('lonDocRoot')."/priv/$cudom/$cuname".$efn;
+	    @f2=&get_split_file($fn,'local');
+	    $r->print('<b>'.&mt('Construction Space Version').'</b>');
+	} else {
+	    my $fn=$r->dir_config('lonDocRoot')."/res/$cudom/$cuname/";
+	    if ($env{'form.versiontwo'}) {
+		my ($main,$suffix,$is_meta)=
+		    &Apache::lonretrieve::get_file_info($efn);
+		# add on to $fn the path information in $efn
+		$fn.=($efn =~m|(.*/)[^/]+|)[0];
+		$fn.=$main.'.'.$env{'form.versiontwo'}.'.'.$suffix;
+		$r->print('<b>'.&mt('Version').' '.$env{'form.versiontwo'}.'</b>');
+	    } else {
+		$fn.=$efn;
+		$r->print('<b>'.&mt('Current Version').'</b>');
+	    }
+	    @f2=&get_split_file($fn,'remote');
+	}
+	$r->print('</span></p>');
+# Run diff
 
-  foreach $chunk (@$diffs) {
+	my $diffs = diff(\@f1, \@f2);
 
-    foreach $line (@$chunk) {
-      my ($sign, $lineno, $text) = @$line;
-      $text=~s/\</\&lt\;/g;
-      $text=~s/\>/\&gt\;/g;
-      $lineno=substr($lineno.'        ',0,7);
-      $r->print('<font color='.(($sign eq '+')?'green':'red').'>'.
-                $sign.' '.$lineno.' '.$text."</font>\n");
+        if (@$diffs) {
+            # Start page output
+            my $chunk;
+            my $line;
+            $r->print('<pre>');
+            foreach $chunk (@$diffs) {
+                foreach $line (@$chunk) {
+                    my ($sign, $lineno, $text) = @$line;
+                    $text=&HTML::Entities::encode($text,'<>&"');
+                    $lineno=substr($lineno.'        ',0,7);
+                    $r->print('<span class="'.(($sign eq '+')?'LC_diff_added'
+                                                             :'LC_diff_removed').'">'.
+                              $sign.' '.$lineno.' '.$text."</span>\n");
+                }
+                $r->print("</pre><hr /><pre>\n");
+            }
+            $r->print('</pre>');
+        } else {
+            $r->print('<p class="LC_info">'.&mt('No differences found').'</p>');
+        }
+    } else {
+	$r->print('<h1><span class="LC_warning">'.&mt('Binary File').'</span></h1>');
     }
-    $r->print("<hr>\n");
-  }
-  $r->print('</pre>');
-
-} else {
-    $r->print('<h1><font color=red>'.&mt('Binary File').'</font></h1>');
-}
-  $r->print('<center><a href="javascript:window.close();">'.&mt('Close This Window').'</a></center></body></html>'); 
-  return OK;  
+    $r->print(&Apache::loncommon::end_page()); 
+    return OK;  
 }
 
 
@@ -212,5 +242,27 @@ sub handler {
 __END__
 
 
+=pod
+
+=head1 NAME
+
+Apache::londiff
+
+=head1 SYNOPSIS
+
+Handler to show difference between two files.
+
+This is part of the LearningOnline Network with CAPA project
+described at http://www.lon-capa.org.
+
+=head1 Subroutines
+
+=over
+
+=item get_split_file()
+
+=item are_different_files()
 
+=item handler()
 
+=cut